@entelligentsia/forgecli 0.15.0 → 0.19.0

package/dist/extensions/forgecli/lib/catalog-types.js

@@ -0,0 +1,161 @@
+// lib/catalog-types.ts — FORGE-S25-T27
+//
+// Authoritative compile-time TS const tuples for domain-specific union types
+// in forge-cli. Zero IO — no fs reads, no subprocess calls.
+//
+// Sources of truth for each const tuple:
+//   - TASK/SPRINT/BUG_STATUS_VALUES: forge/forge/schemas/{task,sprint,bug}.schema.json
+//     status enums (same source as build-enum-catalog.cjs in FORGE-S25-T26).
+//     Verified at runtime by catalog-loader.test.ts (every catalog key appears here).
+//   - SYNTHETIC_EVENT_TYPES: forge-cli internal (hook-dispatcher.ts event taxonomy).
+//   - FRICTION_SUBKINDS: forge/forge/schemas/event.schema.json subkind pattern values
+//     (pattern: ^(skill_unused|skill_failed|skill_missing|skill_stale|skill_redundant|x_[a-z_]+)$).
+//   - ROLE_KINDS: orchestrate_task.md ROLE_TO_NOUN mapping + orchestrator role.
+//   - ACTION_KINDS: event.schema.json action usage + orchestrate_task.md event emission.
+//
+// Design: single file for all compile-time domain constants. When a new state,
+// event type, or role is added, edit this file and the corresponding source of
+// truth (plugin schema or orchestrate_task.md). Regression tests verify alignment.
+//
+// Iron Laws: IL1 (code only in forge-cli/), IL2 (TypeScript + no new deps), IL7 (no silent failures).
+// ── Task status ───────────────────────────────────────────────────────────────
+//
+// Source: forge/forge/schemas/task.schema.json properties.status.enum
+// Last verified: FORGE-S25-T26 (enum-catalog.json generated from this enum)
+export const TASK_STATUS_VALUES = [
+    "draft",
+    "planned",
+    "plan-approved",
+    "implementing",
+    "implemented",
+    "review-approved",
+    "approved",
+    "committed",
+    "plan-revision-required",
+    "code-revision-required",
+    "blocked",
+    "escalated",
+    "abandoned",
+];
+// ── Sprint status ─────────────────────────────────────────────────────────────
+//
+// Source: forge/forge/schemas/sprint.schema.json properties.status.enum
+export const SPRINT_STATUS_VALUES = [
+    "planning",
+    "active",
+    "completed",
+    "retrospective-done",
+    "blocked",
+    "partially-completed",
+    "abandoned",
+];
+// ── Bug status ────────────────────────────────────────────────────────────────
+//
+// Source: forge/forge/schemas/bug.schema.json properties.status.enum
+export const BUG_STATUS_VALUES = [
+    "reported",
+    "triaged",
+    "in-progress",
+    "fixed",
+];
+// ── Command name ──────────────────────────────────────────────────────────────
+//
+// Source: forge/forge/tools/build-enum-catalog.cjs COMMAND_NAMES
+// (subset of forge:* slash commands registered in the plugin)
+export const COMMAND_NAME_VALUES = [
+    "forge:sprint-intake",
+    "forge:plan",
+    "forge:review-plan",
+    "forge:implement",
+    "forge:review-code",
+    "forge:fix-bug",
+    "forge:sprint-plan",
+    "forge:run-task",
+    "forge:run-sprint",
+    "forge:collate",
+    "forge:retrospective",
+    "forge:approve",
+    "forge:commit",
+    "forge:enhance",
+    "forge:quiz-agent",
+    "forge:validate",
+    "forge:init",
+    "forge:health",
+    "forge:regenerate",
+    "forge:update",
+    "forge:add-task",
+    "forge:add-pipeline",
+    "forge:calibrate",
+    "forge:materialize",
+    "forge:remove",
+    "forge:report-bug",
+    "forge:store-query",
+    "forge:store-repair",
+    "forge:config",
+    "forge:ask",
+    "forge:store-custodian",
+    "forge:refresh-kb-links",
+];
+// ── Synthetic event type (forge-cli domain) ───────────────────────────────────
+//
+// Source: forge-cli hook-dispatcher.ts synthetic event taxonomy.
+// Discriminants of the SyntheticEvent union (InitCompleteEvent, etc.).
+// Verified by hook-dispatcher.test.ts.
+export const SYNTHETIC_EVENT_TYPES = [
+    "init-complete",
+    "sprint-collate-complete",
+    "migration-applied",
+];
+// ── Friction subkind ──────────────────────────────────────────────────────────
+//
+// Source: forge/forge/schemas/event.schema.json subkind pattern:
+//   ^(skill_unused|skill_failed|skill_missing|skill_stale|skill_redundant|x_[a-z_]+)$
+// The x_ prefix is for experimental subkinds (not listed here as compile-time values;
+// they are validated via the pattern at runtime by store-cli).
+// Verified by catalog-types.test.ts against the event schema regex.
+export const FRICTION_SUBKINDS = [
+    "skill_unused",
+    "skill_failed",
+    "skill_missing",
+    "skill_stale",
+    "skill_redundant",
+];
+// ── Role kind ─────────────────────────────────────────────────────────────────
+//
+// Source: forge/forge/.forge/workflows/orchestrate_task.md ROLE_TO_NOUN mapping
+// + orchestrator self-identification.
+// Used in: session-registry.ts (PhaseSummary.role), friction-emit.ts (FrictionEmitRuntime.role).
+export const ROLE_KINDS = [
+    "plan",
+    "implement",
+    "update-plan",
+    "update-impl",
+    "commit",
+    "review-plan",
+    "review-code",
+    "validate",
+    "approve",
+    "writeback",
+    "orchestrator",
+];
+// ── Action kind ───────────────────────────────────────────────────────────────
+//
+// Source: event.schema.json action field usage + orchestrate_task.md event emission.
+// Used in: friction-emit.ts (FrictionEmitRuntime.action), run-task.ts event emission.
+export const ACTION_KINDS = [
+    "start",
+    "complete",
+    "plan",
+    "implement",
+    "review",
+    "approve",
+    "commit",
+    "escalated",
+    "gate_failed",
+    "friction_drain",
+    "task_skipped",
+    "subagent_retry",
+    "subagent_escalated",
+    "verdict_malformed",
+];
+//# sourceMappingURL=catalog-types.js.map

