Refactoring Production Code

STARTER_CHARACTER = 🟣

When starting, announce: "🟣 Using REFACTORING skill".

Work autonomously as much as possible. Start with the simplest thing or file and proceed to the more complex ones.

Stages

1. Prep
2. Main Refactoring
3. Final Evaluation
4. Summary

Test Code Policy

Do not change test code during refactoring, except:

• Renames that follow production code renames (imports, function calls)
• Import path updates if something moved

Never change test assertions, test data, or test logic.

1. Prep

• Determine scope: use specified files, or identify related files (imports, shared functionality), or ask the programmer
• Add files in scope to todo list
• Find or create ./test.sh, verify all tests pass
• Remove comments from files in scope (commit per file)

2. Main Refactoring

Code Style

Prefer self-explanatory, readable code over comments.

• Use functional helper methods for clarity
• Remove dead code
• Extract paragraphs into methods
• Use better variable names
• Remove unused imports
• Remove unhelpful local variables
• Look for opportunities to simplify
• Use domain language - name things for what they ARE, not how they're implemented
• Keep consistent abstraction levels within methods

Process

For each refactor:

1. Ensure all tests pass
2. Choose and perform the simplest possible refactoring (one at a time)
3. Ensure all tests pass after the change
4. Commit each successful refactor using Arlo's commit notation:
   • . r <refactoring> if the change is entirely within test code
   • ^ r <refactoring> if the change is in production code with full test suite passing Prefer small granular commits. If applying the same refactoring pattern to multiple locations, change one location at a time and commit each separately.
5. Provide a status update after each refactor

3. Final Evaluation

When you see no more obvious refactoring opportunities, say "🔍 Entering final evaluation."

Shift focus: you've been implementing. Now become a critic. Your job is to find problems, not produce code.

Re-read Code Style guidelines. Look at each file in scope. Consider blind spots - what improvements haven't we even considered that would make the code better, easier, more maintainable?

For each file, find ONE thing that could be better. If you find something:

1. Fix it using the same refactoring process (test, change, test, commit)
2. Look again; fixing one thing often reveals the next

Repeat until you find nothing more to improve.

4. Summary

Provide a high-level summary of the refactoring:

• List each file that was touched
• Describe the key improvements made in each file

Language-Specific

For Java: See references/java.md