@entelligentsia/forgecli 0.19.0 → 0.20.3

package/dist/extensions/forgecli/fix-bug.js

@@ -36,29 +36,29 @@
 //   Reference: lib/orchestrator-preflight.ts (N-H-H, FORGE-S25-T17).
 //
 // N-H-E tag: see inline comment at the materialization skip (~line 707 / checkMaterialization).
+import { spawnSync } from "node:child_process";
 import * as fs from "node:fs";
 import * as path from "node:path";
 import { fileURLToPath } from "node:url";
-import { spawnSync } from "node:child_process";
+import { assertAudience, CallerContextStore } from "./audience-gate.js";
 // ModelRegistry/AuthStorage no longer instantiated here — use ctx.modelRegistry
 // so extension-registered providers (registered against the live session) are
 // visible to validateModelConfig. Creating a fresh registry here would miss
 // them and produce spurious MODEL_UNAVAILABLE warnings (FORGE-BUG-001).
 import { loadLayeredConfig } from "./config-layer.js";
-import { resolveModelForPhase } from "./model-resolver.js";
-import { runOrchestratorPreflight } from "./lib/orchestrator-preflight.js";
-import { assertAudience, CallerContextStore } from "./audience-gate.js";
-import { resolveToCanonicalId, resolveToolDir } from "./store-resolver.js";
-import { checkMaterialization } from "./lib/manifest-checker.js";
-import { readPersonaDir as readPersonaDirBug, readPipelineNames as readPipelineNamesBug } from "./lib/catalog-helpers.js";
 import { loadForgePersona, runForgeSubagent } from "./forge-subagent.js";
 import { getSubagentTools } from "./forge-tools.js";
+import { readPersonaDir as readPersonaDirBug, readPipelineNames as readPipelineNamesBug, } from "./lib/catalog-helpers.js";
 import { discoverForgeConfigCached } from "./lib/forge-config.js";
+import { checkMaterialization } from "./lib/manifest-checker.js";
+import { runOrchestratorPreflight } from "./lib/orchestrator-preflight.js";
+import { resolveModelForPhase } from "./model-resolver.js";
 import { loadWorkflow } from "./parsers/workflow-loader.js";
+import { buildPhaseEvent, drainFrictionFile, emitEvent, findPredecessorIndex, formatLocalTime, isNonInteractive, judgementFromSummary, runPreflightGate, validateId, } from "./run-task.js";
 import { getSessionRegistry } from "./session-registry.js";
+import { resolveToCanonicalId, resolveToolDir } from "./store-resolver.js";
 import { attachViewportObserver } from "./viewport-events.js";
 import { fmtPhaseSummary } from "./viewport-renderer.js";
-import { validateId, isNonInteractive, formatLocalTime, emitEvent, findPredecessorIndex, runPreflightGate, buildPhaseEvent, drainFrictionFile, judgementFromSummary, } from "./run-task.js";
 // ── Bug phase descriptor table ──────────────────────────────────────────────
 //
 // Decoded from .forge/workflows/fix_bug.md and the task prompt's BUG_PHASES.
@@ -79,25 +79,25 @@ export const BUG_PHASES = [
 // Phases mapped to null use update-status bug instead of set-bug-summary
 // for verdict tracking (Option B).
 export const BUG_SUMMARY_KEY_BY_ROLE = {
-    "triage": "triage",
+    triage: "triage",
     "plan-fix": "plan",
     "review-plan": "review_plan",
-    "implement": "implementation",
+    implement: "implementation",
     "review-code": "code_review",
-    "approve": "approve", // read from bug.summaries.approve (set-bug-summary)
-    "commit": null, // commit transitions bug.status → fixed (terminal), no summaries entry
+    approve: "approve", // read from bug.summaries.approve (set-bug-summary)
+    commit: null, // commit transitions bug.status → fixed (terminal), no summaries entry
 };
 // Bug-event type tokens — explicit mapping per review finding #3.
 // Non-review phases always emit the pass token. Review phases select
 // pass or fail based on ec.judgement.verdict.
 export const BUG_TYPE_TOKENS = {
-    "triage": { pass: "bug-triaged", fail: "bug-triaged" },
+    triage: { pass: "bug-triaged", fail: "bug-triaged" },
     "plan-fix": { pass: "fix-planned", fail: "fix-planned" },
     "review-plan": { pass: "fix-review-passed", fail: "fix-review-failed" },
-    "implement": { pass: "fix-implemented", fail: "fix-implemented" },
+    implement: { pass: "fix-implemented", fail: "fix-implemented" },
     "review-code": { pass: "fix-code-review-passed", fail: "fix-code-review-failed" },
-    "approve": { pass: "fix-approved", fail: "fix-revision-requested" },
-    "commit": { pass: "bug-committed", fail: "bug-commit-failed" },
+    approve: { pass: "fix-approved", fail: "fix-revision-requested" },
+    commit: { pass: "bug-committed", fail: "bug-commit-failed" },
 };
 // ── Bug FSM transitions ────────────────────────────────────────────────────
 // Mirrors store-cli BUG_TRANSITIONS. Terminal: `fixed`.
@@ -144,9 +144,7 @@ export function readBugState(cwd, bugId, sessionId) {
                     bestFile = fp;
                 }
             }
