bug: space in folder and file name

Author: kunwarVivek

src/core/mentions/__tests__/index.test.ts

@@ -155,7 +155,17 @@ Detailed commit message with multiple lines
 			jest.spyOn(fs, "readFile").mockResolvedValue(fileContent)
 			jest.spyOn(fs, "stat").mockResolvedValue({ isFile: () => true, isDirectory: () => false } as any)
 
+			// Test with a file path containing spaces
 			const filePath = "/path/with spaces/my file.txt"
+
+			// First, verify that the regex pattern correctly matches the entire path
+			// Import the regex pattern directly to test it
+			const { mentionRegexGlobal } = require("../../../shared/context-mentions")
+			const mentionMatch = `@${filePath}`.match(mentionRegexGlobal)
+			expect(mentionMatch).not.toBeNull()
+			expect(mentionMatch![0]).toBe(`@${filePath}`)
+
+			// Now test the full parseMentions function
 			const result = await parseMentions(`Check out this file @${filePath}`, mockCwd, mockUrlContentFetcher)
 
 			// Verify the file path with spaces was correctly parsed

src/shared/context-mentions.ts

@@ -9,7 +9,7 @@ Mention regex:
   - `/@`:
 	- **@**: The mention must start with the '@' symbol.
 
-  - `((?:\/|\w+:\/\/)(?:[^\s]|\s(?=[^\s]))+?|problems\b|git-changes\b)`:
+  - `((?:\/|\w+:\/\/)[^\r\n]*?(?=\s*$|\s+@|[.,;:!?](?=[\s\r\n]|$))|problems\b|git-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.
@@ -18,10 +18,11 @@ Mention regex:
 	  - `|`: Logical OR.
 	  - `\w+:\/\/`:
 		- **Protocol (`\w+://`)**: Matches URLs that start with a word character sequence followed by '://', such as 'http://', 'https://', 'ftp://', etc.
-	- `(?:[^\s]|\s(?=[^\s]))+?`:
-	  - **Character Pattern**: Matches either a non-whitespace character OR a whitespace character that is followed by a non-whitespace character.
-	  - **This allows spaces within file paths while preventing trailing spaces**.
-	  - **Non-Greedy (`+?`)**: Ensures the smallest possible match, preventing the inclusion of trailing punctuation.
+	- `[^\r\n]*?(?=\s*$|\s+@|[.,;:!?](?=[\s\r\n]|$))`:
+	  - **Character Pattern**: Matches any characters except line breaks.
+	  - **Followed by a lookahead**: Ensures the match ends at the end of the line, before another @ symbol, or before punctuation followed by whitespace or end of line.
+	  - **This allows spaces within file paths while properly handling path boundaries**.
+	  - **Non-Greedy (`*?`)**: Ensures the smallest possible match.
 	- `|`: Logical OR.
 	- `problems\b`:
 	  - **Exact Word ('problems')**: Matches the exact word 'problems'.
@@ -40,6 +41,7 @@ Mention regex:
 - **Summary**:
   - The regex effectively matches:
 	- Mentions that are file or folder paths starting with '/' and can contain spaces within the path (e.g., 'my folder/my file.txt').
+	  The regex now properly handles paths with multiple spaces and ensures the entire path is captured.
 	- 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'.
@@ -51,7 +53,7 @@ Mention regex:
 
 */
 export const mentionRegex =
-	/@((?:\/|\w+:\/\/)(?:[^\s]|\s(?=[^\s]))+?|[a-f0-9]{7,40}\b|problems\b|git-changes\b|terminal\b)(?=[.,;:!?]?(?=[\s\r\n]|$))/
+	/@((?:\/|\w+:\/\/)[^\r\n]*?(?=\s*$|\s+@|[.,;:!?](?=[\s\r\n]|$))|[a-f0-9]{7,40}\b|problems\b|git-changes\b|terminal\b)(?=[.,;:!?]?(?=[\s\r\n]|$))/
 export const mentionRegexGlobal = new RegExp(mentionRegex.source, "g")
 
 export interface MentionSuggestion {

webview-ui/src/utils/context-mentions.ts

@@ -280,8 +280,9 @@ export function shouldShowContextMenu(text: string, position: number): boolean {
 
 	const textAfterAt = beforeCursor.slice(atIndex + 1)
 
-	// Check if there's any whitespace after the '@'
-	if (/\s/.test(textAfterAt)) return false
+	// Check if there's any whitespace immediately after the '@'
+	// This only checks the first character after @ to allow for paths with spaces
+	if (textAfterAt.startsWith(" ")) return false
 
 	// Don't show the menu if it's clearly a URL
 	if (textAfterAt.toLowerCase().startsWith("http")) {