[fix/feat:ui] Capitalize Work Log heading - #3228
Conversation
|
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 Simple copy change that capitalizes a UI heading from "work log" to "Work Log" with a matching test update. No logic or runtime behavior changes. You can customize Macroscope's approvability policy. Learn more. |
What changed
The mixed activity group label now renders as
Work Loginstead ofwork log.Why
This label is a section heading, so it should match the capitalization used by the rest of the activity UI.
UI changes
Before:
After:
Checklist
Testing
vp test apps/web/src/components/chat/MessagesTimeline.test.tsx;vp check;vp run typecheckpassed locally before splitting these PRs.Note
Capitalize 'Work Log' heading in work group sections
Updates the default group label in the
WorkGroupSectioncomponent from'work log'to'Work Log'. The corresponding test in MessagesTimeline.test.tsx is updated to match.Macroscope summarized 651f474.
Note
Low Risk
Copy-only UI tweak with a matching test update; no behavior or data paths change.
Overview
Capitalizes the mixed-activity work group heading from
work logtoWork LoginWorkGroupSection(MessagesTimeline.tsx), so it reads like the other activity section titles. Tool-only groups still use1 tool call/N tool callsand are unchanged.The
MessagesTimelinetest that asserts context compaction appears in that group now expectsWork Login the markup.Reviewed by Cursor Bugbot for commit 651f474. Bugbot is set up for automated code reviews on this repo. Configure here.