@tea-agent/loop-agent 0.39.0-next.17 → 0.39.0-next.19

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

@@ -129,6 +129,26 @@ export function buildNodePrompt(spec, task, upstream, options) {
         convergenceFeedback: options?.convergenceFeedback,
     });
 }
+function frontendStructuredArtifactRetryGuidance(schemaId) {
+    if (schemaId === "frontend-implementation-contract-plan-patch-v1") {
+        return [
+            "Return exactly this artifact and nothing else: one short lead-in line, then exactly one fenced json block containing only the editable RFC 7386 plan patch, immediately followed by exactly one fenced openspec-citations block.",
+            "The runtime applies your patch to its protected contract skeleton. Do not emit schemaVersion, sourceBinding, riskLevel, targets.files, or mockApi.productionDefaultOff; do not emit the full contract.",
+            "Do not emit Markdown headings, bullets, plan prose, explanations, raw JSON, or any other fenced block.",
+        ];
+    }
+    if (schemaId === "frontend-implementation-contract-revision-patch-v1") {
+        return [
+            "Return exactly this artifact and nothing else: one short lead-in line, then exactly one fenced json block containing the RFC 7386 merge-patch delta, immediately followed by exactly one fenced openspec-citations block.",
+            "The delta contains only the fields you change; null deletes a key, arrays and scalars replace, and plain objects merge recursively. Do not emit a full contract or a schemaVersion/targets section.",
+            "Do not emit Markdown headings, bullets, plan prose, explanations, raw JSON, or any other fenced block.",
+        ];
+    }
+    return [
+        "Return exactly this artifact and nothing else: one short lead-in line, then exactly one fenced json block conforming to frontend-implementation-contract-v1, immediately followed by exactly one fenced openspec-citations block.",
+        "The contract JSON fields are the complete implementation plan. Do not emit a separate plan document, Markdown headings, bullets, plan prose, explanations, raw JSON, or any other fenced block.",
+    ];
+}
 function buildAttemptPrompt(task, basePrompt, attemptNumber, previousFailureCategory, previousProtocolReason, recoveryTargetPaths, recoveryDiagnostics) {
     if (attemptNumber <= 1)
         return basePrompt;
@@ -144,27 +164,14 @@ 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,
-            ...guidance,
+            ...frontendStructuredArtifactRetryGuidance(task.structuredContractOutput.schemaId),
+            "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");
     }
@@ -175,9 +182,8 @@ function buildAttemptPrompt(task, basePrompt, attemptNumber, previousFailureCate
             "",
             "<retry_instruction>",
             "Previous attempt was truncated by the provider (stopReason=length) before the JSON contract was completed.",
-            "This attempt: output ONLY the single fenced json contract block, immediately followed by exactly one openspec-citations block.",
-            "Do not emit any Markdown explanation, evidence excerpts, or duplicated upstream context.",
-            "The contract JSON must be complete; the trailing Markdown explanation may be omitted entirely.",
+            ...frontendStructuredArtifactRetryGuidance(task.structuredContractOutput.schemaId),
+            "The JSON artifact must be complete; omit evidence excerpts and duplicated upstream context.",
             "</retry_instruction>",
         ].join("\n");
     }
