Let a box's number leave a thread opened from that box

[dhh]

Pressing 1 inside a thread opened from the Imbox did nothing, and so did 2 inside a thread opened from The Feed. switchBox treated the box already under the thread as "already there" and answered with nothing. The model then passed the key on to the thread, which swallowed it.

• The change: a box's number always lands on that box's list. When that box is the one already under a thread, a search, a bundle or Previously Seen, the key closes whatever is open and shows the list that is already loaded, without re-reading it. Switching to a different box works as before.
• An in-flight read of that box's list is not cancelled, so pressing 1 while the Imbox is still loading doesn't throw that load away.
• Refactor: switching boxes and opening Previously Seen repeated the same "put away the thread, search, bundle and seen screen" steps. switchBox now shares them through closeOverlays.
• Unchanged: pressing the current box's number on its plain list is still a no-op (TestMailViewBoxShortcutNoOp).

New tests open a thread through the model and press 1 and 2, and check that 1 over a search in the Imbox returns to its list.

🤖 Generated with Claude Code

[Copilot]

Copilot review overview

🟡 Changes recommended

Bundle and in-flight request paths remain untested, and the stated cleanup refactor is incomplete.

Review effort: Balanced
Findings: 1 · 1

Open (2)

• Add coverage for active bundle closure and in-flight postings preservation · New
• Reuse closeOverlays in openPreviouslySeen to remove duplicate cleanup · New

What changed in this PR

Updates mail box shortcuts so selecting the current box closes an open thread or overlay and returns to its loaded list.

Changes:

• Adds shared overlay cleanup during box switching.
• Preserves active postings reads.
• Adds thread and search shortcut tests.

[!TIP]
If you aren't ready for review, convert to a draft PR.
Click "Convert to draft" or run gh pr ready --undo.
Click "Ready for review" or run gh pr ready to reengage.

File	Description
internal/​tui/​mail.go	Updates box switching and overlay cleanup.
internal/​tui/​tui_test.go	Tests thread and search shortcut behavior.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.