fix: html diff error with whitespace (#2230)

* fix html diff error with whitespace

* improve logging

* fix: only preserveWhitespace on node content

See [this description for more details](#2230 (comment))

* add test

* fix: preserveWhitespace for our parse rule

* test: add prefix test

---------

Co-authored-by: Nick the Sick <computers@nickthesick.com>

Author: YousefED

examples/09-ai/02-playground/src/data/aimodels.ts

@@ -11,8 +11,8 @@ export const AI_MODELS = [
   "mistral.chat/mistral-medium-latest",
   "mistral.chat/ministral-3b-latest",
   "mistral.chat/ministral-8b-latest",
-  "anthropic.chat/claude-opus-4-1",
-  "anthropic.chat/claude-sonnet-4-0",
+  "anthropic.chat/claude-opus-4-5",
+  "anthropic.chat/claude-sonnet-4-5",
   "anthropic.chat/claude-3-7-sonnet-latest",
   "anthropic.chat/claude-3-5-haiku-latest",
   "albert-etalab.chat/albert-large",

packages/core/src/api/parsers/markdown/parseMarkdown.ts

@@ -93,6 +93,17 @@ export function markdownToHTML(markdown: string): string {
           }
         },
         code,
+        blockquote: (state: any, node: any) => {
+          const result = {
+            type: "element",
+            tagName: "blockquote",
+            properties: {},
+            // The only difference from the original is that we don't wrap the children with line endings
+            children: state.wrap(state.all(node), false),
+          };
+          state.patch(node, result);
+          return state.applyData(node, result);
+        },
       },
     })
     .use(rehypeStringify)

packages/core/src/schema/blocks/createSpec.ts

@@ -70,6 +70,8 @@ export function getParseRules<
 
         return props;
       },
+      // Because we do the parsing ourselves, we want to preserve whitespace for content we've parsed
+      preserveWhitespace: true,
       getContent:
         config.content === "inline" || config.content === "none"
           ? (node, schema) => {
@@ -97,6 +99,7 @@ export function getParseRules<
                 const parser = DOMParser.fromSchema(schema);
                 const parsed = parser.parse(clone, {
                   topNode: schema.nodes.paragraph.create(),
+                  preserveWhitespace: true,
                 });
 
                 return parsed.content;

packages/xl-ai/src/api/formats/base-tools/createUpdateBlockTool.ts

@@ -143,11 +143,13 @@ export function createUpdateBlockTool<T>(config: {
         const block = editor.getBlock(id);
 
         if (!block) {
-          // eslint-disable-next-line no-console
-          console.error("BLOCK NOT FOUND", id);
           return {
             ok: false,
-            error: "block not found",
+            error: new Error("Block not found (update)", {
+              cause: {
+                blockId: id,
+              },
+            }),
           };
         }
 

packages/xl-ai/src/api/formats/base-tools/delete.ts

@@ -62,7 +62,11 @@ export const deleteBlockTool = (
       if (!block) {
         return {
           ok: false,
-          error: "block not found",
+          error: new Error("Block not found (delete)", {
+            cause: {
+              blockId: id,
+            },
+          }),
         };
       }
 

packages/xl-ai/src/api/formats/html-blocks/tools/rebaseTool.ts

@@ -48,8 +48,12 @@ export async function createHTMLRebaseTool(
   );
 
   if (steps.length) {
-    // console.error("html diff", steps);
-    throw new Error("html diff");
+    throw new Error("html diff", {
+      cause: {
+        html,
+        htmlBlock
+      }
+    });
   }
 
   return rebaseTool(editor, tr);

tests/src/unit/core/formatConversion/parse/__snapshots__/html/mixedTextTableCell.json

@@ -15,7 +15,10 @@
                 {
                   "styles": {},
                   "text": "Table Cell
-Table Cell Table Cell",
+Table Cell
+        
+        Table Cell
+",
                   "type": "text",
                 },
               ],

tests/src/unit/core/formatConversion/parse/__snapshots__/html/notion.json

@@ -169,7 +169,8 @@
     "content": [
       {
         "styles": {},
-        "text": "Paragraph With Hard Break",
+        "text": "Paragraph
+With Hard Break",
         "type": "text",
       },
     ],

tests/src/unit/core/formatConversion/parse/__snapshots__/html/trailing whitespace.json

@@ -0,0 +1,19 @@
+[
+  {
+    "children": [],
+    "content": [
+      {
+        "styles": {},
+        "text": "hello, ",
+        "type": "text",
+      },
+    ],
+    "id": "1",
+    "props": {
+      "backgroundColor": "default",
+      "textAlignment": "left",
+      "textColor": "default",
+    },
+    "type": "paragraph",
+  },
+]

tests/src/unit/core/formatConversion/parse/__snapshots__/html/whitespace prefix.json

@@ -0,0 +1,19 @@
+[
+  {
+    "children": [],
+    "content": [
+      {
+        "styles": {},
+        "text": " hello,",
+        "type": "text",
+      },
+    ],
+    "id": "1",
+    "props": {
+      "backgroundColor": "default",
+      "textAlignment": "left",
+      "textColor": "default",
+    },
+    "type": "paragraph",
+  },
+]