Skip to content

fix(ci): isolate upstream CLI help capture - #2976

Open
fengmk2 wants to merge 2 commits into
mainfrom
fix/upgrade-cli-help-isolation
Open

fengmk2 wants to merge 2 commits into
mainfrom
fix/upgrade-cli-help-isolation

Conversation

@fengmk2

@fengmk2 fengmk2 commented Oct 10, 2026

Copy link
Copy Markdown
Member

The dependency upgrade workflow fails because pnpm@12.11.2 applies the repository's workspace overrides to pnpm dlx. It resolves vite to workspace:@voidzero-dev/vite-plus-core@*, which is unavailable in the temporary installation.

The help capture script now runs pnpm dlx from a temporary directory outside the repository. This keeps workspace overrides and patches out of upstream package resolution. The script still reads versions from the repository or upgrade metadata. It removes the temporary directory after success or failure.

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Oct 10, 2026 •

Copy link
Copy Markdown

🚀 Deploying Preview to Cloudflare 🚀

Preview URL: https://fix-upgrade-cli-help-isolation-viteplus-dev.voidzero-docs.workers.dev (commit 151c2ff)

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://e16e0be8-viteplus-dev.voidzero-docs.workers.dev 151c2ff 2026-10-10T11:39:55.046Z Visit the dashboard ↗
  • Build: Success ✅
  • Deployment: Success ✅

View logs ↗
https://4bdd05e7-viteplus-dev.voidzero-docs.workers.dev d4875ae 2026-10-10T11:38:37.519Z Visit the dashboard ↗

@fengmk2
fengmk2 marked this pull request as ready for review October 10, 2026 11:43
@fengmk2 fengmk2 self-assigned this Oct 10, 2026
@fengmk2

fengmk2 commented Oct 10, 2026

Copy link
Copy Markdown
Member Author

@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 requested a review from wan9chi October 10, 2026 13:10

This branch has not been deployed

No deployments
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.

2 participants