Skip to content

chore(deps): upgrade pnp to 0.12.11#286

Open
stormslowly wants to merge 1 commit into
mainfrom
chore/upgrade-pnp-0.12.11
Open

chore(deps): upgrade pnp to 0.12.11#286
stormslowly wants to merge 1 commit into
mainfrom
chore/upgrade-pnp-0.12.11

Conversation

@stormslowly

Copy link
Copy Markdown
Collaborator

Why

Bump the pnp crate from 0.12.9 to 0.12.11 to pick up the loose-mode fallback resolution fix (yarnpkg/pnp-rs#110, fixes yarnpkg/pnp-rs#111).

Before: in loose mode, pnp-rs queried the deduplicated fallback pool directly, so it could resolve a different dependency version than the Yarn runtime (Yarn checks the top-level fallback locator first).

After: fallback resolution matches Yarn's ordering — top-level locator first, then the fallback pool.

0.12.10 also replaces fancy-regex with regress.

Includes the loose-mode fallback resolution fix (yarnpkg/pnp-rs#110,
fixes yarnpkg/pnp-rs#111) and replaces fancy-regex with regress.
Copilot AI review requested due to automatic review settings July 23, 2026 03:41
@coderabbitai

coderabbitai Bot commented Jul 23, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 42924215-d0bf-4c3b-8124-bedc27eeb40b

📥 Commits

Reviewing files that changed from the base of the PR and between 9a600a4 and 62836d3.

⛔ Files ignored due to path filters (1)
  • Cargo.lock is excluded by !**/*.lock
📒 Files selected for processing (1)
  • Cargo.toml

📝 Walkthrough

Walkthrough

Updated the optional pnp dependency in Cargo.toml from version 0.12.9 to 0.12.11.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title precisely describes the dependency upgrade shown in the changeset.
Description check ✅ Passed The description directly explains the pnp version bump and its motivation.
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/upgrade-pnp-0.12.11

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.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the Rust pnp dependency to pick up upstream fixes so loose-mode fallback resolution matches Yarn’s ordering, and incorporates the upstream switch from fancy-regex to regress.

Changes:

  • Bump pnp crate from 0.12.9 to 0.12.11.
  • Regenerate Cargo.lock accordingly (removing fancy-regex and related transitive crates; adding regress).

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File Description
Cargo.toml Updates the pnp dependency version to 0.12.11.
Cargo.lock Updates the lockfile for the new pnp version and its transitive dependency changes.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@codspeed-hq

codspeed-hq Bot commented Jul 23, 2026

Copy link
Copy Markdown

Merging this PR will improve performance by 3.53%

⚡ 1 improved benchmark
✅ 11 untouched benchmarks

Performance Changes

Mode Benchmark BASE HEAD Efficiency
Simulation resolver[pnp resolve] 229 µs 221.2 µs +3.53%

Tip

Curious why this is faster? Comment @codspeedbot explain why this is faster on this PR, or directly use the CodSpeed MCP with your agent.


Comparing chore/upgrade-pnp-0.12.11 (62836d3) with main (9a600a4)

Open in CodSpeed

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.

Loose-mode fallback resolution can select the wrong dependency version

2 participants