Skip to content

Enforce the 80% Cobertura line-coverage gate - #551

Open
drmoisan wants to merge 10 commits into
epic/build-ci-coverage-gate-fidelity-integrationfrom
bug/coverage-threshold-policy-reconciliation-494
Open

Enforce the 80% Cobertura line-coverage gate#551
drmoisan wants to merge 10 commits into
epic/build-ci-coverage-gate-fidelity-integrationfrom
bug/coverage-threshold-policy-reconciliation-494

Conversation

@drmoisan

@drmoisan drmoisan commented Aug 13, 2026 •
edited
Loading

Copy link
Copy Markdown
Owner

Summary

  • Enforces the 80% Cobertura line-coverage threshold before the coverage command completes successfully.
  • Rejects missing, non-numeric, and below-threshold line-coverage summaries while accepting the exact 80% boundary.
  • Preserves the upstream coverage-policy reconciliation prompt as the TaskMaster deliverable and defers upstream application outside this repository.
  • Records scope, regression, and final-review evidence for issue Bug: conflicting-coverage-thresholds-across-policy-docs #494.
  • REVIEW_STATUS: PASS.

Why

The repository had incompatible always-loaded coverage-policy surfaces and no enforced local boundary for Cobertura line coverage. This change establishes the required local 80% gate while preserving the upstream reconciliation work as a deferred, externally owned deliverable.

What Changed

  • Coverage gate: evaluates the generated Cobertura line-coverage summary before successful completion and fails for missing, invalid, or below-80% coverage.
  • Tests: adds deterministic Pester coverage for negative cases, the exact boundary, above-boundary input, and main-entrypoint wiring; selected tests passed 51/0/0.
  • Documentation and evidence: adds corrected scope, acceptance-criteria, protected-path, and final-review evidence while retaining the upstream Claude reconciliation prompt as the complete local deliverable.

Verification

  • REVIEW_STATUS: PASS; the final feature audit reports 10 of 10 acceptance criteria passing.
  • Exact-head CI run 31749877507 for d5e4a99687f22ecf14b88a206db623d85fe9a97f succeeded after workflow_dispatch, including actionlint and Format, build, analyze, and test.
  • Targeted Pester verification passed: 51 passed, 0 failed, 0 skipped.
  • Scope validation passed, including git diff --check; no TaskMaster CLAUDE.md, non-memory .claude/**, or .agents/skills/** runtime-policy path changed.
  • PSScriptAnalyzer retains the approved pre-existing 225-diagnostic baseline with zero branch delta; it requires no issue Bug: conflicting-coverage-thresholds-across-policy-docs #494 remediation.

Scope and Follow-up

The upstream coverage-policy reconciliation remains deferred; the retained prompt defines the future upstream deliverable without changing local runtime policy paths. Re-run workflow-dispatch CI for any later head commit.

GitHub Auto-close

drmoisan added 10 commits August 11, 2026 12:03
@drmoisan
(docs(coverage-policy)): refine #494 plan gates and baseline evidence
69493db 
- Capture Phase 0 policy-read and baseline git-state evidence for issue #494
- Record executor and planner memory for #511 crash handling and Pester/MCP coverage-tool seams
- Revise the feature 494 plan to replace unsatisfiable gates and stale span assumptions

Refs: #494
@drmoisan
docs(coverage-policy): record validation evidence and review blockers
a625677 
- Capture three corrected-arithmetic coverage runs with 6,435 tests passing in each run
- Record the absent upstream release receipt and leave all ten acceptance criteria unresolved
- Add policy, code, and feature audits with a scoped remediation plan

Refs: #494
@drmoisan
docs(coverage-policy): record post-review remediation halt
018e586 
- Add canonical policy, code, and feature review artifacts
- Capture prompt-contract and receipt-gate evidence for the absent upstream release validation receipt
- Complete Phase 0 and Phase 1 checklist items and halt before receipt-dependent Phase 2

Refs: #494
@drmoisan
fix(coverage): enforce 80% Cobertura line coverage threshold
8f36c21
@drmoisan
docs(coverage): reconcile remediation scope for agent-memory records
d25b9a9
@drmoisan
docs(coverage): record final issue #494 audit
51c5426
@drmoisan
docs(coverage-policy): define remediation scope and acceptance source
a762ecd 
- Establish spec.md acceptance criteria as the sole check-off source
- Document local-only scope, protected-path classification, and historical scenarios
- Record scoped validation evidence for the remediation plan

Refs: #494
@drmoisan
docs(coverage-policy): add final feature review audits
b1404d2 
- Record code, feature, and policy audit evidence for issue #494
- Document passing acceptance and Pester results with the analyzer gate finding

Refs: #494
@drmoisan
docs(coverage-policy): align audit verdict with analyzer baseline
73adecd 
- Mark the approved zero-delta analyzer baseline as no-regression evidence
- Record PASS readiness and defer exact-head CI as post-review work

Refs: #494
@drmoisan
docs(coverage-policy): clarify analyzer baseline disposition
d5e4a99 
- Record that the approved zero-delta baseline requires no issue #494 action
- Preserve separate tracking for pre-existing analyzer-warning cleanup

Refs: #494
@drmoisan drmoisan changed the title Enforce the 80% Cobertura line-coverage threshold Enforce the 80% Cobertura line-coverage gate Aug 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@drmoisan