-
Notifications
You must be signed in to change notification settings - Fork 2
reviewability: restore the <5000 installed-package budget on the integrated v1 tree #61
Copy link
Copy link
Closed as not planned
Closed as not planned
Copy link
Labels
area: cliCommand-line interface behavior.Command-line interface behavior.area: correctionCorrection engine and correction UX.Correction engine and correction UX.area: packaging/releasePackaging, artifacts, compatibility, and release qualification.Packaging, artifacts, compatibility, and release qualification.area: wallet/coreWallet integration and Bitcoin Core boundaries.Wallet integration and Bitcoin Core boundaries.bugSomething isn't workingSomething isn't workinggate: adversarial reviewResolve, merge, or explicitly defer before the next full adversarial review.Resolve, merge, or explicitly defer before the next full adversarial review.
Description
Activity
Metadata
Metadata
Assignees
Labels
area: cliCommand-line interface behavior.Command-line interface behavior.area: correctionCorrection engine and correction UX.Correction engine and correction UX.area: packaging/releasePackaging, artifacts, compatibility, and release qualification.Packaging, artifacts, compatibility, and release qualification.area: wallet/coreWallet integration and Bitcoin Core boundaries.Wallet integration and Bitcoin Core boundaries.bugSomething isn't workingSomething isn't workinggate: adversarial reviewResolve, merge, or explicitly defer before the next full adversarial review.Resolve, merge, or explicitly defer before the next full adversarial review.
A disposable integration from
reviewability-v126f933eexposed a release-gate failure that individual PR CI cannot see.Current logical review-line counts (nonblank, non-comment
src/codex32/**/*.py, matchingtest_production_size_budgets_are_enforced):26f933e: 4,893docs/developer/api.mdrequires the installed v1 package to remain below 5,000 lines and says changing that budget requires explicit review and authorization. Do not raise the threshold as part of this issue.Acceptance:
<5000without weakening correct: Interpret mixed-case damage #42 mixed-case correction or wallet: Require the recorded fingerprint before import #57 verify-before-mutate / wallet-record behavior;The disposable integration worktree only produced local synthetic commits; no project branch was changed.