-            catch {
-                continue;
-            }
+            catch { }
         }
     }
     catch {
@@ -183,7 +181,9 @@ export function deleteBugState(cwd, bugId) {
                 try {
                     fs.unlinkSync(path.join(cacheDir, entry));
                 }
-                catch { /* non-fatal */ }
+                catch {
+                    /* non-fatal */
+                }
             }
         }
     }
@@ -228,7 +228,9 @@ export function assignNextBugId(storeCli, cwd) {
                 }
             }
         }
-        catch { /* empty store — start from 1 */ }
+        catch {
+            /* empty store — start from 1 */
+        }
     }
     const next = maxNum + 1;
     return `FORGE-BUG-${String(next).padStart(3, "0")}`;
@@ -322,13 +324,13 @@ export function composeBugBody(subWorkflowMd, bugId, phaseRole, bugStatusBeforeP
         `  node "$FORGE_ROOT/tools/store-cli.cjs" set-bug-summary ${bugId} approve <APPROVE-SUMMARY.json>`,
         `  The summary's "verdict" field MUST be "approved" or "revision". The downstream commit gate reads this, not bug.status.`,
         `- Commit phase: on successful git commit, run \`node "$FORGE_ROOT/tools/store-cli.cjs" update-status bug ${bugId} status fixed\` (terminal).`,
-        `- Do NOT write \"approved\" or \"verified\" to bug.status — those values were removed from the schema in forge v0.44.0.`,
-        `- Do NOT reference task-specific status values (e.g., \"committed\") or task entity kind.`,
+        `- Do NOT write "approved" or "verified" to bug.status — those values were removed from the schema in forge v0.44.0.`,
+        `- Do NOT reference task-specific status values (e.g., "committed") or task entity kind.`,
         "- CRITICAL: All `set-summary` calls must use `set-bug-summary` (not `set-summary`).",
         `  e.g. node "$FORGE_ROOT/tools/store-cli.cjs" set-bug-summary ${bugId} review_plan <jsonFile>`,
         `- Preflight gate: use \`--bug\` flag (not \`--task\`). e.g. node "$FORGE_ROOT/tools/preflight-gate.cjs" --phase review-plan --bug ${bugId}`,
         "- Skip re-running preflight-gate — the orchestrator already checked it. Proceed directly to the review.",
-        "Any workflow text that says \"task\" should be read as \"bug\" for this context.",
+        'Any workflow text that says "task" should be read as "bug" for this context.',
     ];
     // Phase-specific reinforcement when the orchestrator can name the current status.
     if (phaseRole === "approve" && bugStatusBeforePhase) {
@@ -338,7 +340,7 @@ export function composeBugBody(subWorkflowMd, bugId, phaseRole, bugStatusBeforeP
         entityKindLines.push(`- Commit phase: after the git commit lands, transition bug.status from '${bugStatusBeforePhase}' to 'fixed'.`);
     }
     if (phaseRole === "triage") {
-        entityKindLines.push("- Triage phase: in addition to writing TRIAGE.md and TRIAGE-SUMMARY.json, the summary MUST include a `route` field set to `\"A\"` or `\"B\"`.", "  Path A (short-circuit): severity == minor AND single-file fix ≤ ~20 lines AND no schema/API/migration AND regression test obvious from repro.", "  Path B (default): everything else. When in doubt, choose B.", "  The orchestrator reads bug.summaries.triage.route to select the downstream phase list.");
+        entityKindLines.push('- Triage phase: in addition to writing TRIAGE.md and TRIAGE-SUMMARY.json, the summary MUST include a `route` field set to `"A"` or `"B"`.', "  Path A (short-circuit): severity == minor AND single-file fix ≤ ~20 lines AND no schema/API/migration AND regression test obvious from repro.", "  Path B (default): everything else. When in doubt, choose B.", "  The orchestrator reads bug.summaries.triage.route to select the downstream phase list.");
     }
     return [
         `Read the workflow below and follow it. Bug ID: ${bugId}.`,
@@ -414,14 +416,14 @@ export function extractBugIdFromEvents(events) {
 const STATUS_KEY = "forge:fix-bug";
 const MESSAGE_KEY = "forge:fix-bug:message";
 export async function runBugPipeline(opts) {
-    const { bugId: initialBugId, originalArg, isNewBug, cwd, ctx, forgeRoot, storeCli, preflightGate, registry, resumeFromState } = opts;
+    const { bugId: initialBugId, originalArg, isNewBug, cwd, ctx, forgeRoot, storeCli, preflightGate, registry, resumeFromState, } = opts;
     // Mutable bugId — for new bugs, pre-assign a real FORGE-BUG-NNN ID
     // before triage so the subagent never needs to create or discover one.
     // This replaces the fragile PENDING→capture pattern where the subagent was
     // expected to create the bug record and we'd fish the ID from events.
     let bugId = initialBugId;
     let currentPhaseIndex = resumeFromState?.phaseIndex ?? 0;
-    let iterationCounts = resumeFromState?.iterationCounts ?? {};
+    const iterationCounts = resumeFromState?.iterationCounts ?? {};
     let lastModel;
     let lastProvider;
     // ── Per-persona model routing (Plan 16) ─────────────────────────────────
@@ -436,7 +438,12 @@ export async function runBugPipeline(opts) {
         for (const e of layeredConfigErrors) {
             ctx.ui.notify(`× forge:fix-bug — forge-cli config schema error: ${e}`, "error");
         }
-        return { status: "failed", lastPhaseIndex: currentPhaseIndex, iterationCounts, lastError: `forge-cli config schema errors: ${layeredConfigErrors.join("; ")}` };
+        return {
+            status: "failed",
+            lastPhaseIndex: currentPhaseIndex,
+            iterationCounts,
+            lastError: `forge-cli config schema errors: ${layeredConfigErrors.join("; ")}`,
+        };
     }
     // Pre-flight validation — same shape as run-task / run-sprint.
     // FORGE-S25-T17: delegated to lib/orchestrator-preflight.ts (H-13).
@@ -484,7 +491,12 @@ export async function runBugPipeline(opts) {
         const phase = BUG_PHASES[currentPhaseIndex];
         if (!phase) {
             ctx.ui.notify(`× forge:fix-bug — invalid phase index ${currentPhaseIndex}`, "error");
-            return { status: "failed", lastPhaseIndex: currentPhaseIndex, iterationCounts, lastError: `invalid phase index ${currentPhaseIndex}` };
+            return {
+                status: "failed",
+                lastPhaseIndex: currentPhaseIndex,
+                iterationCounts,
+                lastError: `invalid phase index ${currentPhaseIndex}`,
+            };
         }
         ctx.ui.setStatus?.(STATUS_KEY, `fix-bug ${bugId}: phase ${currentPhaseIndex + 1}/${BUG_PHASES.length} (${phase.role})`);
         ctx.ui.notify(`→ ${bugId}: ${phase.role} (phase ${currentPhaseIndex + 1}/${BUG_PHASES.length})`, "info");
@@ -508,7 +520,12 @@ export async function runBugPipeline(opts) {
                 lastError: `sub-workflow read failed: ${e.message ?? "unknown"}`,
                 savedAt: new Date().toISOString(),
             });
-            return { status: "failed", lastPhaseIndex: currentPhaseIndex, iterationCounts, lastError: `sub-workflow read failed: ${e.message ?? "unknown"}` };
+            return {
+                status: "failed",
+                lastPhaseIndex: currentPhaseIndex,
+                iterationCounts,
+                lastError: `sub-workflow read failed: ${e.message ?? "unknown"}`,
+            };
         }
         // ── 6a. Phase skip (state-aware, defense-in-depth) ─────────────
         // Belt-and-suspenders alongside the explicit summaries.triage.route
@@ -523,8 +540,8 @@ export async function runBugPipeline(opts) {
         // removed.
         const PHASE_SKIP_STATES = {
             "plan-fix": new Set(["fixed"]),
-            "implement": new Set(["fixed"]),
-            "commit": new Set(["fixed"]), // commit writes the terminal status; skip if already there
+            implement: new Set(["fixed"]),
+            commit: new Set(["fixed"]), // commit writes the terminal status; skip if already there
         };
         const bugNow = readBugRecord(bugId, storeCli, cwd);
         const skipStates = PHASE_SKIP_STATES[phase.role];
@@ -546,14 +563,19 @@ export async function runBugPipeline(opts) {
                 };
                 const synthFile = path.join(cwd, ".forge", "cache", `synthetic-summary-${bugId}-${summaryKey}.json`);
                 fs.writeFileSync(synthFile, JSON.stringify(synthSummary, null, 2), "utf8");
-                const synthResult = spawnSync("node", [storeCli, "set-bug-summary", bugId, summaryKey, synthFile], { cwd, encoding: "utf8" });
+                const synthResult = spawnSync("node", [storeCli, "set-bug-summary", bugId, summaryKey, synthFile], {
+                    cwd,
+                    encoding: "utf8",
+                });
                 if (synthResult.status !== 0) {
                     ctx.ui.notify(`⚠ forge:fix-bug — synthetic summary write failed for ${phase.role}: ${String(synthResult.stderr).trim()}`, "warning");
                 }
                 try {
                     fs.unlinkSync(synthFile);
                 }
-                catch { /* non-fatal */ }
+                catch {
+                    /* non-fatal */
+                }
             }
             currentPhaseIndex++;
             continue;
@@ -583,7 +605,12 @@ export async function runBugPipeline(opts) {
                     lastError: `preflight gate exit 1 for ${phase.role}`,
                     savedAt: new Date().toISOString(),
                 });
-                return { status: "halted", lastPhaseIndex: currentPhaseIndex, iterationCounts, lastError: `preflight gate exit 1 for ${phase.role}` };
+                return {
+                    status: "halted",
+                    lastPhaseIndex: currentPhaseIndex,
+                    iterationCounts,
+                    lastError: `preflight gate exit 1 for ${phase.role}`,
+                };
             }
             if (preflightResult === "escalate") {
                 ctx.ui.notify(`× forge:fix-bug — preflight gate escalated for phase ${phase.role} (exit 2); manual intervention required.`, "error");
@@ -595,7 +622,12 @@ export async function runBugPipeline(opts) {
                     lastError: `preflight gate exit 2 (escalate) for ${phase.role}`,
                     savedAt: new Date().toISOString(),
                 });
-                return { status: "escalated", lastPhaseIndex: currentPhaseIndex, iterationCounts, lastError: `preflight gate exit 2 (escalate) for ${phase.role}` };
+                return {
+                    status: "escalated",
+                    lastPhaseIndex: currentPhaseIndex,
+                    iterationCounts,
+                    lastError: `preflight gate exit 2 (escalate) for ${phase.role}`,
+                };
             }
         }
         // ── 6. Materialization-marker check ───────────────────────────
