vibecodingmachine-cli 2026.2.20-438 → 2026.2.26-1739

package/src/commands/agents/remove.js

@@ -0,0 +1,351 @@
+/**
+ * CLI Command: Remove Agent
+ * 
+ * Implements "app remove agent" command for CLI interface.
+ * Follows constitutional requirements: <555 lines, test-first approach.
+ */
+
+const AgentAdditionService = require('../../../core/src/agents/AgentAdditionService');
+const AgentConfigManager = require('../../../core/src/agents/config/AgentConfigManager');
+const AgentLogger = require('../../../core/src/agents/logging/AgentLogger');
+const FileManager = require('../../../core/src/agents/storage/FileManager');
+const AgentCommands = require('../../../core/src/rui/commands/AgentCommands');
+const AgentResponseFormatter = require('../../../core/src/rui/commands/AgentResponseFormatter');
+
+/**
+ * CLI remove agent command implementation
+ */
+class RemoveAgentCommand {
+  /**
+   * Create command instance
+   * @param {Object} options - Command options
+   */
+  constructor(options = {}) {
+    this.configManager = options.configManager || new AgentConfigManager();
+    this.logger = options.logger || new AgentLogger({
+      fileManager: new FileManager()
+    });
+    this.fileManager = options.fileManager || new FileManager();
+    
+    // Initialize addition service (handles both add and remove)
+    this.additionService = new AgentAdditionService({
+      configManager: this.configManager,
+      logger: this.logger
+    });
+
+    // Initialize RUI components
+    this.agentCommands = new AgentCommands({
+      configManager: this.configManager,
+      logger: this.logger,
+      fileManager: this.fileManager
+    });
+
+    this.responseFormatter = new AgentResponseFormatter({
+      defaultFormat: 'json',
+      includeTimestamp: true,
+      prettyPrint: options.pretty !== false
+    });
+  }
+
+  /**
+   * Execute remove agent command
+   * @param {Object} args - Command arguments
+   * @param {Object} options - Command options
+   * @returns {Object} - Command result
+   */
+  async execute(args = {}, options = {}) {
+    try {
+      const { agentId } = args;
+      const { force = false, format = 'json' } = options;
+
+      this.logger.info(`Executing remove agent command: ${agentId || 'unknown'}`);
+
+      // Validate agent ID
+      if (!agentId) {
+        const error = 'Agent ID is required';
+        this.logger.error(error);
+        return this.formatResponse({
+          success: false,
+          error
+        }, format);
+      }
+
+      // Check if agent exists
+      const agentExists = await this.additionService.getAgent(agentId);
+      if (!agentExists.success) {
+        const error = `Agent ${agentId} not found`;
+        this.logger.error(error);
+        return this.formatResponse({
+          success: false,
+          error
+        }, format);
+      }
+
+      // Check if agent is enabled (unless force is specified)
+      if (!force && agentExists.agent.enabled) {
+        const error = `Agent ${agentId} is currently enabled. Use --force to remove enabled agents.`;
+        this.logger.error(error);
+        return this.formatResponse({
+          success: false,
+          error,
+          agentId,
+          enabled: true,
+          suggestion: 'Use --force flag to remove enabled agents, or disable the agent first'
+        }, format);
+      }
+
+      // Remove agent
+      const result = await this.additionService.removeAgent(agentId);
+
+      // Log result
+      if (result.success) {
+        this.logger.info(`Successfully removed agent: ${result.agentId}`);
+      } else {
+        this.logger.error(`Failed to remove agent: ${result.error}`);
+      }
+
+      return this.formatResponse(result, format);
+
+    } catch (error) {
+      this.logger.error(`Remove agent command failed: ${error.message}`);
+      return this.formatResponse({
+        success: false,
+        error: error.message
+      }, options.format);
+    }
+  }
+
+  /**
+   * Remove multiple agents
+   * @param {string[]} agentIds - Array of agent IDs to remove
+   * @param {Object} options - Command options
+   * @returns {Object} - Command result
+   */
+  async removeMultiple(agentIds, options = {}) {
+    try {
+      this.logger.info(`Removing multiple agents: ${agentIds.join(', ')}`);
+
+      const results = [];
+      const summary = {
+        total: agentIds.length,
+        successful: 0,
+        failed: 0,
+        skipped: 0
+      };
+
+      for (const agentId of agentIds) {
+        const result = await this.execute({ agentId }, options);
+        results.push({ agentId, result });
+
+        if (result.success) {
+          summary.successful++;
+        } else if (result.error && result.error.includes('not found')) {
+          summary.skipped++;
+        } else {
+          summary.failed++;
+        }
+      }
+
+      return this.formatResponse({
+        success: summary.successful > 0,
+        results,
+        summary,
+        message: `Removed ${summary.successful} of ${summary.total} agents`
+      }, options.format);
+
+    } catch (error) {
+      this.logger.error(`Remove multiple agents failed: ${error.message}`);
+      return this.formatResponse({
+        success: false,
+        error: error.message
+      }, options.format);
+    }
+  }
+
+  /**
+   * Remove agents from file
+   * @param {string} filePath - Path to file containing agent IDs
+   * @param {Object} options - Command options
+   * @returns {Object} - Command result
+   */
+  async removeFromFile(filePath, options = {}) {
+    try {
+      this.logger.info(`Removing agents from file: ${filePath}`);
+
+      // Read file content
+      const fileContent = await this.fileManager.readFile(filePath);
+      const agentIds = JSON.parse(fileContent);
+
+      // Validate agent IDs array
+      if (!Array.isArray(agentIds)) {
+        throw new Error('File must contain an array of agent IDs');
+      }
+
+      // Remove multiple agents
+      return this.removeMultiple(agentIds, options);
+
+    } catch (error) {
+      this.logger.error(`Failed to remove agents from file: ${error.message}`);
+      return this.formatResponse({
+        success: false,
+        error: `Failed to read or parse file: ${error.message}`
+      }, options.format);
+    }
+  }
+
+  /**
+   * List removable agents
+   * @param {Object} options - Command options
+   * @returns {Object} - List of agents that can be removed
+   */
+  async listRemovable(options = {}) {
+    try {
+      this.logger.info('Listing removable agents');
+
+      const agentsResult = await this.additionService.getAgents();
+      
+      if (!agentsResult.success) {
+        return this.formatResponse({
+          success: false,
+          error: agentsResult.error
+        }, options.format);
+      }
+
+      const removableAgents = agentsResult.agents.filter(agent => {
+        // All agents are removable, but enabled ones require --force
+        return {
+          id: agent.id,
+          name: agent.name,
+          enabled: agent.enabled,
+          status: agent.status,
+          requiresForce: agent.enabled
+        };
+      });
+
+      return this.formatResponse({
+        success: true,
+        agents: removableAgents,
+        message: `Found ${removableAgents.length} agents that can be removed`
+      }, options.format);
+
+    } catch (error) {
+      this.logger.error(`Failed to list removable agents: ${error.message}`);
+      return this.formatResponse({
+        success: false,
+        error: error.message
+      }, options.format);
+    }
+  }
+
+  /**
+   * Confirm removal before executing
+   * @param {string} agentId - Agent ID to remove
+   * @param {Object} options - Command options
+   * @returns {Object} - Confirmation result
+   */
+  async confirmRemoval(agentId, options = {}) {
+    try {
+      this.logger.info(`Confirming removal for agent: ${agentId}`);
+
+      // Get agent details
+      const agentResult = await this.additionService.getAgent(agentId);
+      
+      if (!agentResult.success) {
+        return this.formatResponse({
+          success: false,
+          error: agentResult.error
+        }, options.format);
+      }
+
+      const agent = agentResult.agent;
+
+      return this.formatResponse({
+        success: true,
+        agent,
+        confirmation: {
+          message: `Are you sure you want to remove agent '${agent.name}' (${agent.id})?`,
+          details: {
+            id: agent.id,
+            name: agent.name,
+            status: agent.status,
+            enabled: agent.enabled,
+            lastChecked: agent.lastChecked,
+            lastVerified: agent.lastVerified
+          },
+          requiresForce: agent.enabled
+        }
+      }, options.format);
+
+    } catch (error) {
+      this.logger.error(`Failed to confirm removal: ${error.message}`);
+      return this.formatResponse({
+        success: false,
+        error: error.message
+      }, options.format);
+    }
+  }
+
+  /**
+   * Format response according to specified format
+   * @param {Object} data - Response data
+   * @param {string} format - Response format (json, table, etc.)
+   * @returns {Object} - Formatted response
+   */
+  formatResponse(data, format = 'json') {
+    return this.responseFormatter.format(data, format);
+  }
+
+  /**
+   * Get command help
+   * @returns {Object} - Help information
+   */
+  getHelp() {
+    return {
+      name: 'remove-agent',
+      description: 'Remove an agent from the system',
+      usage: [
+        'app remove agent <agent-id>',
+        'app remove agent <agent-id> --force',
+        'app remove agent --multiple <agent-id1,agent-id2>',
+        'app remove agent --file <path>',
+        'app remove agent --list-removable',
+        'app remove agent --confirm <agent-id>'
+      ],
+      options: {
+        '<agent-id>': 'ID of the agent to remove',
+        '--force': 'Remove even if agent is enabled',
+        '--multiple': 'Remove multiple agents (comma-separated IDs)',
+        '--file': 'Remove agents listed in file (JSON array)',
+        '--list-removable': 'List all agents that can be removed',
+        '--confirm': 'Show confirmation dialog before removal',
+        '--format': 'Output format (json, table)',
+        '--pretty': 'Pretty print JSON output'
+      },
+      examples: [
+        {
+          command: 'app remove agent claude-code',
+          description: 'Remove disabled agent claude-code'
+        },
+        {
+          command: 'app remove agent github-copilot --force',
+          description: 'Force remove enabled agent github-copilot'
+        },
+        {
+          command: 'app remove agent --multiple agent1,agent2,agent3',
+          description: 'Remove multiple agents'
+        },
+        {
+          command: 'app remove agent --confirm claude-code',
+          description: 'Show confirmation before removing agent'
+        }
+      ],
+      warnings: [
+        'Removing an agent will delete its configuration and history',
+        'Enabled agents require --force flag to remove',
+        'This action cannot be undone'
+      ]
+    };
+  }
+}
+
+module.exports = RemoveAgentCommand;