opencode/picoclaw
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix error message

Browse source
This commit is contained in:
afjcjsbx 2026-03-18 20:05:50 +01:00
parent 195610f050
commit d97e03bd22
2 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
pkg/tools/filesystem.go
View file

@ -536,7 +536,7 @@ func (t *WriteFileTool) Execute(ctx context.Context, args map[string]any) *ToolR
if !overwrite { if !overwrite {
if _, err := t.fs.Open(path); err == nil { if _, err := t.fs.Open(path); err == nil {
return ErrorResult(fmt.Sprintf("file already exists: %s (set overwrite=true to replace it)", path)) return ErrorResult(fmt.Sprintf("file: %s already exists. Set overwrite=true to replace.", path))
} }
} }

6
pkg/tools/filesystem_test.go
View file

@ -203,7 +203,7 @@ func TestFilesystemTool_WriteFile_OverwriteDefaultBlocked(t *testing.T) {
}) })
assert.True(t, result.IsError, "expected error when overwriting without overwrite=true") assert.True(t, result.IsError, "expected error when overwriting without overwrite=true")
assert.Contains(t, result.ForLLM, "file already exists") assert.Contains(t, result.ForLLM, "already exists")
assert.Contains(t, result.ForLLM, "overwrite=true") assert.Contains(t, result.ForLLM, "overwrite=true")
// Original content must be untouched // Original content must be untouched
@ -267,7 +267,7 @@ func TestFilesystemTool_WriteFile_OverwriteFalseExplicitBlocked(t *testing.T) {
}) })
assert.True(t, result.IsError, "expected error when overwrite=false") assert.True(t, result.IsError, "expected error when overwrite=false")
assert.Contains(t, result.ForLLM, "file already exists") assert.Contains(t, result.ForLLM, "already exists")
data, err := os.ReadFile(testFile) data, err := os.ReadFile(testFile)
assert.NoError(t, err) assert.NoError(t, err)
@ -289,7 +289,7 @@ func TestFilesystemTool_WriteFile_OverwriteSandboxed(t *testing.T) {
"content": "new content", "content": "new content",
}) })
assert.True(t, result.IsError, "expected error in sandbox mode without overwrite=true") assert.True(t, result.IsError, "expected error in sandbox mode without overwrite=true")
assert.Contains(t, result.ForLLM, "file already exists") assert.Contains(t, result.ForLLM, "already exists")
// With overwrite=true → allowed // With overwrite=true → allowed
result = tool.Execute(context.Background(), map[string]any{ result = tool.Execute(context.Background(), map[string]any{