Billy pilot: test runtime enforcement of per-node timeout (on top of #72)#82
Draft
devthejo wants to merge 1 commit into
Draft
Billy pilot: test runtime enforcement of per-node timeout (on top of #72)#82devthejo wants to merge 1 commit into
devthejo wants to merge 1 commit into
Conversation
Parser + C199 validation were tested, but the executeBackend path that
actually derives the bounded context (the feature's behavioral heart)
had no coverage. Add tests proving a short timeout fires with
context.DeadlineExceeded, a generous timeout leaves the happy path
untouched, and a ${VAR:-default} form is expanded and enforced.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.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
1 participant
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.
Pilot run of Billy (
branch-improve-loop) against PR #72 (per-nodetimeout:on agent/judge nodes), part of the e2e cloud dogfood session.Billy reviewed the diff
main...feat/agent-node-timeout, found that the parser + C199 validation were covered but the runtime enforcement path (executeBackendderiving the bounded context — the feature's behavioral heart) was not, and added a test for it. The deterministic build/test gate passed (green) before converging.This draft targets the PR branch so the diff is only Billy's addition (
pkg/backend/model/executor_timeout_test.go, +108). Merge into #72 if you want the coverage, or close.Run locally (sandboxed, devbox toolchain);
finalize_mrcouldn't self-push (no forge_token in the sandbox), so the commit was pushed by hand from the preserved storage branch.🤖 Generated with Claude Code