fix-codecov-gaps

Fix Codecov Coverage Gaps

Restores or improves patch coverage after Codecov flags missing/partial lines in a pull request.

When to Use

• Codecov bot posts a report showing patch coverage below the project baseline
• A PR introduces new lines without accompanying tests
• Coverage regresses (Misses or Partials increase vs. base)

Procedure

1. Fetch the Codecov Report

Use mcp_github_pull_request_read with method: get_comments to read PR comments, then locate the comment from codecov[bot]. Extract:

• Patch coverage % (overall for this PR's changed lines)
• Files with missing lines — collect the file path, patch %, missing count, and partial count for each row in the table
• Coverage diff summary — Hits Δ, Misses Δ, Partials Δ vs. base

Example output to parse:
| src/cli.ts          | 80.30% | 12 Missing + 1 partial |
| src/onboarding/wizard.ts | 83.33% | 0 Missing + 5 partials |

2. Identify Uncovered Lines Per File

For each file listed in the Codecov report:

1. Open the file and read its contents
2. Cross-reference the diff (mcp_github_pull_request_read with method: get_diff) to pinpoint which new/changed lines are uncovered
3. Classify each gap:
   • Missing line — the line was never executed in any test
   • Partial branch — the line ran but not all branches (e.g. both sides of a ternary or ??)

Priority order: tackle files with the most missing lines first. Partials are secondary.

3. Write Targeted Tests

For each uncovered line or partial branch, add or extend a test in src/tests/<mirrored-path>/<module>.test.ts:

• Missing line (error path / guard clause): Write a test that triggers the error condition or the early-return branch
• Missing line (happy path): Write a test that calls the function with valid input that exercises that code path
• Partial branch (ternary / ?? / &&): Add a test case that exercises the other side of the branch

Rules:

• Mirror the source tree: src/cli.ts → src/tests/cli/cli.test.ts
• Use vi.fn() / vi.spyOn() for side effects; never mutate module state across tests
• Each new it() block must contain at least one expect()
• Do not use /* c8 ignore */ to hide the gap — fix the root cause