@vzn/vx 0.0.68 → 0.0.70

package/package.json

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

package/src/cache/inputs.ts

@@ -488,6 +488,19 @@ async function resolveRuntimeValues(
  */
 const OUTPUT_NEVER = ['**/.git/**', '**/.vx/**']
 
+/**
+ * What a set of output globs may not reach: `OUTPUT_NEVER`, and every
+ * `node_modules` unless a glob names one. `**\/*.js` meant the build's
+ * files, and the clean before each run deleted every installed `.js`
+ * under `node_modules` with them (A-13); an install task declares
+ * `node_modules/**` and keeps it. Workspace outputs took none of this, not
+ * even `.git`.
+ */
+function outputExcludes(outputs: readonly string[]): Bun.Glob[] {
+  const namesNodeModules = outputs.some((o) => normalizeGlob(o).split('/').includes('node_modules'))
+  return (namesNodeModules ? OUTPUT_NEVER : [...OUTPUT_NEVER, '**/node_modules/**']).map(globFor)
+}
+
 /** Resolve declared output globs (project-relative) to actual produced files. */
 export async function resolveOutputs(args: {
   projectDir: string
@@ -495,7 +508,7 @@ export async function resolveOutputs(args: {
   nestedProjectDirs: string[]
 }): Promise<string[]> {
   if (args.outputs.length === 0) return []
-  const excludeGlobs = OUTPUT_NEVER.map(globFor)
+  const excludeGlobs = outputExcludes(args.outputs)
   const scanned = [
     ...(await scanUnion(
       asTrees(args.outputs),
@@ -741,7 +754,9 @@ export async function resolveWorkspaceOutputs(args: {
   outputs: string[]
 }): Promise<string[]> {
   if (args.outputs.length === 0) return []
-  const scanned = [...(await scanUnion(asTrees(args.outputs), [], args.workspaceRoot))]
+  const scanned = [
+    ...(await scanUnion(asTrees(args.outputs), outputExcludes(args.outputs), args.workspaceRoot)),
+  ]
   // Same containment as the project twin, anchored one level out. These globs
   // deliberately ignore PROJECT boundaries — that is the escape hatch — but
   // escaping the WORKSPACE was never part of it, and `cleanWorkspaceOutputs`

package/src/graph/scheduler.ts

@@ -428,23 +428,24 @@ export async function runGraph(options: ScheduleOptions): Promise<Map<string, Ta
   const poolOf = options.poolOf
   const poolActive = new Map<string, number>()
   // A pooled task is admitted against ITS pool's capacity, a local one
-  // against `concurrency`; pooled tasks reserve no local resources.
+  // against `concurrency`; pooled tasks reserve no local resources. A
+  // restore is local disk work wherever its task was placed: on the pool's
+  // arm it bypassed the restore lane's cap.
   const hasRoom = (id: string): boolean => {
+    if (inRestoreTier(id)) return restoreRoom()
     const pool = poolOf?.(id)
-    if (pool === undefined) {
-      return inRestoreTier(id) ? restoreRoom() : execRoom()
-    }
+    if (pool === undefined) return execRoom()
     return (poolActive.get(pool.name) ?? 0) < pool.capacity
   }
   const admit = (id: string): (() => void) => {
+    if (inRestoreTier(id)) {
+      activeRestore++
+      return () => {
+        activeRestore--
+      }
+    }
     const pool = poolOf?.(id)
     if (pool === undefined) {
-      if (inRestoreTier(id)) {
-        activeRestore++
-        return () => {
-          activeRestore--
-        }
-      }
       active++
       return () => {
         active--