@@ -611,7 +643,12 @@ export async function runBugPipeline(opts) {
                 for (const marker of markerCheck.missing) {
                     ctx.ui.notify(`× workflow regression: ${marker} not found in ${subWorkflowPath}`, "error");
                 }
-                return { status: "failed", lastPhaseIndex: currentPhaseIndex, iterationCounts, lastError: `materialization markers missing: ${markerCheck.missing.join(", ")}` };
+                return {
+                    status: "failed",
+                    lastPhaseIndex: currentPhaseIndex,
+                    iterationCounts,
+                    lastError: `materialization markers missing: ${markerCheck.missing.join(", ")}`,
+                };
             }
         }
         // ── 5. Audience check ─────────────────────────────────────────
@@ -620,7 +657,8 @@ export async function runBugPipeline(opts) {
         // Skip the audience gate for the monolithic fix_bug.md; only check the
         // true sub-workflows (review_plan, review_code, architect_approve, commit_task)
         // which the subagent does run directly.
-        const audienceOk = phase.workflowFile === "fix_bug" || CallerContextStore.asSubagent(() => assertAudience({ workflowName: phase.workflowFile, audience: subWorkflowAudience }, ctx));
+        const audienceOk = phase.workflowFile === "fix_bug" ||
+            CallerContextStore.asSubagent(() => assertAudience({ workflowName: phase.workflowFile, audience: subWorkflowAudience }, ctx));
         if (!audienceOk) {
             writeBugState(cwd, {
                 bugId,
@@ -630,7 +668,12 @@ export async function runBugPipeline(opts) {
                 lastError: `audience check failed for ${phase.workflowFile}`,
                 savedAt: new Date().toISOString(),
             });
-            return { status: "failed", lastPhaseIndex: currentPhaseIndex, iterationCounts, lastError: `audience check failed for ${phase.workflowFile}` };
+            return {
+                status: "failed",
+                lastPhaseIndex: currentPhaseIndex,
+                iterationCounts,
+                lastError: `audience check failed for ${phase.workflowFile}`,
+            };
         }
         // ── Persona load ──────────────────────────────────────────────
         let persona;
@@ -649,7 +692,12 @@ export async function runBugPipeline(opts) {
                 lastError: `persona load failed: ${e.message ?? "unknown"}`,
                 savedAt: new Date().toISOString(),
             });
-            return { status: "failed", lastPhaseIndex: currentPhaseIndex, iterationCounts, lastError: `persona load failed: ${e.message ?? "unknown"}` };
+            return {
+                status: "failed",
+                lastPhaseIndex: currentPhaseIndex,
+                iterationCounts,
+                lastError: `persona load failed: ${e.message ?? "unknown"}`,
+            };
         }
         // ── Read bug record for current status ────────────────────────
         // Skip for PENDING bugIds (bug doesn't exist yet).
