refactor(svn): Removing output channel function simplifies log processing

Removes the code related to the VS Code output channel, simplifies the implementation of SvnLogger, and uses only console for logging.
At the same time, all interactive logic related to "Show Output" is removed to simplify the display of errors and information prompts.

Author: ubuntutester202

src/activate/registerCommands.ts

@@ -265,34 +265,19 @@ const getCommandsMap = ({ context, outputChannel, provider }: RegisterCommandOpt
 			})
 
 			// Show success message
-			const action = await vscode.window.showInformationMessage(
-				`SVN Debug Complete! Found ${commits.length} commits.\n\nRepository URL: ${repoInfo.repositoryUrl || "Unknown"}\nWorking Copy Root: ${repoInfo.workingCopyRoot || "Unknown"}\n\nCheck "Roo Code - SVN Debug" output channel for detailed information.`,
+			vscode.window.showInformationMessage(
+				`SVN Debug Complete! Found ${commits.length} commits.\n\nRepository URL: ${repoInfo.repositoryUrl || "Unknown"}\nWorking Copy Root: ${repoInfo.workingCopyRoot || "Unknown"}`,
 				{ modal: false },
-				"Show Output",
 			)
-
-			if (action === "Show Output") {
-				SvnLogger.showOutput()
-			}
 		} catch (error) {
 			// This should rarely happen now since individual functions handle their own errors
 			const svnError = error instanceof Error ? error : new Error(String(error))
 			SvnLogger.error("SVN debug failed", svnError)
 
-			vscode.window
-				.showErrorMessage(
-					"SVN debug operation failed",
-					{
-						modal: false,
-						detail: `Unexpected error: ${svnError.message}\n\nPlease check the SVN Debug output channel for more details.`,
-					},
-					"Show Output",
-				)
-				.then((action) => {
-					if (action === "Show Output") {
-						SvnLogger.showOutput()
-					}
-				})
+			vscode.window.showErrorMessage("SVN debug operation failed", {
+				modal: false,
+				detail: `Unexpected error: ${svnError.message}`,
+			})
 		}
 	},
 })

src/utils/__tests__/svn.spec.ts

@@ -22,11 +22,7 @@ vi.mock("vscode", () => ({
 		],
 	},
 	window: {
-		createOutputChannel: vi.fn(() => ({
-			appendLine: vi.fn(),
-			show: vi.fn(),
-		})),
-		showErrorMessage: vi.fn(() => Promise.resolve("Show Output")),
+		showErrorMessage: vi.fn(() => Promise.resolve()),
 		showWarningMessage: vi.fn(() => Promise.resolve()),
 		showInformationMessage: vi.fn(() => Promise.resolve()),
 	},

src/utils/svn.ts

@@ -46,67 +46,24 @@ function convertSvnOutput(output: Buffer | string): string {
 const execAsync = promisify(exec)
 const SVN_OUTPUT_LINE_LIMIT = 500
 
-// SVN Debug Logger
+// SVN Debug Logger - simplified to use console only
 export class SvnLogger {
-	private static outputChannel: vscode.OutputChannel | null = null
-
-	static getOutputChannel(): vscode.OutputChannel {
-		if (!this.outputChannel) {
-			this.outputChannel = vscode.window.createOutputChannel("Roo Code - SVN Debug")
-		}
-		return this.outputChannel
-	}
-
 	static debug(message: string, ...args: any[]) {
 		const timestamp = new Date().toISOString()
-		const logMessage = `[${timestamp}] [DEBUG] ${message}`
-
-		// Log to console
+		const logMessage = `[${timestamp}] [SVN DEBUG] ${message}`
 		console.log(logMessage, ...args)
-
-		// Log to VS Code output channel
-		const channel = this.getOutputChannel()
-		channel.appendLine(logMessage)
-		if (args.length > 0) {
-			channel.appendLine(`  Args: ${JSON.stringify(args, null, 2)}`)
-		}
 	}
 
 	static error(message: string, error?: any) {
 		const timestamp = new Date().toISOString()
-		const logMessage = `[${timestamp}] [ERROR] ${message}`
-
-		// Log to console
+		const logMessage = `[${timestamp}] [SVN ERROR] ${message}`
 		console.error(logMessage, error)
-
-		// Log to VS Code output channel
-		const channel = this.getOutputChannel()
-		channel.appendLine(logMessage)
-		if (error) {
-			channel.appendLine(`  Error: ${error.toString()}`)
-			if (error.stack) {
-				channel.appendLine(`  Stack: ${error.stack}`)
-			}
-		}
 	}
 
 	static info(message: string, ...args: any[]) {
 		const timestamp = new Date().toISOString()
-		const logMessage = `[${timestamp}] [INFO] ${message}`
-
-		// Log to console
+		const logMessage = `[${timestamp}] [SVN INFO] ${message}`
 		console.log(logMessage, ...args)
-
-		// Log to VS Code output channel
-		const channel = this.getOutputChannel()
-		channel.appendLine(logMessage)
-		if (args.length > 0) {
-			channel.appendLine(`  Data: ${JSON.stringify(args, null, 2)}`)
-		}
-	}
-
-	static showOutput() {
-		this.getOutputChannel().show()
 	}
 }
 
@@ -411,20 +368,10 @@ class SvnErrorHandler {
 		}
 
 		// Generic error with context
-		vscode.window
-			.showErrorMessage(
-				`SVN ${context} failed`,
-				{
-					modal: false,
-					detail: `Error: ${error.message}\n\nPlease check the SVN Debug output channel for more details.`,
-				},
-				"Show Output",
-			)
-			.then((action) => {
-				if (action === "Show Output") {
-					SvnLogger.showOutput()
-				}
-			})
+		vscode.window.showErrorMessage(`SVN ${context} failed`, {
+			modal: false,
+			detail: `Error: ${error.message}`,
+		})
 	}
 }