Skip to content

ci(workflows): pin pnpm version via packageManager field#7

Draft
kitsunoff wants to merge 1 commit into
mainfrom
fix/ci-pnpm-version
Draft

ci(workflows): pin pnpm version via packageManager field#7
kitsunoff wants to merge 1 commit into
mainfrom
fix/ci-pnpm-version

Conversation

@kitsunoff
Copy link
Copy Markdown

What

Add "packageManager": "pnpm@10.26.0" to the root package.json.

Why

The test workflow added in #5 has been failing on every run with
No pnpm version is specifiedpnpm/action-setup@v4 requires either
a version: input on the action or a packageManager field in
package.json. The latter is the preferred declaration because it
also pins the version for local developers, not just CI.

Verification

  • pnpm install --frozen-lockfile — clean
  • pnpm typecheck — clean
  • pnpm test — 29/29 passing

Once merged, the test check should go green on main and on
subsequent PRs.

pnpm/action-setup@v4 was failing the test workflow with "No pnpm version
is specified". Declaring packageManager in package.json is the preferred
way to fix this — the action picks it up automatically.

Co-Authored-By: Claude <noreply@anthropic.com>
Signed-off-by: ZverGuy <maximbel2003@gmail.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 13, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 628aef83-3091-4c3c-9e69-830b2d1ae179

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/ci-pnpm-version

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request adds the packageManager field to package.json. The review feedback correctly identifies a syntax error due to a missing trailing comma and points out that the specified pnpm version is invalid, providing a code suggestion to resolve these issues.

Comment thread package.json
"name": "cozystack-ui",
"private": true,
"type": "module",
"packageManager": "pnpm@10.26.0",
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

critical

This line is missing a trailing comma, which makes the package.json file invalid JSON. Any subsequent pnpm or npm commands will fail due to this syntax error.

Additionally, the specified pnpm version 10.26.0 does not appear to be a valid, publicly released version. This will likely cause the CI to fail when pnpm/action-setup tries to download it, as it won't be found in the official pnpm releases.

Please add the comma and use a valid pnpm version to fix both issues.

Suggested change
"packageManager": "pnpm@10.26.0",
"packageManager": "pnpm@9.4.0",

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