Implement extraction and normalization of file paths containing backslashes

Author: robertpiosik

packages/shared/src/utils/extract-path-from-line-of-code.spec.ts

@@ -7,6 +7,12 @@ describe('extract_path_from_line_of_code', () => {
     )
   })
 
+  it('should extract filename from // comment with back slashes', () => {
+    expect(extract_path_from_line_of_code('// path\\to\\file.ts')).toBe(
+      'path/to/file.ts'
+    )
+  })
+
   it('should extract filename with special characters in path components', () => {
     expect(extract_path_from_line_of_code('// [path]/(to)/file.ts')).toBe(
       '[path]/(to)/file.ts'

packages/shared/src/utils/extract-path-from-line-of-code.ts

@@ -19,10 +19,11 @@ export const extract_path_from_line_of_code = (line: string): string | null => {
   // Look for path pattern anywhere in comment with file extension
   // Matches sequences like "path/to/file.ext" or "./file.ext" or "../file.ext" or ".gitignore"
   const path_match = stripped.match(
-    /(?:^|\s)((?:\.|[.\/\w\-\[\]\(\)]+\.)[\w\-]{1,10})(?:\s|$)/
+    /(?:^|\s)((?:\.|[.\/\\\w\-\[\]\(\)]+\.)[\w\-]{1,10})(?:\s|$)/
   )
   if (path_match) {
-    return path_match[1]
+    // Normalize backslashes to forward slashes
+    return path_match[1].replace(/\\/g, '/')
   }
 
   return null

packages/vscode/src/commands/apply-chat-response-command/utils/clipboard-parser/clipboard-parser.spec.ts

@@ -119,6 +119,20 @@ describe('clipboard-parser', () => {
       )
       expect(frontendFile).toBeUndefined()
     })
+
+    it('should parse paths with backslashes', () => {
+      const text = load_clipboard_text('backslash-paths.txt')
+      const result = parse_clipboard_multiple_files({
+        clipboard_text: text,
+        is_single_root_folder_workspace: true
+      })
+
+      expect(result).toHaveLength(2)
+      expect(result[0].file_path).toBe('src/index.ts')
+      expect(result[0].content).toBe('console.log("hello")')
+      expect(result[1].file_path).toBe('src/utils.py')
+      expect(result[1].content).toBe(`print("hello")`)
+    })
   })
 
   describe('parse_file_content_only', () => {

packages/vscode/src/commands/apply-chat-response-command/utils/clipboard-parser/test-clipboards/backslash-paths.txt

@@ -0,0 +1,9 @@
+```typescript
+// src\index.ts
+console.log("hello")
+```
+
+```python
+# src\utils.py
+print("hello")
+```

packages/vscode/src/view/backend/message-handlers/handle-setup-api-tool-code-completions.ts

@@ -182,13 +182,13 @@ export const handle_setup_api_tool_code_completions = async (
     // Step 1: Select provider
     const provider_info = await select_provider()
     if (!provider_info) {
-      return // User cancelled, return to main quick pick
+      return // User cancelled
     }
 
     // Step 2: Select model
     const model = await select_model(provider_info)
     if (!model) {
-      return
+      return // User cancelled
     }
 
     const provider_model_exists = current_configs.some(
@@ -214,6 +214,8 @@ export const handle_setup_api_tool_code_completions = async (
 
     current_configs.push(new_config)
     await providers_manager.save_code_completions_tool_configs(current_configs)
+
+    await edit_configuration(new_config)
   }
 
   async function edit_configuration(config: ToolConfig) {
@@ -351,7 +353,6 @@ export const handle_setup_api_tool_code_completions = async (
         )
       } else {
         console.error('Could not find original config in array to update.')
-        await edit_configuration(config)
         return
       }
     }