@@ -246,6 +252,17 @@ function buildAttemptPrompt(task, basePrompt, attemptNumber, previousFailureCate
         return basePrompt;
     }
     if (task.outputMode === "structured-required") {
+        if (task.structuredContractOutput) {
+            return [
+                basePrompt,
+                "",
+                "<retry_instruction>",
+                "Previous attempt exceeded the structured output size limit.",
+                ...frontendStructuredArtifactRetryGuidance(task.structuredContractOutput.schemaId),
+                "Keep every required field but make values concise; omit evidence excerpts and duplicated upstream context.",
+                "</retry_instruction>",
+            ].join("\n");
+        }
         return [
             basePrompt,
             "",
@@ -792,6 +809,9 @@ export async function executeDagNode(input) {
                     runDir,
                     text: contractText,
                     sourceBinding: spec.sourceBinding,
+                    nodeId,
+                    attempt: attemptNumber,
+                    structuredContractOutput: task.structuredContractOutput,
                 });
                 if (!contractCheck.ok) {
                     // A provider stopReason=length means the response was cut before the
@@ -811,6 +831,13 @@ export async function executeDagNode(input) {
                     previousProtocolReason = contractCheck.reason;
                 }
                 else {
+                    if (contractCheck.normalizedText) {
+                        result = {
+                            ...result,
+                            assistantText: contractCheck.normalizedText,
+                            stdout: contractCheck.normalizedText,
+                        };
+                    }
                     previousProtocolReason = undefined;
                 }
             }

package/dist/workflows/dag/types.js

@@ -713,6 +713,7 @@ export const dagConvergenceSpecSchema = z
  */
 export const structuredContractOutputSchemaIds = [
     "frontend-implementation-contract-v1",
+    "frontend-implementation-contract-plan-patch-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"),
@@ -775,8 +776,25 @@ export const dagTaskSchema = z.object({ id: z.string().regex(/^[a-z][a-z0-9-]*$/
         .object({
         schemaId: z.enum(structuredContractOutputSchemaIds),
         retryOnInvalid: z.boolean().default(true),
+        /**
+         * Optional deterministic base object for patch-producing structured
+         * nodes. The frontend plan patch validator applies the model delta to
+         * this run-owned skeleton before strict contract validation.
+         */
+        skeleton: z.record(z.string(), z.unknown()).optional(),
     })
         .strict()
+        .superRefine((value, ctx) => {
+        if (value.schemaId ===
+            "frontend-implementation-contract-plan-patch-v1" &&
+            !value.skeleton) {
+            ctx.addIssue({
+                code: "custom",
+                message: "frontend plan patch output requires a runtime skeleton",
+                path: ["skeleton"],
+            });
+        }
+    })
         .optional(),
     /**
      * Fail-closed outcome/diff consistency contract for bounded Pi writers.

package/docs/templates/harness.schema.json

@@ -344,14 +344,6 @@
         "HIGH": {
           "$ref": "#/$defs/executorTierModel",
           "description": "HIGH 复杂度 DAG task 的模型覆盖值，优先级高于 defaultModel。可为模型字符串，或 { model, thinking? } 对象。"
-        },
-        "modelOptions": {
-          "type": "array",
-          "items": {
-            "type": "string",
-            "minLength": 1
-          },
-          "description": "Console「模型与认证」页模型矩阵下拉的可选模型列表（provider/model 或裸 model id）。仅供下拉数据使用，不参与 DAG 模型路由。"
         }
       }
     }

package/harness.json

@@ -84,14 +84,7 @@
       "description": "Pi 负责规划、评审、诊断；当 DAG toolProfile=write 时也可做有界写入。模型按复杂度三档配置，支持 provider/model 字符串或带 thinking 的对象；斜杠前为 Pi provider，后为 modelId，勿只写裸 modelId。",
       "LOW": "wizard-local/minimax-m3",
       "MED": "wizard-local/grok-4.6",
-      "HIGH": "wizard-local/gpt-5.6-sol",
-      "modelOptions": [
-        "wizard-local/minimax-m3",
-        "wizard-local/grok-4.6",
-        "wizard-local/gpt-5.6-sol",
-        "deepseek/deepseek-v4-pro",
-        "deepseek/deepseek-v4-flash"
-      ]
+      "HIGH": "wizard-local/gpt-5.6-sol"
     }
   }
 }

package/package.json

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

package/skills/frontend-implementation/references/node-contracts.md

@@ -6,14 +6,14 @@ Pre-write nodes are read-only. Preserve IDs, labels, commands, language, require
 
 - **`frontend-contract-pi`**: `Scope`, `Non-goals`, `Acceptance Criteria`, `UI States`, `Target Runtime Environment`, `Risks`, `Verification Expectations`; no guessed requirements.
 - **`frontend-scout-pi`**: routes, components, tokens, data/API/Mock, scripts, tests, assets; fact vs inference vs gap. Search+read `openspec/schemas/`, `openspec/project-specs/`, `ai_workspace/` first. Output stack, styling, conventions, state/data, test entry points, reuse, risks.
-- **`frontend-plan-pi` + conditional design loop**: AC → observable `expectedOutcome`; interactions → `trigger`+`expectedBehavior`; steps, in-bound files, UI states, reuse, deps, Mock/API strategy, activation/rollback, frozen verify entrypoints, real-integration gap. **JSON-only**: one-line lead-in + **exactly one** fenced `json` block (`frontend-implementation-contract-v1`) + one `openspec-citations` block. No plan prose; plan.md is rendered by the gate. `frontend-contract-json-pi`/`frontend-contract-json-validate-shell` don't exist; IDs+paths ≠ behavior semantics. Use `uiStates: []` for logic-only changes; don't invent UI states. Applicable states need behavior/implementation/verification; non-applicable need reasons. Non-`not-needed` needs frozen Mock verify commands (`frontendMock.verifyCommands`/`package.json` mock script/capability seed); `auto` with absent/ambiguous capability or no command → `not-needed`; `required` without a command → generation-time blocked. Initial pass = original plan; only `request-revision` runs revision+final review; small-risk = one review.
+- **`frontend-plan-pi` + conditional design loop**: AC → observable `expectedOutcome`; interactions → `trigger`+`expectedBehavior`; include steps, in-bound files, UI states, reuse, deps, Mock/API, frozen verify entrypoints and integration gaps. DAG freezes a protected skeleton (`schemaVersion`, `sourceBinding`, `riskLevel`, `targets.files`, `mockApi.productionDefaultOff`). Model returns a **JSON-only** editable RFC 7386 patch (`frontend-implementation-contract-plan-patch-v1`) plus one `openspec-citations` block. Runtime compiles canonical `frontend-implementation-contract-v1` and archives raw/extracted/normalized/validation candidates; unresolved optional symbols are dropped and audited. No plan prose; gate renders plan.md. IDs+paths ≠ behavior semantics. Applicable UI states need behavior/implementation/verification; non-applicable states need reasons. Active Mock needs frozen commands; `auto` without confirmed capability/command → `not-needed`; `required` without command → generation-time blocked. Only `request-revision` runs revision+final review; small-risk gets one review.
   - `uiComponentChoices[]` (optional): `{purpose, component, decision, specReference, rationale}`; `specified` needs `specReference{path,section,line}`; `new` needs null ref+rationale.
 - **`frontend-prewrite-gate-shell`**: sole write authorization. Resolve effective plan (revised if revision ran, else original) + review; require exact pass; retain every REQ/BR/AC id; enforce Mock policy; validate schema/source binding+writeSet containment; materialize `contracts/frontend-implementation-contract.json` from the **single** fenced contract block (multiple candidates fail closed `invalid-output`). Fallback only when conditional primary absent; malformed primary fails closed. Generation-time blocked Mock yields one blocking shell node, no writer.
   - `verificationTarget.commandLabel` must be a **frozen command label** from `verifyEvidence.commandLabels`; others fail closed (`invalid-output`). Empty frozen set (no `run.json`) skips check.
   - `mockApi.strategy !== "not-needed"` fails closed when `mockCommandLabels` empty (`no authorized Mock verification commands`).
   - OpenSpec policy: `task.json.frontendOpenspec.policy` (default `cited`); candidates = `requiredReadPaths` ∪ task-source citations; effective plan must append one fenced `openspec-citations` block (one JSON `{"path","section","line"}` per line). Candidates non-empty but block missing/unparseable → `openspec-citation-block-unparseable`; candidate not cited → `openspec-not-cited`; cited path without read event → `openspec-citation-not-read` (anti-fabrication). Empty candidates skip enforcement. `scan-strict` keeps full-read semantics (`openspec-not-read`); both modes keep `candidate-missing-drift`.
   - Component conformance: missing `uiComponentChoices` (frozen bucket non-empty + UI-visible work) → `component-choices-missing`; bad/uncited `specReference.path` → `component-spec-reference-invalid` / `component-spec-not-cited`.
-  - revisionPatch mode: revision output is an RFC 7386 merge-patch delta applied on the original contract node (`planFallbackFromNodeIds[0]`); legacy DAGs keep full-contract semantics. Size guard: canonical serialization >64KB → `retryable-invalid` + `contract-too-large`.
+  - revisionPatch mode: revision output is an RFC 7386 merge-patch delta applied on the original canonical contract node (`planFallbackFromNodeIds[0]`); protected skeleton paths remain immutable. Legacy DAGs keep full-contract semantics. Size guard: canonical serialization >64KB → `retryable-invalid` + `contract-too-large`.
 - **`frontend-implement-pi`**: sole regular exclusive writer; uses `frontend-bounded-implement` (not this skill). Stay in `writeSet`; real requests default-on. Atomic handler/intercept/adapter with tests. Stop on forbidden paths/guesses. First line `IMPLEMENTATION_OUTCOME: changed|already-satisfied|blocked`; runtime checks against attributed diff. Optional mock-verify when frozen; static+behavior always; behavior must prove page consumption. `not-needed` keeps real integration pending unless real backend has fresh evidence.
 
 ## Contract / trace / stages (M1–M2)

package/dist/worker/console/static/assets/channel-Kf0dWmvg.js

@@ -1 +0,0 @@
-import{ag as o,ah as n}from"./mermaid.core-DeA6yB6l.js";const t=(a,r)=>o.lang.round(n.parse(a)[r]);export{t as c};

package/dist/worker/console/static/assets/classDiagram-JCYQIIEL-DvMFMDx0.js

@@ -1 +0,0 @@
-import{s as a,c as s,a as e,C as t}from"./chunk-GF5L2VYU-CTNubrNg.js";import{_ as i}from"./mermaid.core-DeA6yB6l.js";import"./chunk-5VM5RSS4-aCdt3uIm.js";import"./chunk-XXDRQBXY-Ct6GraPo.js";import"./chunk-KBJHAD2P-51xTGTO2.js";import"./chunk-2GRJ4B5K-DbYJpklA.js";import"./index-BTNvk20w.js";var n={parser:e,get db(){return new t},renderer:s,styles:a,init:i(r=>{r.class||(r.class={}),r.class.arrowMarkerAbsolute=r.arrowMarkerAbsolute},"init")};export{n as diagram};

package/dist/worker/console/static/assets/classDiagram-v2-OCEON4UE-DvMFMDx0.js

@@ -1 +0,0 @@
-import{s as a,c as s,a as e,C as t}from"./chunk-GF5L2VYU-CTNubrNg.js";import{_ as i}from"./mermaid.core-DeA6yB6l.js";import"./chunk-5VM5RSS4-aCdt3uIm.js";import"./chunk-XXDRQBXY-Ct6GraPo.js";import"./chunk-KBJHAD2P-51xTGTO2.js";import"./chunk-2GRJ4B5K-DbYJpklA.js";import"./index-BTNvk20w.js";var n={parser:e,get db(){return new t},renderer:s,styles:a,init:i(r=>{r.class||(r.class={}),r.class.arrowMarkerAbsolute=r.arrowMarkerAbsolute},"init")};export{n as diagram};

package/dist/worker/console/static/assets/stateDiagram-v2-UEYNNEHI-GwsFii7W.js

@@ -1 +0,0 @@
-import{s as r,b as e,a,S as s}from"./chunk-5RXB4S5H-CRl4qEu8.js";import{_ as i}from"./mermaid.core-DeA6yB6l.js";import"./chunk-XXDRQBXY-Ct6GraPo.js";import"./chunk-KBJHAD2P-51xTGTO2.js";import"./chunk-2GRJ4B5K-DbYJpklA.js";import"./index-BTNvk20w.js";var u={parser:a,get db(){return new s(2)},renderer:e,styles:r,init:i(t=>{t.state||(t.state={}),t.state.arrowMarkerAbsolute=t.arrowMarkerAbsolute},"init")};export{u as diagram};

package/dist/worker/console/static/assets/swimlanesDiagram-ULZ7WXOC-D603UkUt.js

@@ -1,8 +0,0 @@
-import{c as r,s as e}from"./flowDiagram-UKHOOZJN-CSuAfPiR.js";import{_ as a}from"./mermaid.core-DeA6yB6l.js";import"./chunk-5VM5RSS4-aCdt3uIm.js";import"./chunk-XXDRQBXY-Ct6GraPo.js";import"./chunk-KBJHAD2P-51xTGTO2.js";import"./chunk-2GRJ4B5K-DbYJpklA.js";import"./channel-Kf0dWmvg.js";import"./index-BTNvk20w.js";var o=a(t=>`${e(t)}
-  .swimlane.cluster rect {
-    stroke: ${t.clusterBorder} !important;
-  }
-  [data-look="neo"].cluster rect {
-    filter: none;
-  }
-`,"getStyles"),s=o,n=r({defaultLayout:"swimlane",styles:s});export{n as diagram};