@entro314labs/ai-changelog-generator 3.2.1 → 3.3.0

package/src/infrastructure/cli/cli.controller.js

@@ -83,6 +83,11 @@ export class CLIController {
   }
 
   async ensureConfig() {
+    // Skip config setup in test environments to prevent hanging
+    if (process.env.NODE_ENV === 'test' || process.env.CI) {
+      return
+    }
+
     const configPath = path.join(process.cwd(), '.env.local')
     try {
       await access(configPath)
@@ -345,17 +350,17 @@ class ValidateCommand extends BaseCommand {
     const validation = await appService.validateConfiguration()
 
     if (validation.valid) {
-      console.log(colors.successMessage('✅ Configuration is valid'))
+      EnhancedConsole.success('✅ Configuration is valid')
     } else {
-      console.log(colors.errorMessage('❌ Configuration has issues:'))
+      EnhancedConsole.error('❌ Configuration has issues:')
       validation.issues.forEach((issue) => {
-        console.log(`  - ${issue}`)
+        EnhancedConsole.log(`  - ${issue}`)
       })
 
       if (validation.recommendations.length > 0) {
-        console.log(colors.infoMessage('\n💡 Recommendations:'))
+        EnhancedConsole.info('\n💡 Recommendations:')
         validation.recommendations.forEach((rec) => {
-          console.log(`  - ${rec}`)
+          EnhancedConsole.log(`  - ${rec}`)
         })
       }
     }
@@ -554,7 +559,7 @@ class ProvidersCommand extends BaseCommand {
         colors.dim('\nUse "ai-changelog providers configure <provider>" to set up a provider')
       )
     } catch (error) {
-      console.error(colors.errorMessage(`Error listing providers: ${error.message}`))
+      EnhancedConsole.error(`Error listing providers: ${error.message}`)
     }
   }
 
@@ -577,7 +582,7 @@ class ProvidersCommand extends BaseCommand {
         )
       }
     } catch (error) {
-      console.error(colors.errorMessage(`Error switching provider: ${error.message}`))
+      EnhancedConsole.error(`Error switching provider: ${error.message}`)
     }
   }
 
@@ -637,7 +642,7 @@ class ProvidersCommand extends BaseCommand {
       console.log(colors.infoMessage('\nAfter adding the configuration, run:'))
       console.log(colors.highlight(`ai-changelog providers validate ${providerName}`))
     } catch (error) {
-      console.error(colors.errorMessage(`Error configuring provider: ${error.message}`))
+      EnhancedConsole.error(`Error configuring provider: ${error.message}`)
     }
   }
 
@@ -676,7 +681,7 @@ class ProvidersCommand extends BaseCommand {
         }
       }
     } catch (error) {
-      console.error(colors.errorMessage(`Error validating provider: ${error.message}`))
+      EnhancedConsole.error(`Error validating provider: ${error.message}`)
     }
   }
 }

package/src/infrastructure/config/configuration.manager.js

