Toggle any pane wherever you put it, and keep the header hidden

[OutThisLife]

Dragging the terminal to the bottom stacks logs into its zone, and every tool-panel toggle broke in that stack — ⌃`, the statusbar button, ⌘J's terminal fallback. A lone terminal never reproduced it, which is why it read as randomly broken: it worked again only after closing the terminal from its own tab, because that broke the stack up.

Auditing the rest of the toggles found the same divergence on ⌘G and ⌘J, so this fixes the class rather than the one key.

The toggles asked a boolean, not the tree

!$terminalTakeover.get() flips a value with no idea what's on screen. Once anything else moved the pane — stacked behind a sibling tab, minimized from the zone menu, closed with ⌘W — the store said "open" while nothing was visible, and the press spent itself re-asserting a value it already held.

A probe confirmed it was never terminal-specific. With review stacked behind files in the right column, or either pane inside a minimized zone, ⌘G and ⌘J were dead in exactly the same way. isPaneVisible / togglePaneVisible now back every toggle. Close still routes through closeTreePane, so each pane keeps its own semantics: a tool panel collapses to its rail, files and review close through their store, anything else is dismissed.

setTreePaneHidden keeps its quiet path — a reactive unhide (a cwd arriving) must not front or un-minimize over what the user is looking at. Only user intent takes the reveal path.

files and review had a closer and no opener

So their booleans went stale the moment anything but the toggle revealed them. bindPaneVisibility moves into the tree store beside bindToolPaneCollapse, documents the two as a pair, and both panes now pass both halves.

Boot revealed instead of leaving the tree alone

bindPaneCollapse ran setPaneCollapsed(id, !open), and false there fronts the pane. Logs binds last, so it stole the active tab from the persisted tree. ⌃` then asked to collapse a terminal that was no longer active — the shared-zone branch declines that by design — so the first press did nothing. Boot now only ever collapses; the persisted tree already records which tab was active.

"Hide header" had two independent clobbers

normalize() deleted headerHidden whenever a zone dropped to one pane — a lone zone is headerless by default, so the flag looked redundant. It isn't: it's the user's standing preference for that zone, and dropping it brought the bar back the moment a pane rejoined. Re-adoption then pinned headerHidden: false unconditionally, which is right for a pane arriving somewhere new and wrong for a zone whose bar was deliberately hidden. Adoption now carries the destination zone's own setting, read before the insert since insertAtGroup pins the flag itself on a center drop.

Min-height

The terminal and logs carried a 7.5rem floor, so dragging the seam down jammed with a sliver of unusable terminal showing. Tool-panel zones now floor at the h-7 header strip, and releasing there minimizes the zone — it folds to its collapsed header, vertical rail or horizontal strip depending on the parent axis. The sliver size is never written, so restoring returns the previous height.

Verification

Eleven regression tests across the two toggle classes, all driven through the real bindings. Reverting the fixes turns them red — including the ⌃`-is-a-dead-key case and the missing opener.

That last one is worth calling out: the tests originally kept a local copy of each binding, and passed with the fix reverted. Both bindings moved into the store so the tests exercise production code, which is what caught the missing files/review opener at all.

1164 tests pass across pane-shell, app/chat, app/shell, store and app/contrib; eslint clean on the changed files.

[github-actions]

૮ >ﻌ< ა ci review

_{ran on feaa325}

ℹ️ Info

Desktop E2E visual evidence · View test artifacts · View job

3 visual diffs.

_{inline evidence is publishing...}