@vzn/vx 0.0.73 → 0.0.75

package/package.json

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

package/src/cache/inputs.ts

@@ -234,6 +234,7 @@ async function resolveWorkspaceFiles(args: {
 }): Promise<string[]> {
   const positive: string[] = []
   const negative: string[] = []
+  refuseOneAlternativeBrace(args.workspaceFiles, 'workspaceFiles')
   for (const entry of args.workspaceFiles) {
     if (entry.startsWith('!')) negative.push(entry.slice(1))
     else positive.push(entry)
@@ -895,6 +896,30 @@ function refuseUndecodable(
   )
 }
 
+const ONE_ALTERNATIVE_BRACE = /(?<!\\)\{[^{},]*\}/
+
+/**
+ * `Bun.Glob` reads `{b}` as a brace of one alternative, so `src/{b}.ts`
+ * matches `src/b.ts` and never a file named `{b}.ts`: that file stays out
+ * of the key and an edit to it is a hit. Neither reading is sure to be
+ * the one meant, so the entry is refused with both spellings.
+ */
+function refuseOneAlternativeBrace(
+  entries: readonly string[],
+  field: 'files' | 'workspaceFiles',
+): void {
+  for (const entry of entries) {
+    const m = ONE_ALTERNATIVE_BRACE.exec(entry)
+    if (m === null) continue
+    const inner = m[0].slice(1, -1)
+    throw new UserError(
+      `cache.inputs.${field}: "${entry}" holds a brace with one alternative, which matches ` +
+        `"${inner}" and never a name holding "${m[0]}". Write "${entry.replace(m[0], inner)}", ` +
+        `or "${entry.replace(m[0], `\\{${inner}\\}`)}" for the braces themselves.`,
+    )
+  }
+}
+
 async function resolveFiles(args: ResolveFilesArgs): Promise<string[]> {
   const positive: string[] = []
   const negative: string[] = []
@@ -902,6 +927,7 @@ async function resolveFiles(args: ResolveFilesArgs): Promise<string[]> {
   if (args.files === undefined) {
     positive.push(...DEFAULT_FILE_GLOBS)
   } else {
+    refuseOneAlternativeBrace(args.files, 'files')
     for (const entry of args.files) {
       if (entry.startsWith('!')) negative.push(entry.slice(1))
       else positive.push(entry)

package/src/cli/run.ts

@@ -1,4 +1,4 @@
-import { appendFile, mkdir } from 'node:fs/promises'
+import { appendFileSync, mkdirSync } from 'node:fs'
 import path from 'node:path'
 import { documentedFlags, seeHelp } from './help.js'
 import { defaultAffectedBase, findWorkspaceRoot } from '../workspace/index.js'
@@ -649,8 +649,10 @@ export async function runCmd(args: readonly string[]): Promise<number> {
         // behaviour change rather than a drop-in.
         // Its directory made first, as `--summarize`, `--profile` and
         // `--graph` make theirs: `nr/r.md` failed alone of the four (item 993).
-        await mkdir(path.dirname(target), { recursive: true })
-        await appendFile(target, md)
+        // Sync: on a Ctrl-C the signal path exits once stdout drains, and
+        // an awaited write lost the race every time (E-23).
+        mkdirSync(path.dirname(target), { recursive: true })
+        appendFileSync(target, md)
       } catch (err) {
         // Same contract as --summarize / --profile: the run already
         // happened, so a write failure is reported, not fatal.

package/src/exec/kill-tree.ts

@@ -87,7 +87,21 @@ function startGuard(): void {
       detached: true,
     })
     guard.unref()
-    guardFd = guard.stdio[3] as number
+    const fd = guard.stdio[3] as number
+    guardFd = fd
+    // A guard that has died (killed, the OOM killer) is handed to no later
+    // spawn: its pipe is broken, and a shell whose printf buffers (bash as
+    // macOS's sh) flushed the failed `+<pgid>` line into the task's own
+    // stdout, cached replay included (B-10).
+    void guard.exited.then(() => {
+      if (guardFd !== fd) return
+      guardFd = null
+      try {
+        closeSync(fd)
+      } catch {
+        // already closed
+      }
+    })
   } catch {
     // The limit as it was: nothing takes the groups down.
   }