umputun / agterm / 38165281896

94%

push

github

Show a session's scratch without switching to it (#730)

* feat(control): show a session's scratch without switching to it

`session scratch on --target A` selected A even when another session was on screen. An agent opening a long-running program in its own session's scratch pulled the window away from what the user was typing, cleared A's unseen badge and reordered Ctrl-Tab recency.

showing a scratch no longer selects its target. `--follow` selects it, the same contract `session overlay open` has. The select was there to stop a hidden scratch from taking the keyboard; the deck's `deckActive` gate already does that for an unselected session, and the scratch gets the keyboard once its session is selected.

behavior change: a script that relied on `scratch on` to switch sessions now passes `--follow`.

`ControlActions` keeps the four-argument `scratchSession` requirement and defaults the new `follow:` overload to it, so the agterm-linux conformer builds unchanged.

Fixed #728

* fix(scratch): run the command when replacing a shown scratch

`session scratch on --command CMD` against a scratch that was already shown answered ok and never ran CMD. The store closes the old scratch and shows it again inside one call, so SwiftUI saw no change in `scratchActive` and kept the torn-down view mounted, where `createSurface` refuses to run.

the scratch view's identity now carries `Session.scratchGeneration`, bumped on every `closeScratch`, so a respawn mounts a fresh view. Same device as `overlaySlotGeneration` on the overlay panel.

Related to #728

* docs(control): correct two comments after the scratch follow change

`ControlArgs.follow` named only `session.overlay.open`. It now lists `session.scratch`, and `pick.open` and `ask.open`, which raise the owning window.

the `pollSessionActiveAndScratch` comment had ended up on the new `pollScratchRealized` helper and still described the removed select-before-show. Each helper has its own comment now.

Related to #728

6 of 9 new or added lines in 5 files covered. (66.67%)

15278 of 16301 relevant lines covered (93.72%)

2503438.7 hits per line