refactor(plugins): remove metadata export requirement

- Plugins now only need to export commands object
- Remove metadata validation from listPlugins
- Simplify plugin detection to check for c8ctl-plugin.js/ts file
- Update all tests to remove metadata checks
- Update documentation and examples

This simplifies plugin development by removing unnecessary boilerplate.

Author: vobu

EXAMPLES.md

@@ -386,7 +386,7 @@ c8 list plugins
 
 **Plugin Development:**
 
-Plugins must be regular Node.js modules with a `c8ctl-plugin.js` or `c8ctl-plugin.ts` file in the root directory. The plugin name must be in the package.json metadata. The `c8ctl` runtime object provides environment information:
+Plugins must be regular Node.js modules with a `c8ctl-plugin.js` or `c8ctl-plugin.ts` file in the root directory. The plugin file must export a `commands` object. The `c8ctl` runtime object provides environment information:
 
 ```typescript
 // c8ctl-plugin.ts
@@ -399,12 +399,6 @@ export const commands = {
     // Custom analysis logic here
   }
 };
-
-export const metadata = {
-  name: 'my-plugin',
-  version: '1.0.0',
-  description: 'My custom c8ctl plugin'
-};
 ```
 
 ---

README.md

@@ -143,8 +143,7 @@ c8 list plugins
 **Plugin Requirements:**
 - Plugin packages must be regular Node.js modules
 - They must include a `c8ctl-plugin.js` or `c8ctl-plugin.ts` file in the root directory
-- The plugin name must be in the package.json metadata
-- The plugin file should export custom commands
+- The plugin file must export a `commands` object
 - Plugins are installed in `node_modules` like regular npm packages
 - The runtime object `c8ctl` provides environment information to plugins
 
@@ -156,12 +155,6 @@ export const commands = {
     console.log('Analyzing...', args);
   }
 };
-
-export const metadata = {
-  name: 'my-plugin',
-  version: '1.0.0',
-  description: 'My custom c8ctl plugin'
-};
 ```
 
 ### Resource Aliases

src/commands/plugins.ts

@@ -4,7 +4,7 @@
 
 import { getLogger } from '../logger.ts';
 import { execSync } from 'node:child_process';
-import { readFileSync } from 'node:fs';
+import { readFileSync, existsSync } from 'node:fs';
 import { join } from 'node:path';
 
 /**
@@ -85,16 +85,13 @@ export function listPlugins(): void {
     for (const [name, version] of Object.entries(allDeps)) {
       try {
         // Try to resolve the package
-        const packagePath = join(process.cwd(), 'node_modules', name, 'package.json');
-        const pkgJson = JSON.parse(readFileSync(packagePath, 'utf-8'));
+        const packageDir = join(process.cwd(), 'node_modules', name);
 
-        // Check if package exports c8ctl-plugin.js or c8ctl-plugin.ts
-        const hasPlugin = pkgJson.main === 'c8ctl-plugin.js' || 
-                         pkgJson.main === 'c8ctl-plugin.ts' ||
-                         pkgJson.exports?.['./c8ctl-plugin.js'] ||
-                         pkgJson.exports?.['./c8ctl-plugin.ts'];
+        // Check if package has c8ctl-plugin.js or c8ctl-plugin.ts file in root
+        const hasPluginFile = existsSync(join(packageDir, 'c8ctl-plugin.js')) ||
+                             existsSync(join(packageDir, 'c8ctl-plugin.ts'));
 
-        if (hasPlugin) {
+        if (hasPluginFile) {
           plugins.push({
             Name: name,
             Version: version as string,

tests/fixtures/plugins/README.md

@@ -10,7 +10,6 @@ TypeScript implementation demonstrating:
 - TypeScript type annotations
 - Access to c8ctl runtime object
 - Multiple command exports (analyze, validate)
-- Metadata export
 
 **Usage in plugins:**
 ```typescript
@@ -29,7 +28,6 @@ JavaScript implementation demonstrating:
 - Command with hyphens ('deploy-all')
 - Argument handling
 - Multiple commands (deploy-all, status, report)
