Visitar URL original
Accept Only .codecov.yml as the Codecov Configuration Name by ptr727 · Pull Request #4077 · ptr727/ProjectTemplate · GitHub
Skip to content

Accept Only .codecov.yml as the Codecov Configuration Name - #4077

Merged
ptr727 merged 2 commits into
developfrom
feature/codecov-dot-name
Nov 11, 2026
Merged

ptr727 merged 2 commits into
developfrom
feature/codecov-dot-name

Conversation

@ptr727

@ptr727 ptr727 commented Nov 11, 2026

Copy link
Copy Markdown
Owner

Summary

The fleet accepted the Codecov configuration as codecov.yml or .codecov.yml, through a baseline alternatives list that only this entry used. The fleet now accepts .codecov.yml alone, and every fleet repo is being renamed by its own pull request.

  • spec/files.json declares .codecov.yml with no alternatives. The hub's own root file and catalog/snippets/configs/codecov.yml are renamed to .codecov.yml.
  • The alternatives mechanism is removed: spec/files.schema.json, spec/validate.py (alternatives_errors), spec/audit.py (declared_alternatives, the multi-path read_baseline_file, the found plumbing through baseline_file_findings, Run.found and intent_trail), and spec/fidelity_honesty.py.
  • WORKFLOW.md D1.6 names .codecov.yml as the one accepted name and calls any other name drift to rename. STANDUP.md, spec/type-model.md, spec/fidelity-model.md, spec/project-types.json, spec/checks.json, docs/reusable-workflows.md, and the PlexCleaner registry note follow. The generated skill includes are rebuilt.
  • A repository holding only the bare codecov.yml now reports file: .codecov.yml absent, which is the finding each fleet rename clears.

Verification

  • Full suite: 3861 passed, 11 skipped.
  • spec/validate.py OK, scripts/build_dist.py --check current, prose_lint.py --diff origin/develop clean.
  • Local strict review: two recorded passes. The first raised four findings. One was fixed (the PlexCleaner driftNote) and the second pass raised none.

Findings Declined or Deferred

  • No audit check for a leftover bare codecov.yml beside .codecov.yml (introduced, declined). The maintainer chose no stale-name guard. The fleet rename uses git mv, so no repo ends up holding both names.
  • spec/workflow_reuse.py fetch duplicates fidelity_honesty.fetch (pre-existing). workflow_reuse.py is untouched here, and consolidating it is a separate change.
  • spec/validate.py runs no unknown-key check on files.json baseline entries (pre-existing). So the schema's additionalProperties: false is unenforced there, and a stray alternatives key would be ignored silently. That is a separate change.

🤖 Generated with Claude Code

ptr727 and others added 2 commits November 11, 2026 10:25
The fleet accepted the Codecov configuration as `codecov.yml` or `.codecov.yml`, through a
baseline `alternatives` list that only this entry used. The fleet now accepts `.codecov.yml`
alone.

- `spec/files.json` declares `.codecov.yml` with no alternatives, and the hub's own file and
  the catalog snippet are renamed to match.
- The `alternatives` mechanism is removed from the schema, the validator, the audit's reader,
  the intent trail, and the fidelity pass, since nothing else declared one.
- WORKFLOW.md D1.6, STANDUP.md, the type and fidelity models, and the reusable-workflows doc
  name the one file, and the generated skill includes are rebuilt.
- A repository holding only the bare `codecov.yml` now reports `.codecov.yml` absent, which
  is the finding the fleet rename clears.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The note named the Codecov file as carried beyond the baseline, which it never was, and a
sentence began in lowercase after the rename.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI lite review requested due to automatic review settings November 11, 2026 18:30
@coderabbitai

coderabbitai Bot commented Nov 11, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 55 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

Run configuration

Commits

Reviewing files that changed from the base of the PR and between dc6bca5 and ec67aca.

