@femtomc/mu-orchestrator 26.2.19 → 26.2.21

package/dist/dag_runner.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"dag_runner.d.ts","sourceRoot":"","sources":["../src/dag_runner.ts"],"names":[],"mappings":"AAGA,OAAO,EAEN,KAAK,QAAQ,EAKb,MAAM,uBAAuB,CAAC;AAC/B,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,mBAAmB,CAAC;AACpD,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,mBAAmB,CAAC;AACpD,OAAO,KAAK,EAAE,cAAc,EAAuB,MAAM,uBAAuB,CAAC;AAGjF,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,iBAAiB,CAAC;AAGrD,MAAM,MAAM,SAAS,GAAG;IACvB,MAAM,EAAE,YAAY,GAAG,oBAAoB,GAAG,qBAAqB,GAAG,OAAO,CAAC;IAC9E,KAAK,EAAE,MAAM,CAAC;IACd,KAAK,EAAE,MAAM,CAAC;CACd,CAAC;AAEF,MAAM,MAAM,uBAAuB,GAAG;IACrC,MAAM,EAAE,MAAM,CAAC;IACf,IAAI,EAAE,MAAM,CAAC;IACb,OAAO,EAAE,MAAM,CAAC;IAChB,IAAI,EAAE,MAAM,GAAG,IAAI,CAAC;IACpB,KAAK,EAAE,MAAM,CAAC;CACd,CAAC;AAEF,MAAM,MAAM,qBAAqB,GAAG;IACnC,MAAM,EAAE,MAAM,CAAC;IACf,IAAI,EAAE,MAAM,CAAC;IACb,OAAO,EAAE,MAAM,CAAC;IAChB,QAAQ,EAAE,MAAM,CAAC;IACjB,QAAQ,EAAE,MAAM,CAAC;IACjB,OAAO,EAAE,MAAM,GAAG,IAAI,CAAC;CACvB,CAAC;AAEF,MAAM,MAAM,yBAAyB,GAAG;IACvC,MAAM,EAAE,MAAM,CAAC;IACf,IAAI,EAAE,MAAM,CAAC;IACb,OAAO,EAAE,MAAM,CAAC;IAChB,SAAS,EAAE,MAAM,CAAC;IAClB,IAAI,EAAE,MAAM,CAAC;CACb,CAAC;AAEF,MAAM,MAAM,cAAc,GAAG;IAC5B,WAAW,CAAC,EAAE,CAAC,EAAE,EAAE,uBAAuB,KAAK,IAAI,GAAG,OAAO,CAAC,IAAI,CAAC,CAAC;IACpE,SAAS,CAAC,EAAE,CAAC,EAAE,EAAE,qBAAqB,KAAK,IAAI,GAAG,OAAO,CAAC,IAAI,CAAC,CAAC;IAChE,aAAa,CAAC,EAAE,CAAC,EAAE,EAAE,yBAAyB,KAAK,IAAI,CAAC;CACxD,CAAC;AAEF,MAAM,MAAM,gBAAgB,GAAG;IAC9B,KAAK,CAAC,EAAE,cAAc,CAAC;CACvB,CAAC;AA2BF,qBAAa,SAAS;;gBAWpB,KAAK,EAAE,UAAU,EACjB,KAAK,EAAE,UAAU,EACjB,QAAQ,EAAE,MAAM,EAChB,IAAI,GAAE;QAAE,OAAO,CAAC,EAAE,aAAa,CAAC;QAAC,MAAM,CAAC,EAAE,QAAQ,CAAC;QAAC,cAAc,CAAC,EAAE,cAAc,CAAA;KAAO;IAwKrF,GAAG,CAAC,MAAM,EAAE,MAAM,EAAE,QAAQ,GAAE,MAAW,EAAE,IAAI,GAAE,gBAAqB,GAAG,OAAO,CAAC,SAAS,CAAC;CAiMjG"}
+{"version":3,"file":"dag_runner.d.ts","sourceRoot":"","sources":["../src/dag_runner.ts"],"names":[],"mappings":"AAGA,OAAO,EAEN,KAAK,QAAQ,EAKb,MAAM,uBAAuB,CAAC;AAC/B,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,mBAAmB,CAAC;AACpD,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,mBAAmB,CAAC;AACpD,OAAO,KAAK,EAAE,cAAc,EAAuB,MAAM,uBAAuB,CAAC;AAGjF,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,iBAAiB,CAAC;AAGrD,MAAM,MAAM,SAAS,GAAG;IACvB,MAAM,EAAE,YAAY,GAAG,oBAAoB,GAAG,qBAAqB,GAAG,OAAO,CAAC;IAC9E,KAAK,EAAE,MAAM,CAAC;IACd,KAAK,EAAE,MAAM,CAAC;CACd,CAAC;AAEF,MAAM,MAAM,uBAAuB,GAAG;IACrC,MAAM,EAAE,MAAM,CAAC;IACf,IAAI,EAAE,MAAM,CAAC;IACb,OAAO,EAAE,MAAM,CAAC;IAChB,IAAI,EAAE,MAAM,GAAG,IAAI,CAAC;IACpB,KAAK,EAAE,MAAM,CAAC;CACd,CAAC;AAEF,MAAM,MAAM,qBAAqB,GAAG;IACnC,MAAM,EAAE,MAAM,CAAC;IACf,IAAI,EAAE,MAAM,CAAC;IACb,OAAO,EAAE,MAAM,CAAC;IAChB,QAAQ,EAAE,MAAM,CAAC;IACjB,QAAQ,EAAE,MAAM,CAAC;IACjB,OAAO,EAAE,MAAM,GAAG,IAAI,CAAC;CACvB,CAAC;AAEF,MAAM,MAAM,yBAAyB,GAAG;IACvC,MAAM,EAAE,MAAM,CAAC;IACf,IAAI,EAAE,MAAM,CAAC;IACb,OAAO,EAAE,MAAM,CAAC;IAChB,SAAS,EAAE,MAAM,CAAC;IAClB,IAAI,EAAE,MAAM,CAAC;CACb,CAAC;AAEF,MAAM,MAAM,cAAc,GAAG;IAC5B,WAAW,CAAC,EAAE,CAAC,EAAE,EAAE,uBAAuB,KAAK,IAAI,GAAG,OAAO,CAAC,IAAI,CAAC,CAAC;IACpE,SAAS,CAAC,EAAE,CAAC,EAAE,EAAE,qBAAqB,KAAK,IAAI,GAAG,OAAO,CAAC,IAAI,CAAC,CAAC;IAChE,aAAa,CAAC,EAAE,CAAC,EAAE,EAAE,yBAAyB,KAAK,IAAI,CAAC;CACxD,CAAC;AAEF,MAAM,MAAM,gBAAgB,GAAG;IAC9B,KAAK,CAAC,EAAE,cAAc,CAAC;CACvB,CAAC;AAkBF,qBAAa,SAAS;;gBAYpB,KAAK,EAAE,UAAU,EACjB,KAAK,EAAE,UAAU,EACjB,QAAQ,EAAE,MAAM,EAChB,IAAI,GAAE;QAAE,OAAO,CAAC,EAAE,aAAa,CAAC;QAAC,MAAM,CAAC,EAAE,QAAQ,CAAC;QAAC,cAAc,CAAC,EAAE,cAAc,CAAA;KAAO;IAiLrF,GAAG,CAAC,MAAM,EAAE,MAAM,EAAE,QAAQ,GAAE,MAAW,EAAE,IAAI,GAAE,gBAAqB,GAAG,OAAO,CAAC,SAAS,CAAC;CAiNjG"}

