@vzn/vx 0.0.299 → 0.0.300

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vzn/vx",
-  "version": "0.0.299",
+  "version": "0.0.300",
   "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.299",
-    "@vzn/vx-linux-arm64": "0.0.299",
-    "@vzn/vx-darwin-x64": "0.0.299",
-    "@vzn/vx-darwin-arm64": "0.0.299"
+    "@vzn/vx-linux-x64": "0.0.300",
+    "@vzn/vx-linux-arm64": "0.0.300",
+    "@vzn/vx-darwin-x64": "0.0.300",
+    "@vzn/vx-darwin-arm64": "0.0.300"
   },
   "dependencies": {
     "@anthropic-ai/sandbox-runtime": "^0.0.76"

package/src/index.ts

@@ -118,6 +118,7 @@ export {
   applyMigration,
   foldScriptHooks,
   PERSISTENT_TODO,
+  pruneOrphanPersistentNotes,
   quoteTsLiteral,
 } from './workspace/index.js'
 export type {

package/src/workspace/index.ts

@@ -64,6 +64,7 @@ export {
   foldScriptHooks,
   PERSISTENT_TASK_NAMES,
   PERSISTENT_TODO,
+  pruneOrphanPersistentNotes,
   quoteTsLiteral,
   type ApplyMigrationArgs,
   type MigrationFormat,

package/src/workspace/migrate-scripts.ts

@@ -39,6 +39,7 @@ import {
   PERSISTENT_TASK_NAMES,
   PERSISTENT_TODO,
   cacheTodo,
+  pruneOrphanPersistentNotes,
 } from './migration.js'
 
 // `lint` is not here: a linter reads sources, and an edge to `build`
@@ -379,6 +380,7 @@ export function migrateScripts(
     }
   }
   breakBuildCycles(projects, metas)
+  pruneOrphanPersistentNotes(projects, PERSISTENT_TODO)
   return {
     headerNotes: [
       // Where to add one is the TODO's to say, when there is one: an Nx repo

package/src/workspace/migration.ts

@@ -97,6 +97,47 @@ export const PERSISTENT_TODO =
   'persistent task — set persistent.readyWhen (regex matched against output) so ' +
   'dependents unblock on readiness, and consider exec.timeout to bound the wait'
 
+// The readiness note a persistent task carries ("add `readyWhen` to gate
+// dependents") is moot for a task nothing depends on: no one waits for it,
+// so there is nothing to gate. refine printed it for 375 `dev` and `start`
+// tasks on every run, none of them anyone's dependency (item 602), and
+// `vx init` wrote it on hoppscotch's 24 `dev`-shaped scripts. Every mapper
+// runs this last, over the whole mapping, since a dependent may sit in
+// another package.
+
+/** Strips the note from every persistent task no `dependsOn` in the mapping names. */
+export function pruneOrphanPersistentNotes(
+  projects: readonly {
+    readonly tasks: readonly {
+      readonly name: string
+      readonly todos: string[]
+      readonly task: Record<string, unknown> | null
+    }[]
+  }[],
+  note: string,
+): void {
+  const referenced = new Set<string>()
+  for (const project of projects) {
+    for (const t of project.tasks) {
+      const deps = t.task?.['dependsOn']
+      if (!Array.isArray(deps)) continue
+      for (const d of deps) {
+        if (typeof d !== 'string') continue
+        // `^name`, `pkg#name` and `name` all name the same task.
+        const hash = d.indexOf('#')
+        referenced.add(hash === -1 ? d.replace(/^\^/, '') : d.slice(hash + 1))
+      }
+    }
+  }
+  for (const project of projects) {
+    for (const t of project.tasks) {
+      if (referenced.has(t.name)) continue
+      const at = t.todos.indexOf(note)
+      if (at !== -1) t.todos.splice(at, 1)
+    }
+  }
+}
+
 /** Verbatim TS expression spliced into a generated array (preset spreads). */
 export interface RawExpr {
   readonly raw: string