Базовые приёмы под зелёным

Рефакторинг по определению сохраняет поведение — если наблюдаемое поведение изменилось, это был не рефакторинг. У слова точный смысл: изменение внутренней структуры кода, не меняющее его внешнего поведения. Те же входы, те же выходы, те же побочные эффекты, те же ошибки. Это не приятное свойство, которое ты стараешься удержать; это определение. В тот миг, когда «рефакторинг» заодно чинит баг или добавляет фичу, он перестаёт быть рефакторингом и становится двумя изменениями под одним именем.

Именно эта точность позволяет набору тестов быть твоей страховочной сеткой. Если поведение не изменилось, зелёный набор до правки обязан остаться зелёным после неё. Красный тест поэтому означает ровно одно: ты нарушил инвариант — твой шаг не сохранил поведение. Чем уже шаг, тем точнее красный указывает на ошибку.

Базовый каталог — это горстка маленьких, именованных, обратимых приёмов. Ты не изобретаешь рефакторинги; ты распознаёшь ситуацию и применяешь подходящий приём. Повседневный набор:

// Extract Function: именованный блок становится функцией
// до
const tax = order.lines.reduce((s, l) => s + l.qty * l.price, 0) * 0.2;
// после
const tax = subtotal(order.lines) * 0.2;
function subtotal(lines: Line[]) {
  return lines.reduce((s, l) => s + l.qty * l.price, 0);
}

// Extract Variable: дать имя подвыражению
// до
if (order.total > 100 && order.lines.length > 3) { /* ... */ }
// после
const isBulkOrder = order.total > 100 && order.lines.length > 3;
if (isBulkOrder) { /* ... */ }

Inline Function и Inline Variable — точные обратные приёмы — применяются, когда имя больше не оправдывает себя. Rename меняет имя везде, где оно связано. Move переносит функцию/поле в тот модуль, которому оно действительно принадлежит. Каждый обратим, и именно это делает весь процесс малорискованным: любой шаг, о котором ты пожалел, отменяется его обратным приёмом, без всякого детективного расследования.

Change Function Declaration и Encapsulate Variable — два приёма, что трогают границу, — делай их микрошагами. Переименование функции, добавление/удаление параметра или перестановка аргументов (Change Function Declaration) расходится по всем вызывающим. Любительская версия — один гигантский find-and-replace. Дисциплинированная версия, когда вызывающих много, — это миграционный вариант:

// 1. добавить новую форму рядом со старой, старая делегирует новой
function chargeOld(cents: number) { return charge({ cents }); }
function charge(opts: { cents: number }) { /* настоящее тело */ }
// 2. мигрировать вызывающих на `charge` по одному, тесты зелёные после каждого
// 3. удалить `chargeOld`, когда не осталось ни одного вызывающего

Encapsulate Variable делает то же с данными: оборачивает голое экспортируемое поле в getter/setter, чтобы будущий доступ шёл через одну точку, а затем мигрирует читателей.

// до: голый изменяемый экспорт — каждый читатель это вызывающий, которого ты не видишь
export let config = { region: "eu" };
// после: доступ направлен через функции, которые позже можно защитить или мемоизировать
let _config = { region: "eu" };
export const getConfig = () => _config;
export const setConfig = (c: Config) => { _config = c; };

Каждый пронумерованный подшаг держит набор зелёным, так что ты можешь остановиться и отгрузить на любой строке.

Ритм — это цикл, а не спринт: крошечный шаг → прогон тестов → коммит → повтор. В этом вся дисциплина. Коммит — не бюрократия; это зуб храповика. После каждого зелёного коммита предыдущее хорошее состояние зафиксировано, поэтому цена любой ошибки ограничена одним шагом — git reset --hard или revert этого единственного коммита, и ты снова в заведомо рабочем состоянии. Пропусти коммиты — и один плохой шаг заражает час работы, который ты теперь не можешь чисто отделить.

loop:
  сделай ОДИН приём из каталога   # выделить, переименовать, встроить, переместить…
  прогони набор тестов            # должен быть зелёным (он был зелёным до)
  зелёный? -> git commit          # "refactor: extract subtotal()"  ← храповик щёлкает
  красный? -> откати ЭТОТ шаг      # шаг не сохранил поведение; отмени, повтори мельче

Два режима отказа это убивает: big-bang-рефакторинг (часы правок, потом красный набор без понятия, какая правка виновата) и необратимый рефакторинг (ты изменил так много, что не можешь вернуться к рабочему коду). Маленькие обратимые шаги делают оба невозможными — набор может только щёлкать вперёд.