Skip to content

docs: raise v1 review budget to 5,200 lines - #62

Merged
BenWestgate merged 1 commit into
reviewability-v1from
codex/61-size-budget-5200
Sep 28, 2026
Merged

BenWestgate merged 1 commit into
reviewability-v1from
codex/61-size-budget-5200

Conversation

@BenWestgate

Copy link
Copy Markdown
Owner

Maintainer authorization now allows the v1 installed-package review budget to increase from <5000 to <5200 rather than performing a riskier pre-release refactor solely to recover ~100 lines.

This focused policy/enforcement change updates only:

  • AGENTS.md
  • docs/developer/api.md
  • test_production_size_budgets_are_enforced

The current reviewability-v1 package measures 4,893 nonblank/noncomment Python lines with the same recursive counter. The previously simulated combined v1 stack measured ~5,102, leaving ~98 lines of headroom under the authorized cap.

Local verification:

  • git diff --check
  • exact production-line counter: 4,893 < 5,200

The full repository test suite is delegated to PR CI because the local system Python lacks the pinned test-only bip32/coincurve oracle dependencies.

This supersedes issue #61's earlier no-budget-increase condition following explicit maintainer authorization.

Closes #61

@BenWestgate BenWestgate added gate: adversarial review Resolve, merge, or explicitly defer before the next full adversarial review. area: packaging/release Packaging, artifacts, compatibility, and release qualification. labels Sep 28, 2026 — with ChatGPT Codex Connector
@BenWestgate
BenWestgate merged commit f19d462 into reviewability-v1 Sep 28, 2026
12 checks passed
@BenWestgate
BenWestgate deleted the codex/61-size-budget-5200 branch September 28, 2026 05:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: packaging/release Packaging, artifacts, compatibility, and release qualification. gate: adversarial review Resolve, merge, or explicitly defer before the next full adversarial review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant