[WIKI-704] fix: hocuspocus error handling

apps/api/plane/utils/content_validator.py

@@ -84,6 +84,7 @@ def validate_binary_data(data):
         "style",
         "start",
         "type",
+        "xmlns",
         # common editor data-* attributes seen in stored HTML
         # (wildcards like data-* are NOT supported by nh3; we add known keys
         # here and dynamically include all data-* seen in the input below)

apps/live/src/extensions/database.ts

@@ -11,6 +11,16 @@ import { getPageService } from "@/services/page/handler";
 // type
 import type { FetchPayloadWithContext, StorePayloadWithContext } from "@/types";
 
+const normalizeToError = (error: unknown, fallbackMessage: string) => {
+  if (error instanceof Error) {
+    return error;
+  }
+
+  const message = typeof error === "string" && error.trim().length > 0 ? error : fallbackMessage;
+
+  return new Error(message);
+};
+
 const fetchDocument = async ({ context, documentName: pageId }: FetchPayloadWithContext) => {
   try {
     const service = getPageService(context.documentType, context);
@@ -29,7 +39,7 @@ const fetchDocument = async ({ context, documentName: pageId }: FetchPayloadWith
     return binaryData;
   } catch (error) {
     logger.error("Error in fetching document", error);
-    throw error;
+    throw normalizeToError(error, `Failed to fetch document: ${pageId}`);
   }
 };
 
@@ -45,10 +55,10 @@ const storeDocument = async ({ context, state: pageBinaryData, documentName: pag
       description_html: contentHTML,
       description: contentJSON,
     };
-    return service.updateDescriptionBinary(pageId, payload);
+    await service.updateDescriptionBinary(pageId, payload);
   } catch (error) {
     logger.error("Error in updating document:", error);
-    throw error;
+    throw normalizeToError(error, `Failed to update document: ${pageId}`);
   }
 };
 

apps/live/src/services/page/core.service.ts

@@ -21,11 +21,11 @@
     })
       .then((response) => response?.data)
       .catch((error) => {
-        throw error?.response?.data;
+        throw error;
       });
   }
 
   async fetchDescriptionBinary(pageId: string): Promise<any> {
     return this.get(`${this.basePath}/pages/${pageId}/description/`, {
       headers: {
         ...this.getHeader(),
@@ -35,7 +35,7 @@
     })
       .then((response) => response?.data)
       .catch((error) => {
-        throw error?.response?.data;
+        throw error;
       });
   }
 
@@ -87,7 +87,7 @@
     }
   }
 
   async updateDescriptionBinary(pageId: string, data: TPageDescriptionPayload): Promise<any> {
     return this.patch(`${this.basePath}/pages/${pageId}/description/`, data, {
       headers: this.getHeader(),
     })