Skip to content

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

Merged
drmoisan merged 10 commits into
epic/build-ci-coverage-gate-fidelity-integrationfrom
bug/coverage-threshold-policy-reconciliation-494
Aug 15, 2026
Merged

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

Conversation

@drmoisan

@drmoisan drmoisan commented Aug 13, 2026

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 drmoisan changed the title Enforce the 80% Cobertura line-coverage threshold Enforce the 80% Cobertura line-coverage gate Aug 13, 2026
@drmoisan
drmoisan force-pushed the bug/coverage-threshold-policy-reconciliation-494 branch from d5e4a99 to 36daf03 Compare August 15, 2026 02:15
- 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
- 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
- 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
- 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
- Record code, feature, and policy audit evidence for issue #494
- Document passing acceptance and Pester results with the analyzer gate finding

Refs: #494
- 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
- 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 force-pushed the bug/coverage-threshold-policy-reconciliation-494 branch from 36daf03 to b8b4b16 Compare August 15, 2026 02:26
@drmoisan
drmoisan merged commit 85ff0c3 into epic/build-ci-coverage-gate-fidelity-integration Aug 15, 2026
drmoisan added a commit that referenced this pull request Aug 15, 2026
docs(epic): refresh epic-status projection from git and gh

Wave 2 closed: feature 494 merged via PR #551 (merge commit 85ff0c3) on
2026-08-15T02:27Z. The projection had it at worktree_created.

Also corrects the "Issues Closed" claim. All nine issues are still open on
GitHub because every child PR targeted the integration branch, and GitHub
auto-closes only on merge into the default branch. Renamed the section to
"Issues - Pending Closure" and recorded that the integration PR body carries
the closing keywords.

Records that the tree has moved past the last green workflow_dispatch run
(31493339489 @ c7d398c): feature 494 and the main merge fb8eff9 (CI parallel
job split #553/PR #556) are both uncovered by it, and the job split renamed the
check runs.

Regenerated from git and gh; artifacts/orchestration/epic-orchestrator-state.json
is absent from this worktree and the commands are authoritative per the cache
doctrine.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@
drmoisan added a commit that referenced this pull request Aug 15, 2026
Wave 2 closed: feature 494 merged via PR #551 (merge commit 85ff0c3) on
2026-08-15T02:27Z. The projection had it at worktree_created.

Also corrects the "Issues Closed" claim. All nine issues are still open on
GitHub because every child PR targeted the integration branch, and GitHub
auto-closes only on merge into the default branch. Renamed the section to
"Issues - Pending Closure" and recorded that the integration PR body carries
the closing keywords.

Records that the tree has moved past the last green workflow_dispatch run
(31493339489 @ c7d398c): feature 494 and the main merge fb8eff9 (CI parallel
job split #553/PR #556) are both uncovered by it, and the job split renamed the
check runs.

Regenerated from git and gh; artifacts/orchestration/epic-orchestrator-state.json
is absent from this worktree and the commands are authoritative per the cache
doctrine.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@drmoisan
drmoisan deleted the bug/coverage-threshold-policy-reconciliation-494 branch August 15, 2026 02:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant