@vzn/vx 0.0.178 → 0.0.180

package/package.json

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

package/src/workspace/config-eval.ts

@@ -39,9 +39,38 @@ import path from 'node:path'
 import { MAX_TIMEOUT_MS, UserError } from '../util/index.js'
 import { nonJsonMessage, nonJsonPaths, type NonJsonValue } from './json-data.js'
 
+/** Why a config's `process.exit` throws, on both load paths (D-65). */
+export const CONFIG_EXIT = 'a config exports its object; it cannot end the run'
+
 const WORKER_SRC = `
 const nonJsonPaths = ${nonJsonPaths.toString()}
+// The parent's stdout is a verb's JSON or vx mcp's JSON-RPC stream, and a
+// config printing on a repeat load wrote into it: the server's own redirect
+// covers the parent thread only (D-64). This worker only evaluates configs,
+// so every route to fd 1 is stderr's, Bun's own included (item 1069).
+const quiet = (async () => {
+  const P = globalThis.process
+  const B = globalThis.Bun
+  const { Console } = await import('node:console')
+  globalThis.console = Object.assign(new Console(P.stderr, P.stderr), {
+    write: (...data) => {
+      const text = data.join('')
+      P.stderr.write(text)
+      return text.length
+    },
+  })
+  P.stdout.write = (...args) => P.stderr.write(...args)
+  const ownBunWrite = B.write
+  B.write = (dest, ...rest) => ownBunWrite(dest === B.stdout ? B.stderr : dest, ...rest)
+  B.stdout.writer = (...args) => B.stderr.writer(...args)
+  // An exit here ended the worker unheard, and the load waited out its
+  // whole deadline to say the worker did not answer (D-65).
+  P.exit = (code) => {
+    throw new Error('process.exit(' + (code ?? '') + ') in a config: ' + ${JSON.stringify(CONFIG_EXIT)})
+  }
+})()
 self.onmessage = async (e) => {
+  await quiet
   const { id, path, env } = e.data
   // A Worker starts with the process's STARTUP environment, not the
   // parent's process.env as written since (probed): a config reading an

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, evaluateConfigFresh } from './config-eval.js'
+import { beginEvalRound, CONFIG_EXIT, evaluateConfigFresh } from './config-eval.js'
 import { hasEsmExport, unprovidedBareImports } from './config-imports.js'
 import {
   configEvalKey,
@@ -136,6 +136,7 @@ async function loadDefaultExport(
     holdSource(hash, source)
   }
   let ns: { default?: unknown }
+  const unguard = guardExit()
   try {
     ns = (await import(specifier)) as { default?: unknown }
   } catch (err) {
@@ -145,6 +146,7 @@ async function loadDefaultExport(
     }
     throw configLoadError(err, configPath, kind) ?? err
   } finally {
+    unguard()
     if (source !== null) releaseSource(hash)
   }
   const mod = ns?.default
@@ -152,6 +154,28 @@ async function loadDefaultExport(
   return mod
 }
 
+/**
+ * A config's `process.exit` ended vx mid-load: `exit(0)` was a green run
+ * that ran nothing and printed nothing (D-65). While any config evaluates
+ * in process it throws instead, at the config's line. vx's own exit is
+ * signal forwarding's, which a run installs after its load and removes
+ * when it ends.
+ */
+let evaluating = 0
+let ownExit: typeof process.exit = process.exit
+function guardExit(): () => void {
+  if (evaluating++ === 0) {
+    ownExit = process.exit
+    process.exit = configExit
+  }
+  return () => {
+    if (--evaluating === 0) process.exit = ownExit
+  }
+}
+const configExit = ((code?: number | string | null) => {
+  throw new Error(`process.exit(${code ?? ''}) in a config: ${CONFIG_EXIT}`)
+}) as typeof process.exit
+
 /**
  * A bare import nothing above the config provides is refused BEFORE the
  * evaluation: left to Bun, a workspace with no `node_modules` would have