Skip to content

Commit c91d564

Browse files
author
Eric Wheeler
committed
fix: conditionally show user edits message in file write response
Make the 'If the user's edits have addressed part of the task...' message conditional based on whether there are actual user edits. This prevents showing irrelevant guidance when no user edits were made to the file. Signed-off-by: Eric Wheeler <[email protected]>
1 parent a119ca4 commit c91d564

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

src/integrations/editor/DiffViewProvider.ts

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -290,9 +290,13 @@ export class DiffViewProvider {
290290
problems: this.newProblemsMessage || undefined,
291291
notice: {
292292
i: [
293-
"You do not need to re-read the file, as you have seen all changes in the diff above.",
293+
"You do not need to re-read the file, as you have seen all changes",
294294
"Proceed with the task using these changes as the new baseline.",
295-
"If the user's edits have addressed part of the task or changed the requirements, adjust your approach accordingly.",
295+
...(this.userEdits
296+
? [
297+
"If the user's edits have addressed part of the task or changed the requirements, adjust your approach accordingly.",
298+
]
299+
: []),
296300
],
297301
},
298302
},

0 commit comments

Comments
 (0)