Skip to content

Make Copilot PR review CLI timeout configurable#8159

Open
WaelAbuSeada wants to merge 1 commit into
mainfrom
fix/copilot-cli-timeout-config
Open

Make Copilot PR review CLI timeout configurable#8159
WaelAbuSeada wants to merge 1 commit 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
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.

2 participants