package/dist/extensions/forgecli/lib/catalog-types.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"catalog-types.js","sourceRoot":"","sources":["../../../../src/extensions/forgecli/lib/catalog-types.ts"],"names":[],"mappings":"AAAA,uCAAuC;AACvC,EAAE;AACF,6EAA6E;AAC7E,4DAA4D;AAC5D,EAAE;AACF,yCAAyC;AACzC,uFAAuF;AACvF,6EAA6E;AAC7E,sFAAsF;AACtF,qFAAqF;AACrF,sFAAsF;AACtF,oGAAoG;AACpG,gFAAgF;AAChF,yFAAyF;AACzF,EAAE;AACF,+EAA+E;AAC/E,+EAA+E;AAC/E,mFAAmF;AACnF,EAAE;AACF,sGAAsG;AAEtG,iFAAiF;AACjF,EAAE;AACF,sEAAsE;AACtE,4EAA4E;AAE5E,MAAM,CAAC,MAAM,kBAAkB,GAAG;IACjC,OAAO;IACP,SAAS;IACT,eAAe;IACf,cAAc;IACd,aAAa;IACb,iBAAiB;IACjB,UAAU;IACV,WAAW;IACX,wBAAwB;IACxB,wBAAwB;IACxB,SAAS;IACT,WAAW;IACX,WAAW;CACiB,CAAC;AAI9B,iFAAiF;AACjF,EAAE;AACF,wEAAwE;AAExE,MAAM,CAAC,MAAM,oBAAoB,GAAG;IACnC,UAAU;IACV,QAAQ;IACR,WAAW;IACX,oBAAoB;IACpB,SAAS;IACT,qBAAqB;IACrB,WAAW;CACiB,CAAC;AAI9B,iFAAiF;AACjF,EAAE;AACF,qEAAqE;AAErE,MAAM,CAAC,MAAM,iBAAiB,GAAG;IAChC,UAAU;IACV,SAAS;IACT,aAAa;IACb,OAAO;CACqB,CAAC;AAI9B,iFAAiF;AACjF,EAAE;AACF,iEAAiE;AACjE,8DAA8D;AAE9D,MAAM,CAAC,MAAM,mBAAmB,GAAG;IAClC,qBAAqB;IACrB,YAAY;IACZ,mBAAmB;IACnB,iBAAiB;IACjB,mBAAmB;IACnB,eAAe;IACf,mBAAmB;IACnB,gBAAgB;IAChB,kBAAkB;IAClB,eAAe;IACf,qBAAqB;IACrB,eAAe;IACf,cAAc;IACd,eAAe;IACf,kBAAkB;IAClB,gBAAgB;IAChB,YAAY;IACZ,cAAc;IACd,kBAAkB;IAClB,cAAc;IACd,gBAAgB;IAChB,oBAAoB;IACpB,iBAAiB;IACjB,mBAAmB;IACnB,cAAc;IACd,kBAAkB;IAClB,mBAAmB;IACnB,oBAAoB;IACpB,cAAc;IACd,WAAW;IACX,uBAAuB;IACvB,wBAAwB;CACI,CAAC;AAI9B,iFAAiF;AACjF,EAAE;AACF,iEAAiE;AACjE,uEAAuE;AACvE,uCAAuC;AAEvC,MAAM,CAAC,MAAM,qBAAqB,GAAG;IACpC,eAAe;IACf,yBAAyB;IACzB,mBAAmB;CACS,CAAC;AAI9B,iFAAiF;AACjF,EAAE;AACF,iEAAiE;AACjE,sFAAsF;AACtF,sFAAsF;AACtF,+DAA+D;AAC/D,oEAAoE;AAEpE,MAAM,CAAC,MAAM,iBAAiB,GAAG;IAChC,cAAc;IACd,cAAc;IACd,eAAe;IACf,aAAa;IACb,iBAAiB;CACW,CAAC;AAI9B,iFAAiF;AACjF,EAAE;AACF,gFAAgF;AAChF,sCAAsC;AACtC,iGAAiG;AAEjG,MAAM,CAAC,MAAM,UAAU,GAAG;IACzB,MAAM;IACN,WAAW;IACX,aAAa;IACb,aAAa;IACb,QAAQ;IACR,aAAa;IACb,aAAa;IACb,UAAU;IACV,SAAS;IACT,WAAW;IACX,cAAc;CACc,CAAC;AAI9B,iFAAiF;AACjF,EAAE;AACF,qFAAqF;AACrF,sFAAsF;AAEtF,MAAM,CAAC,MAAM,YAAY,GAAG;IAC3B,OAAO;IACP,UAAU;IACV,MAAM;IACN,WAAW;IACX,QAAQ;IACR,SAAS;IACT,QAAQ;IACR,WAAW;IACX,aAAa;IACb,gBAAgB;IAChB,cAAc;IACd,gBAAgB;IAChB,oBAAoB;IACpB,mBAAmB;CACS,CAAC"}

package/dist/extensions/forgecli/lib/exec-helpers.d.ts

