Skip to content

refactor(tarko): merge EditFileRenderer into DiffRenderer#1259

Merged
ulivz merged 2 commits intomainfrom
feat/merge-diff-renderers
Aug 25, 2025
Merged

refactor(tarko): merge EditFileRenderer into DiffRenderer#1259
ulivz merged 2 commits intomainfrom
feat/merge-diff-renderers

Conversation

@ulivz
Copy link
Member

@ulivz ulivz commented Aug 25, 2025

Summary

Merged redundant EditFileRenderer into DiffRenderer to consolidate diff rendering functionality. The EditFileRenderer was only used for str_replace_editor content, while DiffRenderer handled standard diff format. Now DiffRenderer handles both cases:

  • str_replace_editor content with old/new content comparison (previously edit_file type)
  • Standard unified diff format content (existing diff_result type)

Updated str_replace_editor renderer condition to use diff_result instead of edit_file type.

Checklist

  • Added or updated necessary tests (Optional).
  • Updated documentation to align with changes (Optional).
  • Verified no breaking changes, or prepared solutions for any occurring breaking changes (Optional).
  • My change does not involve the above items.

@netlify
Copy link

netlify bot commented Aug 25, 2025

Deploy Preview for agent-tars-docs ready!

Name Link
🔨 Latest commit 5796b38
🔍 Latest deploy log https://app.netlify.com/projects/agent-tars-docs/deploys/68ac753123c1a20008423fd3
😎 Deploy Preview https://deploy-preview-1259--agent-tars-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@ulivz ulivz changed the title feat(tarko): merge EditFileRenderer into DiffRenderer refactor(tarko): merge EditFileRenderer into DiffRenderer Aug 25, 2025
@ulivz ulivz merged commit 173a03d into main Aug 25, 2025
10 checks passed
@ulivz ulivz deleted the feat/merge-diff-renderers branch August 25, 2025 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Comments