@vzn/vx 0.0.348 → 0.0.349

package/package.json

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

package/src/orchestrator/execute-task.ts

@@ -37,7 +37,7 @@ import {
 import { isGroupTask, RestoreDemoted, type TaskNode, type TaskOutcome } from '../graph/index.js'
 import { killGraceMs, maskedEmitter, printable, relPosix, secretMask, span } from '../util/index.js'
 import { SIGNAL_SHUTDOWN_GRACE_MS } from './signals.js'
-import { executorLabel } from './plugin-host.js'
+import { executorLabel, nameExecutorFailure } from './plugin-host.js'
 import {
   mayWriteFingerprint,
   type Placeholder,
@@ -791,7 +791,8 @@ async function executeCachedTask(args: ExecuteArgs): Promise<TaskOutcome> {
         assertExecuteResult(args.executor.name, node.id, r)
         return r
       })
-      .catch(async (err: unknown) => {
+      .catch(async (raw: unknown) => {
+        const err = nameExecutorFailure(args.executor, raw)
         const message = err instanceof Error ? err.message : String(err)
         log.taskStderr(node, `${message}\n`)
         await sweepPlaceholders(placeholders)

package/src/orchestrator/plugin-host.ts

@@ -439,6 +439,23 @@ export async function resolveCache(
 /** The plugin each resolved executor came from, so a hook of it that throws is named. */
 const executorPlugin = new WeakMap<TaskExecutor, string>()
 
+/**
+ * A plugin executor's throw from `execute`, named as every other hook's
+ * is: `plugin 'p' (executor 'e') failed in execute: …`. It read `internal
+ * error in <task>: pool down`, naming neither (C-63). The error itself is
+ * kept, its class, cause and code with it, so the scheduler still tells a
+ * refusal from a bug; the floor's own throw is vx's and stays as it is.
+ */
+export function nameExecutorFailure(executor: TaskExecutor, err: unknown): unknown {
+  if (!(err instanceof Error) || !executorPlugin.has(executor)) return err
+  try {
+    err.message = `${executorLabel(executor)} failed in execute: ${err.message}`
+  } catch {
+    // A frozen error keeps its own words.
+  }
+  return err
+}
+
 /** How a message names `executor`: its plugin and its own name, or its name alone (the floor). */
 export function executorLabel(executor: TaskExecutor): string {
   const plugin = executorPlugin.get(executor)