Skip to content

feat: add versioned review configuration - #50

Merged
EmersonBraun merged 3 commits into
mainfrom
codex/issue-39-review-config
Aug 26, 2026
Merged

feat: add versioned review configuration#50
EmersonBraun merged 3 commits into
mainfrom
codex/issue-39-review-config

Conversation

@EmersonBraun

Copy link
Copy Markdown
Member

Summary

  • Adds strict .agentskit-review.json with configVersion: 1.
  • Resolves built-in lens policy, thresholds, budgets, retries, conventions, context policy, and execution metadata with flags taking precedence.
  • Keeps provider/model/transport/trust/redaction/permissions out of project-controlled CI configuration; secrets are rejected and diagnostics are value-free.
  • Adds explicit local-only incomplete-profile handling: output is COMMENT and exit 2, never approval.
  • Documents the contract and adds unit plus CLI end-to-end coverage.

Validation

  • npm run check
  • npm pack --dry-run
  • 38/38 tests passing

Scope boundary

Runtime enforcement of byte/call/retry/context budgets is intentionally consumed by the follow-up issues #44 and #45; this PR establishes and validates their versioned policy input without silently adding a second execution path.

Closes #39

@EmersonBraun
EmersonBraun merged commit 0a5306c into main Aug 26, 2026
5 checks passed
@EmersonBraun
EmersonBraun deleted the codex/issue-39-review-config branch August 26, 2026 01:24
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: add versioned review configuration and lens policy

1 participant