fix: Codex approval policies, terminal snapshot replay, and multi-hunk edit display

- Pass approvalPolicy and sandbox fields through to Codex thread/start and turn/start
- Suppress terminal onData during snapshot replay to prevent escape-sequence echo
- Fix multi-hunk single-file edits incorrectly counted as "multiple files" in compact
  summaries, diff rendering, and turn change tracking
- Add unit tests for patch-utils and tool-formatting
- Update claude-agent-sdk to 0.2.77

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: OpenSource03

electron/src/ipc/codex-sessions.ts

@@ -43,6 +43,10 @@ interface CodexSession {
   eventCounter: number;
   cwd: string;
   model?: string;
+  /** Approval policy for the session — passed to turn/start and lazy thread/start */
+  approvalPolicy?: string;
+  /** Sandbox policy for the session — passed to lazy thread/start */
+  sandbox?: string;
 }
 
 import { SUPPORTED_SERVER_REQUESTS, isSupportedServerRequestMethod, pickModelId } from "@shared/lib/codex-helpers";
@@ -258,6 +262,8 @@ export function register(getMainWindow: () => BrowserWindow | null): void {
           eventCounter: 0,
           cwd: options.cwd,
           model: undefined,
+          approvalPolicy: options.approvalPolicy,
+          sandbox: options.sandbox,
         };
         codexSessions.set(internalId, session);
         setupCodexHandlers(rpc, session, internalId, getMainWindow);
@@ -374,6 +380,8 @@ export function register(getMainWindow: () => BrowserWindow | null): void {
             persistExtendedHistory: false,
           };
           if (session.model) threadParams.model = session.model;
