Skip to content

Commit 03d0c7d

Browse files
committed
Add test case and fix parser to handle file paths in markdown headings without extracting content from first line
1 parent e1c7f23 commit 03d0c7d

4 files changed

Lines changed: 27 additions & 1 deletion

File tree

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

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,22 @@ describe('clipboard-parser', () => {
1212
}
1313

1414
describe('parse_multiple_files', () => {
15+
it("_temp test case that doesn't fail when files are empty", () => {
16+
const test_case = '_temp'
17+
const text = load_test_case_file(test_case, 'temp.txt')
18+
const result = parse_multiple_files({
19+
response: text,
20+
is_single_root_folder_workspace: true
21+
})
22+
23+
expect(result).toHaveLength(1)
24+
expect(result[0]).toMatchObject({
25+
type: 'file',
26+
file_path: 'README.md',
27+
content: load_test_case_file(test_case, '1-file.txt')
28+
})
29+
})
30+
1531
it('parses multiple files when file paths are in comments at the start of code blocks', () => {
1632
const test_case = 'comment-filename'
1733
const text = load_test_case_file(test_case, `${test_case}.txt`)

packages/vscode/src/commands/apply-chat-response-command/utils/clipboard-parser/parsers/multiple-files-parser.ts

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -684,7 +684,11 @@ export const parse_multiple_files = (params: {
684684
}
685685
}
686686

687-
if (is_first_content_line && !xml_file_mode) {
687+
if (
688+
is_first_content_line &&
689+
!xml_file_mode &&
690+
!last_seen_file_path_was_header
691+
) {
688692
const trimmed_line = line.trim()
689693
let extracted_filename: string | null = null
690694

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
Testing...
Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
### Updated file: `README.md`
2+
3+
```
4+
Testing...
5+
```

0 commit comments

Comments
 (0)