ci: improve backend coverage reliability and reduce redundant work - #1711
Merged
Merged
Conversation
7 of 11 tasks
📊 Code coverage
Gate: line coverage must stay at or above the threshold. Ratchet plan (W21): Backend → 55%, CLI → 50%, Frontend → 30% by quarter end. |
ctkm-aelf
marked this pull request as draft
September 30, 2026 13:01
ctkm-aelf
force-pushed
the
fix-backend-coverage-debug-info
branch
3 times, most recently
from
September 30, 2026 16:02
d62750b to
264f39d
Compare
ctkm-aelf
force-pushed
the
fix-backend-coverage-debug-info
branch
from
September 30, 2026 17:04
264f39d to
e1585ea
Compare
ctkm-aelf
marked this pull request as ready for review
September 30, 2026 17:33
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
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.
Result
Improve CI reliability while preserving the full test suite, feature checks and coverage gates. This PR targets main. Final validation of head
e1585ea9includes main14d76f4a(#1716), tested together as merge commit15cf1bf7. It changes CI workflows/helpers and fixes one observed race in the test harness; production behavior is unchanged.Changes
cfg(test).--no-report, retaining instrumented execution, exports and threshold enforcement. Cancel superseded PR CodeQL scans while preserving all four languages. Include Mobile failures/cancellations in the aggregate CI gate.Coverage contract
Keep the full backend and service-adapter suite, standalone billing smoke, production
gcp-kmsbuild and embedded-input guard, AWS/GCP/combined feature builds, CLI, frontend, mobile, SDK, Oracle, wizard and plugin checks. Gates remain 73% backend / 64% CLI / 15% frontend. No test exclusions, additional ignores/skips, retries, looser assertions or reduced feature matrices are introduced. Head coverage is always freshly measured.Evidence and validation
Final head
e1585ea9passes CI, CodeQL and Release. Both independent backend coverage jobs and all four CodeQL language scans passed. Release publishing/build steps are intentionally skipped for a PR.cfg(test)callback-completion hook innyxbot.rs. No measured source scope was removed. Covered lines are 332,009 base / 332,023 head; this small execution variation is not claimed as a CI-induced coverage improvement.--no-reportskips reporting after normal instrumented test execution.Earlier validation exposed issues that were corrected before this final run. A 45-minute coverage limit interrupted a progressing suite; a subsequent passing full backend job took 37m14s, so both full-test and coverage bounds now allow 90 minutes. In run 36741290868, the required gate passed but informational base coverage failed the direct-group test from #1712 (6,857 passed / 1 failed / 2 ignored), revealing the asynchronous test race fixed here. That failed base produced no complete coverage report and is not counted as all-green validation. The final run above passed both head and base.
The September 30 audit found 16 compilation shutdowns and one assertion failure among 17 inspected failed coverage jobs; none failed the percentage gate. The shutdown signature predates the parallel split. Resource observations support more capacity and better diagnostics, but do not establish OOM or exhaustion of GitHub-wide compute as the root cause. A passing pipeline cannot establish zero future flakes or guaranteed speedup. Safe savings here come from removing unused reports and obsolete scans; database semantics and useful parallelism are retained. Historical audit and implementation details:
docs/plans/ci-robustness-and-latency.md.