@entro314labs/ai-changelog-generator 3.2.1 → 3.6.0

package/src/infrastructure/providers/provider-management.service.js

@@ -1,453 +1,167 @@
-import process from 'node:process'
-
-import colors from '../../shared/constants/colors.js'
+import { ProviderManagerService } from './provider-manager.service.js'
 
 /**
- * Provider Management Service
+ * ProviderManagementService - Alias for ProviderManagerService
  *
- * Extracts the missing provider management methods from main class:
- * - listProviders() (40 lines) - Provider listing functionality
- * - switchProvider() (40 lines) - Provider switching functionality
- * - configureProvider() (80 lines) - Provider configuration
- * - validateModels() (95 lines) - Model validation functionality
+ * This service manages AI provider lifecycle, configuration, and operations
+ * including registration, activation, validation, and testing of providers.
  */
-export class ProviderManagementService {
-  constructor(providerManager) {
-    this.providerManager = providerManager
+export class ProviderManagementService extends ProviderManagerService {
+  constructor(config = {}, options = {}) {
+    super(config, options)
   }
 
-  async listProviders(includeCapabilities = false) {
-    console.log(colors.header('🔌 Available AI Providers:'))
-    console.log('')
-
-    try {
-      if (!this.providerManager) {
-        console.log(colors.errorMessage('Provider manager not available'))
-        return
-      }
-
-      const providers = this.providerManager.getAllProviders()
-      const activeProvider = this.providerManager.getActiveProvider()
-
-      providers.forEach((provider) => {
-        const isActive = activeProvider && provider.name === activeProvider.getName()
-        const status = provider.available ? '✅ Available' : '❌ Not configured'
-        const activeIndicator = isActive ? colors.success(' (ACTIVE)') : ''
-
-        console.log(`${colors.highlight(provider.name.toUpperCase())}${activeIndicator}`)
-        console.log(`   ${colors.label('Status')}: ${status}`)
-
-        if (includeCapabilities && provider.available && provider.capabilities) {
-          console.log(
-            `   ${colors.label('Capabilities')}: ${
-              Object.entries(provider.capabilities)
-                .filter(([_key, value]) => value === true)
-                .map(([key]) => key)
-                .join(', ') || 'Basic completion'
-            }`
-          )
-        }
-        console.log('')
-      })
-
-      console.log(colors.infoMessage(`Total providers: ${colors.number(providers.length)}`))
-      console.log(
-        colors.infoMessage(
-          `Active provider: ${colors.highlight(activeProvider?.getName() || 'None')}`
-        )
-      )
-
-      return {
-        providers,
-        activeProvider: activeProvider?.getName() || null,
-        total: providers.length,
-      }
-    } catch (error) {
-      console.error(colors.errorMessage(`Error listing providers: ${error.message}`))
-      throw error
-    }
+  /**
+   * Initialize the provider management service
+   * @returns {Promise<void>}
+   */
+  async initialize() {
+    await this.loadProviders()
+    await this.validateConfiguration()
   }
 
-  async switchProvider(providerName, testConnection = false) {
-    console.log(
-      colors.processingMessage(
-        `🔄 Switching to provider: ${colors.highlight(providerName.toUpperCase())}`
-      )
-    )
-
-    try {
-      if (!this.providerManager) {
-        console.log(colors.errorMessage('Provider manager not available'))
-        return { success: false, error: 'Provider manager not available' }
-      }
-
-      // Switch provider
-      const result = this.providerManager.switchProvider(providerName)
-
-      if (result.success) {
-        console.log(
-          colors.successMessage(
-            `✅ Successfully switched to ${colors.highlight(providerName.toUpperCase())}`
-          )
-        )
-
-        if (testConnection) {
-          console.log(colors.processingMessage('🧪 Testing connection...'))
-          const newProvider = this.providerManager.getActiveProvider()
-          const testResult = await newProvider.testConnection()
-
-          if (testResult.success) {
-            console.log(colors.successMessage('✅ Connection test passed'))
-            if (testResult.model) {
-              console.log(
-                colors.infoMessage(
-                  `   ${colors.label('Model')}: ${colors.highlight(testResult.model)}`
-                )
-              )
-            }
-            return { success: true, provider: providerName, connectionTest: testResult }
-          }
-          console.log(colors.errorMessage(`❌ Connection test failed: ${testResult.error}`))
-          return { success: true, provider: providerName, connectionTest: testResult }
-        }
-
-        return { success: true, provider: providerName }
-      }
-      console.log(colors.errorMessage(`❌ Failed to switch provider: ${result.error}`))
-      return { success: false, error: result.error }
-    } catch (error) {
-      console.error(colors.errorMessage(`Error switching provider: ${error.message}`))
-      return { success: false, error: error.message }
-    }
+  /**
+   * Get all available providers
+   * @returns {Array} Array of provider instances
+   */
+  getAllProviders() {
+    return this.providers
   }
 
-  async configureProvider(providerName = null, testConnection = false, showModels = false) {
-    const activeProvider = this.providerManager?.getActiveProvider()
-    const targetProvider = providerName || activeProvider?.getName()
-
-    if (!targetProvider) {
-      console.log(colors.errorMessage('No provider specified and no active provider found'))
-      return { success: false, error: 'No provider specified' }
-    }
-
-    console.log(
-      colors.header(`🔧 Configuring ${colors.highlight(targetProvider.toUpperCase())} Provider:`)
-    )
-    console.log('')
+  /**
+   * Get provider by name
+   * @param {string} name - Provider name
+   * @returns {Object|null} Provider instance or null
+   */
+  getProviderByName(name) {
+    return this.findProviderByName(name)
+  }
 
+  /**
+   * Register a new provider
+   * @param {string} name - Provider name
+   * @param {Object} providerInstance - Provider instance
+   * @returns {boolean} Success status
+   */
+  registerProvider(name, providerInstance) {
     try {
-      if (!this.providerManager) {
-        console.log(colors.errorMessage('Provider manager not available'))
-        return { success: false, error: 'Provider manager not available' }
-      }
-
-      const provider = this.providerManager.findProviderByName(targetProvider)
-
-      if (!provider) {
-        console.log(colors.errorMessage(`Provider '${targetProvider}' not found`))
-        return { success: false, error: `Provider '${targetProvider}' not found` }
-      }
-
-      // Display current configuration status
-      console.log(colors.subheader('📋 Current Configuration:'))
-      const config = provider.getConfiguration()
-
-      Object.entries(config).forEach(([key, value]) => {
-        const displayValue =
-          key.toLowerCase().includes('key') || key.toLowerCase().includes('secret')
-            ? value
-              ? '***CONFIGURED***'
-              : '❌ NOT SET'
-            : value || '❌ NOT SET'
-        console.log(`   ${colors.label(key)}: ${displayValue}`)
+      this.providers.push({
+        name,
+        instance: providerInstance,
+        available: true,
       })
-
-      // Show required environment variables
-      console.log(colors.subheader('\n🔑 Required Environment Variables:'))
-      const requiredVars = provider.getRequiredEnvVars()
-      requiredVars.forEach((varName) => {
-        const isSet = !!process.env[varName]
-        const status = isSet ? '✅ SET' : '❌ NOT SET'
-        console.log(`   ${colors.label(varName)}: ${status}`)
-      })
-
-      // Configuration instructions
-      console.log(colors.infoMessage('\n💡 Configuration Instructions:'))
-      console.log('   1. Set the required environment variables in your .env.local file')
-      console.log('   2. Or export them in your shell session')
-      console.log('   3. Run the configuration test to verify setup')
-
-      // Test connection if requested
-      if (testConnection) {
-        console.log(colors.processingMessage('\n🧪 Testing connection...'))
-        const testResult = await provider.testConnection()
-
-        if (testResult.success) {
-          console.log(colors.successMessage('✅ Configuration test passed'))
-          if (testResult.model) {
-            console.log(
-              colors.infoMessage(
-                `   ${colors.label('Default model')}: ${colors.highlight(testResult.model)}`
-              )
-            )
-          }
-        } else {
-          console.log(colors.errorMessage(`❌ Configuration test failed: ${testResult.error}`))
-          console.log(
-            colors.warningMessage('   Please check your environment variables and API keys')
-          )
-        }
-      }
-
-      // Show available models if requested
-      if (showModels) {
-        console.log(colors.processingMessage('\n🤖 Fetching available models...'))
-        try {
-          const models = await provider.getAvailableModels()
-          if (models && models.length > 0) {
-            console.log(colors.subheader('Available Models:'))
-            models.forEach((model) => {
-              console.log(`   - ${colors.highlight(model.name || model)}`)
-              if (model.description) {
-                console.log(`     ${colors.dim(model.description)}`)
-              }
-            })
-          } else {
-            console.log(colors.warningMessage('No models found or API error'))
-          }
-        } catch (error) {
-          console.log(colors.errorMessage(`Failed to fetch models: ${error.message}`))
-        }
-      }
-
-      return {
-        success: true,
-        provider: targetProvider,
-        configuration: config,
-        testResult: testConnection ? await provider.testConnection() : null,
-      }
+      return true
     } catch (error) {
-      console.error(colors.errorMessage(`Error configuring provider: ${error.message}`))
-      return { success: false, error: error.message }
+      console.error(`Failed to register provider ${name}:`, error.message)
+      return false
     }
   }
 
-  async validateModels(providerName = null, testModels = false, checkCapabilities = false) {
-    const activeProvider = this.providerManager?.getActiveProvider()
-    const targetProvider = providerName || activeProvider?.getName()
-
-    if (!targetProvider) {
-      console.log(colors.errorMessage('No provider specified and no active provider found'))
-      return { success: false, error: 'No provider specified' }
-    }
-
-    console.log(
-      colors.header(`🔍 Validating Models for ${colors.highlight(targetProvider.toUpperCase())}:`)
-    )
-    console.log('')
-
+  /**
+   * Activate a provider by name
+   * @param {string} name - Provider name to activate
+   * @returns {Promise<boolean>} Success status
+   */
+  async activateProvider(name) {
     try {
-      if (!this.providerManager) {
-        console.log(colors.errorMessage('Provider manager not available'))
-        return { success: false, error: 'Provider manager not available' }
-      }
-
-      const provider = this.providerManager.findProviderByName(targetProvider)
-
+      const provider = this.findProviderByName(name)
       if (!provider) {
-        console.log(colors.errorMessage(`Provider '${targetProvider}' not found`))
-        return { success: false, error: `Provider '${targetProvider}' not found` }
-      }
-
-      // Check if provider is configured
-      const isConfigured = provider.isAvailable()
-      if (!isConfigured) {
-        console.log(colors.errorMessage(`Provider '${targetProvider}' is not properly configured`))
-        return { success: false, error: 'Provider not configured' }
-      }
-
-      // Get available models
-      console.log(colors.processingMessage('📋 Fetching available models...'))
-      const models = await provider.getAvailableModels()
-
-      if (!models || models.length === 0) {
-        console.log(colors.warningMessage('No models found'))
-        return { success: true, models: [] }
-      }
-
-      console.log(colors.successMessage(`✅ Found ${colors.number(models.length)} models`))
-      console.log('')
-
-      const validationResults = []
-
-      // Display and optionally test each model
-      for (const model of models) {
-        const modelName = model.name || model
-        console.log(colors.subheader(`🤖 ${colors.highlight(modelName)}`))
-
-        if (model.description) {
-          console.log(`   ${colors.label('Description')}: ${colors.value(model.description)}`)
-        }
-
-        if (checkCapabilities && model.capabilities) {
-          console.log(
-            `   ${colors.label('Capabilities')}: ${Object.entries(model.capabilities)
-              .filter(([_key, value]) => value === true)
-              .map(([key]) => key)
-              .join(', ')}`
-          )
-        }
-
-        if (model.contextWindow) {
-          console.log(
-            `   ${colors.label('Context Window')}: ${colors.number(model.contextWindow)} tokens`
-          )
-        }
-
-        if (model.maxTokens) {
-          console.log(`   ${colors.label('Max Output')}: ${colors.number(model.maxTokens)} tokens`)
-        }
-
-        // Test model if requested
-        if (testModels) {
-          console.log(colors.processingMessage(`   🧪 Testing ${modelName}...`))
-          try {
-            const testResult = await provider.testModel(modelName)
-            if (testResult.success) {
-              console.log(colors.successMessage('   ✅ Model test passed'))
-              if (testResult.responseTime) {
-                console.log(
-                  `   ${colors.label('Response Time')}: ${colors.number(testResult.responseTime)}ms`
-                )
-              }
-            } else {
-              console.log(colors.errorMessage(`   ❌ Model test failed: ${testResult.error}`))
-            }
-            validationResults.push({ model: modelName, ...testResult })
-          } catch (error) {
-            console.log(colors.errorMessage(`   ❌ Model test error: ${error.message}`))
-            validationResults.push({ model: modelName, success: false, error: error.message })
-          }
-        } else {
-          validationResults.push({ model: modelName, success: true })
-        }
-
-        console.log('')
-      }
-
-      // Summary
-      if (testModels) {
-        const successfulTests = validationResults.filter((r) => r.success).length
-        const failedTests = validationResults.length - successfulTests
-
-        console.log(colors.subheader('📊 Test Summary:'))
-        console.log(`   ${colors.label('Successful')}: ${colors.successMessage(successfulTests)}`)
-        if (failedTests > 0) {
-          console.log(`   ${colors.label('Failed')}: ${colors.errorMessage(failedTests)}`)
-        }
+        throw new Error(`Provider '${name}' not found`)
       }
 
-      return {
-        success: true,
-        provider: targetProvider,
-        models,
-        validationResults: testModels ? validationResults : null,
-        summary: {
-          total: models.length,
-          tested: testModels ? validationResults.length : 0,
-          successful: testModels
-            ? validationResults.filter((r) => r.success).length
-            : models.length,
-        },
-      }
+      this.activeProvider = provider.instance
+      return true
     } catch (error) {
-      console.error(colors.errorMessage(`Error validating models: ${error.message}`))
-      return { success: false, error: error.message }
+      console.error(`Failed to activate provider ${name}:`, error.message)
+      return false
     }
   }
 
-  // Utility methods
-  getProviderStatus(providerName) {
+  /**
+   * Deactivate current provider
+   * @returns {boolean} Success status
+   */
+  deactivateProvider() {
     try {
-      const provider = this.providerManager?.findProviderByName(providerName)
-      if (!provider) {
-        return { available: false, error: 'Provider not found' }
-      }
-
-      return {
-        available: provider.isAvailable(),
-        name: provider.getName(),
-        configuration: provider.getConfiguration(),
-        requiredVars: provider.getRequiredEnvVars(),
-      }
+      this.activeProvider = null
+      return true
     } catch (error) {
-      return { available: false, error: error.message }
+      console.error('Failed to deactivate provider:', error.message)
+      return false
     }
   }
 
-  async getProviderCapabilities(providerName) {
-    try {
-      const provider = this.providerManager?.findProviderByName(providerName)
-      if (!provider) {
-        return { error: 'Provider not found' }
-      }
-
-      const capabilities = await provider.getCapabilities()
-      return { success: true, capabilities }
-    } catch (error) {
-      return { success: false, error: error.message }
+  /**
+   * Get provider lifecycle status
+   * @param {string} name - Provider name
+   * @returns {Object} Lifecycle status information
+   */
+  getProviderLifecycle(name) {
+    const provider = this.findProviderByName(name)
+    return {
+      name,
+      exists: !!provider,
+      available: provider?.available,
+      active: this.activeProvider === provider?.instance,
+      initialized: provider?.instance?.isInitialized?.(),
     }
   }
 
-  async testAllProviders() {
-    console.log(colors.header('🧪 Testing All Configured Providers:'))
-    console.log('')
-
-    const results = []
-
-    try {
-      const providers = this.providerManager.getAllProviders()
-      const configuredProviders = providers.filter((p) => p.available)
-
-      if (configuredProviders.length === 0) {
-        console.log(colors.warningMessage('No configured providers found'))
-        return { success: true, results: [] }
-      }
+  /**
+   * Get provider registry information
+   * @returns {Object} Registry information
+   */
+  getProviderRegistry() {
+    return {
+      total: this.providers.length,
+      active: this.activeProvider ? this.activeProvider.constructor.name : null,
+      available: this.providers.filter((p) => p.available).length,
+      providers: this.providers.map((p) => ({
+        name: p.name,
+        available: p.available,
+        type: p.instance.constructor.name,
+      })),
+    }
+  }
 
-      for (const provider of configuredProviders) {
-        console.log(
-          colors.processingMessage(`Testing ${colors.highlight(provider.name.toUpperCase())}...`)
-        )
+  /**
+   * Validate all provider configurations
+   * @returns {Promise<Object>} Validation results
+   */
+  async validateConfiguration() {
+    const results = {
+      valid: 0,
+      invalid: 0,
+      errors: [],
+    }
 
-        try {
-          const testResult = await provider.testConnection()
-          if (testResult.success) {
-            console.log(colors.successMessage(`✅ ${provider.name} - Connection successful`))
-          } else {
-            console.log(colors.errorMessage(`❌ ${provider.name} - ${testResult.error}`))
-          }
-          results.push({ provider: provider.name, ...testResult })
-        } catch (error) {
-          console.log(colors.errorMessage(`❌ ${provider.name} - ${error.message}`))
-          results.push({ provider: provider.name, success: false, error: error.message })
+    for (const provider of this.providers) {
+      try {
+        const isValid = await provider.instance.isAvailable?.()
+        if (isValid) {
+          results.valid++
+        } else {
+          results.invalid++
+          results.errors.push(`${provider.name}: Not available`)
         }
+      } catch (error) {
+        results.invalid++
+        results.errors.push(`${provider.name}: ${error.message}`)
       }
+    }
 
-      // Summary
-      const successful = results.filter((r) => r.success).length
-      const failed = results.length - successful
-
-      console.log(colors.subheader('\n📊 Test Results Summary:'))
-      console.log(`   ${colors.label('Total Providers')}: ${colors.number(results.length)}`)
-      console.log(`   ${colors.label('Successful')}: ${colors.successMessage(successful)}`)
-      if (failed > 0) {
-        console.log(`   ${colors.label('Failed')}: ${colors.errorMessage(failed)}`)
-      }
+    return results
+  }
 
-      return { success: true, results, summary: { total: results.length, successful, failed } }
-    } catch (error) {
-      console.error(colors.errorMessage(`Error testing providers: ${error.message}`))
-      return { success: false, error: error.message }
-    }
+  /**
+   * Clean up provider resources
+   * @returns {Promise<void>}
+   */
+  async cleanup() {
+    this.activeProvider = null
+    this.providers = []
   }
 }
+
+// Export as default for compatibility
+export default ProviderManagementService

package/src/infrastructure/providers/utils/base-provider-helpers.js

@@ -446,6 +446,17 @@ export function ConfigurationMixin(providerName, defaults = {}) {
       return buildClientOptions(providerConfig, { ...defaults, ...extraDefaults })
     },
 
+    getRequiredEnvVars() {
+      // Default implementation - providers should override if needed
+      const baseKey = `${providerName.toUpperCase()}_API_KEY`
+      return [baseKey]
+    },
+
+    getDefaultModel() {
+      const modelConfig = this.getProviderModelConfig()
+      return modelConfig.standardModel || 'unknown'
+    },
+
     getProviderInfo() {
       const providerConfig = this.getProviderConfig()
       const availableModels = this.getAvailableModels ? this.getAvailableModels() : []

package/src/shared/utils/cli-ui.js

@@ -223,8 +223,13 @@ export class EnhancedConsole {
     EnhancedConsole.log(message, 'ai')
   }
 
-  static metrics(message) {
-    EnhancedConsole.log(message, 'metrics')
+  static metrics(data, title = 'Metrics') {
+    if (typeof data === 'string') {
+      EnhancedConsole.log(data, 'metrics')
+    } else if (data && Object.keys(data).length > 0) {
+      EnhancedConsole.section(title)
+      console.log(colors.formatMetrics(data))
+    }
   }
 
   static section(title, content = '') {
@@ -252,13 +257,6 @@ export class EnhancedConsole {
     }
   }
 
-  static metrics(metrics, title = 'Metrics') {
-    if (metrics && Object.keys(metrics).length > 0) {
-      EnhancedConsole.section(title)
-      console.log(colors.formatMetrics(metrics))
-    }
-  }
-
   static divider(char = '─', length = 60) {
     console.log(colors.separator(char, length))
   }
@@ -438,7 +436,7 @@ export const cliUtils = {
   // Format bytes
   formatBytes(bytes) {
     const sizes = ['B', 'KB', 'MB', 'GB']
-    if (bytes === 0) {
+    if (bytes <= 0 || !Number.isFinite(bytes)) {
       return '0B'
     }
     const i = Math.floor(Math.log(bytes) / Math.log(1024))