@entro314labs/ai-changelog-generator 3.2.1 → 3.6.0

package/src/application/orchestrators/changelog.orchestrator.js

@@ -3,7 +3,9 @@ import process from 'node:process'
 import { AIAnalysisService } from '../../domains/ai/ai-analysis.service.js'
 import { AnalysisEngine } from '../../domains/analysis/analysis.engine.js'
 import { ChangelogService } from '../../domains/changelog/changelog.service.js'
+import { CommitTagger } from '../../domains/git/commit-tagger.js'
 import { GitService } from '../../domains/git/git.service.js'
+import { GitManager } from '../../domains/git/git-manager.js'
 import { InteractiveStagingService } from '../../infrastructure/interactive/interactive-staging.service.js'
 import { InteractiveWorkflowService } from '../../infrastructure/interactive/interactive-workflow.service.js'
 import { ProviderManagerService } from '../../infrastructure/providers/provider-manager.service.js'
@@ -58,9 +60,9 @@ export class ChangelogOrchestrator {
       })
       this.aiProvider = this.providerManager.getActiveProvider()
 
-      // Create lightweight implementations for missing dependencies
-      this.gitManager = await this.createGitManager()
-      this.tagger = await this.createTagger()
+      // Create proper implementations using the new classes
+      this.gitManager = new GitManager()
+      this.tagger = new CommitTagger()
       this.promptEngine = await this.createPromptEngine()
 
       // Initialize domain services with proper dependencies
@@ -121,207 +123,17 @@ export class ChangelogOrchestrator {
     }
   }
 
