@vzn/vx 0.0.61 → 0.0.63

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vzn/vx",
-  "version": "0.0.61",
+  "version": "0.0.63",
   "description": "An open, extensible monorepo task runner.",
   "type": "module",
   "exports": {
@@ -17,10 +17,10 @@
     "node": ">=18"
   },
   "optionalDependencies": {
-    "@vzn/vx-linux-x64": "0.0.61",
-    "@vzn/vx-linux-arm64": "0.0.61",
-    "@vzn/vx-darwin-x64": "0.0.61",
-    "@vzn/vx-darwin-arm64": "0.0.61"
+    "@vzn/vx-linux-x64": "0.0.63",
+    "@vzn/vx-linux-arm64": "0.0.63",
+    "@vzn/vx-darwin-x64": "0.0.63",
+    "@vzn/vx-darwin-arm64": "0.0.63"
   },
   "dependencies": {
     "@anthropic-ai/sandbox-runtime": "^0.0.76"

package/src/cache/inputs.ts

@@ -25,6 +25,7 @@ import {
   isExecutableMissing,
   isLiteralPattern,
   normalizeGlob,
+  slashBraceExpansions,
   taskGlob,
   UserError,
 } from '../util/index.js'
@@ -1062,7 +1063,10 @@ async function scanUnion(
   cwd: string,
 ): Promise<Set<string>> {
   const matches = new Set<string>()
-  for (const pattern of positive) {
+  // `Bun.Glob`'s scan finds nothing for a brace whose alternatives hold a
+  // `/`: `{dist,lib/esm}/**` saved an empty artifact, and a hit restored
+  // nothing over a cleaned tree (A-10). Expanded here, as discovery does.
+  for (const pattern of positive.flatMap(slashBraceExpansions)) {
     const glob = globFor(pattern)
     for (const rel of glob.scanSync({ cwd, onlyFiles: false, followSymlinks: false, dot: true })) {
       if (excludeGlobs.some((g) => g.match(rel))) continue

package/src/orchestrator/run-artifacts.ts

@@ -29,6 +29,8 @@ export interface SummarizeArgs {
   totalMs: number
   /** The run's verdict — the same value the CLI turns into the exit code. */
   ok: boolean
+  /** The process's exit code where it is not `ok ? 0 : 1`: a stopping signal's. */
+  exitCode?: number
   outcomes: readonly TaskOutcome[]
   /** What `detectFlaky` found this run; the footer's Flaky section, typed. */
   flaky?: readonly FlakyFinding[]
@@ -96,12 +98,13 @@ export async function writeRunSummary(args: SummarizeArgs): Promise<string> {
   // parser must not be told less than a human is.
   const aborted = args.outcomes.filter((o) => !isGroupTask(o.node) && o.status === 'aborted')
   const flakyById = new Map((args.flaky ?? []).map((f) => [f.taskId, f]))
+  const exitCode = args.exitCode ?? (args.ok ? 0 : 1)
   const payload = {
     runId: args.runId,
     // The run-level verdict, first: a consumer gating on this artifact must
     // not have to re-derive it by re-implementing the bucket rules.
-    ok: args.ok,
-    exitCode: args.ok ? 0 : 1,
+    ok: exitCode === 0,
+    exitCode,
     startedAt: new Date(args.startedAtMs).toISOString(),
     endedAt: new Date(args.endedAtMs).toISOString(),
     totalMs: args.totalMs,

package/src/orchestrator/run.ts

@@ -58,6 +58,7 @@ import { prepareRun, type PreparedRun } from './prepare.js'
 import { acquireRunLock } from './run-lock.js'
 import {
   forwardedSignal,
+  type StopSignal,
   forwardSignals,
   SIGNAL_SHUTDOWN_GRACE_MS,
   terminateChildren,
@@ -456,7 +457,7 @@ async function runOnBus(
   // handler exited before any outcome landed, and a Ctrl-C'd run then read
   // in `vx last` as FAILED with 0 tasks (item 854). An embedder's abort is
   // its own call and keeps its record.
-  let stoppedBySignal = false
+  let stoppedBy: StopSignal | undefined
   let leftRun = (): void => {}
   const runLeft = new Promise<void>((resolve) => {
     leftRun = resolve
@@ -468,7 +469,7 @@ async function runOnBus(
     liveChildren,
     persistentRegistry,
     stop: (signal) => {
-      stoppedBySignal = true
+      stoppedBy = signal
       stopRun.abort(signal)
     },
     done: runLeft,
@@ -903,7 +904,10 @@ async function runOnBus(
 
     // Optional artifacts. Errors are surfaced to the user but don't
     // change the run's exit code — the run already happened.
-    if (options.summarize !== undefined) {
+    // Written again after the keep-alive wait: a kept server's crash or a
+    // Ctrl-C there is the process's exit, and the first write said ok.
+    const summarize = async (runOk: boolean): Promise<void> => {
+      if (options.summarize === undefined) return
       try {
         const wrote = await writeRunSummary({
           target: options.summarize,
@@ -913,7 +917,8 @@ async function runOnBus(
           startedAtMs: endedAtMsAtStart,
           endedAtMs,
           totalMs,
-          ok,
+          ok: runOk,
+          ...(stoppedBy !== undefined && { exitCode: signalExitCode(stoppedBy) }),
           outcomes: list,
           flaky,
         })
@@ -923,6 +928,7 @@ async function runOnBus(
         log.status(`vx: failed to write summary: ${msg}`)
       }
     }
+    await summarize(ok)
     if (options.profile !== undefined) {
       try {
         const wrote = await writeRunProfile({
@@ -964,7 +970,7 @@ async function runOnBus(
       withTelemetry: telemetry !== undefined,
     })
     try {
-      if (!stoppedBySignal) cache.recordRunBundle(records)
+      if (stoppedBy === undefined) cache.recordRunBundle(records)
     } catch (err) {
       // History is observability: a full cache disk at the very end must
       // not turn a finished run's verdict into a stack and exit 1 (seen as
@@ -1105,6 +1111,7 @@ async function runOnBus(
         )
       }
       await terminateChildren(() => keepAlive.children)
+      await summarize(ok && first.code === 0)
       return { ok: ok && first.code === 0, outcomes: list }
     }
 

package/src/util/index.ts

@@ -30,6 +30,7 @@ export {
   staticPrefix,
   taskGlob,
   wholeSubtreePrefixes,
+  slashBraceExpansions,
 } from './paths.js'
 export { killGraceMs, settleWithin, teardownTimeoutMs } from './settle.js'
 export { formatBytes, parseDuration, parseSize } from './size.js'

package/src/util/paths.ts

@@ -227,3 +227,37 @@ export function asTrees(patterns: readonly string[]): string[] {
   }
   return out
 }
+
+/**
+ * `pattern` with its first brace group expanded, recursively, while that
+ * group holds a `/`; any other pattern as itself, since `Bun.Glob`
+ * expands a slash-free brace on its own. An unbalanced or escaped brace
+ * is left to the glob.
+ */
+export function slashBraceExpansions(pattern: string): string[] {
+  if (pattern.includes('\\')) return [pattern]
+  for (let open = pattern.indexOf('{'); open !== -1; open = pattern.indexOf('{', open + 1)) {
+    let depth = 0
+    const cuts: number[] = []
+    let close = -1
+    for (let i = open; i < pattern.length && close === -1; i++) {
+      const c = pattern[i]
+      if (c === '{') depth++
+      else if (c === '}' && --depth === 0) close = i
+      else if (c === ',' && depth === 1) cuts.push(i)
+    }
+    if (close === -1) return [pattern]
+    const body = pattern.slice(open + 1, close)
+    if (cuts.length === 0 || !body.includes('/')) continue
+    const head = pattern.slice(0, open)
+    const tail = pattern.slice(close + 1)
+    const bounds = [open, ...cuts, close]
+    const out: string[] = []
+    for (let k = 0; k + 1 < bounds.length; k++) {
+      const alt = pattern.slice(bounds[k]! + 1, bounds[k + 1])
+      out.push(...slashBraceExpansions(head + alt + tail))
+    }
+    return [...new Set(out)]
+  }
+  return [pattern]
+}

package/src/workspace/workspace.ts

@@ -2,7 +2,13 @@ import type { Dirent } from 'node:fs'
 import { readdir, stat } from 'node:fs/promises'
 import path from 'node:path'
 import type { ProjectConfig, WorkspaceConfig } from '../config.js'
-import { BUN_GLOB_WILDCARDS, relPosix, UserError, normalizeBunGlob } from '../util/index.js'
+import {
+  BUN_GLOB_WILDCARDS,
+  relPosix,
+  slashBraceExpansions,
+  UserError,
+  normalizeBunGlob,
+} from '../util/index.js'
 import { type LoadReads, readOnce } from './load-reads.js'
 
 export interface PackageJson {
@@ -441,40 +447,6 @@ async function memberDirs(root: string, pattern: string): Promise<string[]> {
   return dirs
 }
 
-/**
- * `pattern` with its first brace group expanded, recursively, while that
- * group holds a `/`; any other pattern as itself, since `Bun.Glob`
- * expands a slash-free brace on its own. An unbalanced or escaped brace
- * is left to the glob.
- */
-function slashBraceExpansions(pattern: string): string[] {
-  if (pattern.includes('\\')) return [pattern]
-  for (let open = pattern.indexOf('{'); open !== -1; open = pattern.indexOf('{', open + 1)) {
-    let depth = 0
-    const cuts: number[] = []
-    let close = -1
-    for (let i = open; i < pattern.length && close === -1; i++) {
-      const c = pattern[i]
-      if (c === '{') depth++
-      else if (c === '}' && --depth === 0) close = i
-      else if (c === ',' && depth === 1) cuts.push(i)
-    }
-    if (close === -1) return [pattern]
-    const body = pattern.slice(open + 1, close)
-    if (cuts.length === 0 || !body.includes('/')) continue
-    const head = pattern.slice(0, open)
-    const tail = pattern.slice(close + 1)
-    const bounds = [open, ...cuts, close]
-    const out: string[] = []
-    for (let k = 0; k + 1 < bounds.length; k++) {
-      const alt = pattern.slice(bounds[k]! + 1, bounds[k + 1])
-      out.push(...slashBraceExpansions(head + alt + tail))
-    }
-    return [...new Set(out)]
-  }
-  return [pattern]
-}
-
 /**
  * The project's config file, by `PROJECT_CONFIG_FILENAMES` precedence, or null.
  *