Skip to content

fix(opencode): default large-repo model run timeout to ~1h - #682

Merged
seonghobae merged 1 commit into
mainfrom
fix/opencode-large-repo-1h-timeout
Jul 31, 2026
Merged

fix(opencode): default large-repo model run timeout to ~1h#682
seonghobae merged 1 commit into
mainfrom
fix/opencode-large-repo-1h-timeout

Conversation

@seonghobae

@seonghobae seonghobae commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Summary

OpenCode pool runner defaulted primary/dynamic timeouts to 600s when env was unset, which is unrealistic for large repositories even when the workflow sets 5400s in some paths.

  • Default OPENCODE_RUN_TIMEOUT_SECONDS and OPENCODE_DYNAMIC_RUN_TIMEOUT_CAP_SECONDS3600
  • Medium/unknown cadence + central fallback defaults → 3600
  • Free-tier failover remains 600s
  • Unit test + quick-gate asserts updated

Test plan

  • pytest tests/test_opencode_model_pool_runner.py::test_dynamic_review_cadence_caps_large_change_queue_budget

Summary by CodeRabbit

  • 개선 사항
    • 대규모 저장소에서 OpenCode 리뷰가 중단되지 않도록 기본 실행 제한시간을 최대 3,600초로 늘렸습니다.
    • 변경 규모에 따른 동적 실행 제한과 전체 작업 예산도 확대되어 긴 리뷰 작업을 더 안정적으로 처리합니다.
    • 환경변수로 지정한 제한시간은 계속 우선 적용됩니다.
    • 무료 티어의 장애 조치 제한시간은 기존과 동일하게 600초로 유지됩니다.
    • 다양한 실행 조건에서 새로운 제한시간 설정이 올바르게 적용되는지 검증을 강화했습니다.

Raise primary OPENCODE_RUN_TIMEOUT and dynamic timeout-cap defaults from
600s to 3600s so big-repo reviews are not killed by the short fallback.
Keep free-tier failover at 600s; workflow env (5400s) still overrides.
@coderabbitai

coderabbitai Bot commented Jul 31, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 53be6ec9-5f8e-4201-843e-8796be7fbd1f

📥 Commits

Reviewing files that changed from the base of the PR and between 59e0f49 and e189f2c.

📒 Files selected for processing (4)
  • docs/nvidia-nim-opencode-hotfix.md
  • scripts/ci/run_opencode_review_model_pool.sh
  • scripts/ci/test_strix_quick_gate.sh
  • tests/test_opencode_model_pool_runner.py

📝 Walkthrough

Walkthrough

OpenCode 모델 풀의 기본 실행 제한시간과 동적 큐 상한을 3600초로 상향했습니다. 동적 큐 총 예산은 7200초로 설정했습니다. 무료 티어 fallback은 600초로 유지하고 관련 테스트와 문서를 갱신했습니다.

Changes

OpenCode 제한시간 설정

Layer / File(s) Summary
기본 제한시간 및 동적 큐 상한
scripts/ci/run_opencode_review_model_pool.sh
동적 큐와 여러 실행 경로의 기본 per-attempt timeout을 3600초로 변경했습니다. 동적 큐 총 예산 상한을 7200초로 변경했습니다.
제한시간 검증 및 문서
docs/nvidia-nim-opencode-hotfix.md, scripts/ci/test_strix_quick_gate.sh, tests/test_opencode_model_pool_runner.py
기본 제한시간과 동적 상한을 검증하도록 테스트를 갱신했습니다. 무료 티어 fallback 제한시간 600초와 새 설정을 문서화했습니다.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

  • ContextualWisdomLab/.github#635: 동일한 OpenCode 모델 풀 스크립트와 테스트를 수정했지만, 제한시간이 아닌 privileged-dispatch 격리를 다룹니다.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/opencode-large-repo-1h-timeout

Comment @coderabbitai help to get the list of available commands.

@seonghobae
seonghobae merged commit c7a4bad into main Jul 31, 2026
24 of 37 checks passed
@seonghobae
seonghobae deleted the fix/opencode-large-repo-1h-timeout branch July 31, 2026 11:47
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.

1 participant