@vzn/vx 0.0.39 → 0.0.41

package/package.json

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

package/src/cli/help.ts

@@ -49,7 +49,7 @@ export function helpText(pluginCommands: readonly string[] = []): string {
     '  vx why [TASK | PKG#TASK] [--run <runId>] [--format pretty|json] [--cache-dir <path>]',
     '  vx last [RUNID] [--list[=N]] [--format pretty|json] [--cache-dir <path>]',
     '  vx completions bash|zsh|fish',
-    '  vx help',
+    '  vx help [VERB]',
     '  vx version',
     '',
     'Selection (for run):',

package/src/cli/index.ts

@@ -32,9 +32,17 @@ export async function run(argv: readonly string[]): Promise<number> {
     case undefined:
     case '--help':
     case '-h':
-    case 'help':
       printHelp(await pluginCommandHelp())
       return 0
+    case 'help': {
+      // `vx help run` is the same question as `vx run --help`. A plugin verb
+      // owns its own help, so anything but a core verb gets the whole
+      // reference, which lists the plugin verbs.
+      const verb = rest[0]
+      const core = verb !== undefined && (CORE_VERBS as readonly string[]).includes(verb)
+      printHelp(await pluginCommandHelp(), core ? verb : undefined)
+      return 0
+    }
     case '--version':
     case 'version':
       process.stdout.write(`vx ${VERSION}\n`)

package/src/cli/workspace-config.ts

@@ -127,6 +127,7 @@ export async function loadCliProjects(
       lock,
       evalCache: {
         store: cache,
+        workspaceRoot,
         workspaceFingerprint: await computeWorkspaceFingerprint(workspaceRoot),
       },
       warn: warnToStderr,

package/src/orchestrator/doctor.ts

@@ -148,6 +148,7 @@ export async function collectInfo(cwd: string, opts: CollectInfoOptions = {}): P
         lock: null,
         evalCache: {
           store: cache,
+          workspaceRoot: root,
           workspaceFingerprint: await computeWorkspaceFingerprint(root, reads),
         },
         warn,

package/src/orchestrator/prepare.ts

@@ -258,7 +258,7 @@ export async function prepareRun(options: RunOptions, log: Logger): Promise<Prep
     projectMetas,
     packageGraph,
     lock,
-    evalCache: { store: localCache, workspaceFingerprint: fingerprints.all },
+    evalCache: { store: localCache, workspaceRoot, workspaceFingerprint: fingerprints.all },
     warn: (m: string) => log.status(m),
   }
   let loaded: LoadedProjects

package/src/orchestrator/projects.ts

@@ -259,6 +259,7 @@ export async function loadResolvedProjects(
       lock: null,
       evalCache: {
         store: cache,
+        workspaceRoot,
         workspaceFingerprint: await computeWorkspaceFingerprint(workspaceRoot, reads),
       },
       warn,

package/src/workspace/config-cache.ts

@@ -109,6 +109,13 @@ export interface ConfigEvalKeyArgs {
    */
   hashFile?: (file: string) => Promise<string>
   bytes: Uint8Array
+  /**
+   * The workspace root, as the config path spells it. A symlink BELOW it on
+   * the way to the config makes the closure unindexable; one above it
+   * (macOS's `/var`) moves nothing the warm path could miss. Absent, any
+   * symlink on the way does.
+   */
+  workspaceRoot?: string | undefined
   workspaceFingerprint: string
 }
 
@@ -132,9 +139,10 @@ const MAX_CLOSURE_FILES = 32
 // literal naming it was stripped before the test ran; listed 2026-09-27
 // (item 957). A key assembled at run time (`'constru' + 'ctor'`) still
 // passes: the gate stops accidental impurity, not a config written to
-// defeat it.
+// defeat it. `random` is denied as a word, not as `Math.random`: a
+// destructured `const { random } = Math` was cached as pure (item 1036).
 const IMPURE_RE =
-  /\b(?:process|Bun|globalThis|global|self|fetch|Date|Temporal|Intl|crypto|performance|navigator|require|eval|Function|constructor|localeCompare|await|toLocale\w*|Reflect|getPrototypeOf|setPrototypeOf|getOwnPropertyNames|getOwnPropertyDescriptor|getOwnPropertyDescriptors|__proto__|prototype|__defineGetter__|__defineSetter__|__lookupGetter__|__lookupSetter__)\b|import\s*\.\s*meta|Math\s*\.\s*random|\bimport\s*\(/
+  /\b(?:process|Bun|globalThis|global|self|fetch|Date|Temporal|Intl|crypto|performance|navigator|require|eval|Function|constructor|localeCompare|await|toLocale\w*|Reflect|getPrototypeOf|setPrototypeOf|getOwnPropertyNames|getOwnPropertyDescriptor|getOwnPropertyDescriptors|__proto__|prototype|__defineGetter__|__defineSetter__|__lookupGetter__|__lookupSetter__|random)\b|import\s*\.\s*meta|\bimport\s*\(/
 
 /** Literal text that names a way to `Function` when used as a computed key. */
 const IMPURE_LITERAL_RE = /constructor|__proto__|prototype/
@@ -143,10 +151,15 @@ const IMPURE_LITERAL_RE = /constructor|__proto__|prototype/
 // matched on `stripLiterals` output with its strings kept as placeholders
 // (`\0<n>\0`): a comment is gone, so an import after `/* … */` on its line
 // is seen, and a string-named binding (`import { 'a-b' as x }`) is a
-// placeholder like any other. `[^;]*?` spans newlines, so multi-line
-// specifier lists match.
+// placeholder like any other. The body spans newlines, so multi-line
+// specifier lists match, but never another statement's `import` or
+// `export`: in a file without semicolons a lazy `[^;]*?` ran from
+// `export type Mode = …` on to the next line's `from '@vzn/vx'`, and the
+// purity check read `type` and passed an impure import. Nor is a line
+// start required: `} export * from './side.ts'` went unscanned, and the
+// fail-closed count below counted only imports (item 1036).
 const IMPORT_RE =
-  /(?:^|[\n;])\s*(?:import|export)\b[^;]*?\bfrom\s*\0(\d+)\0|(?:^|[\n;])\s*import\s*\0(\d+)\0/g
+  /\b(?:import|export)\b(?:(?!\b(?:import|export)\b)[^;])*?\bfrom\s*\0(\d+)\0|\bimport\s*\0(\d+)\0/g
 
 /**
  * Every static import in `source` as its specifier and statement, or `null`
@@ -384,6 +397,31 @@ function realDirOf(file: string): string {
   return realpathSync(path.dirname(file))
 }
 
+/** Each workspace root's real path, asked once per process. */
+const realRoots = new Map<string, string>()
+
+/**
+ * Whether `realDir`, the real directory of `file`, is where its spelling
+ * says it is once the root's own links are resolved. A member directory
+ * linked in from elsewhere is not: the warm path keys the closure by the
+ * paths the last evaluation found, and a link retargeted to a config of
+ * the same bytes whose preset differs moves none of them (item 1036).
+ */
+function reachedThroughNoLink(file: string, realDir: string, root: string | undefined): boolean {
+  const dir = path.dirname(file)
+  // Without a root there is nothing to tell a member's own link from one
+  // above the tree (macOS's `/tmp` → `/private/tmp`), and comparing the
+  // spelling with the real path refused every config under such a link.
+  // Every run passes its root; a caller that does not keeps the index.
+  if (root === undefined) return true
+  let realRoot = realRoots.get(root)
+  if (realRoot === undefined) {
+    realRoot = realpathSync(root)
+    realRoots.set(root, realRoot)
+  }
+  return path.relative(realRoot, realDir) === path.relative(root, dir)
+}
+
 /** A regular file reached through no symlink: the path IS the file. */
 function isCanonicalFile(file: string): boolean {
   try {
@@ -439,10 +477,15 @@ export async function configEvalKey(a: ConfigEvalKeyArgs): Promise<ConfigEvalKey
       // (item 950). The named file is taken without `Bun.resolveSync`,
       // whose directory cache answers a retargeted link with its old
       // target for the rest of the process.
-      try {
-        dir ??= realDirOf(file)
-      } catch {
-        return null
+      if (dir === undefined) {
+        try {
+          dir = realDirOf(file)
+          if (file === a.configPath && !reachedThroughNoLink(file, dir, a.workspaceRoot)) {
+            indexable = false
+          }
+        } catch {
+          return null
+        }
       }
       const named = path.resolve(dir, spec)
       let resolved: string

package/src/workspace/project-loader.ts

@@ -233,7 +233,7 @@ export interface LoadProjectConfigOptions {
    * Serve a provably-pure config from its cached evaluation (see
    * config-cache.ts) and store a fresh one for next time. Off for `fresh`.
    */
-  evalCache?: { store: ConfigEvalStore; workspaceFingerprint: string }
+  evalCache?: { store: ConfigEvalStore; workspaceRoot?: string; workspaceFingerprint: string }
 }
 
 /**
@@ -296,6 +296,7 @@ export async function loadProjectConfigs(
           : await configEvalKey({
               configPath,
               bytes,
+              workspaceRoot: evalCache.workspaceRoot,
               workspaceFingerprint: evalCache.workspaceFingerprint,
               ...slowKeyHash,
             })
@@ -352,6 +353,7 @@ export async function loadProjectConfigs(
         const keyed = await configEvalKey({
           configPath,
           bytes,
+          workspaceRoot: evalCache!.workspaceRoot,
           workspaceFingerprint: evalCache!.workspaceFingerprint,
           ...slowKeyHash,
         })