Skip to content

Remove global npm install from publish workflow - #790

Merged
Hugo-ter-Doest merged 2 commits into
masterfrom
copilot/update-npm-publish-workflow
Oct 22, 2026
Merged

Hugo-ter-Doest merged 2 commits into
masterfrom
copilot/update-npm-publish-workflow

Conversation

Copilot AI commented Oct 22, 2026 •

Copy link
Copy Markdown
Contributor

A publish run failed with MODULE_NOT_FOUND for promise-retry inside npm's own Arborist code, not in project code. This PR stops the workflow from replacing the runner's npm, so it uses the npm bundled with actions/setup-node.

  • .github/workflows/npm-publish.yml
    • Removes the publish-npm step run: npm install --global npm@12.2.1.
    • No other steps change.
       - uses: actions/setup-node@v4
         with:
           node-version: 22.22.2
           registry-url: https://registry.npmjs.org/
-      - run: npm install --global npm@12.2.1
       - run: npm ci

Review notes

  • publish-npm still pins Node 22.22.2, the version the failed run used. If the promise-retry error comes back, the next step is probably pinning Node 20, as the build job already does.
  • The publish step is npm stage publish. Please confirm the npm bundled with Node 22.22.2 supports it, since the removed step was the only thing installing a different npm.

Co-authored-by: Hugo-ter-Doest <7829083+Hugo-ter-Doest@users.noreply.github.com>
Copilot AI changed the title [WIP] Update GitHub Actions workflow to remove npm global install step Remove global npm install from publish workflow Oct 22, 2026
Copilot AI requested a review from Hugo-ter-Doest October 22, 2026 11:47
@Hugo-ter-Doest
Hugo-ter-Doest marked this pull request as ready for review October 22, 2026 11:48
@Hugo-ter-Doest
Hugo-ter-Doest merged commit 991bc2c into master Oct 22, 2026
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