Merge pull request #230 from beNative/codex/ensure-selected-node-is-visible-after-search-clear

Ensure document tree selection remains visible after clearing search

Author: beNative

App.tsx

@@ -218,6 +218,7 @@ export const MainApp: React.FC = () => {
     const [confirmAction, setConfirmAction] = useState<{ title: string; message: React.ReactNode; onConfirm: () => void; } | null>(null);
     const [clipboardNotice, setClipboardNotice] = useState<{ title: string; message: React.ReactNode; helpUrl?: string } | null>(null);
     const [searchTerm, setSearchTerm] = useState('');
+    const previousSearchTermRef = useRef('');
     const [contextMenu, setContextMenu] = useState<{ isOpen: boolean; position: { x: number, y: number }, items: MenuItem[] }>({ isOpen: false, position: { x: 0, y: 0 }, items: [] });
     const [isDraggingFile, setIsDraggingFile] = useState(false);
     const [formatTrigger, setFormatTrigger] = useState(0);
@@ -765,6 +766,23 @@ export const MainApp: React.FC = () => {
         return iterator.done ? null : iterator.value;
     }, [lastClickedId, selectedIds]);
 
+    useEffect(() => {
+        const previousTrimmed = previousSearchTermRef.current.trim();
+        const currentTrimmed = searchTerm.trim();
+
+        if (previousTrimmed && !currentTrimmed) {
+            const primaryId = getPrimarySelectionId();
+            if (primaryId) {
+                const node = items.find(item => item.id === primaryId);
+                if (node) {
+                    ensureNodeVisible(node);
+                }
+            }
+        }
+
+        previousSearchTermRef.current = searchTerm;
+    }, [searchTerm, getPrimarySelectionId, items, ensureNodeVisible]);
+
     const handleDocumentTreeSelectAll = useCallback(() => {
         setSelectedIds(new Set(navigableItems.map(item => item.id)));
     }, [navigableItems, setSelectedIds]);