Skip to content

[Agent] [Chore] Clean coverage artifacts and restore formatter baseline#305

Merged
stranske merged 1 commit intomainfrom
codex/mop-up-artifacts-format
Mar 1, 2026
Merged

[Agent] [Chore] Clean coverage artifacts and restore formatter baseline#305
stranske merged 1 commit intomainfrom
codex/mop-up-artifacts-format

Conversation

@stranske
Copy link
Copy Markdown
Owner

@stranske stranske commented Mar 1, 2026

Why

Two repository hygiene regressions are still present on main:

  1. Tracked .coverage.* artifacts are committed.
  2. Formatting drift causes black --check . to fail.

This PR removes those regressions so the repository baseline is clean again.

What Changed

  • Added /.coverage* to .gitignore.
  • Removed tracked .coverage.* artifacts from git.
  • Ran black . to normalize formatting across files currently out of sync with the formatter baseline.

Validation

  • black --check .
  • ruff check .
  • pytest -q tests/test_counter_risk_cli.py tests/test_chat_session.py tests/test_runner_launch.py

Notes

  • No functional logic changes were intended; edits outside .gitignore are formatter-only.

Copilot AI review requested due to automatic review settings March 1, 2026 01:24
@stranske-keepalive
Copy link
Copy Markdown
Contributor

⚠️ Action Required: Unable to determine source issue for PR #305. The PR title, branch name, or body must contain the issue number (e.g. #123, branch: issue-123, or the hidden marker ).

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR restores repository hygiene by ignoring/removing stray coverage artifacts and re-applying the Black formatter baseline so black --check . is clean again.

Changes:

  • Add /.coverage* to .gitignore and remove tracked .coverage.* files.
  • Apply Black-only formatting normalization across tests, scripts, and source files.
  • Minor whitespace-only cleanup (e.g., trailing blank lines) in a few tests.

Reviewed changes

Copilot reviewed 25 out of 29 changed files in this pull request and generated no comments.

Show a summary per file
File Description
.gitignore Ignores .coverage* artifacts so they aren’t re-committed.
.coverage.runnervmnay03.pid5811.XIBGM8Kx.HQsfEMlJuHGh Removes tracked coverage artifact from git history going forward.
.coverage.runnervmnay03.pid5808.Xif2iyox.HxFiMg0sjQ2h Removes tracked coverage artifact from git history going forward.
.coverage.runnervmnay03.pid5805.XQ6K8i8x.HIUQ70HoAQsh Removes tracked coverage artifact from git history going forward.
scripts/build_runner_workbook.py Black formatting normalization only.
src/counter_risk/chat/providers/base.py Black formatting normalization only.
src/counter_risk/chat/session.py Black formatting normalization only.
src/counter_risk/cli/__init__.py Black formatting normalization only.
src/counter_risk/mosers/template.py Black formatting normalization only.
src/counter_risk/parsers/repo_cash_sources.py Black formatting normalization only.
src/counter_risk/pipeline/run.py Black formatting normalization only.
src/counter_risk/runner_launch.py Black formatting normalization only.
src/counter_risk/writers/historical_update.py Black formatting normalization only.
tests/integration/test_cli_run.py Black formatting normalization only.
tests/spec/test_macro_spec_doc.py Black formatting normalization only.
tests/spec/test_macro_spec_fixtures.py Black formatting normalization only.
tests/test_fixtures_smoke.py Black formatting normalization only.
tests/test_futures_delta.py Black formatting normalization only.
tests/test_mosers_all_programs_output_structure.py Removes trailing whitespace/blank line; no logic change.
tests/test_mosers_ex_trend_output_structure.py Removes trailing whitespace/blank line; no logic change.
tests/test_mosers_trend_output_structure.py Removes trailing whitespace/blank line; no logic change.
tests/test_mosers_workbook.py Black formatting normalization only.
tests/test_nisa_all_programs_parser.py Black formatting normalization only.
tests/test_nisa_parser.py Black formatting normalization only.
tests/test_runner_launch.py Black formatting normalization only (incl. wrapped annotations/defs).
tests/test_runner_vba_module.py Black formatting normalization only.
tests/test_validate_release_workflow_yaml.py Black formatting normalization only.
tests/test_vba_module_inventory.py Black formatting normalization only.
tests/test_vba_runnerlaunch_signature.py Black formatting normalization only.

@stranske-keepalive
Copy link
Copy Markdown
Contributor

🤖 Keepalive Loop Status

PR #305 | Agent: Codex | Iteration 0/5

Current State

Metric Value
Iteration progress [----------] 0/5
Action wait (missing-agent-label)
Disposition skipped (transient)
Gate success
Tasks 0/3 complete
Timeout 45 min (default)
Timeout usage 5m elapsed (12%, 40m remaining)
Keepalive ❌ disabled
Autofix ❌ disabled

🔍 Failure Classification

| Error type | infrastructure |
| Error category | resource |
| Suggested recovery | Confirm the referenced resource exists (repo, PR, branch, workflow, or file). |

@stranske-keepalive
Copy link
Copy Markdown
Contributor

Keepalive Work Log (click to expand)
# Time (UTC) Agent Action Result Files Tasks Progress Commit Gate
0 2026-03-01 01:29:47 Codex wait (missing-agent-label-transient) skipped 0 0/3 success

@stranske stranske merged commit 0e49c00 into main Mar 1, 2026
29 checks passed
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