refactor: address CodeRabbit PR feedback

Improvements based on CodeRabbit review comments:

1. Use getPrimaryWorktreeBranch for consistent branch detection
   - Replace hardcoded 'main' fallback with getPrimaryWorktreeBranch()
   - Ensures auto-generated branch names respect the repo's actual primary branch
   - Handles repos using 'master' or other primary branch names

2. Extract worktree auto-selection logic to helper function
   - Create addAndSelectWorktree helper to eliminate code duplication
   - Use helper in both onWorktreeAutoSelect and handleBulkUpdate
   - Reduces maintenance burden and ensures consistent behavior

These changes improve code consistency and maintainability without affecting functionality.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

apps/ui/src/components/views/board-view.tsx

@@ -422,6 +422,31 @@ export function BoardView() {
   const selectedWorktreeBranch =
     currentWorktreeBranch || worktrees.find((w) => w.isMain)?.branch || 'main';
 
+  // Helper function to add and select a worktree
+  const addAndSelectWorktree = useCallback(
+    (worktreeResult: { path: string; branch: string }) => {
+      if (!currentProject) return;
+
+      const currentWorktrees = getWorktrees(currentProject.path);
+      const existingWorktree = currentWorktrees.find((w) => w.branch === worktreeResult.branch);
+
+      // Only add if it doesn't already exist (to avoid duplicates)
+      if (!existingWorktree) {
+        const newWorktreeInfo = {
+          path: worktreeResult.path,
+          branch: worktreeResult.branch,
+          isMain: false,
+          isCurrent: false,
+          hasWorktree: true,
+        };
+        setWorktrees(currentProject.path, [...currentWorktrees, newWorktreeInfo]);
+      }
+      // Select the worktree (whether it existed or was just added)
+      setCurrentWorktree(currentProject.path, worktreeResult.path, worktreeResult.branch);
+    },
+    [currentProject, getWorktrees, setWorktrees, setCurrentWorktree]
+  );
+
   // Extract all action handlers into a hook
   const {
     handleAddFeature,
@@ -467,26 +492,7 @@ export function BoardView() {
     outputFeature,
     projectPath: currentProject?.path || null,
     onWorktreeCreated: () => setWorktreeRefreshKey((k) => k + 1),
-    onWorktreeAutoSelect: (newWorktree) => {
+    onWorktreeAutoSelect: addAndSelectWorktree,
-      if (!currentProject) return;
-      // Check if worktree already exists in the store (by branch name)
-      const currentWorktrees = getWorktrees(currentProject.path);
-      const existingWorktree = currentWorktrees.find((w) => w.branch === newWorktree.branch);
-
-      // Only add if it doesn't already exist (to avoid duplicates)
-      if (!existingWorktree) {
-        const newWorktreeInfo = {
-          path: newWorktree.path,
-          branch: newWorktree.branch,
-          isMain: false,
-          isCurrent: false,
-          hasWorktree: true,
-        };
-        setWorktrees(currentProject.path, [...currentWorktrees, newWorktreeInfo]);
-      }
-      // Select the worktree (whether it existed or was just added)
-      setCurrentWorktree(currentProject.path, newWorktree.path, newWorktree.branch);
-    },
     currentWorktreeBranch,
   });
 
@@ -507,7 +513,8 @@ export function BoardView() {
           finalBranchName = undefined;
         } else if (workMode === 'auto') {
           // Auto-generate a branch name based on current branch and timestamp
-          const baseBranch = currentWorktreeBranch || 'main';
+          const baseBranch =
+            currentWorktreeBranch || getPrimaryWorktreeBranch(currentProject.path) || 'main';
           const timestamp = Date.now();
           const randomSuffix = Math.random().toString(36).substring(2, 6);
           finalBranchName = `feature/${baseBranch}-${timestamp}-${randomSuffix}`;
@@ -532,28 +539,7 @@ export function BoardView() {
                   }`
                 );
                 // Auto-select the worktree when creating/using it for bulk update
-                const currentWorktrees = getWorktrees(currentProject.path);
+                addAndSelectWorktree(result.worktree);
-                const existingWorktree = currentWorktrees.find(
-                  (w) => w.branch === result.worktree.branch
-                );
-
-                // Only add if it doesn't already exist (to avoid duplicates)
-                if (!existingWorktree) {
-                  const newWorktreeInfo = {
-                    path: result.worktree.path,
-                    branch: result.worktree.branch,
-                    isMain: false,
-                    isCurrent: false,
-                    hasWorktree: true,
-                  };
-                  setWorktrees(currentProject.path, [...currentWorktrees, newWorktreeInfo]);
-                }
-                // Select the worktree (whether it existed or was just added)
-                setCurrentWorktree(
-                  currentProject.path,
-                  result.worktree.path,
-                  result.worktree.branch
-                );
                 // Refresh worktree list in UI
                 setWorktreeRefreshKey((k) => k + 1);
               } else if (!result.success) {
@@ -609,9 +595,8 @@ export function BoardView() {
       updateFeature,
       exitSelectionMode,
       currentWorktreeBranch,
-      getWorktrees,
+      getPrimaryWorktreeBranch,
-      setWorktrees,
+      addAndSelectWorktree,
-      setCurrentWorktree,
       setWorktreeRefreshKey,
     ]
   );