ci(coverage): ratchet AC-17 merged coverage gate floor 80 -> 84 - #390
Merged
Conversation
Merged unit coverage on main reached 84.53% (measured 2026-07-12 on the main CI coverage job), so raise the COVERAGE_MIN default from 80 to 84 to lock in the gain with a small margin without failing current main. Continues the ratchet-upward-toward-90 policy (prior step 66 -> 80 in #124). Updates the hardcoded default in script/ci/ci.sh (_assert_coverage_gate plus the help + section ratchet comment blocks), the justfile.ci coverage / coverage-merge recipe comments, the boundary specs in test/unit/script/ci_spec.bats (80.00/79.99 -> 84.00/83.99), and the doc/TESTING.md gate note. COVERAGE_MIN and COVERAGE_ENFORCE overrides are unchanged. lint + test-unit green in Docker. Claude-Session: https://claude.ai/code/session_01NX5H2vuMTv4mBmjpPYoS3s Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
ycpss91255
enabled auto-merge (squash)
July 15, 2026 10:28
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Ratchet the AC-17 merged-coverage gate floor from 80 to 84.
Merged unit coverage on
mainreached 84.53% (measured 2026-07-12 onthe main CI coverage job), so a floor of 84 locks in the gains with a small
margin and does not fail current
main. This continues the maintainer'sratchet-upward-toward-90 policy (prior step 66 -> 80 in #124).
Changes
script/ci/ci.sh—_assert_coverage_gatedefault${COVERAGE_MIN:-80}->
:-84; help text + "Gate semantics" ratchet comment block updated torecord the new
80 -> 84step (84.53% measured 2026-07-12), keeping theexisting
66 -> 80(ci(coverage): flip AC-17 gate to 80% after boost (ratchet from 66) #124) history.justfile.ci—coverage/coverage-mergerecipe commentsdefault 80->
default 84.test/unit/script/ci_spec.bats— boundary specs and header updated to the84 default (pass at
84.00, fail at83.99); theCOVERAGE_MINoverrideand
COVERAGE_ENFORCEreport-only specs are unchanged.doc/TESTING.md— gate note default updated to 84 with the extendedratchet history (Chinese doc; number-only change).
doc/changelog/CHANGELOG.md— entry under[Unreleased] > Changed.COVERAGE_MINandCOVERAGE_ENFORCEoverrides are unchanged; only thehardcoded default moved.
Test plan
just -f justfile.ci lint— green (Docker).just -f justfile.ci test-unit— green (Docker); updatedci_spec.batsboundary specs pass against the new 84 default.
coverage-mergegate now enforces>= 84— expected to passsince
mainis at 84.53%.