@@ -0,0 +1,25 @@
+import { execFile } from "node:child_process";
+import type { ExtensionCommandContext } from "@earendil-works/pi-coding-agent";
+/**
+ * Promisified `execFile`. Use instead of declaring `promisify(execFile)`
+ * locally in every module.
+ *
+ * Note: `forge-update-command.ts` deliberately keeps its own injectable
+ * `runner` override for test isolation — do NOT migrate that module here.
+ */
+export declare const execFileAsync: typeof execFile.__promisify__;
+/** Convenience type alias for callers that need to annotate the signature. */
+export type ExecFileAsyncType = typeof execFileAsync;
+/**
+ * Invoke a Node.js tool by path and wait for it to complete.
+ * Throws a descriptive error (including tool basename) on non-zero exit.
+ */
+export declare function runTool(toolPath: string, argv: string[], cwd: string, timeout?: number): Promise<void>;
+/**
+ * Invoke a Node.js tool in advisory mode: on failure, notify via `ctx.ui.notify`
+ * and return `false` instead of throwing.
+ *
+ * Returns `true` on success, `false` on failure. Callers that need hard failure
+ * should use `runTool` directly.
+ */
+export declare function runToolAdvisory(toolPath: string, argv: string[], cwd: string, ctx: ExtensionCommandContext, label: string, timeout?: number): Promise<boolean>;

package/dist/extensions/forgecli/lib/exec-helpers.js

@@ -0,0 +1,52 @@
+// lib/exec-helpers.ts — shared tool invocation helpers (N-C-E, FORGE-S25-T23 B-4).
+//
+// Centralises the `promisify(execFile)` declaration that was independently
+// copy-pasted in forge-tools.ts:24 and store-resolver.ts:14, and the
+// `runTool`/`runToolAdvisory` pair extracted from forge-init.ts.
+import * as path from "node:path";
+import { execFile } from "node:child_process";
+import { promisify } from "node:util";
+/**
+ * Promisified `execFile`. Use instead of declaring `promisify(execFile)`
+ * locally in every module.
+ *
+ * Note: `forge-update-command.ts` deliberately keeps its own injectable
+ * `runner` override for test isolation — do NOT migrate that module here.
+ */
+export const execFileAsync = promisify(execFile);
+/**
+ * Invoke a Node.js tool by path and wait for it to complete.
+ * Throws a descriptive error (including tool basename) on non-zero exit.
+ */
+export async function runTool(toolPath, argv, cwd, timeout = 30000) {
+    try {
+        await execFileAsync("node", [toolPath, ...argv], {
+            cwd,
+            timeout,
+            encoding: "utf8",
+        });
+    }
+    catch (err) {
+        const e = err;
+        throw new Error(`Tool ${path.basename(toolPath)} failed: ${e.stderr?.trim() || e.message || "unknown error"}`);
+    }
+}
+/**
+ * Invoke a Node.js tool in advisory mode: on failure, notify via `ctx.ui.notify`
+ * and return `false` instead of throwing.
+ *
+ * Returns `true` on success, `false` on failure. Callers that need hard failure
+ * should use `runTool` directly.
+ */
+export async function runToolAdvisory(toolPath, argv, cwd, ctx, label, timeout = 30000) {
+    try {
+        await runTool(toolPath, argv, cwd, timeout);
+        return true;
+    }
+    catch (err) {
+        const e = err;
+        ctx.ui.notify(`△ ${label}: ${e.message ?? "failed"} — proceeding.`, "warning");
+        return false;
+    }
+}
+//# sourceMappingURL=exec-helpers.js.map

package/dist/extensions/forgecli/lib/exec-helpers.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"exec-helpers.js","sourceRoot":"","sources":["../../../../src/extensions/forgecli/lib/exec-helpers.ts"],"names":[],"mappings":"AAAA,mFAAmF;AACnF,EAAE;AACF,2EAA2E;AAC3E,qEAAqE;AACrE,iEAAiE;AAEjE,OAAO,KAAK,IAAI,MAAM,WAAW,CAAC;AAClC,OAAO,EAAE,QAAQ,EAAE,MAAM,oBAAoB,CAAC;AAC9C,OAAO,EAAE,SAAS,EAAE,MAAM,WAAW,CAAC;AAGtC;;;;;;GAMG;AACH,MAAM,CAAC,MAAM,aAAa,GAAG,SAAS,CAAC,QAAQ,CAAC,CAAC;AAKjD;;;GAGG;AACH,MAAM,CAAC,KAAK,UAAU,OAAO,CAAC,QAAgB,EAAE,IAAc,EAAE,GAAW,EAAE,OAAO,GAAG,KAAK;IAC3F,IAAI,CAAC;QACJ,MAAM,aAAa,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE,GAAG,IAAI,CAAC,EAAE;YAChD,GAAG;YACH,OAAO;YACP,QAAQ,EAAE,MAAM;SAChB,CAAC,CAAC;IACJ,CAAC;IAAC,OAAO,GAAY,EAAE,CAAC;QACvB,MAAM,CAAC,GAAG,GAA4C,CAAC;QACvD,MAAM,IAAI,KAAK,CAAC,QAAQ,IAAI,CAAC,QAAQ,CAAC,QAAQ,CAAC,YAAY,CAAC,CAAC,MAAM,EAAE,IAAI,EAAE,IAAI,CAAC,CAAC,OAAO,IAAI,eAAe,EAAE,CAAC,CAAC;IAChH,CAAC;AACF,CAAC;AAED;;;;;;GAMG;AACH,MAAM,CAAC,KAAK,UAAU,eAAe,CACpC,QAAgB,EAChB,IAAc,EACd,GAAW,EACX,GAA4B,EAC5B,KAAa,EACb,OAAO,GAAG,KAAK;IAEf,IAAI,CAAC;QACJ,MAAM,OAAO,CAAC,QAAQ,EAAE,IAAI,EAAE,GAAG,EAAE,OAAO,CAAC,CAAC;QAC5C,OAAO,IAAI,CAAC;IACb,CAAC;IAAC,OAAO,GAAY,EAAE,CAAC;QACvB,MAAM,CAAC,GAAG,GAA2B,CAAC;QACtC,GAAG,CAAC,EAAE,CAAC,MAAM,CAAC,KAAK,KAAK,KAAK,CAAC,CAAC,OAAO,IAAI,QAAQ,gBAAgB,EAAE,SAAS,CAAC,CAAC;QAC/E,OAAO,KAAK,CAAC;IACd,CAAC;AACF,CAAC"}

