fix: disable comments module + sd update (#22)

Co-authored-by: Caio Pizzol <caiopizzol@icloud.com>

Author: caio-pizzol

demo/package.json

@@ -11,7 +11,7 @@
         "@superdoc-dev/template-builder": "link:../.",
         "react": "^19.2.0",
         "react-dom": "^19.2.0",
-        "superdoc": "^0.31.2"
+        "superdoc": "^0.35.3"
     },
     "devDependencies": {
         "@types/react": "^19.2.6",

demo/pnpm-lock.yaml

@@ -54,7 +54,7 @@
   "peerDependencies": {
     "react": "^16.8.0 || ^17.0.0 || ^18.0.0 || ^19.0.0",
     "react-dom": "^16.8.0 || ^17.0.0 || ^18.0.0 || ^19.0.0",
-    "superdoc": "^0.31.2"
+    "superdoc": "^0.35.3"
   },
   "devDependencies": {
     "@commitlint/cli": "^20.1.0",

package.json

@@ -367,101 +367,100 @@ const SuperDocTemplateBuilder = forwardRef<
     const initSuperDoc = async () => {
       const { SuperDoc } = await import('superdoc');
 
-      const config: Record<string, unknown> = {
-        selector: containerRef.current!,
-        document: document?.source,
-        documentMode: document?.mode || 'editing',
-        onReady: () => {
-          if (instance.activeEditor) {
-            const editor = instance.activeEditor;
-
-            editor.on('update', ({ editor: e }: any) => {
-              const { state } = e;
-              const { from } = state.selection;
-
-              if (from >= trigger.length) {
-                const triggerStart = from - trigger.length;
-                const text = state.doc.textBetween(triggerStart, from);
-
-                if (text === trigger) {
-                  const coords = e.view.coordsAtPos(from);
-                  const bounds = clampToViewport(new DOMRect(coords.left, coords.top, 0, 0));
-
-                  const cleanup = () => {
-                    const editor = superdocRef.current?.activeEditor;
-                    if (!editor) return;
-                    const currentPos = editor.state.selection.from;
-                    const tr = editor.state.tr.delete(triggerStart, currentPos);
-                    (editor as any).view.dispatch(tr);
-                  };
-
-                  triggerCleanupRef.current = cleanup;
-                  menuTriggerFromRef.current = from;
-                  setMenuPosition(bounds);
-                  setMenuVisible(true);
-                  resetMenuFilter();
-
-                  onTrigger?.({
-                    position: { from: triggerStart, to: from },
-                    bounds,
-                    cleanup,
-                  });
-
-                  return;
-                }
-              }
-
-              if (!menuVisibleRef.current) {
-                return;
-              }
+      const modules: Record<string, unknown> = {
+        comments: false,
+        ...(toolbarSettings && {
+          toolbar: {
+            selector: toolbarSettings.selector,
+            toolbarGroups: toolbarSettings.config.toolbarGroups || ['center'],
+            excludeItems: toolbarSettings.config.excludeItems || [],
+            ...toolbarSettings.config,
+          },
+        }),
+      };
 
-              if (menuTriggerFromRef.current == null) {
-                setMenuVisible(false);
+      const handleReady = () => {
+        if (instance.activeEditor) {
+          const editor = instance.activeEditor;
+
+          editor.on('update', ({ editor: e }: any) => {
+            const { state } = e;
+            const { from } = state.selection;
+
+            if (from >= trigger.length) {
+              const triggerStart = from - trigger.length;
+              const text = state.doc.textBetween(triggerStart, from);
+
+              if (text === trigger) {
+                const coords = e.view.coordsAtPos(from);
+                const bounds = clampToViewport(new DOMRect(coords.left, coords.top, 0, 0));
+
+                const cleanup = () => {
+                  const editor = superdocRef.current?.activeEditor;
+                  if (!editor) return;
+                  const currentPos = editor.state.selection.from;
+                  const tr = editor.state.tr.delete(triggerStart, currentPos);
+                  (editor as any).view.dispatch(tr);
+                };
+
+                triggerCleanupRef.current = cleanup;
+                menuTriggerFromRef.current = from;
+                setMenuPosition(bounds);
+                setMenuVisible(true);
                 resetMenuFilter();
-                return;
-              }
 
-              if (from < menuTriggerFromRef.current) {
-                setMenuVisible(false);
-                menuTriggerFromRef.current = null;
-                resetMenuFilter();
+                onTrigger?.({
+                  position: { from: triggerStart, to: from },
+                  bounds,
+                  cleanup,
+                });
+
                 return;
               }
+            }
+
+            if (!menuVisibleRef.current) {
+              return;
+            }
+
+            if (menuTriggerFromRef.current == null) {
+              setMenuVisible(false);
+              resetMenuFilter();
+              return;
+            }
+
+            if (from < menuTriggerFromRef.current) {
+              setMenuVisible(false);
+              menuTriggerFromRef.current = null;
+              resetMenuFilter();
+              return;
+            }
+
+            const queryText = state.doc.textBetween(menuTriggerFromRef.current, from);
+            updateMenuFilter(queryText);
+
+            const coords = e.view.coordsAtPos(from);
+            const bounds = clampToViewport(new DOMRect(coords.left, coords.top, 0, 0));
+            setMenuPosition(bounds);
+          });
 
-              const queryText = state.doc.textBetween(menuTriggerFromRef.current, from);
-              updateMenuFilter(queryText);
-
-              const coords = e.view.coordsAtPos(from);
-              const bounds = clampToViewport(new DOMRect(coords.left, coords.top, 0, 0));
-              setMenuPosition(bounds);
-            });
-
-            editor.on('update', () => {
-              discoverFields(editor);
-            });
-
+          editor.on('update', () => {
             discoverFields(editor);
-          }
+          });
+
+          discoverFields(editor);
+        }
 
-          onReady?.();
-        },
+        onReady?.();
       };
 
       const instance = new SuperDoc({
         selector: containerRef.current!,
+        document: document?.source,
         documentMode: document?.mode || 'editing',
-        ...config,
-        ...(toolbarSettings && {
-          toolbar: toolbarSettings.selector,
-          modules: {
-            toolbar: {
-              selector: toolbarSettings.selector,
-              toolbarGroups: toolbarSettings.config.toolbarGroups || ['center'],
-              excludeItems: toolbarSettings.config.excludeItems || [],
-              ...toolbarSettings.config,
-            },
-          },
-        }),
+        modules,
+        toolbar: toolbarSettings?.selector,
+        onReady: handleReady,
       });
 
       superdocRef.current = instance;