package/dist/dag_runner.js

@@ -2,20 +2,12 @@ import { mkdir } from "node:fs/promises";
 import { join, relative } from "node:path";
 import { currentRunId, fsEventLogFromRepoRoot, getStorePaths, newRunId, runContext, } from "@femtomc/mu-core/node";
 import { resolveModelConfig } from "./model_resolution.js";
-import { parseMuRole, systemPromptForRole } from "./mu_roles.js";
+import { roleFromTags, systemPromptForRole } from "./mu_roles.js";
 import { PiSdkBackend } from "./pi_sdk_backend.js";
 function roundTo(n, digits) {
     const f = 10 ** digits;
     return Math.round(n * f) / f;
 }
-function specRoleFromExecutionSpec(execution_spec) {
-    const role = execution_spec?.role;
-    if (typeof role !== "string") {
-        return null;
-    }
-    const trimmed = role.trim();
-    return trimmed.length > 0 ? trimmed : null;
-}
 function relPath(repoRoot, path) {
     try {
         const rel = relative(repoRoot, path);
@@ -33,6 +25,7 @@ export class DagRunner {
     #backend;
     #modelOverrides;
     #reorchestrateOutcomes = new Set(["failure", "needs_work"]);
+    #attempts = new Map();
     constructor(store, forum, repoRoot, opts = {}) {
         this.#store = store;
         this.#forum = forum;
@@ -41,11 +34,10 @@ export class DagRunner {
         this.#backend = opts.backend ?? new PiSdkBackend();
         this.#modelOverrides = opts.modelOverrides ?? {};
     }
-    async #resolveConfig(issue) {
-        void issue;
+    async #resolveConfig() {
         return resolveModelConfig(this.#modelOverrides);
     }
-    async #renderUserPrompt(issue, rootId, step) {
+    async #renderUserPrompt(issue, rootId, step, attempt = 1) {
         let rendered = issue.title ?? "";
         if (issue.body) {
             rendered += `\n\n${issue.body}`;
@@ -55,12 +47,15 @@ export class DagRunner {
         if (runId) {
             rendered += `Run: ${runId}\n`;
         }
+        if (attempt > 1) {
+            rendered += `\nAttempt: ${attempt} (previous attempt failed — check \`mu forum read issue:${issue.id}\` for context)\n`;
+        }
         return rendered;
     }
     async #executeBackend(issue, cfg, rootId, step, opts = {}) {
-        const role = parseMuRole(specRoleFromExecutionSpec(issue.execution_spec));
+        const role = roleFromTags(issue.tags);
         const logSuffix = opts.logSuffix ?? "";
-        const rendered = await this.#renderUserPrompt(issue, rootId, step);
+        const rendered = await this.#renderUserPrompt(issue, rootId, step, opts.attempt ?? 1);
         const systemPrompt = systemPromptForRole(role);
         const { logsDir } = getStorePaths(this.#repoRoot);
         await mkdir(logsDir, { recursive: true });
@@ -116,7 +111,7 @@ export class DagRunner {
         const reopened = await this.#store.update(issueId, {
             status: "open",
             outcome: null,
-            execution_spec: { role: "orchestrator" },
+            tags: [...before.tags.filter((t) => !t.startsWith("role:")), "role:orchestrator"],
         });
         await this.#events.emit("dag.unstick.reopen", {
             source: "dag_runner",
@@ -152,6 +147,9 @@ export class DagRunner {
                 continue;
             if (row.status !== "closed")
                 continue;
+            // Circuit breaker: skip issues that have exhausted their attempts.
+            if ((this.#attempts.get(row.id) ?? 0) >= 3)
+                continue;
             const outcome = row.outcome;
             if (outcome && this.#reorchestrateOutcomes.has(outcome)) {
                 if (hasOpenChildren(row.id))
@@ -215,9 +213,8 @@ export class DagRunner {
                             ...rootIssue,
                             title: `Repair stuck DAG: ${rootIssue.title}`,
                             body: `${(rootIssue.body || "").trim()}\n\n## Runner Diagnostics\n\n${diag}`.trim(),
-                            execution_spec: null,
                         };
-                        const cfg = await this.#resolveConfig(repairIssue);
+                        const cfg = await this.#resolveConfig();
                         const logSuffix = "unstick";
                         const onBackendLine = hooks?.onBackendLine;
                         const { exitCode, elapsedS } = await this.#executeBackend(repairIssue, cfg, rootId, step, {
@@ -243,8 +240,7 @@ export class DagRunner {
                     }
                     const issue = candidates[0];
                     const issueId = issue.id;
-                    // Validate role early so we don't claim/work an unsupported leaf.
-                    const role = parseMuRole(specRoleFromExecutionSpec(issue.execution_spec));
+                    const role = roleFromTags(issue.tags);
                     await this.#events.emit("dag.step.start", {
                         source: "dag_runner",
                         issueId,
@@ -260,12 +256,16 @@ export class DagRunner {
                         payload: { root_id: rootId, step },
                     });
                     await this.#store.claim(issueId);
+                    // Track attempt count for circuit breaker.
+                    const attempt = (this.#attempts.get(issueId) ?? 0) + 1;
+                    this.#attempts.set(issueId, attempt);
                     // 4. Route + 5. Render + 6. Execute.
-                    const cfg = await this.#resolveConfig(issue);
-                    const logSuffix = "";
+                    const cfg = await this.#resolveConfig();
+                    const logSuffix = attempt > 1 ? `attempt-${attempt}` : "";
                     const onBackendLine = hooks?.onBackendLine;
                     const { exitCode, elapsedS } = await this.#executeBackend(issue, cfg, rootId, step, {
                         logSuffix,
+                        attempt,
                         onLine: onBackendLine
                             ? (line) => onBackendLine({ rootId, step, issueId, logSuffix, line })
                             : undefined,
@@ -277,6 +277,11 @@ export class DagRunner {
                         return final;
                     }
                     if (updated.status !== "closed") {
+                        await this.#events.emit("dag.step.force_close", {
+                            source: "dag_runner",
+                            issueId,
+                            payload: { root_id: rootId, step, role, attempt, reason: "agent_did_not_close" },
+                        });
                         updated = await this.#store.close(issueId, "failure");
                     }
                     // 8. Log to forum.
@@ -309,9 +314,18 @@ export class DagRunner {
                             outcome: updated.outcome,
                         },
                     });
-                    // 9. Re-orchestrate on failure / needs_work.
+                    // 9. Re-orchestrate on failure / needs_work (circuit breaker: max 3 attempts).
                     if (updated.outcome && this.#reorchestrateOutcomes.has(updated.outcome)) {
-                        await this.#reopenForOrchestration(issueId, { reason: `outcome=${updated.outcome}`, step });
+                        if (attempt < 3) {
+                            await this.#reopenForOrchestration(issueId, { reason: `outcome=${updated.outcome}`, step });
+                        }
+                        else {
+                            await this.#events.emit("dag.circuit_breaker", {
+                                source: "dag_runner",
+                                issueId,
+                                payload: { root_id: rootId, step, attempt, outcome: updated.outcome },
+                            });
+                        }
                     }
                 }
                 final = { status: "max_steps_exhausted", steps: maxSteps, error: "" };

package/dist/mu_roles.d.ts

@@ -1,4 +1,5 @@
 export type MuRole = "orchestrator" | "worker";
-export declare function parseMuRole(role: string | null | undefined): MuRole;
+/** Determine role from tags. Defaults to orchestrator if no role tag present. */
+export declare function roleFromTags(tags: readonly string[]): MuRole;
 export declare function systemPromptForRole(role: MuRole): string;
 //# sourceMappingURL=mu_roles.d.ts.map

package/dist/mu_roles.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"mu_roles.d.ts","sourceRoot":"","sources":["../src/mu_roles.ts"],"names":[],"mappings":"AAAA,MAAM,MAAM,MAAM,GAAG,cAAc,GAAG,QAAQ,CAAC;AAE/C,wBAAgB,WAAW,CAAC,IAAI,EAAE,MAAM,GAAG,IAAI,GAAG,SAAS,GAAG,MAAM,CAanE;AA2DD,wBAAgB,mBAAmB,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,CA2DxD"}
+{"version":3,"file":"mu_roles.d.ts","sourceRoot":"","sources":["../src/mu_roles.ts"],"names":[],"mappings":"AAAA,MAAM,MAAM,MAAM,GAAG,cAAc,GAAG,QAAQ,CAAC;AAE/C,iFAAiF;AACjF,wBAAgB,YAAY,CAAC,IAAI,EAAE,SAAS,MAAM,EAAE,GAAG,MAAM,CAM5D;AA2DD,wBAAgB,mBAAmB,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,CAuExD"}

package/dist/mu_roles.js

@@ -1,12 +1,12 @@
-export function parseMuRole(role) {
-    if (role == null) {
-        return "orchestrator";
+/** Determine role from tags. Defaults to orchestrator if no role tag present. */
+export function roleFromTags(tags) {
+    for (const tag of tags) {
+        if (tag === "role:worker")
+            return "worker";
+        if (tag === "role:orchestrator")
+            return "orchestrator";
     }
-    const trimmed = role.trim();
-    if (trimmed === "orchestrator" || trimmed === "worker") {
-        return trimmed;
-    }
-    throw new Error(`unsupported execution_spec.role=${JSON.stringify(trimmed)} (only "orchestrator" and "worker" are supported)`);
+    return "orchestrator";
 }
 /* ------------------------------------------------------------------ */
 /*  mu CLI reference                                                   */
@@ -90,6 +90,18 @@ export function systemPromptForRole(role) {
             "- Assign executable leaves to role=worker.",
             "- If a child requires further decomposition, assign role=orchestrator.",
             "",
+            "## Review Pattern",
+            "",
+            "For work that benefits from verification, create a review issue:",
+            "1. Create worker issues for implementation.",
+            "2. Create a review issue (`--role orchestrator`) blocked by the workers.",
+            "3. The review runs after workers complete — verify, test, or expand further.",
+            "",
+            "Example:",
+            '  mu issues create "Implement X" --parent <id> --role worker',
+            '  mu issues create "Review X" --parent <id> --role orchestrator',
+            "  mu issues dep <worker-id> blocks <review-id>",
+            "",
             MU_CLI_REFERENCE,
         ].join("\n");
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@femtomc/mu-orchestrator",
-  "version": "26.2.19",
+  "version": "26.2.21",
   "type": "module",
   "main": "./dist/index.js",
   "types": "./dist/index.d.ts",
@@ -14,9 +14,9 @@
     "dist/**"
   ],
   "dependencies": {
-    "@femtomc/mu-core": "26.2.19",
-    "@femtomc/mu-forum": "26.2.19",
-    "@femtomc/mu-issue": "26.2.19",
+    "@femtomc/mu-core": "26.2.21",
+    "@femtomc/mu-forum": "26.2.21",
+    "@femtomc/mu-issue": "26.2.21",
     "@mariozechner/pi-agent-core": "^0.52.12",
     "@mariozechner/pi-coding-agent": "^0.52.12",
     "@mariozechner/pi-ai": "^0.52.12"