Merge pull request #49 from ai-action/test/ModelPicker

test(ModelPicker): fix failing tests

Author: remarkablemark

src/components/ModelPicker.test.tsx

@@ -9,7 +9,7 @@ const { mockListModels, mockOnChange } = vi.hoisted(() => ({
 }));
 
 vi.mock('@inkjs/ui', async () => {
-  const { Text, useInput } = await import('ink');
+  const { Text } = await import('ink');
   return {
     Spinner: ({ label }: { label?: string }) => (
       <Text>{`⏳${label ?? ''}`}</Text>
@@ -23,13 +23,6 @@ vi.mock('@inkjs/ui', async () => {
       onChange?: (value: string) => void;
     }) => {
       mockOnChange.mockImplementation((v) => onChange?.(v));
-
-      useInput((_input, key) => {
-        if (key.return && options.length > 0) {
-          onChange?.(options[0].value);
-        }
-      });
-
       return (
         <>
           {options.map(({ value, label }) => (

src/components/ModelPicker.tsx

@@ -21,12 +21,11 @@ export function ModelPicker({ currentModel, onSelect, onClose }: Props) {
   useInput(
     // eslint-disable-next-line @typescript-eslint/no-misused-promises
     async (_input, key) => {
-      if (key.return) {
+      if (!error && options.length && key.return) {
         await time.tick();
         onClose();
       }
     },
-    { isActive: Boolean(options.length) },
   );
 
   useEffect(() => {