@vzn/vx 0.0.180 → 0.0.182

package/package.json

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

package/src/workspace/config-eval.ts

@@ -132,12 +132,12 @@ interface Pending {
 }
 
 /**
- * How long a single config evaluation may take before we treat the worker as
- * wedged. Real evaluations are ~10 ms; this exists only so a killed or hung
- * worker cannot stall a long-lived process indefinitely. Read per call so a
- * test can drive the deadline instead of waiting it out.
+ * How long a single config evaluation may take, in the worker or in process
+ * (D-66). Real evaluations are ~10 ms; this exists only so a hung config or a
+ * killed worker cannot stall a run or a long-lived process indefinitely. Read
+ * per call so a test can drive the deadline instead of waiting it out.
  */
-function workerTimeoutMs(): number {
+export function evalBudgetMs(): number {
   const raw = process.env['VX_CONFIG_WORKER_TIMEOUT_MS']
   // Same kind of value as the teardown deadline, so the same treatment: a
   // BOUND on a worker that may be wedged, with no "no limit" reading. Falls
@@ -266,7 +266,7 @@ export async function evaluateConfigFresh(configPath: string): Promise<unknown>
     // otherwise take milliseconds. Nothing else bounds it: there is no
     // run-level timeout. The budget is enormous next to the ~10 ms a real
     // evaluation costs, so it can only fire on a genuine wedge.
-    const budget = workerTimeoutMs()
+    const budget = evalBudgetMs()
     const reply = await new Promise<WorkerReply>((resolve, reject) => {
       pending.set(id, { resolve, reject })
       timer = setTimeout(() => {

package/src/workspace/json-data.ts

@@ -74,7 +74,14 @@ export function nonJsonPaths(value: unknown): NonJsonValue[] {
       ancestors.push(v)
       for (const key of Object.keys(v)) {
         keys.push(key)
-        walk((v as Record<string, unknown>)[key], false)
+        // Read by descriptor: a getter is code, and each read may answer
+        // differently. `vx show` read one three times, so the key, the
+        // check and the command could each see another value, and a
+        // throwing one surfaced as vx's own stack (D-67). Never called here.
+        const d = Object.getOwnPropertyDescriptor(v, key)
+        if (d !== undefined && d.get !== undefined) found('a getter')
+        else if (d !== undefined && d.set !== undefined) found('a setter')
+        else walk(d?.value, false)
         keys.pop()
       }
       ancestors.pop()

package/src/workspace/project-loader.ts

@@ -3,7 +3,7 @@ import path from 'node:path'
 import type { ProjectConfig, WorkspaceConfig } from '../config.js'
 import { UserError, xxh3hex } from '../util/index.js'
 import { validateProjectConfig, validateWorkspace } from './config-schema.js'
-import { beginEvalRound, CONFIG_EXIT, evaluateConfigFresh } from './config-eval.js'
+import { beginEvalRound, CONFIG_EXIT, evalBudgetMs, evaluateConfigFresh } from './config-eval.js'
 import { hasEsmExport, unprovidedBareImports } from './config-imports.js'
 import {
   configEvalKey,
@@ -137,8 +137,26 @@ async function loadDefaultExport(
   }
   let ns: { default?: unknown }
   const unguard = guardExit()
+  let deadline: ReturnType<typeof setTimeout> | undefined
   try {
-    ns = (await import(specifier)) as { default?: unknown }
+    // The worker's budget, here too: a top-level await that never settled
+    // while a timer kept the loop alive hung `vx run` for good, silently
+    // (D-66).
+    const budget = evalBudgetMs()
+    ns = (await Promise.race([
+      import(specifier),
+      new Promise<never>((_, reject) => {
+        deadline = setTimeout(
+          () =>
+            reject(
+              new UserError(
+                `${kind} config ${configPath} did not finish evaluating within ${budget}ms (VX_CONFIG_WORKER_TIMEOUT_MS)`,
+              ),
+            ),
+          budget,
+        )
+      }),
+    ])) as { default?: unknown }
   } catch (err) {
     // A served module's frames name its specifier; the user wrote the path.
     if (err instanceof Error && err.stack !== undefined) {
@@ -146,6 +164,7 @@ async function loadDefaultExport(
     }
     throw configLoadError(err, configPath, kind) ?? err
   } finally {
+    clearTimeout(deadline)
     unguard()
     if (source !== null) releaseSource(hash)
   }