Skip to content

workflows: Increase Drafter timeout from 15 to 30 minutes - #342

Merged
cgwalters merged 1 commit into
mainfrom
agent/increase-drafter-timeout-462c3232e136d871
Sep 2, 2026
Merged

cgwalters merged 1 commit into
mainfrom
agent/increase-drafter-timeout-462c3232e136d871

Conversation

@bootc-bot

@bootc-bot bootc-bot Bot commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Caution

Protected files were modified in this change.
This pull request is in request_review mode and requires explicit human scrutiny before merge.

Protected files: .github/workflows/drafter.md

Summary

Fixes #332

The Drafter workflow was failing with "Engine Failure: The claude engine terminated unexpectedly" and MCP server connection errors. Analysis of the error logs and workflow configuration revealed that the 15-minute timeout was too short for operations on a Rust project.

Changes

  • Increased timeout-minutes in .github/workflows/drafter.md from 15 to 30 minutes

Rationale

Rust projects require significant time for:

  • Repository exploration and code analysis
  • Cargo compilation (which can be slow, especially for cold builds)
  • Running tests and validation
  • Network operations for crate downloads

The compiled workflow shows that while the top-level activation job has a 60-minute timeout and the safe_outputs job has 45 minutes, the agent step itself only had 15 minutes. This mismatch meant the agent would be terminated mid-execution when working on non-trivial changes.

Post-merge action required

⚠️ The workflow lockfile (.github/workflows/drafter.lock.yml) needs to be recompiled after this PR is merged:

gh aw compile --approve

Or trigger the scheduled upgrade workflow to handle it automatically.

Validation

  • Analyzed the error logs from the failed workflow run
  • Compared timeout settings across different jobs in the compiled workflows
  • Verified that 30 minutes is consistent with other long-running workflow jobs

Warning

Firewall blocked 2 domains

The following domains were blocked by the firewall during workflow execution:

  • api.anthropic.com
  • api.github.com

[!TIP]
api.github.com is blocked because GitHub API access uses the built-in GitHub tools by default. Instead of adding api.github.com to network.allowed, use tools.github.mode: gh-proxy for direct pre-authenticated GitHub CLI access without requiring network access to api.github.com:

tools:
  github:
    mode: gh-proxy

See GitHub Tools for more information on gh-proxy mode.

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "api.anthropic.com"
    - "api.github.com"

See Network Configuration for more information.

Generated by Drafter for #332 · claude · sonnet45 · 73.7 AIC · ⌖ 37.3 AIC · ⊞ 4.4K · ◷

The 15-minute timeout was too short for Rust project operations
(exploration, compilation, testing), causing the agent to terminate
unexpectedly with MCP server connection errors.

Fixes #332

Assisted-by: AI

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

@bootc-bot bootc-bot Bot left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Protected files were modified in this pull request and require manual scrutiny before merge.

Please verify that each protected-file change is intentional, policy-compliant, and safe:

  • Protected files: .github/workflows/drafter.md

@bootc-bot bootc-bot Bot mentioned this pull request Sep 2, 2026
@bootc-bot bootc-bot Bot added agent/review-working The review agent is actively working on this PR agent/lgtm Reviewer agent approved; ready to auto-merge labels Sep 2, 2026

@bootc-bot bootc-bot Bot left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Reviewed the timeout increase from 15 to 30 minutes for the Drafter workflow.

Verified:

  • Change correctly addresses issue #332 (workflow timeouts causing engine termination)
  • Root cause analysis is sound—AbortError logs confirm timeout, and Rust operations (cargo compilation, testing) require significant time
  • 30 minutes is appropriate—doubles the timeout while remaining well under activation (60min) and safe_outputs (45min) limits
  • Minimal, focused change with thorough documentation
  • Follows repository conventions

The lockfile recompilation requirement is clearly documented in the PR description for post-merge action. This change is ready to merge.

@bootc-bot bootc-bot Bot removed the agent/review-working The review agent is actively working on this PR label Sep 2, 2026
@cgwalters
cgwalters marked this pull request as ready for review September 2, 2026 19:46
@cgwalters

Copy link
Copy Markdown
Collaborator

Even 30 minutes is probably too short...

@cgwalters
cgwalters merged commit 47f7d24 into main Sep 2, 2026
43 checks passed
@gursewak1997
gursewak1997 deleted the agent/increase-drafter-timeout-462c3232e136d871 branch September 2, 2026 19:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agent/lgtm Reviewer agent approved; ready to auto-merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[aw] Drafter failed

1 participant