/refactor — Safe Refactoring
Refactor code safely: verify tests pass before AND after.
Execution
- •Baseline — Run
make test-rustand capture results - •Plan — Enter plan mode, identify what to refactor and why
- •Save plan — Write to
working/plans/YYYY-MM-DD_description.md - •Approval — Present plan, wait for user OK
- •Add tests — If coverage gaps exist, add tests FIRST
- •Refactor — Make changes incrementally
- •Verify after each step —
make test-ruststays green - •Full verify —
make build && make test-rust && make lint - •Review — Spawn review subagents:
code
Task(subagent_type="senior-code-reviewer",
prompt="<.claude/agents/architecture-reviewer.md content>\n\nReview this refactoring:\n{changed_files}")
Task(subagent_type="senior-code-reviewer",
prompt="<.claude/agents/code-reviewer.md content>\n\nReview this refactoring:\n{changed_files}")
Arguments
- •
/refactor crates/ha-state-store— scope of refactoring - •
/refactor extract-service— description of refactoring
Rules
- •NO behavior changes — tests must pass identically before and after
- •If a test breaks, the refactoring changed behavior — fix or reconsider
- •Refactor in small, verified steps — not one big change