package/dist/extensions/forgecli/lib/forge-config.d.ts

@@ -0,0 +1,20 @@
+import { type ForgeConfig } from "../forge-root.js";
+/**
+ * Memoized variant of `discoverForgeConfig`.
+ *
+ * On the first call for a given `cwd` the function walks the directory tree
+ * upward to locate `.forge/config.json`; subsequent calls for the same `cwd`
+ * return the cached result immediately.
+ *
+ * @param cwd - Working directory to start the search from.
+ *              Defaults to `process.cwd()` (same default as the underlying
+ *              `discoverForgeConfig`).
+ */
+export declare function discoverForgeConfigCached(cwd?: string): ForgeConfig | null;
+/**
+ * Clears all cached entries.
+ *
+ * Primarily for test isolation — call in `beforeEach`/`afterEach` to
+ * prevent cache bleed between test cases that change the filesystem.
+ */
+export declare function clearForgeConfigCache(): void;

package/dist/extensions/forgecli/lib/forge-config.js

@@ -0,0 +1,43 @@
+// lib/forge-config.ts — memoized Forge config accessor (C-3, S-13).
+//
+// Wraps discoverForgeConfig() with a per-cwd Map cache so repeated calls
+// during a single session don't re-walk the filesystem.
+//
+// CRITICAL INVARIANT: only `ForgeConfig | null` is cached. The `LayeredConfig`
+// produced by `loadLayeredConfig()` is NOT cached here — T20 must retain full
+// access to `LayeredConfig.errors` to surface the N-B-E issue. Callers that
+// need `LayeredConfig` must call `loadLayeredConfig()` directly; this module
+// optimises only the walk-up discovery phase.
+import { discoverForgeConfig } from "../forge-root.js";
+/** Module-level cache keyed by the `cwd` argument (or `""` for the default). */
+const cache = new Map();
+/**
+ * Memoized variant of `discoverForgeConfig`.
+ *
+ * On the first call for a given `cwd` the function walks the directory tree
+ * upward to locate `.forge/config.json`; subsequent calls for the same `cwd`
+ * return the cached result immediately.
+ *
+ * @param cwd - Working directory to start the search from.
+ *              Defaults to `process.cwd()` (same default as the underlying
+ *              `discoverForgeConfig`).
+ */
+export function discoverForgeConfigCached(cwd = process.cwd()) {
+    const key = cwd;
+    if (cache.has(key)) {
+        return cache.get(key);
+    }
+    const result = discoverForgeConfig(cwd);
+    cache.set(key, result);
+    return result;
+}
+/**
+ * Clears all cached entries.
+ *
+ * Primarily for test isolation — call in `beforeEach`/`afterEach` to
+ * prevent cache bleed between test cases that change the filesystem.
+ */
+export function clearForgeConfigCache() {
+    cache.clear();
+}
+//# sourceMappingURL=forge-config.js.map

package/dist/extensions/forgecli/lib/forge-config.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"forge-config.js","sourceRoot":"","sources":["../../../../src/extensions/forgecli/lib/forge-config.ts"],"names":[],"mappings":"AAAA,oEAAoE;AACpE,EAAE;AACF,yEAAyE;AACzE,wDAAwD;AACxD,EAAE;AACF,+EAA+E;AAC/E,8EAA8E;AAC9E,4EAA4E;AAC5E,6EAA6E;AAC7E,8CAA8C;AAE9C,OAAO,EAAoB,mBAAmB,EAAE,MAAM,kBAAkB,CAAC;AAEzE,gFAAgF;AAChF,MAAM,KAAK,GAAG,IAAI,GAAG,EAA8B,CAAC;AAEpD;;;;;;;;;;GAUG;AACH,MAAM,UAAU,yBAAyB,CAAC,MAAc,OAAO,CAAC,GAAG,EAAE;IACpE,MAAM,GAAG,GAAG,GAAG,CAAC;IAChB,IAAI,KAAK,CAAC,GAAG,CAAC,GAAG,CAAC,EAAE,CAAC;QACpB,OAAO,KAAK,CAAC,GAAG,CAAC,GAAG,CAAE,CAAC;IACxB,CAAC;IACD,MAAM,MAAM,GAAG,mBAAmB,CAAC,GAAG,CAAC,CAAC;IACxC,KAAK,CAAC,GAAG,CAAC,GAAG,EAAE,MAAM,CAAC,CAAC;IACvB,OAAO,MAAM,CAAC;AACf,CAAC;AAED;;;;;GAKG;AACH,MAAM,UAAU,qBAAqB;IACpC,KAAK,CAAC,KAAK,EAAE,CAAC;AACf,CAAC"}

package/dist/extensions/forgecli/lib/frontmatter-parser.d.ts

@@ -0,0 +1,13 @@
+/**
+ * Extract persona names from a workflow's YAML frontmatter `deps.personas` list.
+ *
+ * Expects the frontmatter block to begin and end with `---` delimiters.
+ * Looks for:
+ *   ```yaml
+ *   deps:
+ *     personas: [architect, engineer]
+ *   ```
+ *
+ * @returns Array of persona name strings, or empty array if not found.
+ */
+export declare function extractPersonaNames(workflowMd: string): string[];

package/dist/extensions/forgecli/lib/frontmatter-parser.js

