@tea-agent/loop-agent 0.28.6 → 0.28.8

package/CHANGELOG.md

@@ -2,6 +2,27 @@
 
 ## [Unreleased]
 
+## [0.28.8] - 2026-08-05
+
+### 重点更新
+
+- 修复 Night Scheduler 收尾提交被运行态文件污染的问题，确保提交内容仅限源码变更
+
+### 修复
+
+- Night Scheduler 收尾提交现忽略 .harness/ 与 .worktrees/ 运行态目录，仅暂存 writeSet 源码路径，避免 verify 将 dag-runs 等运行态文件误判为 writer 变更
+- Night Scheduler worktree 在 prepare 时尽力确保 .gitignore 包含 .harness/ 与 .worktrees/，防止运行态文件干扰工作区校验
+
+## [0.28.7] - 2026-08-05
+
+### 重点更新
+
+- 修复 Night Scheduler 执行已准备好的 DAG 时因客户端超时中断任务的问题
+
+### 修复
+
+- Night Scheduler 执行已准备好的 DAG 时，`task advance --approve-gate` 改用 `timeoutMs: 0`，避免默认 120 秒超时在 DAG 仍在运行时将其强制终止，现由内核负责执行监督
+
 ## [0.28.6] - 2026-08-05
 
 ### 重点更新

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

@@ -65,6 +65,10 @@ export async function executePreparedTaskSpec(options) {
             cwd: input.workspaceRoot,
             artifactName: `night-execute-${input.workerRunId}`,
             expectJson: true,
+            // Night prepared execute runs a full DAG under task advance; do not use
+            // the short LoopAgentClient default (120s). Zero disables client timeout
+            // so the DAG kernel owns long-running supervision (same as run-task).
+            timeoutMs: 0,
         }));
     const advanceResult = await advance({
         workspaceRoot,

package/dist/worker/scheduler/night-git-finalizer.js

@@ -1,9 +1,13 @@
 import { spawn } from "node:child_process";
 import { parseGitStatusPorcelain, pathMatchesPattern, } from "../../shared/git-progress.js";
+import { isSchedulerRuntimeGitPath } from "./git-base.js";
 import { formatCommitMessage, validateTaskCard } from "./traceability.js";
 /**
  * Create a single closing commit on the night branch after successful execution.
  * Does not merge to base. Fail-closed on writeSet violations or missing task card.
+ *
+ * Runtime facts under `.harness/**` / `.worktrees/**` are never staged.
+ * When writeSet is present, only matching source paths are added (not `git add -A`).
  */
 export async function createNightClosingCommit(input) {
     const card = validateTaskCard(input.schedule.traceability?.workItemRef, {
@@ -30,19 +34,38 @@ export async function createNightClosingCommit(input) {
     if (!status) {
         return { ok: false, reason: "no changes to commit after execution" };
     }
-    const changedPaths = parseGitStatusPorcelain(status).map((entry) => entry.path);
+    const allChangedPaths = parseGitStatusPorcelain(status).map((entry) => entry.path);
+    const sourceChangedPaths = allChangedPaths.filter((p) => !isSchedulerRuntimeGitPath(p));
+    if (sourceChangedPaths.length === 0) {
+        return {
+            ok: false,
+            reason: "no source changes to commit after execution (only runtime .harness/.worktrees dirtiness)",
+            changedPaths: allChangedPaths,
+        };
+    }
     const writeSet = input.writeSet ?? input.admission.dag?.writeSet ?? undefined;
     if (writeSet && writeSet.length > 0) {
-        const offenders = changedPaths.filter((p) => !pathAllowedByWriteSet(p, writeSet));
+        const offenders = sourceChangedPaths.filter((p) => !pathAllowedByWriteSet(p, writeSet));
         if (offenders.length > 0) {
             return {
                 ok: false,
                 reason: `changed paths outside frozen writeSet: ${offenders.join(", ")}`,
-                changedPaths,
+                changedPaths: sourceChangedPaths,
             };
         }
     }
-    await runGit(input.workspaceRoot, ["add", "-A"]);
+    // Stage only source paths (and preferably writeSet-bound ones). Never `git add -A`.
+    const pathsToStage = writeSet && writeSet.length > 0
+        ? sourceChangedPaths.filter((p) => pathAllowedByWriteSet(p, writeSet))
+        : sourceChangedPaths;
+    if (pathsToStage.length === 0) {
+        return {
+            ok: false,
+            reason: "no writeSet-allowed source paths to stage",
+            changedPaths: sourceChangedPaths,
+        };
+    }
+    await runGit(input.workspaceRoot, ["add", "--", ...pathsToStage]);
     const summary = `feat(${input.schedule.taskId.toLowerCase()}): night schedule ${input.schedule.id}`;
     const message = formatCommitMessage({
         summary,
@@ -60,7 +83,7 @@ export async function createNightClosingCommit(input) {
         ok: true,
         commitSha,
         message,
-        changedPaths,
+        changedPaths: pathsToStage,
     };
 }
 function pathAllowedByWriteSet(filePath, writeSet) {

package/dist/worker/scheduler/workspace-adapter.js

@@ -1,4 +1,4 @@
-import { access, realpath, symlink, unlink } from "node:fs/promises";
+import { access, appendFile, readFile, realpath, symlink, unlink, writeFile } from "node:fs/promises";
 import path from "node:path";
 import { ensureOwnedWorktree, verifyOwnedWorktree, } from "../../task/worktree.js";
 import { getSchedulerRoot } from "./paths.js";
@@ -33,6 +33,9 @@ export async function prepareNightWorkspace(spec) {
         ...(spec.ownershipToken ? { ownershipToken: spec.ownershipToken } : {}),
         ...(spec.now ? { now: spec.now } : {}),
     });
+    // Keep runtime roots out of writer/verify dirty trees and closing commits.
+    await ensureNightGitignore(ensured.path);
+    await ensureNightGitignore(spec.controlRepoRoot);
     let nodeModulesLinked = false;
     if (spec.hydrateNodeModules !== false) {
         nodeModulesLinked = await linkNodeModulesIfPresent(spec.controlRepoRoot, ensured.path);
@@ -98,3 +101,28 @@ async function linkNodeModulesIfPresent(controlRepoRoot, worktreePath) {
         return false;
     }
 }
+async function ensureNightGitignore(repoRoot) {
+    const gitignorePath = path.join(repoRoot, ".gitignore");
+    const required = [".harness/", ".worktrees/"];
+    let existing = "";
+    try {
+        existing = await readFile(gitignorePath, "utf-8");
+    }
+    catch {
+        await writeFile(gitignorePath, `# Night Scheduler / loop-agent runtime (auto)
+${required.join("\n")}\n`, "utf-8");
+        return;
+    }
+    const lines = new Set(existing
+        .split(/\r?\n/)
+        .map((line) => line.trim())
+        .filter(Boolean));
+    const missing = required.filter((entry) => !lines.has(entry));
+    if (missing.length === 0)
+        return;
+    const suffix = (existing.endsWith("\n") || existing.length === 0 ? "" : "\n") +
+        `# Night Scheduler / loop-agent runtime (auto)\n` +
+        missing.join("\n") +
+        "\n";
+    await appendFile(gitignorePath, suffix, "utf-8");
+}

package/package.json

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