Skip to content

chore: update pnpm config#4208

Open
btea wants to merge 1 commit intomainfrom
chore/update-pnpm-config
Open

chore: update pnpm config#4208
btea wants to merge 1 commit intomainfrom
chore/update-pnpm-config

Conversation

@btea
Copy link
Copy Markdown
Contributor

@btea btea commented Apr 16, 2026

🔗 Linked issue

❓ Type of change

  • 📖 Documentation (updates to the documentation, readme, or JSdoc annotations)
  • 🐞 Bug fix (a non-breaking change that fixes an issue)
  • 👌 Enhancement (improving an existing functionality like performance)
  • ✨ New feature (a non-breaking change that adds functionality)
  • 🧹 Chore (updates to the build process or auxiliary tools and libraries)
  • ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

dedupePeers refers to https://github.com/pnpm/pnpm/releases/tag/v10.33.0

allowBuilds refers to https://pnpm.io/settings#allowbuilds

📝 Checklist

  • I have linked an issue or discussion.
  • I have updated the documentation accordingly.

@btea btea requested a review from pi0 as a code owner April 16, 2026 13:05
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 16, 2026

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

Project Deployment Actions Updated (UTC)
nitro.build Ready Ready Preview, Comment Apr 16, 2026 1:05pm

Request Review

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 16, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: b83e1d58-8344-4ac9-bb10-48057407c828

📥 Commits

Reviewing files that changed from the base of the PR and between 0a6cd73 and af9f427.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (1)
  • pnpm-workspace.yaml

📝 Walkthrough

Walkthrough

Updates the pnpm workspace configuration by replacing ignoredBuiltDependencies and onlyBuiltDependencies with a new allowBuilds mapping that specifies which packages should have builds enabled or disabled, and adds dedupePeers: true for peer dependency deduplication.

Changes

Cohort / File(s) Summary
pnpm Workspace Configuration
pnpm-workspace.yaml
Replaced legacy build dependency configuration blocks (ignoredBuiltDependencies, onlyBuiltDependencies) with explicit allowBuilds mapping. Disabled builds for keytar, protobufjs, vue-demi; enabled builds for @parcel/watcher, esbuild, ocache, sharp, workerd. Added dedupePeers: true setting.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title 'chore: update pnpm config' follows conventional commits format with appropriate type prefix and clearly describes the configuration update.
Description check ✅ Passed The description is related to the changeset, referencing the two main configuration changes (dedupePeers and allowBuilds) with links to pnpm documentation.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/update-pnpm-config

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.

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Apr 16, 2026

Open in StackBlitz

npm i https://pkg.pr.new/nitro@4208

commit: af9f427

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