`520c30f`

Show error state for failed write/edit tool calls instead of DiffWidget

Authored by

espadonne 1 month ago

Status	File	+	-
M	`src/loader/ui/app.py`	3	3

src/loader/ui/app.pymodified

          # Check if this is an edit tool with diff info
          # Note: old_string can be empty string (inserting), so check `is not None`
 -        if message.tool_name == "edit" and message.new_string and message.old_string is not None:
 +        if message.tool_name == "edit" and message.new_string and message.old_string is not None and not message.is_error:
              # Replace tool widget with diff widget
              self._debug_log(
                  "  -> showing EDIT diff widget "
                  new_string=message.new_string,
+             )
              msg_area.mount(diff_widget)
 -        # Check if this is a write tool - show as diff (new file)
 -        elif message.tool_name == "write" and message.new_string:
 +        # Check if this is a write tool - show as diff (new file), but only on success
 +        elif message.tool_name == "write" and message.new_string and not message.is_error:
              self._debug_log(f"  -> showing WRITE diff widget ({len(message.new_string)} chars)")
              if tool_widget:
                  tool_widget.remove()