@@ -0,0 +1,56 @@
+// lib/frontmatter-parser.ts — FORGE-S25-T16
+//
+// Extracted from 10 byte-identical copies of extractPersonaNames() across
+// kickoff shim files (approve.ts, collate.ts, commit.ts, enhance.ts,
+// implement.ts, plan.ts, retrospective.ts, review-code.ts, review-plan.ts,
+// validate.ts). Finding H-1 (Round-2-validation.md).
+//
+// Parses the YAML frontmatter `deps.personas` array from a workflow markdown
+// file. Used by kickoff shims to determine which persona files to load, and by
+// lib/manifest-checker.ts to validate that personas are referenced in the body.
+/**
+ * Extract persona names from a workflow's YAML frontmatter `deps.personas` list.
+ *
+ * Expects the frontmatter block to begin and end with `---` delimiters.
+ * Looks for:
+ *   ```yaml
+ *   deps:
+ *     personas: [architect, engineer]
+ *   ```
+ *
+ * @returns Array of persona name strings, or empty array if not found.
+ */
+export function extractPersonaNames(workflowMd) {
+    const lines = workflowMd.split(/\r?\n/);
+    if (lines.length === 0 || lines[0] !== "---")
+        return [];
+    let inside = false;
+    let depsBlock = false;
+    for (let i = 0; i < lines.length; i++) {
+        const line = lines[i];
+        if (i === 0 && line === "---") {
+            inside = true;
+            continue;
+        }
+        if (!inside)
+            break;
+        if (line === "---")
+            break;
+        if (/^deps\s*:\s*$/.test(line)) {
+            depsBlock = true;
+            continue;
+        }
+        const m = /^\s*personas\s*:\s*\[([^\]]*)\]\s*$/.exec(line);
+        if (m && (depsBlock || /^\s/.test(line))) {
+            return m[1]
+                .split(",")
+                .map((s) => s.trim().replace(/^["']|["']$/g, ""))
+                .filter((s) => s.length > 0);
+        }
+        if (depsBlock && /^[A-Za-z0-9_-]+\s*:/.test(line) && !/^\s/.test(line)) {
+            depsBlock = false;
+        }
+    }
+    return [];
+}
+//# sourceMappingURL=frontmatter-parser.js.map

package/dist/extensions/forgecli/lib/frontmatter-parser.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"frontmatter-parser.js","sourceRoot":"","sources":["../../../../src/extensions/forgecli/lib/frontmatter-parser.ts"],"names":[],"mappings":"AAAA,4CAA4C;AAC5C,EAAE;AACF,0EAA0E;AAC1E,qEAAqE;AACrE,2EAA2E;AAC3E,qDAAqD;AACrD,EAAE;AACF,6EAA6E;AAC7E,+EAA+E;AAC/E,gFAAgF;AAEhF;;;;;;;;;;;GAWG;AACH,MAAM,UAAU,mBAAmB,CAAC,UAAkB;IACrD,MAAM,KAAK,GAAG,UAAU,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC;IACxC,IAAI,KAAK,CAAC,MAAM,KAAK,CAAC,IAAI,KAAK,CAAC,CAAC,CAAC,KAAK,KAAK;QAAE,OAAO,EAAE,CAAC;IACxD,IAAI,MAAM,GAAG,KAAK,CAAC;IACnB,IAAI,SAAS,GAAG,KAAK,CAAC;IACtB,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,KAAK,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;QACvC,MAAM,IAAI,GAAG,KAAK,CAAC,CAAC,CAAC,CAAC;QACtB,IAAI,CAAC,KAAK,CAAC,IAAI,IAAI,KAAK,KAAK,EAAE,CAAC;YAC/B,MAAM,GAAG,IAAI,CAAC;YACd,SAAS;QACV,CAAC;QACD,IAAI,CAAC,MAAM;YAAE,MAAM;QACnB,IAAI,IAAI,KAAK,KAAK;YAAE,MAAM;QAC1B,IAAI,eAAe,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;YAChC,SAAS,GAAG,IAAI,CAAC;YACjB,SAAS;QACV,CAAC;QACD,MAAM,CAAC,GAAG,qCAAqC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAC3D,IAAI,CAAC,IAAI,CAAC,SAAS,IAAI,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,EAAE,CAAC;YAC1C,OAAO,CAAC,CAAC,CAAC,CAAC;iBACT,KAAK,CAAC,GAAG,CAAC;iBACV,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,OAAO,CAAC,cAAc,EAAE,EAAE,CAAC,CAAC;iBAChD,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC;QAC/B,CAAC;QACD,IAAI,SAAS,IAAI,qBAAqB,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;YACxE,SAAS,GAAG,KAAK,CAAC;QACnB,CAAC;IACF,CAAC;IACD,OAAO,EAAE,CAAC;AACX,CAAC"}

package/dist/extensions/forgecli/lib/manifest-checker.d.ts

@@ -0,0 +1,22 @@
+/**
+ * Result of a workflow materialization check.
+ * `ok` is true iff all required Pack-06 markers are present.
+ * `missing` lists the human-readable names of any absent markers.
+ */
+export interface MaterializationCheck {
+    ok: boolean;
+    missing: string[];
+}
+/**
+ * Verify that a workflow markdown file contains the four Pack-06 materialization
+ * markers required by the forge-cli kickoff shims:
+ *   1. "Store-Write Verification" — confirms write discipline section.
+ *   2. "Iron Laws" — confirms iron laws section.
+ *   3. "forge_store" — confirms tool reference.
+ *   4. deps.personas declared in frontmatter AND at least one persona referenced in body.
+ *
+ * Note: fix-bug.ts intentionally skips this check for fix_bug.md (N-H-E) because
+ * fix_bug.md is orchestrator-owned prose, not a tool-use sub-workflow. That skip
+ * lives in the call site (fix-bug.ts:725), not here.
+ */
+export declare function checkMaterialization(workflowPath: string, workflowMd: string): MaterializationCheck;

package/dist/extensions/forgecli/lib/manifest-checker.js

@@ -0,0 +1,64 @@
+// lib/manifest-checker.ts — FORGE-S25-T16
+//
+// Extracted from 9 byte-identical copies of checkMaterialization() across
+// kickoff shim files (approve.ts, collate.ts, commit.ts, enhance.ts,
+// implement.ts, plan.ts, review-code.ts, review-plan.ts, validate.ts).
+// Finding H-2 (Round-2-validation.md).
+//
+// The 3 orchestrators (run-task.ts, run-sprint.ts, fix-bug.ts) already imported
+// checkMaterialization from plan.ts. After this extraction they import from here.
+//
+// N-H-E: fix-bug.ts:725 intentionally skips checkMaterialization for fix_bug.md.
+// This is a call-site decision in fix-bug.ts, not a function decision — the lib
+// function is unaware of this skip and must NOT encode it.
+import { extractPersonaNames } from "./frontmatter-parser.js";
+/**
+ * Verify that a workflow markdown file contains the four Pack-06 materialization
+ * markers required by the forge-cli kickoff shims:
+ *   1. "Store-Write Verification" — confirms write discipline section.
+ *   2. "Iron Laws" — confirms iron laws section.
+ *   3. "forge_store" — confirms tool reference.
+ *   4. deps.personas declared in frontmatter AND at least one persona referenced in body.
+ *
+ * Note: fix-bug.ts intentionally skips this check for fix_bug.md (N-H-E) because
+ * fix_bug.md is orchestrator-owned prose, not a tool-use sub-workflow. That skip
+ * lives in the call site (fix-bug.ts:725), not here.
+ */
+export function checkMaterialization(workflowPath, workflowMd) {
+    const missing = [];
+    if (!workflowMd.includes("Store-Write Verification")) {
+        missing.push("Store-Write Verification");
+    }
+    if (!workflowMd.includes("Iron Laws")) {
+        missing.push("Iron Laws");
+    }
+    if (!workflowMd.includes("forge_store")) {
+        missing.push("forge_store");
+    }
+    const personas = extractPersonaNames(workflowMd);
+    if (personas.length === 0) {
+        missing.push("deps.personas: declaration");
+    }
+    else {
+        const bodyStart = (() => {
+            if (!workflowMd.startsWith("---\n") && !workflowMd.startsWith("---\r\n"))
+                return 0;
+            const re = /\r?\n---\r?\n/;
+            const m = re.exec(workflowMd);
+            return m ? m.index + m[0].length : 0;
+        })();
+        const body = workflowMd.slice(bodyStart);
+        const anyHit = personas.some((name) => {
+            if (!name)
+                return false;
+            const tokenRegex = new RegExp(`\\b${name.replace(/[.*+?^${}()|[\]\\]/g, "\\$&")}\\b`);
+            return body.includes(`${name}.md`) || tokenRegex.test(body);
+        });
+        if (!anyHit) {
+            missing.push(`persona file path (${personas.join(", ")})`);
+        }
+    }
+    void workflowPath;
+    return { ok: missing.length === 0, missing };
+}
+//# sourceMappingURL=manifest-checker.js.map

package/dist/extensions/forgecli/lib/manifest-checker.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"manifest-checker.js","sourceRoot":"","sources":["../../../../src/extensions/forgecli/lib/manifest-checker.ts"],"names":[],"mappings":"AAAA,0CAA0C;AAC1C,EAAE;AACF,0EAA0E;AAC1E,qEAAqE;AACrE,uEAAuE;AACvE,uCAAuC;AACvC,EAAE;AACF,gFAAgF;AAChF,kFAAkF;AAClF,EAAE;AACF,iFAAiF;AACjF,gFAAgF;AAChF,2DAA2D;AAE3D,OAAO,EAAE,mBAAmB,EAAE,MAAM,yBAAyB,CAAC;AAY9D;;;;;;;;;;;GAWG;AACH,MAAM,UAAU,oBAAoB,CAAC,YAAoB,EAAE,UAAkB;IAC5E,MAAM,OAAO,GAAa,EAAE,CAAC;IAE7B,IAAI,CAAC,UAAU,CAAC,QAAQ,CAAC,0BAA0B,CAAC,EAAE,CAAC;QACtD,OAAO,CAAC,IAAI,CAAC,0BAA0B,CAAC,CAAC;IAC1C,CAAC;IACD,IAAI,CAAC,UAAU,CAAC,QAAQ,CAAC,WAAW,CAAC,EAAE,CAAC;QACvC,OAAO,CAAC,IAAI,CAAC,WAAW,CAAC,CAAC;IAC3B,CAAC;IACD,IAAI,CAAC,UAAU,CAAC,QAAQ,CAAC,aAAa,CAAC,EAAE,CAAC;QACzC,OAAO,CAAC,IAAI,CAAC,aAAa,CAAC,CAAC;IAC7B,CAAC;IAED,MAAM,QAAQ,GAAG,mBAAmB,CAAC,UAAU,CAAC,CAAC;IACjD,IAAI,QAAQ,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QAC3B,OAAO,CAAC,IAAI,CAAC,4BAA4B,CAAC,CAAC;IAC5C,CAAC;SAAM,CAAC;QACP,MAAM,SAAS,GAAG,CAAC,GAAG,EAAE;YACvB,IAAI,CAAC,UAAU,CAAC,UAAU,CAAC,OAAO,CAAC,IAAI,CAAC,UAAU,CAAC,UAAU,CAAC,SAAS,CAAC;gBAAE,OAAO,CAAC,CAAC;YACnF,MAAM,EAAE,GAAG,eAAe,CAAC;YAC3B,MAAM,CAAC,GAAG,EAAE,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;YAC9B,OAAO,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,KAAK,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,CAAC;QACtC,CAAC,CAAC,EAAE,CAAC;QACL,MAAM,IAAI,GAAG,UAAU,CAAC,KAAK,CAAC,SAAS,CAAC,CAAC;QAEzC,MAAM,MAAM,GAAG,QAAQ,CAAC,IAAI,CAAC,CAAC,IAAI,EAAE,EAAE;YACrC,IAAI,CAAC,IAAI;gBAAE,OAAO,KAAK,CAAC;YACxB,MAAM,UAAU,GAAG,IAAI,MAAM,CAAC,MAAM,IAAI,CAAC,OAAO,CAAC,qBAAqB,EAAE,MAAM,CAAC,KAAK,CAAC,CAAC;YACtF,OAAO,IAAI,CAAC,QAAQ,CAAC,GAAG,IAAI,KAAK,CAAC,IAAI,UAAU,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAC7D,CAAC,CAAC,CAAC;QACH,IAAI,CAAC,MAAM,EAAE,CAAC;YACb,OAAO,CAAC,IAAI,CAAC,sBAAsB,QAAQ,CAAC,IAAI,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QAC5D,CAAC;IACF,CAAC;IAED,KAAK,YAAY,CAAC;IAClB,OAAO,EAAE,EAAE,EAAE,OAAO,CAAC,MAAM,KAAK,CAAC,EAAE,OAAO,EAAE,CAAC;AAC9C,CAAC"}

