Skip to content

fix(cli): show full session titles in /resume list (#14082)#31710

Merged
teknium1 merged 1 commit into
mainfrom
fix/resume-list-full-titles
May 24, 2026
Merged

fix(cli): show full session titles in /resume list (#14082)#31710
teknium1 merged 1 commit into
mainfrom
fix/resume-list-full-titles

Conversation

@teknium1

@teknium1 teknium1 commented May 24, 2026

Copy link
Copy Markdown
Contributor

Summary

Long session titles render in full in the /resume list — no more Salvage BytePlus Volcengine PR With Fix… truncation at 30 chars. Fixes #14082.

The truncation existed in cli.py:6178 since the table layout was introduced; the fixed-width Python format spec ({title:<32}) already pads short titles to 32 chars without breaking alignment for longer ones (it left-aligns within the field and overflows visibly past it, which is the desired behavior for an interactive lookup table).

Changes

  • cli.py — drop the [:30] slice on title in _show_recent_sessions. Preview slicing at 38 chars stays.
  • tests/cli/test_cli_init.py — regression test confirming a 41-char title round-trips through the output.

Validation

Before After
41-char title in /resume list Salvage BytePlus Volcengine PR… Salvage BytePlus Volcengine PR With Fixes
Short titles (unchanged, still padded to col) (unchanged)
Preview column truncation 38 chars 38 chars
Alignment for ID column preserved preserved (format-spec pads short titles, long ones overflow visibly)

Targeted tests: tests/cli/test_cli_init.py — 42/42 passing.

Salvage notes

Infographic

resume-cluster-may2026

This infographic covers the full /resume command cluster salvage from May 2026 — PRs #31695, #31709, and #31710. Each PR's body links to the same image because they shipped together as a coherent UX upgrade.

@github-actions

github-actions Bot commented May 24, 2026

Copy link
Copy Markdown
Contributor

🔎 Lint report: fix/resume-list-full-titles vs origin/main

ruff

Total: 0 on HEAD, 0 on base (➖ 0)

🆕 New issues: none

✅ Fixed issues: none

Unchanged: 0 pre-existing issues carried over.

ty (type checker)

Total: 9102 on HEAD, 9102 on base (➖ 0)

🆕 New issues: none

✅ Fixed issues: none

Unchanged: 4849 pre-existing issues carried over.

Diagnostics are surfaced as warnings — this check never fails the build.

@alt-glitch alt-glitch added type/bug Something isn't working comp/cli CLI entry point, hermes_cli/, setup wizard P2 Medium — degraded but workaround exists labels May 24, 2026
@alt-glitch

Copy link
Copy Markdown
Collaborator

Salvage of #14098 — fixes #14082 (title truncation in /resume list). Supersedes #14098.

@teknium1 teknium1 force-pushed the fix/resume-list-full-titles branch from 05fa2e6 to 4f4cba2 Compare May 24, 2026 23:13
@teknium1 teknium1 merged commit 6c44d53 into main May 24, 2026
4 checks passed
@teknium1 teknium1 deleted the fix/resume-list-full-titles branch May 24, 2026 23:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/cli CLI entry point, hermes_cli/, setup wizard P2 Medium — degraded but workaround exists type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

/resume fails on title match because display truncates titles to 30 chars

3 participants