@vzn/vx 0.0.51 → 0.0.53

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vzn/vx",
-  "version": "0.0.51",
+  "version": "0.0.53",
   "description": "An open, extensible monorepo task runner.",
   "type": "module",
   "exports": {
@@ -17,10 +17,10 @@
     "node": ">=18"
   },
   "optionalDependencies": {
-    "@vzn/vx-linux-x64": "0.0.51",
-    "@vzn/vx-linux-arm64": "0.0.51",
-    "@vzn/vx-darwin-x64": "0.0.51",
-    "@vzn/vx-darwin-arm64": "0.0.51"
+    "@vzn/vx-linux-x64": "0.0.53",
+    "@vzn/vx-linux-arm64": "0.0.53",
+    "@vzn/vx-darwin-x64": "0.0.53",
+    "@vzn/vx-darwin-arm64": "0.0.53"
   },
   "dependencies": {
     "@anthropic-ai/sandbox-runtime": "^0.0.76"

package/src/cli/run.ts

@@ -437,9 +437,10 @@ export async function resolveRunOptions(
   // changed-only `[<base>]` (item 287). `--filter '[<base>]'` stays the
   // "only what I touched" form. Merging it into the filter list means the
   // same code path handles plain filter use, --affected alone, and the combo.
-  // It goes FIRST: filters apply in argv order, and appended last it came
-  // after every `!` exclude, so `--affected --filter '!app'` still ran app
-  // (item 955). An include's place does not change a union.
+  // Its place does not change the selection: `applyFilters` takes every
+  // include before any exclude (item 979), so `--affected --filter '!app'`
+  // drops app from either side (item 955 had put it first for that). First
+  // it stays, so a message naming the filters names it first.
   const filterStrings = [...parsed.filters]
   if (parsed.affected !== undefined) {
     const root = await findWorkspaceRoot(cwd)
@@ -461,6 +462,7 @@ export async function resolveRunOptions(
   const anchoredTasks = tasks.filter((t) => t.includes('#'))
   let projects: string[] | undefined
   let staged: RunOptions['staged']
+  let discovered: RunOptions['discovered']
   let selectedByDiff = false
   if (bareTasks.length === 0) {
     projects = undefined
@@ -485,6 +487,7 @@ export async function resolveRunOptions(
       // The graph walk staged every config; the run reuses those entries
       // instead of evaluating and staging them a second time.
       staged = resolved.staged
+      discovered = resolved.discovered
     }
   } else if (parsed.all) {
     projects = undefined
@@ -519,6 +522,7 @@ export async function resolveRunOptions(
   if (projects !== undefined) opts.projects = projects
   if (selectedByDiff) opts.selectedByDiff = true
   if (staged !== undefined) opts.staged = staged
+  if (discovered !== undefined) opts.discovered = discovered
   if (parsed.retries !== undefined) opts.retries = parsed.retries
   if (parsed.timeout !== undefined) opts.timeout = parsed.timeout
   if (parsed.cacheDir !== undefined) opts.cacheDir = parsed.cacheDir

package/src/cli/select.ts

@@ -152,6 +152,8 @@ export type FilterResolution =
       byDiff: boolean
       /** The staged load the graph walk needed, for the run to reuse (`RunOptions.staged`). */
       staged?: ReadonlyMap<string, ProjectEntry>
+      /** The discovery this pass made, for the run to reuse (`RunOptions.discovered`). */
+      discovered: { root: string; projects: ProjectMeta[] }
     }
   | { error: string }
   | { empty: string }
@@ -317,6 +319,7 @@ export async function resolveFilters(
     names: [...selected].sort(),
     byDiff: parsed.some((f) => !f.negate && f.gitSince !== undefined),
     ...(staged !== undefined ? { staged } : {}),
+    discovered: { root, projects },
   }
 }
 

package/src/cli/watch.ts

@@ -466,9 +466,10 @@ export async function watchCmd(args: readonly string[]): Promise<number> {
   // run()'s process.argv fallback put the bin's absolute path there, so
   // `vx last --list` showed `$ /…/bin.ts watch build` beside `$ vx run build`.
   opts.command = ['vx', 'watch', ...args].join(' ')
-  // A staged load from the selection pass is one run's worth of configs;
-  // every cycle after an edit must evaluate live.
+  // A staged load and a discovery from the selection pass are one run's
+  // worth; every cycle after an edit must load and list live.
   delete opts.staged
+  delete opts.discovered
   process.once('SIGINT', () => {
     process.stdout.write('\nvx watch: stopped\n')
     stop.abort('SIGINT')

package/src/index.ts

@@ -15,9 +15,6 @@ export { VERSION } from './version.js'
 // fractional SQL LIMIT is a datatype mismatch, not a smaller page) should
 // have one implementation.
 export { clampInt, UserError, isUserError } from './util/index.js'
-// "Did you mean": the hint core's own verbs give for a near-miss name, for a
-// plugin verb to give the same one.
-export { nearMatches } from './util/index.js'
 // Every hook a plugin may fill, in pipeline order — the one list the
 // loader, the host and `vx info` read; exported so a doc pin outside core
 // (the site's guides) holds its tables to the same list.
@@ -116,7 +113,6 @@ export type { ProjectEntry } from './workspace/index.js'
 export {
   applyMigration,
   foldScriptHooks,
-  PERSISTENT_TASK_NAMES,
   PERSISTENT_TODO,
   quoteTsLiteral,
 } from './workspace/index.js'
@@ -184,18 +180,15 @@ export type {
 // that added them found: with only the raw `TaskStatus` union exported, every
 // consumer rolls its own Set of status literals — and a Set has no
 // compile-time tripwire when the union gains a member, so it silently answers
-// "no" for the new one. `TASK_STATUSES` is the union at runtime, for a
-// consumer that needs the list rather than the predicate.
+// "no" for the new one.
 // `escapeMarkdownCell` is on the façade for the same demonstrated need: a
 // plugin that renders a run as a markdown table takes the same unvalidated
 // task names core does, and the cloud job summary shipped without the escape.
 export {
   escapeMarkdownCell,
   TELEMETRY_SCHEMA_VERSION,
-  deriveCacheSource,
   isCacheHit,
   isPassStatus,
-  TASK_STATUSES,
 } from './orchestrator/index.js'
 // `TaskLogBuffer` is on the façade on the same demonstrated need: EVERY
 // telemetry sink that ships build output has to bound it, and the retention

package/src/orchestrator/options.ts

@@ -8,7 +8,7 @@ import type { TaskOutcome } from '../graph/index.js'
 import type { EventBus } from './events.js'
 import type { TelemetrySink } from './telemetry.js'
 import type { Logger } from './logger.js'
-import type { ProjectEntry } from '../workspace/index.js'
+import type { ProjectEntry, ProjectMeta } from '../workspace/index.js'
 
 export interface RunOptions {
   cwd: string
@@ -39,6 +39,13 @@ export interface RunOptions {
    * it per cycle).
    */
   staged?: ReadonlyMap<string, ProjectEntry>
+  /**
+   * The projects a selection pass in this same process discovered under
+   * `root` (`--filter`, `--affected`). The run takes them instead of
+   * walking the workspace again when its root is the same. One run only,
+   * as `staged` is.
+   */
+  discovered?: { root: string; projects: ProjectMeta[] }
   concurrency?: number
   /**
    * Cache directory override (`--cache-dir <path>`). Absolute, or relative

package/src/orchestrator/prepare.ts

@@ -179,7 +179,10 @@ export async function prepareRun(options: RunOptions, log: Logger): Promise<Prep
     log.status(m),
   )
   mark('workspace config')
-  const projectMetas = await listProjects(workspace)
+  const projectMetas =
+    options.discovered?.root === workspaceRoot
+      ? options.discovered.projects
+      : await listProjects(workspace)
   mark('discover projects')
 
   // SCOPED config loading: configs are programs, and evaluating 1090
@@ -291,14 +294,20 @@ export async function prepareRun(options: RunOptions, log: Logger): Promise<Prep
 
   const requested = expandRequested(options.tasks, candidateProjects, projects)
   let unresolvedTasks = unresolvedRequests(options.tasks, candidateProjects, projects)
-  if (
-    options.selectedByDiff === true &&
-    projects.size < projectsWithConfigs.length &&
-    unresolvedTasks.some((t) => !t.includes('#'))
-  ) {
-    unresolvedTasks = await declaredNowhere(unresolvedTasks, () =>
-      loadProjects({ ...loadArgs, seeds: 'all', closure: false, staged: projects }),
-    )
+  if (options.selectedByDiff === true && unresolvedTasks.some((t) => !t.includes('#'))) {
+    // A name a loaded project declares is no typo, whether or not the load
+    // was whole: an unaffected dependency loaded for the closure declared
+    // it, and the run said no project did (C-3). Only what is still
+    // unjudged pays for the rest of the workspace.
+    unresolvedTasks = undeclaredIn(unresolvedTasks, projects)
+    if (
+      projects.size < projectsWithConfigs.length &&
+      unresolvedTasks.some((t) => !t.includes('#'))
+    ) {
+      unresolvedTasks = await declaredNowhere(unresolvedTasks, () =>
+        loadProjects({ ...loadArgs, seeds: 'all', closure: false, staged: projects }),
+      )
+    }
   }
 
   // Cache seam precedence: an EXPLICITLY injected remote layer
@@ -527,8 +536,16 @@ async function declaredNowhere(
   } catch {
     return [...unresolved]
   }
+  return undeclaredIn(unresolved, all.projects)
+}
+
+/** The bare names in `unresolved` no project in `projects` declares; `a#b` forms pass through. */
+function undeclaredIn(
+  unresolved: readonly string[],
+  projects: LoadedProjects['projects'],
+): string[] {
   const declared = new Set<string>()
-  for (const p of all.projects.values()) {
+  for (const p of projects.values()) {
     for (const t of Object.keys(p.config.tasks ?? {})) declared.add(t)
   }
   return unresolved.filter((t) => t.includes('#') || !declared.has(t))

package/src/workspace/config-cache.ts

@@ -201,8 +201,6 @@ export const PURE_CORE_EXPORTS: ReadonlySet<string> = new Set([
   'normalizeGlob',
   'isLiteralPattern',
   'PLUGIN_HOOKS',
-  'TASK_STATUSES',
-  'PERSISTENT_TASK_NAMES',
 ])
 
 /** Whether an `import`/`export … from '@vzn/vx'` statement takes only pure values. */