-- Metadata with command list
 
 **Usage in plugins:**
 ```javascript
@@ -45,17 +43,14 @@ export const commands = {
 For a package to be recognized as a c8ctl plugin:
 
 1. Must be a regular Node.js module with proper package.json
-2. Plugin name must be defined in package.json metadata
-3. Must have either `c8ctl-plugin.js` or `c8ctl-plugin.ts` file in root directory
-4. Must export a `commands` object with async functions
-5. Should export `metadata` with name, version, description
-6. Can access c8ctl runtime via `import { c8ctl } from 'c8ctl/runtime'`
+2. Must have either `c8ctl-plugin.js` or `c8ctl-plugin.ts` file in root directory
+3. Must export a `commands` object with async functions
+4. Can access c8ctl runtime via `import { c8ctl } from 'c8ctl/runtime'`
 
 ## Testing
 
 These fixtures are used by `tests/unit/plugins.test.ts` to verify:
 - Plugin structure validation
 - Command export format
-- Metadata format
 - Dynamic import capability
 - Runtime object access

tests/fixtures/plugins/c8ctl-plugin.js

@@ -37,10 +37,3 @@ export const commands = {
     console.log(`Output format: ${format}`);
   },
 };
-
-export const metadata = {
-  name: 'sample-js-plugin',
-  version: '2.0.0',
-  description: 'A sample JavaScript plugin for c8ctl',
-  commands: ['deploy-all', 'status', 'report'],
-};

tests/fixtures/plugins/c8ctl-plugin.ts

@@ -29,9 +29,3 @@ export const commands = {
     console.log(`Validating files: ${args.join(', ')}`);
   },
 };
-
-export const metadata = {
-  name: 'sample-ts-plugin',
-  version: '1.0.0',
-  description: 'A sample TypeScript plugin for c8ctl',
-};

tests/unit/plugins.test.ts

@@ -55,17 +55,6 @@ describe('Plugin Structure', () => {
       assert.ok(pluginContent.includes('c8ctl.env'), 'Plugin uses c8ctl.env');
     });
 
-    test('should have metadata export', async () => {
-      const pluginPath = join(process.cwd(), 'tests/fixtures/plugins/c8ctl-plugin.ts');
-      const pluginContent = readFileSync(pluginPath, 'utf-8');
-      
-      // Verify plugin exports metadata
-      assert.ok(pluginContent.includes('export const metadata'), 'Plugin exports metadata');
-      assert.ok(pluginContent.includes('name:'), 'Metadata has name');
-      assert.ok(pluginContent.includes('version:'), 'Metadata has version');
-      assert.ok(pluginContent.includes('description:'), 'Metadata has description');
-    });
-
     test('should be valid TypeScript syntax', async () => {
       const pluginPath = join(process.cwd(), 'tests/fixtures/plugins/c8ctl-plugin.ts');
       const pluginContent = readFileSync(pluginPath, 'utf-8');
@@ -90,17 +79,6 @@ describe('Plugin Structure', () => {
       assert.ok(pluginContent.includes('report:'), 'Plugin has report command');
     });
 
-    test('should have metadata export', async () => {
-      const pluginPath = join(process.cwd(), 'tests/fixtures/plugins/c8ctl-plugin.js');
-      const pluginContent = readFileSync(pluginPath, 'utf-8');
-      
-      // Verify plugin exports metadata
-      assert.ok(pluginContent.includes('export const metadata'), 'Plugin exports metadata');
-      assert.ok(pluginContent.includes('name:'), 'Metadata has name');
-      assert.ok(pluginContent.includes('version:'), 'Metadata has version');
-      assert.ok(pluginContent.includes('commands:'), 'Metadata lists commands');
-    });
-
     test('should use ES6 module syntax', async () => {
       const pluginPath = join(process.cwd(), 'tests/fixtures/plugins/c8ctl-plugin.js');
       const pluginContent = readFileSync(pluginPath, 'utf-8');
@@ -130,7 +108,6 @@ describe('Plugin Structure', () => {
         assert.ok(plugin.commands, 'Plugin has commands export');
         assert.ok(typeof plugin.commands.analyze === 'function', 'analyze is a function');
         assert.ok(typeof plugin.commands.validate === 'function', 'validate is a function');
-        assert.ok(plugin.metadata, 'Plugin has metadata export');
       } catch (error) {
         // If import fails, just verify the file exists
         const pluginPath = join(process.cwd(), 'tests/fixtures/plugins/c8ctl-plugin.ts');
@@ -149,7 +126,6 @@ describe('Plugin Structure', () => {
         assert.ok(typeof plugin.commands['deploy-all'] === 'function', 'deploy-all is a function');
         assert.ok(typeof plugin.commands.status === 'function', 'status is a function');
         assert.ok(typeof plugin.commands.report === 'function', 'report is a function');
-        assert.ok(plugin.metadata, 'Plugin has metadata export');
       } catch (error) {
         // If import fails, just verify the file exists
         const pluginPath = join(process.cwd(), 'tests/fixtures/plugins/c8ctl-plugin.js');