@@ -691,7 +739,9 @@ export async function runBugPipeline(opts) {
                             fs.writeFileSync(debugLogPath, lines.slice(-keep).join("\n"), "utf8");
                         }
                     }
-                    catch { /* file may not exist yet */ }
+                    catch {
+                        /* file may not exist yet */
+                    }
                     fs.appendFileSync(debugLogPath, `${JSON.stringify({ ts: new Date().toISOString(), phase: phase.role, ...rec })}\n`, "utf8");
                 }
                 catch {
@@ -759,7 +809,9 @@ export async function runBugPipeline(opts) {
                 // Keeps every phase of this bug-fix pipeline in a single cache
                 // namespace so the system-prompt + persona prefix stays warm
                 // across the ~10-minute phases.
-                cacheSessionId: typeof bugRecordBefore?.sprintId === "string" ? `forge:${bugRecordBefore.sprintId}` : `forge:bug:${bugId}`,
+                cacheSessionId: typeof bugRecordBefore?.sprintId === "string"
+                    ? `forge:${bugRecordBefore.sprintId}`
+                    : `forge:bug:${bugId}`,
                 onEvent: onSubagentEvent,
                 requestedModel: modelResolution.model,
                 modelRegistry: ctx.modelRegistry,
@@ -778,7 +830,12 @@ export async function runBugPipeline(opts) {
                 lastError: `runForgeSubagent threw: ${e.message ?? "unknown"}`,
                 savedAt: new Date().toISOString(),
             });
-            return { status: "failed", lastPhaseIndex: currentPhaseIndex, iterationCounts, lastError: `runForgeSubagent threw: ${e.message ?? "unknown"}` };
+            return {
+                status: "failed",
+                lastPhaseIndex: currentPhaseIndex,
+                iterationCounts,
+                lastError: `runForgeSubagent threw: ${e.message ?? "unknown"}`,
+            };
         }
         // ── Post-subagent abort detection ─────────────────────────────────
         if (result.stopReason === "aborted" || opts.signal?.aborted) {
@@ -810,7 +867,12 @@ export async function runBugPipeline(opts) {
                 lastError: result.errorMessage ?? result.stopReason ?? "subagent exit non-zero",
                 savedAt: new Date().toISOString(),
             });
