fix(chat): fix the slash-command Enter regression

- `src/components/Chat/CommandMenu.tsx` now uses the local `Suggestions` component instead of
  `SelectPrompt`.
- This avoids `SelectPrompt`’s one-tick disabled state, so Enter is handled immediately and
  still respects arrow-focused command selection.
- `src/components/Chat/CommandMenu.test.tsx` now tests real stdin Enter/Down behavior instead
  of mocking selection callbacks.

Author: remarkablemark

src/components/Chat/CommandMenu.test.tsx

@@ -1,44 +1,18 @@
-import { Text } from 'ink';
 import { render } from 'ink-testing-library';
 
-interface MockSelectPromptProps {
-  highlightText: string;
-  onChange: (value: string) => void;
-  options: { label: string; value: string }[];
-}
-
-const { mockSelectPrompt } = vi.hoisted(() => ({
-  mockSelectPrompt: vi.fn<(props: MockSelectPromptProps) => void>(),
-}));
-
-vi.mock('@/components/SelectPrompt', () => ({
-  SelectPrompt: (props: MockSelectPromptProps) => {
-    mockSelectPrompt(props);
-    return (
-      <>
-        {props.options.map(({ label, value }) => (
-          <Text key={value}>{label}</Text>
-        ))}
-      </>
-    );
-  },
-}));
+import { KEY } from '@/constants';
+import { time } from '@/utils';
 
 import { CommandMenu } from './CommandMenu';
 
 describe('CommandMenu', () => {
-  beforeEach(() => {
-    mockSelectPrompt.mockReset();
-  });
-
   it('returns null when input does not start with a slash', () => {
     const onSubmit = vi.fn();
     const { lastFrame } = render(
       <CommandMenu input="hello" onSubmit={onSubmit} />,
     );
 
     expect(lastFrame()).toBe('');
-    expect(mockSelectPrompt).not.toHaveBeenCalled();
   });
 
   it('returns null when no commands match the slash input', () => {
@@ -48,34 +22,35 @@ describe('CommandMenu', () => {
     );
 
     expect(lastFrame()).toBe('');
-    expect(mockSelectPrompt).not.toHaveBeenCalled();
   });
 
-  it('renders matching commands and forwards selection', () => {
+  it('renders matching commands and selects with Enter', async () => {
     const onSubmit = vi.fn();
-    const { lastFrame } = render(
+    const { lastFrame, stdin } = render(
       <CommandMenu input="/m" onSubmit={onSubmit} />,
     );
 
     expect(lastFrame()).toContain('/model - manage Ollama models');
     expect(lastFrame()).not.toContain('/clear - clear the current session');
-    expect(mockSelectPrompt).toHaveBeenCalledTimes(1);
-
-    const [firstCall] = mockSelectPrompt.mock.calls;
-    expect(firstCall).toBeDefined();
-    const [props] = firstCall;
-    expect(props.highlightText).toBe('/m');
-    expect(props.options).toEqual([
-      {
-        label: '/model - manage Ollama models',
-        value: '/model',
-      },
-    ]);
-    const { onChange } = props;
-    onChange('/model');
+
+    stdin.write(KEY.ENTER);
+    await time.tick();
+
     expect(onSubmit).toHaveBeenCalledWith('/model');
   });
 
+  it('moves focus through slash commands before selecting', async () => {
+    const onSubmit = vi.fn();
+    const { stdin } = render(<CommandMenu input="/" onSubmit={onSubmit} />);
+
+    stdin.write(KEY.DOWN);
+    await time.tick();
+    stdin.write(KEY.ENTER);
+    await time.tick();
+
+    expect(onSubmit).toHaveBeenCalledWith('/session');
+  });
+
   it('includes /search in matching command results', () => {
     const onSubmit = vi.fn();
     const { lastFrame } = render(

src/components/Chat/CommandMenu.tsx

@@ -1,6 +1,6 @@
 import { useMemo } from 'react';
 
-import { SelectPrompt } from '@/components/SelectPrompt';
+import { Suggestions } from '@/components/Suggestions';
 import { COMMAND } from '@/constants';
 
 interface Props {
@@ -30,9 +30,10 @@ export function CommandMenu({ input, onSubmit }: Props) {
   }
 
   return (
-    <SelectPrompt
-      highlightText={input}
-      onChange={onSubmit}
+    <Suggestions
+      onSelect={(option) => {
+        onSubmit(option.value);
+      }}
       options={commandOptions}
     />
   );