fix(superdoc): scrollToElement matches any of nodeId/sdBlockId/id/paraId per node

Block nodes in imported docx files can carry more than one ID-shaped
attr at once — paragraphs from `.docx` carry both `paraId` (from
`w14:paraId` in OOXML) and `sdBlockId` (minted by SuperDoc on import).

The previous walk did `const candidate = a.nodeId ?? a.sdBlockId ?? a.id ?? a.paraId`
and compared the single first-non-null pick to the caller's elementId.
That meant a paragraph carrying both `sdBlockId: "3496bf7f-..."` and
`paraId: "00000001"` would always return `sdBlockId` as the candidate
and never match a caller looking for `"00000001"`.

Compare each attr independently so the walk matches when ANY of the
candidate fields equals elementId. Consumers can now pass whichever
ID handle they have without knowing which ID types a given block
happens to carry.

Adds one regression test asserting that scrollToElement('00000001')
resolves to `true` even when the matching paragraph also has a
non-matching sdBlockId.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

Author: bjohas

packages/superdoc/src/core/SuperDoc.js

@@ -1438,13 +1438,18 @@ export class SuperDoc extends EventEmitter {
     }
 
     // 2. Fall back to a single PM walk looking for matching block-level
-    //    id attributes (nodeId / sdBlockId / id / paraId).
+    //    id attributes. Block nodes can carry multiple ID-shaped attrs
+    //    at once — e.g. paragraphs from a `.docx` carry both `paraId`
+    //    (the OOXML `w14:paraId`) and `sdBlockId` (minted by SuperDoc
+    //    on import). We must compare against each independently rather
+    //    than picking the first non-null and comparing, because the
+    //    caller may have a handle on any one of them and consumers
+    //    shouldn't have to know which ID type a given block carries.
     if (pos == null || !Number.isFinite(pos)) {
       editor.state.doc.descendants((node, p) => {
         if (pos != null) return false;
         const a = node.attrs || {};
-        const candidate = a.nodeId ?? a.sdBlockId ?? a.id ?? a.paraId;
-        if (candidate && candidate === elementId) {
+        if (a.nodeId === elementId || a.sdBlockId === elementId || a.id === elementId || a.paraId === elementId) {
           pos = p;
           return false;
         }

packages/superdoc/src/core/SuperDoc.test.js

@@ -475,6 +475,51 @@ describe('SuperDoc core', () => {
     );
   });
 
+  it('scrollToElement matches paraId even when sdBlockId is also present', async () => {
+    const { superdocStore } = createAppHarness();
+    superdocStore.documents = [{ getPresentationEditor: vi.fn(() => ({ scrollToElement: vi.fn(async () => false) })) }];
+
+    const scrollIntoView = vi.fn();
+    const targetEl = { scrollIntoView };
+    const setCursorById = vi.fn(() => false);
+
+    // A paragraph carrying BOTH a long sdBlockId and a short paraId.
+    // The walk must match against each attr independently — picking
+    // the first non-null and comparing would let sdBlockId mask paraId.
+    const node = {
+      attrs: {
+        sdBlockId: '3496bf7f-b408-489d-9d1d-7a6854c09e70',
+        paraId: '00000001',
+      },
+    };
+    const descendants = (cb) => {
+      cb(node, 7);
+    };
+
+    const instance = new SuperDoc({
+      selector: '#host',
+      document: 'https://example.com/doc.docx',
+      documents: [],
+      modules: { comments: {}, toolbar: {} },
+      onException: vi.fn(),
+    });
+    await flushMicrotasks();
+
+    Object.defineProperty(instance, 'activeEditor', {
+      configurable: true,
+      get: () => ({
+        state: { doc: { descendants, content: { size: 100 } }, selection: { from: null } },
+        commands: { setCursorById },
+        getElementAtPos: vi.fn(() => targetEl),
+      }),
+    });
+
+    await expect(instance.scrollToElement('00000001')).resolves.toBe(true);
+    expect(scrollIntoView).toHaveBeenCalledWith(
+      expect.objectContaining({ block: expect.any(String), inline: 'nearest' }),
+    );
+  });
+
   it('scrollToHeading walks for the Nth heading at the given level and scrolls', async () => {
     const { superdocStore } = createAppHarness();
     // Mock doc with three Heading1 paragraphs at known positions.