Skip to content

Double daily code review agent run timeout#170

Merged
YunchuWang merged 2 commits intomainfrom
copilot/double-daily-code-review-timeout
Mar 13, 2026
Merged

Double daily code review agent run timeout#170
YunchuWang merged 2 commits intomainfrom
copilot/double-daily-code-review-timeout

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 13, 2026

The daily code review agent job and inner timeout command were hitting their limits, truncating agent runs prematurely.

Changes

  • timeout-minutes: 30 → 60 (job-level GitHub Actions cap)
  • Agent timeout command: 1200s (20 min) → 2400s (40 min)
  • Updated timeout warning message to reflect new 40-minute limit

Copilot AI and others added 2 commits March 13, 2026 03:48
Co-authored-by: YunchuWang <12449837+YunchuWang@users.noreply.github.com>
@YunchuWang YunchuWang marked this pull request as ready for review March 13, 2026 03:50
Copilot AI review requested due to automatic review settings March 13, 2026 03:50
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR increases the execution limits for the daily automated code review agent workflow to prevent premature termination by both the GitHub Actions job timeout and the inner timeout wrapper around the Copilot CLI invocation.

Changes:

  • Increased the workflow job timeout from 30 to 60 minutes.
  • Increased the Copilot CLI execution timeout from 20 to 40 minutes and updated the corresponding warning message.

You can also share your feedback on Copilot code review. Take the survey.

@YunchuWang YunchuWang merged commit cc97f44 into main Mar 13, 2026
20 checks passed
@YunchuWang YunchuWang deleted the copilot/double-daily-code-review-timeout branch March 13, 2026 17:09
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.

4 participants