Merge pull request #255 from beNative/fix/focus-loss-title-edit-v2

Fix focus loss on title edit by implementing silent refresh

Author: beNative

hooks/useNodes.ts

@@ -8,16 +8,16 @@ export const useNodes = () => {
   const [nodes, setNodes] = useState<Node[]>([]);
   const [isLoading, setIsLoading] = useState(true);
 
-  const refreshNodes = useCallback(async () => {
-    setIsLoading(true);
+  const refreshNodes = useCallback(async (silent: boolean = false) => {
+    if (!silent) setIsLoading(true);
     try {
       const fetchedNodes = await repository.getNodeTree();
       setNodes(fetchedNodes);
       addLog('DEBUG', 'Node tree refreshed from database.');
     } catch (error) {
       addLog('ERROR', `Failed to refresh nodes: ${error instanceof Error ? error.message : String(error)}`);
     } finally {
-      setIsLoading(false);
+      if (!silent) setIsLoading(false);
     }
   }, [addLog]);
 
@@ -28,44 +28,44 @@ export const useNodes = () => {
   const addNode = useCallback(async (node: Omit<Node, 'node_id' | 'sort_order' | 'created_at' | 'updated_at'>): Promise<Node> => {
     const newNode = await repository.addNode(node);
     addLog('INFO', `New ${node.node_type} created with title: "${node.title}"`);
-    await refreshNodes(); // Refresh to get correct sorting and structure
+    await refreshNodes(true); // Refresh to get correct sorting and structure
     return newNode;
   }, [addLog, refreshNodes]);
 
   const updateNode = useCallback(async (nodeId: string, updates: Partial<Pick<Node, 'title' | 'parent_id'> & { language_hint?: string | null; default_view_mode?: ViewMode | null }>) => {
     await repository.updateNode(nodeId, updates);
     addLog('DEBUG', `Node updated with ID: ${nodeId}. Refreshing tree.`);
-    await refreshNodes();
+    await refreshNodes(true);
   }, [addLog, refreshNodes]);
 
   const deleteNode = useCallback(async (nodeId: string) => {
     await repository.deleteNode(nodeId);
     addLog('INFO', `Deleted node and its descendants starting from root ID: ${nodeId}`);
-    await refreshNodes();
+    await refreshNodes(true);
   }, [addLog, refreshNodes]);
 
   const deleteNodes = useCallback(async (nodeIds: string[]) => {
     if (nodeIds.length === 0) return;
     await repository.deleteNodes(nodeIds);
     addLog('INFO', `Deleted ${nodeIds.length} node(s) and their descendants.`);
-    await refreshNodes();
+    await refreshNodes(true);
   }, [addLog, refreshNodes]);
 
   const duplicateNodes = useCallback(async (nodeIds: string[]) => {
     await repository.duplicateNodes(nodeIds);
     addLog('INFO', `Duplicated ${nodeIds.length} item(s).`);
-    await refreshNodes();
+    await refreshNodes(true);
   }, [addLog, refreshNodes]);
 
   const moveNodes = useCallback(async (draggedIds: string[], targetId: string | null, position: 'before' | 'after' | 'inside') => {
     await repository.moveNodes(draggedIds, targetId, position);
     addLog('INFO', `Moved ${draggedIds.length} item(s).`);
-    await refreshNodes();
+    await refreshNodes(true);
   }, [addLog, refreshNodes]);
-  
+
   const updateDocumentContent = useCallback(async (nodeId: string, newContent: string): Promise<void> => {
-      await repository.updateDocumentContent(nodeId, newContent);
-      addLog('DEBUG', `Content for node ${nodeId} saved.`);
+    await repository.updateDocumentContent(nodeId, newContent);
+    addLog('DEBUG', `Content for node ${nodeId} saved.`);
   }, [addLog]);
 
   const importFiles = useCallback(
@@ -82,7 +82,7 @@ export const useNodes = () => {
     async (payload: DraggedNodeTransfer, targetId: string | null, position: 'before' | 'after' | 'inside') => {
       const createdIds = await repository.importNodesFromTransfer(payload, targetId, position);
       addLog('INFO', `Copied ${createdIds.length} item(s) from external workspace.`);
-      await refreshNodes();
+      await refreshNodes(true);
       return createdIds;
     },
     [addLog, refreshNodes]
@@ -91,7 +91,7 @@ export const useNodes = () => {
   const setNodeLock = useCallback(async (nodeId: string, locked: boolean) => {
     await repository.setNodeLock(nodeId, locked);
     addLog('DEBUG', `Set lock state for node ${nodeId} to ${locked ? 'locked' : 'unlocked'}.`);
-    await refreshNodes();
+    await refreshNodes(true);
   }, [addLog, refreshNodes]);
 
   const createDocumentFromClipboard = useCallback(async (
@@ -106,7 +106,7 @@ export const useNodes = () => {
     if (summary.warnings.length > 0) {
       summary.warnings.forEach(warning => addLog('WARNING', warning));
     }
-    await refreshNodes();
+    await refreshNodes(true);
     return result;
   }, [addLog, refreshNodes]);