Skip to content

fix(ci): remove npm upgrade hack from provenance workflow#1162

Closed
John-David Dalton (jdalton) wants to merge 1 commit intomainfrom
fix/remove-npm-upgrade-hack
Closed

fix(ci): remove npm upgrade hack from provenance workflow#1162
John-David Dalton (jdalton) wants to merge 1 commit intomainfrom
fix/remove-npm-upgrade-hack

Conversation

@jdalton
Copy link
Copy Markdown
Contributor

@jdalton John-David Dalton (jdalton) commented Apr 4, 2026

Node 25.8.2 (.node-version) ships npm 11.11+. The curl+tar npm upgrade step is unnecessary.


Note

Low Risk
Low risk: this only removes a CI workaround step and relies on the Node-provided npm during the publish workflow. Main risk is publish failures if the runner’s bundled npm lacks required trusted publishing/provenance behavior.

Overview
Removes the custom curl | tar npm self-upgrade step from the provenance.yml publish workflow, so publishing now uses the npm version bundled with the Node version from .node-version.

This simplifies the build/publish jobs and avoids mutating the global npm installation mid-workflow.

Reviewed by Cursor Bugbot for commit 9cd7b27. Configure here.

@jdalton
Copy link
Copy Markdown
Contributor Author

Folded into #1161 (SHA bump + npm hack removal combined)

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