Skip to content

docs: refresh README for pnpm global virtual store support#22

Merged
SoonIter merged 3 commits intomainfrom
copilot/update-readme-for-global-virtual-store
Mar 17, 2026
Merged

docs: refresh README for pnpm global virtual store support#22
SoonIter merged 3 commits intomainfrom
copilot/update-readme-for-global-virtual-store

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 17, 2026

The README was still describing pnpm path normalization as if only local .pnpm/... layouts were supported. This updates the docs to reflect current behavior with enableGlobalVirtualStore.

  • README updates

    • Clarify that pnpm path serialization also supports global virtual store layouts
    • Mention both pnpm config forms:
      • enableGlobalVirtualStore
      • enable-global-virtual-store=true in .npmrc
  • Behavior documented

    • Expand the replacePnpmInner feature description to cover both:
      • local virtual store paths: .pnpm/...
      • global virtual store paths: pnpm/store/.../links/...
  • Example refreshed

    • Add a concrete global virtual store path example and its normalized output
<ROOT>/node_modules/../../../../../../Library/pnpm/store/v10/links/react/19.2.4/<hash>/node_modules/react/index.js

becomes:

<PNPM_INNER>/react/index.js

📱 Kick off Copilot coding agent tasks wherever you are with GitHub Mobile, available on iOS and Android.

Copilot AI changed the title [WIP] Update README to reflect enableGlobalVirtualStore support docs: refresh README for pnpm global virtual store support Mar 17, 2026
Copilot AI requested a review from SoonIter March 17, 2026 10:18
Copilot AI and others added 3 commits March 17, 2026 19:57
Co-authored-by: SoonIter <79413249+SoonIter@users.noreply.github.com>
@SoonIter SoonIter force-pushed the copilot/update-readme-for-global-virtual-store branch from 9130565 to f8685c6 Compare March 17, 2026 12:01
@SoonIter SoonIter marked this pull request as ready for review March 17, 2026 12:02
Copilot AI review requested due to automatic review settings March 17, 2026 12:02
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Documentation-only update to reflect that replacePnpmInner also handles pnpm global virtual store paths (enableGlobalVirtualStore), and corrects the example output to show the prefix-stripped <PNPM_INNER> format.

Changes:

  • Added mention of enableGlobalVirtualStore support in the feature list
  • Fixed the example output to show <PNPM_INNER>/css-loader/utils.ts (without <ROOT>/node_modules/ prefix), matching actual behavior
  • Updated the replacePnpmInner feature table description to cover global virtual store paths

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

@SoonIter SoonIter merged commit 153f5c8 into main Mar 17, 2026
9 checks passed
@SoonIter SoonIter deleted the copilot/update-readme-for-global-virtual-store branch March 17, 2026 12:03
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.

3 participants