Skip to content

[codex] Fix trusted publishing npm upgrade step#7

Merged
ejntaylor merged 1 commit into
mainfrom
codex-fix-trusted-publish-node
May 12, 2026
Merged

[codex] Fix trusted publishing npm upgrade step#7
ejntaylor merged 1 commit into
mainfrom
codex-fix-trusted-publish-node

Conversation

@ejntaylor
Copy link
Copy Markdown
Contributor

Summary

  • Removes the failing npm install -g npm@latest self-upgrade step from the publish workflow.
  • Runs the publish job on Node 24 instead, which ships a trusted-publishing-capable npm version.
  • Adds a lightweight tool-version log step so release runs show the exact Node/npm versions used.

Why

The release workflow failed while trying to replace npm globally on the GitHub runner:

npm error Cannot find module 'promise-retry'
npm error Require stack: ... @npmcli/arborist ...

Avoiding npm self-mutation in the release job is more reliable than upgrading npm in place.

Validation

  • npm test
  • npm run typecheck

@ejntaylor ejntaylor merged commit 0f5ce74 into main May 12, 2026
4 checks passed
@ejntaylor ejntaylor deleted the codex-fix-trusted-publish-node branch May 12, 2026 15:13
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