@vzn/vx 0.0.122 → 0.0.124

package/package.json

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

package/src/cache/index.ts

@@ -40,7 +40,9 @@ export {
   type WorkspaceFilesCache,
   cleanOutputPaths,
   ownOutputsSince,
+  ownWorkspaceOutputsSince,
   stampOutputs,
+  stampWorkspaceOutputs,
   type OutputStamp,
 } from './inputs.js'
 export {

package/src/cache/inputs.ts

@@ -643,7 +643,18 @@ export async function stampOutputs(args: {
   outputs: string[]
   nestedProjectDirs: string[]
 }): Promise<Map<string, OutputStamp>> {
-  const files = await resolveOutputs(args)
+  return stampFiles(await resolveOutputs(args))
+}
+
+/** `stampOutputs` for root-anchored `cache.outputs.workspaceFiles` (A-43). */
+export async function stampWorkspaceOutputs(args: {
+  workspaceRoot: string
+  outputs: string[]
+}): Promise<Map<string, OutputStamp>> {
+  return stampFiles(await resolveWorkspaceOutputs(args))
+}
+
+function stampFiles(files: readonly string[]): Map<string, OutputStamp> {
   const out = new Map<string, OutputStamp>()
   for (const f of files) {
     try {
@@ -668,7 +679,21 @@ export async function ownOutputsSince(
   args: { projectDir: string; outputs: string[]; nestedProjectDirs: string[] },
   before: ReadonlyMap<string, OutputStamp>,
 ): Promise<string[]> {
-  const after = await resolveOutputs(args)
+  return changedSince(await resolveOutputs(args), before)
+}
+
+/** `ownOutputsSince` for root-anchored `cache.outputs.workspaceFiles` (A-43). */
+export async function ownWorkspaceOutputsSince(
+  args: { workspaceRoot: string; outputs: string[] },
+  before: ReadonlyMap<string, OutputStamp>,
+): Promise<string[]> {
+  return changedSince(await resolveWorkspaceOutputs(args), before)
+}
+
+function changedSince(
+  after: readonly string[],
+  before: ReadonlyMap<string, OutputStamp>,
+): string[] {
   const own: string[] = []
   for (const f of after) {
     const was = before.get(f)

package/src/orchestrator/execute-task.ts

@@ -9,7 +9,9 @@ import {
   cleanWorkspaceOutputs,
   type OutputStamp,
   ownOutputsSince,
+  ownWorkspaceOutputsSince,
   stampOutputs,
+  stampWorkspaceOutputs,
   FULL_CACHE_POLICY,
   type GitFilesCache,
 } from '../cache/index.js'
@@ -616,6 +618,7 @@ async function executeCachedTask(args: ExecuteArgs): Promise<TaskOutcome> {
   // earlier run are its command's to clean, as they are under Turbo.
   const additive = (node.addsToOutputsOf?.length ?? 0) > 0
   let stampedBefore: ReadonlyMap<string, OutputStamp> | undefined
+  let wsStampedBefore: ReadonlyMap<string, OutputStamp> | undefined
 
   // Cache lookup. On hit, time the user-perceived restore op
   // (clean+restore+log-replay) — that's what the framed-block footer
@@ -756,7 +759,12 @@ async function executeCachedTask(args: ExecuteArgs): Promise<TaskOutcome> {
       endClean()
       args.gitFilesCache?.markOutputsChanged(node.projectDir, cleanedRels)
     }
-    if (willWrite && !deferralRequested && wsOutputs.length > 0) {
+    if (additive && willWrite && !deferralRequested && wsOutputs.length > 0) {
+      // A root-anchored addition is stamped as the project one above: a
+      // glob clean deleted the upstream's files before this task, which
+      // reads them, ran (A-43).
+      wsStampedBefore ??= await stampWorkspaceOutputs(wsCleanArgs)
+    } else if (willWrite && !deferralRequested && wsOutputs.length > 0) {
       // Root-anchored deletions can land in other projects' dirs; mark
       // them so stale per-project git snapshots can't survive the wipe.
       const cleanedWsRels = await cleanWorkspaceOutputs(wsCleanArgs)
@@ -1028,8 +1036,13 @@ async function executeCachedTask(args: ExecuteArgs): Promise<TaskOutcome> {
       additive && stampedBefore !== undefined
         ? await ownOutputsSince(cleanArgs, stampedBefore)
         : undefined
+    const ownWsOutputFiles =
+      additive && wsStampedBefore !== undefined
+        ? await ownWorkspaceOutputsSince(wsCleanArgs, wsStampedBefore)
+        : undefined
     const { landed } = await saveMiss({
       ...(ownOutputFiles !== undefined ? { ownOutputFiles } : {}),
+      ...(ownWsOutputFiles !== undefined ? { ownWsOutputFiles } : {}),
       node,
       hash,
       cache,

package/src/orchestrator/miss-save.ts

@@ -71,6 +71,8 @@ export interface SaveMissArgs {
    * would also take the upstream's files the task adds beside.
    */
   ownOutputFiles?: string[] | undefined
+  /** `ownOutputFiles` for `cache.outputs.workspaceFiles` (A-43). */
+  ownWsOutputFiles?: string[] | undefined
   /** The Tier-3 input fingerprint rows captured by the pre-exec describe. */
   captured: readonly TaskInputComponent[]
   command: string
@@ -101,10 +103,12 @@ export async function saveMiss(a: SaveMissArgs): Promise<{ landed: Promise<void>
       outputs: a.outputs,
       nestedProjectDirs: a.nestedProjectDirs,
     }))
-  const wsOutputFiles = await resolveWorkspaceOutputs({
-    workspaceRoot: a.workspaceRoot,
-    outputs: a.wsOutputs,
-  })
+  const wsOutputFiles =
+    a.ownWsOutputFiles ??
+    (await resolveWorkspaceOutputs({
+      workspaceRoot: a.workspaceRoot,
+      outputs: a.wsOutputs,
+    }))
   endResolve()
   // The mirror of the input warning: declared outputs that resolve to
   // nothing save an empty artifact, and the next hit "restores" it —

package/src/orchestrator/shell-verdict.ts

@@ -1,4 +1,4 @@
-import { existsSync, readFileSync, statSync } from 'node:fs'
+import { accessSync, constants as fsConstants, existsSync, readFileSync, statSync } from 'node:fs'
 import { constants as osConstants } from 'node:os'
 import path from 'node:path'
 import { execWord, exitSignal } from '../exec/index.js'
@@ -46,8 +46,12 @@ function fileVerdict(file: string): string {
   try {
     const st = statSync(file)
     if (st.isDirectory()) return 'is a directory'
-    if ((st.mode & 0o111) === 0) return 'is not executable — chmod +x it'
-    head = readFileSync(file, 'latin1').slice(0, 256)
+    // Ask the kernel: an execute bit for the group or others does not
+    // make a file this user owns executable to this user (root aside).
+    if (!executable(file)) return 'is not executable — chmod +x it'
+    // The kernel reads the #! line as bytes, and a path's bytes are its
+    // UTF-8: a latin1 read named a present non-ASCII interpreter missing.
+    head = readFileSync(file).subarray(0, 256).toString('utf8')
   } catch (err) {
     return `could not be read (${(err as Error).message})`
   }
@@ -115,3 +119,12 @@ const SIGNAL_WHY: Record<string, string> = {
   SIGSYS: 'a system call was refused — a seccomp filter, or the sandbox',
   SIGQUIT: 'the process was sent a quit (Ctrl-\\, or a kill -QUIT); a core dump may be beside it',
 }
+
+function executable(file: string): boolean {
+  try {
+    accessSync(file, fsConstants.X_OK)
+    return true
+  } catch {
+    return false
+  }
+}