fix: remove plugins.json check and update error handling in plugin search (#31)

Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>

Author: yordis

src/commands/plugin-search.ts

@@ -1,6 +1,5 @@
 import { z } from 'zod';
-import { getConfigPaths, loadPluginsConfig } from '../config/loader';
-import { fileExists } from '../helpers/fs';
+import { loadPluginsConfig } from '../config/loader';
 import { resolveMarketplacePath } from '../helpers/git';
 import { defaultIO } from '../helpers/io';
 import { getAvailablePlugins, getMarketplaceType, loadMarketplaceManifest } from '../helpers/marketplace';
@@ -22,17 +21,15 @@ export async function pluginSearch(options: unknown): Promise<void> {
   const cmd = PluginSearchOptionsSchema.parse(options);
 
   const cwd = cmd.cwd || process.cwd();
-  const paths = getConfigPaths(cwd);
 
   try {
-    if (!(await fileExists(paths.plugins))) {
-      const error = new Error("No plugins.json found. Run 'aipm init' first.");
-      defaultIO.logError(error.message);
-      throw error;
-    }
-
     const { config } = await loadPluginsConfig(cwd);
 
+    if (Object.keys(config.marketplaces).length === 0) {
+      defaultIO.logInfo('No marketplaces configured. Add a marketplace first with: aipm marketplace add <name> <path>');
+      return;
+    }
+
     const allPlugins: PluginInfo[] = [];
 
     for (const [marketplaceName, marketplace] of Object.entries(config.marketplaces)) {

tests/commands/plugin-search.test.ts

@@ -1,8 +1,10 @@
-import { afterEach, beforeEach, describe, expect, test } from 'bun:test';
+import { afterEach, beforeEach, describe, expect, spyOn, test } from 'bun:test';
 import { mkdir, mkdtemp, rm, writeFile } from 'node:fs/promises';
 import { tmpdir } from 'node:os';
 import { join } from 'node:path';
 import { pluginSearch } from '../../src/commands/plugin-search';
+import * as claudeCodeConfig from '../../src/helpers/claude-code-config';
+import { defaultIO } from '../../src/helpers/io';
 
 describe('plugin-search', () => {
   let testDir: string;
@@ -277,12 +279,23 @@ describe('plugin-search', () => {
   });
 
   describe('error handling', () => {
-    test('should error if no plugins.json found', async () => {
+    test('should log info if no marketplaces configured', async () => {
       const options = {
         cwd: testDir,
       };
 
-      await expect(pluginSearch(options)).rejects.toThrow('No plugins.json found');
+      // Mock Claude Code as not installed to ensure no auto-discovered marketplaces
+      const claudeSpy = spyOn(claudeCodeConfig, 'isClaudeCodeInstalled').mockResolvedValue(false);
+      const infoSpy = spyOn(defaultIO, 'logInfo');
+
+      await pluginSearch(options);
+
+      expect(infoSpy).toHaveBeenCalledWith(
+        'No marketplaces configured. Add a marketplace first with: aipm marketplace add <name> <path>',
+      );
+
+      claudeSpy.mockRestore();
+      infoSpy.mockRestore();
     });
 
     test('should handle marketplaces with invalid paths', async () => {