feat(svn): 添加对svn-changes提及的支持，主要是让API请求包含svn的status、diff信息

- 在mentionRegex中添加svn-changes匹配项
- 实现获取SVN工作状态的函数并添加调试日志
- 在parseMentions中处理svn-changes提及，返回工作目录变更信息
- 在processUserContentMentions中添加调试日志

Author: ubuntutester202

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 } from "../../utils/svn"
 import { getWorkspacePath } from "../../utils/path"
 
 import { openFile } from "../../integrations/misc/open-file"
@@ -95,6 +96,8 @@ 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 (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 (mention === "terminal") {
@@ -177,6 +180,29 @@ export async function parseMentions(
 			} catch (error) {
 				parsedText += `\n\n<git_working_state>\nError fetching working state: ${error.message}\n</git_working_state>`
 			}
+		} else if (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)

src/core/mentions/processUserContentMentions.ts

@@ -21,6 +21,8 @@ export async function processUserContentMentions({
 	rooIgnoreController?: any
 	showRooIgnoredFiles?: boolean
 }) {
+	console.log("[DEBUG] Processing user content mentions, input userContent:", JSON.stringify(userContent, null, 2))
+
 	// Process userContent array, which contains various block types:
 	// TextBlockParam, ImageBlockParam, ToolUseBlockParam, and ToolResultBlockParam.
 	// We need to apply parseMentions() to:
@@ -31,39 +33,45 @@ export async function processUserContentMentions({
 	// (see askFollowupQuestion), we place all user generated content in
 	// these tags so they can effectively be used as markers for when we
 	// should parse mentions).
-	return Promise.all(
+	const result = await Promise.all(
 		userContent.map(async (block) => {
 			const shouldProcessMentions = (text: string) => text.includes("<task>") || text.includes("<feedback>")
 
 			if (block.type === "text") {
 				if (shouldProcessMentions(block.text)) {
+					console.log("[DEBUG] Processing mentions in text block:", block.text)
+					const processedText = await parseMentions(
+						block.text,
+						cwd,
+						urlContentFetcher,
+						fileContextTracker,
+						rooIgnoreController,
+						showRooIgnoredFiles,
+					)
+					console.log("[DEBUG] Processed text block result:", processedText)
 					return {
 						...block,
-						text: await parseMentions(
-							block.text,
-							cwd,
-							urlContentFetcher,
-							fileContextTracker,
-							rooIgnoreController,
-							showRooIgnoredFiles,
-						),
+						text: processedText,
 					}
 				}
 
 				return block
 			} else if (block.type === "tool_result") {
 				if (typeof block.content === "string") {
 					if (shouldProcessMentions(block.content)) {
+						console.log("[DEBUG] Processing mentions in tool_result string content:", block.content)
+						const processedContent = await parseMentions(
+							block.content,
+							cwd,
+							urlContentFetcher,
+							fileContextTracker,
+							rooIgnoreController,
+							showRooIgnoredFiles,
+						)
+						console.log("[DEBUG] Processed tool_result string content result:", processedContent)
 						return {
 							...block,
-							content: await parseMentions(
-								block.content,
-								cwd,
-								urlContentFetcher,
-								fileContextTracker,
-								rooIgnoreController,
-								showRooIgnoredFiles,
-							),
+							content: processedContent,
 						}
 					}
 
@@ -72,16 +80,22 @@ export async function processUserContentMentions({
 					const parsedContent = await Promise.all(
 						block.content.map(async (contentBlock) => {
 							if (contentBlock.type === "text" && shouldProcessMentions(contentBlock.text)) {
+								console.log(
+									"[DEBUG] Processing mentions in tool_result array content:",
+									contentBlock.text,
+								)
+								const processedText = await parseMentions(
+									contentBlock.text,
+									cwd,
+									urlContentFetcher,
+									fileContextTracker,
+									rooIgnoreController,
+									showRooIgnoredFiles,
+								)
+								console.log("[DEBUG] Processed tool_result array content result:", processedText)
 								return {
 									...contentBlock,
-									text: await parseMentions(
-										contentBlock.text,
-										cwd,
-										urlContentFetcher,
-										fileContextTracker,
-										rooIgnoreController,
-										showRooIgnoredFiles,
-									),
+									text: processedText,
 								}
 							}
 
@@ -98,4 +112,7 @@ export async function processUserContentMentions({
 			return block
 		}),
 	)
+
+	console.log("[DEBUG] Final processed userContent for AI:", JSON.stringify(result, null, 2))
+	return result
 }

src/shared/context-mentions.ts

@@ -9,7 +9,7 @@ Mention regex:
   - `/@`: 
 	- **@**: The mention must start with the '@' symbol.
   
-  - `((?:\/|\w+:\/\/)[^\s]+?|problems\b|git-changes\b)`:
+  - `((?:\/|\w+:\/\/)[^\s]+?|problems\b|git-changes\b|svn-changes\b)`:
 	- **Capturing Group (`(...)`)**: Captures the part of the string that matches one of the specified patterns.
 	- `(?:\/|\w+:\/\/)`: 
 	  - **Non-Capturing Group (`(?:...)`)**: Groups the alternatives without capturing them for back-referencing.
@@ -46,6 +46,7 @@ Mention regex:
 	- URLs that start with a protocol (like 'http://') followed by any non-whitespace characters (including query parameters).
 	- The exact word 'problems'.
 	- The exact word 'git-changes'.
+	- The exact word 'svn-changes'.
     - The exact word 'terminal'.
   - It ensures that any trailing punctuation marks (such as ',', '.', '!', etc.) are not included in the matched mention, allowing the punctuation to follow the mention naturally in the text.
 
@@ -54,7 +55,7 @@ Mention regex:
 
 */
 export const mentionRegex =
-	/(?<!\\)@((?:\/|\w+:\/\/)(?:[^\s\\]|\\ )+?|[a-f0-9]{7,40}\b|problems\b|git-changes\b|terminal\b)(?=[.,;:!?]?(?=[\s\r\n]|$))/
+	/(?<!\\)@((?:\/|\w+:\/\/)(?:[^\s\\]|\\ )+?|[a-f0-9]{7,40}\b|problems\b|git-changes\b|svn-changes\b|terminal\b)(?=[.,;:!?]?(?=[\s\r\n]|$))/
 export const mentionRegexGlobal = new RegExp(mentionRegex.source, "g")
 
 export interface MentionSuggestion {

src/utils/svn.ts

@@ -419,31 +419,45 @@ export async function getSvnWorkingState(cwd: string): Promise<{
 	diff: string
 }> {
 	try {
+		console.log("[DEBUG] getSvnWorkingState called with cwd:", cwd)
+
 		if (!(await checkSvnInstalled()) || !(await checkSvnRepo(cwd))) {
+			console.log("[DEBUG] SVN not installed or not a repository")
 			return { status: "", diff: "" }
 		}
 
 		// Get status
 		let status = ""
 		try {
+			console.log("[DEBUG] Executing 'svn status' command")
 			const { stdout: statusOutput } = await execAsync("svn status", { cwd })
+			console.log("[DEBUG] SVN status raw output:", JSON.stringify(statusOutput))
 			status = truncateOutput(statusOutput, SVN_OUTPUT_LINE_LIMIT)
-		} catch {
+			console.log("[DEBUG] SVN status after truncation:", JSON.stringify(status))
+		} catch (error) {
+			console.log("[DEBUG] SVN status command failed:", error)
 			// Status might not be available
 		}
 
 		// Get diff of working changes
 		let diff = ""
 		try {
+			console.log("[DEBUG] Executing 'svn diff' command")
 			const { stdout: diffOutput } = await execAsync("svn diff", { cwd })
+			console.log("[DEBUG] SVN diff raw output length:", diffOutput.length)
+			console.log("[DEBUG] SVN diff raw output preview:", JSON.stringify(diffOutput.substring(0, 500)))
 			diff = truncateOutput(diffOutput, SVN_OUTPUT_LINE_LIMIT)
-		} catch {
+			console.log("[DEBUG] SVN diff after truncation length:", diff.length)
+		} catch (error) {
+			console.log("[DEBUG] SVN diff command failed:", error)
 			// Diff might not be available
 		}
 
-		return { status, diff }
+		const result = { status, diff }
+		console.log("[DEBUG] getSvnWorkingState returning:", JSON.stringify(result))
+		return result
 	} catch (error) {
-		console.error("Error getting SVN working state:", error)
+		console.error("[DEBUG] Error in getSvnWorkingState:", error)
 		return { status: "", diff: "" }
 	}
 }