loopforge 2.0.0 → 2.6.0

package/dist/engine.d.ts

@@ -7,30 +7,8 @@
  */
 import type { VaultBackend } from "./backends/interface.js";
 import type { LoopStore } from "./loop-store.js";
-import { type AgentLoopResult, type CriterionRevision, type ExecutionEvidence, type LoopForgeRequest, type SelfEvaluation, type SessionState } from "./protocol.js";
-/** Parse ExecutionEvidence from a raw JSON object. Shared by buildSelfEvaluation
- *  and invokeLoopCompile — both parse the same execution_evidence shape. */
-export declare function parseExecutionEvidence(raw: Record<string, unknown> | undefined | null): ExecutionEvidence | undefined;
-/** Parse CriterionRevision[] from a raw JSON array. Shared by buildSelfEvaluation
- *  and invokeLoopCompile — both parse the same revised_success_criteria shape. */
-export declare function parseCriterionRevisions(raw: unknown): CriterionRevision[];
-/** Parse WorkerResult[] from a raw JSON array. Shared by buildSelfEvaluation
- *  and invokeLoopCompile — both parse the same worker_results shape. */
-export declare function parseWorkerResults(raw: unknown): import("./protocol.js").WorkerResult[];
-/** Extract a structured SelfEvaluation from agent output text.
- *  Returns null if no valid self-eval block is found.
- *  The agent is instructed to output JSON between the delimiters. */
-export declare function extractSelfEvaluation(text: string): SelfEvaluation | null;
-/** Build a SelfEvaluation from a parsed JSON object.
- *  Lenient parsing: missing optional fields get sensible defaults.
- *  Used by extractSelfEvaluation() (regex path) and MCP tool handler
- *  (structured evaluation parameter path). */
-export declare function buildSelfEvaluation(raw: Record<string, unknown>): SelfEvaluation;
-/** Fallback heuristic when structured self-eval extraction fails.
- *  Scans agent output for completion and error signals.
- *  Returns a low-confidence SelfEvaluation — the autonomous runner
- *  may choose to warn the user or continue cautiously. */
-export declare function heuristicSelfEvaluation(text: string): SelfEvaluation | null;
+import { type AgentLoopResult, type LoopForgeRequest, type SelfEvaluation, type SessionState } from "./protocol.js";
+export { parseExecutionEvidence, parseCriterionRevisions, parseWorkerResults, extractSelfEvaluation, buildSelfEvaluation, } from "./self-eval.js";
 /** A single sub-agent delegation record (v1.9 — AgentTool mode). */
 export interface DelegationEntry {
     index: number;
@@ -43,19 +21,12 @@ export interface DelegationEntry {
 }
 export interface EngineMetrics {
     vaultWriteErrors: number;
-    vaultWriteTimeouts: number;
-    vaultWriteBytes: number;
-    silentAnalysisErrors: number;
-    hydrateCacheMisses: number;
-    feedbackBufferFlushes: number;
-    feedbackBufferMaxSize: number;
     sessionStart: number;
 }
 export declare class LoopForgeEngine {
     state: SessionState | null;
     private backend;
     private metrics;
-    private feedbackWriteBuffer;
     lastTask: string | null;
     constructor(storeOrBackend?: LoopStore | VaultBackend);
     private resolveBackend;
@@ -65,7 +36,7 @@ export declare class LoopForgeEngine {
     getMetrics(): EngineMetrics;
     private ensureInit;
     private persistFeedbackToVault;
-    flushFeedbackBuffer(): number;
+    private buildFeedbackEntry;
     private persistLoopLineage;
     /** Record sub-agent delegations for this round into the vault.
      *  Written as a lightweight journal entry so the main agent's rolling
@@ -83,7 +54,6 @@ export declare class LoopForgeEngine {
     invokeLoopCompile(request: LoopForgeRequest, hydrateResults?: Record<string, unknown> | null, options?: {
         persistLineage?: boolean;
     }): AgentLoopResult;
-    shouldBreak(): boolean;
 }
 export declare function createEngine(store?: LoopStore): LoopForgeEngine;
 //# sourceMappingURL=engine.d.ts.map

package/dist/engine.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"engine.d.ts","sourceRoot":"","sources":["../src/engine.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAIH,OAAO,KAAK,EAAE,YAAY,EAAc,MAAM,yBAAyB,CAAC;AAExE,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,iBAAiB,CAAC;AACjD,OAAO,EAYL,KAAK,eAAe,EACpB,KAAK,iBAAiB,EACtB,KAAK,iBAAiB,EAItB,KAAK,gBAAgB,EACrB,KAAK,cAAc,EACnB,KAAK,YAAY,EAElB,MAAM,eAAe,CAAC;AAQvB;4EAC4E;AAC5E,wBAAgB,sBAAsB,CACpC,GAAG,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAAG,SAAS,GAAG,IAAI,GAC9C,iBAAiB,GAAG,SAAS,CAwB/B;AAED;kFACkF;AAClF,wBAAgB,uBAAuB,CACrC,GAAG,EAAE,OAAO,GACX,iBAAiB,EAAE,CAWrB;AAED;wEACwE;AACxE,wBAAgB,kBAAkB,CAChC,GAAG,EAAE,OAAO,GACX,OAAO,eAAe,EAAE,YAAY,EAAE,CAqBxC;AAMD;;qEAEqE;AACrE,wBAAgB,qBAAqB,CAAC,IAAI,EAAE,MAAM,GAAG,cAAc,GAAG,IAAI,CAezE;AAED;;;8CAG8C;AAC9C,wBAAgB,mBAAmB,CACjC,GAAG,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAC3B,cAAc,CAoDhB;AAED;;;0DAG0D;AAC1D,wBAAgB,uBAAuB,CAAC,IAAI,EAAE,MAAM,GAAG,cAAc,GAAG,IAAI,CAqB3E;AAMD,oEAAoE;AACpE,MAAM,WAAW,eAAe;IAC9B,KAAK,EAAE,MAAM,CAAC;IACd,OAAO,EAAE,MAAM,CAAC;IAChB,YAAY,EAAE,MAAM,CAAC;IACrB,OAAO,EAAE,MAAM,CAAC;IAChB,aAAa,EAAE,MAAM,CAAC;IACtB,OAAO,EAAE,OAAO,CAAC;IACjB,qBAAqB,EAAE,MAAM,EAAE,CAAC;CACjC;AAED,MAAM,WAAW,aAAa;IAC5B,gBAAgB,EAAE,MAAM,CAAC;IACzB,kBAAkB,EAAE,MAAM,CAAC;IAC3B,eAAe,EAAE,MAAM,CAAC;IACxB,oBAAoB,EAAE,MAAM,CAAC;IAC7B,kBAAkB,EAAE,MAAM,CAAC;IAC3B,qBAAqB,EAAE,MAAM,CAAC;IAC9B,qBAAqB,EAAE,MAAM,CAAC;IAC9B,YAAY,EAAE,MAAM,CAAC;CACtB;AAmBD,qBAAa,eAAe;IAC1B,KAAK,EAAE,YAAY,GAAG,IAAI,CAAQ;IAClC,OAAO,CAAC,OAAO,CAA6B;IAC5C,OAAO,CAAC,OAAO,CAA8B;IAC7C,OAAO,CAAC,mBAAmB,CAAoB;IAC/C,QAAQ,EAAE,MAAM,GAAG,IAAI,CAAQ;gBAEnB,cAAc,CAAC,EAAE,SAAS,GAAG,YAAY;IAQrD,OAAO,CAAC,cAAc;IAStB,iFAAiF;IACjF,UAAU,IAAI,YAAY;IAI1B,6EAA6E;IAC7E,UAAU,IAAI,aAAa;IAO3B,OAAO,CAAC,UAAU;IAelB,OAAO,CAAC,sBAAsB;IAkB9B,mBAAmB,IAAI,MAAM;IA8D7B,OAAO,CAAC,kBAAkB;IA8D1B;;yEAEqE;IACrE,gBAAgB,CACd,MAAM,EAAE,MAAM,EACd,KAAK,EAAE,MAAM,EACb,OAAO,EAAE,eAAe,EAAE,GACzB,IAAI;IAwCP,kBAAkB,CAAC,MAAM,EAAE,MAAM,GAAG,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAAG,IAAI;IA2DlE,cAAc,CACZ,OAAO,EAAE,gBAAgB,EACzB,eAAe,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAAG,IAAI,GAC/C,eAAe;IAmFlB;;;;;gEAK4D;IAC5D,YAAY,CACV,QAAQ,EAAE,cAAc,EACxB,MAAM,EAAE,MAAM,EACd,KAAK,EAAE,MAAM,EACb,IAAI,EAAE,MAAM,EACZ,gBAAgB,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GACzC,OAAO;IAyHV,iBAAiB,CACf,OAAO,EAAE,gBAAgB,EACzB,cAAc,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAAG,IAAI,EAC/C,OAAO,GAAE;QAAE,cAAc,CAAC,EAAE,OAAO,CAAA;KAAO,GACzC,eAAe;IA6IlB,WAAW,IAAI,OAAO;CAkBvB;AAMD,wBAAgB,YAAY,CAC1B,KAAK,CAAC,EAAE,SAAS,GAChB,eAAe,CAEjB"}
+{"version":3,"file":"engine.d.ts","sourceRoot":"","sources":["../src/engine.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAIH,OAAO,KAAK,EAAE,YAAY,EAAc,MAAM,yBAAyB,CAAC;AAExE,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,iBAAiB,CAAC;AACjD,OAAO,EASL,KAAK,eAAe,EAIpB,KAAK,gBAAgB,EACrB,KAAK,cAAc,EACnB,KAAK,YAAY,EAElB,MAAM,eAAe,CAAC;AAWvB,OAAO,EACL,sBAAsB,EACtB,uBAAuB,EACvB,kBAAkB,EAClB,qBAAqB,EACrB,mBAAmB,GACpB,MAAM,gBAAgB,CAAC;AAMxB,oEAAoE;AACpE,MAAM,WAAW,eAAe;IAC9B,KAAK,EAAE,MAAM,CAAC;IACd,OAAO,EAAE,MAAM,CAAC;IAChB,YAAY,EAAE,MAAM,CAAC;IACrB,OAAO,EAAE,MAAM,CAAC;IAChB,aAAa,EAAE,MAAM,CAAC;IACtB,OAAO,EAAE,OAAO,CAAC;IACjB,qBAAqB,EAAE,MAAM,EAAE,CAAC;CACjC;AAED,MAAM,WAAW,aAAa;IAC5B,gBAAgB,EAAE,MAAM,CAAC;IACzB,YAAY,EAAE,MAAM,CAAC;CACtB;AAaD,qBAAa,eAAe;IAC1B,KAAK,EAAE,YAAY,GAAG,IAAI,CAAQ;IAClC,OAAO,CAAC,OAAO,CAA6B;IAC5C,OAAO,CAAC,OAAO,CAA8B;IAC7C,QAAQ,EAAE,MAAM,GAAG,IAAI,CAAQ;gBAEnB,cAAc,CAAC,EAAE,SAAS,GAAG,YAAY;IAQrD,OAAO,CAAC,cAAc;IAStB,iFAAiF;IACjF,UAAU,IAAI,YAAY;IAI1B,6EAA6E;IAC7E,UAAU,IAAI,aAAa;IAO3B,OAAO,CAAC,UAAU;IAelB,OAAO,CAAC,sBAAsB;IAgB9B,OAAO,CAAC,kBAAkB;IAqC1B,OAAO,CAAC,kBAAkB;IA8D1B;;yEAEqE;IACrE,gBAAgB,CACd,MAAM,EAAE,MAAM,EACd,KAAK,EAAE,MAAM,EACb,OAAO,EAAE,eAAe,EAAE,GACzB,IAAI;IAwCP,kBAAkB,CAAC,MAAM,EAAE,MAAM,GAAG,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAAG,IAAI;IA2DlE,cAAc,CACZ,OAAO,EAAE,gBAAgB,EACzB,eAAe,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAAG,IAAI,GAC/C,eAAe;IAyElB;;;;;gEAK4D;IAC5D,YAAY,CACV,QAAQ,EAAE,cAAc,EACxB,MAAM,EAAE,MAAM,EACd,KAAK,EAAE,MAAM,EACb,IAAI,EAAE,MAAM,EACZ,gBAAgB,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GACzC,OAAO;IAqHV,iBAAiB,CACf,OAAO,EAAE,gBAAgB,EACzB,cAAc,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAAG,IAAI,EAC/C,OAAO,GAAE;QAAE,cAAc,CAAC,EAAE,OAAO,CAAA;KAAO,GACzC,eAAe;CAiJnB;AAMD,wBAAgB,YAAY,CAC1B,KAAK,CAAC,EAAE,SAAS,GAChB,eAAe,CAEjB"}

package/dist/engine.js

@@ -8,184 +8,16 @@
 import { randomUUID } from "node:crypto";
 import { getPolicy } from "./policy.js";
 import { FileLoopStore, LoopStoreBackend } from "./loop-store.js";
-import { AgentStatus, Mode, makeExecutionEvidence, makeExecutionFeedback, makeLoopCompileRequest, makeLoopObjective, makeLoopRoundResult, makeSelfEvaluation, makeSessionState, makeTaskId, SELF_EVAL_REGEX, } from "./protocol.js";
+import { AgentStatus, Mode, makeExecutionFeedback, makeLoopCompileRequest, makeLoopObjective, makeLoopRoundResult, makeSessionState, makeTaskId, } from "./protocol.js";
 import { compileLoop } from "./loop-compiler.js";
 import { logEvent } from "./observability.js";
-// ═══════════════════════════════════════════════════════════════════════════
-// Shared helpers
-// ═══════════════════════════════════════════════════════════════════════════
-/** Parse ExecutionEvidence from a raw JSON object. Shared by buildSelfEvaluation
- *  and invokeLoopCompile — both parse the same execution_evidence shape. */
-export function parseExecutionEvidence(raw) {
-    if (!raw || typeof raw !== "object")
-        return undefined;
-    const testResults = raw.test_results;
-    return makeExecutionEvidence({
-        files_changed: Array.isArray(raw.files_changed)
-            ? raw.files_changed.filter((v) => typeof v === "string")
-            : [],
-        test_results: testResults && typeof testResults.passed === "number"
-            ? {
-                passed: testResults.passed,
-                failed: testResults.failed ?? 0,
-                skipped: testResults.skipped ?? 0,
-            }
-            : null,
-        success_criteria_met: Array.isArray(raw.success_criteria_met)
-            ? raw.success_criteria_met.filter((v) => typeof v === "string")
-            : [],
-        success_criteria_remaining: Array.isArray(raw.success_criteria_remaining)
-            ? raw.success_criteria_remaining.filter((v) => typeof v === "string")
-            : [],
-        progress_estimate: typeof raw.progress_estimate === "number"
-            ? Math.max(0, Math.min(1, raw.progress_estimate))
-            : 0.0,
-    });
-}
-/** Parse CriterionRevision[] from a raw JSON array. Shared by buildSelfEvaluation
- *  and invokeLoopCompile — both parse the same revised_success_criteria shape. */
-export function parseCriterionRevisions(raw) {
-    if (!Array.isArray(raw))
-        return [];
-    return raw
-        .filter((v) => typeof v === "object" && v !== null &&
-        typeof v.old === "string" &&
-        typeof v.new === "string")
-        .map((v) => {
-        const r = v;
-        return { old: r.old, new: r.new };
-    });
-}
-/** Parse WorkerResult[] from a raw JSON array. Shared by buildSelfEvaluation
- *  and invokeLoopCompile — both parse the same worker_results shape. */
-export function parseWorkerResults(raw) {
-    if (!Array.isArray(raw))
-        return [];
-    return raw
-        .filter((v) => typeof v === "object" && v !== null &&
-        typeof v.agentId === "string" &&
-        typeof v.subTask === "string" &&
-        typeof v.resultSummary === "string")
-        .map((v) => {
-        const w = v;
-        return {
-            agentId: w.agentId,
-            subAgentType: typeof w.subAgentType === "string" ? w.subAgentType : "general-purpose",
-            subTask: w.subTask,
-            resultSummary: w.resultSummary,
-            success: typeof w.success === "boolean" ? w.success : false,
-            discoveredConstraints: Array.isArray(w.discoveredConstraints)
-                ? w.discoveredConstraints.filter((c) => typeof c === "string")
-                : [],
-        };
-    });
-}
-// ═══════════════════════════════════════════════════════════════════════════
-// Self-Evaluation extraction (v1.1 — autonomous loop feedback)
-// ═══════════════════════════════════════════════════════════════════════════
-/** Extract a structured SelfEvaluation from agent output text.
- *  Returns null if no valid self-eval block is found.
- *  The agent is instructed to output JSON between the delimiters. */
-export function extractSelfEvaluation(text) {
-    const match = text.match(SELF_EVAL_REGEX);
-    if (!match)
-        return null;
-    try {
-        const raw = JSON.parse(match[1]);
-        // Validate required fields
-        if (typeof raw.success !== "boolean")
-            return null;
-        if (typeof raw.output_summary !== "string")
-            return null;
-        if (!Array.isArray(raw.constraint_violations))
-            return null;
-        if (typeof raw.should_continue !== "boolean")
-            return null;
-        return buildSelfEvaluation(raw);
-    }
-    catch {
-        return null;
-    }
-}
-/** Build a SelfEvaluation from a parsed JSON object.
- *  Lenient parsing: missing optional fields get sensible defaults.
- *  Used by extractSelfEvaluation() (regex path) and MCP tool handler
- *  (structured evaluation parameter path). */
-export function buildSelfEvaluation(raw) {
-    // P4: Parse execution evidence from raw JSON
-    const executionEvidence = parseExecutionEvidence(raw.execution_evidence);
-    // P5: Parse corrections
-    const retractedConstraints = Array.isArray(raw.retracted_constraints)
-        ? raw.retracted_constraints.filter((v) => typeof v === "string")
-        : [];
-    const revisedCriteria = parseCriterionRevisions(raw.revised_success_criteria);
-    const wrongAssumptions = Array.isArray(raw.wrong_assumptions)
-        ? raw.wrong_assumptions.filter((v) => typeof v === "string")
-        : [];
-    // Multi-agent: Parse worker delegation results
-    const workerResults = parseWorkerResults(raw.worker_results);
-    return makeSelfEvaluation({
-        success: typeof raw.success === "boolean" ? raw.success : false,
-        output_summary: typeof raw.output_summary === "string" ? raw.output_summary : "",
-        constraint_violations: Array.isArray(raw.constraint_violations)
-            ? raw.constraint_violations.filter((v) => typeof v === "string")
-            : [],
-        should_continue: typeof raw.should_continue === "boolean" ? raw.should_continue : true,
-        // P0–P2: Optional evolution fields
-        discovered_constraints: Array.isArray(raw.discovered_constraints)
-            ? raw.discovered_constraints.filter((v) => typeof v === "string")
-            : [],
-        objective_refinement: typeof raw.objective_refinement === "string"
-            ? raw.objective_refinement
-            : "",
-        emerged_subtasks: Array.isArray(raw.emerged_subtasks)
-            ? raw.emerged_subtasks.filter((v) => typeof v === "string")
-            : [],
-        // P4: Execution evidence
-        execution_evidence: executionEvidence,
-        // P5: Self-correction
-        retracted_constraints: retractedConstraints,
-        revised_success_criteria: revisedCriteria,
-        wrong_assumptions: wrongAssumptions,
-        // Multi-agent: Worker delegation results
-        worker_results: workerResults,
-        // v1.10: Checkpoint compression
-        compression_checkpoint: typeof raw.compression_checkpoint === "boolean" ? raw.compression_checkpoint : false,
-        checkpoint_label: typeof raw.checkpoint_label === "string" ? raw.checkpoint_label : "",
-        // v1.16: Agent's declared next action
-        next_action: typeof raw.next_action === "string" ? raw.next_action : undefined,
-    });
-}
-/** Fallback heuristic when structured self-eval extraction fails.
- *  Scans agent output for completion and error signals.
- *  Returns a low-confidence SelfEvaluation — the autonomous runner
- *  may choose to warn the user or continue cautiously. */
-export function heuristicSelfEvaluation(text) {
-    const lower = text.toLowerCase();
-    const hasError = /error|failed|exception|cannot|unable|失败|错误|异常/.test(lower);
-    const hasCompletion = /done|complete|finished|完成|成功/.test(lower);
-    const hasRemaining = /remaining|continue|still need|next|todo|剩余|继续|下一步/.test(lower);
-    // Extract a reasonable summary from the last meaningful paragraph
-    const paragraphs = text.split(/\n\n+/).filter((p) => p.trim().length > 30);
-    const summary = paragraphs.length > 0
-        ? paragraphs[paragraphs.length - 1].trim().slice(0, 300)
-        : text.trim().slice(0, 300);
-    return makeSelfEvaluation({
-        success: !hasError && (hasCompletion || !hasRemaining),
-        output_summary: summary || "[heuristic fallback — could not parse structured self-eval]",
-        constraint_violations: [],
-        should_continue: hasRemaining && !hasError,
-    });
-}
+import { parseExecutionEvidence, parseCriterionRevisions, parseWorkerResults, } from "./self-eval.js";
+import { parseLoopExtras } from "./loop-extras-parser.js";
+// ── Re-export self-evaluation utilities (moved to self-eval.ts) ──────────
+export { parseExecutionEvidence, parseCriterionRevisions, parseWorkerResults, extractSelfEvaluation, buildSelfEvaluation, } from "./self-eval.js";
 function makeEngineMetrics() {
     return {
         vaultWriteErrors: 0,
-        vaultWriteTimeouts: 0,
-        vaultWriteBytes: 0,
-        silentAnalysisErrors: 0,
-        hydrateCacheMisses: 0,
-        feedbackBufferFlushes: 0,
-        feedbackBufferMaxSize: 0,
         sessionStart: Date.now(),
     };
 }
@@ -196,7 +28,6 @@ export class LoopForgeEngine {
     state = null;
     backend = null;
     metrics = null;
-    feedbackWriteBuffer = [];
     lastTask = null;
     constructor(storeOrBackend) {
         if (storeOrBackend) {
@@ -237,72 +68,53 @@ export class LoopForgeEngine {
         if (this.metrics === null) {
             this.metrics = makeEngineMetrics();
         }
-        this.feedbackWriteBuffer.push(signal);
-        const bufLen = this.feedbackWriteBuffer.length;
-        if (bufLen > this.metrics.feedbackBufferMaxSize) {
-            this.metrics.feedbackBufferMaxSize = bufLen;
+        // v2.6: write directly — the buffer was always flushed immediately
+        // (autoFeedback calls persistFeedbackToVault then immediately flushFeedbackBuffer).
+        const entry = this.buildFeedbackEntry(signal);
+        if (!entry)
+            return;
+        try {
+            this.resolveBackend().appendEntry(entry);
         }
-        const policy = getPolicy();
-        if (bufLen >= policy.engine.feedback_flush_interval) {
-            this.flushFeedbackBuffer();
+        catch {
+            if (this.metrics)
+                this.metrics.vaultWriteErrors++;
+            logEvent("vault_write_error", { error: "feedback_persist" });
         }
     }
-    flushFeedbackBuffer() {
-        if (!this.feedbackWriteBuffer.length)
-            return 0;
-        const records = this.feedbackWriteBuffer.splice(0);
-        if (this.metrics)
-            this.metrics.feedbackBufferFlushes++;
-        const now = new Date().toISOString().replace(/\.\d+Z$/, "");
-        const entries = [];
-        for (const signal of records) {
-            try {
-                const entry = {
-                    id: randomUUID(),
-                    task_id: signal.task_id ?? "feedback",
-                    version_tag: "v1",
-                    is_active: true,
-                    timestamp: now,
-                    user_intent: String(signal.task_type ?? "").slice(0, 200),
+    buildFeedbackEntry(signal) {
+        try {
+            return {
+                id: randomUUID(),
+                task_id: signal.task_id ?? "feedback",
+                version_tag: "v1",
+                is_active: true,
+                timestamp: new Date().toISOString().replace(/\.\d+Z$/, ""),
+                user_intent: String(signal.task_type ?? "").slice(0, 200),
+                success: signal.success ?? false,
+                execution_feedback: JSON.stringify({
                     success: signal.success ?? false,
-                    execution_feedback: JSON.stringify({
-                        success: signal.success ?? false,
-                        status: signal.success
-                            ? "success"
-                            : "partial",
-                        constraint_compliance: {
-                            all_hard_constraints_met: !Array.isArray(signal.violations) || signal.violations.length === 0,
-                            violations: signal.violations ?? [],
-                        },
-                        output_summary: signal.task_type ?? "",
-                        improvement_notes: signal.manual_fixes ?? "",
-                    }),
-                    task_type: signal.task_type ?? "",
-                    tags: signal.skill_used ? [signal.skill_used] : [],
-                    skill_used: signal.skill_used ?? "",
-                    loop_id: signal.loop_id,
-                    loop_lineage: signal.loop_lineage ?? {},
-                };
-                entries.push(entry);
-            }
-            catch {
-                if (this.metrics)
-                    this.metrics.vaultWriteErrors++;
-                logEvent("vault_write_error", { error: "feedback_entry_build" });
-            }
+                    status: signal.success ? "success" : "partial",
+                    constraint_compliance: {
+                        all_hard_constraints_met: !Array.isArray(signal.violations) || signal.violations.length === 0,
+                        violations: signal.violations ?? [],
+                    },
+                    output_summary: signal.task_type ?? "",
+                    improvement_notes: signal.manual_fixes ?? "",
+                }),
+                task_type: signal.task_type ?? "",
+                tags: signal.skill_used ? [signal.skill_used] : [],
+                skill_used: signal.skill_used ?? "",
+                loop_id: signal.loop_id,
+                loop_lineage: signal.loop_lineage ?? {},
+            };
         }
-        if (entries.length > 0) {
-            try {
-                this.resolveBackend().appendEntries(entries);
-            }
-            catch {
-                if (this.metrics)
-                    this.metrics.vaultWriteErrors += entries.length;
-                logEvent("vault_write_error", { error: "feedback_append_entries", count: entries.length });
-                return 0;
-            }
+        catch {
+            if (this.metrics)
+                this.metrics.vaultWriteErrors++;
+            logEvent("vault_write_error", { error: "feedback_entry_build" });
+            return null;
         }
-        return entries.length;
     }
     // ═══════════════════════════════════════════════════════════════════════
     // Lineage persistence
@@ -490,21 +302,12 @@ export class LoopForgeEngine {
             round: loopRound,
         };
         this.persistFeedbackToVault(signal);
-        // Flush immediately so next compile cycle sees success flags
-        this.flushFeedbackBuffer();
         // Update state
         this.state.call_count++;
         this.state.success_trend.push(success);
         if (this.state.success_trend.length > 20) {
             this.state.success_trend = this.state.success_trend.slice(-20);
         }
-        // Circuit breaker
-        if (this.shouldBreak()) {
-            this.state.circuit_breaker_count++;
-        }
-        else {
-            this.state.circuit_breaker_count = 0;
-        }
         logEvent("round_complete", {
             success,
             loopId: loopId ?? "unknown",
@@ -596,7 +399,6 @@ export class LoopForgeEngine {
         };
         if (!alreadyCommitted) {
             this.persistFeedbackToVault(signal);
-            this.flushFeedbackBuffer();
             if (transactionId && !transactionPersisted()) {
                 throw new Error(`Round feedback commit failed: ${transactionId}`);
             }
@@ -624,13 +426,9 @@ export class LoopForgeEngine {
         if (this.state.success_trend.length > 20) {
             this.state.success_trend = this.state.success_trend.slice(-20);
         }
-        // Circuit breaker
-        if (this.shouldBreak()) {
-            this.state.circuit_breaker_count++;
-        }
-        else {
-            this.state.circuit_breaker_count = 0;
-        }
+        // v2.5: Old binary-success circuit breaker removed — stalled loops
+        // are now detected by the enforcement gate (R4/R5) using
+        // progress_estimate gradients rather than success=true/false count.
         logEvent("round_complete", {
             loopId,
             round,
@@ -644,82 +442,88 @@ export class LoopForgeEngine {
     invokeLoopCompile(request, hydrateResults, options = {}) {
         this.ensureInit(request);
         this.lastTask = request.task;
-        // Build LoopCompileRequest from LoopForgeRequest extras
+        // Parse extras via typed extraction pipeline (loop-extras-parser.ts).
+        // Errors are collected but never thrown — the compiler always gets
+        // best-effort defaults so a malformed request doesn't crash the engine.
         const extras = request;
+        const { parsed, ctx } = parseLoopExtras(extras, request.task_id ?? "");
+        if (ctx.errors.length > 0) {
+            logEvent("extras_parse_errors", {
+                errors: ctx.errors.map((e) => `${e.field}: ${e.message}`),
+            });
+        }
         const lcr = makeLoopCompileRequest({
-            loop_id: extras.loop_id ?? request.task_id ?? "",
-            round: extras.round ?? 1,
-            goal_id: extras.goal_id ?? "",
+            loop_id: parsed.loop_id,
+            round: parsed.round,
+            goal_id: parsed.goal_id,
             task: request.task,
-            domain: extras.domain ?? "",
-            next_task_proposal: extras.next_task_proposal ?? "",
-            plan_source: extras.plan_source ?? null,
-            constraints_from_plan: extras.constraints_from_plan ?? [],
-            new_since_last_round: extras.new_since_last_round ?? "",
-            force_level: extras.force_level ?? "auto",
-            health_check_interval: extras.health_check_interval ?? 1,
-            external_context: extras.external_context ?? "",
-            max_rounds: extras.max_rounds ?? undefined,
-            verification_flags: Array.isArray(extras.verification_flags)
-                ? extras.verification_flags
-                : [],
-            attempt: typeof extras.attempt === "number"
-                ? Math.max(1, Math.trunc(extras.attempt))
-                : 1,
-            consecutive_rejections: typeof extras.consecutive_rejections === "number"
-                ? Math.max(0, Math.trunc(extras.consecutive_rejections))
-                : 0,
-            rejection_notice: typeof extras.rejection_notice === "string"
-                ? extras.rejection_notice
-                : "",
+            domain: parsed.domain,
+            next_task_proposal: parsed.next_task_proposal,
+            plan_source: parsed.plan_source,
+            constraints_from_plan: parsed.constraints_from_plan,
+            new_since_last_round: parsed.new_since_last_round,
+            force_level: parsed.force_level,
+            health_check_interval: parsed.health_check_interval,
+            external_context: parsed.external_context,
+            max_rounds: parsed.max_rounds,
+            verification_flags: parsed.verification_flags,
+            attempt: parsed.attempt,
+            consecutive_rejections: parsed.consecutive_rejections,
+            rejection_notice: parsed.rejection_notice,
         });
-        // Convert last_round_result if present
-        const lastRR = extras.last_round_result;
-        if (lastRR) {
-            if (typeof lastRR === "object" && !Array.isArray(lastRR)) {
-                const rr = lastRR;
-                // Parse P4 execution evidence (shared helper)
-                const executionEvidence = parseExecutionEvidence(rr.execution_evidence);
-                // Parse P5 revised_success_criteria (shared parser)
-                const revisedCriteria = parseCriterionRevisions(rr.revised_success_criteria);
-                lcr.last_round_result = makeLoopRoundResult({
-                    round: rr.round ?? 0,
-                    success: rr.success ?? false,
-                    output_summary: rr.output_summary ?? "",
-                    constraint_violations: rr.constraint_violations ?? [],
-                    manual_fixes_needed: rr.manual_fixes_needed ?? "",
-                    // P0–P2: Cognitive evolution fields
-                    discovered_constraints: Array.isArray(rr.discovered_constraints)
-                        ? rr.discovered_constraints.filter((v) => typeof v === "string")
-                        : [],
-                    objective_refinement: typeof rr.objective_refinement === "string"
-                        ? rr.objective_refinement
-                        : "",
-                    emerged_subtasks: Array.isArray(rr.emerged_subtasks)
-                        ? rr.emerged_subtasks.filter((v) => typeof v === "string")
-                        : [],
-                    // P4: Execution evidence
-                    execution_evidence: executionEvidence,
-                    // P5: Self-correction
-                    retracted_constraints: Array.isArray(rr.retracted_constraints)
-                        ? rr.retracted_constraints.filter((v) => typeof v === "string")
-                        : [],
-                    revised_success_criteria: revisedCriteria,
-                    wrong_assumptions: Array.isArray(rr.wrong_assumptions)
-                        ? rr.wrong_assumptions.filter((v) => typeof v === "string")
-                        : [],
-                    // v1.10: Checkpoint boundary
-                    compression_checkpoint: typeof rr.compression_checkpoint === "boolean" ? rr.compression_checkpoint : false,
-                    checkpoint_label: typeof rr.checkpoint_label === "string" ? rr.checkpoint_label : "",
-                    // Multi-agent: Worker delegation results (shared parser)
-                    worker_results: parseWorkerResults(rr.worker_results),
-                });
-            }
+        // Convert last_round_result if present (object already validated by parser)
+        if (parsed.last_round_result) {
+            const rr = parsed.last_round_result;
+            // Parse P4 execution evidence (shared helper)
+            const executionEvidence = parseExecutionEvidence(rr.execution_evidence);
+            // Parse P5 revised_success_criteria (shared parser)
+            const revisedCriteria = parseCriterionRevisions(rr.revised_success_criteria);
+            lcr.last_round_result = makeLoopRoundResult({
+                round: rr.round ?? 0,
+                success: rr.success ?? false,
+                output_summary: rr.output_summary ?? "",
+                constraint_violations: rr.constraint_violations ?? [],
+                manual_fixes_needed: rr.manual_fixes_needed ?? "",
+                // P0–P2: Cognitive evolution fields
+                discovered_constraints: Array.isArray(rr.discovered_constraints)
+                    ? rr.discovered_constraints.filter((v) => typeof v === "string")
+                    : [],
+                objective_refinement: typeof rr.objective_refinement === "string"
+                    ? rr.objective_refinement
+                    : "",
+                emerged_subtasks: Array.isArray(rr.emerged_subtasks)
+                    ? rr.emerged_subtasks.filter((v) => typeof v === "string")
+                    : [],
+                // P4: Execution evidence
+                execution_evidence: executionEvidence,
+                // P5: Self-correction
+                retracted_constraints: Array.isArray(rr.retracted_constraints)
+                    ? rr.retracted_constraints.filter((v) => typeof v === "string")
+                    : [],
+                revised_success_criteria: revisedCriteria,
+                wrong_assumptions: Array.isArray(rr.wrong_assumptions)
+                    ? rr.wrong_assumptions.filter((v) => typeof v === "string")
+                    : [],
+                // v1.10: Checkpoint boundary
+                compression_checkpoint: typeof rr.compression_checkpoint === "boolean" ? rr.compression_checkpoint : false,
+                checkpoint_label: typeof rr.checkpoint_label === "string" ? rr.checkpoint_label : "",
+                // v2.2: Sub-goal lifecycle
+                completed_subtasks: Array.isArray(rr.completed_subtasks)
+                    ? rr.completed_subtasks.filter((v) => typeof v === "string")
+                    : [],
+                blocked_subtasks: Array.isArray(rr.blocked_subtasks)
+                    ? rr.blocked_subtasks.filter((v) => typeof v === "string")
+                    : [],
+                canceled_subtasks: Array.isArray(rr.canceled_subtasks)
+                    ? rr.canceled_subtasks.filter((v) => typeof v === "string")
+                    : [],
+                // Multi-agent: Worker delegation results (shared parser)
+                worker_results: parseWorkerResults(rr.worker_results),
+            });
         }
-        // Convert loop_objective if present
-        const lo = extras.loop_objective;
-        if (lo && typeof lo === "object" && !Array.isArray(lo)) {
-            const obj = lo;
+        // Convert loop_objective if present (object already validated by parser)
+        if (parsed.loop_objective) {
+            const obj = parsed.loop_objective;
             lcr.loop_objective = makeLoopObjective({
                 objective: obj.objective ?? "",
                 success_criteria: obj.success_criteria ?? [],
@@ -760,30 +564,10 @@ export class LoopForgeEngine {
                 error: null,
                 state_file_content: response.state_file_content,
                 prompt_artifact: response.prompt_artifact,
+                warnings: response.warnings,
             },
         };
     }
-    // ═══════════════════════════════════════════════════════════════════════
-    // Circuit breaker
-    // ═══════════════════════════════════════════════════════════════════════
-    shouldBreak() {
-        if (!this.state)
-            return false;
-        const policy = getPolicy();
-        const maxCB = policy.engine.max_circuit_breaker;
-        if (this.state.success_trend.length < maxCB)
-            return false;
-        const recent = this.state.success_trend.slice(-maxCB);
-        // Trip only when all recent rounds are failures (no false-positives on all-success)
-        const allFailed = recent.every((v) => v === false);
-        if (allFailed) {
-            logEvent("circuit_breaker", {
-                trend: recent,
-                totalRounds: this.state.success_trend.length,
-            });
-        }
-        return allFailed;
-    }
 }
 // ═══════════════════════════════════════════════════════════════════════════
 // Factory