@tea-agent/loop-agent 0.39.0-next.10 → 0.39.0-next.11

package/CHANGELOG.md

@@ -23,6 +23,7 @@
 
 ### 修复
 
+- `frontend-plan-revision-pi` 节点级校验与 merge-patch 输出协议矛盾修复：新增 schemaId `frontend-implementation-contract-revision-patch-v1` 与格式级宽松校验 `validateFrontendRevisionPatchNodeOutput`（至少一个 `openspec-citations` 块 + 剥离引用块后恰好一个 JSON 对象，不做完整契约 schema 校验），revision 节点 `structuredContractOutput.schemaId` 切换（plan-pi 仍为 v1）；node-execution 的 invalid-output retry 指令按 schemaId 分支，patch 协议提示 merge-patch delta + 引用块，消除「prompt 要求 patch、retry 要求完整契约」的 retryable 死循环；gate 仍是唯一权威合并/校验点（零改动）。ADR 0015 补「节点级校验」决策 7
 - `frontend-implementation-contract.schema.json` 的 `uiComponentChoices` items 补 `if/then` 决策语义（`specified` ⇒ `specReference` 必填非 null；`new` ⇒ `specReference` 为 null/缺失）；`frontend-plan-render.ts` 空 `section` 不再渲染尾随 `#`；ADR 0016 增补「门禁边界」小节
 
 ### 文档

package/dist/build-stamp.json

@@ -1,6 +1,6 @@
 {
   "schemaVersion": 1,
   "version": "0.37.0",
-  "gitSha": "9fc46a3aa984418eaa0796bc9b5abcdd5d43c857",
-  "builtAt": "2026-08-18T07:50:46.364Z"
+  "gitSha": "7bf68166795e6687e3e804901a8a6ceda3a46825",
+  "builtAt": "2026-08-18T13:08:32.390Z"
 }

package/dist/workflows/dag/contract-validator-registrations.js

@@ -3,6 +3,7 @@
  * its schemaId in the contract output registry. Imported once by the node
  * executor; keeps per-contract imports out of the executor.
  */
-import { validateFrontendContractNodeOutput } from "./frontend-implementation-contract.js";
+import { validateFrontendContractNodeOutput, validateFrontendRevisionPatchNodeOutput, } from "./frontend-implementation-contract.js";
 import { registerStructuredContractValidator } from "./contract-output-registry.js";
 registerStructuredContractValidator("frontend-implementation-contract-v1", validateFrontendContractNodeOutput);
+registerStructuredContractValidator("frontend-implementation-contract-revision-patch-v1", validateFrontendRevisionPatchNodeOutput);

package/dist/workflows/dag/frontend-implementation-contract.js

@@ -1632,6 +1632,46 @@ export async function validateFrontendContractNodeOutput(input) {
         };
     }
 }
+/**
+ * Node-output self-check for the plan revision node, which emits an RFC 7386
+ * merge-patch delta against the original contract instead of a full contract.
+ * Deliberately loose and format-level: the output must contain at least one
+ * ```openspec-citations fenced block and must extract (after the citations
+ * blocks are stripped) to exactly one JSON object. The delta carries no
+ * schemaVersion/targets, so no full-contract schema/semantic checks run here;
+ * content parsing, patch application, and merged-contract validation stay with
+ * the prewrite gate, which remains the only authority.
+ */
+export async function validateFrontendRevisionPatchNodeOutput(input) {
+    // Mirror the prewrite gate's block-presence semantics (a fenced
+    // ```openspec-citations block with a newline-terminated body).
+    if (!/```openspec-citations[ \t]*\r?\n[\s\S]*?\r?\n```/.test(input.text)) {
+        return {
+            ok: false,
+            reason: "revision patch output must include at least one ```openspec-citations fenced block",
+        };
+    }
+    // Strip the citation blocks before extraction: their per-row JSON objects
+    // would otherwise count as competing balanced objects in the extractor.
+    const textWithoutCitations = input.text.replace(/```openspec-citations[ \t]*\r?\n[\s\S]*?\r?\n```/g, "");
+    let patch;
+    try {
+        patch = extractFrontendImplementationJson(textWithoutCitations);
+    }
+    catch (error) {
+        return {
+            ok: false,
+            reason: `revision patch output must contain exactly one valid JSON object: ${error instanceof Error ? error.message : String(error)}`,
+        };
+    }
+    if (!patch || typeof patch !== "object" || Array.isArray(patch)) {
+        return {
+            ok: false,
+            reason: "revision patch output must extract to exactly one JSON object (RFC 7386 merge-patch delta)",
+        };
+    }
+    return { ok: true, contract: patch };
+}
 export async function materializeFrontendImplementationContract(input) {
     const analysis = await analyzeFrontendImplementationContract(input);
     return writeFrontendImplementationContractArtifact({

package/dist/workflows/dag/init-hybrid.js

@@ -2859,7 +2859,7 @@ async function buildFrontendHybridDagFromTask(sources) {
                 outputMode: "structured-required",
                 retryPolicy: STRUCTURED_REQUIRED_PI_RETRY_POLICY,
                 structuredContractOutput: {
-                    schemaId: "frontend-implementation-contract-v1",
+                    schemaId: "frontend-implementation-contract-revision-patch-v1",
                     retryOnInvalid: true,
                 },
                 allowedPaths: readOnlyPaths,

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

@@ -144,15 +144,27 @@ function buildAttemptPrompt(task, basePrompt, attemptNumber, previousFailureCate
     if (previousFailureCategory === "invalid-output" &&
         task.structuredContractOutput &&
         previousProtocolReason) {
+        // Revision nodes emit an RFC 7386 merge-patch delta (no schemaVersion /
+        // targets), so the retry instruction must not push them toward a full
+        // contract rewrite; plan nodes keep the strict v1 guidance.
+        const guidance = task.structuredContractOutput.schemaId ===
+            "frontend-implementation-contract-revision-patch-v1"
+            ? [
+                "Return exactly one fenced json object containing the RFC 7386 merge-patch delta against the original contract (null deletes a key; arrays and scalars replace; plain objects merge recursively) + exactly one openspec-citations block.",
+                "The delta contains only the fields you change; do not emit a full contract or a schemaVersion/targets section.",
+            ]
+            : [
+                "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.",
+                "The Markdown explanation may be omitted; prioritize a complete contract.",
+            ];
         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.",
-            "The Markdown explanation may be omitted; prioritize a complete contract.",
+            ...guidance,
             "</retry_instruction>",
         ].join("\n");
     }

package/dist/workflows/dag/types.js

@@ -704,6 +704,7 @@ export const dagConvergenceSpecSchema = z
  */
 export const structuredContractOutputSchemaIds = [
     "frontend-implementation-contract-v1",
+    "frontend-implementation-contract-revision-patch-v1",
 ];
 export const dagTaskSchema = z.object({ id: z.string().regex(/^[a-z][a-z0-9-]*$/, "task id must be kebab-case"),
     depends_on: z.array(z.string()).default([]),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tea-agent/loop-agent",
-  "version": "0.39.0-next.10",
+  "version": "0.39.0-next.11",
   "type": "module",
   "bin": {
     "loop-agent": "bin/loop-agent.js",