@vzn/vx 0.0.123 → 0.0.125

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vzn/vx",
-  "version": "0.0.123",
+  "version": "0.0.125",
   "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.123",
-    "@vzn/vx-linux-arm64": "0.0.123",
-    "@vzn/vx-darwin-x64": "0.0.123",
-    "@vzn/vx-darwin-arm64": "0.0.123"
+    "@vzn/vx-linux-x64": "0.0.125",
+    "@vzn/vx-linux-arm64": "0.0.125",
+    "@vzn/vx-darwin-x64": "0.0.125",
+    "@vzn/vx-darwin-arm64": "0.0.125"
   },
   "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/workspace/config-cache.ts

@@ -142,9 +142,11 @@ const MAX_CLOSURE_FILES = 32
 // defeat it. `random` is denied as a word, not as `Math.random`: a
 // destructured `const { random } = Math` was cached as pure (item 1036).
 // `prompt`, `confirm` and `alert` read the user's terminal, and a cached
-// evaluation replayed the first answer (D-25).
+// evaluation replayed the first answer (D-25). A CommonJS config reaches
+// `require` without naming it as `arguments[1]`, and one reading a file
+// outside its closure so replayed the old bytes (D-36).
 const IMPURE_RE =
-  /\b(?:process|Bun|globalThis|global|self|fetch|Date|Temporal|Intl|crypto|performance|navigator|require|eval|Function|constructor|localeCompare|await|toLocale\w*|Reflect|getPrototypeOf|setPrototypeOf|getOwnPropertyNames|getOwnPropertyDescriptor|getOwnPropertyDescriptors|__proto__|prototype|__defineGetter__|__defineSetter__|__lookupGetter__|__lookupSetter__|random|prompt|confirm|alert)\b|import\s*\.\s*meta|\bimport\s*\(/
+  /\b(?:process|Bun|globalThis|global|self|fetch|Date|Temporal|Intl|crypto|performance|navigator|require|eval|Function|constructor|localeCompare|await|toLocale\w*|Reflect|getPrototypeOf|setPrototypeOf|getOwnPropertyNames|getOwnPropertyDescriptor|getOwnPropertyDescriptors|__proto__|prototype|__defineGetter__|__defineSetter__|__lookupGetter__|__lookupSetter__|random|prompt|confirm|alert|arguments)\b|import\s*\.\s*meta|\bimport\s*\(/
 
 /** Literal text that names a way to `Function` when used as a computed key. */
 const IMPURE_LITERAL_RE = /constructor|__proto__|prototype/