@theholocron/astromech 5.0.0-alpha.77 → 5.0.0-alpha.78

package/dist/index.d.mts

@@ -332,10 +332,14 @@ interface Astromech {
    * (not just an orchestrator) silently vanishes from `turbo run <task>`.
    * Pass `pnpm-workspace.yaml`'s current content (or `""` if none exists —
    * a no-op either way, since no `packages:` key means single-package mode
-   * already) and root `package.json`'s own `scripts` keys. `changed: false`
-   * means don't write anything — the file is already correct.
+   * already) and root `package.json`'s own `scripts` object — values and
+   * all, not just keys, so a task-name-matching script that's already the
+   * generated `holocron run <task> --` wrapper (predating this call, not
+   * written by it) is correctly excluded rather than treated as proof root
+   * is directly buildable (#846). `changed: false` means don't write
+   * anything — the file is already correct.
    */
-  ensureRootWorkspaceMember(workspaceYaml: string, rootScripts: readonly string[]): EnsureRootWorkspaceMemberResult;
+  ensureRootWorkspaceMember(workspaceYaml: string, rootScripts: Readonly<Record<string, string>>): EnsureRootWorkspaceMemberResult;
   /**
    * `turboConfig()` writes a `turbo.json` but never touches `package.json`
    * — a repo with no local `turbo` devDependency falls back to whatever

package/dist/index.mjs

@@ -1544,6 +1544,17 @@ function turboConfig(config) {
 * (`Packages in scope: docs, root-lib`) without disturbing the sibling
 * package's own caching.
 *
+* A matching script key alone isn't enough — its *value* has to be a real
+* command, not the generated `holocron run <task> --` wrapper (#846, the
+* counterpart `mergePackageJsonScripts()` already had to learn for #747/
+* #748). A repo whose root predates this tooling can already have that exact
+* wrapper sitting in a task-name-matching script with nothing ever having
+* generated it this session; adding `.` there fires #747's recursion
+* immediately (`turbo run <task>` → root's own `<task>` script → the wrapper
+* → `holocron run <task>` sees `turbo.json` still defines it → `turbo run
+* <task>` again, forever). Only a matching script whose value is *not* that
+* wrapper counts as "root is genuinely a directly-buildable package".
+*
 * A targeted line-based edit, not a full YAML parse/reserialize — same
 * pattern `codecov.ts`'s `mergeCodecovComponents()` already uses for editing
 * an existing generated file. `pnpm-workspace.yaml` routinely carries a
@@ -1555,7 +1566,10 @@ function turboConfig(config) {
 * every repo checked actually uses.
 */
 function ensureRootWorkspaceMember(workspaceYaml, rootScripts, taskNames) {
-	if (!taskNames.some((t) => rootScripts.includes(t))) return {
+	if (!taskNames.some((t) => {
+		const script = rootScripts[t];
+		return script !== void 0 && !isGeneratedRunWrapper(script, t);
+	})) return {
 		content: workspaceYaml,
 		changed: false
 	};
@@ -1589,6 +1603,21 @@ function ensureRootWorkspaceMember(workspaceYaml, rootScripts, taskNames) {
 	};
 }
 /**
+* Matches the exact shape `packageScripts()` generates for `taskName` —
+* `<bin> run <taskName> --`, `<bin>` normally `holocron` but substitutable
+* (`holocronScript`, itself allowed to be multi-word, e.g. `"node
+* packages/cli/dist/cli.mjs"` — see its own test). Checking the *suffix*
+* rather than the whole string means an old wrapper survives a
+* `holocronScript` rename intact — it's still recognized as a wrapper (safe
+* to update to the new one), not mistaken for a real command. Mirrors
+* `sync.ts`'s private `isGeneratedRunWrapper()` — same predicate, needed on
+* both sides of #692/#747: whether to *write* the wrapper, and here, whether
+* an *existing* one should count as "root is directly buildable".
+*/
+function isGeneratedRunWrapper(command, taskName) {
+	return command.endsWith(` run ${taskName} --`);
+}
+/**
 * Fallback pin when a repo's own `pnpm-workspace.yaml` has no `turbo`
 * catalog entry yet — matches this repo's own catalog version (the
 * canonical source-of-truth repo for the whole org's tooling).

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@theholocron/astromech",
-  "version": "5.0.0-alpha.77",
+  "version": "5.0.0-alpha.78",
   "description": "The Holocron task runner — one task manifest drives `holocron run`, `holocron ci`, the CI workflows, package.json scripts, linters, and required checks.",
   "keywords": [
     "ci",
@@ -37,7 +37,7 @@
     "dist"
   ],
   "dependencies": {
-    "@theholocron/datapad": "5.0.0-alpha.77"
+    "@theholocron/datapad": "5.0.0-alpha.78"
   },
   "devDependencies": {
     "@theholocron/eslint-config": "^8.4.5",
@@ -53,7 +53,7 @@
     "tsdown": "^0.22.14",
     "typescript": "^5.9.3",
     "vitest": "^4.1.11",
-    "@theholocron/rollup-plugin-transform-template": "5.0.0-alpha.77"
+    "@theholocron/rollup-plugin-transform-template": "5.0.0-alpha.78"
   },
   "engines": {
     "node": ">=22"