From 3b06bed4441b4d7c044f74c89a50e40222e67673 Mon Sep 17 00:00:00 2001 From: Claude Date: Thu, 29 Jan 2026 19:33:49 +0000 Subject: [PATCH] fix: remove automatic file filter when clicking Review When clicking the Review button in the diff sidebar, a sub-chat filter was automatically applied (filteredSubChatIdAtom set to activeSubChatId), causing a "Filtered" badge to appear and hiding files not associated with the active sub-chat. This was confusing as users didn't expect or request the filtering. The review should operate on all changes by default; users can still manually apply a sub-chat filter if needed. Co-Authored-By: Claude Opus 4.5 --- src/renderer/features/agents/main/active-chat.tsx | 10 +--------- 1 file changed, 1 insertion(+), 9 deletions(-) diff --git a/src/renderer/features/agents/main/active-chat.tsx b/src/renderer/features/agents/main/active-chat.tsx index 5bc5bf06..f80edc0f 100644 --- a/src/renderer/features/agents/main/active-chat.tsx +++ b/src/renderer/features/agents/main/active-chat.tsx @@ -4645,9 +4645,6 @@ export function ChatView({ const [isCreatingPr, setIsCreatingPr] = useAtom(isCreatingPrAtom) // Review loading state const [isReviewing, setIsReviewing] = useState(false) - // Subchat filter setter - used by handleReview to filter by active subchat - const setFilteredSubChatId = useSetAtom(filteredSubChatIdAtom) - // Determine if we're in sandbox mode const chatSourceMode = useAtomValue(chatSourceModeAtom) @@ -5300,11 +5297,6 @@ export function ChatView({ return } - // Set filter to show only files from the active subchat - if (activeSubChatId) { - setFilteredSubChatId(activeSubChatId) - } - // Generate review message and set it for ChatViewInner to send const message = generateReviewMessage(context) setPendingReviewMessage(message) @@ -5316,7 +5308,7 @@ export function ChatView({ } finally { setIsReviewing(false) } - }, [chatId, activeSubChatId, setPendingReviewMessage, setFilteredSubChatId]) + }, [chatId, setPendingReviewMessage]) // Handle Fix Conflicts - sends a message to Claude to sync with main and fix merge conflicts const setPendingConflictResolutionMessage = useSetAtom(pendingConflictResolutionMessageAtom)