Fix incorrect pluralization of “entry”#3933
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 fix correcting pluralization of 'entry' to 'entries'. No runtime behavior changes - only affects displayed text strings. You can customize Macroscope's approvability policy. Learn more. |
What changed
Updated the hidden-entry label to use the correct singular or plural form:
1 log entry2 log entries1 tool call2 tool callsWhy
The previous implementation formed the plural by appending
s, which produced the incorrect wordentrys.UI Changes
Before:

After:
Same but "entrys" is changed to "entries"
Checklist
Note
Fix incorrect pluralization of 'log entry' in collapsed toggle label
The
labelNouninMessagesTimeline.WorkGroupToggleTimelineRownow selects the full singular or plural form ('log entry'/'log entries','tool call'/'tool calls') based onrow.hiddenCount, replacing a suffix-appending approach that produced'log entrys'.Macroscope summarized eeb8c72.
Note
Low Risk
Copy-only change in a chat timeline UI label with no logic, data, or API impact.
Overview
Fixes the collapsed work-group toggle copy in
MessagesTimelineso hidden counts read naturally instead of producing "log entrys".WorkGroupToggleTimelineRownow picks the full noun phrase fromrow.hiddenCountandrow.onlyToolEntries(log entry/log entries,tool call/tool calls) and drops the old suffix-based pluralization.Reviewed by Cursor Bugbot for commit eeb8c72. Bugbot is set up for automated code reviews on this repo. Configure here.