Rebuild: update dist source maps & metadata#1848
Conversation
coliff
commented
Apr 5, 2026
- Regenerated build artifacts and updated package metadata.
- Inline sourceMappingURL entries were updated in several dist/cli files after recompiling TypeScript sources; src/cli files, tsconfig.json and package.json were also modified.
- Minor project configuration updates.
Regenerated build artifacts and updated package metadata. Inline sourceMappingURL entries were updated in several dist/cli files after recompiling TypeScript sources; src/cli files, tsconfig.json and package.json were also modified. This commit reflects a rebuild and minor project configuration updates.
There was a problem hiding this comment.
Code Review
This pull request updates the project's TypeScript configuration to use Node16 module resolution and upgrades several dependencies, including @typescript-eslint and astro. The changes involve refactoring imports across the CLI and core modules to ensure compatibility with the new module system. Review feedback correctly identifies that the Node.js version (22.22.2) specified in the website's package.json and volta configuration is non-existent and should be corrected to a valid version like 22.12.0 to prevent installation failures.
There was a problem hiding this comment.
Pull request overview
This PR refreshes generated build artifacts (including inline source maps) after recompiling the TypeScript sources, while also updating a few dependency/tooling versions and Node engine constraints (notably for the website package).
Changes:
- Bumped website dependencies (Astro) and the website’s pinned/required Node version.
- Switched the repo TypeScript compilation settings to
module/moduleResolution: Node16and adjusted a few CommonJS interop imports accordingly. - Regenerated
dist/outputs and updated lockfiles to reflect the rebuild and dependency updates.
Reviewed changes
Copilot reviewed 6 out of 15 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| website/package.json | Updates Astro version and bumps website Node engine/Volta pin. |
| website/package-lock.json | Lockfile refresh for the website dependency changes (Astro and transitive deps). |
| tsconfig.json | Switches TS module and resolution strategy to Node16. |
| src/cli/parse-glob.ts | Adjusts is-glob import style for CommonJS interop under the new TS module settings. |
| src/cli/htmlhint.ts | Adjusts strip-json-comments import style for CommonJS interop under the new TS module settings. |
| src/cli/formatters/junit.ts | Adjusts xml import style for CommonJS interop under the new TS module settings. |
| src/cli/formatters/checkstyle.ts | Adjusts xml import style for CommonJS interop under the new TS module settings. |
| package.json | Bumps @typescript-eslint/* devDependencies. |
| package-lock.json | Lockfile refresh for the root devDependency bumps and transitive reshaping. |
| dist/core/rules/index.js | Rebuilt output reflecting updated TS emit (interop helpers) and refreshed inline source map. |
| dist/core/core.js | Rebuilt output reflecting updated TS emit (interop helpers) and refreshed inline source map. |
| dist/cli/parse-glob.js | Rebuilt output with refreshed inline source map metadata. |
| dist/cli/htmlhint.js | Rebuilt output with refreshed inline source map metadata. |
| dist/cli/formatters/junit.js | Rebuilt output with refreshed inline source map metadata. |
| dist/cli/formatters/checkstyle.js | Rebuilt output with refreshed inline source map metadata. |
Files not reviewed (1)
- website/package-lock.json: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.