Repository navigation
chore: adopt the existing custom-code budget on next - #1248
jbeckwith-oai wants to merge 1 commit into
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
markstuart-oai
left a comment
There was a problem hiding this comment.
Reviewed 7660ced37598f6e4f0a522542943e8ea4d572136. No actionable source findings. This one-file PR adopts on next the exact schema 1 / 3,000-line policy blob already enforced by main. The complete diff contains only .castiron-ratchet.json; it does not raise the effective main budget or change counting, exclusions, or generation metadata.
Source-only review; candidate tests were not run. All nine observed non-skipped exact-head hosted checks passed, including CI / required; the Maven publishing check was skipped. The requested human policy approval and applicable checks are still required before merge. This automated source review does not substitute for that human approval.
|
Closing this budget-only proposal as obsolete after removal of customization budgets. It contains no next_policy baseline pin. Next-major baseline adoption and validation will proceed separately with the simplified compiler workflow. |
Summary
Adopt on
nextthe exact custom-code policy already enforced onmain: schema version 1 and a ceiling of 3,000 added-plus-deleted custom patch lines. The dormant next branch currently has no policy. This is a policy-only prerequisite for bringing the current SDK baseline into next-major development.The file is byte-for-byte identical to main at
199cf8169a49dc72bcf7fd869817917383e5ca77. No higher ceiling, counting changes, exclusions, generation metadata, runtime changes, or CI changes are included. Main's trusted checker measured the proposed reporting candidate at 2,516/3,000; next candidates must receive their own fresh checks.Validation and approval
JSON parses, exact comparison against main passes, and the commit changes only
.castiron-ratchet.json. Human approving review is required before merge; agents must not self-approve or bypass the budget gate. This does not authorize package publication.Unblocks #1247 after this policy is merged normally.