@entro314labs/ai-changelog-generator 3.2.1 → 3.6.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

@@ -5,15 +5,7 @@ import process from 'node:process'
 import yaml from 'js-yaml'
 
 import colors from '../../shared/constants/colors.js'
-
-// Model configurations - moved from lib/utils/model-config.js
-const MODEL_CONFIGS = {
-  'gpt-4': { maxTokens: 8192, cost: 0.03 },
-  'gpt-3.5-turbo': { maxTokens: 4096, cost: 0.002 },
-  'claude-3-opus': { maxTokens: 200000, cost: 0.015 },
-  'claude-3-sonnet': { maxTokens: 200000, cost: 0.003 },
-  'claude-3-haiku': { maxTokens: 200000, cost: 0.00025 },
-}
+import { MODEL_CONFIGS } from '../providers/utils/model-config.js'
 
 /**
  * Unified Configuration Manager
@@ -151,7 +143,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 +521,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 +571,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

@@ -9,6 +9,7 @@ import fs from 'node:fs'
 import path, { dirname } from 'node:path'
 import process from 'node:process'
 import { fileURLToPath } from 'node:url'
+import { execSync } from 'node:child_process'
 
 import { Server } from '@modelcontextprotocol/sdk/server/index.js'
 import { StdioServerTransport } from '@modelcontextprotocol/sdk/server/stdio.js'
@@ -18,9 +19,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)
@@ -61,12 +62,17 @@ class AIChangelogMCPServer {
       // Set MCP server mode to suppress verbose logging
       process.env.MCP_SERVER_MODE = 'true'
 
+      // Initialize configuration
       this.configManager = new ConfigurationManager()
+
+      // Initialize application service
       this.applicationService = new ApplicationService()
+
+      // Initialize orchestrator with proper dependencies (it will create its own services)
       this.changelogOrchestrator = new ChangelogOrchestrator(this.configManager)
-      this.gitAnalyzer = new GitRepositoryAnalyzer()
-      this.analysisEngine = new AnalysisEngine()
-      this.providerService = new ProviderManagementService()
+
+      // Initialize provider service
+      this.providerService = new ProviderManagerService(this.configManager.getAll())
 
       // Log available configuration
       const hasProvider = process.env.AI_PROVIDER
@@ -80,6 +86,9 @@ class AIChangelogMCPServer {
       } else {
         console.error(`[MCP] Configured with provider: ${process.env.AI_PROVIDER}`)
       }
+
+      // Wait for orchestrator to initialize its services
+      this.initPromise = this.changelogOrchestrator.ensureInitialized()
     } catch (error) {
       console.error('[MCP] Failed to initialize services:', error.message)
       console.error('[MCP] Server will start but tools may require configuration')
@@ -296,27 +305,55 @@ class AIChangelogMCPServer {
     } = args
 
     try {
+      // Ensure services are initialized
+      if (this.initPromise) {
+        await this.initPromise
+      }
+
       let result
 
       if (source === 'working-dir' || (source === 'auto' && this.hasWorkingDirectoryChanges())) {
-        // Generate from working directory changes
-        result = await this.changelogOrchestrator.generateWorkspaceChangelog({
-          version,
+        // Generate from working directory changes using ChangelogService
+        // Access the changelogService through the orchestrator
+        await this.changelogOrchestrator.ensureInitialized()
+        const changelogService = this.changelogOrchestrator.changelogService
+
+        result = await changelogService.generateWorkspaceChangelog(version, {
           analysisMode,
           includeAttribution,
         })
+
+        // Format result to match expected structure
+        if (result && result.changelog) {
+          result = {
+            content: result.changelog,
+            metadata: {
+              version,
+              source: 'working-directory',
+              filesProcessed: result.filesProcessed,
+              filesSkipped: result.filesSkipped,
+            }
+          }
+        }
       } else {
-        // Generate from commits
-        result = await this.changelogOrchestrator.generateChangelog({
-          version,
-          since,
-          analysisMode,
-          includeAttribution,
-        })
+        // Generate from commits - use orchestrator's generateChangelog(version, since)
+        result = await this.changelogOrchestrator.generateChangelog(version, since)
+
+        // Wrap result if needed
+        if (result && typeof result === 'string') {
+          result = {
+            content: result,
+            metadata: {
+              version,
+              since,
+              source: 'commits'
+            }
+          }
+        }
       }
 
       // Write file if requested
-      if (writeFile && result.content) {
+      if (writeFile && result && result.content) {
         const changelogPath = path.join(process.cwd(), 'AI_CHANGELOG.md')
         try {
           fs.writeFileSync(changelogPath, result.content, 'utf8')
@@ -330,10 +367,10 @@ class AIChangelogMCPServer {
         content: [
           {
             type: 'text',
-            text: result.content || 'No changelog content generated',
+            text: result?.content || 'No changelog content generated',
           },
         ],
-        metadata: result.metadata,
+        metadata: result?.metadata,
       }
     } catch (error) {
       throw new Error(`Changelog generation failed: ${error.message}`)
@@ -344,23 +381,30 @@ class AIChangelogMCPServer {
     const { analysisType = 'comprehensive', includeRecommendations = true, commitLimit = 50 } = args
 
     try {
+      // Ensure services are initialized
+      if (this.initPromise) {
+        await this.initPromise
+      }
+      await this.changelogOrchestrator.ensureInitialized()
+
       let result
 
       switch (analysisType) {
         case 'health':
-          result = await this.gitAnalyzer.assessRepositoryHealth(includeRecommendations)
+          result = await this.changelogOrchestrator.gitService.assessRepositoryHealth(includeRecommendations)
           break
         case 'commits':
-          result = await this.analysisEngine.analyzeRecentCommits(commitLimit)
+          result = await this.changelogOrchestrator.analysisEngine.analyzeRecentCommits(commitLimit)
           break
         case 'branches':
-          result = await this.gitAnalyzer.analyzeBranches()
+          result = await this.changelogOrchestrator.gitService.analyzeBranches()
           break
         case 'working-dir':
-          result = await this.analysisEngine.analyzeCurrentChanges()
+          result = await this.changelogOrchestrator.analysisEngine.analyzeCurrentChanges()
           break
         default:
-          result = await this.gitAnalyzer.analyzeComprehensive(includeRecommendations)
+          // Comprehensive analysis
+          result = await this.changelogOrchestrator.analyzeRepository({ type: analysisType })
           break
       }
 
@@ -381,7 +425,13 @@ class AIChangelogMCPServer {
     const { includeAIAnalysis = true, includeAttribution = true } = args
 
     try {
-      const result = await this.analysisEngine.analyzeCurrentChanges({
+      // Ensure services are initialized
+      if (this.initPromise) {
+        await this.initPromise
+      }
+      await this.changelogOrchestrator.ensureInitialized()
+
+      const result = await this.changelogOrchestrator.analysisEngine.analyzeCurrentChanges({
         includeAIAnalysis,
         includeAttribution,
       })
@@ -419,15 +469,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 +510,7 @@ class AIChangelogMCPServer {
         content: [
           {
             type: 'text',
-            text: result,
+            text: typeof result === 'string' ? result : JSON.stringify(result, null, 2),
           },
         ],
       }
@@ -448,7 +522,6 @@ class AIChangelogMCPServer {
   hasWorkingDirectoryChanges() {
     try {
       // Simple check for working directory changes
-      const { execSync } = require('node:child_process')
       const result = execSync('git status --porcelain', { encoding: 'utf8' })
       return result.trim().length > 0
     } catch (_error) {

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