cli: clarify session-title cap is a content cap (Copilot review) · Entire
cli: clarify session-title cap is a content cap (Copilot review)
a7dcecd→main·
Soph·1w ago·1 file·+5 added/-2 removed
The 120 is a cap on the display-name content, matching entire.io's `slice(0, 120) + "…"`; the ellipsis is appended on top, so the rendered title can be 121 runes. Reword the comment so it doesn't read as a hard total-length cap. No behavior change.
Co-Authored-By: Claude Opus 4.8 (1M context) noreply@anthropic.com
Sessions
01KWVW5EG0CXK0W1F2C8MPNJBVView transcript
[?
Fix Sessions Endpoint in Activity FeedClaude Code·1 step](/content/gh/entireio/cli/session/1b0292dc-2cb1-49e9-91f5-41188abf9147#timeline-01KWVW5EG0CXK0W1F2C8MPNJBV/index.html)
Changes
1
cmd/entire/cli
Mactivity_render.go+5/-2
525 unmodified lines
526
527
528
529
530
529
530
531
532
533
534
535
536
525 unmodified lines
\t}
// sessionTitleMaxRunes mirrors entire.io's 120-char displayName cap on the
// Overview row (width-based truncation may shorten it further on the terminal).
// sessionTitleMaxRunes caps the display-name *content* at 120 runes, matching
// entire.io's Overview row (`displayName.slice(0, 120) + "…"`): the ellipsis is
// appended as an overflow marker on top of the 120, so the rendered title can
// be 121 runes — this is a content cap, not a hard total-length cap. On a real
// terminal the width-based truncation below usually shortens it further first.
const sessionTitleMaxRunes = 120
func renderSessionList(w io.Writer, sty activityStyles, days []sessionDay) {
Mcmd/entire/cli/activity_render.go+5/-2