@entro314labs/ai-changelog-generator 3.2.1 → 3.6.0

package/src/shared/utils/diff-processor.js

@@ -16,18 +16,18 @@ export class DiffProcessor {
 
   getDefaultMaxSize() {
     const sizes = {
-      standard: 12000,
-      detailed: 20000,
-      enterprise: 30000,
+      standard: 50000, // Increased from 12000 - modern LLMs can handle more
+      detailed: 80000, // Increased from 20000
+      enterprise: 120000, // Increased from 30000
     }
     return sizes[this.analysisMode] || sizes.standard
   }
 
   getDefaultPriorityFiles() {
     const counts = {
-      standard: 15,
-      detailed: 25,
-      enterprise: 40,
+      standard: 30, // Increased from 15 - analyze more files in detail
+      detailed: 50, // Increased from 25
+      enterprise: 80, // Increased from 40
     }
     return counts[this.analysisMode] || counts.standard
   }
@@ -101,7 +101,7 @@ export class DiffProcessor {
    * Process a single file's diff with intelligent compression
    */
   processFileDiff(file, options = {}) {
-    const { budget = 1500, patterns = {}, isHighPriority = false } = options
+    const { budget = 4000, patterns = {}, isHighPriority = false } = options // Increased from 1500
 
     if (!file.diff || file.diff === 'No diff available') {
       return {
@@ -119,17 +119,18 @@ export class DiffProcessor {
       processedDiff = this.applyDiffFiltering(processedDiff, file)
     }
 
-    // Check if file matches bulk patterns
-    const patternMatch = this.matchesBulkPattern(file, patterns)
-    if (patternMatch) {
-      return {
-        ...file,
-        diff: `[Bulk ${patternMatch.type}]: ${patternMatch.description}`,
-        processed: true,
-        compressionApplied: true,
-        bulkPattern: patternMatch.type,
-      }
-    }
+    // Temporarily disable bulk pattern matching to preserve more detail
+    // TODO: Make this smarter - only apply to truly repetitive patterns
+    // const patternMatch = this.matchesBulkPattern(file, patterns)
+    // if (patternMatch) {
+    //   return {
+    //     ...file,
+    //     diff: `[Bulk ${patternMatch.type}]: ${patternMatch.description}`,
+    //     processed: true,
+    //     compressionApplied: true,
+    //     bulkPattern: patternMatch.type,
+    //   }
+    // }
 
     // Apply intelligent truncation if still too large
     if (processedDiff.length > budget) {
@@ -313,7 +314,8 @@ export class DiffProcessor {
       return trimmed.match(/^(import|require|from\s+['"])/)
     })
 
-    if (importLines.length > 10) {
+    if (importLines.length > 25) {
+      // Increased threshold from 10 to 25
       // If many import changes, summarize them
       filteredDiff = filteredDiff.replace(/^[+-]\s*(import|require|from\s+['"]).*$/gm, '')
       filteredDiff = `[${importLines.length} import/require changes summarized]\n${filteredDiff}`

package/src/shared/utils/error-classes.js

@@ -355,6 +355,39 @@ export class NetworkError extends AIChangelogError {
   }
 }
 
+/**
+ * Parse error for malformed data/responses
+ */
+export class ParseError extends Error {
+  constructor(message, data = null, originalError = null) {
+    super(message)
+    this.name = 'ParseError'
+    this.data = data
+    this.originalError = originalError
+
+    if (Error.captureStackTrace) {
+      Error.captureStackTrace(this, ParseError)
+    }
+  }
+}
+
+/**
+ * Concurrency limit error for resource management
+ */
+export class ConcurrencyLimitError extends Error {
+  constructor(message, limit, active, originalError = null) {
+    super(message)
+    this.name = 'ConcurrencyLimitError'
+    this.limit = limit
+    this.active = active
+    this.originalError = originalError
+
+    if (Error.captureStackTrace) {
+      Error.captureStackTrace(this, ConcurrencyLimitError)
+    }
+  }
+}
+
 /**
  * Error context builder utility
  */

package/src/shared/utils/utils.js

@@ -17,57 +17,12 @@ import fs from 'node:fs'
 
 import colors from '../constants/colors.js'
 import { DiffProcessor } from './diff-processor.js'
+import { AbstractMethodError, AIChangelogError, ProviderError } from './error-classes.js'
 import JsonUtils from './json-utils.js'
 
-// ========================================
-// ERROR HANDLING UTILITIES
-// ========================================
-
-/**
- * Base error class for domain-specific errors
- */
-export class AIChangelogError extends Error {
-  constructor(message, type, context = {}, originalError = null) {
-    super(message)
-    this.name = this.constructor.name
-    this.type = type
-    this.context = context
-    this.originalError = originalError
-    this.timestamp = new Date().toISOString()
-  }
-
-  toJSON() {
-    return {
-      name: this.name,
-      message: this.message,
-      type: this.type,
-      context: this.context,
-      timestamp: this.timestamp,
-      stack: this.stack,
-    }
-  }
-}
-
-/**
- * Abstract method error for base classes
- */
-export class AbstractMethodError extends AIChangelogError {
-  constructor(message, className, methodName, context = {}) {
-    super(message, 'abstract', { className, methodName, ...context })
-    this.className = className
-    this.methodName = methodName
-  }
-}
-
-/**
- * Provider-specific error
- */
-export class ProviderError extends AIChangelogError {
-  constructor(message, providerName, context = {}, originalError = null) {
-    super(message, 'provider', { providerName, ...context }, originalError)
-    this.providerName = providerName
-  }
-}
+// Note: Error classes have been moved to './error-classes.js' to avoid duplication
+// and provide better organization. Import them from there instead of defining here.
+export { AbstractMethodError, AIChangelogError, ProviderError }
 
 // ========================================
 // DATA MANIPULATION UTILITIES
@@ -1110,9 +1065,22 @@ export function buildEnhancedPrompt(commitAnalysis, analysisMode = 'standard') {
     riskAssessment = {},
   } = commitAnalysis
 
-  // Initialize DiffProcessor with analysis mode
+  // Detect merge commits and upgrade analysis mode for better specificity
+  const isMergeCommit = subject && subject.toLowerCase().includes('merge')
+  const effectiveAnalysisMode = isMergeCommit && files.length > 10 ? 'detailed' : analysisMode
+
+  // Debug logging for merge commits (disabled)
+  // if (isMergeCommit) {
+  //   console.log('\n=== MERGE COMMIT DEBUG (EARLY) ===')
+  //   console.log('Subject:', subject)
+  //   console.log('Files count:', files.length)
+  //   console.log('Effective analysis mode:', effectiveAnalysisMode)
+  //   console.log('=== END EARLY DEBUG ===\n')
+  // }
+
+  // Initialize DiffProcessor with effective analysis mode
   const diffProcessor = new DiffProcessor({
-    analysisMode,
+    analysisMode: effectiveAnalysisMode,
     enableFiltering: true,
     enablePatternDetection: true,
   })
@@ -1148,10 +1116,41 @@ export function buildEnhancedPrompt(commitAnalysis, analysisMode = 'standard') {
     })
     .join('')
 
+  // Check if we have enhanced merge summary from GitService (isMergeCommit already detected above)
+  const enhancedMergeSummary =
+    files.length > 0 && files[0].enhancedMergeSummary ? files[0].enhancedMergeSummary : null
+
   const prompt = `Analyze this git commit for changelog generation.
 
-**COMMIT:** ${subject}
-**FILES:** ${files.length} files (${filesProcessed} analyzed, ${filesSkipped} summarized), ${insertions + deletions} lines changed${patternSummary}
+**COMMIT:** ${subject}${isMergeCommit ? ' (MERGE COMMIT - categorize as "merge")' : ''}
+**FILES:** ${files.length} files (${filesProcessed} analyzed, ${filesSkipped} summarized), ${insertions + deletions} lines changed${patternSummary}${
+    enhancedMergeSummary
+      ? `
+
+**ENHANCED MERGE SUMMARY:**
+${enhancedMergeSummary}
+
+**IMPORTANT FOR MERGE COMMITS:** Use the above enhanced summary as your description. Do NOT create a generic summary. Instead, convert the bullet points above into a flowing description that includes the specific file names, numbers, and technical details provided.`
+      : ''
+  }
+
+  // Debug logging for merge commits with "pull request"
+  if (isMergeCommit && subject.includes('pull request')) {
+    try {
+      const fs = require('fs')
+      const debugContent = '=== EXACT AI INPUT FOR fd97ab7 ===\n' +
+        'SUBJECT: ' + subject + '\n' +
+        'FILES COUNT: ' + files.length + '\n' +
+        'ENHANCED MERGE SUMMARY:\n' + (enhancedMergeSummary || 'None') + '\n\n' +
+        'FILES SECTION (first 3000 chars):\n' + filesSection.substring(0, 3000) + '\n\n' +
+        'FULL PROMPT PREVIEW:\n' + prompt.substring(0, 2000) + '\n=== END DEBUG ==='
+
+      fs.writeFileSync('AI_INPUT_DEBUG.txt', debugContent)
+      console.log('*** AI INPUT SAVED TO AI_INPUT_DEBUG.txt ***')
+    } catch (error) {
+      console.log('DEBUG ERROR:', error.message)
+    }
+  }
 
 **TARGET AUDIENCE:** End users and project stakeholders who need to understand what changed and why it matters.
 
@@ -1161,30 +1160,36 @@ export function buildEnhancedPrompt(commitAnalysis, analysisMode = 'standard') {
 1. **First, categorize correctly** using the rules below
 2. **Then, focus on user impact** - what can they do now that they couldn't before?
 3. **Keep technical details minimal** - only what's necessary for understanding
+4. **Be definitive and factual** - never use uncertain language like "likely", "probably", "appears to", "seems to", or "possibly"
+5. **Base analysis on actual code changes** - only describe what you can verify from the diff content
+6. **For merge commits** - ALWAYS categorize as "merge" regardless of content
+7. **Use enhanced merge summary** - If an "ENHANCED MERGE SUMMARY" section is provided above, DO NOT analyze individual file diffs. Instead, directly incorporate the specific bullet points from the enhanced summary into your response. Use the exact technical details, file names, and numbers provided in the enhanced summary.
 
 **PROCESSED DIFFS:**${filesSection}
 
 **CATEGORIZATION RULES (STRICTLY ENFORCED):**
+- **merge**: Any commit with "Merge" in the subject line (branch merges, pull request merges)
 - **fix**: ONLY actual bug fixes - broken functionality now works correctly
-- **feature**: New capabilities, tools, or major functionality additions
+- **feature**: New capabilities, tools, or major functionality additions (NOT merges)
 - **refactor**: Code restructuring without changing what users can do
 - **perf**: Performance improvements users will notice
 - **docs**: Documentation updates only
 - **build/chore**: Build system, dependencies, maintenance
 
 **CRITICAL VALIDATION:**
-- Large additions (>10 files OR >1000 lines) = "feature" or "refactor", NEVER "fix"
-- New modules/classes/tools = "feature"
+- Commits with "Merge" in subject = "merge" category ALWAYS
+- Large additions (>10 files OR >1000 lines) = "feature" or "refactor", NEVER "fix" (unless merge)
+- New modules/classes/tools = "feature" (unless merge)
 - Only actual bug repairs = "fix"
 
-Provide a JSON response with ONLY these fields:
+Provide a JSON response with ONLY these fields (use definitive language - NO "likely", "probably", "appears", etc.):
 {
   "summary": "${subject}",
   "impact": "critical|high|medium|low|minimal",
   "category": "feature|fix|security|breaking|docs|style|refactor|perf|test|chore",
-  "description": "One clear sentence describing what users can now do (for features) or what now works correctly (for fixes)",
-  "technicalDetails": "Maximum 2 sentences about key technical changes - focus on the most important functions/modules/APIs added or modified",
-  "businessValue": "Brief user benefit in 1 sentence",
+  "description": "One clear, factual sentence describing what users can now do (for features) or what now works correctly (for fixes)",
+  "technicalDetails": "Maximum 2 factual sentences about key technical changes - focus on the most important functions/modules/APIs added or modified",
+  "businessValue": "Brief, definitive user benefit in 1 sentence",
   "riskFactors": ["minimal", "list"],
   "recommendations": ["minimal", "list"],
   "breakingChanges": false,
@@ -1434,6 +1439,15 @@ function extractCategoryFromText(content) {
  */
 /**
  * Get raw working directory changes from git status
+ *
+ * Note: This is a lightweight utility function that shells out to git directly
+ * for simple status checks. For comprehensive git operations, services should
+ * use GitService/GitManager, but this utility is kept for:
+ * - Performance (avoids service initialization overhead)
+ * - Simplicity (standalone function for basic status checks)
+ * - Backward compatibility (widely used across codebase)
+ *
+ * @returns {Array} Array of change objects with status and filePath
  */
 export function getWorkingDirectoryChanges() {
   try {
@@ -1821,11 +1835,11 @@ function assessWorkspaceComplexity(categorizedChanges, totalFiles) {
  * Handle unified output for analysis commands
  */
 export function handleUnifiedOutput(data, config) {
-  const { format = 'markdown', outputFile, silent } = config
+  const { format = 'markdown', outputFile, silent, dryRun } = config
   if (format === 'json') {
     const jsonOutput = safeJsonStringify(data)
 
-    if (outputFile) {
+    if (outputFile && !dryRun) {
       // Write to file
       try {
         fs.writeFileSync(outputFile, jsonOutput, 'utf8')
@@ -1839,13 +1853,16 @@ export function handleUnifiedOutput(data, config) {
         }
       }
     } else if (!silent) {
+      if (dryRun && outputFile) {
+        console.log(colors.infoMessage(`[DRY RUN] Would save changelog to: ${colors.file(outputFile)}`))
+      }
       console.log(jsonOutput)
     }
   } else {
     // Markdown format (default)
     const markdownOutput = typeof data === 'string' ? data : safeJsonStringify(data)
 
-    if (outputFile) {
+    if (outputFile && !dryRun) {
       try {
         fs.writeFileSync(outputFile, markdownOutput, 'utf8')
         if (!silent) {
@@ -1858,6 +1875,9 @@ export function handleUnifiedOutput(data, config) {
         }
       }
     } else if (!silent) {
+      if (dryRun && outputFile) {
+        console.log(colors.infoMessage(`[DRY RUN] Would save changelog to: ${colors.file(outputFile)}`))
+      }
       console.log(markdownOutput)
     }
   }

package/types/index.d.ts

@@ -310,51 +310,42 @@ export class AIChangelogGenerator {
   constructor(constructorOptions?: {
     repositoryPath?: string
     configPath?: string
+    analysisMode?: AnalysisMode
+    modelOverride?: AIModel
+    dryRun?: boolean
+    silent?: boolean
   })
 
-  // Core methods
-  run(): Promise<void>
-  generateChangelog(changelogOptions?: ChangelogOptions): Promise<string>
-  analyzeCommits(commitOptions?: {
-    since?: string
-    limit?: number
-    repositoryPath?: string
-  }): Promise<CommitAnalysis>
+  // Core methods (these actually exist in the implementation)
+  generateChangelog(version?: string | null, since?: string | null): Promise<string>
 
-  // Enhanced analysis methods
-  analyzeCurrentChanges(analysisOptions?: {
-    includeAIAnalysis?: boolean
-    repositoryPath?: string
-  }): Promise<CurrentChangesAnalysis>
+  // Analysis methods
+  analyzeRepository(config?: { type?: string }): Promise<any>
+  analyzeCurrentChanges(): Promise<CurrentChangesAnalysis>
+  analyzeRecentCommits(limit?: number): Promise<CommitAnalysis>
+  analyzeBranches(config?: any): Promise<BranchAnalysis>
+  analyzeComprehensive(config?: any): Promise<RepositoryHealthCheck>
 
-  analyzeBranches(branchOptions?: {
-    includeAllBranches?: boolean
-    repositoryPath?: string
-  }): Promise<BranchAnalysis>
+  // Repository health
+  assessRepositoryHealth(includeRecommendations?: boolean): Promise<any>
 
-  analyzeRepository(repoOptions?: { repositoryPath?: string }): Promise<RepositoryHealthCheck>
+  // Interactive mode
+  runInteractive(): Promise<void>
 
-  // Configuration and validation
-  validateConfig(): Promise<{
-    success: boolean
-    provider?: AIProvider
-    model?: AIModel
-    capabilities?: ModelCapabilities
-    error?: string
-  }>
+  // Configuration
+  setAnalysisMode(mode: AnalysisMode): void
+  setModelOverride(model: AIModel): void
 
-  validateModels(): Promise<{
-    success: boolean
-    models?: Array<{
-      name: AIModel
-      available: boolean
-      capabilities?: ModelCapabilities
-    }>
-    error?: string
-  }>
+  // Validation
+  validateConfiguration(): Promise<void>
 
-  // Git operations
-  getGitInfo(gitOptions?: { includeStats?: boolean; repositoryPath?: string }): Promise<GitInfo>
+  // Metrics
+  getMetrics(): {
+    startTime: number
+    commitsProcessed: number
+    apiCalls: number
+    errors: number
+  }
 }
 
 export class AIChangelogMCPServer {
@@ -362,53 +353,55 @@ export class AIChangelogMCPServer {
 
   run(): Promise<void>
 
-  // MCP Tools
+  // MCP Tools (these match the actual implementation)
   generateChangelog(params: {
     repositoryPath?: string
+    source?: 'commits' | 'working-dir' | 'auto'
     analysisMode?: AnalysisMode
-    outputFormat?: OutputFormat
     since?: string
     version?: string
-    includeUnreleased?: boolean
-    model?: AIModel
-  }): Promise<string>
+    includeAttribution?: boolean
+    writeFile?: boolean
+  }): Promise<{
+    content: Array<{
+      type: 'text'
+      text: string
+    }>
+    metadata?: any
+  }>
 
-  analyzeCommits(params: {
+  analyzeRepository(params: {
     repositoryPath?: string
-    limit?: number
-    since?: string
-  }): Promise<CommitAnalysis>
+    analysisType?: 'health' | 'commits' | 'branches' | 'working-dir' | 'comprehensive'
+    includeRecommendations?: boolean
+    commitLimit?: number
+  }): Promise<{
+    content: Array<{
+      type: 'text'
+      text: string
+    }>
+  }>
 
   analyzeCurrentChanges(params: {
     repositoryPath?: string
     includeAIAnalysis?: boolean
-  }): Promise<CurrentChangesAnalysis>
-
-  getGitInfo(params: { repositoryPath?: string; includeStats?: boolean }): Promise<GitInfo>
-
-  configureAIProvider(params: {
-    provider?: AIProvider
-    showModels?: boolean
-    testConnection?: boolean
+    includeAttribution?: boolean
   }): Promise<{
-    success: boolean
-    provider?: AIProvider
-    models?: AIModel[]
-    error?: string
+    content: Array<{
+      type: 'text'
+      text: string
+    }>
   }>
 
-  validateModels(params: {
-    provider?: AIProvider
-    checkCapabilities?: boolean
-    testModels?: boolean
+  configureProviders(params: {
+    action?: 'list' | 'switch' | 'test' | 'configure' | 'validate'
+    provider?: string
+    testConnection?: boolean
   }): Promise<{
-    success: boolean
-    models?: Array<{
-      name: AIModel
-      available: boolean
-      capabilities?: ModelCapabilities
+    content: Array<{
+      type: 'text'
+      text: string
     }>
-    error?: string
   }>
 }