eng-1135 define internal-error

Author: maparent

apps/roam/src/components/CreateRelationDialog.tsx

@@ -14,7 +14,6 @@ import AutocompleteInput from "roamjs-components/components/AutocompleteInput";
 import getPageTitleByPageUid from "roamjs-components/queries/getPageTitleByPageUid";
 import getPageUidByPageTitle from "roamjs-components/queries/getPageUidByPageTitle";
 import getAllPageNames from "roamjs-components/queries/getAllPageNames";
-import sendErrorEmail from "~/utils/sendErrorEmail";
 import { getSetting } from "~/utils/extensionSettings";
 import getDiscourseRelations, {
   type DiscourseRelation,
@@ -23,7 +22,7 @@ import { createReifiedRelation } from "~/utils/createReifiedBlock";
 import { findDiscourseNodeByTitleAndUid } from "~/utils/findDiscourseNode";
 import { getDiscourseNodeFormatInnerExpression } from "~/utils/getDiscourseNodeFormatExpression";
 import type { DiscourseNode } from "~/utils/getDiscourseNodes";
-import type { Result } from "~/utils/types";
+import internalError from "~/utils/internalError";
 import getDiscourseNodes from "~/utils/getDiscourseNodes";
 
 export type CreateRelationDialogProps = {
@@ -41,15 +40,6 @@ type ExtendedCreateRelationDialogProps = CreateRelationDialogProps & {
   selectedSourceType: DiscourseNode;
 };
 
-const internalError = (msg: string) => {
-  process.env.NODE_ENV === "development"
-    ? console.error(msg)
-    : void sendErrorEmail({
-        error: new Error(msg),
-        type: "Create Relation Dialog Failed",
-      }).catch(() => {});
-};
-
 const CreateRelationDialog = ({
   onClose,
   sourceNodeUid,
@@ -102,7 +92,11 @@ const CreateRelationDialog = ({
     });
     if (selectedTargetType === false) {
       // should not happen at this point, since the pattern was vetted at input.
-      internalError("Could not find identify node downstream");
+      internalError({
+        type: "create-relation-error",
+        error:
+          "Create Relation dialog: Could not find identify node downstream",
+      });
       return null;
     }
     const candidateRelations = relDataByTag[selectedRelationName].filter(
@@ -122,14 +116,18 @@ const CreateRelationDialog = ({
     );
     if (candidateRelations.length === 0) {
       // also should not happen
-      internalError("Could not find the relation");
+      internalError({
+        type: "create-relation-error",
+        error: "Create Relation dialog: Could not find the relation",
+      });
       return null;
     }
     if (candidateRelations.length !== 1) {
       // This seems to happen... I need more data.
-      internalError(
-        `Too many relations between ${selectedTargetType.type} and ${selectedSourceType.type}: ${candidateRelations.map((r) => r.id).join(",")}`,
-      );
+      internalError({
+        type: "create-relation-error",
+        error: `Create Relation dialog: Too many relations between ${selectedTargetType.type} and ${selectedSourceType.type}: ${candidateRelations.map((r) => r.id).join(",")}`,
+      });
       return null;
     }
     return candidateRelations[0];
@@ -276,7 +274,10 @@ const prepareRelData = (
   });
   if (!nodeSchema) {
     // should not happen at this point, since the pattern was vetted at input.
-    internalError("Could not find identify node downstream");
+    internalError({
+      error: "Create Relation dialog: Could not find identify node downstream",
+      type: "create-relation-error",
+    });
     return [];
   }
   // note the same relation could be used in both directions

apps/roam/src/components/Export.tsx

@@ -54,7 +54,7 @@ import {
 import calcCanvasNodeSizeAndImg from "~/utils/calcCanvasNodeSizeAndImg";
 import { DiscourseNodeShape } from "~/components/canvas/DiscourseNodeUtil";
 import { MAX_WIDTH } from "~/components/canvas/Tldraw";
-import sendErrorEmail from "~/utils/sendErrorEmail";
+import internalError from "~/utils/internalError";
 import { getSetting, setSetting } from "~/utils/extensionSettings";
 
 const ExportProgress = ({ id }: { id: string }) => {
@@ -222,7 +222,7 @@ const ExportDialog: ExportDialogComponent = ({
   const addToSelectedCanvas = async (pageUid: string) => {
     if (typeof results !== "object") return;
 
-    let props: Record<string, unknown> = getBlockProps(pageUid);
+    const props: Record<string, unknown> = getBlockProps(pageUid);
 
     const PADDING_BETWEEN_SHAPES = 20;
     const COMMON_BOUNDS_XOFFSET = 250;
@@ -309,10 +309,10 @@ const ExportDialog: ExportDialogComponent = ({
       let minY = Number.MAX_SAFE_INTEGER;
 
       shapes.forEach((shape) => {
-        let rightX = shape.x + shape.w;
-        let leftX = shape.x;
-        let topY = shape.y;
-        let bottomY = shape.y - shape.h;
+        const rightX = shape.x + shape.w;
+        const leftX = shape.x;
+        const topY = shape.y;
+        const bottomY = shape.y - shape.h;
 
         if (rightX > maxX) maxX = rightX;
         if (leftX < minX) minX = leftX;
@@ -468,13 +468,12 @@ const ExportDialog: ExportDialogComponent = ({
         id: "query-builder-export-success",
       });
     } catch (e) {
-      const error = e as Error;
-      renderToast({
-        content: "Looks like there was an error. The team has been notified.",
-        intent: "danger",
-        id: "discourse-graphs-error",
+      internalError({
+        error: e as Error,
+        type: "export-error",
+        userMessage:
+          "Looks like there was an error. The team has been notified.",
       });
-      sendErrorEmail({ error, type: "Export Dialog Failed" }).catch(() => {});
     } finally {
       setLoading(false);
       onClose();
@@ -688,18 +687,13 @@ const ExportDialog: ExportDialogComponent = ({
                     setError(`Unsupported export type: ${exportType}`);
                   }
                 } catch (e) {
-                  const error = e as Error;
-                  renderToast({
-                    id: "export-error",
-                    content:
+                  internalError({
+                    error: e as Error,
+                    type: "export-error",
+                    userMessage:
                       "Looks like there was an error. The team has been notified.",
-                    intent: "danger",
-                  });
-                  sendErrorEmail({
-                    error,
-                    type: "Export Dialog Failed",
                     context: { activeExportType, filename, results },
-                  }).catch(() => {});
+                  });
                   setDialogOpen(true);
                   setError((e as Error).message);
                 } finally {

apps/roam/src/components/canvas/Clipboard.tsx

@@ -57,7 +57,7 @@ import { MAX_WIDTH } from "./Tldraw";
 import getBlockProps from "~/utils/getBlockProps";
 import setBlockProps from "~/utils/setBlockProps";
 import { measureCanvasNodeText } from "~/utils/measureCanvasNodeText";
-import sendErrorEmail from "~/utils/sendErrorEmail";
+import internalError from "~/utils/internalError";
 
 export type ClipboardPage = {
   uid: string;
@@ -79,21 +79,6 @@ const ClipboardContext = createContext<ClipboardContextValue | null>(null);
 
 const CLIPBOARD_PROP_KEY = "pages";
 
-const toError = (error: unknown, fallbackMessage: string): Error => {
-  if (error instanceof Error) {
-    return error;
-  }
-  if (typeof error === "string") {
-    return new Error(error);
-  }
-  try {
-    const serialized = JSON.stringify(error);
-    return new Error(serialized || fallbackMessage);
-  } catch {
-    return new Error(fallbackMessage);
-  }
-};
-
 const getOrCreateClipboardBlock = async (
   canvasPageTitle: string,
   userUid: string,
@@ -155,11 +140,13 @@ export const ClipboardProvider = ({
       try {
         const userUid = getCurrentUserUid();
         if (!userUid) {
-          sendErrorEmail({
-            error: new Error("Missing current user UID"),
-            type: "Canvas Clipboard: Missing current user UID",
-            context: { canvasPageTitle },
-          }).catch(() => {});
+          internalError({
+            error: new Error("Canvas Clipboard: Missing current user UID"),
+            type: "canvas-clipboard-missing-uid",
+            context: {
+              canvasPageTitle,
+            },
+          });
           setIsInitialized(true);
           return;
         }
@@ -179,13 +166,12 @@ export const ClipboardProvider = ({
         ) {
           setPages(storedPages as ClipboardPage[]);
         }
-      } catch (e) {
-        const normalizedError = toError(e, "Failed to initialize clipboard");
-        sendErrorEmail({
-          error: normalizedError,
-          type: "Canvas Clipboard: Failed to initialize",
+      } catch (error) {
+        internalError({
+          error,
+          type: "canvas-clipboard-initialization-error",
           context: { canvasPageTitle },
-        }).catch(() => {});
+        });
       } finally {
         setIsInitialized(true);
       }
@@ -201,13 +187,12 @@ export const ClipboardProvider = ({
       setBlockProps(clipboardBlockUid, {
         [CLIPBOARD_PROP_KEY]: pages,
       });
-    } catch (e) {
-      const normalizedError = toError(e, "Failed to persist clipboard state");
-      sendErrorEmail({
-        error: normalizedError,
-        type: "Canvas Clipboard: Failed to persist state",
+    } catch (error) {
+      internalError({
+        error,
+        type: "canvas-clipboard-state-save-error",
         context: { clipboardBlockUid, pageCount: pages.length },
-      }).catch(() => {});
+      });
     }
   }, [pages, clipboardBlockUid, isInitialized]);
 
@@ -287,8 +272,8 @@ const AddPageModal = ({ isOpen, onClose, onConfirm }: AddPageModalProps) => {
         // eslint-disable-next-line @typescript-eslint/await-thenable
         const raw = await window.roamAlphaAPI.data.backend.q(
           `
-            [:find ?text ?uid 
-            :where 
+            [:find ?text ?uid
+            :where
             [?e :node/title ?text]
             [?e :block/uid ?uid]]`,
         );
@@ -447,15 +432,11 @@ const ClipboardPageSection = ({
         );
         setDiscourseNodes(nodes);
       } catch (error) {
-        const normalizedError = toError(
+        internalError({
           error,
-          "Failed to fetch discourse nodes",
-        );
-        sendErrorEmail({
-          error: normalizedError,
-          type: "Canvas Clipboard: Failed to fetch discourse nodes",
+          type: "canvas-clipboard-discourse-node-error",
           context: { pageTitle: page.text },
-        }).catch(() => {});
+        });
         setDiscourseNodes([]);
       } finally {
         setIsLoading(false);
@@ -610,11 +591,11 @@ const ClipboardPageSection = ({
 
       const nodeType = findDiscourseNode(node.uid);
       if (!nodeType) {
-        sendErrorEmail({
-          error: new Error("Node type not found"),
-          type: "Canvas Clipboard: Node type not found",
+        internalError({
+          error: new Error("Canvas Clipboard: Node type not found"),
+          type: "canvas-clipboard-type-not-found",
           context: { uid: node.uid },
-        }).catch(() => {});
+        });
         return;
       }
 

apps/roam/src/components/canvas/Tldraw.tsx

@@ -86,13 +86,12 @@ import { createMigrations } from "./DiscourseRelationShape/discourseRelationMigr
 import ToastListener, { dispatchToastEvent } from "./ToastListener";
 import { CanvasDrawerPanel } from "./CanvasDrawer";
 import { ClipboardPanel, ClipboardProvider } from "./Clipboard";
-import sendErrorEmail from "~/utils/sendErrorEmail";
+import internalError from "~/utils/internalError";
 import { AUTO_CANVAS_RELATIONS_KEY } from "~/data/userSettings";
 import { getSetting } from "~/utils/extensionSettings";
 import { isPluginTimerReady, waitForPluginTimer } from "~/utils/pluginTimer";
 import { HistoryEntry } from "@tldraw/store";
 import { TLRecord } from "@tldraw/tlschema";
-import getCurrentUserDisplayName from "roamjs-components/queries/getCurrentUserDisplayName";
 
 declare global {
   interface Window {
@@ -509,16 +508,14 @@ const TldrawCanvas = ({ title }: { title: string }) => {
         error.stack = e.detail.stack;
       }
 
-      sendErrorEmail({
+      internalError({
         error,
-        type: "Tldraw Error",
+        type: "tldraw-error",
         context: {
           title: title,
           lastActions: lastActionsRef.current,
         },
-      }).catch(() => {});
-
-      console.error("Tldraw Error:", e.detail);
+      });
     };
 
     document.addEventListener(

apps/roam/src/components/canvas/useRoamStore.ts

@@ -25,8 +25,7 @@ import {
 } from "tldraw";
 import { AddPullWatch } from "roamjs-components/types";
 import { LEGACY_SCHEMA } from "~/data/legacyTldrawSchema";
-import sendErrorEmail from "~/utils/sendErrorEmail";
-import getCurrentUserDisplayName from "roamjs-components/queries/getCurrentUserDisplayName";
+import internalError from "~/utils/internalError";
 
 const THROTTLE = 350;
 
@@ -142,21 +141,20 @@ export const useRoamStore = ({
       error: Error;
       errorMessage: string;
     }): void => {
-      console.error(errorMessage, error);
       setError(error);
       setLoading(false);
       const snapshotSize = initialSnapshot
         ? JSON.stringify(initialSnapshot).length
         : 0;
-      sendErrorEmail({
+      internalError({
         error,
-        type: errorMessage,
+        type: "roam-store-error",
         context: {
           pageUid,
           snapshotSize,
           ...(snapshotSize < 10000 ? { initialSnapshot } : {}),
         },
-      }).catch(() => {});
+      });
     };
 
     // eslint-disable-next-line @typescript-eslint/naming-convention
@@ -305,14 +303,13 @@ export const useRoamStore = ({
       setNeedsUpgrade(false);
       setInitialSnapshot(null);
       setError(error);
-      sendErrorEmail({
+      internalError({
         error,
-        type: "Failed to perform Canvas upgrade",
+        type: "canvas-upgrade-error",
         context: {
           data: { oldData },
         },
-      }).catch(() => {});
-      console.error("Failed to perform Canvas upgrade", error);
+      });
     }
   };