feat: add commands to remove services and plugins with options for confirmation and file retention

Author: meenu155

README.md

@@ -167,6 +167,8 @@ create-polyglot dev --docker
 | `create-polyglot init <name>` | Scaffold a new workspace (root invocation without `init` is deprecated). |
 | `create-polyglot add service <name>` | Add a service after init (`--type`, `--port`, `--yes`). |
 | `create-polyglot add plugin <name>` | Create plugin skeleton under `plugins/<name>`. |
+| `create-polyglot remove service <name>` | Remove a service from the workspace (`--keep-files`, `--yes`). |
+| `create-polyglot remove plugin <name>` | Remove a plugin from the workspace (`--keep-files`, `--yes`). |
 | `create-polyglot dev [--docker]` | Run Node & frontend services locally or all via compose. |
 
 ## Init Options
@@ -227,6 +229,50 @@ my-org/
 4. Edit services under `services/<name>`.
 5. Extend infra / databases inside `compose.yaml`.
 
+## Managing Services & Plugins
+
+### Adding Components
+Add services after initial scaffolding:
+```bash
+create-polyglot add service payments --type node --port 4100
+create-polyglot add plugin analytics
+```
+
+### Removing Components
+Clean removal of services and plugins:
+```bash
+# Remove a service (including files and configuration)
+create-polyglot remove service payments --yes
+
+# Remove a plugin  
+create-polyglot remove plugin analytics --yes
+
+# Keep files but remove from configuration
+create-polyglot remove service payments --keep-files --yes
+```
+
+The remove commands:
+- **Remove from configuration**: Updates `polyglot.json`, `compose.yaml`, and root `package.json` scripts
+- **Clean up files**: Removes service/plugin directories unless `--keep-files` is specified  
+- **Update dependencies**: Cleans up related logs and docker configurations
+- **Interactive confirmation**: Prompts for confirmation unless `--yes` is used
+- **Plugin management**: Also available via `create-polyglot plugin remove <name>`
+
+### Service Removal Details
+When removing a service:
+- Service entry removed from `polyglot.json`
+- Service definition removed from `compose.yaml`
+- Service directory deleted from `services/<name>` (unless `--keep-files`)
+- Related scripts removed from root `package.json`
+- Log files cleaned up
+
+### Plugin Removal Details
+When removing a plugin:
+- Plugin configuration removed from `polyglot.json`
+- Plugin directory deleted from `plugins/<name>` (unless `--keep-files`)
+- Plugin unloaded from the plugin system
+- External plugin references removed
+
 ### Basic Dev Runner
 When no preset is chosen, `npm run dev` uses `npx create-polyglot dev`:
 1. Detects package manager (pnpm > yarn > bun > npm fallback)
@@ -291,7 +337,6 @@ Generates ESLint + Prettier base configs at the root. Extend rules per service i
 - Automatic test harness & CI workflow template
 - Language-specific shared libs (Python package, Go module)
 - Hot reload integration aggregator
-- Remove service / remove plugin commands
 
 ## Contributing
 Contributions welcome! See `CONTRIBUTING.md` for guidelines. Please run tests before submitting a PR:
@@ -489,7 +534,6 @@ See [`docs/automated-release-notes.md`](docs/automated-release-notes.md) for det
 - Automatic test harness & CI workflow template
 - Language-specific shared libs (Python package, Go module)
 - Hot reload integration aggregator
-- Remove service / remove plugin commands
 
 ## License
 MIT

bin/index.js

@@ -2,7 +2,7 @@
 import { Command } from 'commander';
 import chalk from 'chalk';
 
-import { scaffoldMonorepo, addService, scaffoldPlugin } from './lib/scaffold.js';
+import { scaffoldMonorepo, addService, scaffoldPlugin, removeService, removePlugin } from './lib/scaffold.js';
 import fs from 'fs';
 import path from 'path';
 import { renderServicesTable } from './lib/ui.js';
