WIP

apps/client/src/features/page/components/page-import-modal.tsx

@@ -31,6 +31,8 @@ import { getFileImportSizeLimit, isCloud } from "@/lib/config.ts";
 import { formatBytes } from "@/lib";
 import { workspaceAtom } from "@/features/user/atoms/current-user-atom.ts";
 import { getFileTaskById } from "@/features/file-task/services/file-task-service.ts";
+import { queryClient } from "@/main.tsx";
+import { useQueryEmit } from "@/features/websocket/use-query-emit.ts";
 
 interface PageImportModalProps {
   spaceId: string;
@@ -79,6 +81,7 @@ function ImportFormatSelection({ spaceId, onClose }: ImportFormatSelection) {
   const [treeData, setTreeData] = useAtom(treeDataAtom);
   const [workspace] = useAtom(workspaceAtom);
   const [fileTaskId, setFileTaskId] = useState<string | null>(null);
+  const emit = useQueryEmit();
 
   const canUseConfluence = isCloud() || workspace?.hasLicenseKey;
 
@@ -94,16 +97,13 @@ function ImportFormatSelection({ spaceId, onClose }: ImportFormatSelection) {
       notifications.show({
         id: "import",
         title: t("Importing pages"),
-        message: t(
+        message: t("Page import is in progress."),
-          "Page import is in progress. Refresh this tab after a while.",
-        ),
         loading: true,
         withCloseButton: false,
         autoClose: false,
       });
 
       setFileTaskId(importTask.id);
-      console.log("taskId set", importTask.id);
     } catch (err) {
       console.log("Failed to import page", err);
       notifications.update({
@@ -140,6 +140,17 @@ function ImportFormatSelection({ spaceId, onClose }: ImportFormatSelection) {
           });
           clearInterval(intervalId);
           setFileTaskId(null);
+
+          await queryClient.refetchQueries({
+            queryKey: ["root-sidebar-pages", fileTask.spaceId],
+          });
+
+          setTimeout(() => {
+            emit({
+              operation: "refetchRootTreeNodeEvent",
+              spaceId: spaceId,
+            });
+          }, 50);
         }
 
         if (status === "failed") {

apps/client/src/features/page/tree/components/space-tree.tsx

@@ -24,7 +24,10 @@ import {
   IconPointFilled,
   IconTrash,
 } from "@tabler/icons-react";
-import { appendNodeChildrenAtom, treeDataAtom } from "@/features/page/tree/atoms/tree-data-atom.ts";
+import {
+  appendNodeChildrenAtom,
+  treeDataAtom,
+} from "@/features/page/tree/atoms/tree-data-atom.ts";
 import clsx from "clsx";
 import EmojiPicker from "@/components/ui/emoji-picker.tsx";
 import { useTreeMutation } from "@/features/page/tree/hooks/use-tree-mutation.ts";
@@ -32,6 +35,7 @@ import {
   appendNodeChildren,
   buildTree,
   buildTreeWithChildren,
+  mergeRootTrees,
   updateTreeNodeIcon,
 } from "@/features/page/tree/utils/utils.ts";
 import { SpaceTreeNode } from "@/features/page/tree/types.ts";
@@ -104,17 +108,17 @@ export default function SpaceTree({ spaceId, readOnly }: SpaceTreeProps) {
       const allItems = pagesData.pages.flatMap((page) => page.items);
       const treeData = buildTree(allItems);
 
-      if (data.length < 1 || data?.[0].spaceId !== spaceId) {
+      setData((prev) => {
-        //Thoughts
+        // fresh space; full reset
-        // don't reset if there is data in state
+        if (prev.length === 0 || prev[0]?.spaceId !== spaceId) {
-        // we only expect to call this once on initial load
+          setIsDataLoaded(true);
-        // even if we decide to refetch, it should only update
+          setOpenTreeNodes({});
-        // and append root pages instead of resetting the entire tree
+          return treeData;
-        // which looses async loaded children too
+        }
-        setData(treeData);
+
-        setIsDataLoaded(true);
+        // same space; append only missing roots
-        setOpenTreeNodes({});
+        return mergeRootTrees(prev, treeData);
-      }
+      });
     }
   }, [pagesData, hasNextPage]);
 
@@ -297,17 +301,19 @@ function Node({ node, style, dragHandle, tree }: NodeRendererProps<any>) {
 
   const handleEmojiSelect = (emoji: { native: string }) => {
     handleUpdateNodeIcon(node.id, emoji.native);
-    updatePageMutation.mutateAsync({ pageId: node.id, icon: emoji.native }).then((data) => {
+    updatePageMutation
-      setTimeout(() => {
+      .mutateAsync({ pageId: node.id, icon: emoji.native })
-        emit({
+      .then((data) => {
-          operation: "updateOne",
+        setTimeout(() => {
-          spaceId: node.data.spaceId,
+          emit({
-          entity: ["pages"],
+            operation: "updateOne",
-          id: node.id,
+            spaceId: node.data.spaceId,
-          payload: { icon: emoji.native, parentPageId: data.parentPageId},
+            entity: ["pages"],
-        });
+            id: node.id,
-      }, 50);
+            payload: { icon: emoji.native, parentPageId: data.parentPageId },
-    });
+          });
+        }, 50);
+      });
   };
 
   const handleRemoveEmoji = () => {
@@ -570,7 +576,7 @@ interface PageArrowProps {
 
 function PageArrow({ node, onExpandTree }: PageArrowProps) {
   useEffect(() => {
-    if(node.isOpen){
+    if (node.isOpen) {
       onExpandTree();
     }
   }, []);

apps/client/src/features/page/tree/utils/utils.ts

@@ -121,7 +121,6 @@ export const deleteTreeNode = (
     .filter((node) => node !== null);
 };
 
-
 export function buildTreeWithChildren(items: SpaceTreeNode[]): SpaceTreeNode[] {
   const nodeMap = {};
   let result: SpaceTreeNode[] = [];
@@ -167,10 +166,12 @@ export function appendNodeChildren(
   // Preserve deeper children if they exist and remove node if deleted
   return treeItems.map((node) => {
     if (node.id === nodeId) {
-      const newIds = new Set(children.map(c => c.id));
+      const newIds = new Set(children.map((c) => c.id));
 
       const existingMap = new Map(
-        (node.children ?? []).filter(c => newIds.has(c.id)).map(c => [c.id, c])
+        (node.children ?? [])
+          .filter((c) => newIds.has(c.id))
+          .map((c) => [c.id, c]),
       );
 
       const merged = children.map((newChild) => {
@@ -196,3 +197,21 @@ export function appendNodeChildren(
     return node;
   });
 }
+
+/**
+ * Merge root nodes; keep existing ones intact, append new ones,
+ */
+export function mergeRootTrees(
+  prevRoots: SpaceTreeNode[],
+  incomingRoots: SpaceTreeNode[],
+): SpaceTreeNode[] {
+  const seen = new Set(prevRoots.map((r) => r.id));
+
+  // add new roots that were not present before
+  const merged = [...prevRoots];
+  incomingRoots.forEach((node) => {
+    if (!seen.has(node.id)) merged.push(node);
+  });
+
+  return sortPositionKeys(merged);
+}

apps/client/src/features/websocket/types/types.ts

@@ -47,15 +47,28 @@ export type MoveTreeNodeEvent = {
     parentId: string;
     index: number;
     position: string;
-  }
+  };
 };
 
 export type DeleteTreeNodeEvent = {
   operation: "deleteTreeNode";
   spaceId: string;
   payload: {
-    node: SpaceTreeNode
+    node: SpaceTreeNode;
-  }
+  };
 };
 
-export type WebSocketEvent = InvalidateEvent | InvalidateCommentsEvent | UpdateEvent | DeleteEvent | AddTreeNodeEvent | MoveTreeNodeEvent | DeleteTreeNodeEvent;
+export type RefetchRootTreeNodeEvent = {
+  operation: "refetchRootTreeNodeEvent";
+  spaceId: string;
+};
+
+export type WebSocketEvent =
+  | InvalidateEvent
+  | InvalidateCommentsEvent
+  | UpdateEvent
+  | DeleteEvent
+  | AddTreeNodeEvent
+  | MoveTreeNodeEvent
+  | DeleteTreeNodeEvent
+  | RefetchRootTreeNodeEvent;

apps/client/src/features/websocket/use-query-subscription.ts

@@ -12,6 +12,7 @@ import {
   invalidateOnUpdatePage,
 } from "../page/queries/page-query";
 import { RQ_KEY } from "../comment/queries/comment-query";
+import { queryClient } from "@/main.tsx";
 
 export const useQuerySubscription = () => {
   const queryClient = useQueryClient();
@@ -84,6 +85,17 @@ export const useQuerySubscription = () => {
           );
       */
           break;
+        case "refetchRootTreeNodeEvent": {
+          const spaceId = data.spaceId;
+          queryClient.refetchQueries({
+            queryKey: ["root-sidebar-pages", spaceId],
+          });
+
+          queryClient.invalidateQueries({
+            queryKey: ["recent-changes", spaceId],
+          });
+          break;
+        }
       }
     });
   }, [queryClient, socket]);

apps/server/src/integrations/import/dto/file-task-dto.ts

@@ -5,3 +5,14 @@ export class FileTaskIdDto {
   @IsUUID()
   fileTaskId: string;
 }
+
+export type ImportPageNode = {
+  id: string;
+  slugId: string;
+  name: string;
+  content: string;
+  position?: string | null;
+  parentPageId: string | null;
+  fileExtension: string;
+  filePath: string;
+};

apps/server/src/integrations/import/import.module.ts

@@ -6,6 +6,7 @@ import { FileTaskService } from './services/file-task.service';
 import { FileTaskProcessor } from './processors/file-task.processor';
 import { ImportAttachmentService } from './services/import-attachment.service';
 import { FileTaskController } from './file-task.controller';
+import { PageModule } from '../../core/page/page.module';
 
 @Module({
   providers: [
@@ -16,6 +17,6 @@ import { FileTaskController } from './file-task.controller';
   ],
   exports: [ImportService, ImportAttachmentService],
   controllers: [ImportController, FileTaskController],
-  imports: [StorageModule],
+  imports: [StorageModule, PageModule],
 })
 export class ImportModule {}

apps/server/src/integrations/import/processors/file-task.processor.ts

@@ -44,35 +44,23 @@ export class FileTaskProcessor extends WorkerHost implements OnModuleDestroy {
       `Error processing ${job.name} job. Reason: ${job.failedReason}`,
     );
 
-    const MAX_JOB_ATTEMPTS = 3;
+    try {
-    const fileTaskId = job.data.fileTaskId;
+      const fileTaskId = job.data.fileTaskId;
-
-    if (job.attemptsMade >= MAX_JOB_ATTEMPTS) {
-      this.logger.error(`Max import attempts reached for Task ${fileTaskId}.`);
       await this.fileTaskService.updateTaskStatus(
         fileTaskId,
         FileTaskStatus.Failed,
         job.failedReason,
       );
 
-      try {
+      const fileTask = await this.fileTaskService.getFileTask(fileTaskId);
-        const fileTask = await this.fileTaskService.getFileTask(fileTaskId);
+      if (fileTask) {
-        if (fileTask) {
+        await this.storageService.delete(fileTask.filePath);
-          await this.storageService.delete(fileTask.filePath);
-        }
-      } catch (err) {
-        this.logger.error(err);
       }
+    } catch (err) {
+      this.logger.error(err);
     }
   }
 
-  @OnWorkerEvent('stalled')
-  async onStalled(job: Job) {
-    this.logger.error(
-      `Stalled processing ${job.name} job. Reason: ${job.failedReason}`,
-    );
-  }
-
   @OnWorkerEvent('completed')
   onCompleted(job: Job) {
     this.logger.log(

apps/server/src/integrations/import/services/file-task.service.ts

@@ -29,6 +29,8 @@ import { executeTx } from '@docmost/db/utils';
 import { BacklinkRepo } from '@docmost/db/repos/backlink/backlink.repo';
 import { ImportAttachmentService } from './import-attachment.service';
 import { ModuleRef } from '@nestjs/core';
+import { PageService } from '../../../core/page/services/page.service';
+import { ImportPageNode } from '../dto/file-task-dto';
 
 @Injectable()
 export class FileTaskService {
@@ -37,6 +39,7 @@ export class FileTaskService {
   constructor(
     private readonly storageService: StorageService,
     private readonly importService: ImportService,
+    private readonly pageService: PageService,
     private readonly backlinkRepo: BacklinkRepo,
     @InjectKysely() private readonly db: KyselyDB,
     private readonly importAttachmentService: ImportAttachmentService,
@@ -55,6 +58,10 @@ export class FileTaskService {
       return;
     }
 
+    if (fileTask.status === FileTaskStatus.Failed) {
+      return;
+    }
+
     if (fileTask.status === FileTaskStatus.Success) {
       this.logger.log('Imported task already processed.');
       return;
@@ -118,6 +125,8 @@ export class FileTaskService {
       }
       try {
         await this.updateTaskStatus(fileTaskId, FileTaskStatus.Success, null);
+        await cleanupTmpFile();
+        await cleanupTmpDir();
         // delete stored file on success
         await this.storageService.delete(fileTask.filePath);
       } catch (err) {
@@ -142,19 +151,7 @@ export class FileTaskService {
     const allFiles = await collectMarkdownAndHtmlFiles(extractDir);
     const attachmentCandidates = await buildAttachmentCandidates(extractDir);
 
-    const pagesMap = new Map<
+    const pagesMap = new Map<string, ImportPageNode>();
-      string,
-      {
-        id: string;
-        slugId: string;
-        name: string;
-        content: string;
-        position?: string | null;
-        parentPageId: string | null;
-        fileExtension: string;
-        filePath: string;
-      }
-    >();
 
     for (const absPath of allFiles) {
       const relPath = path
@@ -201,14 +198,42 @@ export class FileTaskService {
     });
 
     // generate position keys
-    const siblingsMap = new Map<string | null, typeof Array.prototype>();
+    const siblingsMap = new Map<string | null, ImportPageNode[]>();
+
     pagesMap.forEach((page) => {
-      const sibs = siblingsMap.get(page.parentPageId) || [];
+      const group = siblingsMap.get(page.parentPageId) ?? [];
-      sibs.push(page);
+      group.push(page);
-      siblingsMap.set(page.parentPageId, sibs);
+      siblingsMap.set(page.parentPageId, group);
     });
-    siblingsMap.forEach((sibs) => {
+
+    // get root pages
+    const rootSibs = siblingsMap.get(null);
+
+    if (rootSibs?.length) {
+      rootSibs.sort((a, b) => a.name.localeCompare(b.name));
+
+      // get first position key from the server
+      const nextPosition = await this.pageService.nextPagePosition(
+        fileTask.spaceId,
+      );
+
+      let prevPos: string | null = null;
+      rootSibs.forEach((page, idx) => {
+        if (idx === 0) {
+          page.position = nextPosition;
+        } else {
+          page.position = generateJitteredKeyBetween(prevPos, null);
+        }
+        prevPos = page.position;
+      });
+    }
+
+    // non-root buckets (children & deeper levels)
+    siblingsMap.forEach((sibs, parentId) => {
+      if (parentId === null) return; // root already done
+
       sibs.sort((a, b) => a.name.localeCompare(b.name));
+
       let prevPos: string | null = null;
       for (const page of sibs) {
         page.position = generateJitteredKeyBetween(prevPos, null);
@@ -216,6 +241,7 @@ export class FileTaskService {
       }
     });
 
+    // internal page links
     const filePathToPageMetaMap = new Map<
       string,
       { id: string; title: string; slugId: string }

apps/server/src/integrations/import/services/import-attachment.service.ts

@@ -260,7 +260,11 @@ export class ImportAttachmentService {
     }
 
     // wait for all uploads & DB inserts
-    await Promise.all(attachmentTasks);
+    try {
+      await Promise.all(attachmentTasks);
+    } catch (err) {
+      this.logger.log('Import attachment upload error', err);
+    }
 
     return $.root().html() || '';
   }

apps/server/src/integrations/queue/queue.module.ts

@@ -54,6 +54,7 @@ import { BacklinksProcessor } from './processors/backlinks.processor';
       defaultJobOptions: {
         removeOnComplete: true,
         removeOnFail: true,
+        attempts: 1,
       },
     }),
   ],