mirror of
https://github.com/lukaszraczylo/filepuff-mcp.git
synced 2026-10-12 04:54:27 +00:00
fix(edit): point empty-replace error at operation=delete
A replace with empty new_content failed with a bare 'new_content is required' message, leaving the model to guess. Append a hint to use operation=delete when the intent is removal. Adds TestValidateReplaceEmptySuggestsDelete.
This commit is contained in:
@@ -101,6 +101,22 @@ func TestValidateEdit(t *testing.T) {
|
||||
}
|
||||
}
|
||||
|
||||
// A replace with empty new_content should point the caller at operation=delete.
|
||||
func TestValidateReplaceEmptySuggestsDelete(t *testing.T) {
|
||||
e := NewEngine(parser.NewRegistry())
|
||||
err := e.validateASTEdit(&ASTEdit{
|
||||
File: "test.go",
|
||||
Operation: EditReplace,
|
||||
Selector: ASTSelector{Kind: "function_declaration"},
|
||||
})
|
||||
if err == nil {
|
||||
t.Fatal("expected error for replace with empty new_content")
|
||||
}
|
||||
if !strings.Contains(err.Error(), "operation=delete") {
|
||||
t.Errorf("error should suggest operation=delete, got: %v", err)
|
||||
}
|
||||
}
|
||||
|
||||
func TestResolveSelector(t *testing.T) {
|
||||
registry := parser.NewRegistry()
|
||||
defer registry.Close()
|
||||
|
||||
Reference in New Issue
Block a user