@@ -90,14 +90,38 @@ program
       } else if (entity === 'plugin') {
         await scaffoldPlugin(projectDir, name);
       } else {
-        console.log(chalk.red(`Unknown entity '${entity}'. Use service or plugin.`));
+        console.error(chalk.red(`Unknown entity '${entity}'. Use service or plugin.`));
         process.exit(1);
       }
     } catch (e) {
       console.error(chalk.red('Failed to add:'), e.message);
       process.exit(1);
     }
   });
+
+program
+  .command('remove')
+  .description('Remove a service or plugin')
+  .argument('<entity>', 'service | plugin')
+  .argument('<name>', 'Name of the service or plugin')
+  .option('--keep-files', 'Keep service files, only remove from configuration')
+  .option('--yes', 'Skip confirmation prompt')
+  .action(async (entity, name, opts) => {
+    const projectDir = process.cwd();
+    try {
+      if (entity === 'service') {
+        await removeService(projectDir, name, opts);
+      } else if (entity === 'plugin') {
+        await removePlugin(projectDir, name, opts);
+      } else {
+        console.error(chalk.red(`Unknown entity '${entity}'. Use service or plugin.`));
+        process.exit(1);
+      }
+    } catch (e) {
+      console.error(chalk.red('Failed to remove:'), e.message);
+      process.exit(1);
+    }
+  });
 
 program
   .command('dev')
@@ -407,6 +431,23 @@ pluginCmd
     }
   });
 
+pluginCmd
+  .command('remove')
+  .description('Remove a plugin')
+  .argument('<name>', 'Plugin name')
+  .option('--keep-files', 'Keep plugin files, only remove from configuration')
+  .option('--yes', 'Skip confirmation prompt')
+  .action(async (name, opts) => {
+    try {
+      const { removePlugin } = await import('./lib/scaffold.js');
+      const cwd = process.cwd();
+      await removePlugin(cwd, name, opts);
+    } catch (e) {
+      console.error(chalk.red('Failed to remove plugin:'), e.message);
+      process.exit(1);
+    }
+  });
+
 pluginCmd
   .command('stats')
   .description('Show plugin system statistics')

bin/lib/plugin-system.js

@@ -128,25 +128,35 @@ class PluginSystem {
   }
 
   /**
-   * Discover local plugins in the .polyglot/plugins directory
+   * Discover local plugins in both .polyglot/plugins and plugins directories
    */
   async discoverLocalPlugins() {
-    const pluginsDir = path.join(this.projectDir, '.polyglot', 'plugins');
     const localPlugins = [];
-
-    if (await fs.pathExists(pluginsDir)) {
-      const entries = await fs.readdir(pluginsDir, { withFileTypes: true });
-      
-      for (const entry of entries) {
-        if (entry.isDirectory()) {
-          const pluginPath = path.join(pluginsDir, entry.name, 'index.js');
-          if (await fs.pathExists(pluginPath)) {
-            localPlugins.push({
-              name: entry.name,
-              type: 'local',
-              path: pluginPath,
-              enabled: this.config?.plugins?.[entry.name]?.enabled !== false
-            });
+    
+    // Check both .polyglot/plugins and plugins directories for backward compatibility
+    const pluginDirs = [
+      path.join(this.projectDir, '.polyglot', 'plugins'),
+      path.join(this.projectDir, 'plugins')
+    ];
+
+    for (const pluginsDir of pluginDirs) {
+      if (await fs.pathExists(pluginsDir)) {
+        const entries = await fs.readdir(pluginsDir, { withFileTypes: true });
+        
+        for (const entry of entries) {
+          if (entry.isDirectory()) {
+            const pluginPath = path.join(pluginsDir, entry.name, 'index.js');
+            if (await fs.pathExists(pluginPath)) {
+              // Only add if not already added from another directory
+              if (!localPlugins.find(p => p.name === entry.name)) {
+                localPlugins.push({
+                  name: entry.name,
+                  type: 'local',
+                  path: pluginPath,
+                  enabled: this.config?.plugins?.[entry.name]?.enabled !== false
+                });
+              }
+            }
           }
         }
       }