-            return { status: "failed", lastPhaseIndex: currentPhaseIndex, iterationCounts, lastError: result.errorMessage ?? result.stopReason ?? "subagent exit non-zero" };
+            return {
+                status: "failed",
+                lastPhaseIndex: currentPhaseIndex,
+                iterationCounts,
+                lastError: result.errorMessage ?? result.stopReason ?? "subagent exit non-zero",
+            };
         }
         // Capture model/provider from subagent result.
         if (result.model)
@@ -838,19 +900,29 @@ export async function runBugPipeline(opts) {
                             const recent = bugs
                                 .filter((b) => b.reportedAt && b.reportedAt >= pipelineStartIso)
                                 .sort((a, b) => String(b.reportedAt).localeCompare(String(a.reportedAt)))[0];
-                            if (recent && recent.bugId && typeof recent.bugId === "string" && recent.bugId.startsWith("FORGE-BUG-")) {
+                            if (recent &&
+                                recent.bugId &&
+                                typeof recent.bugId === "string" &&
+                                recent.bugId.startsWith("FORGE-BUG-")) {
                                 bugId = recent.bugId;
                                 ctx.ui.notify(`forge:fix-bug — captured bug ID via store fallback: ${bugId}`, "info");
                             }
                         }
                     }
-                    catch { /* parse failure — fall through to assertion */ }
+                    catch {
+                        /* parse failure — fall through to assertion */
+                    }
                 }
             }
             // Defensive guard: if bugId is still PENDING after triage, pipeline cannot proceed.
             if (bugId.startsWith("PENDING-")) {
                 ctx.ui.notify("× forge:fix-bug — failed to capture real bug ID after triage. Cannot proceed with PENDING placeholder.", "error");
-                return { status: "failed", lastPhaseIndex: currentPhaseIndex, iterationCounts, lastError: "bugId still PENDING after triage" };
+                return {
+                    status: "failed",
+                    lastPhaseIndex: currentPhaseIndex,
+                    iterationCounts,
+                    lastError: "bugId still PENDING after triage",
+                };
             }
             // Re-initialize debug log now that real bugId is available.
             if (!debugLogDisabled) {
@@ -868,7 +940,9 @@ export async function runBugPipeline(opts) {
                                 fs.writeFileSync(debugLogPath, lines.slice(-keep).join("\n"), "utf8");
                             }
                         }
