Skip to content

Fix regression-runner scan timeout ownership - #186

Merged
DavidBakerEffendi merged 1 commit into
mainfrom
dave/latest-regression-test-failed-pretty
Sep 1, 2026
Merged

Fix regression-runner scan timeout ownership#186
DavidBakerEffendi merged 1 commit into
mainfrom
dave/latest-regression-test-failed-pretty

Conversation

@DavidBakerEffendi

Copy link
Copy Markdown
Collaborator

Keep Bifrost scan deadline policy in UsageBench without restoring the removed max_duration_secs tool argument. The runner now scopes the configured budget to the scan call, cancels timed-out MCP work, and leaves subsequent requests usable.

Key Changes:

  • Replace the session-wide timeout mutation with a per-call deadline covering readiness retries and backoff.
  • Send standard MCP notifications/cancelled on transport timeout and preserve explicit runner-error reporting.
  • Add regressions for omitted server arguments, timeout error mapping, cancellation payloads, stale responses, next-call recovery, zero duration, and logical deadline bounds.

Touch Points:

  • src/runners/mcp.rs
  • src/runners/bifrost.rs

@DavidBakerEffendi
DavidBakerEffendi merged commit 66eab05 into main Sep 1, 2026
5 checks passed
@DavidBakerEffendi
DavidBakerEffendi deleted the dave/latest-regression-test-failed-pretty branch September 1, 2026 13:33
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