package/dist/extensions/forgecli/lib/orchestrator-preflight.d.ts

@@ -0,0 +1,46 @@
+import type { MergedConfig } from "../config-layer.js";
+import type { OrchestratorResult } from "./orchestrator-types.js";
+/** Minimal context interface — only the bits preflight needs. */
+export interface PreflightContext {
+    ui: {
+        notify(message: string, type?: "error" | "info" | "warning"): void;
+    };
+}
+export interface RunOrchestratorPreflightOptions {
+    /** "task" runs validateModelConfig; "ceremony" skips it. */
+    mode: "task" | "ceremony";
+    /** Minimal context for notifications. */
+    ctx: PreflightContext;
+    /** Prefix for notify messages, e.g. "forge:run-task" or "forge:fix-bug". */
+    notifyPrefix: string;
+    /** Persona name catalogue from readPersonaDir. */
+    personaCatalogue: string[];
+    /** Pipeline name catalogue from readPipelineNames; null if no config found. */
+    pipelineCatalogue: string[] | null;
+    /** Merged model-routing config from loadLayeredConfig. */
+    modelRoutingConfig: MergedConfig;
+    /** Available models from ctx.modelRegistry.getAvailable(). */
+    availableModels: Array<{
+        provider: string;
+        id: string;
+    }>;
+}
+/** Discriminated union: proceed=true → continue; proceed=false → return result early. */
+export type OrchestratorPreflightResult = {
+    proceed: true;
+} | {
+    proceed: false;
+    result: OrchestratorResult;
+};
+/**
+ * Run the orchestrator pre-flight check.
+ *
+ * In mode="task": calls validateModelConfig and surfaces warnings/errors via
+ * ctx.ui.notify. Returns `{ proceed: false, result }` with status="failed" if
+ * there are errors; returns `{ proceed: true }` otherwise.
+ *
+ * In mode="ceremony": returns `{ proceed: true }` immediately without calling
+ * validateModelConfig. This matches the existing behaviour of ceremony dispatch
+ * which uses lookupPersonaModel rather than the full validator.
+ */
+export declare function runOrchestratorPreflight(opts: RunOrchestratorPreflightOptions): OrchestratorPreflightResult;

