feat: Add SVN commit support with GUI interface similar to Git commits

packages/types/src/global-settings.ts

@@ -100,6 +100,7 @@ export const globalSettingsSchema = z.object({
 	maxOpenTabsContext: z.number().optional(),
 	maxWorkspaceFiles: z.number().optional(),
 	showRooIgnoredFiles: z.boolean().optional(),
+	enableSvnContext: z.boolean().optional(),
 	maxReadFileLine: z.number().optional(),
 
 	terminalOutputLineLimit: z.number().optional(),

packages/types/src/vscode.ts

@@ -53,6 +53,8 @@ export const commandIds = [
 	"focusInput",
 	"acceptInput",
 	"focusPanel",
+
+	"debugSvn",
 ] as const
 
 export type CommandId = (typeof commandIds)[number]

src/activate/registerCommands.ts

@@ -16,6 +16,7 @@ import { CodeIndexManager } from "../services/code-index/manager"
 import { importSettingsWithFeedback } from "../core/config/importExport"
 import { MdmService } from "../services/mdm/MdmService"
 import { t } from "../i18n"
+import { checkSvnInstalled, checkSvnRepo, getSvnRepositoryInfo, searchSvnCommits, SvnLogger } from "../utils/svn"
 
 /**
  * Helper to get the visible ClineProvider instance or log if not found.
@@ -218,6 +219,82 @@ const getCommandsMap = ({ context, outputChannel, provider }: RegisterCommandOpt
 
 		visibleProvider.postMessageToWebview({ type: "acceptInput" })
 	},
+	debugSvn: async () => {
+		const workspaceFolders = vscode.workspace.workspaceFolders
+		if (!workspaceFolders || workspaceFolders.length === 0) {
+			vscode.window.showErrorMessage("No workspace folder is open")
+			return
+		}
+
+		const workspaceRoot = workspaceFolders[0].uri.fsPath
+		SvnLogger.info(`Starting SVN debug for workspace: ${workspaceRoot}`)
+
+		try {
+			// Check SVN installation and repository status
+			// Note: These functions now handle their own user-friendly error messages
+			SvnLogger.info("Checking if SVN is installed...")
+			const svnInstalled = await checkSvnInstalled()
+			SvnLogger.info(`SVN installed: ${svnInstalled}`)
+
+			if (!svnInstalled) {
+				// Error message already shown by checkSvnInstalled
+				return
+			}
+
+			SvnLogger.info("Checking if current directory is an SVN repository...")
+			const isSvnRepo = await checkSvnRepo(workspaceRoot)
+			SvnLogger.info(`Is SVN repository: ${isSvnRepo}`)
+
+			if (!isSvnRepo) {
+				// Error message already shown by checkSvnRepo
+				return
+			}
+
+			// Get SVN repository information
+			SvnLogger.info("Getting SVN repository information...")
+			const repoInfo = await getSvnRepositoryInfo(workspaceRoot)
+			SvnLogger.info(`Repository info: ${JSON.stringify(repoInfo, null, 2)}`)
+
+			// Search for recent commits
+			SvnLogger.info("Searching for recent commits...")
+			const commits = await searchSvnCommits("", workspaceRoot)
+			SvnLogger.info(`Found ${commits.length} commits`)
+
+			commits.forEach((commit, index) => {
+				SvnLogger.info(`Commit ${index + 1}: r${commit.revision} - ${commit.message}`)
+			})
+
+			// 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.`,
+				{ 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()
+					}
+				})
+		}
+	},
 })
 
 export const openClineInNewTab = async ({ context, outputChannel }: Omit<RegisterCommandOptions, "provider">) => {

src/api/providers/bedrock.ts

@@ -224,8 +224,8 @@ export class AwsBedrockHandler extends BaseProvider implements SingleCompletionH
 
 		if (this.options.awsUseApiKey && this.options.awsApiKey) {
 			// Use API key/token-based authentication if enabled and API key is set
-			clientConfig.token = { token: this.options.awsApiKey }
-			clientConfig.authSchemePreference = ["httpBearerAuth"] // Otherwise there's no end of credential problems.
+			;(clientConfig as any).token = { token: this.options.awsApiKey }
+			;(clientConfig as any).authSchemePreference = ["httpBearerAuth"] // Otherwise there's no end of credential problems.
 		} else if (this.options.awsUseProfile && this.options.awsProfile) {
 			// Use profile-based credentials if enabled and profile is set
 			clientConfig.credentials = fromIni({

src/core/mentions/index.ts

@@ -7,6 +7,7 @@ import { isBinaryFile } from "isbinaryfile"
 import { mentionRegexGlobal, unescapeSpaces } from "../../shared/context-mentions"
 
 import { getCommitInfo, getWorkingState } from "../../utils/git"
+import { getSvnWorkingState, getSvnCommitInfoForMentions } from "../../utils/svn"
 import { getWorkspacePath } from "../../utils/path"
 
 import { openFile } from "../../integrations/misc/open-file"
@@ -80,6 +81,7 @@ export async function parseMentions(
 	fileContextTracker?: FileContextTracker,
 	rooIgnoreController?: RooIgnoreController,
 	showRooIgnoredFiles: boolean = true,
+	enableSvnContext: boolean = false,
 	includeDiagnosticMessages: boolean = true,
 	maxDiagnosticMessages: number = 50,
 ): Promise<string> {
@@ -97,8 +99,12 @@ export async function parseMentions(
 			return `Workspace Problems (see below for diagnostics)`
 		} else if (mention === "git-changes") {
 			return `Working directory changes (see below for details)`
+		} else if (enableSvnContext && mention === "svn-changes") {
+			return `Working directory changes (see below for details)`
 		} else if (/^[a-f0-9]{7,40}$/.test(mention)) {
 			return `Git commit '${mention}' (see below for commit info)`
+		} else if (enableSvnContext && /^r\d+$/.test(mention)) {
+			return `SVN revision '${mention}' (see below for commit info)`
 		} else if (mention === "terminal") {
 			return `Terminal Output (see below for output)`
 		}
@@ -179,13 +185,43 @@ export async function parseMentions(
 			} catch (error) {
 				parsedText += `\n\n<git_working_state>\nError fetching working state: ${error.message}\n</git_working_state>`
 			}
+		} else if (enableSvnContext && mention === "svn-changes") {
+			try {
+				const svnWorkingState = await getSvnWorkingState(cwd)
+				console.log("[DEBUG] SVN working state object:", JSON.stringify(svnWorkingState, null, 2))
+
+				// Format the SVN working state properly
+				let formattedState = ""
+				if (svnWorkingState.status) {
+					formattedState += `Status:\n${svnWorkingState.status}\n\n`
+				}
+				if (svnWorkingState.diff) {
+					formattedState += `Diff:\n${svnWorkingState.diff}`
+				}
+				if (!formattedState.trim()) {
+					formattedState = "No changes detected in working directory."
+				}
+
+				console.log("[DEBUG] Formatted SVN state for AI:", formattedState)
+				parsedText += `\n\n<svn_working_state>\n${formattedState}\n</svn_working_state>`
+			} catch (error) {
+				console.error("[DEBUG] Error fetching SVN working state:", error)
+				parsedText += `\n\n<svn_working_state>\nError fetching SVN working state: ${error.message}\n</svn_working_state>`
+			}
 		} else if (/^[a-f0-9]{7,40}$/.test(mention)) {
 			try {
 				const commitInfo = await getCommitInfo(mention, cwd)
 				parsedText += `\n\n<git_commit hash="${mention}">\n${commitInfo}\n</git_commit>`
 			} catch (error) {
 				parsedText += `\n\n<git_commit hash="${mention}">\nError fetching commit info: ${error.message}\n</git_commit>`
 			}
+		} else if (enableSvnContext && /^r\d+$/.test(mention)) {
+			try {
+				const commitInfo = await getSvnCommitInfoForMentions(mention, cwd)
+				parsedText += `\n\n<svn_commit revision="${mention}">\n${commitInfo}\n</svn_commit>`
+			} catch (error) {
+				parsedText += `\n\n<svn_commit revision="${mention}">\nError fetching commit info: ${error.message}\n</svn_commit>`
+			}
 		} else if (mention === "terminal") {
 			try {
 				const terminalOutput = await getLatestTerminalOutput()

src/core/mentions/processUserContentMentions.ts

@@ -13,6 +13,7 @@ export async function processUserContentMentions({
 	fileContextTracker,
 	rooIgnoreController,
 	showRooIgnoredFiles = true,
+	enableSvnContext = false,
 	includeDiagnosticMessages = true,
 	maxDiagnosticMessages = 50,
 }: {
@@ -22,6 +23,7 @@ export async function processUserContentMentions({
 	fileContextTracker: FileContextTracker
 	rooIgnoreController?: any
 	showRooIgnoredFiles?: boolean
+	enableSvnContext?: boolean
 	includeDiagnosticMessages?: boolean
 	maxDiagnosticMessages?: number
 }) {
@@ -50,6 +52,7 @@ export async function processUserContentMentions({
 							fileContextTracker,
 							rooIgnoreController,
 							showRooIgnoredFiles,
+							enableSvnContext,
 							includeDiagnosticMessages,
 							maxDiagnosticMessages,
 						),
@@ -69,6 +72,7 @@ export async function processUserContentMentions({
 								fileContextTracker,
 								rooIgnoreController,
 								showRooIgnoredFiles,
+								enableSvnContext,
 								includeDiagnosticMessages,
 								maxDiagnosticMessages,
 							),
@@ -89,6 +93,7 @@ export async function processUserContentMentions({
 										fileContextTracker,
 										rooIgnoreController,
 										showRooIgnoredFiles,
+										enableSvnContext,
 										includeDiagnosticMessages,
 										maxDiagnosticMessages,
 									),

src/core/task/Task.ts

@@ -1228,6 +1228,7 @@ export class Task extends EventEmitter<ClineEvents> {
 
 		const {
 			showRooIgnoredFiles = true,
+			enableSvnContext = false,
 			includeDiagnosticMessages = true,
 			maxDiagnosticMessages = 50,
 		} = (await this.providerRef.deref()?.getState()) ?? {}
@@ -1239,6 +1240,7 @@ export class Task extends EventEmitter<ClineEvents> {
 			fileContextTracker: this.fileContextTracker,
 			rooIgnoreController: this.rooIgnoreController,
 			showRooIgnoredFiles,
+			enableSvnContext,
 			includeDiagnosticMessages,
 			maxDiagnosticMessages,
 		})

src/core/webview/ClineProvider.ts

@@ -1439,6 +1439,8 @@ export class ClineProvider
 			profileThresholds,
 			alwaysAllowFollowupQuestions,
 			followupAutoApproveTimeoutMs,
+			diagnosticsEnabled,
+			enableSvnContext,
 			includeDiagnosticMessages,
 			maxDiagnosticMessages,
 		} = await this.getState()
@@ -1561,6 +1563,8 @@ export class ClineProvider
 			hasOpenedModeSelector: this.getGlobalState("hasOpenedModeSelector") ?? false,
 			alwaysAllowFollowupQuestions: alwaysAllowFollowupQuestions ?? false,
 			followupAutoApproveTimeoutMs: followupAutoApproveTimeoutMs ?? 60000,
+			diagnosticsEnabled: diagnosticsEnabled ?? true,
+			enableSvnContext: enableSvnContext ?? false,
 			includeDiagnosticMessages: includeDiagnosticMessages ?? true,
 			maxDiagnosticMessages: maxDiagnosticMessages ?? 50,
 		}
@@ -1701,6 +1705,7 @@ export class ClineProvider
 			browserToolEnabled: stateValues.browserToolEnabled ?? true,
 			telemetrySetting: stateValues.telemetrySetting || "unset",
 			showRooIgnoredFiles: stateValues.showRooIgnoredFiles ?? true,
+			enableSvnContext: stateValues.enableSvnContext ?? false,
 			maxReadFileLine: stateValues.maxReadFileLine ?? -1,
 			maxConcurrentFileReads: stateValues.maxConcurrentFileReads ?? 5,
 			historyPreviewCollapsed: stateValues.historyPreviewCollapsed ?? false,

src/core/webview/__tests__/ClineProvider.spec.ts

@@ -542,6 +542,9 @@ describe("ClineProvider", () => {
 			profileThresholds: {},
 			hasOpenedModeSelector: false,
 			diagnosticsEnabled: true,
+			enableSvnContext: true,
+			includeDiagnosticMessages: true,
+			maxDiagnosticMessages: 50,
 		}
 
 		const message: ExtensionMessage = {

src/core/webview/webviewMessageHandler.ts

@@ -37,6 +37,7 @@ import { fileExistsAtPath } from "../../utils/fs"
 import { playTts, setTtsEnabled, setTtsSpeed, stopTts } from "../../utils/tts"
 import { singleCompletionHandler } from "../../utils/single-completion-handler"
 import { searchCommits } from "../../utils/git"
+import { searchSvnCommits } from "../../utils/svn"
 import { exportSettings, importSettingsWithFeedback } from "../config/importExport"
 import { getOpenAiModels } from "../../api/providers/openai"
 import { getVsCodeLmModels } from "../../api/providers/vscode-lm"
@@ -1257,6 +1258,10 @@ export const webviewMessageHandler = async (
 			await updateGlobalState("showRooIgnoredFiles", message.bool ?? true)
 			await provider.postStateToWebview()
 			break
+		case "enableSvnContext":
+			await updateGlobalState("enableSvnContext", message.bool ?? true)
+			await provider.postStateToWebview()
+			break
 		case "hasOpenedModeSelector":
 			await updateGlobalState("hasOpenedModeSelector", message.bool ?? true)
 			await provider.postStateToWebview()
@@ -1410,6 +1415,24 @@ export const webviewMessageHandler = async (
 			}
 			break
 		}
+		case "searchSvnCommits": {
+			const cwd = provider.cwd
+			if (cwd) {
+				try {
+					const svnCommits = await searchSvnCommits(message.query || "", cwd)
+					await provider.postMessageToWebview({
+						type: "svnCommitSearchResults",
+						svnCommits,
+					})
+				} catch (error) {
+					provider.log(
+						`Error searching SVN commits: ${JSON.stringify(error, Object.getOwnPropertyNames(error), 2)}`,
+					)
+					vscode.window.showErrorMessage("Error searching SVN commits")
+				}
+			}
+			break
+		}
 		case "searchFiles": {
 			const workspacePath = getWorkspacePath()
 

src/integrations/editor/DiffViewProvider.ts

@@ -187,7 +187,10 @@ export class DiffViewProvider {
 		}
 	}
 
-	async saveChanges(diagnosticsEnabled: boolean = true, writeDelayMs: number = DEFAULT_WRITE_DELAY_MS): Promise<{
+	async saveChanges(
+		diagnosticsEnabled: boolean = true,
+		writeDelayMs: number = DEFAULT_WRITE_DELAY_MS,
+	): Promise<{
 		newProblemsMessage: string | undefined
 		userEdits: string | undefined
 		finalContent: string | undefined
@@ -222,22 +225,22 @@ export class DiffViewProvider {
 		// and can address them accordingly. If problems don't change immediately after
 		// applying a fix, won't be notified, which is generally fine since the
 		// initial fix is usually correct and it may just take time for linters to catch up.
-		
+
 		let newProblemsMessage = ""
-		
+
 		if (diagnosticsEnabled) {
 			// Add configurable delay to allow linters time to process and clean up issues
 			// like unused imports (especially important for Go and other languages)
 			// Ensure delay is non-negative
 			const safeDelayMs = Math.max(0, writeDelayMs)
-			
+
 			try {
 				await delay(safeDelayMs)
 			} catch (error) {
 				// Log error but continue - delay failure shouldn't break the save operation
 				console.warn(`Failed to apply write delay: ${error}`)
 			}
-			
+
 			const postDiagnostics = vscode.languages.getDiagnostics()
 
 			// Get diagnostic settings from state

src/package.json

@@ -174,6 +174,11 @@
 				"command": "roo-cline.acceptInput",
 				"title": "%command.acceptInput.title%",
 				"category": "%configuration.title%"
+			},
+			{
+				"command": "roo-cline.debugSvn",
+				"title": "%command.debugSvn.title%",
+				"category": "%configuration.title%"
 			}
 		],
 		"menus": {

src/package.nls.json

@@ -25,6 +25,7 @@
 	"command.terminal.fixCommand.title": "Fix This Command",
 	"command.terminal.explainCommand.title": "Explain This Command",
 	"command.acceptInput.title": "Accept Input/Suggestion",
+	"command.debugSvn.title": "Debug SVN Integration",
 	"configuration.title": "Roo Code",
 	"commands.allowedCommands.description": "Commands that can be auto-executed when 'Always approve execute operations' is enabled",
 	"commands.deniedCommands.description": "Command prefixes that will be automatically denied without asking for approval. In case of conflicts with allowed commands, the longest prefix match takes precedence. Add * to deny all commands.",

src/services/checkpoints/excludes.ts

@@ -200,6 +200,7 @@ const getLfsPatterns = async (workspacePath: string) => {
 
 export const getExcludePatterns = async (workspacePath: string) => [
 	".git/",
+	".svn/",
 	...getBuildArtifactPatterns(),
 	...getMediaFilePatterns(),
 	...getCacheFilePatterns(),