Skip to content

fix: replace pnpm/action-setup@v6 with corepack in all CI workflows - #1219

Merged
hotlong merged 1 commit into
mainfrom
claude/fix-ci-pnpm
Apr 13, 2026
Merged

fix: replace pnpm/action-setup@v6 with corepack in all CI workflows#1219
hotlong merged 1 commit into
mainfrom
claude/fix-ci-pnpm

Conversation

@Claude

@ClaudeClaudeAI commented Apr 13, 2026

Copy link
Copy Markdown
Contributor

CI workflows fail with ERR_PNPM_BROKEN_LOCKFILE due to pnpm/action-setup@v6's npm-based self-installer misparsingthe lockfile. Applied the same fix used in framework#1122.

Changes

Replaced pnpm/action-setup@v6corepack enable across all 9 workflows:

# Before
- name: Setup pnpmuses: pnpm/action-setup@v6with:
version: 10.31.0# After
- name: Enable Corepackrun: corepack enable
- name: Verify pnpm versionrun: pnpm --version

Updated workflows:

  • ci.yml (4 jobs: test, build, e2e, docs)
  • lint.yml
  • release.yml
  • changeset-release.yml
  • performance-budget.yml
  • shadcn-check.yml
  • storybook-deploy.yml
  • storybook-tests.yml
  • dependabot-auto-merge.yml

Corepack reads the packageManager field from package.json (pnpm@10.31.0) with SHA-512 verification, ensuring identical binaries between local dev and CI environments.

All CI workflows updated to use corepack instead of pnpm/action-setup@v6:
- ci.yml (4 jobs)
- lint.yml
- release.yml
- changeset-release.yml
- performance-budget.yml
- shadcn-check.yml
- storybook-deploy.yml
- storybook-tests.yml
- dependabot-auto-merge.yml
Corepack reads the exact packageManager field from package.json with SHA verification, ensuring consistent pnpm version across local dev and CI.
Agent-Logs-Url: https://github.com/objectstack-ai/objectui/sessions/e2dacbe8-6e60-47e1-9acd-5416a058077e
Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>
@vercel

vercelBot commented Apr 13, 2026

Copy link
Copy Markdown

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

ProjectDeploymentActionsUpdated (UTC)
objectui-demoReadyReadyPreview, CommentApr 13, 2026 1:11pm
1 Skipped Deployment
ProjectDeploymentActionsUpdated (UTC)
objectuiIgnoredIgnoredApr 13, 2026 1:11pm

Request Review

@hotlong
hotlong marked this pull request as ready for review April 13, 2026 13:11
CopilotAI review requested due to automatic review settings April 13, 2026 13:11
@hotlong
hotlong merged commit 2bb5b98 into mainApr 13, 2026
8 checks passed

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates GitHub Actions CI workflows to stop using pnpm/action-setup@v6 (which is causing lockfile parse failures) and instead rely on Node’s Corepack to provision the repo-pinned pnpm version via packageManager in package.json.

Changes:

  • Replaced pnpm/action-setup@v6 with corepack enable across the affected workflows.
  • Added a pnpm --version verification step after enabling Corepack.

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 12 comments.

Show a summary per file
FileDescription
.github/workflows/ci.ymlSwitches pnpm setup in all CI jobs from pnpm/action-setup to Corepack + version verification.
.github/workflows/lint.ymlReplaces pnpm setup with Corepack + version verification.
.github/workflows/release.ymlReplaces pnpm setup with Corepack + version verification in release pipeline.
.github/workflows/changeset-release.ymlReplaces pnpm setup with Corepack + version verification for changesets release flow.
.github/workflows/performance-budget.ymlReplaces pnpm setup with Corepack + version verification for bundle analysis.
.github/workflows/shadcn-check.ymlReplaces pnpm setup with Corepack + version verification for shadcn sync checks.
.github/workflows/storybook-deploy.ymlReplaces pnpm setup with Corepack + version verification for Storybook build/deploy.
.github/workflows/storybook-tests.ymlReplaces pnpm setup with Corepack + version verification for Storybook CI tests.
.github/workflows/dependabot-auto-merge.ymlReplaces pnpm setup with Corepack + version verification for Dependabot automerge flow.

Comment on lines +24 to +28
- name: Enable Corepack
run: corepack enable

- name: Verify pnpm version
run: pnpm --version
Comment on lines +66 to +70
- name: Enable Corepack
run: corepack enable

- name: Verify pnpm version
run: pnpm --version
Comment on lines +122 to +126
- name: Enable Corepack
run: corepack enable

- name: Verify pnpm version
run: pnpm --version
Comment on lines +210 to +216
- name: Enable Corepack
if: steps.docs-changes.outputs.should_run == 'true'
uses: pnpm/action-setup@v6
with:
version: 10.31.0
run: corepack enable

- name: Verify pnpm version
if: steps.docs-changes.outputs.should_run == 'true'
run: pnpm --version
Comment on lines +24 to +28
- name: Enable Corepack
run: corepack enable

- name: Verify pnpm version
run: pnpm --version
Comment on lines +22 to +26
- name: Enable Corepack
run: corepack enable

- name: Verify pnpm version
run: pnpm --version
Comment on lines +35 to +39
- name: Enable Corepack
run: corepack enable

- name: Verify pnpm version
run: pnpm --version
Comment on lines +31 to +35
- name: Enable Corepack
run: corepack enable

- name: Verify pnpm version
run: pnpm --version
Comment on lines +34 to +38
- name: Enable Corepack
run: corepack enable

- name: Verify pnpm version
run: pnpm --version
Comment on lines +21 to +25
- name: Enable Corepack
run: corepack enable

- name: Verify pnpm version
run: pnpm --version
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@Claude@hotlong