package/dist/extensions/forgecli/lib/orchestrator-preflight.js

@@ -0,0 +1,64 @@
+// lib/orchestrator-preflight.ts — FORGE-S25-T17 (H-13, N-H-D)
+//
+// Single shared preflight helper for orchestrator pipelines. Replaces the
+// duplicate pre-flight validation blocks in run-task.ts and fix-bug.ts.
+//
+// Design: a `mode` parameter selects behaviour:
+//
+//   mode="task"     — full model-config validation via validateModelConfig.
+//                     Used by runTaskPipeline and runBugPipeline.
+//   mode="ceremony" — skips validateModelConfig entirely.
+//                     Aligns with dispatchSprintCeremony which uses
+//                     lookupPersonaModel (not the full validator). The
+//                     run-sprint.ts ceremony call does NOT use this helper
+//                     directly; it was already skip-validation by design
+//                     (N-H-D). This mode is available for future callers
+//                     that need a ceremony-style preflight.
+//
+// Why run-sprint.ts is NOT migrated to this helper:
+//   run-sprint.ts contains a separate sprint-entry validation block
+//   (lines 470–498) that validates sprint-level config, not task/phase-level.
+//   That block is architecturally distinct and is explicitly out of scope
+//   per the plan-review finding (N-H-D resolved, reviewer-approved).
+import { validateModelConfig } from "../model-validator.js";
+// ---------------------------------------------------------------------------
+// runOrchestratorPreflight
+// ---------------------------------------------------------------------------
+/**
+ * Run the orchestrator pre-flight check.
+ *
+ * In mode="task": calls validateModelConfig and surfaces warnings/errors via
+ * ctx.ui.notify. Returns `{ proceed: false, result }` with status="failed" if
+ * there are errors; returns `{ proceed: true }` otherwise.
+ *
+ * In mode="ceremony": returns `{ proceed: true }` immediately without calling
+ * validateModelConfig. This matches the existing behaviour of ceremony dispatch
+ * which uses lookupPersonaModel rather than the full validator.
+ */
+export function runOrchestratorPreflight(opts) {
+    if (opts.mode === "ceremony") {
+        return { proceed: true };
+    }
+    // mode="task" — full validation
+    const strict = process.env.FORGE_STRICT_MODELS === "1";
+    const { errors, warnings } = validateModelConfig(opts.personaCatalogue, opts.pipelineCatalogue, opts.modelRoutingConfig, opts.availableModels, strict);
+    for (const w of warnings) {
+        opts.ctx.ui.notify(`⚠ ${opts.notifyPrefix} — model routing: ${w.message}`, "warning");
+    }
+    if (errors.length > 0) {
+        for (const e of errors) {
+            opts.ctx.ui.notify(`× ${opts.notifyPrefix} — model routing: ${e.message}`, "error");
+        }
+        return {
+            proceed: false,
+            result: {
+                status: "failed",
+                lastPhaseIndex: 0,
+                iterationCounts: {},
+                lastError: `model config validation failed: ${errors.map((e) => e.code).join(", ")}`,
+            },
+        };
+    }
+    return { proceed: true };
+}
+//# sourceMappingURL=orchestrator-preflight.js.map

