@vzn/vx 0.0.194 → 0.0.196

package/package.json

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

package/src/cli/watch-set.ts

@@ -88,6 +88,7 @@ export async function sweepConfigs(
   projects: readonly ProjectMeta[],
   workspaceRoot: string,
   load: CliLoadOptions = {},
+  tasks?: readonly string[],
 ): Promise<ConfigSweep> {
   const outputs = new Map<string, string[]>()
   const inputs = new Map<string, string[][]>()
@@ -115,8 +116,22 @@ export async function sweepConfigs(
     addTo(outputs, dir, globs)
   const workspaceInputs = new Set<string>()
   const uncached = new Set<string>()
+  const folded: [string, ProjectConfig][] = []
   const fold = (dir: string, config: ProjectConfig): void => {
-    for (const task of Object.values(config.tasks ?? {})) {
+    folded.push([dir, config])
+  }
+  // Only the tasks the run can reach are judged: `vx watch build` over a
+  // turbo() package whose `lint` reads `**/*` took build's own `dist/`
+  // write for lint's input, and re-ran build.
+  const foldAll = (): void => {
+    const names = tasks === undefined ? null : reachableNames(folded, tasks)
+    for (const [dir, config] of folded) foldTasks(dir, config, names)
+  }
+  const foldTasks = (dir: string, config: ProjectConfig, names: Set<string> | null): void => {
+    for (const [name, task] of Object.entries(config.tasks ?? {})) {
+      add(dir, task.cache?.outputs?.files)
+      add(workspaceRoot, task.cache?.outputs?.workspaceFiles)
+      if (names !== null && !names.has(name)) continue
       if (task.cache === undefined && task.exec !== undefined && task.exec.persistent === undefined)
         uncached.add(dir)
       for (const g of task.cache?.inputs?.workspaceFiles ?? []) workspaceInputs.add(g)
@@ -126,8 +141,6 @@ export async function sweepConfigs(
         task.cache?.inputs?.workspaceFiles,
         task.cache?.outputs?.workspaceFiles,
       )
-      add(dir, task.cache?.outputs?.files)
-      add(workspaceRoot, task.cache?.outputs?.workspaceFiles)
     }
   }
   const result = (staged: Map<string, ProjectEntry> | null): ConfigSweep => ({
@@ -159,6 +172,7 @@ export async function sweepConfigs(
   }
   if (staged !== null) {
     for (const p of staged.values()) fold(p.dir, p.config)
+    foldAll()
     return result(staged)
   }
   await Promise.all(
@@ -171,9 +185,40 @@ export async function sweepConfigs(
       }
     }),
   )
+  foldAll()
   return result(null)
 }
 
+/**
+ * The task names a run of `requested` reaches through `dependsOn`, by name
+ * in any project: a superset of the graph's tasks, so a file is never
+ * dropped that one of them reads. Null when a name pattern (`build.*`)
+ * makes the set unknowable without the graph: every task then counts.
+ */
+function reachableNames(
+  configs: ReadonlyArray<readonly [string, ProjectConfig]>,
+  requested: readonly string[],
+): Set<string> | null {
+  const name = (spec: string): string => spec.slice(spec.indexOf('#') + 1).replace(/^\^/, '')
+  const names = new Set(requested.map(name))
+  for (let grew = true; grew;) {
+    grew = false
+    for (const [, config] of configs) {
+      for (const [task, def] of Object.entries(config.tasks ?? {})) {
+        if (!names.has(task)) continue
+        for (const dep of def.dependsOn ?? []) {
+          const n = name(dep)
+          if (!names.has(n)) {
+            names.add(n)
+            grew = true
+          }
+        }
+      }
+    }
+  }
+  return [...names].some((n) => n.includes('*')) ? null : names
+}
+
 /**
  * The directory entries under a package glob's directory that can be
  * members: directories (or links) whose name is not dotted and not

package/src/cli/watch.ts

@@ -201,7 +201,7 @@ export async function watchCmd(args: readonly string[]): Promise<number> {
     ...(opts.cacheDir !== undefined ? { cacheDir: opts.cacheDir } : {}),
     ...(opts.frozen === true ? { frozen: true } : {}),
   }
-  const swept = await sweepConfigs(allProjects, workspaceRoot, load)
+  const swept = await sweepConfigs(allProjects, workspaceRoot, load, opts.tasks)
   const watched = await watchedProjects(workspaceRoot, allProjects, scope, load, swept.staged)
   const ws = await loadCliWorkspace(workspaceRoot)
   return await runWatchLoop({
@@ -227,7 +227,7 @@ export async function watchCmd(args: readonly string[]): Promise<number> {
     rediscover: async () => {
       const workspace = await loadWorkspace(workspaceRoot)
       const all = await listProjects(workspace)
-      const sweep = await sweepConfigs(all, workspaceRoot, load)
+      const sweep = await sweepConfigs(all, workspaceRoot, load, opts.tasks)
       const now = await watchedProjects(workspaceRoot, all, inScope(all), load, sweep.staged)
       return {
         projects: now,

package/src/workspace/project-loader.ts

@@ -515,6 +515,16 @@ export async function loadProjectConfigs(
 const WATCHED_BUILTINS: ReadonlyArray<readonly [string, object]> = [
   ['Object.prototype', Object.prototype],
   ['Array.prototype', Array.prototype],
+  // What vx itself runs on: `Bun.hash.xxHash3 = () => 7n` gave every task
+  // the key 00000000, and a changed command replayed the old output (D-75).
+  ['Bun', Bun],
+  ['Bun.hash', Bun.hash],
+  ['JSON', JSON],
+  ['Math', Math],
+  ['String.prototype', String.prototype],
+  ['Map.prototype', Map.prototype],
+  ['Set.prototype', Set.prototype],
+  ['Promise.prototype', Promise.prototype],
 ]
 
 type BuiltinSnapshot = ReadonlyArray<ReadonlyMap<PropertyKey, PropertyDescriptor>>
@@ -530,7 +540,7 @@ function builtinSnapshot(): BuiltinSnapshot {
 function builtinsChanged(changed: readonly string[], configPath?: string): UserError {
   const who = configPath === undefined ? 'a project config' : configPath
   return new UserError(
-    `${who} changed ${changed.join(', ')} while it was evaluated — a config must not change the built-ins: every other config is read through them, and the cache key does not see what they add`,
+    `${who} changed ${changed.join(', ')} while it was evaluated — a config must not change the built-ins vx runs on: other configs are read through them and cache keys are made with them`,
   )
 }