storkit: done 418_refactor_split_pool_auto_assign_rs_into_submodules

This commit is contained in:
dave
2026-03-28 08:48:05 +00:00
parent d375c4b1d3
commit c0ea5f0cb8
2 changed files with 28 additions and 22 deletions
@@ -1,22 +0,0 @@
---
name: "Unblock command to reset blocked stories"
---
# Story 422: Unblock command to reset blocked stories
## User Story
As a ..., I want ..., so that ...
## Acceptance Criteria
- [ ] Bot command `unblock <story_id>` clears blocked flag and resets retry_count to 0 on the story front matter
- [ ] Replies with confirmation including story ID and name
- [ ] Returns clear error if story is not found or not blocked
- [ ] Works from all transports (Matrix, WhatsApp, Slack)
- [ ] Exposed as an MCP tool so agents can unblock stories programmatically
- [ ] Works as a slash command in the web UI
## Out of Scope
- TBD
@@ -0,0 +1,28 @@
---
name: "Split pool/auto_assign.rs into submodules"
---
# Refactor 418: Split pool/auto_assign.rs into submodules
## Current State
- TBD
## Desired State
Refactor the monolithic server/src/agents/pool/auto_assign.rs (1813 lines) into focused submodules.
## Acceptance Criteria
- [ ] auto_assign.rs contains auto_assign_available_work and its unit tests
- [ ] reconcile.rs contains reconcile_on_startup and its unit tests
- [ ] watchdog.rs contains run_watchdog_once, spawn_watchdog, check_orphaned_agents and their unit tests
- [ ] scan.rs contains scan_stage_items, is_story_assigned_for_stage, count_active_agents_for_stage, find_free_agent_for_stage, is_agent_free and their unit tests
- [ ] story_checks.rs contains read_story_front_matter_agent, has_review_hold, is_story_blocked, has_merge_failure and their unit tests
- [ ] mod.rs wires the submodules and re-exports all public items
- [ ] Unit tests live in their respective module files
- [ ] No public API changes — all existing imports continue to work
## Out of Scope
- TBD