fix: Various bug fixes (listed) (#544)

- Remove folders that no longer exist on disk
- Add scroll bar to session view
- Fix file changes detection bug report
- Fix Dangerous Auto-Accept on Click (From Array Feedback PDF)
- Fix cursor glow not respecting the cursor glow setting

Author: charlesvien

apps/twig/src/renderer/components/TorchGlow.tsx

@@ -1,3 +1,4 @@
+import { useSettingsStore } from "@features/settings/stores/settingsStore";
 import { useThemeStore } from "@stores/themeStore";
 import { useEffect, useState } from "react";
 
@@ -7,6 +8,7 @@ interface TorchGlowProps {
 
 export function TorchGlow({ containerRef }: TorchGlowProps) {
   const isDarkMode = useThemeStore((state) => state.isDarkMode);
+  const cursorGlow = useSettingsStore((state) => state.cursorGlow);
   const [mousePos, setMousePos] = useState<{ x: number; y: number } | null>(
     null,
   );
@@ -41,8 +43,8 @@ export function TorchGlow({ containerRef }: TorchGlowProps) {
     };
   }, [containerRef]);
 
-  // Only show in dark mode when hovering
-  if (!isDarkMode || !isHovering || !mousePos) return null;
+  // Only show in dark mode when hovering and cursor glow is enabled
+  if (!isDarkMode || !cursorGlow || !isHovering || !mousePos) return null;
 
   return (
     <>

apps/twig/src/renderer/features/panels/hooks/usePanelLayoutHooks.tsx

@@ -1,6 +1,9 @@
 import { FileIcon } from "@components/ui/FileIcon";
 import { TabContentRenderer } from "@features/task-detail/components/TabContentRenderer";
-import { useTaskExecutionStore } from "@features/task-detail/stores/taskExecutionStore";
+import {
+  selectTaskRepoPath,
+  useTaskExecutionStore,
+} from "@features/task-detail/stores/taskExecutionStore";
 import { ChatCenteredText, Terminal } from "@phosphor-icons/react";
 import type { Task } from "@shared/types";
 import { useCallback, useEffect, useMemo, useRef } from "react";
@@ -82,9 +85,7 @@ export function useTabInjection(
   closeTab: (taskId: string, panelId: string, tabId: string) => void,
 ): Tab[] {
   const worktreePath = useWorkspaceStore(selectWorktreePath(taskId));
-  const storedRepoPath = useTaskExecutionStore(
-    (state) => state.taskStates[taskId]?.repoPath ?? null,
-  );
+  const storedRepoPath = useTaskExecutionStore(selectTaskRepoPath(taskId));
   const repoPath = worktreePath || storedRepoPath || "";
 
   return useMemo(

apps/twig/src/renderer/features/right-sidebar/stores/fileTreeStore.ts

@@ -38,3 +38,8 @@ export const useFileTreeStore = create<FileTreeStore>()((set) => ({
       },
     })),
 }));
+
+// Selector factory for checking if a path is expanded
+export const selectIsPathExpanded =
+  (taskId: string, path: string) => (state: FileTreeStore) =>
+    state.expandedPaths[taskId]?.has(path) ?? false;

apps/twig/src/renderer/features/sessions/components/ConversationView.tsx

