Fix spurious saves on note open with discrete editor update

The editor.update() in InitialContentPlugin was async (batched),
causing onInitComplete to fire before the import settled. The
OnChangeMarkdownPlugin baseline was recorded from the pre-import
state, so the first post-init update saw a different export and
triggered onChange, causing every note open to schedule a save.

Fix: use { discrete: true } so the import settles synchronously
before the baseline is recorded.

Author: ChristianChiarulli

src/components/editor/plugins/initial-content-plugin.tsx

@@ -26,39 +26,45 @@ export default function InitialContentPlugin({
     const mode = isNew ? "new" : !markdown.trim() ? "empty" : "existing";
     console.log(`[editor:init] mode=${mode} markdown=${markdown.length} chars`);
 
-    editor.update(() => {
-      if (isNew) {
-        if (markdown === "- [ ] ") {
-          // Todo mode: create an empty checklist item directly
-          // (marked can't parse "- [ ] " without text after it)
-          const root = $getRoot();
-          root.clear();
-          const checkList = $createListNode("check");
-          const checkItem = $createListItemNode(false);
-          checkList.append(checkItem);
-          root.append(checkList);
-          checkItem.selectEnd();
+    editor.update(
+      () => {
+        if (isNew) {
+          if (markdown === "- [ ] ") {
+            // Todo mode: create an empty checklist item directly
+            // (marked can't parse "- [ ] " without text after it)
+            const root = $getRoot();
+            root.clear();
+            const checkList = $createListNode("check");
+            const checkItem = $createListItemNode(false);
+            checkList.append(checkItem);
+            root.append(checkList);
+            checkItem.selectEnd();
+          } else {
+            // Normal new note: import markdown and place cursor at heading end
+            $importMarkdown(markdown);
+            const root = $getRoot();
+            const firstChild = root.getFirstChild();
+            if ($isHeadingNode(firstChild)) {
+              firstChild.selectEnd();
+            }
+          }
+        } else if (!markdown.trim()) {
+          // Empty existing note: leave the default empty paragraph.
+          // (Lexical initializes with one ParagraphNode by default.)
+          $setSelection(null);
         } else {
-          // Normal new note: import markdown and place cursor at heading end
           $importMarkdown(markdown);
-          const root = $getRoot();
-          const firstChild = root.getFirstChild();
-          if ($isHeadingNode(firstChild)) {
-            firstChild.selectEnd();
-          }
+          $setSelection(null);
         }
-      } else if (!markdown.trim()) {
-        // Empty existing note: leave the default empty paragraph.
-        // (Lexical initializes with one ParagraphNode by default.)
-        $setSelection(null);
-      } else {
-        $importMarkdown(markdown);
-        $setSelection(null);
-      }
 
-      const root = $getRoot();
-      console.log(`[editor:init] imported ${root.getChildrenSize()} nodes`);
-    });
+        const root = $getRoot();
+        console.log(`[editor:init] imported ${root.getChildrenSize()} nodes`);
+      },
+      // Use discrete so the update settles synchronously before onInitComplete.
+      // This ensures the OnChangeMarkdownPlugin baseline is recorded from the
+      // post-import state, not the pre-import state.
+      { discrete: true },
+    );
 
     onInitComplete();
   }, [editor, isNew, markdown, onInitComplete]);