Skip to content

fix(oar): make auto-review bounded and timeout-safe - #67

Merged
johnnygreco merged 3 commits into
mainfrom
johnny/oar-review-resilience
Sep 11, 2026
Merged

johnnygreco merged 3 commits into
mainfrom
johnny/oar-review-resilience

Conversation

@johnnygreco

Copy link
Copy Markdown
Collaborator

Summary

  • bound new-project reviews around complete documentation and representative implementation evidence
  • raise agent, inference, and job timeouts while classifying OAR timeouts separately
  • report timeout-only reviews as incomplete and non-blocking without ever calling them a pass
  • replace the full patch and repository upload with compact change context
  • redact configured model IDs from OAR command diagnostics

Validation

  • make check (155 tests)
  • make build
  • repository OAR review tests (65 tests, 133 subtests)
  • documentation tests (23 passed, 3 skipped)
  • scripts/build-docs.sh
  • actionlint v1.7.12
  • shell, JSON, and diff validation

@github-actions

github-actions Bot commented Sep 11, 2026

Copy link
Copy Markdown

Documentation preview

The preview has been removed.

@johnnygreco
johnnygreco merged commit 3e0318e into main Sep 11, 2026
19 checks passed
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