futpib / slopd / 23721329514

83%

master: 85%

Pull #25

github

Fix race condition: hook handler waits briefly for pane registration

When slopctl run creates a new tmux pane, mock_claude can fire a
SessionStart hook before the Run handler's managed_panes.insert()
executes. In tokio's multi-threaded scheduler there is a small window
between output().await resolving and the continuation running; a
concurrent hook handler task can check managed_panes.contains() in
that window and silently discard the hook.

Fix: add PaneRegistered Notify that is signalled immediately after
managed_panes.insert(). The hook handler, on seeing an unregistered
pane, waits up to PANE_REGISTRATION_WAIT (2 s) for a registration
notification before concluding the pane is genuinely external and
discarding the hook.

Agent-Logs-Url: https://github.com/futpib/slopd/sessions/6a8f20e3-1d0f-475f-9988-390d05f5bf51

Co-authored-by: futpib <4330357+futpib@users.noreply.github.com>

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

1927 of 2314 relevant lines covered (83.28%)

160.39 hits per line