Skip to content

fix: require text-capable local reviewer models - #5642

Merged
atomantic merged 1 commit into
mainfrom
codex/pr-reviewer-text-capability
Sep 1, 2026
Merged

fix: require text-capable local reviewer models#5642
atomantic merged 1 commit into
mainfrom
codex/pr-reviewer-text-capability

Conversation

@atomantic

Copy link
Copy Markdown
Owner

Summary

  • require the shared tool-free local selection policy to expose a text-capable model
  • reject embedding-only and unknown-capability models in both UI and server preflight validation
  • keep the existing no-tools and local-provider boundary fail-closed

Validation

  • client focused tests: 295 passed
  • server focused scheduler/security tests: 408 passed
  • client lint and production build passed
  • full server suite: 36,899 passed; unrelated image/sprite suites timed out under full-suite load

No contributor branches are checked out or executed by this change.

@atomantic
atomantic merged commit 5331693 into main Sep 1, 2026
7 checks passed
@atomantic
atomantic deleted the codex/pr-reviewer-text-capability branch September 1, 2026 21:01
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