smartrte-core 1.0.0-beta.5 → 1.0.0-beta.6

package/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## 1.0.0-beta.6
+
+- Fix pasting a long, list-free document (many short paragraphs, no images) being slow enough to look broken — the Office/Google Docs list normalizers recursed into every paragraph hunting for nested lists a `<p>` can never structurally contain. Pasting a few thousand blocks now completes in well under a second instead of multiple seconds to tens of seconds. See `docs/bugs/clipboard-paste-slow-on-large-list-free-documents.md` for the full investigation, including a known remaining limitation for very large (10,000+ block) pastes.
+
 ## 1.0.0-beta.5
 
 - Add a per-block `lineHeight` attribute (paragraph, heading, blockquote, code block) — a unitless multiplier matching CSS `line-height`'s own unitless mode (`1`, `1.5`, `2`, ...), consistent with how `align`/`indentLevel` are represented. Absence means no override (the browser/font's own natural line-height), not a forced `1`. Full HTML round-trip fidelity via a dedicated `data-smart-line-height` marker — deliberately does **not** fall back to reading a bare CSS `line-height` value on import (unlike `align`/`indentLevel`'s own CSS fallbacks): real captured clipboard fixtures show `line-height` is frequently an ambient, whole-document default baked onto every paragraph, not a deliberate per-paragraph choice, and recognizing it would have silently misattributed that default as an explicit override on every pasted document. Real Word line spacing (`w:spacing`/`w:line`/`w:lineRule="auto"`) on DOCX export; declared `lossy` there (confirmed empirically that DOCX reimport via mammoth drops it) and `unsupported` (content preserved) in Markdown. Inherits `full` PDF fidelity for free, since this package's "Save as PDF" prints the same HTML export.

package/dist/foundation/clipboard/normalizers.js

@@ -2,6 +2,20 @@ import { parseCanonicalListMarkdown, serializeCanonicalListHtml } from "../list/
 const listTags = new Set(["UL", "OL"]);
 const blockSelector = ":scope > p, :scope > h1, :scope > h2, :scope > h3, :scope > h4, :scope > h5, :scope > h6, :scope > ul, :scope > ol, :scope > blockquote, :scope > pre, :scope > table, :scope > div";
 const msoListPattern = /(?:^|;)\s*mso-list\s*:\s*([^\s;]+)\s+level(\d+)\s+([^\s;]+)/i;
+/**
+ * Elements whose HTML content model is inline-only - a real HTML parser
+ * never lets a <p>/<ul>/<ol>/<div> nest inside one of these (an outer <p>
+ * auto-closes first), so a nested Office mso-list paragraph or a
+ * declared-level <ul>/<ol> can never appear inside one. normalizeMsoLists/
+ * normalizeDeclaredLevelLists below recurse looking for exactly those two
+ * things - recursing into one of these tags is always wasted work. See
+ * docs/bugs/clipboard-paste-normalizer-quadratic-blowup-on-large-documents.md:
+ * on a real document with thousands of top-level paragraphs (the common
+ * case for a long pasted document with no lists at all), skipping this
+ * recursion is what turns an O(n) visit count with an accidentally
+ * document-size-scaled per-call cost back into genuinely linear work.
+ */
+const INLINE_LEAF_TAGS = new Set(["P", "SPAN", "B", "STRONG", "I", "EM", "U", "S", "A", "CODE", "BR", "IMG", "SUP", "SUB"]);
 const unwrap = (element) => element.replaceWith(...Array.from(element.childNodes));
 const normalizeTransparentContainers = (document) => {
     const docsWrapper = document.body.querySelector('[id^="docs-internal-guid-"]');
@@ -71,7 +85,8 @@ const normalizeMsoLists = (document) => {
         for (let index = 0; index < children.length;) {
             const first = msoListEntry(children[index]);
             if (!first) {
-                visit(children[index]);
+                if (!INLINE_LEAF_TAGS.has(children[index].tagName))
+                    visit(children[index]);
                 index += 1;
                 continue;
             }
@@ -167,7 +182,8 @@ const normalizeDeclaredLevelLists = (document) => {
             run[0].replaceWith(...roots);
             run.slice(1).forEach((list) => list.remove());
         }
-        Array.from(parent.children).forEach((child) => visit(child));
+        Array.from(parent.children).forEach((child) => { if (!INLINE_LEAF_TAGS.has(child.tagName))
+            visit(child); });
     };
     visit(document.body);
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "smartrte-core",
-  "version": "1.0.0-beta.5",
+  "version": "1.0.0-beta.6",
   "description": "Framework-agnostic document model and editing primitives for Smart RTE",
   "repository": {
     "type": "git",