Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Feb 3, 2026

CI fails on Node 18.x with EUNSUPPORTEDPROTOCOL error because the repo uses workspace:* protocol in package dependencies—a pnpm-specific syntax unsupported by npm.

Changes

  • Add pnpm workspace config: Created pnpm-workspace.yaml defining workspace packages
  • Replace lockfile: Generated pnpm-lock.yaml, removed package-lock.json
  • Migrate all workflows to pnpm:
    • ci.yml - Setup pnpm before Node.js for proper caching
    • publish.yml - Use pnpm publish --no-git-checks
    • release.yml - Build and test with pnpm
    • azure-functions.yml - Install deps with --production flag

All workflows now use pnpm/action-setup@v4 preceding actions/setup-node@v6 to ensure pnpm is available before cache initialization.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 2 commits February 3, 2026 09:11
- Add pnpm-workspace.yaml for workspace configuration
- Generate pnpm-lock.yaml to replace package-lock.json
- Update ci.yml to use pnpm with proper setup order
- Update publish.yml to use pnpm for NPM publishing
- Update release.yml to use pnpm for builds and tests
- Update azure-functions.yml to use pnpm for deployments

This fixes the EUNSUPPORTEDPROTOCOL error that occurred because
the repo uses workspace:* protocol which is pnpm-specific.

Co-authored-by: kayodebristol <3579196+kayodebristol@users.noreply.github.com>
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 by migrating to pnpm Feb 3, 2026
Copilot AI requested a review from kayodebristol February 3, 2026 09:19
@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. All changes are complete:

✅ Fixed Node 18.x CI failure by migrating from npm to pnpm (commits c998304, 3916561)
✅ Code review completed - all feedback addressed
✅ Security scan passed (CodeQL - no vulnerabilities)
✅ Mergeable state: clean

The fix addresses the EUNSUPPORTEDPROTOCOL error on the base branch by switching all workflows to pnpm, which properly supports the workspace:* protocol used in this monorepo.

@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