Skip to content

ci: single-source the pnpm version via packageManager - #17

Merged
bufo24 merged 1 commit into
mainfrom
fix/pnpm-version-single-source
Jul 7, 2026
Merged

ci: single-source the pnpm version via packageManager#17
bufo24 merged 1 commit into
mainfrom
fix/pnpm-version-single-source

Conversation

@bufo24

@bufo24 bufo24 commented Jul 7, 2026

Copy link
Copy Markdown
Member

Problem

The publish job failed on pnpm/action-setup@v4:

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

PR #12 bumped the workflow's version: to 10.19.0 for OIDC but left package.json's packageManager at pnpm@10.5.2. action-setup v4 treats the mismatch as a hard error.

Fix

One source of truth:

  • package.jsonpackageManager: pnpm@10.19.0 (must be an OIDC-capable version; can't stay at 10.5.2).
  • Removed the redundant version: pin from both release-please.yml and ci.yml, so action-setup derives the version from packageManager.

Verified locally: corepack switches to 10.19.0 and pnpm install --frozen-lockfile resolves cleanly against the existing lockfile (same pnpm 10.x lockfile format).

Follow-up

@ambosstech/core@0.1.1 was tagged by release-please but never published (this is the run that failed), so it's a skipped version on npm. Once this lands, the next release will publish through the corrected workflow.

pnpm/action-setup errored with ERR_PNPM_BAD_PM_VERSION because the
workflow pinned version 10.19.0 while package.json packageManager still
said pnpm@10.5.2. Bump packageManager to 10.19.0 (OIDC-capable) and drop
the redundant version: pins from both workflows so action-setup derives
the version from packageManager — one source of truth, no conflict.
@bufo24
bufo24 merged commit 844b4fb into main Jul 7, 2026
2 checks passed
@bufo24
bufo24 deleted the fix/pnpm-version-single-source branch July 7, 2026 10:41
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