Skip to content

fix: drop ./ prefix from bin path so npm 11 keeps the executable - #17

Merged
nicosampler merged 1 commit into
mainfrom
fix/bin-path-npm11
Jul 29, 2026
Merged

nicosampler merged 1 commit into
mainfrom
fix/bin-path-npm11

Conversation

@nicosampler

Copy link
Copy Markdown
Collaborator

Summary

No related issue. The last publish attempt logged "bin[canton-barebones]" script name bin/canton-barebones.js was invalid and removed — npm 11 (now installed in release.yml for trusted publishing) strips bin entries whose path starts with ./ instead of just normalizing them like npm 10 did. Published like that, the package would ship without the canton-barebones command.

Changes

  • Remove the ./ prefix from the bin path in package.json
  • Add a patch changeset — this also exercises the full release flow end to end (changeset → Version Packages PR → publish via OIDC)

Acceptance criteria

  • The next published version installs the canton-barebones executable (npx @bootnodedev/cbn --help works)
  • No bin ... removed warning in the publish log

Test plan

Automated tests

No automated tests added; npm test runs as the publish gate in release.yml.

Manual verification

  • Merge this PR → the bot opens the Version Packages PR bumping to 0.1.3
  • Merge that PR → release.yml publishes via trusted publishing (OIDC)
  • Verify: npm view @bootnodedev/cbn version shows 0.1.3 and npx @bootnodedev/cbn@latest status resolves the bin

Breaking changes

None.

Checklist

  • Self-reviewed my own diff
  • Tests added or updated
  • Docs updated (if applicable)
  • No unrelated changes bundled in

Screenshots

None.

@gemini-code-assist

Copy link
Copy Markdown

Caution

The consumer version of Gemini Code Assist on GitHub has been sunset. All code review activity has officially ceased.

@nicosampler
nicosampler merged commit 333675b into main Jul 29, 2026
3 checks passed
@nicosampler
nicosampler deleted the fix/bin-path-npm11 branch July 29, 2026 18:33
@github-project-automation github-project-automation Bot moved this from Backlog to Done in Canton - dAppBooster (#390) Jul 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants