@entro314labs/ai-changelog-generator 3.7.1 → 3.8.1

package/src/domains/git/git.service.js

@@ -41,7 +41,8 @@ export class GitService {
             .trim()
             .split(' ')
           isMergeCommit = parents.length > 1
-        } catch {
+        } catch (error) {
+          console.warn(colors.warningMessage(`Failed to check parent commits: ${error.message}`))
           isMergeCommit = false
         }
       }
@@ -203,7 +204,8 @@ export class GitService {
           } else {
             diff = 'New empty file created'
           }
-        } catch {
+        } catch (error) {
+          console.warn(colors.warningMessage(`Failed to read new file content: ${error.message}`))
           diff = 'New file created (binary or inaccessible)'
         }
       } else if (status === 'D') {
@@ -217,7 +219,10 @@ export class GitService {
           } else {
             diff = 'File deleted from working directory (content was empty)'
           }
-        } catch {
+        } catch (error) {
+          console.warn(
+            colors.warningMessage(`Failed to read deleted file content: ${error.message}`)
+          )
           diff = 'File deleted from working directory (content unavailable)'
         }
       } else if (status === 'M' || status.includes('M')) {
@@ -230,7 +235,8 @@ export class GitService {
             const stagedDiff = this.gitManager.execGitSafe(`git diff --cached -- "${filePath}"`)
             diff = stagedDiff || 'No diff available (binary or identical)'
           }
-        } catch {
+        } catch (error) {
+          console.warn(colors.warningMessage(`Failed to generate diff: ${error.message}`))
           diff = 'Modified file (diff unavailable)'
         }
       } else if (status === 'R') {
@@ -250,7 +256,8 @@ export class GitService {
         try {
           const headResult = this.gitManager.execGitSafe(`git show HEAD:"${filePath}"`)
           beforeContent = headResult ? headResult.slice(0, 1000) : ''
-        } catch {
+        } catch (error) {
+          console.warn(colors.warningMessage(`Failed to get before content: ${error.message}`))
           beforeContent = ''
         }
         // afterContent stays empty for deleted files
@@ -260,7 +267,10 @@ export class GitService {
           // Get HEAD version
           const headResult = this.gitManager.execGitSafe(`git show HEAD:"${filePath}"`)
           beforeContent = headResult ? headResult.slice(0, 1000) : ''
-        } catch {
+        } catch (error) {
+          console.warn(
+            colors.warningMessage(`Failed to get before content (modified): ${error.message}`)
+          )
           beforeContent = ''
         }
 
@@ -268,7 +278,8 @@ export class GitService {
           // Get current working directory version
           const currentResult = this.gitManager.execGitSafe(`cat "${filePath}"`)
           afterContent = currentResult ? currentResult.slice(0, 1000) : ''
-        } catch {
+        } catch (error) {
+          console.warn(colors.warningMessage(`Failed to get after content: ${error.message}`))
           afterContent = ''
         }
       }
@@ -301,7 +312,11 @@ export class GitService {
         language: 'unknown',
         importance: 'medium',
         complexity: { score: 1 },
-        semanticChanges: { changeType: 'unknown', patterns: [], frameworks: [] },
+        semanticChanges: {
+          changeType: 'unknown',
+          patterns: [],
+          frameworks: [],
+        },
         functionalImpact: { scope: 'local', severity: 'low' },
       }
     }
@@ -328,7 +343,8 @@ export class GitService {
       }
 
       return { files: 0, insertions: 0, deletions: 0 }
-    } catch {
+    } catch (error) {
+      console.warn(colors.warningMessage(`Failed to get diff stats: ${error.message}`))
       return { files: 0, insertions: 0, deletions: 0 }
     }
   }
