clean debug outputs

Author: MarcoWang3

packages/core/src/amazonq/autoDebug/diagnostics/diagnosticsMonitor.ts

@@ -41,7 +41,6 @@ export class DiagnosticsMonitor implements vscode.Disposable {
         // Monitor diagnostic changes from all language servers
         this.disposables.push(
             vscode.languages.onDidChangeDiagnostics((event) => {
-                this.logger.debug('DiagnosticsMonitor: Diagnostic change detected for %d URIs', event.uris.length)
                 this.handleDiagnosticChange(event)
             })
         )
@@ -119,8 +118,6 @@ export class DiagnosticsMonitor implements vscode.Disposable {
      * Captures a baseline snapshot of current diagnostics
      */
     public async captureBaseline(): Promise<DiagnosticSnapshot> {
-        this.logger.debug('DiagnosticsMonitor: Capturing diagnostic baseline')
-
         const diagnostics = await this.getCurrentDiagnostics(false)
         const snapshot: DiagnosticSnapshot = {
             diagnostics,
@@ -168,7 +165,6 @@ export class DiagnosticsMonitor implements vscode.Disposable {
 
             // Only emit if diagnostics actually changed
             if (!this.lastDiagnostics || !this.areDiagnosticsEqual(this.lastDiagnostics, currentDiagnostics)) {
-                this.logger.debug('DiagnosticsMonitor: Emitting diagnostic change event')
                 this.lastDiagnostics = currentDiagnostics
                 this.diagnosticsChangeEmitter.fire(currentDiagnostics)
             }
@@ -230,8 +226,6 @@ export class DiagnosticsMonitor implements vscode.Disposable {
     }
 
     public dispose(): void {
-        this.logger.debug('DiagnosticsMonitor: Disposing diagnostic monitor')
-
         if (this.debounceTimer) {
             clearTimeout(this.debounceTimer)
         }

packages/core/src/amazonq/autoDebug/diagnostics/errorContext.ts

@@ -45,12 +45,6 @@ export class ErrorContextFormatter {
      * Converts a Problem to detailed ErrorContext
      */
     public async createErrorContext(problem: Problem): Promise<ErrorContext> {
-        this.logger.debug(
-            'ErrorContextFormatter: Creating error context for %s at line %d',
-            problem.uri.fsPath,
-            problem.diagnostic.range.start.line + 1
-        )
-
         const surroundingCode = await this.extractSurroundingCode(problem.uri, problem.diagnostic.range)
 
         const context: ErrorContext = {
@@ -67,17 +61,13 @@ export class ErrorContextFormatter {
             relatedInformation: problem.diagnostic.relatedInformation,
             surroundingCode,
         }
-
-        this.logger.debug('ErrorContextFormatter: Created error context for %s', context.location.file)
         return context
     }
 
     /**
      * Formats multiple error contexts into a comprehensive report
      */
     public formatErrorReport(contexts: ErrorContext[]): FormattedErrorReport {
-        this.logger.debug('ErrorContextFormatter: Formatting error report for %d contexts', contexts.length)
-
         const summary = this.createSummary(contexts)
         const details = this.createDetails(contexts)
         const contextualCode = this.createContextualCode(contexts)
@@ -95,8 +85,6 @@ export class ErrorContextFormatter {
      * Formats a single error context for display
      */
     public formatSingleError(context: ErrorContext): string {
-        this.logger.debug('ErrorContextFormatter: Formatting single error for %s', context.location.file)
-
         const parts = [
             `**${context.severity.toUpperCase()}** in ${context.location.file}`,
             `Line ${context.location.line}, Column ${context.location.column}`,
@@ -134,8 +122,6 @@ export class ErrorContextFormatter {
      * Creates a problems string similar to the reference implementation
      */
     public formatProblemsString(problems: Problem[], cwd: string): string {
-        this.logger.debug('ErrorContextFormatter: Formatting problems string for %d problems', problems.length)
-
         let result = ''
         const fileGroups = this.groupProblemsByFile(problems)
 

packages/core/src/amazonq/autoDebug/diagnostics/problemDetector.ts

@@ -5,7 +5,6 @@
 
 import * as vscode from 'vscode'
 import * as path from 'path'
-import { getLogger } from '../../../shared/logger/logger'
 import { DiagnosticCollection, DiagnosticSnapshot } from './diagnosticsMonitor'
 import { mapDiagnosticSeverity } from '../shared/diagnosticUtils'
 
@@ -28,22 +27,10 @@ export interface CategorizedProblems {
  * Detects new problems by comparing diagnostic snapshots and filtering relevant issues.
  */
 export class ProblemDetector {
-    private readonly logger = getLogger('amazonqLsp')
-
-    constructor() {
-        this.logger.debug('ProblemDetector: Initializing problem detector')
-    }
-
     /**
      * Detects new problems by comparing baseline and current diagnostics
      */
     public detectNewProblems(baseline: DiagnosticSnapshot, current: DiagnosticSnapshot): Problem[] {
-        this.logger.debug(
-            'ProblemDetector: Detecting new problems between baseline %s and current %s',
-            baseline.id,
-            current.id
-        )
-
         const newProblems: Problem[] = []
         const baselineMap = this.createDiagnosticMap(baseline.diagnostics)
 
@@ -56,21 +43,13 @@ export class ProblemDetector {
                 }
             }
         }
-
-        this.logger.debug('ProblemDetector: Found %d new problems', newProblems.length)
         return newProblems
     }
 
     /**
      * Filters problems to only include those relevant to changed files
      */
     public filterRelevantProblems(problems: Problem[], changedFiles: string[]): Problem[] {
-        this.logger.debug(
-            'ProblemDetector: Filtering %d problems for %d changed files',
-            problems.length,
-            changedFiles.length
-        )
-
         if (changedFiles.length === 0) {
             return problems
         }
@@ -81,17 +60,13 @@ export class ProblemDetector {
             const problemFile = path.normalize(problem.uri.fsPath)
             return changedFileSet.has(problemFile) || this.isRelatedFile(problemFile, changedFiles)
         })
-
-        this.logger.debug('ProblemDetector: Filtered to %d relevant problems', relevantProblems.length)
         return relevantProblems
     }
 
     /**
      * Categorizes problems by severity level
      */
     public categorizeBySeverity(problems: Problem[]): CategorizedProblems {
-        this.logger.debug('ProblemDetector: Categorizing %d problems by severity', problems.length)
-
         const categorized: CategorizedProblems = {
             errors: [],
             warnings: [],
@@ -115,45 +90,28 @@ export class ProblemDetector {
                     break
             }
         }
-
-        this.logger.debug(
-            'ProblemDetector: Categorized problems - errors: %d, warnings: %d, info: %d, hints: %d',
-            categorized.errors.length,
-            categorized.warnings.length,
-            categorized.info.length,
-            categorized.hints.length
-        )
-
         return categorized
     }
 
     /**
      * Gets all problems from a diagnostic collection
      */
     public getAllProblems(diagnostics: DiagnosticCollection): Problem[] {
-        this.logger.debug('ProblemDetector: Converting diagnostic collection to problems')
-
         const problems: Problem[] = []
 
         for (const [uri, fileDiagnostics] of diagnostics.diagnostics) {
             for (const diagnostic of fileDiagnostics) {
                 problems.push(this.createProblem(uri, diagnostic, false))
             }
         }
-
-        this.logger.debug('ProblemDetector: Converted %d diagnostics to problems', problems.length)
         return problems
     }
 
     /**
      * Filters problems by source (TypeScript, ESLint, etc.)
      */
     public filterBySource(problems: Problem[], sources: string[]): Problem[] {
-        this.logger.debug('ProblemDetector: Filtering problems by sources: %O', sources)
-
         const filtered = problems.filter((problem) => sources.length === 0 || sources.includes(problem.source))
-
-        this.logger.debug('ProblemDetector: Filtered to %d problems', filtered.length)
         return filtered
     }
 

packages/core/src/amazonq/autoDebug/ide/contextMenuProvider.ts

@@ -200,14 +200,11 @@ export class ContextMenuProvider implements vscode.Disposable {
      * Starts a new auto debug session
      */
     private async startAutoDebugSession(): Promise<void> {
-        this.logger.debug('ContextMenuProvider: Starting auto debug session')
-
         try {
             const session = await this.autoDebugController.startSession()
             void vscode.window.showInformationMessage(
                 `Auto Debug session started (ID: ${session.id.substring(0, 8)}...)`
             )
-            this.logger.debug('ContextMenuProvider: Auto debug session started successfully')
         } catch (error) {
             this.logger.error('ContextMenuProvider: Error starting auto debug session: %s', error)
             void vscode.window.showErrorMessage('Failed to start Auto Debug session')
@@ -223,7 +220,6 @@ export class ContextMenuProvider implements vscode.Disposable {
         try {
             await this.autoDebugController.endSession()
             void vscode.window.showInformationMessage('Auto Debug session ended')
-            this.logger.debug('ContextMenuProvider: Auto debug session ended successfully')
         } catch (error) {
             this.logger.error('ContextMenuProvider: Error ending auto debug session: %s', error)
             void vscode.window.showErrorMessage('Failed to end Auto Debug session')