@tea-agent/loop-agent 0.16.25 → 0.17.0

package/dist/worker/observe/static/views/dag-inspector.js

@@ -10,12 +10,14 @@ import {
 	UI_TEXT,
 } from "../constants.js";
 import { clearNode, el } from "../dom.js";
-import { fetchJson } from "../api.js";
+import { fetchJson, fetchSpecEvidenceFile } from "../api.js";
 import { badge } from "../format.js";
 import { renderMarkdown } from "../markdown-render.js";
 import {
 	uiState,
 	makeSessionEventIdentity,
+	openSpecEvidenceDetail,
+	closeSpecEvidenceDetail,
 	updateDagTimelineViewportState,
 	updateDagOutputViewportState,
 	computeFollowLatest,
@@ -44,6 +46,10 @@ export function selectDagNode(dagRunId, nodeId) {
 	if (nodeChanged) {
 		uiState.sessionEventOffset = 0;
 		uiState.sessionEvents = [];
+		// Drop any open spec-evidence preview so a stale A detail cannot survive
+		// into node B's spec-evidence panel (AC-007 regression guard).
+		uiState.specEvidenceDetail = null;
+		uiState.specEvidenceListEvidence = null;
 		// Bump the session-event identity token so any in-flight response for the
 		// previous node is rejected before it can mutate the current node cache.
 		uiState.sessionEventIdentity = makeSessionEventIdentity(dagRunId, nodeId);
@@ -188,15 +194,20 @@ export function captureDagTimelineViewportState(panel) {
 	}
 }
 
-async function renderSpecEvidence(content, dagRunId, nodeId) {
-	const evidence = await fetchJson(
+async function renderSpecEvidence(content, dagRunId, nodeId, cachedEvidence) {
+	const evidence = cachedEvidence ?? await fetchJson(
 		`/api/dag-runs/${encodeURIComponent(dagRunId)}/nodes/${encodeURIComponent(nodeId)}/spec-evidence`,
 	);
 	if (!content.isConnected || content.dataset.dagNodeId !== String(nodeId)) return;
+	if (uiState.specEvidenceDetail) {
+		renderSpecEvidenceDetail(content, dagRunId, nodeId);
+		return;
+	}
 	if (!evidence) {
 		content.appendChild(el("p", "empty", "无法加载规范证据。"));
 		return;
 	}
+	uiState.specEvidenceListEvidence = evidence;
 
 	const statusLabels = {
 		"spec-read": "已读取规范",
@@ -241,9 +252,24 @@ async function renderSpecEvidence(content, dagRunId, nodeId) {
 	});
 	appendListSection("已绑定任务源", evidence.sourceBinding?.sources, (source) => {
 		const item = el("li", null);
+		const button = document.createElement("button");
+		button.type = "button";
+		button.className = "spec-evidence-file-btn";
+		button.setAttribute("data-source", "binding");
+		button.setAttribute("data-path", source.path);
+		button.id = `spec-evidence-btn-binding-${source.path}`;
 		const icon = el("i", "ri-links-line");
 		icon.setAttribute("aria-hidden", "true");
-		item.append(icon, document.createTextNode(` ${source.kind}: `), el("code", null, source.path), document.createTextNode(` · sha256 ${source.sha256.slice(0, 12)}…`));
+		button.append(
+			icon,
+			document.createTextNode(` ${source.kind}: `),
+			el("code", null, source.path),
+			document.createTextNode(` · sha256 ${source.sha256.slice(0, 12)}…`),
+		);
+		button.addEventListener("click", () =>
+			onSpecEvidenceFileClick(content, dagRunId, nodeId, "binding", source.path, button),
+		);
+		item.appendChild(button);
 		return item;
 	});
 	appendListSection("显式需求编号", evidence.sourceBinding?.requirementIds, (id) => el("li", null, id));
@@ -252,11 +278,20 @@ async function renderSpecEvidence(content, dagRunId, nodeId) {
 		evidence.specReads,
 		(read) => {
 			const item = el("li", null);
+			const button = document.createElement("button");
+			button.type = "button";
+			button.className = "spec-evidence-file-btn";
+			button.setAttribute("data-source", "read");
+			button.setAttribute("data-path", read.path);
+			if (read.timestamp) {
+				button.setAttribute("data-read-at", read.timestamp);
+			}
+			button.id = `spec-evidence-btn-read-${read.path}`;
 			const icon = el("i", "ri-file-text-line");
 			icon.setAttribute("aria-hidden", "true");
-			item.append(icon, el("code", null, read.path));
+			button.append(icon, el("code", null, read.path));
 			if (read.timestamp) {
-				item.appendChild(
+				button.appendChild(
 					el(
 						"span",
 						"spec-evidence-time",
@@ -264,6 +299,10 @@ async function renderSpecEvidence(content, dagRunId, nodeId) {
 					),
 				);
 			}
+			button.addEventListener("click", () =>
+				onSpecEvidenceFileClick(content, dagRunId, nodeId, "read", read.path, button),
+			);
+			item.appendChild(button);
 			return item;
 		},
 	);
@@ -308,7 +347,135 @@ async function renderSpecEvidence(content, dagRunId, nodeId) {
 	}
 }
 
+/**
+ * Click handler for spec-evidence file buttons (AC-001/002). Opens the detail
+ * view in the same spec-evidence panel (no modal), records the trigger button
+ * so the back action can restore keyboard focus (AC-005).
+ */
+function onSpecEvidenceFileClick(content, dagRunId, nodeId, source, relPath, button) {
+	openSpecEvidenceDetail(source, relPath, button);
+	clearNode(content);
+	const detail = uiState.specEvidenceDetail;
+	renderSpecEvidenceDetail(content, dagRunId, nodeId);
+	void (async () => {
+		try {
+			const data = await fetchSpecEvidenceFile(dagRunId, nodeId, source, relPath);
+			if (
+				!content.isConnected ||
+				content.dataset.dagNodeId !== String(nodeId) ||
+				!uiState.specEvidenceDetail ||
+				uiState.specEvidenceDetail.source !== source ||
+				uiState.specEvidenceDetail.path !== relPath
+			) {
+				return;
+			}
+			detail.loading = false;
+			detail.data = data;
+		} catch (error) {
+			if (
+				!content.isConnected ||
+				content.dataset.dagNodeId !== String(nodeId) ||
+				!uiState.specEvidenceDetail ||
+				uiState.specEvidenceDetail.source !== source ||
+				uiState.specEvidenceDetail.path !== relPath
+			) {
+				return;
+			}
+			detail.loading = false;
+			detail.error = error instanceof Error ? error.message : String(error);
+		}
+		renderSpecEvidenceDetail(content, dagRunId, nodeId);
+	})();
+}
+
+/**
+ * Render the in-inspector file preview (AC-005). Single scroll container for
+ * the body, always-visible back button, and accessible loading/error states.
+ */
+function renderSpecEvidenceDetail(content, dagRunId, nodeId) {
+	const detail = uiState.specEvidenceDetail;
+	clearNode(content);
+	if (!detail) {
+		renderSpecEvidence(content, dagRunId, nodeId, uiState.specEvidenceListEvidence);
+		return;
+	}
+
+	const wrapper = el("div", "spec-evidence-detail");
+	const back = document.createElement("button");
+	back.type = "button";
+	back.className = "spec-evidence-back";
+	back.setAttribute("aria-label", "返回规范证据");
+	const backIcon = el("i", "ri-arrow-left-line");
+	backIcon.setAttribute("aria-hidden", "true");
+	back.append(backIcon, document.createTextNode(" 返回规范证据"));
+	back.addEventListener("click", () => {
+		const closed = closeSpecEvidenceDetail();
+		clearNode(content);
+		// Restore the list synchronously from the cached evidence so the user
+		// sees no flicker and the trigger button is immediately focusable.
+		renderSpecEvidence(content, dagRunId, nodeId, uiState.specEvidenceListEvidence);
+		if (closed?.triggerId) {
+			const trigger = document.getElementById(closed.triggerId);
+			trigger?.focus();
+		}
+	});
+	wrapper.appendChild(back);
+
+	const meta = el("div", "spec-evidence-detail-meta");
+	meta.appendChild(el("code", null, detail.path));
+	if (detail.source === "binding" && detail.data) {
+		meta.appendChild(
+			el(
+				"span",
+				"spec-evidence-detail-hash",
+				`绑定 sha256 ${String(detail.data.sha256 ?? "").slice(0, 12)}… · 当前 ${detail.data.hashMatch ? "一致" : "不一致"}`,
+			),
+		);
+	} else if (detail.source === "read" && detail.data?.readAt) {
+		meta.appendChild(
+			el(
+				"span",
+				"spec-evidence-detail-time",
+				`读取于 ${formatSessionEventTime({ timestamp: detail.data.readAt })}`,
+			),
+		);
+	}
+	wrapper.appendChild(meta);
+
+	if (detail.loading) {
+		const loading = el("p", "muted", "正在加载文件内容…");
+		loading.setAttribute("role", "status");
+		loading.setAttribute("aria-live", "polite");
+		wrapper.appendChild(loading);
+	} else if (detail.error) {
+		const error = el("p", "spec-evidence-detail-error", `加载失败：${detail.error}`);
+		error.setAttribute("role", "alert");
+		wrapper.appendChild(error);
+	} else if (detail.data) {
+		const body = el("pre", "spec-evidence-detail-content");
+		body.textContent = detail.data.content ?? "";
+		wrapper.appendChild(body);
+		if (detail.data.truncated) {
+			const note = el(
+				"p",
+				"spec-evidence-detail-truncated",
+				`已截断，仅显示前 ${detail.data.maxBytes ?? 0} 字节。`,
+			);
+			wrapper.appendChild(note);
+		}
+	}
+	content.appendChild(wrapper);
+}
+
 	function fillDagInspectorContent(content, dagRunId, node) {
+		const previousRenderedNode = content.dataset.dagNodeId ?? "";
+		const nodeChanged = previousRenderedNode !== String(node.nodeId ?? "");
+		if (nodeChanged) {
+			// Drop any open spec-evidence preview + cached list so a stale A detail
+			// cannot survive into node B (AC-007 regression guard).
+			uiState.specEvidenceDetail = null;
+			uiState.specEvidenceListEvidence = null;
+		}
 		const activeTab = ["timeline", "spec-evidence"].includes(
 			uiState.dagInspectorTab,
 		)

package/dist/workflows/dag/backend-test-analysis-contract.js

@@ -356,20 +356,47 @@ function coerceOptionalEvidence(value, kind) {
         out.sourceRef = sourceRef;
     return out;
 }
+const EVIDENCE_GAP_INPUT_KEYS = new Set([
+    "description",
+    "text",
+    "requirementId",
+    "acId",
+    "id",
+    "sourceRef",
+    "sourceRefs",
+]);
 function coerceEvidenceGap(value) {
+    if (typeof value === "string") {
+        if (!value.trim())
+            throw new Error("evidence gap string must be non-empty");
+        return { description: value.trim() };
+    }
     const record = asRecord(value);
     if (!record)
-        return null;
+        throw new Error("evidence gap must be a string or object");
+    for (const key of Object.keys(record)) {
+        if (!EVIDENCE_GAP_INPUT_KEYS.has(key))
+            throw new Error(`evidence gap has unrecognized key: ${key}`);
+    }
+    for (const key of ["description", "text", "requirementId", "acId", "id", "sourceRef"]) {
+        if (record[key] !== undefined && typeof record[key] !== "string")
+            throw new Error(`evidence gap ${key} must be a string`);
+    }
+    if (record.sourceRefs !== undefined && (!Array.isArray(record.sourceRefs) || record.sourceRefs.some((item) => typeof item !== "string"))) {
+        throw new Error("evidence gap sourceRefs must be a string array");
+    }
     const description = (typeof record.description === "string" && record.description.trim()) ||
         (typeof record.text === "string" && record.text.trim()) ||
         "";
     if (!description)
-        return null;
+        throw new Error("evidence gap description must be non-empty");
     const out = { description };
-    if (typeof record.requirementId === "string" && record.requirementId.trim())
-        out.requirementId = record.requirementId.trim();
-    else if (typeof record.acId === "string" && record.acId.trim())
-        out.requirementId = record.acId.trim();
+    const requirementId = (typeof record.requirementId === "string" && record.requirementId.trim()) ||
+        (typeof record.acId === "string" && record.acId.trim()) ||
+        (typeof record.id === "string" && record.id.trim()) ||
+        "";
+    if (requirementId)
+        out.requirementId = requirementId;
     const sourceRef = firstSourceRef(record, "");
     if (sourceRef)
         out.sourceRef = sourceRef;
@@ -403,9 +430,7 @@ export function coerceBackendTestAnalysisInput(value) {
     next.risks = asArray(record.risks)
         .map((item) => coerceOptionalEvidence(item, "risk"))
         .filter((item) => item !== null);
-    next.evidenceGaps = asArray(record.evidenceGaps)
-        .map(coerceEvidenceGap)
-        .filter((item) => item !== null);
+    next.evidenceGaps = asArray(record.evidenceGaps).map(coerceEvidenceGap);
     return next;
 }
 function normalizeAnalysis(value) {

package/dist/workflows/dag/dynamic-runtime/shared.js

@@ -5,6 +5,7 @@ export function freshNodeRecord(task) {
         status: "PENDING",
         executor: task.executor,
         complexity: task.complexity,
+        ...(task.outputMode ? { outputMode: task.outputMode } : {}),
     };
 }
 export function sha256Json(value) {

package/dist/workflows/dag/frontend-repair.js

@@ -320,16 +320,7 @@ export async function runFrontendRepairContractGate(input) {
     return { ok: true, assessment };
 }
 export function formatFrontendFailureAssessStdout(assessment) {
-    return [
-        "Frontend failure assess: ok",
-        `eligible=${assessment.eligible}`,
-        `failureClass=${assessment.failureClass}`,
-        `failedNodeIds=${assessment.failedNodeIds.join(",") || "(none)"}`,
-        `attempt=${assessment.attempt}/${assessment.maxAttempts}`,
-        `reason=${assessment.reason}`,
-        `browser=${assessment.browserStatus}`,
-        `artifact=contracts/frontend-repair-assessment.json`,
-    ].join("\n");
+    return JSON.stringify(assessment);
 }
 export function formatFrontendRepairContractStdout(assessment) {
     return [