@@ -420,7 +436,9 @@ export class GitService {
           category: categorizeFile(filePath),
           language: detectLanguage(filePath),
           importance: assessFileImportance(filePath, status),
-          complexity: { level: changes > 100 ? 'high' : changes > 20 ? 'medium' : 'low' },
+          complexity: {
+            level: changes > 100 ? 'high' : changes > 20 ? 'medium' : 'low',
+          },
           semanticChanges: { patterns: ['merge-commit'] },
           functionalImpact: { level: changes > 50 ? 'high' : 'medium' },
           isMergeCommit: true,
@@ -662,7 +680,9 @@ export class GitService {
             details.push(techDetail)
           }
         }
-      } catch (_error) {}
+      } catch (error) {
+        console.warn(colors.warningMessage(`Failed to extract technical details: ${error.message}`))
+      }
     }
 
     return details
@@ -830,7 +850,10 @@ export class GitService {
           .filter((line) => line.trim())
           .map((line) => {
             const parts = line.split(' ')
-            return { size: parseInt(parts[1], 10), path: parts.slice(2).join(' ') }
+            return {
+              size: parseInt(parts[1], 10),
+              path: parts.slice(2).join(' '),
+            }
           })
           .filter((file) => file.size > 10 * 1024 * 1024) // 10MB threshold
 
@@ -1101,7 +1124,7 @@ ${
         )
         if (firstCommitOutput.trim()) {
           const firstCommitDate = new Date(firstCommitOutput.trim())
-          const ageInDays = Math.floor((new Date() - firstCommitDate) / (1000 * 60 * 60 * 24))
+          const ageInDays = Math.floor((Date.now() - firstCommitDate) / (1000 * 60 * 60 * 24))
           analysis.metrics.repositoryAgeInDays = ageInDays
         }
       } catch (error) {

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

@@ -306,7 +306,10 @@ export class CLIController {
       .option('output', { alias: 'o', type: 'string', description: 'Output file path' })
       .option('since', { type: 'string', description: 'Analyze changes since this git ref' })
       .option('author', { alias: 'a', type: 'string', description: 'Filter commits by author' })
-      .option('tag-range', { type: 'string', description: 'Generate changelog between tags (v1.0..v2.0)' })
+      .option('tag-range', {
+        type: 'string',
+        description: 'Generate changelog between tags (v1.0..v2.0)',
+      })
       .option('silent', { type: 'boolean', description: 'Suppress non-essential output' })
       .option('dry-run', { type: 'boolean', description: 'Preview without writing files' })
       .option('detailed', { type: 'boolean', description: 'Use detailed analysis mode' })
@@ -544,35 +547,33 @@ class WorkingDirCommand extends BaseCommand {
 class FromCommitsCommand extends BaseCommand {
   async execute(argv, appService) {
     const _config = this.processStandardFlags(argv, appService)
-    console.log(colors.processingMessage(`🔍 Generating changelog from commits: ${argv.commits.join(', ')}`))
+    EnhancedConsole.processing(`Generating changelog from commits: ${argv.commits.join(', ')}`)
 
     try {
       const result = await appService.generateChangelogFromCommits(argv.commits)
 
       if (result?.changelog) {
-        console.log(colors.successMessage('\n✅ Changelog generated successfully!'))
-        console.log(colors.dim('─'.repeat(50)))
+        EnhancedConsole.success('Changelog generated successfully!')
+        EnhancedConsole.divider()
         console.log(result.changelog)
       } else {
-        console.log(colors.warningMessage('No changelog could be generated from the specified commits.'))
+        EnhancedConsole.warn('No changelog could be generated from the specified commits.')
       }
     } catch (error) {
-      console.error(colors.errorMessage(`Error generating changelog: ${error.message}`))
+      EnhancedConsole.error(`Error generating changelog: ${error.message}`)
     }
   }
 }
 
 class CommitMessageCommand extends BaseCommand {
   async execute(_argv, appService) {
-    console.log(
-      colors.processingMessage('🤖 Analyzing current changes for commit message suggestions...')
-    )
+    EnhancedConsole.processing('Analyzing current changes for commit message suggestions...')
 
     try {
       const result = await appService.generateCommitMessage()
 
       if (result?.suggestions && result.suggestions.length > 0) {
-        console.log(colors.successMessage('\n✅ Generated commit message suggestions:'))
+        EnhancedConsole.success('Generated commit message suggestions:')
         result.suggestions.forEach((suggestion, index) => {
           console.log(`${colors.number(index + 1)}. ${colors.highlight(suggestion)}`)
         })
@@ -581,18 +582,18 @@ class CommitMessageCommand extends BaseCommand {
           console.log(colors.dim(`\nContext: ${result.context}`))
         }
       } else {
-        console.log(colors.warningMessage('No commit message suggestions could be generated.'))
-        console.log(colors.infoMessage('Make sure you have uncommitted changes.'))
+        EnhancedConsole.warn('No commit message suggestions could be generated.')
+        EnhancedConsole.info('Make sure you have uncommitted changes.')
       }
     } catch (error) {
-      console.error(colors.errorMessage(`Error generating commit message: ${error.message}`))
+      EnhancedConsole.error(`Error generating commit message: ${error.message}`)
     }
   }
 }
 
 class CommitCommand extends BaseCommand {
   async execute(argv, appService) {
-    console.log(colors.processingMessage('🚀 Starting interactive commit workflow...'))
+    EnhancedConsole.processing('Starting interactive commit workflow...')
 
     try {
       // Process flags and model override
@@ -611,29 +612,25 @@ class CommitCommand extends BaseCommand {
 
       if (result?.success) {
         if (argv.dryRun) {
-          console.log(colors.successMessage('✅ Commit workflow completed (dry-run mode)'))
+          EnhancedConsole.success('Commit workflow completed (dry-run mode)')
           console.log(colors.highlight(`Proposed commit message:\n${result.commitMessage}`))
         } else {
-          console.log(colors.successMessage('✅ Changes committed successfully!'))
+          EnhancedConsole.success('Changes committed successfully!')
           console.log(colors.highlight(`Commit: ${result.commitHash}`))
           console.log(colors.dim(`Message: ${result.commitMessage}`))
         }
       } else {
-        console.log(colors.warningMessage('Commit workflow cancelled or no changes to commit.'))
+        EnhancedConsole.warn('Commit workflow cancelled or no changes to commit.')
       }
     } catch (error) {
-      console.error(colors.errorMessage(`Commit workflow failed: ${error.message}`))
+      EnhancedConsole.error(`Commit workflow failed: ${error.message}`)
 
       // Provide helpful suggestions based on error type
       if (error.message.includes('No changes')) {
-        console.log(
-          colors.infoMessage('💡 Try making some changes first, then run the commit command.')
-        )
+        EnhancedConsole.info('Try making some changes first, then run the commit command.')
       } else if (error.message.includes('git')) {
-        console.log(
-          colors.infoMessage(
-            '💡 Make sure you are in a git repository and git is properly configured.'
-          )
+        EnhancedConsole.info(
+          'Make sure you have uncommitted changes.'
         )
       }
     }
@@ -664,8 +661,8 @@ class ProvidersCommand extends BaseCommand {
         await this.listModels(appService, argv.provider)
         break
       default:
-        console.log(colors.errorMessage('Unknown provider subcommand'))
-        console.log(colors.infoMessage('Available subcommands: list, switch, configure, validate, status, models'))
+        EnhancedConsole.error('Unknown provider subcommand')
+        EnhancedConsole.info('Available subcommands: list, switch, configure, validate, status, models')
     }
   }
 
@@ -702,8 +699,8 @@ class ProvidersCommand extends BaseCommand {
 
   async switchProvider(appService, providerName) {
     if (!providerName) {
-      console.log(colors.errorMessage('Please specify a provider name'))
-      console.log(colors.infoMessage('Usage: ai-changelog providers switch <provider>'))
+      EnhancedConsole.error('Please specify a provider name')
+      EnhancedConsole.info('Usage: ai-changelog providers switch <provider>')
       return
     }
 
@@ -711,12 +708,10 @@ class ProvidersCommand extends BaseCommand {
       const result = await appService.switchProvider(providerName)
 
       if (result.success) {
-        console.log(colors.successMessage(`✅ Switched to ${providerName} provider`))
+        EnhancedConsole.success(`Switched to ${providerName} provider`)
       } else {
-        console.log(colors.errorMessage(`❌ Failed to switch provider: ${result.error}`))
-        console.log(
-          colors.infoMessage('Use "ai-changelog providers list" to see available providers')
-        )
+        EnhancedConsole.error(`Failed to switch provider: ${result.error}`)
+        EnhancedConsole.info('Use "ai-changelog providers list" to see available providers')
       }
     } catch (error) {
       EnhancedConsole.error(`Error switching provider: ${error.message}`)
@@ -743,7 +738,7 @@ class ProvidersCommand extends BaseCommand {
       }
 
       console.log(colors.header(`\n🔧 Configuring ${providerName.toUpperCase()} Provider`))
-      console.log(colors.infoMessage('Please add the following to your .env.local file:\n'))
+      EnhancedConsole.info('Please add the following to your .env.local file:\n')
 
       switch (providerName.toLowerCase()) {
         case 'openai':
@@ -776,7 +771,7 @@ class ProvidersCommand extends BaseCommand {
           console.log(colors.code(`${providerName.toUpperCase()}_API_KEY=your_api_key_here`))
       }
 
-      console.log(colors.infoMessage('\nAfter adding the configuration, run:'))
+      EnhancedConsole.info('\nAfter adding the configuration, run:')
       console.log(colors.highlight(`ai-changelog providers validate ${providerName}`))
     } catch (error) {
       EnhancedConsole.error(`Error configuring provider: ${error.message}`)
@@ -895,7 +890,9 @@ class ProvidersCommand extends BaseCommand {
 
       // Summary
       const healthy = healthResults.filter((r) => r.status === 'healthy').length
-      const unhealthy = healthResults.filter((r) => ['unhealthy', 'error'].includes(r.status)).length
+      const unhealthy = healthResults.filter((r) =>
+        ['unhealthy', 'error'].includes(r.status)
+      ).length
       const unconfigured = healthResults.filter((r) => r.status === 'unconfigured').length
 
       console.log(colors.dim('─'.repeat(40)))
@@ -918,7 +915,9 @@ class ProvidersCommand extends BaseCommand {
         const provider = providers.find((p) => p.name.toLowerCase() === providerName.toLowerCase())
         if (!provider) {
           console.log(colors.errorMessage(`Provider '${providerName}' not found.`))
-          console.log(colors.infoMessage('Use "ai-changelog providers list" to see available providers.'))
+          console.log(
+            colors.infoMessage('Use "ai-changelog providers list" to see available providers.')
+          )
           return
         }
 
@@ -974,13 +973,31 @@ class ProvidersCommand extends BaseCommand {
 
   getKnownModelsForProvider(providerName) {
     const knownModels = {
-      openai: ['gpt-4o', 'gpt-4o-mini', 'gpt-4-turbo', 'gpt-4', 'gpt-3.5-turbo', 'o1', 'o1-mini', 'o3-mini'],
-      anthropic: ['claude-sonnet-4-20250514', 'claude-3-5-sonnet-20241022', 'claude-3-opus-20240229', 'claude-3-haiku-20240307'],
+      openai: [
+        'gpt-4o',
+        'gpt-4o-mini',
+        'gpt-4-turbo',
+        'gpt-4',
+        'gpt-3.5-turbo',
+        'o1',
+        'o1-mini',
+        'o3-mini',
+      ],
+      anthropic: [
+        'claude-sonnet-4-20250514',
+        'claude-3-5-sonnet-20241022',
+        'claude-3-opus-20240229',
+        'claude-3-haiku-20240307',
+      ],
       google: ['gemini-2.0-flash-exp', 'gemini-1.5-pro', 'gemini-1.5-flash', 'gemini-pro'],
       azure: ['gpt-4o', 'gpt-4-turbo', 'gpt-35-turbo'],
       ollama: ['llama3.2', 'llama3.1', 'mistral', 'codellama', 'deepseek-coder'],
       lmstudio: ['local-model'],
-      bedrock: ['anthropic.claude-3-sonnet', 'anthropic.claude-3-haiku', 'amazon.titan-text-express'],
+      bedrock: [
+        'anthropic.claude-3-sonnet',
+        'anthropic.claude-3-haiku',
+        'amazon.titan-text-express',
+      ],
       vertex: ['gemini-1.5-pro', 'gemini-1.5-flash'],
       huggingface: ['mistralai/Mistral-7B-Instruct-v0.2', 'google/flan-t5-xxl'],
     }
@@ -1109,7 +1126,9 @@ class StashCommand extends BaseCommand {
         console.log(colors.header('\n📋 Stash Changelog:\n'))
         console.log(`## Stashed Changes\n`)
         console.log(`**Message:** ${details.message}`)
-        console.log(`**Stats:** ${details.stats.filesChanged} files, +${details.stats.insertions}/-${details.stats.deletions}`)
+        console.log(
+          `**Stats:** ${details.stats.filesChanged} files, +${details.stats.insertions}/-${details.stats.deletions}`
+        )
 
         console.log(`\n**Files affected:**`)
         details.files.forEach((f) => {

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

@@ -111,9 +111,11 @@ export class ConfigurationManager {
         const content = fs.readFileSync(this.configPath, 'utf8')
         const envVars = this.parseEnvFile(content)
         Object.assign(defaults, envVars)
-      } catch (_error) {
+      } catch (error) {
         console.warn(
-          colors.warningMessage(`Warning: Could not load config from ${this.configPath}`)
+          colors.warningMessage(
+            `Warning: Could not load config from ${this.configPath}: ${error.message}`
+          )
         )
       }
     }
@@ -179,10 +181,10 @@ export class ConfigurationManager {
 
       // Deep merge with defaults
       return this.deepMergeConfig(defaultConfig, yamlConfig)
-    } catch (_error) {
+    } catch (error) {
       console.warn(
         colors.warningMessage(
-          `Warning: Could not load changelog config from ${this.changelogConfigPath}, using defaults`
+          `Warning: Could not load changelog config from ${this.changelogConfigPath}: ${error.message}, using defaults`
         )
       )
       return defaultConfig

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

@@ -1,4 +1,5 @@
 import process from 'node:process'
+import { execSync } from 'node:child_process'
 
 import colors from '../../shared/constants/colors.js'
 import { EnhancedConsole, SimpleSpinner } from '../../shared/utils/cli-ui.js'
@@ -101,28 +102,40 @@ export class InteractiveWorkflowService {
       // Use AI to suggest better commit message
       if (this.aiAnalysisService.hasAI) {
         const prompt = message
-          ? `Improve this commit message following conventional commit format and best practices:
+          ? `Improve this commit message following conventional commit format.
 
 Original: "${message}"
 
+Provide EXACTLY 3 alternative commit messages, one per line, numbered 1-3.
+Do NOT include any other text, explanations, or conversational phrases.
+
 Requirements:
-- Use conventional commit format (type(scope): description)
+- Use conventional commit format: type(scope): description
 - Keep first line under 72 characters
 - Use imperative mood
 - Be specific and descriptive
 
-Provide 3 alternative suggestions.`
-          : `Suggest a commit message for these changes:
+Example output format:
+1. feat(auth): add OAuth2 authentication support
+2. fix(api): resolve timeout in user endpoint
+3. refactor(db): optimize query performance`
+          : `Generate commit messages for these changes:
 
 ${analysisContext}
 
+Provide EXACTLY 3 commit message suggestions, one per line, numbered 1-3.
+Do NOT include any other text, explanations, or conversational phrases.
+
 Requirements:
-- Use conventional commit format (type(scope): description)
+- Use conventional commit format: type(scope): description
 - Keep first line under 72 characters
 - Use imperative mood
 - Be specific and descriptive
 
-Provide 3 suggestions.`
+Example output format:
+1. feat(auth): add OAuth2 authentication support
+2. fix(api): resolve timeout in user endpoint
+3. refactor(db): optimize query performance`
 
         const response = await this.aiAnalysisService.aiProvider.generateCompletion(
           [
@@ -195,6 +208,71 @@ Provide 3 suggestions.`
       summary += ` (${details.join(', ')})`
     }
 
+    // Add actual change summaries from git diff for better context
+    summary += '\n\nKey changes:\n'
+
+    try {
+      // Get a concise summary of actual changes, limiting to top 10 files
+      const filesToAnalyze = changes.slice(0, 10)
+
+      filesToAnalyze.forEach((change) => {
+        const filePath = change.filePath || change.path
+        const status = change.status
+
+        try {
+          if (status === 'M' || status === 'MM') {
+            // For modified files, get a brief diff stat
+            const diffStat = execSync(`git diff HEAD -- "${filePath}" | head -50`, {
+              encoding: 'utf8',
+              maxBuffer: 1024 * 100,
+            })
+
+            if (diffStat) {
+              // Extract some meaningful info from diff
+              const addedLines = (diffStat.match(/^\+(?!\+)/gm) || []).length
+              const removedLines = (diffStat.match(/^-(?!-)/gm) || []).length
+
+              // Try to extract function/class names or key changes
+              const meaningfulLines = diffStat
+                .split('\n')
+                .filter(
+                  (line) =>
+                    line.startsWith('+') &&
+                    (line.includes('function') ||
+                      line.includes('class') ||
+                      line.includes('const') ||
+                      line.includes('export') ||
+                      line.includes('import') ||
+                      line.includes('async') ||
+                      line.includes('//'))
+                )
+                .slice(0, 2)
+                .map((line) => line.substring(1).trim())
+
+              if (meaningfulLines.length > 0) {
+                summary += `- ${filePath}: +${addedLines}/-${removedLines} lines (${meaningfulLines[0].substring(0, 60)}...)\n`
+              } else {
+                summary += `- ${filePath}: +${addedLines}/-${removedLines} lines\n`
+              }
+            }
+          } else if (status === 'A' || status === '??') {
+            summary += `- ${filePath}: New file\n`
+          } else if (status === 'D') {
+            summary += `- ${filePath}: Deleted\n`
+          }
+        } catch (fileError) {
+          // Skip files that can't be diffed
+        }
+      })
+
+      if (changes.length > 10) {
+        summary += `... and ${changes.length - 10} more files\n`
+      }
+    } catch (error) {
+      // If diff analysis fails, fall back to basic summary
+      console.warn('Could not analyze diffs:', error.message)
+    }
+
     return summary
   }
 
@@ -380,17 +458,46 @@ Provide 3 suggestions.`
   }
 
   parseCommitSuggestions(content) {
-    // Parse AI response to extract suggestions
+    // Parse AI response to extract valid commit message suggestions
     const lines = content.split('\n').filter((line) => line.trim())
     const suggestions = []
 
+    // Valid conventional commit types
+    const validTypes = [
+      'feat',
+      'fix',
+      'docs',
+      'style',
+      'refactor',
+      'test',
+      'chore',
+      'perf',
+      'ci',
+      'build',
+      'revert',
+    ]
+
     lines.forEach((line) => {
       // Remove numbering and clean up
-      const cleaned = line
-        .replace(/^\d+\.\s*/, '')
-        .replace(/^-\s*/, '')
-        .trim()
-      if (cleaned && cleaned.length > 10 && cleaned.includes(':')) {
+      const cleaned = line.replace(/^\d+\.\s*/, '').replace(/^-\s*/, '').trim()
+
+      // Must be a reasonable length
+      if (!cleaned || cleaned.length < 10 || cleaned.length > 100) {
+        return
+      }
+
+      // Must have a colon (conventional format)
+      if (!cleaned.includes(':')) {
+        return
+      }
+
+      // Must start with a valid conventional commit type
+      const startsWithValidType = validTypes.some((type) => {
+        const pattern = new RegExp(`^${type}(\\(|:)`, 'i')
+        return pattern.test(cleaned)
+      })
+
+      if (startsWithValidType) {
         suggestions.push(cleaned)
       }
     })

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

@@ -5,7 +5,8 @@
  * Provides Model Context Protocol interface for changelog generation
  */
 
-import { execSync } from 'node:child_process'
+import { exec } from 'node:child_process'
+import { promisify } from 'node:util'
 import fs from 'node:fs'
 import path, { dirname } from 'node:path'
 import process from 'node:process'
@@ -18,8 +19,7 @@ import { CallToolRequestSchema, ListToolsRequestSchema } from '@modelcontextprot
 import { ChangelogOrchestrator } from '../../application/orchestrators/changelog.orchestrator.js'
 // Import application services
 import { ApplicationService } from '../../application/services/application.service.js'
-import { AnalysisEngine } from '../../domains/analysis/analysis.engine.js'
-import { GitService } from '../../domains/git/git.service.js'
+
 import { ConfigurationManager } from '../config/configuration.manager.js'
 import { ProviderManagerService } from '../providers/provider-manager.service.js'
 
@@ -312,7 +312,7 @@ class AIChangelogMCPServer {
 
       let result
 
-      if (source === 'working-dir' || (source === 'auto' && this.hasWorkingDirectoryChanges())) {
+      if (source === 'working-dir' || (source === 'auto' && (await this.hasWorkingDirectoryChanges()))) {
         // Generate from working directory changes using ChangelogService
         // Access the changelogService through the orchestrator
         await this.changelogOrchestrator.ensureInitialized()
@@ -324,7 +324,7 @@ class AIChangelogMCPServer {
         })
 
         // Format result to match expected structure
-        if (result && result.changelog) {
+        if (result?.changelog) {
           result = {
             content: result.changelog,
             metadata: {
@@ -522,12 +522,15 @@ class AIChangelogMCPServer {
     }
   }
 
-  hasWorkingDirectoryChanges() {
+  async hasWorkingDirectoryChanges() {
     try {
-      // Simple check for working directory changes
-      const result = execSync('git status --porcelain', { encoding: 'utf8' })
-      return result.trim().length > 0
-    } catch (_error) {
+      const execAsync = promisify(exec)
+      const { stdout } = await execAsync('git status --porcelain', { encoding: 'utf8' })
+      return stdout.trim().length > 0
+    } catch (error) {
+      // If git is not installed or not a git repo, we can assume no changes for our purpose
+      // but we should log the error for debugging
+      console.warn(`[MCP] Git check warning: ${error.message}`)
       return false
     }
   }

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

@@ -87,7 +87,8 @@ export class BaseProvider {
   async selectOptimalModel(commitInfo) {
     try {
       return this.getModelRecommendation(commitInfo)
-    } catch (_error) {
+    } catch (error) {
+      console.warn(`Warning: Model selection failed: ${error.message}. Using default model.`)
       return { model: this.getDefaultModel(), reason: 'fallback' }
     }
   }

package/src/infrastructure/providers/utils/model-config.js

@@ -14,7 +14,9 @@ function addWarningToCache(key) {
     warningCache.clear()
     // Keep last 80% of entries
     const keepCount = Math.floor(MAX_WARNING_CACHE_SIZE * 0.8)
-    entries.slice(-keepCount).forEach((entry) => warningCache.add(entry))
+    entries.slice(-keepCount).forEach((entry) => {
+      warningCache.add(entry)
+    })
   }
   warningCache.add(key)
 }

package/src/infrastructure/validation/commit-message-validation.service.js

@@ -483,7 +483,7 @@ export class CommitMessageValidationService {
    * Display validation results with colors
    */
   displayValidationResults(validationResult) {
-    const { valid, errors, warnings, suggestions, score, summary } = validationResult
+    const { valid, errors, warnings, suggestions, summary } = validationResult
 
     EnhancedConsole.section('🔍 Commit Message Validation')
     console.log(colors.secondary(`${summary}`))