Migrate linter from tslint to @rubensworks/eslint-config (#75)

* Changes before error encountered

Agent-Logs-Url: https://github.com/rubensworks/rdfa-streaming-parser.js/sessions/2dc0ad2e-9480-4194-94cd-520bf5ceeb37

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

* chore: migrate from tslint to eslint (partial fixes)

Agent-Logs-Url: https://github.com/rubensworks/rdfa-streaming-parser.js/sessions/3244748a-5ae5-4252-a0f9-7f7e0b1e696e

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

* Changes before error encountered

Agent-Logs-Url: https://github.com/rubensworks/rdfa-streaming-parser.js/sessions/5a05c36d-c144-42ef-bfe4-ae02d5954db7

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

* fix: resolve CI failures - TypeScript errors, ESLint errors, and test issues

Agent-Logs-Url: https://github.com/rubensworks/rdfa-streaming-parser.js/sessions/adc4fdf2-bbd3-4c49-8311-99ed24dc63b4

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

---------

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

Author: Copilot

.gitignore

@@ -18,6 +18,7 @@ yarn-error.log
 **/index.js.map
 **/index.d.ts
 coverage
+.eslintcache
 documentation
 .rdf-test-suite-cache
 

README.md

@@ -35,7 +35,6 @@ _or_
 const RdfaParser = require("rdfa-streaming-parser").RdfaParser;
 ```
 
-
 ## Usage
 
 `RdfaParser` is a Node [Transform stream](https://nodejs.org/api/stream.html#stream_class_stream_transform)

eslint.config.js

@@ -0,0 +1,91 @@
+const config = require('@rubensworks/eslint-config');
+
+module.exports = config([
+  {
+    // Exclude performance benchmark scripts from linting
+    ignores: [ 'perf/**' ],
+  },
+  {
+    files: [ '**/*.ts' ],
+    languageOptions: {
+      parserOptions: {
+        tsconfigRootDir: __dirname,
+        project: [ './tsconfig.eslint.json' ],
+      },
+    },
+  },
+  {
+    rules: {
+      // Allow importing Node.js built-in modules (used in tests and webpack config)
+      'import/no-nodejs-modules': 'off',
+
+      // Allow file extensions in import paths (required for JSON imports)
+      'import/extensions': 'off',
+    },
+  },
+  {
+    files: [ '**/*.ts' ],
+    rules: {
+      // This rule requires strictNullChecks, which is not enabled in this project
+      'ts/prefer-nullish-coalescing': 'off',
+
+      // Extended naming conventions for this project
+      '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: 'typeLike',
+          format: [ 'PascalCase' ],
+        },
+        {
+          selector: [ 'typeParameter' ],
+          format: [ 'PascalCase' ],
+          prefix: [ 'T' ],
+        },
+        {
+          selector: 'interface',
+          format: [ 'PascalCase' ],
+          custom: {
+            regex: '^I[A-Z]',
+            match: true,
+          },
+        },
+        {
+          // Allow UPPER_CASE for static class properties (e.g., namespace constants)
+          selector: 'classProperty',
+          modifiers: [ 'static' ],
+          format: [ 'camelCase', 'UPPER_CASE' ],
+          leadingUnderscore: 'forbid',
+          trailingUnderscore: 'forbid',
+        },
+        {
+          // Allow leading underscore for class methods (e.g., _transform, _flush from Node.js Transform API)
+          selector: 'classMethod',
+          format: [ 'camelCase' ],
+          leadingUnderscore: 'allow',
+          trailingUnderscore: 'forbid',
+        },
+        {
+          // Allow any format for object literal property names (e.g., MIME type keys like 'text/html')
+          selector: 'objectLiteralProperty',
+          format: null,
+        },
+      ],
+    },
+  },
+]);

lib/IActiveTag.ts

@@ -1,13 +1,13 @@
-import * as RDF from "@rdfjs/types";
-import {IRdfaPattern} from "./IRdfaPattern";
+import type * as RDF from '@rdfjs/types';
+import type { IRdfaPattern } from './IRdfaPattern';
 
 /**
  * Data holder for the RDFa state in XML tags.
  */
 export interface IActiveTag {
   name: string;
-  prefixesAll: {[prefix: string]: string};
-  prefixesCustom: {[prefix: string]: string};
+  prefixesAll: Record<string, string>;
+  prefixesCustom: Record<string, string>;
   subject?: RDF.NamedNode | RDF.BlankNode | boolean;
   explicitNewSubject?: boolean;
   predicates?: RDF.NamedNode[];
@@ -21,10 +21,10 @@ export interface IActiveTag {
   collectChildTagsForCurrentTag?: boolean;
   collectedPatternTag?: IRdfaPattern;
   interpretObjectAsTime?: boolean;
-  incompleteTriples?: { predicate: RDF.Quad_Predicate, reverse: boolean, list?: boolean }[];
+  incompleteTriples?: { predicate: RDF.Quad_Predicate; reverse: boolean; list?: boolean }[];
   inlist: boolean;
-  listMapping: {[predicate: string]: (RDF.Term|boolean)[]};
-  listMappingLocal: {[predicate: string]: (RDF.Term|boolean)[]};
+  listMapping: Record<string, (RDF.Term | boolean)[]>;
+  listMappingLocal: Record<string, (RDF.Term | boolean)[]>;
   skipElement: boolean;
   localBaseIRI?: RDF.NamedNode;
 }

lib/IHtmlParseListener.ts

@@ -7,23 +7,23 @@ export interface IHtmlParseListener {
    * @param {string} name The tag name.
    * @param {{[p: string]: string}} attributes A hash of attributes.
    */
-  onTagOpen(name: string, attributes: {[s: string]: string}): void;
+  onTagOpen: (name: string, attributes: Record<string, string>) => void;
 
   /**
    * Called when a tag is closed.
    */
-  onTagClose(): void;
+  onTagClose: () => void;
 
   /**
    * Called when text contents are parsed.
    * Note that this can be called multiple times per tag,
    * when for example the string is spread over multiple chunks.
    * @param {string} data A string.
    */
-  onText(data: string): void;
+  onText: (data: string) => void;
 
   /**
    * Called when parsing has ended.
    */
-  onEnd(): void;
+  onEnd: () => void;
 }

lib/IRdfaPattern.ts

@@ -1,13 +1,13 @@
-import * as RDF from "@rdfjs/types";
-import {IActiveTag} from "./IActiveTag";
+import type * as RDF from '@rdfjs/types';
+import type { IActiveTag } from './IActiveTag';
 
 /**
  * A datastructure for storing an rdfa:Pattern.
  */
 export interface IRdfaPattern {
   rootPattern: boolean;
   name: string;
-  attributes: {[s: string]: string};
+  attributes: Record<string, string>;
   text: string[];
   children: IRdfaPattern[];
   referenced: boolean;