diff options
| author | Taylor Mullen <[email protected]> | 2025-05-17 23:04:26 -0700 |
|---|---|---|
| committer | N. Taylor Mullen <[email protected]> | 2025-05-17 23:06:50 -0700 |
| commit | f0b9199a772d23f8c9375c8cd627da345bc205ae (patch) | |
| tree | 30ea82879a4c6c35c12a065d4dc731636d8ef6c9 /packages/server/src | |
| parent | 5bddf40fd19f8a9d402aec8c941e257266aa03c8 (diff) | |
refactor: Remove console.error from WriteFileTool
- Removes an unnecessary `console.error` call from the `shouldConfirmExecute` method in the `WriteFileTool` class.
- This logging was redundant as validation errors are already handled and returned by the method.
- Additionally, `console.error` is not suitable for this scenario, as incorrect arguments can be provided by the LLM, and these are anticipated and managed without needing an error log.
Fixes https://b.corp.google.com/issues/418491206
Diffstat (limited to 'packages/server/src')
| -rw-r--r-- | packages/server/src/tools/write-file.ts | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/packages/server/src/tools/write-file.ts b/packages/server/src/tools/write-file.ts index c628ce75..c9f95ea9 100644 --- a/packages/server/src/tools/write-file.ts +++ b/packages/server/src/tools/write-file.ts @@ -132,9 +132,6 @@ export class WriteFileTool extends BaseTool<WriteFileToolParams, ToolResult> { const validationError = this.validateToolParams(params); if (validationError) { - console.error( - `[WriteFile Wrapper] Attempted confirmation with invalid parameters: ${validationError}`, - ); return false; } |