Uh oh!
There was an error while loading. Please reload this page.
Hide exited sessions by default, and keep them out of the switcher - #75
Merged
Conversation
The list folds them behind an "N exited sessions · Show" line, and the Cmd+K switcher drops them from its resting rows, pinned badges, and the prev/next cycle. A search brings a match back, marked exited and ranked after everything running. Ghost rows for offline machines stay: an offline machine comes back, an exited shell does not. Closes#70 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Uh oh!
There was an error while loading. Please reload this page.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes#70.
An exited session is history, not somewhere to go. Two changes, plus one the issue implies.
The sessions list folds them away
DEFAULT_VIEW.showExitedflips tofalse. The fold is not a silent filter: a line under the list reads "N exited sessions · Show", counted after the search so the sentence matches the list it sits under (hiddenExitedinsessions/view.ts). Show sets the sameshowExitedthe display options checkbox owns, so the reveal is a view edit like any other — the dirty flag lights, the choice persists, saved views keep working.When everything on the fleet has ended, the screen no longer claims "No sessions yet"; the fold line stands in for the rows it hides.
The switcher stops offering them
At rest the palette is a "take me there" control, and a dead session is not a there:
A search brings them back, per the issue's suggestion: typing the name of a session you know ended and getting nothing would be its own kind of wrong. Matches rank after everything running and carry the word exited where a ghost says "unreachable".
The blind chords agree
cycleOrder(⌃⇧[ ] and the ⌃⇧1–9 pinned chords outside the palette) now skips exited sessions too. Not doing so would have ⌃⇧2 mean one session with the palette open and another with it shut, since the palette's badges no longer count the dead. Pressed from inside a session that has just ended, the cycle still leaves —stepCyclealready starts from the top for a key it does not hold.Deliberately untouched
ExitedRetentionand reaping are as they were; this is a client-side view decision.Tests
pnpm vitest run: 68 files, 1372 tests, all passing.pnpm run lintclean. New coverage: the fold line and its count, the all-ended empty state, resting-palette exclusion, badge renumbering, exited-visit skipping, search re-inclusion and ranking, cycle behavior.🤖 Generated with Claude Code