@vzn/vx 0.0.547 → 0.0.549

package/package.json

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

package/src/orchestrator/execute-task.ts

@@ -507,11 +507,13 @@ async function executePersistentTask(args: ExecuteArgs): Promise<TaskOutcome> {
     }
     // The reason rides the outcome (every label reads it), and a child that
     // exited before ready keeps its own exit code rather than a made-up 1.
+    // One the readiness timeout is killing reports the signal's, as an
+    // ordinary timeout does (X-24).
     const ready = err instanceof PersistentReadyError ? err : undefined
     return {
       node,
       status: 'failed',
-      exitCode: ready?.exitCode ?? 1,
+      exitCode: ready?.reason === 'timeout' ? await spawn.child.exited : (ready?.exitCode ?? 1),
       durationMs: spawn.readyMs(),
       ...(ready !== undefined ? { notReady: ready.reason } : {}),
       wallclockStartNs,

package/src/orchestrator/run-report.ts

@@ -10,6 +10,7 @@
 // too. `--report-file` is the sink that carries the property end to end.
 
 import { failedLabel, skippedLabel, type OutcomeView } from './events.js'
+import { neverStarted } from './summary.js'
 
 /**
  * One finished run, reduced to what a report needs. Used to live in
@@ -35,8 +36,10 @@ import { tallyViews, type Tally } from './tally.js'
  * a row claiming `success | miss | 0ms`.
  */
 interface ReportTally extends Tally {
-  /** Wall-clock ms summed over tasks that actually executed (misses). */
+  /** Wall-clock ms summed over tasks that ran a command, an aborted one's time so far included. */
   executedMs: number
+  /** The aborted tasks the stop reached before they ran, as the terminal splits them. */
+  notRun: number
   /**
    * Ms of work the cache SKIPPED — the sum of the hits' STORED exec times.
    * Deliberately not `durationMs`, which for a hit is the restore this run
@@ -46,7 +49,7 @@ interface ReportTally extends Tally {
 }
 
 function tally(outcomes: readonly OutcomeView[]): ReportTally {
-  const t: ReportTally = { ...tallyViews(outcomes), executedMs: 0, savedMs: 0 }
+  const t: ReportTally = { ...tallyViews(outcomes), executedMs: 0, savedMs: 0, notRun: 0 }
   for (const o of outcomes) {
     if (o.isGroup === true) continue
     switch (o.status) {
@@ -54,6 +57,10 @@ function tally(outcomes: readonly OutcomeView[]): ReportTally {
       case 'failed':
         t.executedMs += o.durationMs
         break
+      case 'aborted':
+        if (neverStarted(o)) t.notRun++
+        else t.executedMs += o.durationMs
+        break
       case 'cache-hit':
       case 'cache-hit-remote':
         // An outcome that doesn't know what it skipped contributes nothing:
@@ -78,6 +85,8 @@ function statusWord(o: OutcomeView): string {
       return failedLabel(o.exitCode, o.timedOut, o.sandboxViolations, o.notReady)
     case 'skipped':
       return skippedLabel(o.blockedBy)
+    case 'aborted':
+      return neverStarted(o) ? 'not run' : 'aborted'
     default:
       return o.status
   }
@@ -145,12 +154,21 @@ export function formatRunReportMarkdown(result: RunResult): string {
       : '0 cached',
   ]
   if (t.skipped > 0) parts.push(`${t.skipped} skipped`)
-  if (t.aborted > 0) parts.push(`${t.aborted} aborted`)
   parts.push(`${fmtDuration(t.executedMs)} total`)
   if (t.savedMs > 0) parts.push(`${fmtDuration(t.savedMs)} saved`)
+  // The terminal summary's words: neither is in `total`, and a task the stop
+  // reached before it ran was never aborted.
+  const aborted = t.aborted - t.notRun
+  const uncounted: string[] = []
+  if (aborted > 0) uncounted.push(`${aborted} aborted`)
+  if (t.notRun > 0) uncounted.push(`${t.notRun} not run`)
+  if (uncounted.length > 0) parts.push(`not counted: ${uncounted.join(', ')}`)
 
+  // Only a stop (a signal, `RunOptions.signal`) makes an `aborted` outcome,
+  // so one is how an interrupted run tells itself from a failed one.
+  const heading = t.aborted > 0 ? 'interrupted' : result.ok ? 'passed' : 'failed'
   const lines: string[] = []
-  lines.push(`## vx run — ${result.ok ? 'passed' : 'failed'}`)
+  lines.push(`## vx run — ${heading}`)
   lines.push('')
   lines.push(parts.join(' · '))
   lines.push('')
@@ -163,7 +181,7 @@ export function formatRunReportMarkdown(result: RunResult): string {
     if (o.isGroup === true) continue
     lines.push(
       `| ${cell(o.taskId)} | ${cell(statusWord(o))} | ${cell(cacheWord(o))} | ${cell(
-        fmtDuration(o.durationMs),
+        neverStarted(o) ? '—' : fmtDuration(o.durationMs),
       )} |`,
     )
   }

package/src/orchestrator/summary.ts

@@ -381,7 +381,9 @@ export function formatRunSummary(
  * it an invented exit 1; only a started task carries a wall-clock start, so
  * that is what tells "never ran" from "killed" (item 1062).
  */
-export function neverStarted(o: TaskOutcome): boolean {
+export function neverStarted(
+  o: Pick<TaskOutcome, 'status'> & { wallclockStartNs?: unknown },
+): boolean {
   return o.status === 'aborted' && o.wallclockStartNs === undefined
 }