Skip to content

Fix CI build (26.x) by removing conflicting pnpm setup#137

Draft
Copilot wants to merge 2 commits into
mainfrom
copilot/fix-failing-github-actions-job
Draft

Fix CI build (26.x) by removing conflicting pnpm setup#137
Copilot wants to merge 2 commits into
mainfrom
copilot/fix-failing-github-actions-job

Conversation

Copilot AI commented Jun 9, 2026

Copy link
Copy Markdown

The build (26.x) GitHub Actions job fails before dependency install because pnpm is configured from two sources with different versions. pnpm/action-setup now hard-fails when the workflow version input and package.json#packageManager disagree.

  • Root cause

    • Workflow defined pnpm twice:
      • explicit pnpm/action-setup in .github/workflows/action.yml (version: 11)
      • composite module-cache flow deriving version from package.json (pnpm@11.5.2)
  • Change

    • Removed the explicit Setup pnpm step from the build job in .github/workflows/action.yml.
    • Kept pnpm initialization centralized in ./.github/module-cache/action.yaml (single source of truth from packageManager).
  • Resulting workflow shape

    • Build job now performs checkout, then calls ./.github/build-artifacts, which internally sets up pnpm via module-cache.
# .github/workflows/action.yml (build job)
steps:
  - uses: actions/checkout@v6.0.3
  - name: Setup Node, install dependencies, and cache node modules, build dist artifacts
    uses: ./.github/build-artifacts
    with:
      node-version: ${{ matrix.node }}

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jun 9, 2026

Copy link
Copy Markdown

Deploying samples with  Cloudflare Pages  Cloudflare Pages

Latest commit: 68aba6b
Status:🚫  Build failed.

View logs

Copilot AI changed the title [WIP] Fix failing GitHub Actions job "build (26.x)" Fix CI build (26.x) by removing conflicting pnpm setup Jun 9, 2026
Copilot AI requested a review from SanderElias June 9, 2026 08:55
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