@vzn/vx 0.0.415 → 0.0.417

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vzn/vx",
-  "version": "0.0.415",
+  "version": "0.0.417",
   "description": "The fastest task runner and build cache for JavaScript monorepos.",
   "type": "module",
   "exports": {
@@ -21,10 +21,10 @@
     "node": ">=18"
   },
   "optionalDependencies": {
-    "@vzn/vx-linux-x64": "0.0.415",
-    "@vzn/vx-linux-arm64": "0.0.415",
-    "@vzn/vx-darwin-x64": "0.0.415",
-    "@vzn/vx-darwin-arm64": "0.0.415"
+    "@vzn/vx-linux-x64": "0.0.417",
+    "@vzn/vx-linux-arm64": "0.0.417",
+    "@vzn/vx-darwin-x64": "0.0.417",
+    "@vzn/vx-darwin-arm64": "0.0.417"
   },
   "dependencies": {
     "@anthropic-ai/sandbox-runtime": "^0.0.76"

package/src/cli/watch.ts

@@ -42,6 +42,7 @@ import {
 } from './watch-filter.js'
 import { CLOSED, fsClockNow, type WatchHandle, WatcherPool } from './watch-fs.js'
 import { ChangeJudge } from './watch-judge.js'
+import { restartTimings } from '../util/index.js'
 import { memberEntries, sameMembers, sweepConfigs, watchedProjects } from './watch-set.js'
 
 /** One line for a watcher or re-read the OS refused; the loop goes on without it. */
@@ -357,6 +358,7 @@ async function runWatchLoop(args: WatchLoopArgs): Promise<number> {
         try {
           await held?.stop()
           held = undefined
+          restartTimings()
           const start = Date.now()
           held = (await runOrchestrator(opts)).persistent
           changes.lastCycle = { start, end: Date.now() }

package/src/util/index.ts

@@ -18,7 +18,7 @@ export {
   OUT_OF_FDS_HINT,
 } from './errors.js'
 export { xxh3, xxh3hex } from './hash.js'
-export { mark, printTimings, span } from './timing.js'
+export { mark, printTimings, restartTimings, span } from './timing.js'
 export { clampInt, formatElapsed, MAX_TIMEOUT_MS, parseDecimalInt } from './num.js'
 export {
   asTrees,

package/src/util/timing.ts

@@ -7,7 +7,7 @@
 // of it sit outside the table; the table shows each stage's own share.
 
 const enabled = process.env.VX_TIMING !== undefined && process.env.VX_TIMING !== ''
-const t0 = Bun.nanoseconds()
+let t0 = Bun.nanoseconds()
 const marks: Array<[label: string, ns: number]> = []
 
 /** Record the end of a stage. No-op unless `VX_TIMING` is set. */
@@ -19,6 +19,18 @@ export function mark(label: string): void {
 const spans = new Map<string, [count: number, ns: number]>()
 const noop = (): void => {}
 
+/**
+ * Start a new table: `vx watch` runs one per cycle in one process, and
+ * without this a cycle's table reprinted every earlier cycle's rows and
+ * its first stage counted the idle wait before it.
+ */
+export function restartTimings(): void {
+  if (!enabled) return
+  t0 = Bun.nanoseconds()
+  marks.length = 0
+  spans.clear()
+}
+
 /**
  * Time one occurrence of a repeated operation (a per-task probe, a restore):
  * `const end = span('probe'); …; end()`. Accumulated by label and printed

package/src/workspace/migrate-scripts.ts

@@ -557,6 +557,9 @@ function usesPnp(dir: string): boolean {
 const PNP_NOTE =
   "Yarn Plug'n'Play installs this repo: a package's bins live in `.pnp.cjs`, not `node_modules/.bin`, so a task's `tsc` is not found under vx — set `nodeLinker: node-modules` in `.yarnrc.yml` and run `yarn install`, or write each command as `yarn exec '<command>'`"
 
+/** A runner of this package's scripts by name: `run-p build:*`. */
+const RUNS_OWN = /(?:^|\s)(?:run-s|run-p|npm-run-all)(?:\s|$)/
+
 /** A package manager running a script by name: `pnpm build`, `npm run x`, `bun run x`. */
 const RUNS_SCRIPT =
   /(?:^|[\s;&|(])(?:pnpm|pn|npm|yarn|bun)\s+(?:run(?:-script)?\s+)?([^\s;&|()'"-][^\s;&|()'"]*)/g
@@ -835,6 +838,9 @@ export function migrateScripts(
     }
     const tasks: GeneratedTask[] = []
     let readsManifest = false
+    const text = Object.entries(scripts).filter(
+      (e): e is [string, string] => typeof e[1] === 'string',
+    )
     for (const name of names) {
       if (!isTask(name)) continue
 
@@ -886,19 +892,34 @@ export function migrateScripts(
       // pnpm run lint`) ran each again inside the command, beside the task
       // of that name: `vx run check` built twice. The chain's order may
       // matter, so it is named rather than turned into a group.
+      // Through `run-s` / `run-p` / `npm-run-all` too (`build: run-p
+      // build:*`), each segment naming every task it runs (M-48). A
+      // persistent one has PERSISTENT_TODO.
+      const persistent = isPersistent(name, scripts)
+      let ranTasks = 0
       const ownRuns = own
         .split(/&&|\|\||;/)
         .map((part) => part.trim())
         .filter((part) => {
           const d = delegatedScript(part)
-          return d !== null && d !== name && isTask(d)
+          if (d !== null) {
+            if (d === name || !isTask(d)) return false
+            ranTasks++
+            return true
+          }
+          if (persistent || !RUNS_OWN.test(part)) return false
+          const refs = new Set(scriptRefs(part, text).filter((r) => r !== name && isTask(r)))
+          ranTasks += refs.size
+          return refs.size > 0
         })
       if (ownRuns.length > 0) {
         const list = ownRuns.map((r) => `\`${r}\``).join(', ')
         todos.push(
-          ownRuns.length === 1
-            ? `${list} runs this package's own task again inside the command, beside that task: name it under dependsOn and drop it from the command`
-            : `${list} run this package's own tasks again inside the command, beside those tasks: name them under dependsOn and drop them from the command`,
+          `${list} ${ownRuns.length === 1 ? 'runs' : 'run'} this package's own ${
+            ranTasks === 1
+              ? 'task again inside the command, beside that task: name it'
+              : 'tasks again inside the command, beside those tasks: name them'
+          } under dependsOn and drop ${ownRuns.length === 1 ? 'it' : 'them'} from the command`,
         )
       }
       const exec: Record<string, unknown> = { command }
@@ -911,7 +932,7 @@ export function migrateScripts(
         )
       }
       const task: Record<string, unknown> = { exec }
-      if (isPersistent(name, scripts)) {
+      if (persistent) {
         exec['persistent'] = {}
         todos.push(PERSISTENT_TODO)
       }

package/src/workspace/workspace.ts

@@ -290,6 +290,11 @@ function parsePackageJson(text: string, file: string): PackageJson {
   if (name !== undefined && (typeof name !== 'string' || name.trim() !== name)) {
     throw new UserError(`${file}: "name" must be a string with no surrounding whitespace`)
   }
+  // A task is `<project>#<task>` split at the first `#`: `"a#b"` planned
+  // under `--all` but no run spec or dependsOn could reach it (D-130).
+  if (typeof name === 'string' && name.includes('#')) {
+    throw new UserError(`${file}: "name" cannot hold "#" — vx addresses a task as <name>#<task>`)
+  }
   return pkg as PackageJson
 }