umputun / agterm / 29160675921

98%

push

github

fix: let agtermctl font target a split or scratch pane (#188)

`agtermctl font inc|dec|reset` only accepted `--target`/`--window` and always resized the session's main/left surface, so a split pane's font was unreachable from the CLI even though the two panes hold independent sizes (#186).

**Pane targeting**

Adds a `--pane left|right|scratch` option, mirroring the existing `session type`/`session text` pane addressing:

- omitted or `left` targets the main pane (unchanged behavior, via `addressableSurface`, which also reaches a promoted split survivor once the primary pane's shell exits)
- `right` targets the split pane (`session has no split pane` when there is none)
- `scratch` targets the session's scratch terminal, settable even while hidden (`session has no scratch terminal` when none is open)

Pane resolution and validation run server-side in the `ControlServer` font arm, so a raw socket client can't bypass them; the CLI validates via the shared `validatePaneArgument`. `performBindingAction` now returns a discardable `Bool`, so the font arm reports `session not realized` when a pane's surface isn't libghostty-realized yet instead of a silent false ok, matching `session.type`'s `inject()`.

**Read-back**

`agtermctl tree` now exposes each pane's live font size so a script can confirm a change: `fontSize` (the default/left pane, resolved via `addressableSurface`, so the main pane or the promoted split survivor), `splitFontSize` (the split pane), and `scratchFontSize` (the scratch terminal). Each is omitted when its pane isn't realized. Only the main pane's size persists across relaunch; the split/scratch surfaces' `onFontSizeChange` is deliberately unwired, matching a GUI cmd +/- on those panes, so read the live values back from the tree rather than the snapshot.

**Tests + docs**

Threaded through `ControlDispatcher` and the `agtermctl` CLI. Adds dispatcher and CLI unit tests, `AppStore`/protocol round-trip tests for the three tree fields (incl... (continued)

23 of 23 new or added lines in 4 files covered. (100.0%)

4431 of 4495 relevant lines covered (98.58%)

6688644.35 hits per line