@@ -121,7 +121,7 @@ export function ConversationView({
     <div className="relative flex-1">
       <div
         ref={scrollRef}
-        className="scrollbar-hide absolute inset-0 overflow-auto bg-white p-2 pb-16 dark:bg-gray-1"
+        className="absolute inset-0 overflow-auto bg-white p-2 pb-16 dark:bg-gray-1"
       >
         <div className="flex flex-col gap-3">
           {items.map((item) =>

apps/twig/src/renderer/features/sessions/components/InlinePermissionSelector.tsx

@@ -202,17 +202,6 @@ export function InlinePermissionSelector({
     [customInput, options, onSelect],
   );
 
-  const handleOptionClick = (index: number) => {
-    if (disabled) return;
-    const opt = allOptions[index];
-    if (isCustomOption(opt.optionId)) {
-      setSelectedIndex(index);
-      setIsCustomInputMode(true);
-    } else {
-      onSelect(opt.optionId);
-    }
-  };
-
   return (
     <Box
       ref={containerRef}
@@ -258,10 +247,9 @@ export function InlinePermissionSelector({
               key={option.optionId}
               align="center"
               gap="1"
-              className={`cursor-pointer py-0.5 ${
+              className={`py-0.5 ${
                 isSelected ? "text-gray-12" : "text-gray-10"
               }`}
-              onClick={() => handleOptionClick(index)}
             >
               <span
                 className={`inline-block w-3 text-xs ${

apps/twig/src/renderer/features/task-detail/components/FileTreePanel.tsx

@@ -1,7 +1,10 @@
 import { FileIcon } from "@components/ui/FileIcon";
 import { PanelMessage } from "@components/ui/PanelMessage";
 import { isFileTabActiveInTree, usePanelLayoutStore } from "@features/panels";
-import { useFileTreeStore } from "@features/right-sidebar/stores/fileTreeStore";
+import {
+  selectIsPathExpanded,
+  useFileTreeStore,
+} from "@features/right-sidebar/stores/fileTreeStore";
 import { useTaskData } from "@features/task-detail/hooks/useTaskData";
 import { CaretRight, FolderIcon, FolderOpenIcon } from "@phosphor-icons/react";
 import { Box, Flex, Text } from "@radix-ui/themes";
@@ -40,9 +43,7 @@ function LazyTreeItem({
   repoPath,
   isFileActive,
 }: LazyTreeItemProps) {
-  const isExpanded = useFileTreeStore(
-    (state) => state.expandedPaths[taskId]?.has(entry.path) ?? false,
-  );
+  const isExpanded = useFileTreeStore(selectIsPathExpanded(taskId, entry.path));
   const togglePath = useFileTreeStore((state) => state.togglePath);
   const collapseAll = useFileTreeStore((state) => state.collapseAll);
   const openFile = usePanelLayoutStore((state) => state.openFile);

apps/twig/src/renderer/features/task-detail/components/TaskDetail.tsx

@@ -11,7 +11,11 @@ import { useStatusBar } from "@hooks/useStatusBar";
 import { Box, Code, Flex, Text } from "@radix-ui/themes";
 import type { Task } from "@shared/types";
 import { useMemo } from "react";
-import { useWorkspaceStore } from "@/renderer/features/workspace/stores/workspaceStore";
+import {
+  selectWorkspace,
+  selectWorktreePath,
+  useWorkspaceStore,
+} from "@/renderer/features/workspace/stores/workspaceStore";
 
 interface TaskDetailProps {
   task: Task;
@@ -23,9 +27,7 @@ export function TaskDetail({ task: initialTask }: TaskDetailProps) {
     initialTask,
   });
 
-  const worktreePath = useWorkspaceStore(
-    (state) => state.workspaces[initialTask.id]?.worktreePath,
-  );
+  const worktreePath = useWorkspaceStore(selectWorktreePath(initialTask.id));
   const effectiveRepoPath = worktreePath ?? taskData.repoPath;
 
   useFileWatcher(effectiveRepoPath, taskData.task.id);
@@ -55,10 +57,7 @@ export function TaskDetail({ task: initialTask }: TaskDetailProps) {
 
   useWorkspaceEvents(taskId);
   const task = taskData.task;
-  const workspace = useWorkspaceStore(
-    (state) => state.workspaces[taskId] ?? null,
-  );
-
+  const workspace = useWorkspaceStore(selectWorkspace(taskId));
   const branchName = workspace?.branchName;
 
   const headerContent = useMemo(

apps/twig/src/renderer/features/task-detail/hooks/useTaskData.ts

@@ -1,4 +1,8 @@
-import { useTaskExecutionStore } from "@features/task-detail/stores/taskExecutionStore";
+import {
+  selectTaskRepoExists,
+  selectTaskRepoPath,
+  useTaskExecutionStore,
+} from "@features/task-detail/stores/taskExecutionStore";
 import { useTasks } from "@features/tasks/hooks/useTasks";
 import type { Task } from "@shared/types";
 import { cloneStore } from "@stores/cloneStore";
@@ -27,12 +31,8 @@ export function useTaskData({ taskId, initialTask }: UseTaskDataParams) {
   }, [initializeRepoPath, taskId, task]);
 
   // Subscribe to specific fields reactively to avoid unnecessary rerenders
-  const repoPath = useTaskExecutionStore(
-    (state) => state.taskStates[taskId]?.repoPath ?? null,
-  );
-  const repoExists = useTaskExecutionStore(
-    (state) => state.taskStates[taskId]?.repoExists ?? null,
-  );
+  const repoPath = useTaskExecutionStore(selectTaskRepoPath(taskId));
+  const repoExists = useTaskExecutionStore(selectTaskRepoExists(taskId));
 
   const repository = getTaskRepository(task);
 

apps/twig/src/renderer/features/task-detail/stores/taskExecutionStore.ts

@@ -147,3 +147,12 @@ export const useTaskExecutionStore = create<TaskExecutionStore>()(
     },
   ),
 );
+
+// Selector factories for parameterized access (taskId-based)
+export const selectTaskRepoPath =
+  (taskId: string) => (state: TaskExecutionStore) =>
+    state.taskStates[taskId]?.repoPath ?? null;
+
+export const selectTaskRepoExists =
+  (taskId: string) => (state: TaskExecutionStore) =>
+    state.taskStates[taskId]?.repoExists ?? null;

apps/twig/src/renderer/stores/registeredFoldersStore.ts

@@ -43,11 +43,23 @@ export const useRegisteredFoldersStore = create<RegisteredFoldersState>()(
     (async () => {
       try {
         const loadedFolders = await loadFolders();
+
+        // Remove folders that no longer exist on disk
+        const deletedFolders = loadedFolders.filter((f) => f.exists === false);
+        for (const folder of deletedFolders) {
+          trpcVanilla.folders.removeFolder
+            .mutate({ folderId: folder.id })
+            .catch((err) =>
+              log.error(`Failed to remove deleted folder ${folder.path}:`, err),
+            );
+        }
+        const existingFolders = loadedFolders.filter((f) => f.exists !== false);
+
         // Merge with existing state to preserve folders added during load
         set((state) => {
           // Dedupe by id, local state wins for freshness
           const byId = new Map<string, RegisteredFolder>();
-          for (const f of loadedFolders) byId.set(f.id, f);
+          for (const f of existingFolders) byId.set(f.id, f);
           for (const f of state.folders) byId.set(f.id, f);
           return { folders: Array.from(byId.values()), isLoaded: true };
         });
@@ -140,6 +152,7 @@ export const useRegisteredFoldersStore = create<RegisteredFoldersState>()(
 
       getRecentFolders: (limit = 5) => {
         return [...get().folders]
+          .filter((f) => f.exists !== false)
           .sort(
             (a, b) =>
               new Date(b.lastAccessed).getTime() -