refactorlisted

<objective> Test-first refactoring. Audit coverage, add characterization tests if missing, apply changes with safety net. NOT for: - bug fixes (use `/develop:fix`) - new features (use `/develop:feature`) - `.claude/` config changes (use `/foundry:manage` (requires foundry plugin)) - non-Python projects (JS/TS/Go/Rust) — toolchain assumes pytest; use language-native toolchain instead - mixed refactor+feature tasks — run /develop:refactor first, then /develop:feature; do not attempt both in single skill run </objective> <constants> - MAX_INNER_CYCLES: 5 (change-test cycles per outer session — Step 4 safety break) </constants> <workflow> <!-- Agent Resolution: resolved at runtime via $_DEV_SHARED; source at plugins/develop/skills/_shared/agent-resolution.md --> ## Agent Resolution ```bash _PATHS=$(python "${CLAUDE_PLUGIN_ROOT:-plugins/develop}/bin/dev_shared_resolve.py" --foundry 2>/dev/null) # timeout: 5000 _DEV_SHARED=$(echo "$_PATHS" | head -1) _FOUNDRY_SHARED=$(echo "$_PATHS" | tail -1) ``` Read `$_DEV_SHARED/agent-resolution.md`. Contains: foundry check + fallback table. If foundry not installed: use table to substitute each `foundry:X` with `general-purpose`. Agents skill uses: `foundry:sw-engineer`, `foundry:qa-specialist`, `foundry:linting-expert`, `foundry:challenger`. Read `$_DEV_SHARED/task-hygiene.md`. ## Anti-Rationalizations | Temptation | Reality | | --- | --- | | "The code is simple enough — I can skip characterization tests" | No safety net = no p