Skip to content

refactor: remove PM fallback from vp run, always delegate to vite-plus - #1179

Merged
graphite-app[bot] merged 1 commit into
mainfrom
vp-run-no-fallback
Mar 30, 2026
Merged

refactor: remove PM fallback from vp run, always delegate to vite-plus#1179
graphite-app[bot] merged 1 commit into
mainfrom
vp-run-no-fallback

Conversation

@fengmk2

Copy link
Copy Markdown
Member

The run_or_delegate module previously fell back to <pm> run <script>
when vite-plus was not a project dependency. This is no longer needed
because the global CLI ships with its own vite-plus installation, and
delegate::execute() already resolves local vite-plus first then falls
back to the global one.

Now vp run and vpr always delegate through the vite-plus task runner,
regardless of whether the project has vite-plus as a dependency.

Closes#1132

@fengmk2fengmk2 self-assigned this Mar 28, 2026
@fengmk2Graphite App

fengmk2 commented Mar 28, 2026

Copy link
Copy Markdown
MemberAuthor

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
fengmk2force-pushed the vp-run-no-fallback branch 2 times, most recently from f0fe3df to 3af4839CompareMarch 28, 2026 08:40
@fengmk2
fengmk2force-pushed the vp-run-no-fallback branch from 3af4839 to 3048c0dCompareMarch 29, 2026 11:36
@fengmk2
fengmk2force-pushed the vp-run-no-fallback branch from 3048c0d to ad0a0c7CompareMarch 29, 2026 14:52
@graphite-app
graphite-appBotforce-pushed the vp-run-alias-vpr branch 2 times, most recently from 959f493 to cf48911CompareMarch 29, 2026 15:43
@fengmk2
fengmk2force-pushed the vp-run-no-fallback branch from 6c24f5a to 707c56aCompareMarch 30, 2026 02:39
@fengmk2
fengmk2 marked this pull request as ready for review March 30, 2026 02:43
@fengmk2
fengmk2 requested review from cpojer and wan9chiMarch 30, 2026 02:43
@fengmk2
fengmk2force-pushed the vp-run-no-fallback branch 2 times, most recently from 9a5c382 to e4862b2CompareMarch 30, 2026 06:49
@fengmk2
fengmk2force-pushed the vp-run-no-fallback branch from e4862b2 to f898075CompareMarch 30, 2026 07:25
@fengmk2
fengmk2force-pushed the vp-run-no-fallback branch from f898075 to 0e5b9c2CompareMarch 30, 2026 07:40
@graphite-app

graphite-appBot commented Mar 30, 2026

Copy link
Copy Markdown

Merge activity

…lus (#1179)
The `run_or_delegate` module previously fell back to `<pm> run <script>`
when vite-plus was not a project dependency. This is no longer needed
because the global CLI ships with its own vite-plus installation, and
`delegate::execute()` already resolves local vite-plus first then falls
back to the global one.
Now `vp run` and `vpr` always delegate through the vite-plus task runner,
regardless of whether the project has vite-plus as a dependency.
Closes#1132
Base automatically changed from vp-run-alias-vpr to mainMarch 30, 2026 08:31
@graphite-app
graphite-appBot merged commit c682cb1 into mainMar 30, 2026
29 checks passed
@graphite-app
graphite-appBot deleted the vp-run-no-fallback branch March 30, 2026 08:37
Sign up for freeto 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.

vp run is not interactive if called without additional parameters

2 participants

@fengmk2@cpojer