Skip to content

fix(upgrade): isolate pnpm install from parent workspaces - #2644

Open
fengmk2 wants to merge 2 commits into
mainfrom
fix/upgrade-parent-workspace
Open

fix(upgrade): isolate pnpm install from parent workspaces#2644
fengmk2 wants to merge 2 commits into
mainfrom
fix/upgrade-parent-workspace

Conversation

@fengmk2

@fengmk2 fengmk2 commented Sep 9, 2026

Copy link
Copy Markdown
Member

vp upgrade can use a parent pnpm workspace and fail when its lockfile is stale. With a valid lockfile, it can report success without its own dependencies.

The installer now adds --ignore-workspace to pnpm install. Two snapshot tests check the global package and the unchanged parent workspace for both lockfile states.

Fixes #2639.

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Sep 9, 2026

Copy link
Copy Markdown

🚀 Deploying Preview to Cloudflare 🚀

Preview URL: https://fix-upgrade-parent-workspace-viteplus-dev.voidzero-docs.workers.dev (commit 7064ecd)

This URL reflects your latest Preview deployment

Preview Deployments by commit

Status Deployment URL Commit Updated (UTC) See this deployment's details
  • Build: Success ✅
  • Deployment: Success ✅

View logs ↗
https://08c5fda3-viteplus-dev.voidzero-docs.workers.dev 7064ecd 2026-09-09T14:17:05.318Z Visit the dashboard ↗
  • Build: Success ✅
  • Deployment: Success ✅

View logs ↗
https://02a39682-viteplus-dev.voidzero-docs.workers.dev 9164254 2026-09-09T13:52:19.366Z Visit the dashboard ↗

@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

CLI artifact sizes (7064ecd)

Final release artifacts built by the canonical build-upstream and build-windows-cli actions.
The dist rows use the Linux build. The core total excludes .node files to match the release artifact.

Artifact Format Base PR Change
packages/cli/dist Directory total 2.14 MiB 2.14 MiB 0 B (0.00%)
packages/core/dist Directory total 3.92 MiB 3.92 MiB 0 B (0.00%)
Combined package dist Directory total 6.06 MiB 6.06 MiB 0 B (0.00%)
vp (Linux x64) Binary 10.99 MiB 10.99 MiB 0 B (0.00%)
vp (Linux x64) gzip -9 4.76 MiB 4.76 MiB +22 B (+0.00%)
NAPI (Linux x64) Binary 31.94 MiB 31.94 MiB 0 B (0.00%)
NAPI (Linux x64) gzip -9 12.61 MiB 12.61 MiB 0 B (0.00%)
vp (macOS ARM64) Binary 8.19 MiB 8.19 MiB 0 B (0.00%)
vp (macOS ARM64) gzip -9 4.15 MiB 4.15 MiB +73 B (+0.00%)
NAPI (macOS ARM64) Binary 39.55 MiB 39.55 MiB 0 B (0.00%)
NAPI (macOS ARM64) gzip -9 16.92 MiB 16.92 MiB 0 B (0.00%)
vp (Windows x64) Binary 8.86 MiB 8.86 MiB 0 B (0.00%)
vp (Windows x64) gzip -9 3.88 MiB 3.88 MiB +118 B (+0.00%)
NAPI (Windows x64) Binary 26.82 MiB 26.82 MiB 0 B (0.00%)
NAPI (Windows x64) gzip -9 10.71 MiB 10.71 MiB -2 B (-0.00%)
Trampoline (Windows x64) Binary 14.00 KiB 14.00 KiB 0 B (0.00%)
Trampoline (Windows x64) gzip -9 7.09 KiB 7.09 KiB 0 B (0.00%)
Installer (Windows x64) Binary 4.50 MiB 4.50 MiB 0 B (0.00%)
Installer (Windows x64) gzip -9 2.11 MiB 2.11 MiB +96 B (+0.00%)

@fengmk2
fengmk2 marked this pull request as ready for review September 9, 2026 14:27
@fengmk2
fengmk2 requested review from cpojer and liangmiQwQ September 9, 2026 14:27
@fengmk2 fengmk2 self-assigned this Sep 9, 2026

@liangmiQwQ liangmiQwQ left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM 💯

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.

I can't seem to upgrade via vp upgrade

2 participants