package/dist/extensions/forgecli/lib/orchestrator-preflight.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"orchestrator-preflight.js","sourceRoot":"","sources":["../../../../src/extensions/forgecli/lib/orchestrator-preflight.ts"],"names":[],"mappings":"AAAA,8DAA8D;AAC9D,EAAE;AACF,0EAA0E;AAC1E,wEAAwE;AACxE,EAAE;AACF,gDAAgD;AAChD,EAAE;AACF,4EAA4E;AAC5E,kEAAkE;AAClE,0DAA0D;AAC1D,oEAAoE;AACpE,uEAAuE;AACvE,2EAA2E;AAC3E,yEAAyE;AACzE,yEAAyE;AACzE,4DAA4D;AAC5D,EAAE;AACF,oDAAoD;AACpD,oEAAoE;AACpE,8EAA8E;AAC9E,0EAA0E;AAC1E,qEAAqE;AAErE,OAAO,EAAE,mBAAmB,EAAE,MAAM,uBAAuB,CAAC;AAqC5D,8EAA8E;AAC9E,2BAA2B;AAC3B,8EAA8E;AAE9E;;;;;;;;;;GAUG;AACH,MAAM,UAAU,wBAAwB,CACvC,IAAqC;IAErC,IAAI,IAAI,CAAC,IAAI,KAAK,UAAU,EAAE,CAAC;QAC9B,OAAO,EAAE,OAAO,EAAE,IAAI,EAAE,CAAC;IAC1B,CAAC;IAED,gCAAgC;IAChC,MAAM,MAAM,GAAG,OAAO,CAAC,GAAG,CAAC,mBAAmB,KAAK,GAAG,CAAC;IACvD,MAAM,EAAE,MAAM,EAAE,QAAQ,EAAE,GAAG,mBAAmB,CAC/C,IAAI,CAAC,gBAAgB,EACrB,IAAI,CAAC,iBAAiB,EACtB,IAAI,CAAC,kBAAkB,EACvB,IAAI,CAAC,eAAe,EACpB,MAAM,CACN,CAAC;IAEF,KAAK,MAAM,CAAC,IAAI,QAAQ,EAAE,CAAC;QAC1B,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,MAAM,CAAC,KAAK,IAAI,CAAC,YAAY,qBAAqB,CAAC,CAAC,OAAO,EAAE,EAAE,SAAS,CAAC,CAAC;IACvF,CAAC;IAED,IAAI,MAAM,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QACvB,KAAK,MAAM,CAAC,IAAI,MAAM,EAAE,CAAC;YACxB,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,MAAM,CAAC,KAAK,IAAI,CAAC,YAAY,qBAAqB,CAAC,CAAC,OAAO,EAAE,EAAE,OAAO,CAAC,CAAC;QACrF,CAAC;QACD,OAAO;YACN,OAAO,EAAE,KAAK;YACd,MAAM,EAAE;gBACP,MAAM,EAAE,QAAQ;gBAChB,cAAc,EAAE,CAAC;gBACjB,eAAe,EAAE,EAAE;gBACnB,SAAS,EAAE,mCAAmC,MAAM,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE;aACpF;SACD,CAAC;IACH,CAAC;IAED,OAAO,EAAE,OAAO,EAAE,IAAI,EAAE,CAAC;AAC1B,CAAC"}

package/dist/extensions/forgecli/lib/orchestrator-types.d.ts

@@ -0,0 +1,20 @@
+/** Shared status values used by both task and bug pipelines. */
+export type OrchestratorStatus = "completed" | "halted" | "escalated" | "failed" | "cancelled";
+/**
+ * Base shape shared by RunTaskPipelineResult and RunBugPipelineResult.
+ *
+ * Both concrete result interfaces must be structurally compatible with this
+ * type — i.e. a value of either concrete type must be assignable to
+ * OrchestratorResult. The type-level tests in orchestrator-types.test.ts
+ * enforce this at compile time.
+ */
+export interface OrchestratorResult {
+    status: OrchestratorStatus;
+    lastPhaseIndex: number;
+    iterationCounts: Record<string, number>;
+    lastError?: string;
+    /** Model captured from last successful phase's subagent result. */
+    model?: string;
+    /** Provider captured from last successful phase's subagent result. */
+    provider?: string;
+}

package/dist/extensions/forgecli/lib/orchestrator-types.js

@@ -0,0 +1,14 @@
+// lib/orchestrator-types.ts — FORGE-S25-T17 (H-10)
+//
+// Shared orchestrator result type used by both runTaskPipeline (run-task.ts)
+// and runBugPipeline (fix-bug.ts). Extracting this to a shared lib module
+// allows callers (e.g. run-sprint.ts) to reference a single type when
+// consuming pipeline results without importing from either concrete handler.
+//
+// Design: OrchestratorResult is the base interface. RunTaskPipelineResult and
+// RunBugPipelineResult extend it (via structural compatibility — TypeScript's
+// structural type system means no `extends` keyword is required; they just
+// must satisfy the shape). Verify with the type-level tests in
+// test/extensions/forgecli/lib/orchestrator-types.test.ts.
+export {};
+//# sourceMappingURL=orchestrator-types.js.map

package/dist/extensions/forgecli/lib/orchestrator-types.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"orchestrator-types.js","sourceRoot":"","sources":["../../../../src/extensions/forgecli/lib/orchestrator-types.ts"],"names":[],"mappings":"AAAA,mDAAmD;AACnD,EAAE;AACF,6EAA6E;AAC7E,0EAA0E;AAC1E,sEAAsE;AACtE,6EAA6E;AAC7E,EAAE;AACF,8EAA8E;AAC9E,8EAA8E;AAC9E,2EAA2E;AAC3E,+DAA+D;AAC/D,2DAA2D"}