Skip to content

test(eve): hold busy-worker until steering begins - #3139

Closed
ruiconti wants to merge 1 commit into
mainfrom
ruiconti/busy-worker-gate
Closed

test(eve): hold busy-worker until steering begins#3139
ruiconti wants to merge 1 commit into
mainfrom
ruiconti/busy-worker-gate

Conversation

@ruiconti

@ruiconti ruiconti commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Summary

The busy-worker eval can miss the state it intends to test: in the failed run, the child completed before the later continuation arrived. Hold the child at an approval gate and wait until that gate is visible before steering it. The eval then checks that steering cancels the held task and completes a replacement on the same agent, without relying on the five-second delay.

Extracted from #3073; this fixture fix is independent of workflow authorization.

Validation

  • pnpm typecheck: 44 tasks passed, including building and typechecking fixture-tasks.
  • pnpm --filter fixture-tasks validate:transitions: 19 declarations passed.
  • pnpm fmt and pnpm lint passed.
  • E2E execution is CI-only; the extracted branch needs its own CI run.

Checklist

  • This change was requested or approved by a maintainer
  • I ran the relevant checks from CONTRIBUTING.md
  • I added tests and documentation where relevant
  • I added a changeset if this touches the published eve package
  • DCO sign-off passes for every commit (git commit --signoff)

Signed-off-by: Rui Conti <ruiconti@gmail.com>
@vercel

vercel Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
eve-docs Ready Ready Preview, v0 Sep 8, 2026 12:59pm UTC
1 Skipped Deployment
Project Deployment Actions Updated
eve-pkg Skipped Skipped v0 Sep 8, 2026 12:59pm UTC

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