-                        catch { /* file may not exist yet */ }
+                        catch {
+                            /* file may not exist yet */
+                        }
                         fs.appendFileSync(debugLogPath, `${JSON.stringify({ ts: new Date().toISOString(), phase: phase.role, ...rec })}\n`, "utf8");
                     }
                     catch {
@@ -981,7 +1055,12 @@ export async function runBugPipeline(opts) {
                     lastError: `verdict missing for ${phase.role}`,
                     savedAt: new Date().toISOString(),
                 });
-                return { status: "failed", lastPhaseIndex: currentPhaseIndex, iterationCounts, lastError: `verdict missing for ${phase.role}` };
+                return {
+                    status: "failed",
+                    lastPhaseIndex: currentPhaseIndex,
+                    iterationCounts,
+                    lastError: `verdict missing for ${phase.role}`,
+                };
             }
             if (verdict === "revision") {
                 iterationCounts[phase.role] = (iterationCounts[phase.role] ?? 0) + 1;
@@ -996,7 +1075,12 @@ export async function runBugPipeline(opts) {
                         lastError: `revision cap reached for ${phase.role}`,
                         savedAt: new Date().toISOString(),
                     });
-                    return { status: "escalated", lastPhaseIndex: currentPhaseIndex, iterationCounts, lastError: `revision cap reached for ${phase.role}` };
+                    return {
+                        status: "escalated",
+                        lastPhaseIndex: currentPhaseIndex,
+                        iterationCounts,
+                        lastError: `revision cap reached for ${phase.role}`,
+                    };
                 }
                 // Transition bug back to in-progress before re-dispatching implement.
                 // This is required for review-code → implement and approve → implement loops.
