Skip to content

test(coverage): ratchet autogen branch coverage gate#11

Merged
OgeonX-Ai merged 3 commits into
mainfrom
feat/phase-26-coverage-gates
Jul 8, 2026
Merged

test(coverage): ratchet autogen branch coverage gate#11
OgeonX-Ai merged 3 commits into
mainfrom
feat/phase-26-coverage-gates

Conversation

@OgeonX-Ai

Copy link
Copy Markdown
Contributor

Summary

Implements the Phase 26-02 coverage ratchet for autogen and records the verified local gate results.

Changes

  • updates the pytest coverage command to enforce total coverage at 73
  • adds branch coverage telemetry enforcement at 54.6 from coverage.xml
  • adds focused tests for maf setup and phase 3 routing paths
  • aligns requirements with the verified repo-local virtualenv stack

Why

Phase 26-02 requires a measurable branch coverage ratchet backed by repo-declared dependencies and local verification evidence.

Validation

  • repo-local .venv full suite passed with coverage
  • total coverage reached 73.26 percent
  • branch coverage reached 54.65 percent
  • pip check passed

Risks

  • fresh scratch install no longer has the earlier resolver conflict, but a full install was not proven end-to-end before timeout
  • remote CI still needs confirmation on this PR

Related

  • root planning phase 26-02

Reviewer notes

Start with requirements.txt and .github/workflows/ci.yml, then review the added routing and maf setup tests.

@OgeonX-Ai OgeonX-Ai changed the title Phase 26-02: ratchet autogen branch coverage gate test(coverage): ratchet autogen branch coverage gate Jul 6, 2026

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 7e929035e3

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .github/workflows/ci.yml Outdated
- name: Emit branch coverage telemetry
if: always()
run: |
python - <<'PY'

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Use a cross-platform shell for telemetry

This job still runs on windows-latest, and this step does not override the default shell; GitHub's workflow syntax docs state that an unspecified Windows run step uses pwsh, while this python - <<'PY' heredoc is Bash syntax, so the Windows matrix leg will fail parsing the telemetry step even when tests pass. Set shell: bash for this step or use shell: python/a script file instead. See GitHub docs: https://docs.github.com/actions/using-workflows/workflow-syntax-for-github-actions#jobsjob_idstepsshell

Useful? React with 👍 / 👎.

Aitomates added 2 commits July 7, 2026 00:57
Use a cross-platform Python shell for coverage telemetry and align maf setup tests with the current tool surface and fallback helper access.
@OgeonX-Ai OgeonX-Ai enabled auto-merge (squash) July 8, 2026 17:02
@OgeonX-Ai OgeonX-Ai merged commit 91d12d3 into main Jul 8, 2026
6 checks passed
@OgeonX-Ai OgeonX-Ai deleted the feat/phase-26-coverage-gates branch July 8, 2026 17: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.

2 participants