chore: remove AnnotatableFileDiff leftovers, rename file#3488
chore: remove AnnotatableFileDiff leftovers, rename file#3488ipanasenko wants to merge 1 commit into
AnnotatableFileDiff leftovers, rename file#3488Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
ApprovabilityVerdict: Approved Pure cleanup PR removing unused You can customize Macroscope's approvability policy. Learn more. |
What Changed
Removed unused
AnnotatableFileDiffcomponent. Looks like it's a leftover after #3169Renamed file
AnnotatableFileDiff.tsx=>AnnotatableCodeView.tsxWhy
Just cleaning things up a bit
UI Changes
None
Checklist
I included before/after screenshots for any UI changes- N/AI included a video for animation/interaction changes- N/ANote
Remove
AnnotatableFileDiffcomponent and rename file toAnnotatableCodeViewDeletes the
AnnotatableFileDiffcomponent and its associated props interface from AnnotatableCodeView.tsx, removing theFileDiff-based rendering path along with its annotation state and handlers. Updates DiffPanel.tsx to import from the renamed module path.Macroscope summarized e1ab551.
Note
Low Risk
Dead-code removal and import path rename with no remaining references to the removed component.
Overview
Cleanup after consolidating diff review UI on
CodeView. The PR deletes the leftoverAnnotatableFileDiffimplementation (single-fileFileDiffwrapper with its own annotation state) from what is nowAnnotatableCodeView.tsx, and drops the unusedFileDiffimport from@pierre/diffs/react.DiffPanelonly changes the import path/handle type to./diffs/AnnotatableCodeView; it already rendersAnnotatableCodeViewfor multi-file diffs, so behavior should be unchanged.Reviewed by Cursor Bugbot for commit e1ab551. Bugbot is set up for automated code reviews on this repo. Configure here.