Clean up debug logging after successful fixes

Author: chrisbbreuer

src/buddy.ts

@@ -84,15 +84,6 @@ export class Buddy {
         ? this.groupUpdatesByConfig(updates)
         : groupUpdates(updates)
 
-      // Debug logging for group composition
-      console.log(`🐛 DEBUG: Created ${groups.length} update groups:`)
-      groups.forEach(group => {
-        console.log(`🐛   Group: "${group.name}" (${group.updateType}) - ${group.updates.length} updates`)
-        group.updates.forEach(update => {
-          console.log(`🐛     - ${update.name}: ${update.currentVersion} -> ${update.newVersion} (${update.file})`)
-        })
-      })
-
       const scanDuration = Date.now() - startTime
 
       const result: UpdateScanResult = {
@@ -502,14 +493,8 @@ export class Buddy {
     if (composerUpdates.length > 0) {
       try {
         const { generateComposerUpdates } = await import('./utils/composer-parser')
-        // Debug logging to see what composer updates are being passed
-        console.log(`🐛 DEBUG: Passing ${composerUpdates.length} composer updates to parser:`)
-        composerUpdates.forEach(update => {
-          console.log(`🐛   - ${update.name}: ${update.currentVersion} -> ${update.newVersion}`)
-        })
         // Pass only the composer updates for this specific group to prevent cross-contamination
         const compUpdates = await generateComposerUpdates(composerUpdates)
-        console.log(`🐛 DEBUG: Composer parser returned ${compUpdates.length} file updates`)
         fileUpdates.push(...compUpdates)
       }
       catch (error) {

src/utils/composer-parser.ts

@@ -183,11 +183,6 @@ export async function generateComposerUpdates(updates: Array<{ name: string, new
   const fileUpdates: Array<{ path: string, content: string, type: 'update' }> = []
   const composerUpdates = updates.filter(update => update.file.endsWith('composer.json'))
 
-  console.log(`🐛 COMPOSER PARSER DEBUG: Received ${updates.length} total updates, ${composerUpdates.length} composer updates`)
-  composerUpdates.forEach(update => {
-    console.log(`🐛   - Processing: ${update.name} -> ${update.newVersion}`)
-  })
-
   if (composerUpdates.length === 0) {
     return fileUpdates
   }
@@ -205,31 +200,23 @@ export async function generateComposerUpdates(updates: Array<{ name: string, new
     })
   }
 
-  console.log(`🐛 COMPOSER PARSER DEBUG: Processing ${updatesByFile.size} file(s)`)
-
   // Process each composer.json file
   for (const [filePath, fileUpdates_] of updatesByFile) {
     try {
-      console.log(`🐛 COMPOSER PARSER DEBUG: Processing file ${filePath} with ${fileUpdates_.length} updates`)
-
       // Read current composer.json content
       const fs = await import('node:fs')
       let composerContent = fs.readFileSync(filePath, 'utf-8')
 
-      console.log(`🐛 COMPOSER PARSER DEBUG: Original composer.json content length: ${composerContent.length}`)
-
       // Parse to understand structure
       const composerData: ComposerPackage = JSON.parse(composerContent)
 
       // Apply updates using string replacement to preserve formatting
       for (const update of fileUpdates_) {
-        console.log(`🐛 COMPOSER PARSER DEBUG: Applying update for ${update.name} -> ${update.newVersion}`)
         let packageFound = false
 
         // Check in require section
         if (composerData.require && composerData.require[update.name]) {
           const currentVersionInFile = composerData.require[update.name]
-          console.log(`🐛   Found ${update.name} in require: ${currentVersionInFile}`)
 
           // For complex constraints like ">=6.0,<7.0", preserve the constraint format
           // and just update the version numbers
@@ -248,8 +235,6 @@ export async function generateComposerUpdates(updates: Array<{ name: string, new
             newVersion = `${originalPrefix}${update.newVersion}`
           }
 
-          console.log(`🐛   Updating ${update.name}: ${currentVersionInFile} -> ${newVersion}`)
-
           // Create regex to find the exact line with this package and version
           const packageRegex = new RegExp(
             `("${update.name.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')}"\\s*:\\s*")([^"]+)(")`,
@@ -263,7 +248,6 @@ export async function generateComposerUpdates(updates: Array<{ name: string, new
         // Check in require-dev section
         if (!packageFound && composerData['require-dev'] && composerData['require-dev'][update.name]) {
           const currentVersionInFile = composerData['require-dev'][update.name]
-          console.log(`🐛   Found ${update.name} in require-dev: ${currentVersionInFile}`)
 
           // For complex constraints like ">=6.0,<7.0", preserve the constraint format
           // and just update the version numbers
@@ -282,8 +266,6 @@ export async function generateComposerUpdates(updates: Array<{ name: string, new
             newVersion = `${originalPrefix}${update.newVersion}`
           }
 
-          console.log(`🐛   Updating ${update.name}: ${currentVersionInFile} -> ${newVersion}`)
-
           const packageRegex = new RegExp(
             `("${update.name.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')}"\\s*:\\s*")([^"]+)(")`,
             'g',
@@ -298,8 +280,6 @@ export async function generateComposerUpdates(updates: Array<{ name: string, new
         }
       }
 
-      console.log(`🐛 COMPOSER PARSER DEBUG: Final composer.json content length: ${composerContent.length}`)
-
       fileUpdates.push({
         path: filePath,
         content: composerContent,
@@ -311,6 +291,5 @@ export async function generateComposerUpdates(updates: Array<{ name: string, new
     }
   }
 
-  console.log(`🐛 COMPOSER PARSER DEBUG: Returning ${fileUpdates.length} file updates`)
   return fileUpdates
 }