Skip to content

feat: enforce review preflight coverage and budgets - #56

Merged
EmersonBraun merged 2 commits into
mainfrom
codex/issue-45-coverage-preflight
Aug 26, 2026
Merged

feat: enforce review preflight coverage and budgets#56
EmersonBraun merged 2 commits into
mainfrom
codex/issue-45-coverage-preflight

Conversation

@EmersonBraun

Copy link
Copy Markdown
Member

Summary

Closes #45.

  • Adds provider-free --plan / --dry-run output, including JSON.
  • Refuses file, byte, snapshot, and provider-call overages before the first model call, with reductions.
  • Enforces bounded maxCalls (absolute ceiling 1000), CLI concurrency default 1, and one retry only for invalid structured output.
  • Marks required-lens failures and unreviewed coverage as INCOMPLETE with exit code 2; --no-fail remains findings-only.
  • Adds config, CLI smoke, retry, and preflight coverage tests plus docs/changelog updates.

Verification

  • npm run check
  • npm pack --dry-run
  • git diff --check
  • 50 tests passing locally.

Comment thread test/fixtures/bin/codex Fixed
@EmersonBraun
EmersonBraun merged commit 732cd27 into main Aug 26, 2026
5 checks passed
@EmersonBraun
EmersonBraun deleted the codex/issue-45-coverage-preflight branch August 26, 2026 01:59
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.

feat: enforce review coverage, budgets, and preflight

2 participants