Skip to content

chore: bump pnpm in mise#7000

Merged
LesnyRumcajs merged 2 commits intomainfrom
bump-pnpm
May 5, 2026
Merged

chore: bump pnpm in mise#7000
LesnyRumcajs merged 2 commits intomainfrom
bump-pnpm

Conversation

@LesnyRumcajs
Copy link
Copy Markdown
Member

@LesnyRumcajs LesnyRumcajs commented May 5, 2026

Summary of changes

Changes introduced in this pull request:

Reference issue to close (if applicable)

Closes

Other information and links

Change checklist

  • I have performed a self-review of my own code,
  • I have made corresponding changes to the documentation. All new code adheres to the team's documentation standards,
  • I have added tests that prove my fix is effective or that my feature works (if possible),
  • I have made sure the CHANGELOG is up-to-date. All user-facing changes should be reflected in this document.

Outside contributions

  • I have read and agree to the CONTRIBUTING document.
  • I have read and agree to the AI Policy document. I understand that failure to comply with the guidelines will lead to rejection of the pull request.

Summary by CodeRabbit

  • Chores
    • Updated the pnpm package manager version used in development configuration.
    • Removed top-level package resolution overrides from the docs configuration.
    • Adjusted workspace settings to pin select dependency versions, enable stricter exotic-subdependency blocking, and disable builds for certain build tools.

@LesnyRumcajs LesnyRumcajs requested a review from a team as a code owner May 5, 2026 07:36
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 5, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 2f126943-1733-4f58-b110-1e03f451c367

📥 Commits

Reviewing files that changed from the base of the PR and between cf46a14 and 173107f.

📒 Files selected for processing (3)
  • docs/package.json
  • docs/pnpm-workspace.yaml
  • pnpm-workspace.yaml
💤 Files with no reviewable changes (1)
  • docs/package.json
✅ Files skipped from review due to trivial changes (2)
  • pnpm-workspace.yaml
  • docs/pnpm-workspace.yaml

Walkthrough

Bumps tools.pnpm from 10 → 11 in mise.toml; removes the top-level resolutions block from docs/package.json; adds allowBuilds and overrides to docs/pnpm-workspace.yaml and disables builds for certain packages in pnpm-workspace.yaml. All changes are configuration-only.

Changes

Workspace & Dependency overrides

Layer / File(s) Summary
Tool version
mise.toml
Bumps tools.pnpm from 10 to 11.
Workspace policy
pnpm-workspace.yaml, docs/pnpm-workspace.yaml
Enables blockExoticSubdeps: true; adds allowBuilds entries to disable builds for specific packages (esbuild, workerd in root; core-js, core-js-pure, esbuild, puppeteer in docs).
Dependency pinning / overrides
docs/pnpm-workspace.yaml
Adds overrides to enforce versions for dompurify, lodash, lodash-es, serialize-javascript, and uuid.
Removed duplicate pinning
docs/package.json
Removes the top-level resolutions block that previously specified overrides for the same dependencies.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • hanabi1224
  • sudo-shashank
  • akaladarshi
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'chore: bump pnpm in mise' accurately reflects the primary change: updating the pnpm version in mise.toml from 10 to 11.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ 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 bump-pnpm
✨ Simplify code
  • Create PR with simplified code
  • Commit simplified code in branch bump-pnpm

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

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@mise.toml`:
- Line 272: Update the pnpm version entry in mise.toml to "11" (pnpm = "11") and
then run pnpm setup to handle the new global bin layout; also verify there are
no pnpm-specific settings in package.json (no package.json#pnpm field), .npmrc,
or pnpm-workspace.yaml so the upgrade remains minimal and the lockfile will
auto-migrate on first pnpm install.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 3af17634-accb-42aa-9467-c7b1cd1dc331

📥 Commits

Reviewing files that changed from the base of the PR and between a6e42ff and 491a388.

📒 Files selected for processing (1)
  • mise.toml

Comment thread mise.toml
sudo-shashank
sudo-shashank previously approved these changes May 5, 2026
akaladarshi
akaladarshi previously approved these changes May 5, 2026
@LesnyRumcajs LesnyRumcajs enabled auto-merge May 5, 2026 07:46
@codecov
Copy link
Copy Markdown

codecov Bot commented May 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 64.10%. Comparing base (ed59373) to head (173107f).
⚠️ Report is 2 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files

see 13 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a6e42ff...173107f. Read the comment docs.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@LesnyRumcajs LesnyRumcajs added this pull request to the merge queue May 5, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks May 5, 2026
hanabi1224
hanabi1224 previously approved these changes May 5, 2026
@hanabi1224 hanabi1224 enabled auto-merge May 5, 2026 08:48
@hanabi1224 hanabi1224 added this pull request to the merge queue May 5, 2026
@LesnyRumcajs LesnyRumcajs removed this pull request from the merge queue due to a manual request May 5, 2026
@LesnyRumcajs LesnyRumcajs force-pushed the bump-pnpm branch 2 times, most recently from 00f053f to cf46a14 Compare May 5, 2026 09:08
@LesnyRumcajs LesnyRumcajs enabled auto-merge May 5, 2026 09:08
sudo-shashank
sudo-shashank previously approved these changes May 5, 2026
@LesnyRumcajs LesnyRumcajs added this pull request to the merge queue May 5, 2026
Merged via the queue into main with commit 420a5ab May 5, 2026
46 checks passed
@LesnyRumcajs LesnyRumcajs deleted the bump-pnpm branch May 5, 2026 10:01
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.

4 participants