Skip to content

Commit 2784a6f

Browse files
committed
docs: add coverage verification instruction to CLAUDE.md
Require verifying 100% branch coverage before pushing commits. The project enforces fail_under=100 with branch=true in CI, so catching coverage gaps locally prevents failed CI runs.
1 parent 239d682 commit 2784a6f

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

CLAUDE.md

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,9 @@ This document contains critical information about working with this codebase. Fo
2828
- Bug fixes require regression tests
2929
- IMPORTANT: The `tests/client/test_client.py` is the most well designed test file. Follow its patterns.
3030
- IMPORTANT: Be minimal, and focus on E2E tests: Use the `mcp.client.Client` whenever possible.
31+
- IMPORTANT: Before pushing, verify 100% branch coverage on changed files by running
32+
`uv run --frozen pytest -x` (coverage is configured in `pyproject.toml` with `fail_under = 100`
33+
and `branch = true`). If any branch is uncovered, add a test for it before pushing.
3134

3235
Test files mirror the source tree: `src/mcp/client/streamable_http.py``tests/client/test_streamable_http.py`
3336
Add tests to the existing file for that module.

0 commit comments

Comments
 (0)