@tea-agent/loop-agent 0.36.4-beta.0 → 0.37.1-beta.0

package/dist/workflows/dag/lifecycle.js

@@ -743,7 +743,6 @@ function findDoctorFailureNode(state) {
             nodeId: state.pausedByNodeId,
             status: node?.status,
             rawFailureCategory: node?.failureCategory,
-            skippedReason: node?.skippedReason,
         };
     }
     const errorEntry = Object.entries(state.nodes).find(([, node]) => node.status === "ERROR");
@@ -759,7 +758,6 @@ function findDoctorFailureNode(state) {
         nodeId: selected[0],
         status: selected[1].status,
         rawFailureCategory: selected[1].failureCategory,
-        skippedReason: selected[1].skippedReason,
     };
 }
 async function readRunOwnedBackendTestClassification(runDir) {
@@ -788,7 +786,6 @@ async function resolveDoctorFailureRouting(input) {
             normalizedFailureCategory: "unknown",
             nodeId: input.nodeId,
             productLineFailureCategory: classifiedCategory,
-            skippedReason: input.skippedReason,
         });
     }
     return routeDagFailure(input);
@@ -810,7 +807,6 @@ async function formatDagDoctorMarkdown(repoRoot, runId) {
         rawFailureCategory,
         normalizedFailureCategory: normalizedCategory,
         nodeId: failure.nodeId,
-        skippedReason: failure.skippedReason,
     });
     const evidence = failure.nodeId
         ? path.join(located.runDir, failure.nodeId, "result.summary.md")

package/dist/workflows/dag/node-execution.js

@@ -4,7 +4,6 @@ import { readFile } from "node:fs/promises";
 import path from "node:path";
 import { pathMatchesPattern } from "../../shared/git-progress.js";
 import { recordDecisionEnvelopeForNode, shouldPauseOnHumanEscalation, writeHumanEscalationArtifacts, } from "./decision-envelope.js";
-import { FRONTEND_PREWRITE_RESULT_SOURCE_ARTIFACT, FRONTEND_WRITER_NODE_IDS, isFrontendWriterAuthorized, readFrontendPrewriteResult, } from "./scheduler.js";
 import { writeNodeRecord, writeNodeSkillArtifacts } from "./run-store.js";
 import { resolveContextPolicy } from "./context-policy.js";
 import { buildDagNodePromptEnvelope, formatConvergenceFeedbackBlock, } from "./prompt.js";
@@ -13,8 +12,6 @@ import { buildOutputLimitRecoverySection, loadBackendTestWriterProgressForRetry,
 import { computeBackoffDelayMs, isRetryablePiFailureCategory, isSafeReadOnlyPiRetryCandidate, isWriterEmptyDiffRetryCandidate, isWriterTransportRetryCandidate, } from "./retry-policy.js";
 import { applyNodeActivity, evaluateNodeLiveness, resolveLivenessPolicy, } from "./liveness-policy.js";
 import { buildProtocolRetryInstruction, normalizeReviewVerdictAfterRetries, parseJsonReviewVerdict, validateOutputProtocol, } from "./output-protocol.js";
-import { getStructuredContractValidator } from "./contract-output-registry.js";
-import "./contract-validator-registrations.js";
 import { writeDagNodeJsonArtifact } from "../../infrastructure/harness/artifact-store.js";
 import { buildProjectGovernanceContext, readCompletedWriterChangeManifests, writeProjectGovernanceContext, } from "./project-governance-context.js";
 import { assertSkillSnapshotCoversSpec, buildNodePromptFromSnapshot, isDagSkillSnapshotIntegrityError, readSkillSnapshot, } from "./skill-snapshot.js";
@@ -141,20 +138,6 @@ function buildAttemptPrompt(task, basePrompt, attemptNumber, previousFailureCate
             buildProtocolRetryInstruction(task.outputProtocol, previousProtocolReason),
         ].join("\n");
     }
