From 66b8fa71e411e3c3c2db1eabb85a4d0182b87733 Mon Sep 17 00:00:00 2001 From: Dave Date: Mon, 23 Feb 2026 23:43:51 +0000 Subject: [PATCH] story-kit: create 129_story_test_coverage_http_mcp_rs --- .../129_story_test_coverage_http_mcp_rs.md | 29 +++++++++++++++++++ 1 file changed, 29 insertions(+) create mode 100644 .story_kit/work/1_upcoming/129_story_test_coverage_http_mcp_rs.md diff --git a/.story_kit/work/1_upcoming/129_story_test_coverage_http_mcp_rs.md b/.story_kit/work/1_upcoming/129_story_test_coverage_http_mcp_rs.md new file mode 100644 index 0000000..2d59884 --- /dev/null +++ b/.story_kit/work/1_upcoming/129_story_test_coverage_http_mcp_rs.md @@ -0,0 +1,29 @@ +--- +name: "Add test coverage for http/mcp.rs (72% -> 85%+)" +--- + +# Story 129: Add test coverage for http/mcp.rs + +Currently at 72% line coverage (1826 lines, 475 missed). This is the MCP tool server — the largest module and the interface agents use to interact with the system. + +## What to test + +- Uncovered MCP tool handlers (check which tools lack test coverage) +- Tool argument validation and error messages +- Edge cases in existing tool handlers +- The merge-queue and watcher-pause logic (added in story 119) +- `resolve_simple_conflicts` edge cases +- Tool dispatch routing + +## Notes + +- This is a large file — focus on the uncovered handlers rather than trying to test everything +- Run `cargo llvm-cov --html` to identify specific uncovered lines/functions +- The merge-related tools are the most critical gaps given recent changes +- 475 missed lines is a lot — even covering half would be a big win + +## Acceptance Criteria + +- [ ] Line coverage for `http/mcp.rs` reaches 85%+ +- [ ] Tests pass with `cargo test` +- [ ] `cargo clippy` clean