Skip to content

fix: rename bin/ghouls to bin/ghouls.js to fix npm publish#39

Merged
ericanderson merged 1 commit intomainfrom
fix-publish-build
Mar 24, 2026
Merged

fix: rename bin/ghouls to bin/ghouls.js to fix npm publish#39
ericanderson merged 1 commit intomainfrom
fix-publish-build

Conversation

@ericanderson-bot
Copy link
Copy Markdown
Collaborator

Summary

  • npm 11+ treats bin entries without .js extension as invalid and silently removes them during npm publish, resulting in a published package with no ghouls CLI command
  • Renamed bin/ghoulsbin/ghouls.js and updated the bin field in package.json to use the correct path without ./ prefix

Test plan

  • npm publish --dry-run no longer shows "bin[ghouls] script name was invalid and removed" warning
  • All 269 tests pass
  • pnpm compile succeeds

npm 11+ treats bin entries without .js extension as invalid and silently
removes them during publish, resulting in a package with no CLI command.
@ericanderson ericanderson merged commit 87d1aaf into main Mar 24, 2026
3 checks passed
ericanderson pushed a commit that referenced this pull request Mar 24, 2026
npm 11+ treats bin entries without .js extension as invalid and silently
removes them during publish, resulting in a package with no CLI command.

Co-authored-by: ericanderson-bot <179007828+ericanderson-bot@users.noreply.github.com>
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.

3 participants