@tea-agent/loop-agent 0.16.10 → 0.16.12

package/CHANGELOG.md

@@ -18,6 +18,18 @@
 - Pi SDK 执行长推理或大段结构化输出时不再把高频流式增量事件无界累积到内存；同一响应在多个生命周期事件中重复出现的 Token 用量只统计一次，避免 `Invalid string length` 和成本数据虚高。
 - 后端测试复合执行节点继续保持 clean environment、失败分类和 fail-closed outcome，并为 initial/final Result、repair eligibility、traceability 与 Observe 投影保留结构化运行证据。
 
+## [0.16.12] - 2026-07-20
+
+### 修复
+
+- `backend-test` materialize / Git checkpoint 自动并入 `testcase/**`（与 `docs/test-reports/**`）运行时写根，避免 TaskSpec 仍写 `tests/api/**` 时绿跑无法 promote、任务卡在 Running。
+
+## [0.16.11] - 2026-07-20
+
+### 修复
+
+- Worker 报告决策不再把条件分支的 `SKIPPED` 节点当成失败；backend-test 等混合 DAG 在 revise/repair 未选中时，只要 ERROR 不存在且 run finished，即可 `report-completed` 并进入 Outcome/promote。
+
 ## [0.16.10] - 2026-07-20
 
 ### 修复

package/dist/worker/delivery/git-transaction.js

@@ -6,6 +6,8 @@ import path from "node:path";
 import { z } from "zod";
 import { getTaskPoolRoot } from "../pool/run-store.js";
 import { assertSafeRuntimeId } from "../follow-up/paths.js";
+import { expandAllowedPathsForWorkflow } from "../materialize/harness-task-materializer.js";
+import { resolveWorkflow } from "../task-spec/workflow-routing.js";
 const execFileAsync = promisify(execFile);
 const checkpointSchema = z.object({
     taskId: z.string().min(1),
@@ -275,13 +277,14 @@ function auditChangedPaths(changes, taskSpec) {
         throw new Error(audit.violations[0]);
 }
 function auditChangedPathsReport(changes, taskSpec) {
+    const allowedPaths = expandAllowedPathsForWorkflow(resolveWorkflow(taskSpec).workflow, taskSpec.constraints.allowed_paths);
     const violations = [];
     for (const changed of changes) {
         if (/(^|\/)(\.env(?:\.|$)|[^/]*\.(?:pem|key|p12|pfx))$/i.test(changed))
             violations.push(`changed path may contain sensitive material: ${changed}`);
         else if (taskSpec.constraints.forbidden_paths.some((glob) => matchesGlob(changed, glob)))
             violations.push(`changed path is forbidden for ${taskSpec.id}: ${changed}`);
-        else if (!taskSpec.constraints.allowed_paths.some((glob) => matchesGlob(changed, glob)))
+        else if (!allowedPaths.some((glob) => matchesGlob(changed, glob)))
             violations.push(`changed path is outside allowed_paths for ${taskSpec.id}: ${changed}`);
     }
     return { ok: violations.length === 0, violations };

package/dist/worker/materialize/harness-task-materializer.js

@@ -34,13 +34,17 @@ export async function materializeTaskSpec(options) {
     }
     const paths = getTaskPaths(options.repoRoot, harnessTaskId);
     const baseConfig = await loadTaskConfig(options.repoRoot, harnessTaskId);
+    // backend-test hybrid DAG writes under testcase/** (pytest + md cases + helpers).
+    // TaskSpecs often list legacy tests/api/** surfaces; union runtime write roots so
+    // git checkpoint / path guard can promote green BE-TEST runs.
+    const allowedPaths = expandAllowedPathsForWorkflow(resolvedWorkflow.workflow, options.taskSpec.constraints.allowed_paths);
     const taskConfig = {
         ...baseConfig,
         taskId: harnessTaskId,
         title: options.taskSpec.title,
         taskKind: resolvedWorkflow.taskKind,
         featureId: options.taskSpec.feature_id,
-        allowedPaths: options.taskSpec.constraints.allowed_paths,
+        allowedPaths,
         forbiddenPaths: options.taskSpec.constraints.forbidden_paths,
         hardConstraints: options.taskSpec.constraints.hard_constraints,
         complexity: mapRiskLevelToComplexity(options.taskSpec.risk_level),
@@ -312,6 +316,13 @@ function bulletLines(values) {
         return ["- None"];
     return values.map((value) => `- ${value}`);
 }
+/** Union TaskSpec allowed_paths with workflow-owned write roots. */
+export function expandAllowedPathsForWorkflow(workflow, allowedPaths) {
+    const extras = workflow === "backend-test"
+        ? ["testcase/**", "docs/test-reports/**"]
+        : [];
+    return Array.from(new Set([...allowedPaths, ...extras]));
+}
 async function loadExistingTaskConfig(repoRoot, taskId) {
     try {
         return await loadTaskConfig(repoRoot, taskId);

package/dist/worker/run-task/run-task.js

@@ -455,7 +455,8 @@ function isPathInside(root, target) {
     const relative = path.relative(root, target);
     return relative !== "" && relative !== ".." && !relative.startsWith(`..${path.sep}`) && !path.isAbsolute(relative);
 }
-function decideFromReport(workerRunId, result) {
+/** Exported for unit tests: map DAG report JSON into Worker report decision. */
+export function decideFromReport(workerRunId, result) {
     if (!result.ok || !result.json) {
         return {
             succeeded: false,
@@ -473,11 +474,16 @@ function decideFromReport(workerRunId, result) {
     }
     const runStatus = readString(run, "status");
     const nodes = readObjectArray(run, "nodes");
+    // Condition-branch SKIPPED nodes are expected on green backend-test / hybrid
+    // paths (e.g. revise/repair not selected). Do not treat SKIPPED as failure.
+    // ERROR and non-success failureCategory on executed nodes still fail closed.
     const failedNode = nodes.find((node) => {
         const status = readString(node, "status");
         const failureCategory = readString(node, "failureCategory");
+        if (status === "SKIPPED") {
+            return false;
+        }
         return (status === "ERROR" ||
-            status === "SKIPPED" ||
             (Boolean(failureCategory) && failureCategory !== "success"));
     });
     if (runStatus !== "completed" && runStatus !== "finished") {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tea-agent/loop-agent",
-  "version": "0.16.10",
+  "version": "0.16.12",
   "type": "module",
   "bin": {
     "loop-agent": "bin/loop-agent.js",