Skip to content

fix: repair npm trusted publishing workflow - #11

Merged
TDanks2000 merged 1 commit into
mainfrom
agent/fix-release-trusted-publishing
Aug 11, 2026
Merged

fix: repair npm trusted publishing workflow#11
TDanks2000 merged 1 commit into
mainfrom
agent/fix-release-trusted-publishing

Conversation

@TDanks2000

Copy link
Copy Markdown
Collaborator

What

  • run releases on Node 24 instead of Node 20
  • install npm 11.18.0 so npm trusted publishing/OIDC is supported
  • fail early if the Node/npm trusted-publishing minimums are not met
  • align checkout/setup-node/setup-bun versions with the working Api-Wrappers/tmdb-wrapper release workflow
  • run the release job through the npm GitHub environment
  • restrict version PR creation and npm publishing to main
  • ignore .changeset/README.md when detecting pending changesets

Root cause

The failed 1.1.1 release ran Node 20.20.2 with npm 10.8.2. npm trusted publishing requires Node 22.14.0+ and npm 11.5.1+. Although the workflow had id-token: write, npm 10 could not use OIDC for registry authentication and the publish fell through to the setup-node placeholder token, producing npm's misleading scoped-package E404 response.

The package itself exists on npm and the build/test/package verification all passed before the publish step.

Validation

  • matched the trusted-publishing runtime already used by Api-Wrappers/tmdb-wrapper
  • retained id-token: write and the npm registry configuration
  • no package version or source changes; this is release infrastructure only

After this merges, the existing @api-wrappers/igdb-wrapper@1.1.1 version is still unpublished, so the next main release run should retry that version automatically.

@TDanks2000
TDanks2000 marked this pull request as ready for review August 11, 2026 05:01
@TDanks2000
TDanks2000 merged commit 9a50569 into main Aug 11, 2026
1 check passed
@TDanks2000
TDanks2000 deleted the agent/fix-release-trusted-publishing branch August 11, 2026 05:01
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