Skip to content

Rebuild: update dist source maps & metadata#1848

Merged
coliff merged 1 commit intomainfrom
dev/coliff/tsconfig-fix
Apr 5, 2026
Merged

Rebuild: update dist source maps & metadata#1848
coliff merged 1 commit intomainfrom
dev/coliff/tsconfig-fix

Conversation

@coliff
Copy link
Copy Markdown
Member

@coliff 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.
Copilot AI review requested due to automatic review settings April 5, 2026 23:47
@coliff coliff added the dependencies Pull requests that update a dependency file label Apr 5, 2026
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

@coliff coliff merged commit 27f1ff1 into main Apr 5, 2026
26 checks passed
@coliff coliff deleted the dev/coliff/tsconfig-fix branch April 5, 2026 23:50
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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: Node16 and 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants