comunica / traqula / 27698923081

100%

main: 100%

Pull #149

github

refactor(@traqula/test-utils): use dist/cjs/package.json double-packaging instead of .cts extension

The previous fix used lib/index.cts -> dist/cjs/lib/index.cjs, relying on the .cjs
file extension to force CJS interpretation.  This is inconsistent with the rest of
the monorepo, where dist/cjs/package.json { "type": "commonjs" } already makes every
.js file under dist/cjs/ a CommonJS module — no special extension needed.

Changes:
- lib/index.cts -> lib/indexCjs.ts: rename to a plain .ts source file; TypeScript
  compiles it to dist/cjs/lib/indexCjs.js, which is treated as CJS by the existing
  dist/cjs/package.json { "type": "commonjs" } mechanism, exactly like all other
  packages in the monorepo.
- tsconfig.json: exclude lib/indexCjs.ts from the ESM build (it uses __dirname which
  is meaningless in ESM; the ESM entry lib/index.ts uses import.meta.url instead).
- tsconfig.cjs.json: remove the explicit lib/**/*.cts include (no longer needed);
  lib/indexCjs.ts is picked up automatically via lib/**/*.ts.
- package.json: point "require" to ./dist/cjs/lib/indexCjs.js (plain .js, not .cjs);
  remove the dist/*/lib/**/*.cjs and dist/*/lib/**/*.cjs.map globs from "files" since
  no .cjs files are produced anymore.
- test/cjs.test.ts: update path reference from index.cjs to indexCjs.js.
- vitest.config.ts: exclude lib/indexCjs.ts from v8 coverage — it cannot be imported
  in the ESM Vitest context; its behaviour is validated indirectly by cjs.test.ts
  loading dist/cjs/lib/generators/utils.js and calling _initStaticsRoot with the
  indexCjs.js directory.

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

759 of 759 branches covered (100.0%)

Branch coverage included in aggregate %.

20 of 20 new or added lines in 3 files covered. (100.0%)

5474 of 5474 relevant lines covered (100.0%)

282861.01 hits per line