Add inline steering queue support (#4)

* Add inline steering queue support

* Harden steering recovery after Stop

* Wait for session readiness after Stop

* Make Stop lifecycle tests event driven

* Isolate real RPC tests from shared environment

* Stop real RPC fixtures deterministically

Author: sansari

.2119/verdicts/REQ-003.4.1--fdc1dc54680b.json

@@ -0,0 +1,8 @@
+{
+  "reviewId": "REQ-003.4.1--fdc1dc54680b",
+  "requirementId": "REQ-003.4.1",
+  "hash": "fdc1dc54680b",
+  "verdict": "pass",
+  "summary": "testRunningChatDoesNotBlockSelectingAnotherChatOrNewChatSurface exercises the real select()/startNewChat() code path (no mocking of the gate) and asserts selectedSessionID actually changes while the first chat's isConversationRunning stays true, which would fail if a blocking guard (like the archive-flow's blockedNavigationAlert) were added to select().",
+  "timestamp": "2026-09-02T22:28:30.820Z"
+}

.2119/verdicts/REQ-003.4.2--bdf1ed3cc242.json

@@ -0,0 +1,8 @@
+{
+  "reviewId": "REQ-003.4.2--bdf1ed3cc242",
+  "requirementId": "REQ-003.4.2",
+  "hash": "bdf1ed3cc242",
+  "verdict": "pass",
+  "summary": "Test starts a running chat in project A, selects a chat in project B, and asserts the selection actually switches (selectedProjectID/selectedSessionID) while project A's conversation remains running — a real behavioral check since select(projectID:) has no running-state guard that could make this pass vacuously.",
+  "timestamp": "2026-09-02T22:28:30.392Z"
+}

.2119/verdicts/REQ-003.4.5--77bc109ed8ea.json

@@ -0,0 +1,8 @@
+{
+  "reviewId": "REQ-003.4.5--77bc109ed8ea",
+  "requirementId": "REQ-003.4.5",
+  "hash": "77bc109ed8ea",
+  "verdict": "pass",
+  "summary": "testRuntimeCallbacksRouteOutputToOwningChatAfterSelectionChanges routes a real event through AppModel/ConversationModel while a different chat is selected, asserts the delta appends only as the final item of the owning chat's cachedTranscript (order preserved via dropLast equality) and is absent from the other chat's transcript — genuine, non-mocked coverage of the append-only-to-owning-chat requirement.",
+  "timestamp": "2026-09-02T22:28:27.839Z"
+}

.2119/verdicts/REQ-003.4.7--ea2a4fa4eaf8.json

@@ -0,0 +1,8 @@
+{
+  "reviewId": "REQ-003.4.7--ea2a4fa4eaf8",
+  "requirementId": "REQ-003.4.7",
+  "hash": "ea2a4fa4eaf8",
+  "verdict": "pass",
+  "summary": "Test verifies selection actually switches to another chat (selectedSessionID/selectedProjectID change) while a queued-prompt chat and a pending-startup chat both remain in working state; covers both boundary terms (queued prompt, pending startup) with real AppModel state, not mocked-away.",
+  "timestamp": "2026-09-02T22:28:31.084Z"
+}

.2119/verdicts/REQ-003.4.8--e7eb0b484688.json

@@ -0,0 +1,8 @@
+{
+  "reviewId": "REQ-003.4.8--e7eb0b484688",
+  "requirementId": "REQ-003.4.8",
+  "hash": "e7eb0b484688",
+  "verdict": "pass",
+  "summary": "Test opens new-chat surface and creates+sends a new chat while another session is actively running (isConversationRunning true throughout), verifying via non-mocked AppModel state that neither startNewChat() nor sendNewChatPrompt() are blocked or altered by in-flight work; assertions would fail if a guard were added to gate these on running state.",
+  "timestamp": "2026-09-02T22:28:51.080Z"
+}

.2119/verdicts/REQ-003.5.2--014fb242ad97.json

@@ -0,0 +1,8 @@
+{
+  "reviewId": "REQ-003.5.2--014fb242ad97",
+  "requirementId": "REQ-003.5.2",
+  "hash": "014fb242ad97",
+  "verdict": "pass",
+  "summary": "Unit tests (mock delay + real fake-RPC subprocess) and UI test verify late text_delta/tool_execution_start events after Stop and after a later prompt starts do not append to transcript, while new-turn output still appends; negative-space (forbidden late output) and positive-space (allowed new output) both covered without over-mocking.",
+  "timestamp": "2026-09-08T22:20:16.554Z"
+}

.2119/verdicts/REQ-003.5.2--0419965f8453.json

@@ -0,0 +1,8 @@
+{
+  "reviewId": "REQ-003.5.2--0419965f8453",
+  "requirementId": "REQ-003.5.2",
+  "hash": "0419965f8453",
+  "verdict": "pass",
+  "summary": "Tests exercise real RPC race conditions (delayed late deltas/tool starts from stopped and superseded turns, incl. real subprocess) and assert absence of late output in transcript/UI; covers both stop-only and stop-then-new-turn boundary cases genuinely.",
+  "timestamp": "2026-09-02T22:23:05.454Z"
+}

.2119/verdicts/REQ-003.5.2--06c255642356.json

@@ -0,0 +1,8 @@
+{
+  "reviewId": "REQ-003.5.2--06c255642356",
+  "requirementId": "REQ-003.5.2",
+  "hash": "06c255642356",
+  "verdict": "pass",
+  "summary": "Genuine coverage: unit tests use real timing races and a real fake-RPC subprocess to confirm late/stopped-turn output never appends while distinct later-turn output does append; UI test confirms no assistant transcript node after Stop before new send.",
+  "timestamp": "2026-09-08T21:25:03.765Z"
+}

.2119/verdicts/REQ-003.5.2--09c4e4ebdbbc.json

@@ -0,0 +1,8 @@
+{
+  "reviewId": "REQ-003.5.2--09c4e4ebdbbc",
+  "requirementId": "REQ-003.5.2",
+  "hash": "09c4e4ebdbbc",
+  "verdict": "pass",
+  "summary": "Genuine coverage: mocked and real-RPC-process tests confirm late/delayed turn output after Stop (and after a subsequent new turn starts) never appends to transcript, both at model and UI level.",
+  "timestamp": "2026-09-08T20:42:56.451Z"
+}

.2119/verdicts/REQ-003.5.2--177331f1f38e.json

@@ -0,0 +1,8 @@
+{
+  "reviewId": "REQ-003.5.2--177331f1f38e",
+  "requirementId": "REQ-003.5.2",
+  "hash": "177331f1f38e",
+  "verdict": "pass",
+  "summary": "Tests genuinely assert stale output from a stopped turn is excluded from transcript items after stop, both immediately and after a later turn starts, using mocked and real (non-mocked) RPC subprocess paths; negative assertions would fail under the violation.",
+  "timestamp": "2026-09-08T22:04:05.193Z"
+}