Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Feb 3, 2026

The repository uses workspace:* protocol in monorepo packages, which requires pnpm. CI workflows were using npm, causing EUNSUPPORTEDPROTOCOL failures.

Changes

  • Package manager configuration

    • Added packageManager: "pnpm@9.15.4" to package.json
    • Created pnpm-workspace.yaml for workspace definitions
    • Generated pnpm-lock.yaml lockfile
  • GitHub Actions workflows (ci.yml, publish.yml, release.yml, azure-functions.yml)

    • Enable Corepack before Node.js setup
    • Switch cache from npm to pnpm
    • Replace npm cipnpm install --frozen-lockfile
    • Replace npm runpnpm run
  • Dependency bump

    • actions/upload-artifact: 4 → 6

💡 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 3 commits February 3, 2026 02:17
Co-authored-by: kayodebristol <3579196+kayodebristol@users.noreply.github.com>
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 chore(deps): bump actions/upload-artifact from 4 to 6 + fix pnpm CI configuration Feb 3, 2026
Copilot AI requested a review from kayodebristol February 3, 2026 02:20
@kayodebristol
Copy link
Contributor

Heads-up: this is a stacked DRAFT PR targeting the dependabot branch (#147). Please keep stacked changes minimal and be prepared to close it once #147 is fixed/merged; avoid multiple parallel fix strategies.

@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.

@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