ref(cmdk): Remove onAction prop in favor of internal action execution

static/app/components/commandPalette/__stories__/components.tsx

@@ -1,31 +1,11 @@
-import {useCallback} from 'react';
-
 import {addSuccessMessage} from 'sentry/actionCreators/indicator';
-import {CommandPaletteProvider} from 'sentry/components/commandPalette/ui/cmdk';
-import {CMDKAction} from 'sentry/components/commandPalette/ui/cmdk';
-import type {CMDKActionData} from 'sentry/components/commandPalette/ui/cmdk';
-import type {CollectionTreeNode} from 'sentry/components/commandPalette/ui/collection';
+import {
+  CMDKAction,
+  CommandPaletteProvider,
+} from 'sentry/components/commandPalette/ui/cmdk';
 import {CommandPalette} from 'sentry/components/commandPalette/ui/commandPalette';
-import {normalizeUrl} from 'sentry/utils/url/normalizeUrl';
-import {useNavigate} from 'sentry/utils/useNavigate';
 
 export function CommandPaletteDemo() {
-  const navigate = useNavigate();
-
-  const handleAction = useCallback(
-    (
-      action: CollectionTreeNode<CMDKActionData>,
-      _options?: {modifierKeys?: {shiftKey: boolean}}
-    ) => {
-      if ('to' in action) {
-        navigate(normalizeUrl(action.to));
-      } else if ('onAction' in action) {
-        action.onAction();
-      }
-    },
-    [navigate]
-  );
-
   return (
     <CommandPaletteProvider>
       <CMDKAction display={{label: 'Go to Flex story'}} to="/stories/layout/flex/" />
@@ -49,7 +29,7 @@ export function CommandPaletteDemo() {
           onAction={() => addSuccessMessage('Deselect all')}
         />
       </CMDKAction>
-      <CommandPalette onAction={handleAction} />
+      <CommandPalette />
     </CommandPaletteProvider>
   );
 }

static/app/components/commandPalette/ui/cmdk.tsx

@@ -50,13 +50,6 @@ export const CMDKCollection = makeCollection<CMDKActionData>();
 /**
  * Root provider for the command palette. Wrap the component tree that
  * contains CMDKAction registrations and the CommandPalette UI.
- *
- * Slot outlets are rendered separately in the navigation (see
- * CommandPaletteSlotOutlets in navigation/index.tsx) in task → page → global
- * DOM order so that presortBySlotRef's compareDocumentPosition sorting works
- * correctly. Keeping the outlets in the navigation (rather than here) means
- * this provider introduces no DOM nodes — tests that assert an empty container
- * are unaffected.
  */
 export function CommandPaletteProvider({children}: {children: React.ReactNode}) {
   return (