fix: use None check instead of falsy (#4705)

This commit is contained in:
Ryan H. Tran 2024-11-02 23:44:03 +07:00 committed by GitHub
parent 7b8241e424
commit 4446d3180f
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194

View File

@ -46,13 +46,13 @@ class EditTool:
if command == 'view':
return self.view(_path, view_range)
elif command == 'create':
if not file_text:
if file_text is None:
raise ToolError('Parameter `file_text` is required for command: create')
self.write_file(_path, file_text)
self._file_history[_path].append(file_text)
return ToolResult(output=f'File created successfully at: {_path}')
elif command == 'str_replace':
if not old_str:
if old_str is None:
raise ToolError(
'Parameter `old_str` is required for command: str_replace'
)
@ -62,7 +62,7 @@ class EditTool:
raise ToolError(
'Parameter `insert_line` is required for command: insert'
)
if not new_str:
if new_str is None:
raise ToolError('Parameter `new_str` is required for command: insert')
return self.insert(_path, insert_line, new_str)
elif command == 'undo_edit':