comunica / traqula / 27758988697

100%

push

github

Fix: double packaging test-utils through side-effect endpoint  (#149)

TLDR: our previous approach did not work on CJS. Now, we have two index files that, as a side effect register the directory of our statics files regardless of working directory. This approach uses either `__dirName` or `import.meta.url` depending on cjs or ESM runtime.

---


* fix(test-utils): resolve statics relative to installed package location

Replace `path.resolve('packages/test-utils/statics')` (which relied on
CWD being the monorepo root) with `import.meta.url`-based resolution
that walks up the directory tree to find the `statics/` folder.

Changes:
- `lib/generators/utils.ts`: use `import.meta.url` + walk-upward to
  locate `statics/` regardless of CWD or call site depth. A
  `/* v8 ignore */` guard covers the unreachable filesystem-root throw.
- `tsconfig.cjs.json`: exclude `lib/generators/utils.ts` from the CJS
  TypeScript build (file is still compiled via imports, but the output
  is replaced by the post-build script).
- `scripts/write-cjs-utils.cjs` (new): post-build script that writes
  a `__dirname`-based CJS equivalent into `dist/cjs/lib/generators/`.
- `package.json`: wire the post-build script into `build:cjs`, bump
  patch version to 1.1.4.
- `eslint.config.js`: ignore `task.md` and the build scripts directory
  from linting.

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

* cleanup

* fix(@traqula/test-utils): replace string-template CJS shim with proper dual-entry compilation

Root cause: dist/cjs/lib/generators/utils.js contained import.meta.url, causing
SyntaxError: Cannot use 'import.meta' outside a module when loaded with require().
The previous workaround (write-cjs-utils.cjs) patched the dist file after the fact
via a string template, bypassing TypeScript type-checking and linting, and was never
wired into the build scripts.

Changes:
- lib/generators/utils.ts: Remove import.meta.url entirely. Add _initStaticsRoot(dir)
  for lazy... (continued)

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