fix(tools): treat no-op file writes and edits as terminal tool-loop results

wm0018 · wm0018 · commit 7caf70222b35 · 2026-06-26T17:54:17.000+08:00
diff --git a/src/agents/sessions/tools/edit.ts b/src/agents/sessions/tools/edit.ts
@@ -393,7 +393,7 @@ export function createEditToolDefinition(
           return {
             ...textResult(
               `No changes: one or more edits have identical old and new text in ${path}.`,
-              { status: "blocked" as const, reason: "no-op-edit" as const },
+              undefined,
             ),
             terminate: true,
           };
@@ -430,10 +430,7 @@ export function createEditToolDefinition(
           // No changes after applying edits — no-op.
           if (baseContent === newContent) {
             return {
-              ...textResult(`No changes: edits produced no diff in ${path}.`, {
-                status: "blocked" as const,
-                reason: "no-op-edit" as const,
-              }),
+              ...textResult(`No changes: edits produced no diff in ${path}.`, undefined),
               terminate: true,
             };
           }
diff --git a/src/agents/sessions/tools/write.test.ts b/src/agents/sessions/tools/write.test.ts
@@ -98,7 +98,7 @@ describe("write tool", () => {
       type: "text",
       text: `No changes: content is identical to existing ${filePath}`,
     });
-    expect(result.details).toEqual({ status: "blocked", reason: "no-op-write" });
+    expect(result.details).toBeUndefined();
     expect(result.terminate).toBe(true);
   });
 
diff --git a/src/agents/sessions/tools/write.ts b/src/agents/sessions/tools/write.ts
@@ -396,10 +396,7 @@ export function createWriteToolDefinition(
         const precheck = await readOriginalWriteState(absolutePath, content, ops);
         if (precheck.state === "same") {
           return {
-            ...textResult(`No changes: content is identical to existing ${path}`, {
-              status: "blocked" as const,
-              reason: "no-op-write" as const,
-            }),
+            ...textResult(`No changes: content is identical to existing ${path}`, undefined),
             terminate: true,
           };
         }
