fix: 🤖 Trigger refresh after Track All to update UI state (#321)

Follow-up to PR #318 (untracked files feature). Includes two
improvements:

## 1. Trigger refresh after Track All

After staging untracked files via Track All button, the UI should
automatically refresh to reflect the changes (updated hunks, tree, and
untracked count).

**Implementation:**
- Pass `onRefresh` callback from ReviewPanel through ReviewControls to
UntrackedStatus
- Call `onRefresh()` after successful `git add` operation in Track All
handler
- This refresh only happens from user action (clicking Track All),
preventing infinite loops

## 2. Rename 'Dirty' to 'Uncommitted' for clarity

The checkbox was labeled "Dirty" but actually includes ALL uncommitted
changes (staged + unstaged) via `git diff HEAD`. "Uncommitted" is more
accurate and less ambiguous.

**Breaking change:** localStorage keys changed from
`review-include-dirty` to `review-include-uncommitted`. Users will lose
this preference per workspace (acceptable for early development phase).

## Testing

1. Open Code Review panel
2. Create an untracked file
3. Click the untracked badge to open tooltip
4. Click "Track All" button
5. ✅ UI automatically refreshes showing the files are now staged
6. ✅ Untracked count updates to 0
7. ✅ Hunks and tree reflect the new staged changes
8. ✅ Checkbox now labeled "Uncommitted" instead of "Dirty"

_Generated with `cmux`_

Author: ammar-agent

src/components/RightSidebar/CodeReview/ReviewControls.tsx

@@ -162,8 +162,8 @@ export const ReviewControls: React.FC<ReviewControlsProps> = ({
     }
   };
 
-  const handleDirtyToggle = (e: React.ChangeEvent<HTMLInputElement>) => {
-    onFiltersChange({ ...filters, includeDirty: e.target.checked });
+  const handleUncommittedToggle = (e: React.ChangeEvent<HTMLInputElement>) => {
+    onFiltersChange({ ...filters, includeUncommitted: e.target.checked });
   };
 
   const handleSetDefault = () => {
@@ -202,14 +202,19 @@ export const ReviewControls: React.FC<ReviewControlsProps> = ({
       )}
 
       <CheckboxLabel>
-        <input type="checkbox" checked={filters.includeDirty} onChange={handleDirtyToggle} />
-        Dirty
+        <input
+          type="checkbox"
+          checked={filters.includeUncommitted}
+          onChange={handleUncommittedToggle}
+        />
+        Uncommitted
       </CheckboxLabel>
 
       <UntrackedStatus
         workspaceId={workspaceId}
         workspacePath={workspacePath}
         refreshTrigger={refreshTrigger}
+        onRefresh={onRefresh}
       />
 
       <Separator />

src/components/RightSidebar/CodeReview/ReviewPanel.tsx

@@ -236,12 +236,12 @@ interface DiagnosticInfo {
 }
 
 /**
- * Build git diff command based on diffBase and includeDirty flag
+ * Build git diff command based on diffBase and includeUncommitted flag
  * Shared logic between numstat (file tree) and diff (hunks) commands
  */
 function buildGitDiffCommand(
   diffBase: string,
-  includeDirty: boolean,
+  includeUncommitted: boolean,
   pathFilter: string,
   command: "diff" | "numstat"
 ): string {
@@ -258,8 +258,8 @@ function buildGitDiffCommand(
     cmd = `git diff ${diffBase}...HEAD${flag}${pathFilter}`;
   }
 
-  // Append dirty changes if requested
-  if (includeDirty) {
+  // Append uncommitted changes if requested
+  if (includeUncommitted) {
     cmd += ` && git diff HEAD${flag}${pathFilter}`;
   }
 
@@ -295,17 +295,17 @@ export const ReviewPanel: React.FC<ReviewPanelProps> = ({
   // Persist diff base per workspace (falls back to global default)
   const [diffBase, setDiffBase] = usePersistedState(`review-diff-base:${workspaceId}`, defaultBase);
 
-  // Persist includeDirty flag per workspace
-  const [includeDirty, setIncludeDirty] = usePersistedState(
-    `review-include-dirty:${workspaceId}`,
+  // Persist includeUncommitted flag per workspace
+  const [includeUncommitted, setIncludeUncommitted] = usePersistedState(
+    `review-include-uncommitted:${workspaceId}`,
     false
   );
 
   const [filters, setFilters] = useState<ReviewFiltersType>({
     showReviewed: true,
     statusFilter: "all",
     diffBase: diffBase,
-    includeDirty: includeDirty,
+    includeUncommitted: includeUncommitted,
   });
 
   // Load file tree - when workspace, diffBase, or refreshTrigger changes
@@ -317,7 +317,7 @@ export const ReviewPanel: React.FC<ReviewPanelProps> = ({
       try {
         const numstatCommand = buildGitDiffCommand(
           filters.diffBase,
-          filters.includeDirty,
+          filters.includeUncommitted,
           "", // No path filter for file tree
           "numstat"
         );
@@ -352,7 +352,7 @@ export const ReviewPanel: React.FC<ReviewPanelProps> = ({
     return () => {
       cancelled = true;
     };
-  }, [workspaceId, workspacePath, filters.diffBase, filters.includeDirty, refreshTrigger]);
+  }, [workspaceId, workspacePath, filters.diffBase, filters.includeUncommitted, refreshTrigger]);
 
   // Load diff hunks - when workspace, diffBase, selected path, or refreshTrigger changes
   useEffect(() => {
@@ -369,7 +369,7 @@ export const ReviewPanel: React.FC<ReviewPanelProps> = ({
 
         const diffCommand = buildGitDiffCommand(
           filters.diffBase,
-          filters.includeDirty,
+          filters.includeUncommitted,
           pathFilter,
           "diff"
         );
@@ -435,7 +435,7 @@ export const ReviewPanel: React.FC<ReviewPanelProps> = ({
     workspaceId,
     workspacePath,
     filters.diffBase,
-    filters.includeDirty,
+    filters.includeUncommitted,
     selectedFilePath,
     refreshTrigger,
   ]);
@@ -445,10 +445,10 @@ export const ReviewPanel: React.FC<ReviewPanelProps> = ({
     setDiffBase(filters.diffBase);
   }, [filters.diffBase, setDiffBase]);
 
-  // Persist includeDirty when it changes
+  // Persist includeUncommitted when it changes
   useEffect(() => {
-    setIncludeDirty(filters.includeDirty);
-  }, [filters.includeDirty, setIncludeDirty]);
+    setIncludeUncommitted(filters.includeUncommitted);
+  }, [filters.includeUncommitted, setIncludeUncommitted]);
 
   // For MVP: No review state tracking, just show all hunks
   const filteredHunks = hunks;

src/components/RightSidebar/CodeReview/UntrackedStatus.tsx

@@ -10,6 +10,7 @@ interface UntrackedStatusProps {
   workspaceId: string;
   workspacePath: string;
   refreshTrigger?: number;
+  onRefresh?: () => void;
 }
 
 const Container = styled.div`
@@ -128,19 +129,25 @@ export const UntrackedStatus: React.FC<UntrackedStatusProps> = ({
   workspaceId,
   workspacePath,
   refreshTrigger,
+  onRefresh,
 }) => {
   const [untrackedFiles, setUntrackedFiles] = useState<string[]>([]);
   const [isLoading, setIsLoading] = useState(false);
   const [showTooltip, setShowTooltip] = useState(false);
   const [isTracking, setIsTracking] = useState(false);
   const containerRef = useRef<HTMLDivElement>(null);
   const hasLoadedOnce = useRef(false);
+  const loadingRef = useRef(false); // Prevent concurrent loads
 
   // Load untracked files
   useEffect(() => {
     let cancelled = false;
 
     const loadUntracked = async () => {
+      // Prevent concurrent loads
+      if (loadingRef.current) return;
+      loadingRef.current = true;
+
       // Only show loading on first load ever, not on subsequent refreshes
       if (!hasLoadedOnce.current) {
         setIsLoading(true);
@@ -167,6 +174,7 @@ export const UntrackedStatus: React.FC<UntrackedStatusProps> = ({
       } catch (err) {
         console.error("Failed to load untracked files:", err);
       } finally {
+        loadingRef.current = false;
         setIsLoading(false);
       }
     };
@@ -208,8 +216,11 @@ export const UntrackedStatus: React.FC<UntrackedStatusProps> = ({
       );
 
       if (result.success) {
-        setUntrackedFiles([]);
+        // Close tooltip first
         setShowTooltip(false);
+        // Trigger refresh - this will reload untracked files from git
+        // Don't clear untrackedFiles optimistically to avoid flicker
+        onRefresh?.();
       } else {
         console.error("Failed to track files:", result.error);
       }

src/types/review.ts

@@ -82,8 +82,8 @@ export interface ReviewFilters {
   filePathFilter?: string;
   /** Base reference to diff against (e.g., "HEAD", "main", "origin/main") */
   diffBase: string;
-  /** Whether to include uncommitted dirty changes in the diff */
-  includeDirty: boolean;
+  /** Whether to include uncommitted changes (staged + unstaged) in the diff */
+  includeUncommitted: boolean;
 }
 
 /**