-    if (previousFailureCategory === "invalid-output" &&
-        task.structuredContractOutput &&
-        previousProtocolReason) {
-        return [
-            basePrompt,
-            "",
-            "<retry_instruction>",
-            "Previous attempt produced an invalid frontend implementation contract:",
-            previousProtocolReason,
-            "Return exactly one fenced json block conforming to the frontend-implementation-contract-v1 schema.",
-            "Fix every reported field violation: do not emit null for optional fields, do not misspell field names, and match the required types exactly.",
-            "</retry_instruction>",
-        ].join("\n");
-    }
     if (previousFailureCategory === "writer-empty-diff") {
         const maxAttempts = task.retryPolicy?.maxAttempts ?? 3;
         // When a completeness progress exists for this writer, fold the concrete
@@ -216,28 +199,18 @@ function buildAttemptPrompt(task, basePrompt, attemptNumber, previousFailureCate
             "</retry_instruction>",
         ].join("\n");
     }
-    if (previousFailureCategory !== "output-too-large") {
+    if (task.outputMode !== "structured-required" ||
+        previousFailureCategory !== "output-too-large") {
         return basePrompt;
     }
-    if (task.outputMode === "structured-required") {
-        return [
-            basePrompt,
-            "",
-            "<retry_instruction>",
-            "Previous attempt exceeded the structured output size limit.",
-            "Return only the compact structured artifact required by this node's output contract.",
-            "Do not include explanatory prose, duplicated upstream context, long evidence excerpts, or additional markdown sections.",
-            "If a fenced JSON object is required, output exactly one fenced json block and nothing else.",
-            "</retry_instruction>",
-        ].join("\n");
-    }
     return [
         basePrompt,
         "",
         "<retry_instruction>",
-        "Previous attempt exceeded the output size limit.",
-        "Return a minimal plan: ordered steps, narrow writeSet boundaries, and verification commands.",
-        "Drop verbatim upstream quotes, long evidence excerpts, and repeated context.",
+        "Previous attempt exceeded the structured output size limit.",
+        "Return only the compact structured artifact required by this node's output contract.",
+        "Do not include explanatory prose, duplicated upstream context, long evidence excerpts, or additional markdown sections.",
+        "If a fenced JSON object is required, output exactly one fenced json block and nothing else.",
         "</retry_instruction>",
     ].join("\n");
 }
@@ -422,21 +395,6 @@ export async function executeDagNode(input) {
         await notifyNodeObserver(input.observer, "onNodeFinish", nodeId, state);
     };
     const failSkillSnapshot = (error) => failBeforePrompt(error, "skill-snapshot-integrity");
