Migrate to eslint (#82)

* chore: install eslint with rubensworks config, add eslint.config.js and tsconfig.eslint.json

Agent-Logs-Url: https://github.com/rdfjs/rdfxml-streaming-parser.js/sessions/46b14265-34c1-4bd0-b41d-2a2e1a911a1d

Co-authored-by: rubensworks <440384+rubensworks@users.noreply.github.com>

* fix: resolve all ESLint linting errors

- lib/ParseError.ts: Add public constructor modifier, fix type assertion syntax
- lib/RdfXmlParser.ts:
  - Fix NCNAME_MATCHER regex (misleading char class, long line) using RegExp constructor
  - Add public/return type annotations throughout (constructor, _transform, validateNcname, attachSaxListeners, onTag, onTagResource, onTagProperty, emitTriple, claimNodeId, onText, onCloseTag, onDoctype, setDirection, setVersion)
  - Fix unsafe type assertions using angle-bracket syntax
  - Fix SaxesParser field type to SaxesParser<{ xmlns: true }> for type safety
  - Fix no-fallthrough in switch statement with 'Falls through' comment
  - Fix capitalized-comments violations
  - Refactor createLiteral to avoid indentation/ternary issues
  - Format all emitTriple/push calls with consistent multi-line argument style
  - Fix onDoctype regex: add u flag, use replaceAll, type callback params
  - Fix validateUri defaulting to true when not specified
  - Add es2021.string to tsconfig lib arrays for replaceAll support
- test/RdfXmlParser-test.ts:
  - Replace all return expect(...) with await expect(...)
  - Make non-async callbacks async
  - Fix duplicate test title
  - Fix toThrow() missing message
  - Add eslint-disable-next-line for unavoidably long test title

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Co-authored-by: rubensworks <440384+rubensworks@users.noreply.github.com>

* chore: fix all eslint errors after tslint migration

Agent-Logs-Url: https://github.com/rdfjs/rdfxml-streaming-parser.js/sessions/46b14265-34c1-4bd0-b41d-2a2e1a911a1d

Co-authored-by: rubensworks <440384+rubensworks@users.noreply.github.com>

* Ignore .eslintcache

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: rubensworks <440384+rubensworks@users.noreply.github.com>
Co-authored-by: Ruben Taelman <ruben.taelman@ugent.be>

Author: Copilot

.gitignore

@@ -21,3 +21,4 @@ coverage
 documentation
 .rdf-test-suite-cache
 earl.ttl
+.eslintcache

eslint.config.js

@@ -0,0 +1,90 @@
+const config = require('@rubensworks/eslint-config');
+
+module.exports = config([
+  {
+    ignores: [
+      'node_modules',
+      'coverage',
+      '**/*.js',
+      '**/*.d.ts',
+      '**/*.js.map',
+      '**/*.yml',
+      '**/*.yaml',
+      '**/*.md',
+      'perf/**',
+    ],
+  },
+  {
+    files: [ '**/*.ts' ],
+    languageOptions: {
+      parserOptions: {
+        tsconfigRootDir: __dirname,
+        project: [ './tsconfig.eslint.json' ],
+      },
+    },
+  },
+  {
+    // Project-specific overrides for TypeScript files
+    files: [ '**/*.ts' ],
+    rules: {
+      // Project uses strictNullChecks: false, so this rule cannot be applied
+      'ts/prefer-nullish-coalescing': 'off',
+      // This is a Node.js library that legitimately imports Node.js built-in modules
+      'import/no-nodejs-modules': 'off',
+      // Allow UPPER_CASE for class properties (public API constants) and enum members
+      'ts/naming-convention': [
+        'error',
+        {
+          selector: 'default',
+          format: [ 'camelCase' ],
+          leadingUnderscore: 'forbid',
+          trailingUnderscore: 'forbid',
+        },
+        {
+          selector: 'import',
+          format: null,
+        },
+        {
+          selector: 'variable',
+          format: [ 'camelCase', 'UPPER_CASE' ],
+          leadingUnderscore: 'forbid',
+          trailingUnderscore: 'forbid',
+        },
+        {
+          selector: 'classProperty',
+          format: [ 'camelCase', 'UPPER_CASE' ],
+          leadingUnderscore: 'forbid',
+          trailingUnderscore: 'forbid',
+        },
+        {
+          selector: 'enumMember',
+          format: [ 'camelCase', 'UPPER_CASE' ],
+        },
+        {
+          selector: 'typeLike',
+          format: [ 'PascalCase' ],
+        },
+        {
+          selector: [ 'typeParameter' ],
+          format: [ 'PascalCase' ],
+          prefix: [ 'T' ],
+        },
+        {
+          selector: 'interface',
+          format: [ 'PascalCase' ],
+          custom: {
+            regex: '^I[A-Z]',
+            match: true,
+          },
+        },
+        {
+          // Allow leading underscore for Node.js Transform stream override methods
+          selector: 'method',
+          format: [ 'camelCase' ],
+          leadingUnderscore: 'allow',
+          trailingUnderscore: 'forbid',
+        },
+      ],
+    },
+  },
+]);

lib/ParseError.ts

@@ -1,14 +1,12 @@
-import {SaxesParser} from "@rubensworks/saxes";
-import {RdfXmlParser} from "./RdfXmlParser";
+import type { SaxesParser } from '@rubensworks/saxes';
+import type { RdfXmlParser } from './RdfXmlParser';
 
 /**
  * An error that includes line and column in the error message.
  */
 export class ParseError extends Error {
-
-  constructor(parser: RdfXmlParser, message: string) {
-    const saxParser: SaxesParser = (<any>parser).saxParser;
+  public constructor(parser: RdfXmlParser, message: string) {
+    const saxParser: SaxesParser = (<{ saxParser: SaxesParser }><unknown>parser).saxParser;
     super(parser.trackPosition ? `Line ${saxParser.line} column ${saxParser.column + 1}: ${message}` : message);
   }
-
 }