umputun / agterm / 33095977333

97%

push

github

refactor: let Session own the captured-command reset (#490)

`restore.clear` and the non-last-window close both spelled out the same
four-field reset inline, over different session sets. That scattering is
what #452 fixed in `closeSplit`, where `splitForegroundCommand` was left
out of an otherwise complete clear, so the next capture slot added would
have to find two hand-written copies.

`Session.clearCapturedForegroundCommands()` drops the persisted pair and
the unconsumed pending pair together, beside `clearPendingForegroundCommands`
it calls. Both call sites keep their own scope, `library.allOpenSessions()`
for the app-global command and the closing window's store for the close,
and no behaviour changes.

Clearing the persisted fields alone would leave a launch-armed replay
running, so the test presets both pairs plus a session.restore pin and
asserts the pins survive. With the pending call removed from the new
method it fails on the split slot.

Closes the capture-slot-reset backlog item.

5 of 5 new or added lines in 1 file covered. (100.0%)

8185 of 8357 relevant lines covered (97.94%)

4882750.99 hits per line