story-kit: accept 319_refactor_split_workflow_rs_into_story_bug_and_test_result_modules

This commit is contained in:
Dave
2026-03-20 01:30:10 +00:00
parent f5b1103bf6
commit a9d5e9f6f8
2 changed files with 0 additions and 0 deletions

View File

@@ -0,0 +1,32 @@
---
name: "Split mcp.rs into domain-specific tool modules"
---
# Refactor 318: Split mcp.rs into domain-specific tool modules
## Current State
- TBD
## Desired State
mcp.rs is 4,925 lines containing all MCP tool definitions and handlers in a single file. Split by domain:
- mcp/mod.rs — JSON-RPC dispatch, tools list registration
- mcp/story_tools.rs — create_story, update_story, check_criterion, add_criterion, accept_story, move_story
- mcp/agent_tools.rs — start_agent, stop_agent, list_agents, get_agent_output, wait_for_agent
- mcp/merge_tools.rs — merge_agent_work, get_merge_status, move_story_to_merge, report_merge_failure
- mcp/qa_tools.rs — request_qa, approve_qa, reject_qa, launch_qa_app
- mcp/diagnostics.rs — get_server_logs, get_pipeline_status, get_token_usage, rebuild_and_restart
- Tests split into corresponding test modules
## Acceptance Criteria
- [ ] mcp.rs split into domain-specific modules under http/mcp/
- [ ] JSON-RPC dispatch remains in mod.rs, delegates to domain handlers
- [ ] All tool handler signatures unchanged
- [ ] All existing tests pass
- [ ] No public API changes
## Out of Scope
- TBD

View File

@@ -0,0 +1,30 @@
---
name: "Split workflow.rs into story, bug, and test result modules"
---
# Refactor 319: Split workflow.rs into story, bug, and test result modules
## Current State
- TBD
## Desired State
workflow.rs is 2,117 lines handling story file manipulation, bug file creation, test result reading/writing, and section replacement utilities all in one file. Split into:
- workflow/mod.rs — shared utilities (find_story_file, replace_section_content)
- workflow/story_ops.rs — create_story_file, update_story_in_file, story acceptance criteria operations
- workflow/bug_ops.rs — create_bug_file, close_bug operations
- workflow/test_results.rs — write_test_results_to_story_file, read_test_results_from_story_file, coverage operations
- Tests split into corresponding test modules
## Acceptance Criteria
- [ ] workflow.rs split into focused modules under http/workflow/
- [ ] Shared utilities remain in mod.rs and are re-exported
- [ ] All public function signatures unchanged
- [ ] All existing tests pass
- [ ] No public API changes
## Out of Scope
- TBD