umputun / agterm / 33419214656

97%

push

github

Add sidebar.width control command for the sidebar divider position (#512)

sidebar width is persisted per window and adjustable by dragging, but it had no control command or read-back. Dragging was the only user-facing way to change it.

that matters beyond tidiness. Terminal width is the window width minus the sidebar width minus a 1pt divider, and the window has a hard 640pt minimum (`agtermApp.swift:127`) that `window resize` silently clamps to. With the sidebar visible, a controller could not reach a terminal width below the floor set by that minimum and the current sidebar width.

```
agtermctl sidebar width <points> [--window W]
```

clamps to the existing 160...560pt drag bounds and echoes the stored width. Without the echo a clamped request and an honored one both answer bare ok. The echo is the stored value, not a measured on-screen width, and the CLI preserves it without fixed-decimal rounding: rounding an honored 271.34 to 271.3 would report a clamp that never happened. Points are `Double` because the drag writes a fractional cursor x, so an `Int` could not express every width the GUI can reach. The comparison callers are told to make is numeric, since the echo is a double and `300` comes back `300.0`.

read-back is the tree's top-level `sidebarWidth`, deliberately not `window list`. No caller needs width discovery across windows, which is all a `window list` copy would add. Takes `--window` like `sidebar expand` rather than being frontmost-only like `sidebar` and `sidebar mode`.

`ControlActions.setSidebarWidth` has a default unsupported response, so existing conformers need no new method. A downstream can still need source changes for exhaustive `Command` switches, as with any added command.

**two supporting moves, both forced by file length limits**

`AppStore+Sidebar.swift` takes the new setter plus `clampSidebarWidth`, which replaces a clamp expression that `restore()` and the drag handler each spelled separately. The split divider al... (continued)

136 of 139 new or added lines in 8 files covered. (97.84%)

9102 of 9328 relevant lines covered (97.58%)

4374494.73 hits per line