Skip to content

Commit 9a48e6f

Browse files
saagar210claude
andcommitted
fix(components): complete handleWorkspacePersonalizationChange deps
The useCallback for handleWorkspacePersonalizationChange listed only [savedDraftId] in its dep array while the body closed over setCaseIntake. setCaseIntake comes from useDraftIntake and wraps a stable useState dispatcher, so the omission wasn't a practical bug, but it was the last of the four react-hooks/exhaustive-deps warnings originally cited against DraftTab in the audit. Adds setCaseIntake to the dep array. DraftTab is now fully clean under the react-hooks/exhaustive-deps rule. Project-wide lint warnings: 6 -> 5. Remaining warnings are all outside DraftTab (RatingPanel, PilotDashboard, three stale eslint-disable directives in useInitialize). No behavior change. Typecheck + vitest (241/241) clean. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 15d1de9 commit 9a48e6f

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

src/components/Draft/DraftTab.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -406,7 +406,7 @@ export const DraftTab = forwardRef<DraftTabHandle, DraftTabProps>(
406406
return next;
407407
});
408408
},
409-
[savedDraftId],
409+
[savedDraftId, setCaseIntake],
410410
);
411411

412412
const {

0 commit comments

Comments
 (0)