fix: ignore empty runs

Author: VladaHarbour

packages/super-editor/src/editors/v1/core/layout-adapter/converters/paragraph.test.ts

@@ -352,6 +352,44 @@ describe('getLastParagraphFont', () => {
     const result = getLastParagraphFont(blocks);
     expect(result).toBeUndefined();
   });
+
+  it('skips empty first runs even when they carry stale font values', () => {
+    const blocks: FlowBlock[] = [
+      {
+        kind: 'paragraph',
+        id: '0-paragraph',
+        runs: [
+          {
+            kind: 'text',
+            text: 'Valid',
+            fontFamily: 'ValidFont',
+            fontSize: 11,
+            pmStart: 0,
+            pmEnd: 5,
+          },
+        ],
+        attrs: {},
+      },
+      {
+        kind: 'paragraph',
+        id: '1-paragraph',
+        runs: [
+          {
+            kind: 'text',
+            text: '',
+            fontFamily: 'StaleFont',
+            fontSize: 42,
+            pmStart: 5,
+            pmEnd: 5,
+          },
+        ],
+        attrs: {},
+      },
+    ];
+
+    const result = getLastParagraphFont(blocks);
+    expect(result).toEqual({ fontFamily: 'ValidFont', fontSize: 11 });
+  });
 });
 
 describe('paragraph converters', () => {

packages/super-editor/src/editors/v1/core/layout-adapter/converters/paragraph.ts

@@ -1077,7 +1077,10 @@ export function getLastParagraphFont(blocks: FlowBlock[]): ParagraphFont | undef
       const para = block as ParagraphBlock;
       const firstRun = para.runs?.[0];
       if (!firstRun) continue;
-      const run = firstRun as { fontFamily?: string; fontSize?: number };
+      const run = firstRun as { text?: string; fontFamily?: string; fontSize?: number };
+      if (typeof run.text === 'string' && run.text.length === 0) {
+        continue;
+      }
       const fontFamily = typeof run.fontFamily === 'string' ? run.fontFamily.trim() : '';
       const fontSize = typeof run.fontSize === 'number' && Number.isFinite(run.fontSize) ? run.fontSize : NaN;
       if (fontFamily.length > 0 && fontSize > 0) {

packages/super-editor/src/editors/v1/core/layout-adapter/list-marker-font.test.ts

@@ -155,6 +155,8 @@ describe('syncListMarkerFontFromParagraphRuns', () => {
 
     expect(block.attrs.wordLayout?.marker?.run?.fontFamily).toContain('Georgia');
     expect(block.attrs.wordLayout?.marker?.run?.fontSize).toBe(30);
+    expect(block.runs[0]?.fontFamily).toContain('Georgia');
+    expect(block.runs[0]?.fontSize).toBe(30);
   });
 
   it('does not fall back to stale cached runs on cache hits without textStyle or previousParagraphFont', () => {

packages/super-editor/src/editors/v1/core/layout-adapter/list-marker-font.ts

@@ -134,6 +134,16 @@ export const syncListMarkerFontFromParagraphRuns = ({
   const contentFont = resolveContentFont(block, para, contentFontSource, previousParagraphFont);
   if (!contentFont) return;
 
+  // Cache-hit path may reuse stale empty runs. Normalize empty run font so subsequent
+  // getLastParagraphFont() reads the current inherited font instead of cached values.
+  if (contentFontSource === 'paragraph') {
+    const firstRun = block.runs[0];
+    if (firstRun && isTextRun(firstRun) && firstRun.text.length === 0) {
+      if (contentFont.fontFamily) firstRun.fontFamily = contentFont.fontFamily;
+      if (contentFont.fontSize) firstRun.fontSize = contentFont.fontSize;
+    }
+  }
+
   const numberingOverrides = getNumberingMarkerFontOverrides(block.attrs?.numberingProperties, converterContext);
 
   if (!numberingOverrides.fontFamily && contentFont.fontFamily) {