-    const skipFrontendWriter = async (admission) => {
-        const skippedAt = new Date().toISOString();
-        node.startedAt ??= skippedAt;
-        node.frontendWriterAdmission = admission;
-        node.status = "SKIPPED";
-        node.skippedReason = "frontend-prewrite-not-authorized";
-        node.finishedAt = skippedAt;
-        node.lastActivityAt = skippedAt;
-        node.durationMs = durationBetween(node.startedAt, node.finishedAt);
-        node.timing = { retryBackoffMs: 0, settlementCleanupMs: 0 };
-        state.nodes[nodeId].nodeRecordPath = path.join(runDir, `${nodeId}.json`);
-        await writeNodeRecord(runDir, nodeId, state.nodes[nodeId]);
-        await input.persistState();
-        await notifyNodeObserver(input.observer, "onNodeFinish", nodeId, state);
-    };
     if (task.finalWriteSetApproval) {
         const authorization = parseAndValidateFinalWriteSetApproval({ task, spec, state });
         if (!authorization.ok) {
@@ -465,30 +423,6 @@ export async function executeDagNode(input) {
             effectiveWriteSet: [...authorization.effectiveWriteSet],
         };
     }
-    if (FRONTEND_WRITER_NODE_IDS.includes(nodeId)) {
-        const admission = await readFrontendPrewriteResult(runDir);
-        if (!admission.ok) {
-            await skipFrontendWriter(undefined);
-            return;
-        }
-        const decision = isFrontendWriterAuthorized(admission.result);
-        const record = {
-            schemaVersion: 1,
-            writerNodeId: nodeId,
-            decision,
-            sourceArtifact: FRONTEND_PREWRITE_RESULT_SOURCE_ARTIFACT,
-            artifactHash: admission.artifactHash,
-            checkedAt: new Date().toISOString(),
-            reason: decision === "denied"
-                ? `classification: ${admission.result.classification}`
-                : null,
-        };
-        if (decision === "denied") {
-            await skipFrontendWriter(record);
-            return;
-        }
-        node.frontendWriterAdmission = record;
-    }
     let projectGovernanceContext;
     if (task.governanceStandardReview) {
         try {
@@ -755,39 +689,6 @@ export async function executeDagNode(input) {
                 previousProtocolReason = undefined;
             }
         }
-        // R1: structured contract nodes self-validate their output so schema,
-        // typo, and null violations surface as retryable invalid-output at the
-        // producing node instead of failing the whole run at the prewrite gate.
-        if (result.ok && task.structuredContractOutput) {
-            const validator = getStructuredContractValidator(task.structuredContractOutput.schemaId);
-            if (validator) {
-                const contractText = canonicalNodeOutput(result);
-                const contractCheck = await validator({
-                    runDir,
-                    text: contractText,
-                    sourceBinding: spec.sourceBinding,
-                });
-                if (!contractCheck.ok) {
-                    result = {
-                        ...result,
-                        ok: false,
-                        failureCategory: "invalid-output",
-                        stderr: [result.stderr, contractCheck.reason]
-                            .filter(Boolean)
-                            .join("\n"),
-                    };
-                    previousProtocolReason = contractCheck.reason;
-                }
-                else {
-                    previousProtocolReason = undefined;
-                }
-            }
-            else {
-                // DAG spec validation rejects unknown schemaIds; this is a
-                // defensive fallback for a registry that has not been populated.
-                console.warn(`[run-dag] warning: no structured contract validator registered for schemaId ${task.structuredContractOutput.schemaId}; skipping node self-check`);
-            }
-        }
         const attemptFinishedAt = new Date().toISOString();
         const attemptWallDurationMs = durationBetween(attemptStartedAt, attemptFinishedAt);
         totalAttemptWallDurationMs += attemptWallDurationMs;

package/dist/workflows/dag/recovery-recommendation.js

@@ -269,61 +269,3 @@ export function planDagRecovery(input) {
     }
     return planByNormalizedCategory(input);
 }
-/**
- * Phase 6: route a frontend recovery outcome to a recovery CTA (read-only; never
- * mutates a run's status). The six outcomes map to the existing recovery actions
- * so the Console recovery page reuses the current CTA surface.
- */
-export function recommendFrontendRecoveryOutcome(outcome) {
-    switch (outcome) {
-        case "none":
-            return {
-                action: "none",
-                summary: "未发生前端恢复；按普通终态展示。",
-                reason: "该 frontend-implementation 根没有触发自动恢复。",
-                humanRequired: false,
-                autoRetryEligible: false,
-            };
-        case "recovered":
-            return {
-                action: "none",
-                summary: "自动恢复成功；child 已重新实现并通过验证。",
-                reason: "parent 的 transient 失败已由 child 恢复，全链按 root 计一次成功。",
-                humanRequired: false,
-                autoRetryEligible: false,
-            };
-        case "candidate-contract-invalid":
-            return {
-                action: "manual-review",
-                summary: "方案输出无效：candidate JSON 无法解析/推导，重试用尽。",
-                reason: "建议人工检查需求或重新生成方案后再跑。",
-                humanRequired: true,
-                autoRetryEligible: false,
-            };
-        case "prewrite-blocked":
-            return {
-                action: "manual-review",
-                summary: "治理门禁阻断：source stale / writeSet 越界 / Mock policy / contract 违规。",
-                reason: "需人工修正任务边界/事实后重跑，不自动重试。",
-                humanRequired: true,
-                autoRetryEligible: false,
-            };
-        case "repair-exhausted":
-            return {
-                action: "rerun-after-fix",
-                summary: "自动修复未通过：verify 失败，1 次 repair 后 reverify 仍失败。",
-                reason: "建议 `dag rerun` 或人工排查。",
-                humanRequired: true,
-                autoRetryEligible: false,
-                commandHint: "loop-agent dag rerun --run-id <runId>",
-            };
-        case "auto-recovery-blocked":
-            return {
-                action: "manual-review",
-                summary: "自动恢复被阻止：rollback 无法证明完整或存在外部/越界修改。",
-                reason: "必须人工介入，禁止自动覆盖用户文件。",
-                humanRequired: true,
-                autoRetryEligible: false,
-            };
-    }
-}

package/dist/workflows/dag/report.js

@@ -441,7 +441,6 @@ export async function buildDagRunReportEntry(input) {
                 normalizedFailureCategory,
                 nodeId,
                 executor: node.executor,
-                skippedReason: node.skippedReason,
             });
             const followUp = node.status === "ERROR" || node.status === "SKIPPED"
                 ? recommendFollowUpForFailureCategory(node.failureCategory)
