Repository navigation
test(collaboration): recover complete per-turn delegation context - #8165
Merged
Merged
Conversation
Signed-off-by: LoopX Agent <337587101+loopx-agent@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Goal And Delivered Outcome
Restore the existing acceptance test for unfinished delegations across consecutive source-checkout CLI Turns. On
24027a34736b1e50c2eb431a6039009028e73d8d, all eight original cases fail before checking the delegation facts: machine-envelope consumers assume inline JSON, and native-reader consumers assume a bareloopxcommand. The native owner can also legitimately provide a fresh full capture on the second Turn.The test now requests inline transport only for its machine child, executes the native producer's complete interpreter argv, and reads the current Turn's selected context through every page. The original semantic assertions remain; the test additionally carries a queued return and an unconsumed peer result alongside an already-read/adopted request whose linked Todo is done. All three remain visible on both Turns, and the collaboration receipt bytes remain unchanged.
main.Author Declaration
loopx-agent.24027a34736b1e50c2eb431a6039009028e73d8d.Python remains the existing test/transport consumer. There is no new decision owner, protocol vocabulary, frontend configuration or runtime language migration.
Scope And Continuation
One existing test file changes. A single subprocess adapter serves the explicit console and emitted native readers. Reader pages must advance monotonically and cover the complete UTF-8 byte range; the test no longer requires an incidental single-page result or unconditional warm delta. Parent context settings remain unchanged in both
onandoffcases.This repairs source acceptance coverage. Installed desktop/WebView, actual model writing, online Lark delivery, requester adoption and full-cost qualification remain with the existing delivery work. No merge, installation, release or paid execution is included.
Validation
e15a6566c49304b11e3967bbe4986a1c908f4276.uv run --extra test python -m pytest tests/test_delegation_turn_context.py tests/test_delegation_remaining_work.py tests/test_inbox_pagination.py -q: 39 passed. Disposable local File and SQLite authority stores; no online provider or model.Exact-scope quality receipt:
cqr_87133dea7f9ff5a10d5d, verified valid for fingerprint87133dea7f9ff5a10d5dba83d4a55b8eb35651e7d890f02734d16c1adef08645; safe-fix allowed/applied in one bounded pass, zero blockers/warnings/advisories. Intermediate configuration failures remain failed observations and are superseded only by the later passing run.Frontend / Visual Evidence
UI impact: none. The changed surface is the existing CLI acceptance test; no screenshot, packaged UI or installed-product improvement is claimed.
Type And Boundary