@@ -151,7 +151,7 @@ export class ConfigurationManager {
         releaseTagGlobPattern: 'v[0-9]*.[0-9]*.[0-9]*',
       },
       changelog: {
-        commitTypes: ['feat', 'fix', 'perf', 'refactor', 'docs'],
+        commitTypes: ['feat', 'fix', 'perf', 'refactor', 'docs', 'merge'],
         includeInvalidCommits: true,
         commitIgnoreRegexPattern: '^WIP ',
         headlines: {
@@ -529,7 +529,21 @@ export class ConfigurationManager {
   }
 
   getCommitTypes() {
-    return this.changelogConfig?.convention?.commitTypes || ['feat', 'fix', 'docs', 'style', 'refactor', 'perf', 'test', 'build', 'ci', 'chore', 'revert']
+    return (
+      this.changelogConfig?.convention?.commitTypes || [
+        'feat',
+        'fix',
+        'docs',
+        'style',
+        'refactor',
+        'perf',
+        'test',
+        'build',
+        'ci',
+        'chore',
+        'revert',
+      ]
+    )
   }
 
   getChangelogCommitTypes() {
@@ -565,4 +579,12 @@ export class ConfigurationManager {
     const pattern = this.changelogConfig.changelog.commitIgnoreRegexPattern
     return pattern ? new RegExp(pattern) : /^WIP /
   }
+
+  /**
+   * Get the complete configuration object
+   * @returns {Object} Configuration object
+   */
+  getConfig() {
+    return this.config || {}
+  }
 }

package/src/infrastructure/interactive/interactive-workflow.service.js

@@ -198,6 +198,10 @@ Provide 3 suggestions.`
     return summary
   }
 
+  async generateChangelogForCommitHashes(commitHashes) {
+    return await this.generateChangelogForCommits(commitHashes)
+  }
+
   async selectSpecificCommits() {
     return await selectSpecificCommits()
   }
@@ -207,7 +211,10 @@ Provide 3 suggestions.`
       const spinner = new SimpleSpinner(`Generating changelog for recent ${count} commits...`)
       spinner.start()
 
-      const commits = await this.gitService.getCommitsSince(null)
+      const commits =
+        (await this.gitService.getCommitsSince?.(null)) ||
+        (await this.gitService.getCommitAnalysis?.()) ||
+        []
       const recentCommits = commits.slice(0, count)
 
       if (recentCommits.length === 0) {

package/src/infrastructure/mcp/mcp-server.service.js

@@ -18,9 +18,9 @@ import { ChangelogOrchestrator } from '../../application/orchestrators/changelog
 // Import application services
 import { ApplicationService } from '../../application/services/application.service.js'
 import { AnalysisEngine } from '../../domains/analysis/analysis.engine.js'
-import { GitRepositoryAnalyzer } from '../../domains/git/git-repository.analyzer.js'
+import { GitService } from '../../domains/git/git.service.js'
 import { ConfigurationManager } from '../config/configuration.manager.js'
-import { ProviderManagementService } from '../providers/provider-management.service.js'
+import { ProviderManagerService } from '../providers/provider-manager.service.js'
 
 const __filename = fileURLToPath(import.meta.url)
 const __dirname = dirname(__filename)
@@ -64,9 +64,9 @@ class AIChangelogMCPServer {
       this.configManager = new ConfigurationManager()
       this.applicationService = new ApplicationService()
       this.changelogOrchestrator = new ChangelogOrchestrator(this.configManager)
-      this.gitAnalyzer = new GitRepositoryAnalyzer()
+      this.gitAnalyzer = new GitService()
       this.analysisEngine = new AnalysisEngine()
-      this.providerService = new ProviderManagementService()
+      this.providerService = new ProviderManagerService(this.configManager.getConfig())
 
       // Log available configuration
       const hasProvider = process.env.AI_PROVIDER
@@ -419,15 +419,39 @@ class AIChangelogMCPServer {
             result += `\n${testResult}`
           }
           break
-        case 'test':
-          result = await this.providerService.testCurrentProvider()
+        case 'test': {
+          const activeProvider = this.providerService.getActiveProvider()
+          if (!activeProvider) {
+            throw new Error('No active provider found')
+          }
+          result = await this.providerService.testProvider(activeProvider.getName())
           break
-        case 'configure':
-          result = await this.providerService.configureProvider(provider)
+        }
+        case 'configure': {
+          if (!provider) {
+            throw new Error('Provider required for configure action')
+          }
+          const providerData = this.providerService.findProviderByName(provider)
+          if (!providerData) {
+            throw new Error(`Provider '${provider}' not found`)
+          }
+          result = {
+            name: provider,
+            available: providerData.available,
+            configuration: providerData.instance.getConfiguration
+              ? providerData.instance.getConfiguration()
+              : {},
+            requiredVars: providerData.instance.getRequiredEnvVars
+              ? providerData.instance.getRequiredEnvVars()
+              : [],
+          }
           break
-        case 'validate':
-          result = await this.providerService.validateModels(provider)
+        }
+        case 'validate': {
+          const validationResults = await this.providerService.validateAll()
+          result = validationResults
           break
+        }
         default:
           throw new Error(`Unknown action: ${action}`)
       }
@@ -436,7 +460,7 @@ class AIChangelogMCPServer {
         content: [
           {
             type: 'text',
-            text: result,
+            text: typeof result === 'string' ? result : JSON.stringify(result, null, 2),
           },
         ],
       }

package/src/infrastructure/providers/core/base-provider.js

@@ -2,7 +2,7 @@
  * Abstract Base Provider for AI models.
  * Defines the interface that all provider plugins must implement.
  */
-import { AbstractMethodError } from '../../../shared/utils/utils.js'
+import { AbstractMethodError } from '../../../shared/utils/error-classes.js'
 
 export class BaseProvider {
   constructor(config) {

package/src/infrastructure/providers/implementations/anthropic.js

@@ -2,7 +2,6 @@ import Anthropic from '@anthropic-ai/sdk'
 
 import { BaseProvider } from '../core/base-provider.js'
 import { applyMixins, ProviderResponseHandler } from '../utils/base-provider-helpers.js'
-import { buildClientOptions } from '../utils/provider-utils.js'
 
 export class AnthropicProvider extends BaseProvider {
   constructor(config) {
@@ -14,7 +13,7 @@ export class AnthropicProvider extends BaseProvider {
   }
 
   initializeClient() {
-    const clientOptions = buildClientOptions(this.getProviderConfig(), {
+    const clientOptions = this.buildClientOptions({
       timeout: 60000,
       maxRetries: 2,
       defaultHeaders: {
@@ -24,7 +23,7 @@ export class AnthropicProvider extends BaseProvider {
     })
 
     this.anthropic = new Anthropic({
-      apiKey: clientOptions.apiKey,
+      apiKey: clientOptions.ANTHROPIC_API_KEY,
       baseURL: clientOptions.baseURL,
       timeout: clientOptions.timeout,
       maxRetries: clientOptions.maxRetries,
@@ -45,7 +44,8 @@ export class AnthropicProvider extends BaseProvider {
   }
 
   getDefaultModel() {
-    return 'claude-sonnet-4-20250514'
+    const modelConfig = this.getProviderModelConfig()
+    return modelConfig.standardModel
   }
 
   async generateCompletion(messages, options = {}) {
@@ -115,127 +115,32 @@ export class AnthropicProvider extends BaseProvider {
 
   async getAvailableModels() {
     // Anthropic doesn't provide a models endpoint, return known models
+    // Use direct model names to avoid circular dependency with mixins
     return [
-      {
-        name: 'claude-sonnet-4-20250514',
-        id: 'claude-sonnet-4-20250514',
-        description: 'Claude Sonnet 4 - Latest balanced model (2025)',
-        contextWindow: 200000,
-        capabilities: {
-          reasoning: true,
-          function_calling: true,
-          json_mode: true,
-          multimodal: true,
-          largeContext: true,
-          toolUse: true,
-        },
-      },
       {
         name: 'claude-opus-4-20250514',
         id: 'claude-opus-4-20250514',
         description: 'Claude Opus 4 - Most capable model for complex tasks (2025)',
-        contextWindow: 200000,
-        capabilities: {
-          reasoning: true,
-          function_calling: true,
-          json_mode: true,
-          multimodal: true,
-          largeContext: true,
-          toolUse: true,
-          advancedReasoning: true,
-        },
+      },
+      {
+        name: 'claude-sonnet-4-20250514',
+        id: 'claude-sonnet-4-20250514',
+        description: 'Claude Sonnet 4 - Latest balanced model (2025)',
       },
       {
         name: 'claude-3.7-sonnet-20250219',
         id: 'claude-3.7-sonnet-20250219',
         description: 'Claude 3.7 Sonnet - Enhanced reasoning capabilities',
-        contextWindow: 200000,
-        capabilities: {
-          reasoning: true,
-          function_calling: true,
-          json_mode: true,
-          multimodal: true,
-          largeContext: true,
-          toolUse: true,
-        },
-      },
-      {
-        name: 'claude-3-5-sonnet-20241022',
-        id: 'claude-3-5-sonnet-20241022',
-        description: 'Claude 3.5 Sonnet - Previous generation',
-        contextWindow: 200000,
-        capabilities: {
-          reasoning: true,
-          function_calling: true,
-          json_mode: true,
-          multimodal: true,
-        },
       },
       {
         name: 'claude-3-5-haiku-20241022',
         id: 'claude-3-5-haiku-20241022',
         description: 'Claude 3.5 Haiku - Fast and efficient',
-        contextWindow: 200000,
-        capabilities: {
-          reasoning: true,
-          function_calling: true,
-          json_mode: true,
-          multimodal: true,
-        },
       },
     ]
   }
 
-  async validateModelAvailability(modelName) {
-    try {
-      const models = await this.getAvailableModels()
-      const model = models.find((m) => m.name === modelName)
-
-      if (model) {
-        return {
-          available: true,
-          model: modelName,
-          capabilities: model.capabilities,
-          contextWindow: model.contextWindow,
-        }
-      }
-      const availableModels = models.map((m) => m.name)
-      return {
-        available: false,
-        error: `Model '${modelName}' not available`,
-        alternatives: availableModels.slice(0, 5),
-      }
-    } catch (error) {
-      return {
-        available: false,
-        error: error.message,
-        alternatives: [
-          'claude-sonnet-4-20250514',
-          'claude-opus-4-20250514',
-          'claude-3.7-sonnet-20250219',
-        ],
-      }
-    }
-  }
-
-  async testConnection() {
-    try {
-      const response = await this.generateCompletion([{ role: 'user', content: 'Hello' }], {
-        max_tokens: 5,
-      })
-
-      return {
-        success: true,
-        model: response.model,
-        message: 'Connection successful',
-      }
-    } catch (error) {
-      return {
-        success: false,
-        error: error.message,
-      }
-    }
-  }
+  // testConnection() and validateModelAvailability() now provided by mixins
 
   async testModel(modelName) {
     try {
@@ -258,73 +163,11 @@ export class AnthropicProvider extends BaseProvider {
     }
   }
 
-  getCapabilities(_modelName) {
-    return {
-      completion: true,
-      streaming: true,
-      function_calling: true,
-      json_mode: true,
-      reasoning: true,
-      multimodal: true,
-      large_context: true,
-    }
-  }
-
-  getModelRecommendation(commitDetails) {
-    const { files = 0, lines = 0, breaking = false, complex = false } = commitDetails
-
-    // Use the most capable model for complex or breaking changes
-    if (breaking || complex || files > 20 || lines > 1000) {
-      return {
-        model: 'claude-3-5-sonnet-20241022',
-        reason: 'Complex or breaking change requiring advanced reasoning',
-      }
-    }
-
-    // Use standard model for medium changes
-    if (files > 5 || lines > 200) {
-      return {
-        model: 'claude-3-sonnet-20240229',
-        reason: 'Medium-sized change requiring good analysis',
-      }
-    }
-
-    // Use efficient model for small changes
-    return {
-      model: 'claude-3-haiku-20240307',
-      reason: 'Small change, optimized for efficiency',
-    }
-  }
-
-  async selectOptimalModel(commitInfo) {
-    const recommendation = this.getModelRecommendation(commitInfo)
-    const validation = await this.validateModelAvailability(recommendation.model)
-
-    if (validation.available) {
-      return {
-        model: recommendation.model,
-        reason: recommendation.reason,
-        capabilities: validation.capabilities,
-      }
-    }
-    return {
-      model: this.getDefaultModel(),
-      reason: 'Fallback to default model',
-      capabilities: this.getCapabilities(this.getDefaultModel()),
-    }
-  }
-
-  getProviderModelConfig() {
-    return {
-      smallModel: 'claude-3-5-haiku-20241022',
-      mediumModel: 'claude-3.7-sonnet-20250219',
-      standardModel: 'claude-sonnet-4-20250514',
-      complexModel: 'claude-opus-4-20250514',
-      default: 'claude-sonnet-4-20250514',
-      temperature: 0.3,
-      maxTokens: 4096,
-    }
-  }
+  // All common methods now provided by mixins:
+  // - getCapabilities() from CapabilitiesMixin
+  // - getModelRecommendation() from ModelRecommendationMixin
+  // - selectOptimalModel() from ModelRecommendationMixin
+  // - getProviderModelConfig() from ConfigurationMixin
 }
 
 // Apply mixins to add standard provider functionality

package/src/infrastructure/providers/implementations/azure.js

@@ -3,6 +3,7 @@ import { AzureOpenAI } from 'openai'
 
 import { BaseProvider } from '../core/base-provider.js'
 import { applyMixins, ProviderResponseHandler } from '../utils/base-provider-helpers.js'
+import { getProviderModelConfig } from '../utils/model-config.js'
 
 class AzureOpenAIProvider extends BaseProvider {
   constructor(config) {
@@ -168,59 +169,11 @@ class AzureOpenAIProvider extends BaseProvider {
 
   // Azure-specific helper methods
   getDeploymentName() {
-    return this.config.AZURE_OPENAI_DEPLOYMENT_NAME || this.getProviderModelConfig().standardModel
+    const modelConfig = this.getProviderModelConfig()
+    return this.config.AZURE_OPENAI_DEPLOYMENT_NAME || modelConfig.standardModel
   }
 
-  getModelContextWindow(modelName) {
-    const contextWindows = {
-      // Latest 2025 models (Azure exclusive)
-      o4: 500000,
-      'o4-mini': 200000,
-      o3: 300000,
-      'o3-mini': 150000,
-      // Standard 2025 models
-      'gpt-4o': 128000,
-      'gpt-4o-mini': 128000,
-      o1: 200000,
-      'o1-mini': 128000,
-      'gpt-4.1': 200000,
-      'gpt-4.1-mini': 200000,
-      'gpt-4.1-nano': 200000,
-      // Legacy models
-      'gpt-4': 8192,
-      'gpt-4-32k': 32768,
-      'gpt-4-turbo': 128000,
-      'gpt-35-turbo': 4096,
-      'gpt-35-turbo-16k': 16384,
-    }
-    return contextWindows[modelName] || 128000
-  }
-
-  getModelCapabilities(modelName) {
-    return {
-      reasoning:
-        modelName.includes('o1') ||
-        modelName.includes('o3') ||
-        modelName.includes('o4') ||
-        modelName.includes('gpt-4'),
-      function_calling: !(
-        modelName.includes('o1') ||
-        modelName.includes('o3') ||
-        modelName.includes('o4')
-      ), // o-series models don't support function calling
-      json_mode: true,
-      multimodal: modelName.includes('gpt-4o') || modelName.includes('gpt-4.1'),
-      largeContext:
-        modelName.includes('4.1') ||
-        modelName.includes('o1') ||
-        modelName.includes('o3') ||
-        modelName.includes('o4') ||
-        modelName.includes('4o'),
-      promptCaching: modelName.includes('4.1'),
-      advancedReasoning: modelName.includes('o3') || modelName.includes('o4'),
-      azureExclusive: modelName.includes('o3') || modelName.includes('o4'),
-    }
-  }
+  // Model capabilities now handled by centralized CapabilitiesMixin
 
   // Azure-specific method for testing deployment availability
   async testDeployment(deploymentName) {
@@ -257,19 +210,19 @@ class AzureOpenAIProvider extends BaseProvider {
       return this._cachedDeployments
     }
 
-    // Get base config directly to avoid recursion
-    const baseConfig = {
-      commonDeployments: ['o4', 'o3', 'gpt-4.1', 'gpt-4o', 'gpt-35-turbo', 'o1'],
-      fallbacks: ['gpt-4.1', 'gpt-4o', 'o1', 'gpt-35-turbo'],
-    }
+    // Get model config from centralized system (avoiding circular dependency)
+    const modelConfig = getProviderModelConfig('azure', this.config)
 
     const potentialDeployments = [
       // User configured deployment
       this.config.AZURE_OPENAI_DEPLOYMENT_NAME,
-      // Common deployment names
-      ...baseConfig.commonDeployments,
+      // Models from config
+      modelConfig.complexModel,
+      modelConfig.standardModel,
+      modelConfig.mediumModel,
+      modelConfig.smallModel,
       // Fallback models
-      ...baseConfig.fallbacks,
+      ...modelConfig.fallbacks,
     ]
       .filter(Boolean)
       .filter((v, i, a) => a.indexOf(v) === i) // Remove duplicates
@@ -294,16 +247,16 @@ class AzureOpenAIProvider extends BaseProvider {
 
       if (availableDeployments.length === 0) {
         console.warn('⚠️  No Azure deployments found. Using configured deployment name as fallback.')
-        // Fallback to configured deployment even if untested, preferring latest models
-        const fallback = this.config.AZURE_OPENAI_DEPLOYMENT_NAME || 'o4' || 'gpt-4.1'
+        // Fallback to configured deployment even if untested
+        const fallback = this.config.AZURE_OPENAI_DEPLOYMENT_NAME || modelConfig.standardModel
         return [fallback]
       }
 
       return availableDeployments
     } catch (error) {
       console.warn('⚠️  Failed to detect Azure deployments:', error.message)
-      // Return common deployments as fallback
-      return baseConfig.commonDeployments
+      // Return fallback models from config
+      return modelConfig.fallbacks
     }
   }
 

package/src/infrastructure/providers/implementations/dummy.js

@@ -3,8 +3,9 @@
  * Fallback provider when no other providers are available
  */
 
-import { ProviderError } from '../../../shared/utils/utils.js'
+import { ProviderError } from '../../../shared/utils/error-classes.js'
 import { BaseProvider } from '../core/base-provider.js'
+import { applyMixins } from '../utils/base-provider-helpers.js'
 
 class DummyProvider extends BaseProvider {
   constructor(config = {}) {
@@ -12,22 +13,22 @@ class DummyProvider extends BaseProvider {
     this.name = 'dummy'
   }
 
-  /**
-   * Get provider name
-   * @returns {string} Provider name
-   */
   getName() {
     return this.name
   }
 
-  /**
-   * Check if provider is available
-   * @returns {boolean} Always true for dummy provider
-   */
   isAvailable() {
     return true
   }
 
+  getRequiredEnvVars() {
+    return []
+  }
+
+  getDefaultModel() {
+    return 'rule-based'
+  }
+
   /**
    * Generate completion (always fails with informative error)
    * @param {Array} messages - Messages for completion
@@ -93,20 +94,16 @@ class DummyProvider extends BaseProvider {
     }
   }
 
-  getAvailableModels() {
+  async getAvailableModels() {
     return [
       {
         id: 'rule-based',
         name: 'Rule-based Fallback',
-        contextWindow: 0,
-        maxOutput: 0,
-        inputCost: 0,
-        outputCost: 0,
-        features: [],
         description: 'No AI model available - configure a provider',
       },
     ]
   }
 }
 
-export default DummyProvider
+// Apply minimal mixins (error handling only - dummy provider doesn't need full functionality)
+export default applyMixins ? applyMixins(DummyProvider, 'dummy', []) : DummyProvider