rayslava / revdiff / 33397094208
92%
master: 92%

Ran 31 Aug 2026 01:28PM UTC

Jobs 1

Files 79

Run time 1min

Badge

Committed 31 Aug 2026 01:27PM UTC coverage: 91.498%. Remained the same

Build # 33397094208

Build Type

push

github

Committed by

Commit Message

feat(launcher): herdr pane-scoped overlay (REVDIFF_HERDR_PANE=1)

under herdr a review replaces the whole tab view, so the agent pane that asked for it disappears for as long as the review lasts. herdr's working layout is an agent in one pane and you beside it, but the backend only ever calls `herdr tab create` + `pane run`, so the review arrives as a new fullscreen tab. that matters while annotating, because the agent's pane holds the context the diff has to be judged against. `REVDIFF_AGTERM_PANE=1` already exists for the same reason on agterm, which has the same layout model; herdr had no equivalent.

`REVDIFF_HERDR_PANE=1` splits the caller's own pane and zooms it, so the review still gets the full tab area but the agent pane is one keypress away instead of a tab switch away, and no tab is created. it copies the shape agterm settled on: env-var opt-in, capability probe before passing a new flag, quiet degradation to the existing overlay. unset, the `tab create` block is reached byte-identical to master.

pane mode is a self-contained block ahead of the tab path rather than a mode flag threaded through it. an earlier revision threaded a shared teardown through both, which re-indented ~50 untouched lines and quietly changed the default path (the EXIT trap began closing the tab on a signal, where master leaves it open). the self-contained shape costs a short duplicated exit tail, matching `launch-plan-review.sh`, and buys a default path that cannot regress.

**never guess which pane to close.** if `pane split` reports success but returns no parseable id, the launcher warns and exits 1 rather than diffing `pane list` to recover it. the pane population can change in between, so a recovered id may belong to another herdr client and closing it destroys someone else's work. the same rule rejects an id equal to the caller's own pane, which the positional grep fallback can produce on a jq-less box.

**liveness.** `{"error":{"code":"pane_not_found"}}` is aut... (continued)

Coverage Stats

11816 of 12914 relevant lines covered (91.5%)

3458.97 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	33397094208.1	31 Aug 2026 01:28PM UTC	79	91.5	GitHub Action Run

Source Files on build 33397094208