story-kit: done 319_refactor_split_workflow_rs_into_story_bug_and_test_result_modules
This commit is contained in:
@@ -1,29 +0,0 @@
|
||||
---
|
||||
name: "Split pool.rs into pipeline, auto-assign, and agent management modules"
|
||||
---
|
||||
|
||||
# Refactor 317: Split pool.rs into pipeline, auto-assign, and agent management modules
|
||||
|
||||
## Current State
|
||||
|
||||
- TBD
|
||||
|
||||
## Desired State
|
||||
|
||||
pool.rs is 5,852 lines and handles agent pool management, pipeline advancement, auto-assign logic, completion handling, and has massive test suites all in one file. Split into focused modules:
|
||||
- pool.rs — core agent pool (start, stop, list, inject)
|
||||
- pipeline.rs — run_pipeline_advance and stage transition logic
|
||||
- auto_assign.rs — auto_assign_available_work and assignment logic
|
||||
- Tests split into corresponding test modules
|
||||
|
||||
## Acceptance Criteria
|
||||
|
||||
- [ ] pool.rs reduced to core agent management (start, stop, list)
|
||||
- [ ] Pipeline advancement logic extracted to agents/pipeline.rs
|
||||
- [ ] Auto-assign logic extracted to agents/auto_assign.rs
|
||||
- [ ] All existing tests pass without modification to test logic
|
||||
- [ ] No public API changes — callers still access everything through AgentPool
|
||||
|
||||
## Out of Scope
|
||||
|
||||
- TBD
|
||||
@@ -1,32 +0,0 @@
|
||||
---
|
||||
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
|
||||
@@ -1,30 +0,0 @@
|
||||
---
|
||||
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
|
||||
Reference in New Issue
Block a user