+          if (session.approvalPolicy) threadParams.approvalPolicy = session.approvalPolicy;
+          if (session.sandbox) threadParams.sandbox = session.sandbox;
           const threadResult = await session.rpc.request<CodexThreadStartResponse>("thread/start", threadParams);
           session.threadId = threadResult.thread.id;
           log(
@@ -388,7 +396,7 @@ export function register(getMainWindow: () => BrowserWindow | null): void {
 
       log(
         "codex",
-        ` Send requested: session=${shortId(data.sessionId, 12)} thread=${shortId(session.threadId, 12)} text_len=${data.text.length} images=${data.images?.length ?? 0} effort=${data.effort ?? "default"} collab=${data.collaborationMode?.mode ?? "none"} activeTurn=${session.activeTurnId ? shortId(session.activeTurnId, 12) : "none"}`,
+        ` Send requested: session=${shortId(data.sessionId, 12)} thread=${shortId(session.threadId, 12)} text_len=${data.text.length} images=${data.images?.length ?? 0} effort=${data.effort ?? "default"} collab=${data.collaborationMode?.mode ?? "none"} approval=${session.approvalPolicy ?? "default"} activeTurn=${session.activeTurnId ? shortId(session.activeTurnId, 12) : "none"}`,
       );
 
       try {
@@ -405,6 +413,7 @@ export function register(getMainWindow: () => BrowserWindow | null): void {
           ...(session.model ? { model: session.model } : {}),
           ...(data.effort ? { effort: data.effort } : {}),
           ...(data.collaborationMode ? { collaborationMode: data.collaborationMode } : {}),
+          ...(session.approvalPolicy ? { approvalPolicy: session.approvalPolicy } : {}),
         };
 
 
@@ -703,6 +712,8 @@ export function register(getMainWindow: () => BrowserWindow | null): void {
           eventCounter: 0,
           cwd: data.cwd,
           model: data.model,
+          approvalPolicy: data.approvalPolicy,
+          sandbox: data.sandbox,
         };
         codexSessions.set(internalId, session);
         setupCodexHandlers(rpc, session, internalId, getMainWindow);

package.json

@@ -1,6 +1,6 @@
 {
   "name": "harnss",
-  "version": "0.21.2",
+  "version": "0.21.3",
   "productName": "Harnss",
   "description": "Harness your AI coding agents — one desktop app for Claude Code, Codex, and any ACP agent",
   "author": {
@@ -31,7 +31,7 @@
   "packageManager": "pnpm@10.26.0",
   "dependencies": {
     "@agentclientprotocol/sdk": "^0.15.0",
-    "@anthropic-ai/claude-agent-sdk": "^0.2.76",
+    "@anthropic-ai/claude-agent-sdk": "^0.2.77",
     "@huggingface/transformers": "^3.8.1",
     "@modelcontextprotocol/sdk": "^1.26.0",
     "@monaco-editor/react": "^4.7.0",

pnpm-lock.yaml

@@ -229,6 +229,7 @@ function TerminalInstance({
   const lastSeqRef = useRef(0);
   const pendingChunksRef = useRef<Array<{ seq: number; data: string }>>([]);
   const hydratedRef = useRef(false);
+  const suppressInputRef = useRef(false);
   const [ready, setReady] = useState(false);
 
   // Initialize xterm
@@ -277,6 +278,7 @@ function TerminalInstance({
 
       // Wire up input → PTY
       term.onData((data) => {
+        if (suppressInputRef.current) return;
         window.claude.terminal.write(terminalId, data);
       });
 
@@ -302,7 +304,16 @@ function TerminalInstance({
       if (disposed) return;
 
       if (snapshot.output) {
-        term.write(snapshot.output);
+        // Restoring historical terminal output into a fresh xterm instance can
+        // re-trigger terminal capability responses. Suppress onData while the
+        // snapshot is replayed so old escape-sequence replies do not leak into
+        // the live PTY as random input after a space switch.
+        suppressInputRef.current = true;
+        try {
+          term.write(snapshot.output);
+        } finally {
+          suppressInputRef.current = false;
+        }
       }
       lastSeqRef.current = snapshot.seq ?? 0;
       term.options.disableStdin = !!snapshot.exited;
@@ -336,6 +347,7 @@ function TerminalInstance({
       lastSeqRef.current = 0;
       pendingChunksRef.current = [];
       hydratedRef.current = false;
+      suppressInputRef.current = false;
     };
   }, [terminalId]);
 

src/components/ToolsPanel.tsx

@@ -0,0 +1,32 @@
+import { describe, expect, it } from "vitest";
+import type { UIMessage } from "@/types";
+import { formatCompactSummary } from "./tool-formatting";
+
+describe("formatCompactSummary", () => {
+  it("does not report Claude multi-hunk single-file edits as multiple files", () => {
+    const message: UIMessage = {
+      id: "edit-1",
+      role: "tool_call",
+      content: "",
+      toolName: "Edit",
+      toolInput: {
+        file_path: "/repo/src/LtiTeacherAssignmentPreview.tsx",
+        old_string: "old",
+        new_string: "new",
+        replace_all: false,
+      },
+      toolResult: {
+        filePath: "/repo/src/LtiTeacherAssignmentPreview.tsx",
+        oldString: "old",
+        newString: "new",
+        structuredPatch: [
+          { oldStart: 1, oldLines: 1, newStart: 1, newLines: 1, lines: ["-old", "+new"] },
+          { oldStart: 20, oldLines: 1, newStart: 20, newLines: 1, lines: ["-old2", "+new2"] },
+        ],
+      },
+      timestamp: 0,
+    };
+
+    expect(formatCompactSummary(message)).toBe("LtiTeacherAssignmentPreview.tsx");
+  });
+});

src/components/lib/tool-formatting.test.ts

@@ -1,7 +1,7 @@
 import type { UIMessage, SubagentToolStep } from "@/types";
 import { getMcpCompactSummary } from "@/components/McpToolContent";
 import { getTodoItems } from "@/lib/todo-utils";
-import { getStructuredPatches } from "@/lib/patch-utils";
+import { getDistinctPatchPaths, getStructuredPatches } from "@/lib/patch-utils";
 
 // ── Compact summary for collapsed tool line ──
 
@@ -65,8 +65,9 @@ export function formatCompactSummary(message: UIMessage): string {
   if (input.command) return String(input.command).split("\n")[0];
   // Multi-file Codex edits: show file count instead of single filename
   const patches = getStructuredPatches(result);
-  if (input.file_path && patches.length > 1) {
-    return `${patches.length} files`;
+  const patchPaths = getDistinctPatchPaths(patches);
+  if (input.file_path && patchPaths.length > 1) {
+    return `${patchPaths.length} files`;
   }
   if (input.file_path) return String(input.file_path).split("/").pop() ?? "";
   if (filePathFromResult) return filePathFromResult.split("/").pop() ?? filePathFromResult;

src/components/lib/tool-formatting.ts

@@ -4,7 +4,13 @@ import { parseUnifiedDiffFromUnknown } from "@/lib/unified-diff";
 import { DiffViewer } from "@/components/DiffViewer";
 import { UnifiedPatchViewer } from "@/components/UnifiedPatchViewer";
 import { firstDefinedString } from "@/components/lib/tool-formatting";
-import { getStructuredPatches, getPatchPath, filterValidPatches, type StructuredPatchEntry } from "@/lib/patch-utils";
+import {
+  getStructuredPatches,
+  getPatchPath,
+  filterValidPatches,
+  isMultiFileStructuredPatch,
+  type StructuredPatchEntry,
+} from "@/lib/patch-utils";
 import { GenericContent } from "./GenericContent";
 
 // ── Multi-file rendering (Codex fileChange with N > 1 changes) ──
@@ -41,7 +47,7 @@ export function EditContent({ message }: { message: UIMessage }) {
   const structuredPatch = getStructuredPatches(message.toolResult);
 
   // Multi-file Codex fileChange: render each file's diff separately
-  if (structuredPatch.length > 1) {
+  if (isMultiFileStructuredPatch(structuredPatch)) {
     const validPatches = filterValidPatches(structuredPatch);
     if (validPatches.length === 0) return <GenericContent message={message} />;
     return (

src/components/tool-renderers/EditContent.tsx

@@ -8,7 +8,13 @@ import { useResolvedThemeClass } from "@/hooks/useResolvedThemeClass";
 import { parseUnifiedDiff } from "@/lib/unified-diff";
 import { UnifiedPatchViewer } from "@/components/UnifiedPatchViewer";
 import { OpenInEditorButton } from "@/components/OpenInEditorButton";
-import { getStructuredPatches, getPatchPath, filterValidPatches, type StructuredPatchEntry } from "@/lib/patch-utils";
+import {
+  getStructuredPatches,
+  getPatchPath,
+  filterValidPatches,
+  isMultiFileStructuredPatch,
+  type StructuredPatchEntry,
+} from "@/lib/patch-utils";
 import { GenericContent } from "./GenericContent";
 
 // ── Stable style constants (avoid re-creating on every render) ──
@@ -55,7 +61,7 @@ export function WriteContent({ message }: { message: UIMessage }) {
   const structuredPatch = getStructuredPatches(message.toolResult);
 
   // Multi-file Codex fileChange: render each new file separately
-  if (structuredPatch.length > 1) {
+  if (isMultiFileStructuredPatch(structuredPatch)) {
     const validPatches = filterValidPatches(structuredPatch);
     if (validPatches.length === 0) return <GenericContent message={message} />;
     return (

src/components/tool-renderers/WriteContent.tsx

@@ -0,0 +1,43 @@
+import { describe, expect, it } from "vitest";
+import {
+  getDistinctPatchPaths,
+  isMultiFileStructuredPatch,
+  type StructuredPatchEntry,
+} from "./patch-utils";
+
+describe("structured patch helpers", () => {
+  it("treats Claude multi-hunk patches without file paths as single-file", () => {
+    const patches: StructuredPatchEntry[] = [
+      {
+        oldStart: 1,
+        oldLines: 2,
+        newStart: 1,
+        newLines: 2,
+        lines: [" context", "-old", "+new"],
+      },
+      {
+        oldStart: 20,
+        oldLines: 2,
+        newStart: 20,
+        newLines: 2,
+        lines: [" context", "-old2", "+new2"],
+      },
+    ];
+
+    expect(getDistinctPatchPaths(patches)).toEqual([]);
+    expect(isMultiFileStructuredPatch(patches)).toBe(false);
+  });
+
+  it("treats multiple distinct patch file paths as multi-file", () => {
+    const patches: StructuredPatchEntry[] = [
+      { filePath: "/repo/src/a.ts", diff: "diff --git a/src/a.ts b/src/a.ts" },
+      { filePath: "/repo/src/b.ts", diff: "diff --git a/src/b.ts b/src/b.ts" },
+    ];
+
+    expect(getDistinctPatchPaths(patches)).toEqual([
+      "/repo/src/a.ts",
+      "/repo/src/b.ts",
+    ]);
+    expect(isMultiFileStructuredPatch(patches)).toBe(true);
+  });
+});

src/lib/patch-utils.test.ts

@@ -19,6 +19,11 @@ export interface StructuredPatchEntry {
   diff?: string;
   oldString?: string;
   newString?: string;
+  oldStart?: number;
+  oldLines?: number;
+  newStart?: number;
+  newLines?: number;
+  lines?: string[];
 }
 
 // ── Extraction ──
@@ -43,3 +48,27 @@ export function filterValidPatches(
 ): StructuredPatchEntry[] {
   return patches.filter((p) => getPatchPath(p) !== "");
 }
+
+/** Return the distinct non-empty file paths represented in a structuredPatch array. */
+export function getDistinctPatchPaths(
+  patches: StructuredPatchEntry[],
+): string[] {
+  const seen = new Set<string>();
+  const paths: string[] = [];
+
+  for (const patch of patches) {
+    const patchPath = getPatchPath(patch);
+    if (!patchPath || seen.has(patchPath)) continue;
+    seen.add(patchPath);
+    paths.push(patchPath);
+  }
+
+  return paths;
+}
+
+/** True only when structuredPatch entries clearly represent multiple files. */
+export function isMultiFileStructuredPatch(
+  patches: StructuredPatchEntry[],
+): boolean {
+  return getDistinctPatchPaths(patches).length > 1;
+}