NW-REFACTOR: Systematic Code Refactoring

NW-REFACTOR: Systematic Code Refactoring

Wave: CROSS_WAVE Agent: Crafty (nw-software-crafter) Command: *refactor

Overview

Applies the Refactoring Priority Premise (RPP) — 6-level hierarchy L1 Readability|L2 Complexity|L3 Responsibilities|L4 Abstractions|L5 Design Patterns|L6 SOLID++. For complex multi-class refactorings, agent applies Mikado Method internally.

Execution Model — Batch-Then-Verify (default, unconditional)

/nw-refactor runs batch-then-verify by default, regardless of test-suite speed:

1. Cascade governs PLANNING ORDER only — analyze and plan lower levels before higher (L2 transformations build on L1, L3 on L2, etc.). The cascade is a planning discipline, not a test-gating sequence.
2. Edits applied as one coherent batch — apply all planned L1-L6 transformations in a single editing session. NO interleaved test runs between levels.
3. Run the test suite exactly ONCE, at the very end, after all L1-L6 edits.
4. If RED after the batch: diagnose the breakage and fix the production code. Do NOT modify tests to make them pass. A test that must change to pass means either (a) the refactor altered observable behavior — revert that transformation — or (b) the test encoded an implementation detail rather than behavior — flag this to the user explicitly before touching it. Tests changing during a refactor is a signal, not a step.

Legacy incremental variant (opt-in only): the nw-progressive-refactoring skill documents the incremental L1→test→L2→test cycle. It is NOT the default — use it only when explicitly requested. Anchor: feedback_refactor_batch_when_test_suite_slow_2026_05_19 (the prior "only batch when suite slow" conditional is removed — batch is now unconditional).

Context Files Required

• src/* - Production codebase
• tests/* - Test codebase

Agent Invocation

@nw-software-crafter

Execute *refactor for {target-class-or-module}.

Context Files:

• src/*
• tests/*

Configuration:

• level: 3 # Shorthand: --from=1 --to=3 (RPP range)
• from: 1 # Start RPP level (default: 1)
• to: 3 # End RPP level (default: same as level)