@vzn/vx 0.0.197 → 0.0.199

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vzn/vx",
-  "version": "0.0.197",
+  "version": "0.0.199",
   "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.197",
-    "@vzn/vx-linux-arm64": "0.0.197",
-    "@vzn/vx-darwin-x64": "0.0.197",
-    "@vzn/vx-darwin-arm64": "0.0.197"
+    "@vzn/vx-linux-x64": "0.0.199",
+    "@vzn/vx-linux-arm64": "0.0.199",
+    "@vzn/vx-darwin-x64": "0.0.199",
+    "@vzn/vx-darwin-arm64": "0.0.199"
   },
   "dependencies": {
     "@anthropic-ai/sandbox-runtime": "^0.0.76"

package/src/cli/run.ts

@@ -16,7 +16,7 @@ import {
 import type { ContinueMode } from '../graph/index.js'
 import { type CachePolicy, FULL_CACHE_POLICY, parseCachePolicy } from '../cache/index.js'
 import { findCwdProject, pickTask, resolveFilters } from './select.js'
-import { nxTargetHint } from './task-verb.js'
+import { nxTargetHint, taskNamesHere } from './task-verb.js'
 import { MAX_TIMEOUT_MS, isUserError, parseDecimalInt, machineParallelism } from '../util/index.js'
 import { formatGraphDot, formatPlanJson, formatPlanText } from './plan-format.js'
 
@@ -555,6 +555,9 @@ export async function resolveRunOptions(
   return opts
 }
 
+/** How many task names a run with no task names before `and N more`. */
+const TASKS_SHOWN = 12
+
 export async function runCmd(args: readonly string[]): Promise<number> {
   const parsed = parseRunArgs(args)
   if (parsed.error) {
@@ -570,8 +573,15 @@ export async function runCmd(args: readonly string[]): Promise<number> {
   // explicit anchored positional.
   if (tasks.length === 0) {
     if (!process.stdin.isTTY) {
+      const names = await taskNamesHere(cwd)
+      const shown = (names ?? []).slice(0, TASKS_SHOWN)
+      const more = (names?.length ?? 0) - shown.length
+      const tasksHere =
+        shown.length === 0
+          ? 'vx run <task>, e.g. vx run build'
+          : `tasks here: ${shown.join(', ')}${more > 0 ? ` and ${more} more` : ''}`
       process.stderr.write(
-        `vx run: missing task name (stdin is not a TTY, so no picker; vx run <task>, e.g. vx run build)${seeHelp('run')}\n`,
+        `vx run: missing task name (stdin is not a TTY, so no picker; ${tasksHere})${seeHelp('run')}\n`,
       )
       return 1
     }

package/src/cli/task-verb.ts

@@ -43,6 +43,22 @@ export async function nxTargetHint(tasks: readonly string[], cwd: string): Promi
   return `\`${tasks.join(' ')}\` is Nx's project:target: vx run ${specs.join(' ')}`
 }
 
+/**
+ * The task names `vx run <task>` can take here: the cwd project's, else
+ * every project's, sorted; null when no workspace loads. What a run with
+ * no task and no terminal for the picker names instead of a guess.
+ */
+export async function taskNamesHere(cwd: string): Promise<string[] | null> {
+  const projects = await workspaceProjects(cwd)
+  if (projects === null) return null
+  const own = await findCwdProject(cwd)
+  const names = new Set<string>()
+  for (const p of projects.values())
+    if (own === null || p.name === own)
+      for (const t of Object.keys(p.config.tasks ?? {})) names.add(t)
+  return [...names].sort()
+}
+
 async function workspaceProjects(
   cwd: string,
 ): Promise<Awaited<ReturnType<typeof loadCliProjects>> | null> {

package/src/workspace/project-loader.ts

@@ -387,6 +387,7 @@ export async function loadProjectConfigs(
     closure?: readonly [string, readonly string[]]
   }
   const builtins = builtinSnapshot()
+  const env = { ...process.env }
   const loadOne = async (entry: (typeof prepared)[number]): Promise<Loaded> => {
     const { configPath, cacheKey } = entry
     // A fast key that missed: the closure is stale or the file changed.
@@ -427,7 +428,7 @@ export async function loadProjectConfigs(
       : await loadDefaultExport(configPath, 'Project', bytes!)
     // Before anything reads through them: a replaced `Array.prototype.includes`
     // turned the JSON-data walk's own check into "a cyclic reference".
-    const changed = repeat ? [] : restoreBuiltins(builtins)
+    const changed = repeat ? [] : [...restoreBuiltins(builtins), ...restoreEnv(env)]
     if (changed.length > 0) throw builtinsChanged(changed, configPath)
     assertDefaultObject(mod, 'Project', configPath)
     // Validation runs HERE, on whichever object we ended up with, so a
@@ -485,7 +486,7 @@ export async function loadProjectConfigs(
         if (r.closure !== undefined) learnedClosures.push(r.closure)
       }
     }
-    const changed = restoreBuiltins(builtins)
+    const changed = [...restoreBuiltins(builtins), ...restoreEnv(env)]
     if (first !== undefined) throw first.failed
     if (changed.length > 0) throw builtinsChanged(changed)
     return results.map((r) => (r as Loaded).config)
@@ -539,8 +540,11 @@ function builtinSnapshot(): BuiltinSnapshot {
 /** Loads run together, so the config named is the one whose load saw the change. */
 function builtinsChanged(changed: readonly string[], configPath?: string): UserError {
   const who = configPath === undefined ? 'a project config' : configPath
+  const env = changed.some((c) => c.startsWith('process.env.'))
+    ? '; a task gets an env var through `exec.env.define` or `passThrough`'
+    : ''
   return new UserError(
-    `${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`,
+    `${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${env}`,
   )
 }
 
@@ -567,6 +571,31 @@ function restoreBuiltins(before: BuiltinSnapshot): string[] {
   return changed
 }
 
+/**
+ * A first load's `process.env.X = …` gave every other project's task that
+ * value through `passThrough` and reached vx's own `VX_*` reads, while a
+ * repeat load, in a worker, left both unchanged: one config, two
+ * environments, by load order (D-76). Assigned back, not defined:
+ * `process.env` refuses `defineProperty`.
+ */
+function restoreEnv(before: Readonly<Record<string, string | undefined>>): string[] {
+  const changed: string[] = []
+  const live = process.env
+  for (const key of Object.keys(live)) {
+    if (live[key] === before[key]) continue
+    changed.push(`process.env.${key}`)
+    if (key in before) live[key] = before[key]
+    else delete live[key]
+  }
+  for (const key of Object.keys(before)) {
+    if (!(key in live)) {
+      changed.push(`process.env.${key}`)
+      live[key] = before[key]
+    }
+  }
+  return changed
+}
+
 function sameDescriptor(a: PropertyDescriptor, b: PropertyDescriptor): boolean {
   return (
     a.value === b.value &&