fix(presentation-editor): route comment navigation through body editor

Author: luccas-harbour

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

@@ -5942,7 +5942,8 @@ export class PresentationEditor extends EventEmitter {
     }
 
     if (target.entityType === 'comment') {
-      const setCursorById = this.getActiveEditor()?.commands?.setCursorById;
+      const bodyEditor = this.#resolveBodyEditorForNavigation();
+      const setCursorById = bodyEditor?.commands?.setCursorById;
       if (typeof setCursorById !== 'function') return false;
 
       return Boolean(
@@ -5957,6 +5958,17 @@ export class PresentationEditor extends EventEmitter {
     return false;
   }
 
+  #resolveBodyEditorForNavigation(): Editor | null {
+    if (!this.#editor) return null;
+
+    const sessionMode = this.#headerFooterSession?.session?.mode ?? 'body';
+    if (sessionMode !== 'body') {
+      this.#exitHeaderFooterMode();
+    }
+
+    return this.#editor;
+  }
+
   async #navigateToTrackedChange(id: string): Promise<boolean> {
     const activeEditor = this.getActiveEditor();
     const setCursorById = activeEditor?.commands?.setCursorById;

packages/super-editor/src/editors/v1/core/presentation-editor/tests/PresentationEditor.goToAnchor.test.ts

@@ -639,7 +639,8 @@ describe('PresentationEditor - goToAnchor', () => {
       documentId: 'test-doc',
     });
 
-    mockActiveEditor.commands.setCursorById = vi.fn(() => true);
+    const bodyEditor = editor.getActiveEditor();
+    bodyEditor.commands.setCursorById = vi.fn(() => true);
     editor.getActiveEditor = vi.fn(() => mockActiveEditor as never);
 
     const result = await editor.navigateTo({
@@ -649,10 +650,11 @@ describe('PresentationEditor - goToAnchor', () => {
     });
 
     expect(result).toBe(true);
-    expect(mockActiveEditor.commands.setCursorById).toHaveBeenCalledWith('comment-1', {
+    expect(bodyEditor.commands.setCursorById).toHaveBeenCalledWith('comment-1', {
       preferredActiveThreadId: 'comment-1',
       activeCommentId: 'comment-1',
     });
+    expect(mockActiveEditor.commands.setCursorById).toBeUndefined();
   });
 
   it('routes tracked change navigation through the raw tracked-change id when given a canonical id', async () => {