fix: unintended tear down of agent loop (#483)

fixes #465

codex-cli/src/components/chat/terminal-chat.tsx

@@ -305,15 +305,10 @@ export default function TerminalChat({
       agentRef.current = undefined;
       forceUpdate(); // re‑render after teardown too
     };
-  }, [
+    // We intentionally omit 'approvalPolicy' and 'confirmationPrompt' from the deps
-    model,
+    // so switching modes or showing confirmation dialogs doesn’t tear down the loop.
-    provider,
+    // eslint-disable-next-line react-hooks/exhaustive-deps
-    config,
+  }, [model, provider, config, requestConfirmation, additionalWritableRoots]);
-    approvalPolicy,
-    confirmationPrompt,
-    requestConfirmation,
-    additionalWritableRoots,
-  ]);
 
   // Whenever loading starts/stops, reset or start a timer — but pause the
   // timer while a confirmation overlay is displayed so we don't trigger a