Skip to content

test(cli): real-binary E2E for the child handoff and verify gate (#383, 3/3) - #398

Merged
ajianaz merged 1 commit into
developfrom
test/child-handoff-e2e
Oct 16, 2026
Merged

ajianaz merged 1 commit into
developfrom
test/child-handoff-e2e

Conversation

@ajianaz

@ajianaz ajianaz commented Oct 16, 2026

Copy link
Copy Markdown
Contributor

What

Part 3 of 3 of the child-agent handoff: the real-binary integration test from the issue's testing section.

Why

#383. The wiring merged in #397 is covered by unit tests; this proves the whole path with real child processes.

Changes

  • tests/child_handoff_e2e.rs (test only): the test plays the parent model with AgentStartTool / AgentPollTool and starts real tole children through a wrapper that pins a scripted mock provider and a no-op uteke (hermetic: the real memory store is never touched).
    1. an implement child submits a handoff whose verification failed: agent_poll reports verify_failed (not closed) with a fix prompt naming the command and the artifact;
    2. a fix child started from that prompt reads the artifact by reference with its own file tool (the mock only lets it report saw artifact if it received the file body), submits passing verification: closed, no fix prompt;
    3. an implement child that closes without any handoff is unverified.

Testing

Refs #383

@github-actions

Copy link
Copy Markdown

🔍 Cora AI Code Review

✅ No issues found. Code looks good!


Review powered by cora-code · BYOK · MIT

@ajianaz
ajianaz merged commit 778704c into develop Oct 16, 2026
17 checks passed
@ajianaz
ajianaz deleted the test/child-handoff-e2e branch October 16, 2026 06:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant