Skip to content

Updates dependencies and migrates project to ESM#56

Merged
milanholemans merged 1 commit into
pnp:mainfrom
milanholemans:dep-update
Apr 23, 2026
Merged

Updates dependencies and migrates project to ESM#56
milanholemans merged 1 commit into
pnp:mainfrom
milanholemans:dep-update

Conversation

@milanholemans

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings April 23, 2026 22:35

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Migrates the project to ESM and updates the build/lint/tooling stack accordingly.

Changes:

  • Switch TypeScript compilation to modern ESM (NodeNext) and update internal imports to use .js extensions.
  • Replace ncc packaging with an esbuild-based bundling step to dist/index.js.
  • Refresh dependencies/dev tooling and introduce a flat ESLint config.

Reviewed changes

Copilot reviewed 9 out of 15 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
tsconfig.json Updates TS target/libs and switches module + resolution to NodeNext for ESM builds.
src/validate.ts Updates ESM import path; reformats validation logic.
src/validate.spec.ts Updates ESM import path; reformats tests.
src/utils.ts Updates ESM import path; reformats helper utilities.
src/utils.spec.ts Updates ESM import paths; reformats tests.
src/main.ts Updates ESM import paths; reformats main action entrypoint.
src/constants.ts Reformats exported constants object.
src/commands.ts Updates ESM import paths; reformats command builders.
src/commands.spec.ts Updates ESM import paths; reformats tests.
package.json Marks package as ESM, replaces packaging script with esbuild, adds lint scripts, bumps deps/devDeps.
package-lock.json Lockfile updated to reflect new dependency graph/tooling.
eslint.config.mjs Adds ESLint flat config for TS + stylistic rules.
dist/sourcemap-register.js Removes previously committed sourcemap registration artifact.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread package.json
Comment thread eslint.config.mjs
Comment thread src/validate.ts
Comment thread src/main.ts
Comment thread src/commands.ts
Co-authored-by: Copilot <copilot@github.com>
@milanholemans milanholemans merged commit 1bc8877 into pnp:main Apr 23, 2026
6 checks passed
@milanholemans milanholemans deleted the dep-update branch April 23, 2026 22:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants