Skill: Code Refactoring

Name: refactoring Description: Domain-based module extraction and code organization Trigger: "Refactor this file", large files (>200 lines), code organization

Objective

Break large files (>200 lines) into focused domain modules.

Domain-Based Folder Structure

src/
  large_module/
    mod.rs              # Public interface, re-exports
    domain_a.rs         # First domain (focused responsibility)
    domain_b.rs         # Second domain (focused responsibility)
    domain_c.rs         # Third domain (focused responsibility)

Refactoring Workflow

1. Identify domains - Group related functions/structs by responsibility
2. Create new files - One file per domain
3. Move code - Cut from old file, paste into new domain files
4. Update mod.rs - Add mod domain_a; and pub use domain_a::*;
5. Fix imports - Update use statements across codebase
6. Validate - cargo check after each domain extraction
7. Quality gates - clippy, fmt, tests

Example: cli_logic.rs Refactoring

Before: 1,358 lines in cli_logic.rs

After: 10 focused modules

• cli_logic/mod.rs - Public interface (50 lines)
• cli_logic/cli_logic_entry_point.rs - Main coordination (150 lines)
• cli_logic/cli_logic_interactive.rs - Interactive interface (200 lines)
• cli_logic/cli_logic_tool_execution.rs - Tool execution (120 lines)
• cli_logic/cli_logic_update_operations.rs - Update logic (130 lines)
• cli_logic/cli_logic_info_operations.rs - Info display (90 lines)
• cli_logic/cli_logic_list_operations.rs - List operations (80 lines)

Benefits

• Each file has single, clear responsibility
• Easy to navigate and understand
• Easier to test in isolation
• Reduced merge conflicts
• Better code organization

Validation After Each Step

cargo check        # Must compile
cargo clippy -- -D warnings  # No warnings
cargo fmt --all    # Properly formatted
cargo test         # Tests pass