fix!: update vp run argument order for flags-before-task breaking change

vite-task PR voidzero-dev/vite-task#286 and
voidzero-dev/vite-task#290 changed trailing_var_arg
to the command level, meaning flags like `-r` must now appear before the
task name. Tokens after the task name are passed through verbatim to the
task process.

- `vp run -r test` (correct) — `-r` is the recursive flag
- `vp run test -r` (broken) — `-r` is passed through to the task

Add snap test `command-run-argument-order` verifying both behaviors.
Update monorepo template, docs, and RFCs to use the new syntax.

Author: fengmk2

CLAUDE.md

@@ -13,9 +13,9 @@ vp test                            # Run Vitest (dedicated command)
 vp lint                            # Run oxlint (dedicated command)
 
 # Run tasks across packages (explicit mode)
-vp run build -r                    # recursive with topological ordering
+vp run -r build                    # recursive with topological ordering
 vp run app#build web#build         # specific packages
-vp run build -r --no-topological   # recursive without implicit deps
+vp run -r --no-topological build   # recursive without implicit deps
 
 # Run task in current package (implicit mode - for non-built-in tasks)
 vp run dev                         # runs dev script from package.json

packages/cli/snap-tests-global/new-vite-monorepo/snap.txt

@@ -16,7 +16,7 @@ vite.config.ts
   "private": true,
   "type": "module",
   "scripts": {
-    "ready": "vp fmt && vp lint && vp run test -r && vp run build -r",
+    "ready": "vp fmt && vp lint && vp run -r test && vp run -r build",
     "dev": "vp run website#dev",
     "prepare": "vp config"
   },
@@ -126,7 +126,7 @@ vite.config.ts
   "private": true,
   "type": "module",
   "scripts": {
-    "ready": "vp fmt && vp lint && vp run test -r && vp run build -r",
+    "ready": "vp fmt && vp lint && vp run -r test && vp run -r build",
     "dev": "vp run website#dev",
     "prepare": "vp config"
   },

packages/cli/snap-tests/command-run-argument-order/package.json

@@ -0,0 +1,6 @@
+{
+  "name": "run-argument-order",
+  "workspaces": [
+    "packages/*"
+  ]
+}

packages/cli/snap-tests/command-run-argument-order/packages/app-a/package.json

@@ -0,0 +1,9 @@
+{
+  "name": "app-a",
+  "scripts": {
+    "hello": "echo hello from app-a"
+  },
+  "dependencies": {
+    "lib-b": "workspace:*"
+  }
+}

packages/cli/snap-tests/command-run-argument-order/packages/lib-b/package.json

@@ -0,0 +1,6 @@
+{
+  "name": "lib-b",
+  "scripts": {
+    "hello": "echo hello from lib-b"
+  }
+}

packages/cli/snap-tests/command-run-argument-order/snap.txt

@@ -0,0 +1,14 @@
+> vp run -r hello # new syntax: -r before task, runs recursively
+~/packages/lib-b$ echo hello from lib-b ⊘ cache disabled
+hello from lib-b
+
+~/packages/app-a$ echo hello from app-a ⊘ cache disabled
+hello from app-a
+
+---
+vp run: 0/2 cache hit (0%). (Run `vp run --last-details` for full details)
+
+[1]> vp run hello -r # old syntax: -r after task, passed through as arg
+Task "hello" not found. Did you mean:
+  app-a#hello: echo hello from app-a
+  lib-b#hello: echo hello from lib-b

packages/cli/snap-tests/command-run-argument-order/steps.json

@@ -0,0 +1,10 @@
+{
+  "ignoredPlatforms": ["win32"],
+  "env": {
+    "VITE_DISABLE_AUTO_INSTALL": "1"
+  },
+  "commands": [
+    "vp run -r hello # new syntax: -r before task, runs recursively",
+    "vp run hello -r # old syntax: -r after task, passed through as arg"
+  ]
+}

packages/cli/templates/monorepo/README.md

@@ -13,13 +13,13 @@ vp run ready
 - Run the tests:
 
 ```bash
-vp run test -r
+vp run -r test
 ```
 
 - Build the monorepo:
 
 ```bash
-vp run build -r
+vp run -r build
 ```
 
 - Run the development server:

packages/cli/templates/monorepo/package.json

@@ -9,7 +9,7 @@
   ],
   "type": "module",
   "scripts": {
-    "ready": "vp fmt && vp lint && vp run test -r && vp run build -r",
+    "ready": "vp fmt && vp lint && vp run -r test && vp run -r build",
     "dev": "vp run website#dev"
   },
   "engines": {

rfcs/check-command.md

@@ -10,7 +10,7 @@ Currently, running a full code quality check requires chaining multiple commands
 
 ```bash
 # From the monorepo template's "ready" script:
-vp fmt && vp lint --type-aware && vp run test -r && vp run build -r
+vp fmt && vp lint --type-aware && vp run -r test && vp run -r build
 ```
 
 Pain points:
@@ -221,7 +221,7 @@ Options:
 With `vp check`, the monorepo template's "ready" script simplifies to:
 
 ```json
-"ready": "vp check && vp run test -r && vp run build -r"
+"ready": "vp check && vp run -r test && vp run -r build"
 ```
 
 ## Comparison with Other Tools