fix: backward compat for toggleOrderedList

Author: chittolina

packages/super-editor/src/headless-toolbar/create-headless-toolbar.test.ts

@@ -554,15 +554,13 @@ describe('createHeadlessToolbar', () => {
     controller.destroy();
   });
 
-  // PR-2873 (SD-2527): registry rename has compatibility implications.
-  // Hosts that wrap or mock the OLD command names (toggleBulletList,
-  // toggleOrderedList) without exposing the new style-aware ones now silently
-  // fail — this test documents that contract.
-  it('returns false for bullet-list when only the old toggleBulletList exists on the host', () => {
+  // PR-2873 (SD-2527): the registry prefers the new style-aware commands
+  // but falls back to the legacy ones so hosts that only expose
+  // toggleBulletList / toggleOrderedList keep working.
+  it('falls back to toggleBulletList when toggleBulletListStyle is unavailable', () => {
     const toggleBulletList = vi.fn(() => true);
     const superdoc = createActiveEditorHost({
       commands: {
-        // legacy command only — no toggleBulletListStyle
         toggleBulletList,
       },
       state: createSelectionState({
@@ -581,13 +579,13 @@ describe('createHeadlessToolbar', () => {
       commands: ['bullet-list'],
     });
 
-    expect(controller.execute?.('bullet-list')).toBe(false);
-    expect(toggleBulletList).not.toHaveBeenCalled();
+    expect(controller.execute?.('bullet-list')).toBe(true);
+    expect(toggleBulletList).toHaveBeenCalledTimes(1);
 
     controller.destroy();
   });
 
-  it('returns false for numbered-list when only the old toggleOrderedList exists on the host', () => {
+  it('falls back to toggleOrderedList when toggleOrderedListStyle is unavailable', () => {
     const toggleOrderedList = vi.fn(() => true);
     const superdoc = createActiveEditorHost({
       commands: {
@@ -609,8 +607,8 @@ describe('createHeadlessToolbar', () => {
       commands: ['numbered-list'],
     });
 
-    expect(controller.execute?.('numbered-list')).toBe(false);
-    expect(toggleOrderedList).not.toHaveBeenCalled();
+    expect(controller.execute?.('numbered-list')).toBe(true);
+    expect(toggleOrderedList).toHaveBeenCalledTimes(1);
 
     controller.destroy();
   });

packages/super-editor/src/headless-toolbar/helpers/paragraph.ts

@@ -144,6 +144,25 @@ export const createIndentIncreaseExecute =
     return createDirectCommandExecute('increaseTextIndent')({ context });
   };
 
+const createListToggleExecute =
+  (styleCommand: string, legacyCommand: string) =>
+  ({ context, payload }: { context: ToolbarContext | null; payload?: unknown }) => {
+    const editor = resolveStateEditor(context);
+    const commands = editor?.commands;
+    if (typeof commands?.[styleCommand] === 'function') {
+      const result = payload === undefined ? commands[styleCommand]() : commands[styleCommand](payload);
+      return Boolean(result);
+    }
+    if (typeof commands?.[legacyCommand] === 'function') {
+      return Boolean(commands[legacyCommand]());
+    }
+    return false;
+  };
+
+export const createBulletListExecute = () => createListToggleExecute('toggleBulletListStyle', 'toggleBulletList');
+
+export const createOrderedListExecute = () => createListToggleExecute('toggleOrderedListStyle', 'toggleOrderedList');
+
 export const createIndentDecreaseExecute =
   () =>
   ({ context }: { context: ToolbarContext | null; payload?: unknown }) => {

packages/super-editor/src/headless-toolbar/toolbar-registry.ts

@@ -28,11 +28,13 @@ import {
   createUnderlineExecute,
 } from './helpers/formatting.js';
 import {
+  createBulletListExecute,
   createIndentDecreaseExecute,
   createIndentIncreaseExecute,
   createLineHeightStateDeriver,
   createLinkedStyleStateDeriver,
   createListStateDeriver,
+  createOrderedListExecute,
   createTextAlignStateDeriver,
 } from './helpers/paragraph.js';
 import { createDirectCommandExecute, createDisabledStateDeriver } from './helpers/general.js';
@@ -121,11 +123,13 @@ export const createToolbarRegistry = (): Partial<Record<PublicToolbarItemId, Bui
       id: 'bullet-list',
       directCommandName: 'toggleBulletListStyle',
       state: createListStateDeriver('bullet'),
+      execute: createBulletListExecute(),
     },
     'numbered-list': {
       id: 'numbered-list',
       directCommandName: 'toggleOrderedListStyle',
       state: createListStateDeriver('ordered'),
+      execute: createOrderedListExecute(),
     },
     'indent-increase': {
       id: 'indent-increase',