diff options
| author | Wen Sun <[email protected]> | 2025-07-22 06:24:02 +0900 |
|---|---|---|
| committer | GitHub <[email protected]> | 2025-07-21 21:24:02 +0000 |
| commit | b4d00ab4fb338631de039dc445936f7af2c5b79e (patch) | |
| tree | 8f54586974f51418d28240791479523cf60901f3 /packages/core/src/tools/edit.test.ts | |
| parent | 1969d805f2fd559e2227a8822de5be66d3d8a184 (diff) | |
feat(edit): Prevent no-op edits (#3520)
Diffstat (limited to 'packages/core/src/tools/edit.test.ts')
| -rw-r--r-- | packages/core/src/tools/edit.test.ts | 13 |
1 files changed, 13 insertions, 0 deletions
diff --git a/packages/core/src/tools/edit.test.ts b/packages/core/src/tools/edit.test.ts index 84ad1daf..4ff33ff4 100644 --- a/packages/core/src/tools/edit.test.ts +++ b/packages/core/src/tools/edit.test.ts @@ -608,6 +608,19 @@ describe('EditTool', () => { /User modified the `new_string` content/, ); }); + + it('should return error if old_string and new_string are identical', async () => { + const initialContent = 'This is some identical text.'; + fs.writeFileSync(filePath, initialContent, 'utf8'); + const params: EditToolParams = { + file_path: filePath, + old_string: 'identical', + new_string: 'identical', + }; + const result = await tool.execute(params, new AbortController().signal); + expect(result.llmContent).toMatch(/No changes to apply/); + expect(result.returnDisplay).toMatch(/No changes to apply/); + }); }); describe('getDescription', () => { |