Files ignored due to path filters: 5

  • .claude-plugin/fleet-skills/.source-digests/workflow-ci-contract is excluded by !.claude-plugin/fleet-skills/**
  • .claude-plugin/fleet-skills/skills/workflow-ci-contract/references/d-guarantees.md is excluded by !.claude-plugin/fleet-skills/**
  • .claude-plugin/fleet-skills/skills/workflow-ci-contract/references/test-methodology.md is excluded by !.claude-plugin/fleet-skills/**
  • .github/skills/workflow-ci-contract/references/d-guarantees.md is excluded by !.github/skills/**
  • .github/skills/workflow-ci-contract/references/test-methodology.md is excluded by !.github/skills/**
Files selected for processing: 22

  • .agents/skills/workflow-ci-contract/references/d-guarantees.md
  • .agents/skills/workflow-ci-contract/references/test-methodology.md
  • .codecov.yml
  • STANDUP.md
  • WORKFLOW.md
  • catalog/snippets/configs/.codecov.yml
  • docs/reusable-workflows.md
  • registry/repos.json
  • spec/audit.py
  • spec/checks.json
  • spec/fidelity-model.md
  • spec/fidelity_honesty.py
  • spec/files.json
  • spec/files.schema.json
  • spec/project-types.json
  • spec/secret_names.py
  • spec/type-model.md
  • spec/validate.py
  • tests/test_audit.py
  • tests/test_audit_helpers.py
  • tests/test_fidelity_honesty.py
  • tests/test_spec_validate.py
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

@ptr727

ptr727 commented Nov 11, 2026

Copy link
Copy Markdown
Owner Author

A recorded local strict-review pass covers head ec67acaa2619ed7f082a85e7c26cff23a70384ee, the content this pull request carries at that commit against develop, and it recorded 0 findings.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🔵 Needs a closer look

The change affects fleet-wide baseline and audit semantics while dependent repository renames are landing separately, requiring maintainer confirmation of coordinated rollout and historical-report handling.

No open issues 🎉

What changed in this PR

This PR standardizes the fleet on .codecov.yml as the sole accepted Codecov configuration name and removes the unused baseline-alternatives mechanism from validation, auditing, fidelity checks, and documentation.

Changes:

  • Renames the hub and catalog Codecov configurations to .codecov.yml.
  • Removes alternatives handling from schemas, validators, audit logic, fidelity analysis, and tests.
  • Updates workflow contracts, specifications, documentation, registry notes, and generated skill distributions.
File Description
WORKFLOW.md Defines .codecov.yml as the only accepted Codecov configuration name.
tests/​test_spec_validate.py Tests the manifest's single .codecov.yml entry and removal of alternatives.
tests/​test_fidelity_honesty.py Tests fidelity reads at the declared .codecov.yml path.
tests/​test_audit.py Tests audit acceptance of .codecov.yml and rejection of the bare filename.
tests/​test_audit_helpers.py Updates audit helper tests for single-path reads and intent tracking.
STANDUP.md Updates baseline setup guidance to use .codecov.yml.
spec/​validate.py Removes validation for baseline alternatives.
spec/​type-model.md Documents the single accepted Codecov filename.
spec/​secret_names.py Updates Codecov coverage documentation strings.
spec/​project-types.json Updates the Python coverage contract to reference .codecov.yml.
spec/​files.schema.json Removes the alternatives schema property.
spec/​files.json Declares .codecov.yml as the canonical baseline path.
spec/​fidelity-model.md Removes the alternatives fidelity model.
spec/​fidelity_honesty.py Reads intent files only at their declared paths.
spec/​checks.json Simplifies the baseline presence-check description.
spec/​audit.py Removes alternative-path lookup and found-path tracking from audits.
registry/​repos.json Updates the PlexCleaner drift note to the new filename.
docs/​reusable-workflows.md Updates reusable workflow guidance to reference .codecov.yml.
catalog/​snippets/​configs/​.codecov.yml Provides the renamed canonical Codecov configuration snippet.
.github/​skills/​workflow-ci-contract/​references/​test-methodology.md Regenerates workflow-contract methodology guidance.
.github/​skills/​workflow-ci-contract/​references/​d-guarantees.md Regenerates workflow-contract guarantee documentation.
.codecov.yml Adds the hub's root Codecov configuration under the accepted name.
.claude-plugin/​fleet-skills/​skills/​workflow-ci-contract/​references/​test-methodology.md Updates the generated Claude skill methodology reference.
.claude-plugin/​fleet-skills/​skills/​workflow-ci-contract/​references/​d-guarantees.md Updates the generated Claude skill guarantees reference.
.claude-plugin/​fleet-skills/​.source-digests/​workflow-ci-contract Refreshes the generated workflow-contract source digest.
.agents/​skills/​workflow-ci-contract/​references/​test-methodology.md Updates the authored workflow-contract methodology reference.
.agents/​skills/​workflow-ci-contract/​references/​d-guarantees.md Updates the authored workflow-contract guarantees reference.

Review effort: Lite

@codecov

codecov Bot commented Nov 11, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.29%. Comparing base (dc6bca5) to head (ec67aca).

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #4077      +/-   ##
===========================================
- Coverage    82.34%   82.29%   -0.05%     
===========================================
  Files           38       38              
  Lines        12951    12917      -34     
===========================================
- Hits         10664    10630      -34     
  Misses        2287     2287              
Flag Coverage Δ
python-3.13 82.29% <100.00%> (-0.05%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ptr727

ptr727 commented Nov 11, 2026

Copy link
Copy Markdown
Owner Author

@coderabbitai review

@ptr727

ptr727 commented Nov 11, 2026

Copy link
Copy Markdown
Owner Author

/review

@coderabbitai

coderabbitai Bot commented Nov 11, 2026 •

Copy link
Copy Markdown
Action not completed

Review rate limited.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@ptr727

ptr727 commented Nov 11, 2026

Copy link
Copy Markdown
Owner Author

Declined: the coordinated rollout is the maintainer's request and is under way. Every fleet repo carrying a Codecov file is getting its own rename PR into develop, and four are already merged (MediaTools #127, AudioCleaner #26, aiopurpleair #291, Blog #601). Until a repo's rename reaches its main, the audit reports .codecov.yml absent there, and the rename clears that. The historical reports under reports/ are point-in-time records and stay as written. No code change is owed here.

@ptr727
ptr727 merged commit 728f0e3 into develop Nov 11, 2026
14 checks passed
@ptr727
ptr727 deleted the feature/codecov-dot-name branch November 11, 2026 18:43
ptr727 added a commit that referenced this pull request Nov 12, 2026
…s Rule, Companion-Free Lint, Root hub_fetch_run, .codecov.yml (#4083)

## Summary

Promotes burndown round 3 of the attended run of 2026-11-10, together
with two pull requests from other hub sessions, from `develop` to
`main`.

- #4082: `configure.sh apply` refuses a repository the registry does not
list, before any write. A repository whose entry is on `develop` but not
yet on `main` is configured and audited from a hub checkout at
`develop`. `OPERATIONS.md`, `AUDIT.md`, `GOVERNANCE.md` "Hub-Hosted
Tooling", and the standup, audit, and branch-protection skill text now
say so.
- #4081: `WORKFLOW.md`'s contradictory trigger, merge, and release rules
are reconciled. D1.5 requires a gate to trigger on `pull_request`, and
the audit reports a gate that does not. A retarget is cleared by a push
or a close and reopen.
- #4078: `CODESTYLE.md` gains an Interrupts rule with Python and Shell
mechanics. `build_dist.py` no longer carries a file's trailing link
definitions into an include of its last heading.
- #4076: `docker_lint.py` imports only the standard library, so the Lint
tasks fetch no companions. The provenance label and the ruff task labels
are corrected. `GOVERNANCE.md` "Repo-Scoped Secrets" states the
owner-first host secret stem.
- #4080, from another session: `hub_fetch_run.py` lives in the
repository root for both hook shapes.
- #4077, from another session: only `.codecov.yml` is accepted as the
Codecov configuration name.

Fixes #4061
Fixes #4011
Fixes #4051
Fixes #4048
Fixes #4060
Fixes #3947
Fixes #4072
Fixes #3915
Fixes #4059
Fixes #4071

🤖 Generated with [Claude Code](https://claude.com/claude-code)


<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

<!-- coderabbit-ai-generated:v1 -->

## Summary by CodeRabbit

* **Documentation**
* Clarified when repository checks and configuration use the `develop`
branch, and documented that configuration changes are not applied to
unregistered repositories.
* Expanded scripting guidance for handling interruptions, cleaning up
resources, and saving persistent state safely.
* Updated workflow, coverage, and hook setup guidance to reflect current
requirements and locations.

* **Bug Fixes**
* Audits now check only the declared baseline filename; repositories
using a different filename are reported as missing.
* Markdown heading extraction no longer includes trailing link
definitions in the heading body.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->
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