Skip to content

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

Merged
fengmk2 merged 3 commits into
mainfrom
fix/upgrade-parent-workspace
Sep 11, 2026
Merged

fix(upgrade): isolate pnpm install from parent workspaces#2644
fengmk2 merged 3 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 f4769a8)

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://db7df430-viteplus-dev.voidzero-docs.workers.dev f4769a8 2026-09-11T02:06:37.636Z Visit the dashboard ↗
  • Build: Success ✅
  • Deployment: Success ✅

View logs ↗
https://00f4daaa-viteplus-dev.voidzero-docs.workers.dev 8377415 2026-09-10T15:08:16.327Z Visit the dashboard ↗
  • 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 (f4769a8)

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 11.14 MiB 11.14 MiB 0 B (0.00%)
vp (Linux x64) gzip -9 4.82 MiB 4.82 MiB +121 B (+0.00%)
NAPI (Linux x64) Binary 32.03 MiB 32.03 MiB 0 B (0.00%)
NAPI (Linux x64) gzip -9 12.64 MiB 12.64 MiB 0 B (0.00%)
vp (macOS ARM64) Binary 8.30 MiB 8.30 MiB 0 B (0.00%)
vp (macOS ARM64) gzip -9 4.19 MiB 4.19 MiB +111 B (+0.00%)
NAPI (macOS ARM64) Binary 39.61 MiB 39.61 MiB 0 B (0.00%)
NAPI (macOS ARM64) gzip -9 16.96 MiB 16.96 MiB -1 B (-0.00%)
vp (Windows x64) Binary 9.01 MiB 9.01 MiB 0 B (0.00%)
vp (Windows x64) gzip -9 3.93 MiB 3.93 MiB +116 B (+0.00%)
NAPI (Windows x64) Binary 26.92 MiB 26.92 MiB 0 B (0.00%)
NAPI (Windows x64) gzip -9 10.74 MiB 10.74 MiB -6 B (-0.00%)
Trampoline (Windows x64) Binary 13.50 KiB 13.50 KiB 0 B (0.00%)
Trampoline (Windows x64) gzip -9 7.03 KiB 7.03 KiB 0 B (0.00%)
Installer (Windows x64) Binary 4.52 MiB 4.52 MiB 0 B (0.00%)
Installer (Windows x64) gzip -9 2.11 MiB 2.11 MiB -77 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 💯

@fengmk2
fengmk2 removed the request for review from cpojer September 10, 2026 15:03
@fengmk2
fengmk2 force-pushed the fix/upgrade-parent-workspace branch from 7064ecd to 8377415 Compare September 10, 2026 15:06
@fengmk2
fengmk2 requested review from cpojer and removed request for cpojer September 10, 2026 15:08
@fengmk2
fengmk2 merged commit 17ad495 into main Sep 11, 2026
70 checks passed
@fengmk2
fengmk2 deleted the fix/upgrade-parent-workspace branch September 11, 2026 02:20
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

3 participants