rdfjs / rdfxml-streaming-parser.js / 24336514144

100%

push

github

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@user... (continued)

376 of 376 branches covered (100.0%)

Branch coverage included in aggregate %.

155 of 155 new or added lines in 2 files covered. (100.0%)

373 of 373 relevant lines covered (100.0%)

751.98 hits per line