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.
This commit is contained in:
1 parent
bc1f24d60c
commit
d42d895f2a
2 files changed
+2
-3
No files matched your search
@@ -84,14 +84,12 @@ export function App() {
|
|||||||
company.companyGoals,
|
company.companyGoals,
|
||||||
company.companyProjects,
|
company.companyProjects,
|
||||||
), [
|
), [
|
||||||
ui.navItems,
|
ui,
|
||||||
ui.paletteQuery,
|
|
||||||
company.companyAgents,
|
company.companyAgents,
|
||||||
company.tasks,
|
company.tasks,
|
||||||
company.runs,
|
company.runs,
|
||||||
company.companyGoals,
|
company.companyGoals,
|
||||||
company.companyProjects,
|
company.companyProjects,
|
||||||
|
|
||||||
]);
|
]);
|
||||||
|
|
||||||
const paletteActions = useMemo(
|
const paletteActions = useMemo(
|
||||||
|
|||||||
@@ -40,6 +40,7 @@ export function ConsoleView({
|
|||||||
// logRevision is intentionally part of deps: it bumps on every run-log
|
// 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
|
// push into the ref, forcing this memo (and this component) to re-evaluate
|
||||||
// the ref contents on re-render.
|
// the ref contents on re-render.
|
||||||
|
// eslint-disable-next-line react-hooks/exhaustive-deps
|
||||||
[selectedRun, logRevision, logBufferRef],
|
[selectedRun, logRevision, logBufferRef],
|
||||||
);
|
);
|
||||||
const runSelectOptions = useMemo(() => {
|
const runSelectOptions = useMemo(() => {
|
||||||
|
|||||||
Reference in new issue
Block a user