chore: upgrade pnpm to 11.10, turbo to latest, fix dependabot alert - #9734
Conversation
…r-parser - packageManager: pnpm@11.3.0 -> pnpm@11.10.0 - turbo catalog: 2.9.18 -> 2.10.11 (latest) - override postcss-selector-parser >=7.1.0 <7.1.3 to 7.1.3 (dependabot alert #345, DoS via uncontrolled AST recursion)
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Team Run ID: 📒 Files selected for processing (1)
💤 Files with no reviewable changes (1)
Included review availability: Your plan provides up to 10 included reviews per hour; 8 remain after this review. 📝 WalkthroughWalkthroughThe repository updates its required pnpm version, upgrades the Turbo catalog entry, and pins affected ChangesToolchain updates
Estimated code review effort: 1 (Trivial) | ~5 minutes Merge Risk: ⚪ Minimal · up to This PR updates pnpm and Turbo and pins the affected transitive dependency to a fixed version; no actionable merge-blocking risk remains after normal checks and review. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Full details: Description checkExplanation The description clearly explains the version updates, the security override, the affected dependency path, and the verification results. It does not use the template headings or include a Type of Change selection, but the required change and test information is present. Full details: Docstring CoverageExplanation No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.) ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Pull request overview
Updates the repo’s tooling baseline (pnpm + Turborepo) and mitigates a low-severity security advisory by forcing a safe transitive dependency version via pnpm overrides.
Changes:
- Bumped
packageManagerin rootpackage.jsontopnpm@11.10.0. - Updated the workspace catalog entry for
turboto2.10.11and reflected it in the lockfile. - Added a scoped pnpm override to pin vulnerable
postcss-selector-parserversions to7.1.3, with corresponding lockfile resolution.
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| pnpm-workspace.yaml | Updates turbo catalog version and adds a scoped override to pin postcss-selector-parser to a non-vulnerable version. |
| pnpm-lock.yaml | Regenerates lockfile to reflect the turbo upgrade and the postcss-selector-parser override (resolving to 7.1.3). |
| package.json | Updates the root packageManager field to pnpm 11.10.0. |
Files not reviewed (1)
- pnpm-lock.yaml: Generated file
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Changes
11.3.0→11.10.0(packageManagerin rootpackage.json)2.9.18→2.10.11(latest, via catalog inpnpm-workspace.yaml)postcss-selector-parserDoS via uncontrolled AST recursion (>=7.1.0 <7.1.3). Pulled in transitively bypostcss-modules-local-by-default/postcss-modules-scope; pinned to7.1.3with a scoped override inpnpm-workspace.yaml.Verification
pnpm installpasses on pnpm v11.10.0; lockfile resolves onlypostcss-selector-parser@7.1.3turbo --version→ 2.10.11;turbo run check:types --drypasses against existingturbo.jsonSummary by CodeRabbit
Security
Maintenance