Skip to content

Fix daily-rig-game-gen workflow timeout - #387

Merged
pelikhan merged 3 commits into
mainfrom
copilot/aw-daily-rig-game-generator-timeout-fix
Aug 9, 2026
Merged

Fix daily-rig-game-gen workflow timeout#387
pelikhan merged 3 commits into
mainfrom
copilot/aw-daily-rig-game-generator-timeout-fix

Conversation

Copilot AI commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

The Daily Rig Game Generator workflow hit its 30-minute limit and produced no safe output. Its debugging-reflection loop runs up to 3 attempts, each spawning nested rig CLI subprocesses (typecheck + full --server execution, itself a separate LLM session) with no bound on how long any single subprocess could run — leaving no margin for a job with several compounding LLM round-trips.

Changes

  • Timeout budget: raised timeout-minutes from 30 → 45, in line with other workflows that use similarly nested subprocess/agent loops (daily-rig-claude-compat at 45).
  • Loop bound: reduced MAX_ATTEMPTS from 3 → 2 to cap the worst-case loop duration.
  • Subprocess guard: added a 10-minute timeout in runRigEntry that SIGKILLs a hung child process and surfaces it as a failed attempt rather than letting it silently consume the entire job budget.
const timer = setTimeout(() => {
  timedOut = true;
  stderr += `\n[timed out after ${SUBPROCESS_TIMEOUT_MS / 60000} minutes]`;
  child.kill("SIGKILL");
}, SUBPROCESS_TIMEOUT_MS);

Recompiled with gh aw compile daily-rig-game-gen --strict and committed the updated .lock.yml.

Copilot AI linked an issue Aug 9, 2026 that may be closed by this pull request
Copilot AI and others added 2 commits August 9, 2026 15:23
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix timeout issue in daily rig game generator Fix daily-rig-game-gen workflow timeout Aug 9, 2026
Copilot AI requested a review from pelikhan August 9, 2026 15:27
@pelikhan
pelikhan marked this pull request as ready for review August 9, 2026 15:28
@pelikhan
pelikhan merged commit 0570ba3 into main Aug 9, 2026
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.

[aw] Daily Rig Game Generator timed out

2 participants