fix(editor): cancel focus-scroll RAF on intentional scroll

Store the RAF handle from wrapHiddenEditorFocus on the instance and
cancel it when scrollToPosition is called. Prevents the safety net
from undoing intentional scrolls like search navigation.

Author: tupizz

packages/super-editor/src/core/presentation-editor/PresentationEditor.ts

@@ -284,6 +284,7 @@ export class PresentationEditor extends EventEmitter {
   #errorBannerMessage: HTMLElement | null = null;
   #renderScheduled = false;
   #pendingDocChange = false;
+  #focusScrollRafId: number | null = null;
   #pendingMapping: Mapping | null = null;
   #isRerendering = false;
   #selectionSync = new SelectionSyncCoordinator();
@@ -747,7 +748,13 @@ export class PresentationEditor extends EventEmitter {
       // Safety net: the browser may asynchronously scroll after ProseMirror's
       // selectionToDOM() modifies the DOM selection inside the hidden editor.
       // A single requestAnimationFrame catches this post-layout scroll.
-      win.requestAnimationFrame(() => {
+      // The RAF ID is stored so scrollToPosition() can cancel it — otherwise
+      // intentional scrolls (e.g. search navigation) would be undone.
+      if (this.#focusScrollRafId != null) {
+        win.cancelAnimationFrame(this.#focusScrollRafId);
+      }
+      this.#focusScrollRafId = win.requestAnimationFrame(() => {
+        this.#focusScrollRafId = null;
         if (win.scrollX !== beforeX || win.scrollY !== beforeY) {
           win.scrollTo(beforeX, beforeY);
         }
@@ -2021,6 +2028,14 @@ export class PresentationEditor extends EventEmitter {
     pos: number,
     options: { block?: 'start' | 'center' | 'end' | 'nearest'; behavior?: ScrollBehavior } = {},
   ): boolean {
+    // Cancel any pending focus-scroll RAF so this intentional scroll is not undone
+    // by the wrapHiddenEditorFocus safety net (e.g. search navigation after focus).
+    if (this.#focusScrollRafId != null) {
+      const win = this.#visibleHost.ownerDocument?.defaultView;
+      if (win) win.cancelAnimationFrame(this.#focusScrollRafId);
+      this.#focusScrollRafId = null;
+    }
+
     const activeEditor = this.getActiveEditor();
     const doc = activeEditor?.state?.doc;
     if (!doc) return false;

packages/super-editor/src/core/presentation-editor/tests/PresentationEditor.focusWrapping.test.ts

@@ -371,6 +371,23 @@ describe('PresentationEditor - Focus Wrapping (#wrapHiddenEditorFocus)', () => {
       expect(rafSpy).toHaveBeenCalledTimes(1);
       expect(rafSpy).toHaveBeenCalledWith(expect.any(Function));
     });
+
+    it('cancels focus-scroll RAF when scrollToPosition is called', () => {
+      editor = new PresentationEditor({
+        element: container,
+        documentId: 'test-doc',
+        pageSize: { w: 612, h: 792 },
+      });
+
+      const cancelSpy = vi.spyOn(window, 'cancelAnimationFrame');
+
+      editor.editor.view.focus();
+
+      // scrollToPosition will fail (no layout) but should still cancel the RAF
+      editor.scrollToPosition(0);
+
+      expect(cancelSpy).toHaveBeenCalled();
+    });
   });
 
   describe('mock detection', () => {