Skip to content

feat(cli): add vp node shorthand for vp env exec node#1359

Merged
fengmk2 merged 3 commits intomainfrom
add-shorthand-vp-node
Apr 13, 2026
Merged

feat(cli): add vp node shorthand for vp env exec node#1359
fengmk2 merged 3 commits intomainfrom
add-shorthand-vp-node

Conversation

@fengmk2
Copy link
Copy Markdown
Member

@fengmk2 fengmk2 commented Apr 11, 2026

Rewrite vp node [args...] to vp env exec node [args...] in the
pre-clap argument normalizer so the shorthand reuses the existing shim
dispatch (automatic Node version resolution, auto-download, recursion
prevention). No changes to the Commands enum — avoids growing the
derived clap parser.

Closes #1325

Rewrite `vp node [args...]` to `vp env exec node [args...]` in the
pre-clap argument normalizer so the shorthand reuses the existing shim
dispatch (automatic Node version resolution, auto-download, recursion
prevention). No changes to the Commands enum — avoids growing the
derived clap parser.

Closes #1325
@netlify
Copy link
Copy Markdown

netlify bot commented Apr 11, 2026

Deploy Preview for viteplus-preview canceled.

Name Link
🔨 Latest commit a3b2562
🔍 Latest deploy log https://app.netlify.com/projects/viteplus-preview/deploys/69dc63019c91ee000891eb17

@fengmk2 fengmk2 self-assigned this Apr 11, 2026
Copy link
Copy Markdown
Member Author

fengmk2 commented Apr 11, 2026


How to use the Graphite Merge Queue

Add the label auto-merge to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

@fengmk2 fengmk2 force-pushed the add-shorthand-vp-node branch from 5f45cc3 to a7ead2b Compare April 12, 2026 15:28
@fengmk2 fengmk2 marked this pull request as ready for review April 13, 2026 00:55
@fengmk2 fengmk2 requested review from branchseer and cpojer April 13, 2026 00:55
Copy link
Copy Markdown
Member

@cpojer cpojer left a comment

Choose a reason for hiding this comment

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

Elegant.

@graphite-app
Copy link
Copy Markdown

graphite-app bot commented Apr 13, 2026

Merge activity

@fengmk2 fengmk2 merged commit eb7370a into main Apr 13, 2026
37 checks passed
@fengmk2 fengmk2 deleted the add-shorthand-vp-node branch April 13, 2026 06:11
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.

Feature Request: Native Node.js Script Execution via vp node or vp run

2 participants