Repository navigation
Point stale queue and live-output tests at their owners - #1378
Conversation
…heir API LiveQueueScope.session_id is AcpSessionId; the helper passed a str. TurnOwner.response_stream and ResponseStream(turn_id=) no longer exist since 94eda01 grouped text by its append claim; the tests now take streams from CommsUpdateConsumer and vary source_claim. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01XbFMSpSd8zQxyozgEV2Re7
There was a problem hiding this comment.
Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota. Try requesting a review at a lower effort level.
An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing.
|
Original55cf retained in owned integration2dc2102 and combined branch integ/queue-refusal-and-salvage at7bab354b with #1389. Main nominal imports plus the sibling AcpSessionId fixture corrected; production unchanged by1378 salvage. Commander runs the three affected paths with paired refusal controls. No author branch rewrite or old test-count acceptance. |
Red on main: 7 failed, 3 passed in test_queue_request_results + test_live_output_streams. Now: 10 passed, installed runtime core-dabeef407.
strtoLiveQueueScope.session_id: AcpSessionId. Test defect.TurnOwner.response_streamandResponseStream(turn_id=)are gone (94eda01). Same behaviour now asserted throughCommsUpdateConsumer.response_stream, distinctsource_claims andCompleteResponseStream. No source change, no test deleted.toad#1368 touches neither file.
🤖 Generated with Claude Code
https://claude.ai/code/session_01XbFMSpSd8zQxyozgEV2Re7