Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Feb 3, 2026

Node 18.20.8 ships with npm 10.8.2, which doesn't support the workspace:* protocol used in package dependencies. This causes npm ci to fail with EUNSUPPORTEDPROTOCOL.

Changes

  • Added npm upgrade step in CI workflow, conditional on Node 18.x matrix
  • Upgrades to npm 10.9.0+ before dependency installation to support workspace protocol
- name: Upgrade npm
  run: npm install -g npm@latest
  if: matrix.node-version == '18.x'

Node 20.x unaffected (already ships with compatible npm version).


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: kayodebristol <3579196+kayodebristol@users.noreply.github.com>
Copilot AI changed the title [WIP] Update actions/upload-artifact from 4 to 6 Fix Node 18.x CI failure: upgrade npm for workspace protocol support Feb 3, 2026
Copilot AI requested a review from kayodebristol February 3, 2026 02:15
@kayodebristol
Copy link
Contributor

Draft PR noted by automated sweep. If this is ready, mark it Ready for review; otherwise please add the remaining checklist items/blockers. [@copilot] can help finalize when ready.

Copy link
Contributor Author

Copilot AI commented Feb 3, 2026

Draft PR noted by automated sweep. If this is ready, mark it Ready for review; otherwise please add the remaining checklist items/blockers. [@copilot] can help finalize when ready.

This PR is ready for review. The fix upgrades npm to latest on Node 18.x builds (commit 453a6c4), resolving the workspace protocol compatibility issue. Code review and security checks passed with no issues.

@kayodebristol
Copy link
Contributor

This PR is currently Draft.\n\nMinimal next steps:\n- Mark as Ready for review when safe\n- Ensure CI is green\n- Request review / address feedback\n\nOnce checks are green + required reviews are satisfied, we can squash merge.

@kayodebristol
Copy link
Contributor

This PR is currently Draft.

Minimal next steps:

  • Mark as Ready for review when safe
  • Ensure CI is green
  • Request review / address feedback

Once checks are green + required reviews are satisfied, we can squash merge.

@kayodebristol
Copy link
Contributor

Closing as superseded by #155 (single consolidated fix for the Node 18/20 CI + workspace:* issue). If we need to resurrect any approach from this PR, we can reopen.

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