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
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