Skip to content

fix(contextful-gate): give each part a 2700 s exec deadline - #198

Merged
debuggingfuture merged 2 commits into
mainfrom
fix/contextful-gate-part-cap
Oct 11, 2026
Merged

debuggingfuture merged 2 commits into
mainfrom
fix/contextful-gate-part-cap

Conversation

@debuggingfuture

@debuggingfuture debuggingfuture commented Oct 11, 2026 •

Copy link
Copy Markdown
Member

Each contextful gate part now gets a 2700 s exec deadline instead of 1800 s.

Problem & Insight

  • features.binary-*, workspace.cli, evaluate and budget.full each need 1250–1750 s on a good run: a cold build of 550–760 s (a release build of 1145–1518 s for budget.full) plus the suite. The ones above 1800 s end in ExecTimeout and report no verdict (contextful#465).
  • Splitting those parts into more parts makes the whole gate slower. Parts run two at a time (STAGE_CONCURRENCY = 2), and each extra part repeats its own cold build.

Take

  • STAGE_TIMEOUT_SEC = 2700 in runs/contextful-gate.ts. The slowest passing part so far ran ~1750 s, about 65% of the new deadline.
  • check.limits.maxDurationSec goes from 3600 to 4500, keeping the same 1800 s of checkout, upload and retry headroom above a full exec.
  • The batch wait is derived as queue age + the check run ceiling (now 165 min), replacing the hard-coded 130 min, so a part still inside its own budget is never reported failed.
  • A test pins the 2700 s deadline each part receives, under the check run's ceiling.

Discovery execs keep 1800 s; they run in minutes.

A contextful gate part that builds the binary cold and runs its CLI suite
needs up to ~1750 s, so the 1800 s deadline ends most of them in
ExecTimeout. Parts now get 2700 s, the check run's ceiling covers it, and
each batch waits out queue age plus the deadline.

Co-Authored-By: Claude <noreply@anthropic.com>

@flaredispatch-fractalboxdev flaredispatch-fractalboxdev Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ pr-review could not complete: { "step": "review", "cause": Error: ModelCallFailed: openai returned 429: {"success":false,"result":[],"messages":[],"error":[{"code":2003,"message":"Rate limited"}],"name":"AiGatewayError","httpCode":429,"internalCode":2003,"message":"Rate limited","description":"Rate limited"}, "summaryMd": undefined }

📋 View full logs & reviewed diff ↗

@debuggingfuture
debuggingfuture merged commit 3eec27f into main Oct 11, 2026
5 checks passed
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