@@ -1065,7 +1149,13 @@ export async function runBugPipeline(opts) {
     }
     // ── All phases complete ───────────────────────────────────────────
     deleteBugState(cwd, bugId);
-    return { status: "completed", lastPhaseIndex: BUG_PHASES.length - 1, iterationCounts, model: lastModel, provider: lastProvider };
+    return {
+        status: "completed",
+        lastPhaseIndex: BUG_PHASES.length - 1,
+        iterationCounts,
+        model: lastModel,
+        provider: lastProvider,
+    };
 }
 export function registerFixBug(pi, options = {}) {
     pi.registerCommand("forge:fix-bug", {
@@ -1097,8 +1187,7 @@ export function registerFixBug(pi, options = {}) {
             let isNewBug = false;
             // Check if arg looks like it could be a bug ID (prefixed or unprefixed).
             // Covers: FORGE-BUG-042, BUG-042, B042.
-            const looksLikeBugId = /^(?:[A-Z0-9]+-)?(?:BUG-?\d+|B\d+)$/i.test(rawArg) ||
-                /^BUG-\d+$/i.test(rawArg);
+            const looksLikeBugId = /^(?:[A-Z0-9]+-)?(?:BUG-?\d+|B\d+)$/i.test(rawArg) || /^BUG-\d+$/i.test(rawArg);
             if (/^FORGE-BUG-\d+$/.test(rawArg)) {
                 // Canonical bug ID — verify it exists
                 bugId = rawArg;
@@ -1119,7 +1208,10 @@ export function registerFixBug(pi, options = {}) {
                 // Unprefixed bug ID — resolve through the store cascade.
                 // Issue #20: unprefixed entity IDs silently poisoned substitutions.
                 const toolDir = resolveToolDir(forgeRoot);
-                const resolvedBugId = await resolveToCanonicalId(rawArg, toolDir, cwd, "bug", { ctx, commandLabel: "forge:fix-bug" });
+                const resolvedBugId = await resolveToCanonicalId(rawArg, toolDir, cwd, "bug", {
+                    ctx,
+                    commandLabel: "forge:fix-bug",
+                });
                 if (!resolvedBugId) {
                     // Error already emitted by resolver
                     ctx.ui.setStatus?.(STATUS_KEY, undefined);
@@ -1189,9 +1281,7 @@ export function registerFixBug(pi, options = {}) {
                     // (explicit failure), halted=false (cancelled/interrupted), and
                     // any state with existing.status set.
                     const stateStatus = existing.status ?? (existing.halted ? "halted" : "interrupted");
-                    const statusLabel = stateStatus === "cancelled" ? "cancelled"
-                        : stateStatus === "halted" ? "halted"
-                            : "interrupted";
+                    const statusLabel = stateStatus === "cancelled" ? "cancelled" : stateStatus === "halted" ? "halted" : "interrupted";
                     const phaseRole = BUG_PHASES[existing.phaseIndex]?.role ?? existing.phaseIndex;
                     if (!isNonInteractive()) {
                         const resume = await ctx.ui.confirm(`Resume ${bugId}?`, `Cached state — phase ${existing.phaseIndex} (${phaseRole}), ${statusLabel}, ` +
@@ -1247,7 +1337,7 @@ export function registerFixBug(pi, options = {}) {
             // needs the real ID before startSession is called.
             if (isNewBug && bugId.startsWith("PENDING-")) {
                 const realBugId = assignNextBugId(storeCli, cwd);
-                const title = (rawArg && !rawArg.startsWith("@")) ? rawArg.slice(0, 120) : "New bug (pending triage)";
+                const title = rawArg && !rawArg.startsWith("@") ? rawArg.slice(0, 120) : "New bug (pending triage)";
                 if (preCreateBug(realBugId, title, storeCli, cwd)) {
                     ctx.ui.notify(`forge:fix-bug — pre-assigned bug ID: ${realBugId}`, "info");
                     bugId = realBugId;