[MAJOR] §2: README badges point to personal account, not organization

[mvillmow]

Finding

Severity: MAJOR
Section: 2
Evidence: README.md:7-9
Principle: POLA

The Quality Gates and Code Coverage badges reference github.com/mvillmow/ProjectKeystone (a personal account) rather than github.com/HomericIntelligence/ProjectKeystone (the organization). The badges are static with hardcoded values ('481 passing', '86.2%') rather than live CI status. These badges are misleading for consumers of the repository.

---

Part of #504

[mvillmow]

Implementation Plan

Now I have everything I need. Here's the implementation plan:

---

Implementation Plan: Issue #516 — Fix README Badges

1. Objective

Fix three problems in README.md lines 7–10:

1. Quality Gates badge URL points to personal account mvillmow instead of org HomericIntelligence
2. Code Coverage badge has a hardcoded static value (86.2%) rather than a live CI-driven badge
3. Tests badge has a hardcoded static value (481 passing) rather than a live CI-driven badge

---

2. Approach

Key decisions:

• Fix the org slug in the Quality Gates badge URL (mvillmow → HomericIntelligence)
• Replace the static coverage badge with a dynamic Codecov badge (Codecov is the standard for C++ projects with CI; it integrates with GitHub Actions via the extras.yml workflow which already produces coverage reports)
• Remove the static tests-count badge entirely — test counts change with every PR, the badge drifts immediately, and there's no CI-generated artifact to back a dynamic version; replace with a link to the workflow run log instead
• Keep the badge link target pointing to the correct org workflow URL

Workflow files to reference:

• Primary CI: .github/workflows/_required.yml (name: Required Checks)
• Coverage: .github/workflows/extras.yml (name: Extras (non-blocking))

---

3. Files to Create

None — this is a documentation-only fix.

---

4. Files to Modify

README.md — lines 8–10

Line	Current	Change
8	[](https://github.com/mvillmow/ProjectKeystone/actions/workflows/quality.yml)	Fix org slug to HomericIntelligence; update workflow filename to _required.yml (the actual file)
9	[](...)	Replace with dynamic Codecov badge: [](https://codecov.io/gh/HomericIntelligence/ProjectKeystone)
10	[](.github/workflows/ci.yml)	Remove static count badge; replace with [](https://github.com/HomericIntelligence/ProjectKeystone/actions/workflows/_required.yml) or remove entirely

Also update line 29 prose: 481 unit and E2E tests — remove the hardcoded count or replace with a non-numeric description (e.g., "comprehensive unit and E2E tests").

---

5. Implementation Order

1. Edit README.md badge block (lines 8–10)

   • Change mvillmow → HomericIntelligence in Quality Gates badge URL and link
   • Change workflow filename from quality.yml → _required.yml in both badge URL and link
   • Replace static Code Coverage badge with Codecov dynamic badge
   • Replace static Tests badge with a second live CI badge (Extras workflow) or remove it

2. Edit README.md prose (line 29)

   • Remove 481 hardcoded count from "Comprehensive Testing" bullet

3. Verify markdown renders correctly (visual check of badge syntax)

---

6. Verification

• Manual: Open the edited README.md in a markdown previewer and confirm all three badges render (no broken image icons)
• Link correctness: Confirm badge URLs follow the pattern https://github.com/HomericIntelligence/ProjectKeystone/actions/workflows/<name>.yml/badge.svg — the workflow filenames _required.yml and extras.yml exist in .github/workflows/
• No static values: Grep the badge block for hardcoded percentages or test counts: grep -n 'badge/coverage\|badge/tests\|passing\|86\.2' README.md should return no results after the fix
• CI check: The existing markdownlint job in _required.yml will validate markdown syntax on the next PR

---

7. Skills Used

Invoked during planning:

• None (direct file inspection was sufficient)

Team knowledge base skills referenced (from Prior Learnings):

• readme-ci-badges — used to confirm the live badge URL pattern for GitHub Actions
• add-ci-badge-to-readme — confirmed the org slug fix approach
• replace-hardcoded-counts-with-live-commands — informed the decision to remove the static 481 passing and 86.2% values rather than update them
• badge-count-drift-prevention — confirmed that removing the static count badge is preferable to a manual-update approach

---

Generated by Claude Code Planner