Skip to content

Make Copilot PR review CLI timeout configurable#8159

Open
WaelAbuSeada wants to merge 2 commits into
mainfrom
fix/copilot-cli-timeout-config
Open

Make Copilot PR review CLI timeout configurable#8159
WaelAbuSeada wants to merge 2 commits into
mainfrom
fix/copilot-cli-timeout-config

Conversation

@WaelAbuSeada
Copy link
Copy Markdown
Member

@WaelAbuSeada WaelAbuSeada commented May 13, 2026

Why

Copilot PR review can exceed the current hardcoded 20-minute CLI timeout on larger pull requests (for example, runs with many changed files).

What changed

  • Added script env var support: COPILOT_REVIEW_CLI_TIMEOUT_MINUTES (default 30)
  • Replaced hardcoded WaitForExit(1200000) with computed timeout based on that variable
  • Updated timeout exception message to include the configured value
  • Added workflow wiring in CopilotPRReviewRunner.yaml:
    • COPILOT_REVIEW_CLI_TIMEOUT_MINUTES: ${{ vars.COPILOT_REVIEW_CLI_TIMEOUT_MINUTES || '30' }}

Operational guidance

  • Default remains bounded at 30 minutes to avoid runaway processes
  • For larger PRs, set repository/environment variable COPILOT_REVIEW_CLI_TIMEOUT_MINUTES to 45 or 60 as needed

Validation

  • No diagnostics in updated script and workflow files

Fixes AB#635094

@WaelAbuSeada WaelAbuSeada requested review from a team as code owners May 13, 2026 19:52
@github-actions github-actions Bot added the Build: Automation Workflows and other setup in .github folder label May 13, 2026
@github-actions github-actions Bot added this to the Version 29.0 milestone May 13, 2026
@mazhelez
Copy link
Copy Markdown
Contributor

@copilot please pull main the PR branch

…ut-config

Co-authored-by: mazhelez <43066499+mazhelez@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

Copilot AI commented May 15, 2026

@copilot please pull main the PR branch

Pulled main into this PR branch and pushed the merge commit: f325d67.

Copilot AI requested a review from mazhelez May 15, 2026 10:09
@github-actions
Copy link
Copy Markdown
Contributor

⚠️ Stale Status Check Deleted

The Pull Request Build workflow run for this PR was older than 72 hours and has been deleted.

📋 Why was it deleted?

Status checks that are too old may no longer reflect the current state of the target branch. To ensure this PR is validated against the latest code and passes up-to-date checks, a fresh build is required.


🔄 How to trigger a new status check:

  1. 📤 Push a new commit to the PR branch, or
  2. 🔁 Close and reopen the PR

This will automatically trigger a new Pull Request Build workflow run.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Build: Automation Workflows and other setup in .github folder

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants