🤖 refactor: move modal constant to module scope, remove wrapper callbacks

Simplifications in App.tsx for net negative LoC:
- Moved INITIAL_WORKSPACE_MODAL_STATE to module scope (prevents recreation)
- Removed thin wrapper callbacks (handleAddProjectCallback, etc.)
- Directly pass functions to LeftSidebar (addProject, handleAddWorkspace, handleRemoveProject)

Net impact: 774 → 757 lines (-17 LoC)

Generated with `cmux`

Author: ammar-agent

src/App.tsx

@@ -32,6 +32,17 @@ import { parseRuntimeString } from "./utils/chatCommands";
 
 const THINKING_LEVELS: ThinkingLevel[] = ["off", "low", "medium", "high"];
 
+const INITIAL_WORKSPACE_MODAL_STATE = {
+  isOpen: false,
+  projectPath: null as string | null,
+  projectName: "",
+  branches: [] as string[],
+  defaultTrunk: undefined as string | undefined,
+  loadError: null as string | null,
+  startMessage: undefined as string | undefined,
+  model: undefined as string | undefined,
+};
+
 function AppInner() {
   // Get app-level state from context
   const {
@@ -47,17 +58,6 @@ function AppInner() {
     setSelectedWorkspace,
   } = useApp();
 
-  const INITIAL_WORKSPACE_MODAL_STATE = {
-    isOpen: false,
-    projectPath: null as string | null,
-    projectName: "",
-    branches: [] as string[],
-    defaultTrunk: undefined as string | undefined,
-    loadError: null as string | null,
-    startMessage: undefined as string | undefined,
-    model: undefined as string | undefined,
-  };
-
   const [workspaceModalState, setWorkspaceModalState] = useState(INITIAL_WORKSPACE_MODAL_STATE);
   const workspaceModalProjectRef = useRef<string | null>(null);
 
@@ -236,25 +236,6 @@ function AppInner() {
     []
   );
 
-  // Memoize callbacks to prevent LeftSidebar/ProjectSidebar re-renders
-  const handleAddProjectCallback = useCallback(() => {
-    void addProject();
-  }, [addProject]);
-
-  const handleAddWorkspaceCallback = useCallback(
-    (projectPath: string) => {
-      void handleAddWorkspace(projectPath);
-    },
-    [handleAddWorkspace]
-  );
-
-  const handleRemoveProjectCallback = useCallback(
-    (path: string) => {
-      void handleRemoveProject(path);
-    },
-    [handleRemoveProject]
-  );
-
   const handleCreateWorkspace = async (
     branchName: string,
     trunkBranch: string,
@@ -686,9 +667,9 @@ function AppInner() {
       <div className="bg-bg-dark flex h-screen overflow-hidden [@media(max-width:768px)]:flex-col">
         <LeftSidebar
           onSelectWorkspace={handleWorkspaceSwitch}
-          onAddProject={handleAddProjectCallback}
-          onAddWorkspace={handleAddWorkspaceCallback}
-          onRemoveProject={handleRemoveProjectCallback}
+          onAddProject={addProject}
+          onAddWorkspace={handleAddWorkspace}
+          onRemoveProject={handleRemoveProject}
           lastReadTimestamps={lastReadTimestamps}
           onToggleUnread={onToggleUnread}
           collapsed={sidebarCollapsed}