@vzn/vx 0.0.67 → 0.0.69

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vzn/vx",
-  "version": "0.0.67",
+  "version": "0.0.69",
   "description": "An open, extensible monorepo task runner.",
   "type": "module",
   "exports": {
@@ -17,10 +17,10 @@
     "node": ">=18"
   },
   "optionalDependencies": {
-    "@vzn/vx-linux-x64": "0.0.67",
-    "@vzn/vx-linux-arm64": "0.0.67",
-    "@vzn/vx-darwin-x64": "0.0.67",
-    "@vzn/vx-darwin-arm64": "0.0.67"
+    "@vzn/vx-linux-x64": "0.0.69",
+    "@vzn/vx-linux-arm64": "0.0.69",
+    "@vzn/vx-darwin-x64": "0.0.69",
+    "@vzn/vx-darwin-arm64": "0.0.69"
   },
   "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/exec/executor.ts

@@ -190,7 +190,8 @@ export interface TaskExecutor {
   /**
    * How many tasks this executor runs at once. Its tasks then occupy a pool
    * of this size instead of the local worker slots, so a remote pool is not
-   * throttled by the local CPU count. Absent = the local pool.
+   * throttled by the local CPU count. A positive integer; anything else
+   * is refused when the executor is resolved. Absent = the local pool.
    */
   readonly capacity?: number
   /** Per-task opt-out at placement time. Absent = accepts every task it is offered. */

package/src/orchestrator/plugin-host.ts

@@ -419,6 +419,13 @@ export async function resolveExecutors(
     if (typeof executor.name !== 'string' || executor.name.length === 0) {
       throw new UserError(`plugin '${plugin.name}' returned an executor with no name`)
     }
+    // The pool's room is `active < capacity`: 0 or NaN parks its tasks for good.
+    const capacity: unknown = executor.capacity
+    if (capacity !== undefined && !(Number.isInteger(capacity) && (capacity as number) > 0)) {
+      throw new UserError(
+        `plugin '${plugin.name}' returned executor '${executor.name}' with capacity ${typeof capacity === 'number' ? String(capacity) : JSON.stringify(capacity)}: it must be a positive integer`,
+      )
+    }
     executorPlugin.set(executor, plugin.name)
     executors.push(executor)
   }