Skip to content

ci: fix CLI e2e pnpm matrix failing after packageManager pin - #1156

Open
devin-ai-integration[bot] wants to merge 2 commits into
mainfrom
devin/1789065228-fix-cli-e2e-pnpm-setup
Open

ci: fix CLI e2e pnpm matrix failing after packageManager pin#1156
devin-ai-integration[bot] wants to merge 2 commits into
mainfrom
devin/1789065228-fix-cli-e2e-pnpm-setup

Conversation

@devin-ai-integration

@devin-ai-integration devin-ai-integration Bot commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

What

All 14 pnpm jobs in the CLI e2e matrix have failed since #1067 added "packageManager": "pnpm@10.33.0" to the root package.json. pnpm/action-setup@v6 refuses to run when both its version input and packageManager are set:

Error: Multiple versions of pnpm specified:
  - version 10 in the GitHub Action config with the key "version"
  - version pnpm@10.33.0 in the package.json with the key "packageManager"

Changes

In .github/workflows/cli-e2e.yml (matrix job only):

  • Replace the pnpm/action-setup step with npm install --global pnpm@<major>, mirroring how the npm variants are already installed. The matrix intentionally tests pnpm 9/10/11, so it can't honour the repo's packageManager pin.
  • Run the "Verify package manager version" step from runner.temp instead of the repo checkout. pnpm ≥10 self-switches to the packageManager version when invoked inside the repo, so pnpm --version there reported 10.33.0 even with pnpm 11 installed. All subsequent steps already run in runner.temp, so the pin never applies to the generated app.

Test Plan

  • Not applicable (explain why)
  • Verified locally — CI on this PR runs the full matrix; all 14 pnpm + 12 npm jobs green.

Checklist

Link to Devin session: https://app.devin.ai/sessions/e6ce4156b3a24609b00846430fc39ea0
Open in Devin Desktop: https://app.devin.ai/desktop/session/e6ce4156b3a24609b00846430fc39ea0?variant=devin

…nflict

Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

@vercel

vercel Bot commented Sep 10, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
openui-docs Ready Ready Preview Sep 10, 2026 6:39pm UTC

Request Review

…itch it

Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
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.

1 participant