@vzn/vx 0.0.487 → 0.0.489

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vzn/vx",
-  "version": "0.0.487",
+  "version": "0.0.489",
   "description": "The fastest task runner and build cache for JavaScript monorepos.",
   "type": "module",
   "exports": {
@@ -21,10 +21,10 @@
     "node": ">=18"
   },
   "optionalDependencies": {
-    "@vzn/vx-linux-x64": "0.0.487",
-    "@vzn/vx-linux-arm64": "0.0.487",
-    "@vzn/vx-darwin-x64": "0.0.487",
-    "@vzn/vx-darwin-arm64": "0.0.487"
+    "@vzn/vx-linux-x64": "0.0.489",
+    "@vzn/vx-linux-arm64": "0.0.489",
+    "@vzn/vx-darwin-x64": "0.0.489",
+    "@vzn/vx-darwin-arm64": "0.0.489"
   },
   "dependencies": {
     "@anthropic-ai/sandbox-runtime": "^0.0.76"

package/src/exec/env.ts

@@ -74,6 +74,11 @@ export function buildIsolatedEnv(opts: BuildEnvOptions): NodeJS.ProcessEnv {
   for (const [name, value] of Object.entries(opts.define)) {
     out[name] = value
   }
+  // A task's output is a pipe, so a tool asks no TTY and prints plain. vx
+  // forces colour, as Nx and Turbo do, and strips it where its own output
+  // is plain (`plainOutput`), so the cache holds one form for every
+  // machine. Any FORCE_COLOR the task already sees, or a NO_COLOR, wins.
+  if (out['FORCE_COLOR'] === undefined && !out['NO_COLOR']) out['FORCE_COLOR'] = '1'
 
   // A directory holding the delimiter cannot be named in PATH: split, it
   // became two entries naming nothing, the second RELATIVE, so resolved

package/src/orchestrator/plain-output.ts

@@ -0,0 +1,57 @@
+import type { TaskNode } from '../graph/index.js'
+import type { Logger } from './logger.js'
+
+// A prefix that may yet complete as a CSI or OSC sequence. An escape cut
+// at a chunk's end is held for the next chunk: stripped alone, its
+// remainder (`1m`) reached the log as text.
+// eslint-disable-next-line no-control-regex -- ESC is the point
+const PARTIAL_ESCAPE = /^\x1b(?:\[[0-?]*[ -/]*|\][^\x07\x1b]*\x1b?)?$/
+
+/**
+ * `sink` with terminal escapes stripped from task output. Tasks run with
+ * `FORCE_COLOR` (exec/env.ts), so their bytes carry colour wherever vx's
+ * own output is plain: a pipe, CI, `NO_COLOR`.
+ */
+export function plainOutput(sink: Logger): Logger {
+  const held = new Map<string, string>()
+  const strip = (key: string, chunk: string): string => {
+    const prior = held.get(key)
+    const text = prior === undefined ? chunk : prior + chunk
+    if (prior !== undefined) held.delete(key)
+    const esc = text.lastIndexOf('\x1b')
+    if (esc === -1) return text
+    if (PARTIAL_ESCAPE.test(text.slice(esc))) {
+      held.set(key, text.slice(esc))
+      return Bun.stripANSI(text.slice(0, esc))
+    }
+    return Bun.stripANSI(text)
+  }
+  const flush = (node: TaskNode, key: string, write: (n: TaskNode, c: string) => void): void => {
+    const rest = held.get(key)
+    if (rest === undefined) return
+    held.delete(key)
+    const text = Bun.stripANSI(rest)
+    if (text !== '') write(node, text)
+  }
+  // Delegated one by one, never spread: an embedder's logger may be a class
+  // whose methods live on its prototype and read `this`.
+  return {
+    status: (line) => sink.status(line),
+    ...(sink.runStart && { runStart: (info) => sink.runStart!(info) }),
+    ...(sink.taskStart && { taskStart: (node) => sink.taskStart!(node) }),
+    ...(sink.runEnd && { runEnd: () => sink.runEnd!() }),
+    taskStdout(node, chunk) {
+      const text = strip(`${node.id}\0out`, chunk)
+      if (text !== '' || chunk === '') sink.taskStdout(node, text)
+    },
+    taskStderr(node, chunk) {
+      const text = strip(`${node.id}\0err`, chunk)
+      if (text !== '' || chunk === '') sink.taskStderr(node, text)
+    },
+    taskComplete(node, outcome) {
+      flush(node, `${node.id}\0out`, (n, c) => sink.taskStdout(n, c))
+      flush(node, `${node.id}\0err`, (n, c) => sink.taskStderr(n, c))
+      sink.taskComplete(node, outcome)
+    },
+  }
+}

package/src/orchestrator/run.ts

@@ -59,6 +59,7 @@ import { assembleRunSummary, isPassStatus } from './telemetry.js'
 import type { RunContextRecord } from './telemetry.js'
 import { defaultLogger, resolveOutputView, type Logger } from './logger.js'
 import { detectColors, type ColorSupport } from './colors.js'
+import { plainOutput } from './plain-output.js'
 import { formatPersistentList } from './framed-output.js'
 import { LocalHistoryProvider } from './history.js'
 import { plan, type RunPlan } from './plan.js'
@@ -307,7 +308,12 @@ export async function run(options: RunOptions): Promise<RunSummary> {
   // outlives the run, and a renderer left behind reported the next run on
   // it twice (item 635). Otherwise a fresh internal bus.
   const bus = options.bus ?? createEventBus()
-  const unsubscribeTerminal = bus.subscribe(terminalSubscriber(sink))
+  // Tasks run with colour forced (exec/env.ts); where the terminal prints
+  // plain, their escapes are stripped. A custom logger gets the task's
+  // bytes as written, as the cache holds them: it formats its own output.
+  const unsubscribeTerminal = bus.subscribe(
+    terminalSubscriber(terminal === null || colors.enabled ? sink : plainOutput(sink)),
+  )
   const detach = (): void => {
     unsubscribeTerminal()
     terminal?.settle()

package/src/workspace/project-loader.ts

@@ -733,6 +733,7 @@ const BUN_MEMBERS_VX_READS: readonly PropertyKey[] = [
   'spawnSync',
   'stdout',
   'stringWidth',
+  'stripANSI',
   'version',
   'which',
   'write',