Здесь нет правильного или неправильного ответа, поскольку это во многом зависит от обстоятельств.
Если код является живым, работает, не документирован и не содержит инфраструктуры тестирования, то я бы не стал его трогать.Если кто-то вернется в будущем и захочет получить новые функции, я постараюсь использовать их в существующем коде, изменяя при этом как можно меньше.
Если код содержит ошибки, проблемы, отсутствуют функции и был написанпрограммист, который больше не работает с компанией, тогда я, вероятно, перепроектировал бы и переписал бы все это.Я всегда мог ссылаться на код этого программиста для конкретного решения конкретной проблемы, но это помогло бы мне реорганизовать все в моем уме и в источнике.В этой ситуации все, вероятно, плохо спроектировано, и в нем можно было бы полностью переосмыслить.
Для всего, что между ними, я бы использовал изложенный вами подход.Я бы начал с косметической очистки всего, чтобы я мог видеть, что происходит.Затем я начинал работать над тем кодом, который требовал наибольшей работы.Я хотел бы добавить документацию, поскольку я понимаю, как она работает, чтобы я мог вспомнить, что происходит.
В конечном счете, помните, что если вы собираетесь поддерживать код сейчас, он должен соответствовать вашим стандартам.Где это не так, вы должны потратить время, чтобы привести его в соответствие со своими стандартами - что бы это ни потребовало.Это сэкономит вам много времени, усилий и разочарования в будущем.