feat(notion): refactor space handling and improve error reporting

This commit refactors how Notion spaces are handled by:
1. Moving space retrieval to a dedicated `getSpaces` method
2. Adding a `getSpaceId` helper method
3. Using the new methods consistently across the service

Additionally, it improves error reporting in the frontend by showing error stacks in error messages. This helps with debugging when issues occur.

The changes make the code more maintainable by centralizing space-related logic and provide better visibility into errors when they happen.

Author: DiamondYuan

src/common/backend/services/notion/service.ts

@@ -76,14 +76,8 @@ export default class NotionDocumentService implements DocumentService {
       this.userContent = await this.getUserContent();
     }
 
-    if (!this.userContent.recordMap.space) {
-      return [];
-    }
-
-    const spaces = this.userContent.recordMap.space;
-
     const userId = Object.keys(this.userContent.recordMap.notion_user)[0] as string;
-
+    const spaces = (await this.getSpaces(userId)) as any;
     const result: Array<NotionRepository[]> = await Promise.all(
       Object.keys(spaces).map(async (p) => {
         const space = spaces[p];
@@ -96,6 +90,17 @@ export default class NotionDocumentService implements DocumentService {
     return this.repositories;
   };
 
+  getSpaces = async (userId: string) => {
+    const response = await this.requestWithCookie.post<{
+      users: {
+        [id: string]: {
+          space: any;
+        };
+      };
+    }>('/api/v3/getSpacesInitial');
+    return response.data.users[userId].space;
+  };
+
   createDocument = async ({
     repositoryId,
     title,
@@ -118,7 +123,7 @@ export default class NotionDocumentService implements DocumentService {
     if (!this.userContent) {
       this.userContent = await this.getUserContent();
     }
-    const spaceId = Object.values(this.userContent.recordMap.space)[0].value.id;
+    const spaceId = await this.getSpaceId();
     await this.requestWithCookie.post('api/v3/enqueueTask', {
       task: {
         eventName: 'importFile',
@@ -140,11 +145,21 @@ export default class NotionDocumentService implements DocumentService {
     };
   };
 
+  getSpaceId = async () => {
+    if (!this.userContent) {
+      this.userContent = await this.getUserContent();
+    }
+
+    const userId = Object.keys(this.userContent.recordMap.notion_user)[0] as string;
+    const spaces = (await this.getSpaces(userId)) as any;
+    return Object.keys(spaces)[0];
+  };
+
   createEmptyFile = async (repository: NotionRepository, title: string) => {
     if (!this.userContent) {
       this.userContent = await this.getUserContent();
     }
-    const spaceId = Object.values(this.userContent.recordMap.space)[0].value.id;
+    const spaceId = await this.getSpaceId();
     const documentId = generateUuid();
     const parentId = repository.id;
     const userId = Object.values(this.userContent.recordMap.notion_user)[0].value.id;

src/pages/app.tsx

@@ -49,15 +49,16 @@ export default async () => {
   const app = dva({
     namespacePrefixWarning: false,
     history: createHashHistory(),
-    onError: e => {
+    onError: (e) => {
       (e as any).preventDefault();
       message.destroy();
       message.error(e.message);
+      message.error(e.stack);
     },
   });
   app.use(createLoading());
 
-  app.router(router => {
+  app.router((router) => {
     return (
       <LocalWrapper>
         <Router history={router!.history}>