@@ -506,7 +505,6 @@ export async function buildDagRunReportEntry(input) {
             normalizedFailureCategory: normalizeDagFailureCategory(pausedNode.failureCategory, pausedNode.status),
             failureCategory: pausedNode.failureCategory,
             nodeId: input.state.pausedByNodeId,
-            skippedReason: pausedNode.skippedReason,
         }
         : firstActionableNode
             ? {
@@ -515,7 +513,6 @@ export async function buildDagRunReportEntry(input) {
                     normalizeDagFailureCategory(firstActionableNode.failureCategory, firstActionableNode.status),
                 failureCategory: firstActionableNode.failureCategory,
                 nodeId: firstActionableNode.nodeId,
-                skippedReason: input.state.nodes[firstActionableNode.nodeId]?.skippedReason,
             }
             : {
                 status: input.state.status,
@@ -532,9 +529,6 @@ export async function buildDagRunReportEntry(input) {
         rawFailureCategory: runRecoverySource.failureCategory,
         normalizedFailureCategory: runRecoverySource.normalizedFailureCategory,
         nodeId: "nodeId" in runRecoverySource ? runRecoverySource.nodeId : undefined,
-        skippedReason: "skippedReason" in runRecoverySource
-            ? runRecoverySource.skippedReason
-            : undefined,
     });
     const runFollowUp = input.state.status === "failed" ||
         input.state.status === "partial_failed"

package/dist/workflows/dag/retry-policy.js

@@ -118,17 +118,6 @@ export const STRUCTURED_REQUIRED_PI_RETRY_POLICY = {
     ...DEFAULT_READ_ONLY_PI_RETRY_POLICY,
     retryCategories: [...STRUCTURED_REQUIRED_DAG_RETRY_CATEGORIES],
 };
-/**
- * Planner read-only nodes (standard DAG contract-pi / plan-pi) may retry with
- * a compact-output instruction when the model produced an oversized assistant
- * response. Only `output-too-large` is added on top of the default set;
- * report/review and other read-only roles keep the default set so they do not
- * silently learn new output semantics.
- */
-export const PLANNER_OUTPUT_LIMIT_RETRY_POLICY = {
-    ...DEFAULT_READ_ONLY_PI_RETRY_POLICY,
-    retryCategories: [...DEFAULT_DAG_RETRY_CATEGORIES, STRUCTURED_OUTPUT_RETRY_CATEGORY],
-};
 /**
  * Default retry for reviewer / recovery nodes that declare outputProtocol.
  * Includes protocol-invalid so missing VERDICT lines are corrected in-node.