(MOT-4290) fix: harden worker presence and diagnostics - #661
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
skill-check — worker0 verified, 51 skipped (no docs/).
Four for four. Nicely done. |
4441f72 to
ada0747
Compare
f97ae14 to
36ac6b9
Compare
36ac6b9 to
6ae656b
Compare
Summary
harness::turn-started,harness::message-queued, and terminalharness::turn-completedtriggers; the scripted gate uses an early-release latch, with no status/gate polling or synchronization sleeps.Context
The recording exposed several misleading states: a running manager process could be shown as a healthy worker without a registered connection, stopped workers could collapse into an empty state without diagnostics, the chat composer could appear unusable during a running turn, and a stalled trace had no actionable feedback.
The backend stop and queued-message scenarios (INT-011/INT-012) are now on
main; this branch builds on them with UI-003 through the production Console surface. The worker-manager port collision remains an external runtime/environment issue, so the E2E launcher now fails before boot on an occupied port and reports early child-process exits with log tails.Lifecycle hooks were intentionally not used as test observers: they are synchronous execution boundaries that can veto or mutate a turn. The async lifecycle trigger types expose the exact facts this scenario needs without inserting test behavior into the production turn path.
Validation
git diff --checkpassed.Fixes MOT-4290