Fix: no agents selected when installing find-skills

Author: quuu

src/add-prompt.test.ts

@@ -26,15 +26,16 @@ describe('promptForAgents', () => {
     vi.clearAllMocks();
   });
 
-  it('should use empty initial selection when no history exists', async () => {
+  it('should use default agents (claude-code, opencode, codex) when no history exists', async () => {
     vi.mocked(skillLock.getLastSelectedAgents).mockResolvedValue(undefined);
     vi.mocked(searchMultiselectModule.searchMultiselect).mockResolvedValue(['opencode']);
 
     await promptForAgents('Select agents', choices);
 
+    // Should default to claude-code, opencode, codex (filtered by available choices)
     expect(searchMultiselectModule.searchMultiselect).toHaveBeenCalledWith(
       expect.objectContaining({
-        initialSelected: [],
+        initialSelected: ['claude-code', 'opencode'],
       })
     );
   });
@@ -65,15 +66,17 @@ describe('promptForAgents', () => {
     );
   });
 
-  it('should use empty list if all history agents are invalid', async () => {
+  it('should use default agents if all history agents are invalid', async () => {
     vi.mocked(skillLock.getLastSelectedAgents).mockResolvedValue(['invalid-agent']);
     vi.mocked(searchMultiselectModule.searchMultiselect).mockResolvedValue(['opencode']);
 
     await promptForAgents('Select agents', choices);
 
+    // When history is invalid, should fall back to defaults (claude-code, opencode, codex)
+    // filtered by available choices
     expect(searchMultiselectModule.searchMultiselect).toHaveBeenCalledWith(
       expect.objectContaining({
-        initialSelected: [],
+        initialSelected: ['claude-code', 'opencode'],
       })
     );
   });

src/add.ts

@@ -115,17 +115,27 @@ export async function promptForAgents(
 
   const validAgents = choices.map((c) => c.value);
 
+  // Default agents to pre-select when no valid history exists
+  const defaultAgents: AgentType[] = ['claude-code', 'opencode', 'codex'];
+  const defaultValues = defaultAgents.filter((a) => validAgents.includes(a));
+
   let initialValues: AgentType[] = [];
 
   if (lastSelected && lastSelected.length > 0) {
     // Filter stored agents against currently valid agents
     initialValues = lastSelected.filter((a) => validAgents.includes(a as AgentType)) as AgentType[];
   }
 
+  // If no valid selection from history, use defaults
+  if (initialValues.length === 0) {
+    initialValues = defaultValues;
+  }
+
   const selected = await searchMultiselect({
     message,
     items: choices,
     initialSelected: initialValues,
+    required: true,
   });
 
   if (!isCancelled(selected)) {

src/prompts/search-multiselect.ts

@@ -20,6 +20,8 @@ export interface SearchMultiselectOptions<T> {
   items: SearchItem<T>[];
   maxVisible?: number;
   initialSelected?: T[];
+  /** If true, require at least one item to be selected before submitting */
+  required?: boolean;
 }
 
 const S_STEP_ACTIVE = pc.green('◆');
@@ -38,7 +40,7 @@ export const cancelSymbol = Symbol('cancel');
 export async function searchMultiselect<T>(
   options: SearchMultiselectOptions<T>
 ): Promise<T[] | symbol> {
-  const { message, items, maxVisible = 8, initialSelected = [] } = options;
+  const { message, items, maxVisible = 8, initialSelected = [], required = false } = options;
 
   return new Promise((resolve) => {
     const rl = readline.createInterface({
@@ -177,6 +179,10 @@ export async function searchMultiselect<T>(
     };
 
     const submit = (): void => {
+      // If required, don't allow submitting with no selection
+      if (required && selected.size === 0) {
+        return;
+      }
       render('submit');
       cleanup();
       resolve(Array.from(selected));