-  async createGitManager() {
-    const { execSync } = await this._getCachedImport('child_process')
-
-    return {
-      isGitRepo: (() => {
-        try {
-          execSync('git rev-parse --git-dir', { stdio: 'ignore' })
-          return true
-        } catch {
-          return false
-        }
-      })(),
-
-      execGit(command) {
-        try {
-          return execSync(command, { encoding: 'utf8', stdio: 'pipe' })
-        } catch (error) {
-          // Enhanced error handling with more specific messages
-          if (error.code === 128) {
-            throw new Error(
-              `Git repository error: ${error.message.includes('not a git repository') ? 'Not in a git repository' : error.message}`
-            )
-          }
-          if (error.code === 129) {
-            throw new Error(`Git command syntax error: ${command}`)
-          }
-          throw new Error(`Git command failed (${command}): ${error.message}`)
-        }
-      },
-
-      execGitSafe(command) {
-        try {
-          return execSync(command, { encoding: 'utf8', stdio: 'pipe' })
-        } catch {
-          return ''
-        }
-      },
-
-      execGitShow(command) {
-        try {
-          return execSync(command, { encoding: 'utf8', stdio: 'pipe' })
-        } catch (_error) {
-          // console.warn(`Git command failed: ${command}`);
-          // console.warn(`Error: ${error.message}`);
-          return null
-        }
-      },
-
-      validateCommitHash(hash) {
-        try {
-          execSync(`git cat-file -e ${hash}`, { stdio: 'ignore' })
-          return true
-        } catch {
-          return false
-        }
-      },
-
-      getAllBranches() {
-        try {
-          const output = execSync('git branch -a', { encoding: 'utf8' })
-          return output
-            .split('\n')
-            .filter(Boolean)
-            .map((branch) => branch.trim().replace(/^\*\s*/, ''))
-        } catch {
-          return []
-        }
-      },
-
-      getUnmergedCommits() {
-        try {
-          const output = execSync('git log --oneline --no-merges HEAD ^origin/main', {
-            encoding: 'utf8',
-          })
-          return output.split('\n').filter(Boolean)
-        } catch {
-          return []
-        }
-      },
-
-      getDanglingCommits() {
-        try {
-          const output = execSync('git fsck --no-reflog | grep "dangling commit"', {
-            encoding: 'utf8',
-          })
-          return output.split('\n').filter(Boolean)
-        } catch {
-          return []
-        }
-      },
-
-      getUntrackedFiles() {
-        try {
-          const output = execSync('git ls-files --others --exclude-standard', { encoding: 'utf8' })
-          return output.split('\n').filter(Boolean)
-        } catch {
-          return []
-        }
-      },
-
-      getRecentCommits(limit = 10) {
-        try {
-          const output = execSync(`git log --oneline -${limit}`, { encoding: 'utf8' })
-          return output.split('\n').filter(Boolean)
-        } catch {
-          return []
-        }
-      },
-
-      getComprehensiveAnalysis() {
-        return {
-          totalCommits: this.getRecentCommits(1000).length,
-          branches: this.getAllBranches(),
-          untrackedFiles: this.getUntrackedFiles(),
-        }
-      },
-
-      hasFile(filename) {
-        try {
-          execSync(`test -f ${filename}`, { stdio: 'ignore' })
-          return true
-        } catch {
-          return false
-        }
-      },
-    }
-  }
-
-  async createTagger() {
-    const { analyzeSemanticChanges, analyzeFunctionalImpact } = await import(
-      '../../shared/utils/utils.js'
-    )
-
-    return {
-      analyzeCommit(commit) {
-        const semanticChanges = []
-        const breakingChanges = []
-        const categories = []
-        const tags = []
-
-        // Basic analysis based on commit message
-        const message = commit.message.toLowerCase()
-
-        if (message.includes('breaking') || message.includes('!:')) {
-          breakingChanges.push('Breaking change detected in commit message')
-          categories.push('breaking')
-          tags.push('breaking')
-        }
-
-        if (message.startsWith('feat')) {
-          categories.push('feature')
-          tags.push('feature')
-        } else if (message.startsWith('fix')) {
-          categories.push('fix')
-          tags.push('bugfix')
-        } else if (message.startsWith('docs')) {
-          categories.push('docs')
-          tags.push('documentation')
-        }
-
-        // Analyze files if available
-        if (commit.files && commit.files.length > 0) {
-          commit.files.forEach((file) => {
-            const semantic = analyzeSemanticChanges('', file.path)
-            if (semantic.frameworks) {
-              semanticChanges.push(...semantic.frameworks)
-            }
-          })
-        }
-
-        // Determine importance
-        let importance = 'medium'
-        if (breakingChanges.length > 0 || commit.files?.length > 10) {
-          importance = 'high'
-        } else if (categories.includes('docs') || commit.files?.length < 3) {
-          importance = 'low'
-        }
-
-        return {
-          semanticChanges,
-          breakingChanges,
-          categories,
-          importance,
-          tags,
-        }
-      },
-    }
-  }
-
   async createPromptEngine() {
-    const { buildEnhancedPrompt } = await import('../../shared/utils/utils.js')
-
     return {
       systemPrompts: {
         master:
-          'You are an expert software analyst specializing in code change analysis and changelog generation.',
-        standard: 'Provide clear, concise analysis focusing on the practical impact of changes.',
+          'You are an expert software analyst specializing in code change analysis and changelog generation. Analyze code changes with precision and provide definitive, factual assessments. Never use uncertain language like "likely", "probably", "appears to", "seems to", or "possibly". Base your analysis solely on the actual changes shown in the code diffs.',
+        standard:
+          'Provide clear, concise analysis focusing on the practical impact of changes. Be definitive and factual in your descriptions.',
         detailed:
-          'Provide comprehensive technical analysis with detailed explanations and implications.',
+          'Provide comprehensive technical analysis with detailed explanations and implications. Use precise, confident language backed by evidence from the code changes.',
         enterprise:
-          'Provide enterprise-grade analysis suitable for stakeholder communication and decision-making.',
+          'Provide enterprise-grade analysis suitable for stakeholder communication and decision-making. Use authoritative, factual language that conveys confidence and accuracy.',
         changesAnalysis: 'You are an expert at analyzing code changes and their business impact.',
       },
 
@@ -356,7 +168,7 @@ export class ChangelogOrchestrator {
       }
 
       // Generate changelog using the service
-      const result = await this.changelogService.generateChangelog(version, since)
+      const result = await this.changelogService.generateChangelog(version, since, this.options)
 
       if (!result) {
         console.log(colors.warningMessage('No changelog generated'))
@@ -399,9 +211,13 @@ export class ChangelogOrchestrator {
   async runInteractive() {
     await this.ensureInitialized()
 
-    const { runInteractiveMode, selectSpecificCommits } = await import(
-      '../../shared/utils/utils.js'
-    )
+    // Check for interactive environment
+    if (!process.stdin.isTTY && !process.env.CI) {
+      console.log(colors.warningMessage('Interactive mode requires a TTY terminal.'))
+      return { interactive: false, status: 'skipped' }
+    }
+
+    const { runInteractiveMode } = await import('../../shared/utils/utils.js')
     const { confirm } = await this._getCachedImport('@clack/prompts')
 
     console.log(colors.processingMessage('🎮 Starting interactive mode...'))
@@ -1276,7 +1092,7 @@ Generate only the commit message, no explanation.`
     if (!this.aiAnalysisService?.aiProvider?.isAvailable()) {
       throw new Error('AI provider not available for suggestions')
     }
-    
+
     const { select } = await this._getCachedImport('@clack/prompts')
 
     console.log(colors.processingMessage('🤖 Generating AI suggestions...'))

package/src/cli.js

@@ -1,5 +1,17 @@
 #!/usr/bin/env node
 
+/**
+ * DEPRECATED: Legacy CLI Entry Point
+ *
+ * This file is deprecated and maintained only for backward compatibility.
+ * Please use bin/ai-changelog.js which delegates to CLIController instead.
+ *
+ * @deprecated Use bin/ai-changelog.js with CLIController
+ */
+
+console.warn('\n⚠️  WARNING: This CLI entry point is deprecated.')
+console.warn('Please use the main CLI: ai-changelog (or bin/ai-changelog.js)\n')
+
 /**
  * CLI Entry Point for AI Changelog Generator
  * Provides command-line interface for changelog generation functionality
@@ -11,7 +23,6 @@ import { hideBin } from 'yargs/helpers'
 import yargs from 'yargs/yargs'
 
 import { AIChangelogGenerator } from './ai-changelog-generator.js'
-import colors from './shared/constants/colors.js'
 import { EnhancedConsole, SimpleSpinner } from './shared/utils/cli-ui.js'
 
 async function runCLI() {
@@ -91,14 +102,14 @@ async function runCLI() {
     }
 
     if (argv.validate) {
-      console.log(colors.infoMessage('🔧 Validating configuration...'))
+      EnhancedConsole.info('🔧 Validating configuration...')
       await generator.validateConfiguration()
-      console.log(colors.successMessage('✅ Configuration is valid'))
+      EnhancedConsole.success('✅ Configuration is valid')
       return
     }
 
     if (argv.interactive) {
-      console.log(colors.infoMessage('🎮 Starting interactive mode...'))
+      EnhancedConsole.info('🎮 Starting interactive mode...')
       await generator.runInteractive()
       return
     }
@@ -170,7 +181,7 @@ export { runCLI }
 // If this file is run directly, execute CLI
 if (import.meta.url === `file://${process.argv[1]}`) {
   runCLI().catch((error) => {
-    console.error('CLI Error:', error.message)
+    EnhancedConsole.error('CLI Error:', error.message)
     process.exit(1)
   })
 }

package/src/domains/ai/ai-analysis.service.js

@@ -143,6 +143,8 @@ export class AIAnalysisService {
         temperature: 0.3,
       })
 
+      // Debug logging removed
+
       if (response?.usage) {
         this.metrics.totalTokens +=
           (response.usage.prompt_tokens || 0) + (response.usage.completion_tokens || 0)