Skip to content

show full shell session labels instead of truncating to 140px#497

Merged
atomantic merged 2 commits into
mainfrom
cos/task-mpm56x8j/agent-ec11f870
May 26, 2026
Merged

show full shell session labels instead of truncating to 140px#497
atomantic merged 2 commits into
mainfrom
cos/task-mpm56x8j/agent-ec11f870

Conversation

@atomantic

Copy link
Copy Markdown
Owner

Summary

The Shell page truncated session names with an ellipsis (truncate max-w-[140px]), which hid the agent id portion of labels like Claude Code TUI agent-ec11f870 — so the user couldn't tell sessions apart at a glance. Drop the width cap and switch the tab strip from horizontal scroll (overflow-x-auto) to flex-wrap so longer labels can flow onto multiple lines.

Test plan

  • Open the Shell page with multiple sessions (including agent TUI sessions whose labels include <provider name> <agent id>)
  • Confirm full labels render without ellipsis
  • Open with 4–5 sessions so the strip overflows the viewport width; confirm tabs wrap to a second row instead of clipping or horizontally scrolling
  • On mobile width, confirm tabs wrap cleanly and remain tappable

The Shell page truncated session names with an ellipsis (max-w-[140px]),
which hid the agent id portion of labels like 'Claude Code TUI agent-ec11f870'.
Drop the width cap and switch the tab strip from horizontal scroll to
flex-wrap so longer labels can flow onto multiple lines.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the Shell page session tab strip to display full session labels (including agent IDs) by removing the fixed truncation width and allowing the tab list to wrap onto multiple lines instead of horizontally scrolling.

Changes:

  • Switch session tab container from horizontal scroll (overflow-x-auto) to wrapping layout (flex-wrap).
  • Remove truncate max-w-[140px] from session label rendering so full labels are visible.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread client/src/pages/Shell.jsx
Copilot flagged that with truncation removed, a very long unbroken label
(e.g. a long cwd basename or agent id) could push the flex tab wider
than the viewport. flex-wrap on the parent strip only wraps between
tabs, not inside one. Add min-w-0 break-all on the label span so the
label itself wraps and the tab stays within the viewport.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@atomantic atomantic merged commit ccd26c9 into main May 26, 2026
3 checks passed
@atomantic atomantic deleted the cos/task-mpm56x8j/agent-ec11f870 branch May 26, 2026 05:11
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.

2 participants