remove codes for loop debug

Author: MarcoWang3

packages/amazonq/src/lsp/autoDebug/controller.ts

@@ -4,10 +4,8 @@
  */
 
 import * as vscode from 'vscode'
-import { getLogger } from 'aws-core-vscode/shared'
-import { ErrorContextFormatter, ErrorContext } from './diagnostics/errorContext'
+import { getLogger, randomUUID } from 'aws-core-vscode/shared'
 import { AutoDebugLspClient } from './lsp/autoDebugLspClient'
-import { randomUUID } from 'aws-core-vscode/shared'
 import { mapDiagnosticSeverity } from './shared/diagnosticUtils'
 
 export interface AutoDebugConfig {
@@ -21,7 +19,6 @@ export interface Problem {
     readonly diagnostic: vscode.Diagnostic
     readonly severity: 'error' | 'warning' | 'info' | 'hint'
     readonly source: string
-    readonly isNew: boolean
 }
 
 /**
@@ -30,7 +27,6 @@ export interface Problem {
  */
 export class AutoDebugController implements vscode.Disposable {
     private readonly logger = getLogger()
-    private readonly errorFormatter: ErrorContextFormatter
     private readonly lspClient: AutoDebugLspClient
     private readonly disposables: vscode.Disposable[] = []
 
@@ -44,63 +40,16 @@ export class AutoDebugController implements vscode.Disposable {
             ...config,
         }
 
-        this.errorFormatter = new ErrorContextFormatter()
         this.lspClient = new AutoDebugLspClient(client, encryptionKey)
     }
 
-    /**
-     * Creates formatted error contexts for AI debugging
-     */
-    public async createErrorContexts(problems: Problem[]): Promise<ErrorContext[]> {
-        const contexts: ErrorContext[] = []
-
-        for (const problem of problems) {
-            try {
-                const context = await this.errorFormatter.createErrorContext(problem)
-                contexts.push(context)
-            } catch (error) {
-                this.logger.warn('AutoDebugController: Failed to create context for problem: %s', error)
-            }
-        }
-        return contexts
-    }
-
-    /**
-     * Formats problems for display or AI consumption
-     */
-    public formatProblemsForChat(problems: Problem[]): string {
-        const cwd = vscode.workspace.workspaceFolders?.[0]?.uri.fsPath || process.cwd()
-        return this.errorFormatter.formatProblemsString(problems, cwd)
-    }
-
-    /**
-     * Gets the current configuration
-     */
-    public getConfig(): AutoDebugConfig {
-        return this.config
-    }
-
-    /**
-     * Updates the configuration
-     */
-    public updateConfig(newConfig: Partial<AutoDebugConfig>): void {
-        this.config = { ...this.config, ...newConfig }
-    }
-
     /**
      * Sets the language client for LSP communication
      */
     public setLanguageClient(client: any): void {
         this.lspClient.setLanguageClient(client)
     }
 
-    /**
-     * Sends a chat message through the LSP client (public interface)
-     */
-    public async sendChatMessage(message: string, source: string): Promise<void> {
-        await this.sendMessageToChat(message)
-    }
-
     /**
      * Fix specific problems in the code
      */
@@ -130,12 +79,11 @@ export class AutoDebugController implements vscode.Disposable {
                 diagnostic,
                 severity: mapDiagnosticSeverity(diagnostic.severity),
                 source: diagnostic.source || 'unknown',
-                isNew: false,
             }))
 
             // Create fix message
             const fixMessage = this.createFixMessage(filePath, problems)
-            await this.sendChatMessage(fixMessage, 'specificFix')
+            await this.sendMessageToChat(fixMessage)
         } catch (error) {
             this.logger.error('AutoDebugController: Error fixing specific problems: %s', error)
             throw error
@@ -183,12 +131,11 @@ export class AutoDebugController implements vscode.Disposable {
                 diagnostic,
                 severity: mapDiagnosticSeverity(diagnostic.severity),
                 source: diagnostic.source || 'unknown',
-                isNew: false,
             }))
 
             // Create fix message
             const fixMessage = this.createFixMessage(filePath, problems)
-            await this.sendChatMessage(fixMessage, 'singleFix')
+            await this.sendMessageToChat(fixMessage)
         } catch (error) {
             this.logger.error('AutoDebugController: Error in fix process: %s', error)
         }
@@ -223,55 +170,17 @@ export class AutoDebugController implements vscode.Disposable {
                 diagnostic,
                 severity: mapDiagnosticSeverity(diagnostic.severity),
                 source: diagnostic.source || 'unknown',
-                isNew: false,
             }))
 
             // Create explanation message
             const explainMessage = this.createExplainMessage(filePath, problems)
-            await this.sendChatMessage(explainMessage, 'explain')
+            await this.sendMessageToChat(explainMessage)
         } catch (error) {
             this.logger.error('AutoDebugController: Error explaining problems: %s', error)
             throw error
         }
     }
 
