`19f8bdf`

fix: show DiffWidget for edits even when old_string is empty

- Changed condition from 'old_string and new_string' to 'old_string is not None'
- This handles edits where old_string='' (inserting new content)
- Added debug logging with character counts

Authored by

espadonne 4 months ago

SHA: 19f8bdf4c75b168bf72bfde28301da671a277da9
Parents: 5946d38
Tree: 8ca69c9

1 changed file

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

src/loader/ui/app.pymodified

          tool_widget = self._tool_widget_queue.pop(0) if self._tool_widget_queue else None
          # Check if this is an edit tool with diff info
 -        if message.tool_name == "edit" and message.old_string and message.new_string:
 +        # 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:
              # Replace tool widget with diff widget
 -            self._debug_log("  -> showing EDIT diff widget")
 +            self._debug_log(f"  -> showing EDIT diff widget (old={len(message.old_string)} chars, new={len(message.new_string)} chars)")
              if tool_widget:
                  tool_widget.remove()
              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:
 -            self._debug_log("  -> showing WRITE diff widget")
 +            self._debug_log(f"  -> showing WRITE diff widget ({len(message.new_string)} chars)")
              if tool_widget:
                  tool_widget.remove()