Skip to content

chore: write the bin path the way npm publishes it - #79

Merged
kkdev92 merged 2 commits into
mainfrom
chore/normalize-bin-path
Sep 28, 2026
Merged

kkdev92 merged 2 commits into
mainfrom
chore/normalize-bin-path

Conversation

@kkdev92

@kkdev92 kkdev92 commented Sep 28, 2026

Copy link
Copy Markdown
Owner

Writes the bin path the way npm publishes it, so the release log stops warning. The published package does not change.

Why

Every release logs:

npm warn publish npm auto-corrected some errors in your package.json when publishing.  Please run "npm pkg fix" to address these errors.
npm warn publish errors corrected:
npm warn publish "bin[vscode-ext-kit]" script name bin/vscode-ext-kit.mjs was invalid and removed

package.json wrote the entry as ./bin/vscode-ext-kit.mjs, and npm normalises it before publishing. The registry already has bin/vscode-ext-kit.mjs (npm view @kkdev92/vscode-ext-kit bin), so the published command was always right.

What changed

bin["vscode-ext-kit"]: ./bin/vscode-ext-kit.mjs → bin/vscode-ext-kit.mjs. This is exactly what npm pkg fix changes, and the only thing it changes (run on a copy of package.json).

Verification

  • Nothing in scripts/, tests/ or src/ refers to the path string.
  • CI's package job packs the tarball, installs it into a throwaway consumer and runs the checks in verify-package.mjs against it.

🤖 Generated with Claude Code

kkdev92 and others added 2 commits September 28, 2026 14:18
Every release logged "npm auto-corrected some errors in your package.json
when publishing": the bin entry was written ./bin/vscode-ext-kit.mjs, and
npm normalises it to bin/vscode-ext-kit.mjs before publishing. The published
bin was always right; this writes it in that form so the warning stops and
package.json matches the registry. It is what `npm pkg fix` changes, and
the only thing it changes.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@kkdev92
kkdev92 merged commit 1bf2da4 into main Sep 28, 2026
10 checks passed
@kkdev92
kkdev92 deleted the chore/normalize-bin-path branch September 28, 2026 05:25
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.

1 participant