-    /**
-     * Detect problems in the current file
-     */
-    async detectProblems(): Promise<void> {
-        try {
-            const editor = vscode.window.activeTextEditor
-            if (!editor) {
-                throw new Error('No active editor found')
-            }
-
-            const filePath = editor.document.uri.fsPath
-
-            // Get all diagnostics for the current file
-            const allDiagnostics = vscode.languages.getDiagnostics(editor.document.uri)
-
-            if (allDiagnostics.length === 0) {
-                return
-            }
-
-            // Convert diagnostics to problems
-            const problems = allDiagnostics.map((diagnostic) => ({
-                uri: editor.document.uri,
-                diagnostic,
-                severity: mapDiagnosticSeverity(diagnostic.severity),
-                source: diagnostic.source || 'unknown',
-                isNew: false,
-            }))
-
-            // Create detection message
-            const detectMessage = this.createDetectMessage(filePath, problems)
-            await this.sendChatMessage(detectMessage, 'detect')
-        } catch (error) {
-            this.logger.error('AutoDebugController: Error detecting problems: %s', error)
-            throw error
-        }
-    }
-
     private createFixMessage(filePath: string, problems: Problem[]): string {
         const parts = [`Please help me fix the following errors in ${filePath}`]
 
@@ -302,35 +211,6 @@ export class AutoDebugController implements vscode.Disposable {
         return parts.join('\n')
     }
 
-    private createDetectMessage(filePath: string, problems: Problem[]): string {
-        const errorCount = problems.filter((p) => p.severity === 'error').length
-        const warningCount = problems.filter((p) => p.severity === 'warning').length
-
-        const parts = [`I detected ${problems.length} problems in ${filePath}:`]
-
-        if (errorCount > 0) {
-            parts.push(`- ${errorCount} errors`)
-        }
-        if (warningCount > 0) {
-            parts.push(`- ${warningCount} warnings`)
-        }
-
-        parts.push('\nHere are the details:')
-
-        for (const problem of problems.slice(0, 10)) {
-            // Limit to first 10
-            const line = problem.diagnostic.range.start.line + 1
-            const source = problem.source !== 'unknown' ? problem.source : 'Unknown'
-            parts.push(`${problem.severity.toUpperCase()}: ${problem.diagnostic.message} (Line ${line}, ${source})`)
-        }
-
-        if (problems.length > 10) {
-            parts.push(`... and ${problems.length - 10} more problems`)
-        }
-
-        return parts.join('\n')
-    }
-
     /**
      * Sends message directly to language server bypassing webview connectors
      * This ensures messages go through the proper LSP chat system

packages/amazonq/src/lsp/autoDebug/shared/diagnosticUtils.ts

@@ -5,10 +5,6 @@
 
 import * as vscode from 'vscode'
 
-/**
- * Shared utility functions for diagnostic operations across AutoDebug components
- */
-
 /**
  * Maps VSCode DiagnosticSeverity to string representation
  */
@@ -26,51 +22,3 @@ export function mapDiagnosticSeverity(severity: vscode.DiagnosticSeverity): 'err
             return 'error'
     }
 }
-
-/**
- * Maps string severity to VSCode DiagnosticSeverity
- */
-export function mapSeverityToVSCode(severity: 'error' | 'warning' | 'info' | 'hint'): vscode.DiagnosticSeverity {
-    switch (severity) {
-        case 'error':
-            return vscode.DiagnosticSeverity.Error
-        case 'warning':
-            return vscode.DiagnosticSeverity.Warning
-        case 'info':
-            return vscode.DiagnosticSeverity.Information
-        case 'hint':
-            return vscode.DiagnosticSeverity.Hint
-        default:
-            return vscode.DiagnosticSeverity.Error
-    }
-}
-
-/**
- * Gets diagnostics for a specific range in a document
- */
-export function getDiagnosticsForRange(uri: vscode.Uri, range?: vscode.Range): vscode.Diagnostic[] {
-    const allDiagnostics = vscode.languages.getDiagnostics(uri)
-
-    if (!range) {
-        return allDiagnostics
-    }
-
-    return allDiagnostics.filter((diagnostic) => diagnostic.range.intersection(range) !== undefined)
-}
-
-/**
- * Filters diagnostics by severity levels
- */
-export function filterDiagnosticsBySeverity(
-    diagnostics: vscode.Diagnostic[],
-    severities: vscode.DiagnosticSeverity[]
-): vscode.Diagnostic[] {
-    return diagnostics.filter((diagnostic) => severities.includes(diagnostic.severity))
-}
-
-/**
- * Gets only error-level diagnostics
- */
-export function getErrorDiagnostics(diagnostics: vscode.Diagnostic[]): vscode.Diagnostic[] {
-    return filterDiagnosticsBySeverity(diagnostics, [vscode.DiagnosticSeverity.Error])
-}