refactoring

Refactoring Skill

Restructure code safely while preserving behavior.

Workflow

1. Verify Test Coverage

Before refactoring, ensure tests exist:

# Run existing tests
npm test
pytest
go test ./...

If no tests cover the code:

1. Write characterization tests first
2. These capture current behavior (even if buggy)
3. Then refactor with confidence

2. Understand Current Structure

Map the code:

• What are the dependencies?
• Who calls this code?
• What does it depend on?
• What's the data flow?

# Find usages
grep -r "functionName" --include="*.ts"

# Find dependencies
grep -r "import.*from.*module" --include="*.ts"

3. Plan the Refactoring

Identify the target state:

• What's the problem with current code?
• What's the desired structure?
• What's the smallest step toward that?

Choose your approach:

Situation	Approach
Large function	Extract methods
Duplicated code	Extract shared function
Complex conditionals	Replace with polymorphism
Long parameter list	Introduce parameter object
Feature envy	Move method to data's class
Data clump	Extract class

4. Execute in Small Steps

The safe refactoring cycle:

1. Make ONE small change
2. Run tests
3. Commit if green
4. Repeat

Never skip steps:

• Don't combine multiple refactorings
• Don't "fix bugs" while refactoring
• Don't add features while refactoring

5. Verify Behavior Preserved

# Run full test suite
npm test

# Check for regressions
npm run test:e2e

# Manual smoke test if needed

Quick Reference

Safe Refactorings (Automated)

Most IDEs can do these automatically:

Refactoring	Shortcut (VS Code)
Rename	F2
Extract function	Ctrl+Shift+R
Extract variable	Ctrl+Shift+R
Inline variable	Ctrl+Shift+R
Move to file	Drag in explorer

Common Patterns

Extract Function

// Before
function processOrder(order) {
  // validate
  if (!order.items.length) throw new Error('Empty order');
  if (!order.customer) throw new Error('No customer');

  // calculate total
  let total = 0;
  for (const item of order.items) {
    total += item.price * item.quantity;
  }

  // apply discount
  if (order.coupon) {
    total *= (1 - order.coupon.discount);
  }

  return { ...order, total };
}

// After
function processOrder(order) {
  validateOrder(order);
  const total = calculateTotal(order);
  return { ...order, total };
}

function validateOrder(order) {
  if (!order.items.length) throw new Error('Empty order');
  if (!order.customer) throw new Error('No customer');
}

function calculateTotal(order) {
  const subtotal = order.items.reduce(
    (sum, item) => sum + item.price * item.quantity,
    0
  );
  return order.coupon
    ? subtotal * (1 - order.coupon.discount)
    : subtotal;
}