🤖 Port changes from main commits 0a1aea1 and f147297

Apply changes that landed in main after our rebase base point:

1. Review Panel (0a1aea1):
   - Remove extractCommonPrefix import and usage
   - Remove commonPrefix state
   - Simplify file tree to show full directory structure

2. FileTree (0a1aea1):
   - Remove commonPrefix prop from component interface
   - Remove startNode computation logic
   - Remove CommonPrefix display element
   - Render from root node directly

3. ProjectSidebar (f147297):
   - Wrap Add Project button in TooltipWrapper
   - Show tooltip on hover matching other UI elements
   - Remove redundant title attribute

These are non-styling logic changes that need to be preserved
in the Tailwind migration.

Author: kylecarbs

src/components/ProjectSidebar.tsx

@@ -426,14 +426,18 @@ const ProjectSidebarInner: React.FC<ProjectSidebarProps> = ({
                 <h2 className="m-0 text-[13px] font-semibold text-[#cccccc] uppercase tracking-[0.8px]">
                   Projects
                 </h2>
-                <button
-                  onClick={onAddProject}
-                  title="Add Project"
-                  aria-label="Add project"
-                  className="w-6 h-6 bg-transparent text-[#cccccc] border border-transparent rounded cursor-pointer text-lg flex items-center justify-center p-0 transition-all duration-200 hover:bg-[#2a2a2b] hover:border-[#3c3c3c]"
-                >
-                  +
-                </button>
+                <TooltipWrapper inline>
+                  <button
+                    onClick={onAddProject}
+                    aria-label="Add project"
+                    className="w-6 h-6 bg-transparent text-[#cccccc] border border-transparent rounded cursor-pointer text-lg flex items-center justify-center p-0 transition-all duration-200 hover:bg-[#2a2a2b] hover:border-[#3c3c3c]"
+                  >
+                    +
+                  </button>
+                  <Tooltip className="tooltip" align="right">
+                    Add Project
+                  </Tooltip>
+                </TooltipWrapper>
               </div>
               <div className="flex-1 overflow-y-auto">
                 {projects.size === 0 ? (

src/components/RightSidebar/CodeReview/FileTree.tsx

@@ -55,7 +55,6 @@ const TreeNodeContent: React.FC<{
   depth: number;
   selectedPath: string | null;
   onSelectFile: (path: string | null) => void;
-  commonPrefix: string | null;
   getFileReadStatus?: (filePath: string) => { total: number; read: number } | null;
   expandStateMap: Record<string, boolean>;
   setExpandStateMap: (
@@ -66,7 +65,6 @@ const TreeNodeContent: React.FC<{
   depth,
   selectedPath,
   onSelectFile,
-  commonPrefix,
   getFileReadStatus,
   expandStateMap,
   setExpandStateMap,
@@ -213,7 +211,6 @@ const TreeNodeContent: React.FC<{
             depth={depth + 1}
             selectedPath={selectedPath}
             onSelectFile={onSelectFile}
-            commonPrefix={commonPrefix}
             getFileReadStatus={getFileReadStatus}
             expandStateMap={expandStateMap}
             setExpandStateMap={setExpandStateMap}
@@ -228,7 +225,6 @@ interface FileTreeExternalProps {
   selectedPath: string | null;
   onSelectFile: (path: string | null) => void;
   isLoading?: boolean;
-  commonPrefix?: string | null;
   getFileReadStatus?: (filePath: string) => { total: number; read: number } | null;
   workspaceId: string;
 }
@@ -238,7 +234,6 @@ export const FileTree: React.FC<FileTreeExternalProps> = ({
   selectedPath,
   onSelectFile,
   isLoading = false,
-  commonPrefix = null,
   getFileReadStatus,
   workspaceId,
 }) => {
@@ -249,23 +244,6 @@ export const FileTree: React.FC<FileTreeExternalProps> = ({
     { listener: true }
   );
 
-  // Find the node at the common prefix path to start rendering from
-  const startNode = React.useMemo(() => {
-    if (!commonPrefix || !root) return root;
-
-    // Navigate to the node at the common prefix path
-    const parts = commonPrefix.split("/");
-    let current = root;
-
-    for (const part of parts) {
-      const child = current.children.find((c) => c.name === part);
-      if (!child) return root; // Fallback if path not found
-      current = child;
-    }
-
-    return current;
-  }, [root, commonPrefix]);
-
   return (
     <>
       <div className="py-2 px-3 border-b border-[#3e3e42] text-xs font-medium text-[#ccc] font-primary flex items-center gap-2">
@@ -279,23 +257,17 @@ export const FileTree: React.FC<FileTreeExternalProps> = ({
           </button>
         )}
       </div>
-      {commonPrefix && (
-        <div className="py-1.5 px-3 bg-[#1e1e1e] border-b border-[#3e3e42] text-[11px] text-[#888] font-monospace">
-          {commonPrefix}/
-        </div>
-      )}
       <div className="flex-1 min-h-0 p-3 overflow-y-auto font-monospace text-xs">
-        {isLoading && !startNode ? (
+        {isLoading && !root ? (
           <div className="py-5 text-[#888] text-center">Loading file tree...</div>
-        ) : startNode ? (
-          startNode.children.map((child) => (
+        ) : root ? (
+          root.children.map((child) => (
             <TreeNodeContent
               key={child.path}
               node={child}
               depth={0}
               selectedPath={selectedPath}
               onSelectFile={onSelectFile}
-              commonPrefix={commonPrefix}
               getFileReadStatus={getFileReadStatus}
               expandStateMap={expandStateMap}
               setExpandStateMap={setExpandStateMap}

src/components/RightSidebar/CodeReview/ReviewPanel.tsx

@@ -31,12 +31,7 @@ import { useReviewState } from "@/hooks/useReviewState";
 import { parseDiff, extractAllHunks } from "@/utils/git/diffParser";
 import { getReviewSearchStateKey } from "@/constants/storage";
 import { Tooltip, TooltipWrapper } from "@/components/Tooltip";
-import {
-  parseNumstat,
-  buildFileTree,
-  extractNewPath,
-  extractCommonPrefix,
-} from "@/utils/git/numstatParser";
+import { parseNumstat, buildFileTree, extractNewPath } from "@/utils/git/numstatParser";
 import type { DiffHunk, ReviewFilters as ReviewFiltersType } from "@/types/review";
 import type { FileTreeNode } from "@/utils/git/numstatParser";
 import { matchesKeybind, KEYBINDS, formatKeybind } from "@/utils/ui/keybinds";
@@ -137,7 +132,6 @@ export const ReviewPanel: React.FC<ReviewPanelProps> = ({
   const [isPanelFocused, setIsPanelFocused] = useState(false);
   const [refreshTrigger, setRefreshTrigger] = useState(0);
   const [fileTree, setFileTree] = useState<FileTreeNode | null>(null);
-  const [commonPrefix, setCommonPrefix] = useState<string | null>(null);
   // Map of hunkId -> toggle function for expand/collapse
   const toggleExpandFnsRef = useRef<Map<string, () => void>>(new Map());
 
@@ -220,13 +214,9 @@ export const ReviewPanel: React.FC<ReviewPanelProps> = ({
           const numstatOutput = numstatResult.data.output ?? "";
           const fileStats = parseNumstat(numstatOutput);
 
-          // Extract common prefix for display (don't modify paths)
-          const prefix = extractCommonPrefix(fileStats);
-
           // Build tree with original paths (needed for git commands)
           const tree = buildFileTree(fileStats);
           setFileTree(tree);
-          setCommonPrefix(prefix);
         }
       } catch (err) {
         console.error("Failed to load file tree:", err);
@@ -710,7 +700,6 @@ export const ReviewPanel: React.FC<ReviewPanelProps> = ({
                 selectedPath={selectedFilePath}
                 onSelectFile={setSelectedFilePath}
                 isLoading={isLoadingTree}
-                commonPrefix={commonPrefix}
                 getFileReadStatus={getFileReadStatus}
                 workspaceId={workspaceId}
               />