From d42d895f2a579e79a23fcbfb1d412a92af2af5a0 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E9=9B=B7=E7=94=B5=E8=8A=BD=E8=A1=A3?=
 <zhangyanghaha0407@outlook.com>
Date: Fri, 17 Apr 2026 01:44:50 -0400
Subject: [PATCH] chore(lint): fix react-hooks/exhaustive-deps warnings

- App.tsx: replace per-field ui.navItems/ui.paletteQuery with the ui
  object so the memo captures every accessed field.
- ConsoleView.tsx: add eslint-disable for the logRevision dep; it is
  intentionally tracked to force ref re-evaluation.
---
 src/renderer/App.tsx               | 4 +---
 src/renderer/views/ConsoleView.tsx | 1 +
 2 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/src/renderer/App.tsx b/src/renderer/App.tsx
index cd506154..050a7d7c 100644
--- a/src/renderer/App.tsx
+++ b/src/renderer/App.tsx
@@ -84,14 +84,12 @@ export function App() {
       company.companyGoals,
       company.companyProjects,
     ), [
-      ui.navItems,
-      ui.paletteQuery,
+      ui,
       company.companyAgents,
       company.tasks,
       company.runs,
       company.companyGoals,
       company.companyProjects,
-     
     ]);
 
   const paletteActions = useMemo(
diff --git a/src/renderer/views/ConsoleView.tsx b/src/renderer/views/ConsoleView.tsx
index f5ec1832..ac8b107e 100644
--- a/src/renderer/views/ConsoleView.tsx
+++ b/src/renderer/views/ConsoleView.tsx
@@ -40,6 +40,7 @@ export function ConsoleView({
     // logRevision is intentionally part of deps: it bumps on every run-log
     // push into the ref, forcing this memo (and this component) to re-evaluate
     // the ref contents on re-render.
+    // eslint-disable-next-line react-hooks/exhaustive-deps
     [selectedRun, logRevision, logBufferRef],
   );
   const runSelectOptions = useMemo(() => {