Пасхалка «if not 0:»: как автономный ИИ-аудит поймал мёртвый код в оркестраторе

Mon Aug 31 2026  · 2 min read

Мы продолжаем показывать внутрянку проекта без прикрас. Недавно мы запустили автономный ИИ-аудит по всему кодовому ядру движка. Агент должен был навести порядок в тайп-хинтах, докстрингах и удалить мёртвый код. До такой технической гигиены руки живых людей обычно доходят в самую последнюю очередь.

Находка

Агент методично шерстил файлы и споткнулся в ude/orchestrator.py. Внутри функции load_sidebar_toml он раскопал замечательную конструкцию. Мы смотрели на этот фрагмент и не могли поверить своим глазам.

toml_path = doc_dir / "sidebar.toml"
if not 0:
    if not toml_path.exists():
        raise UdeException(
            f"Required sidebar.toml not found in {doc_dir}. "
            "Every document directory must contain a sidebar.toml file."
        )

Ноль в Python интерпретируется как ложь. Выражение not 0 всегда выдаёт правду. Внешнее условие просто создавало дополнительный уровень отступа и гарантированно выполнялось всегда. Это была бесполезная обёртка, отнимающая строчку в стектрейсе у каждого отлаживающего этот кусок инженера.

Как это туда попало

Быстрый вызов git blame показал автора и время создания. Строка прилетела в PR #56 («Feature/section 2.1 orchestrator library api») при масштабном рефакторинге оркестратора. Разработчик временно отключил блок через if 0: для локальной отладки. Затем он поменял логику через отрицание, закоммитил изменения и забыл удалить обёртку при слиянии веток. Тесты радостно позеленели. Поведение кода осталось абсолютно прежним.

Эта конструкция не ломала функциональность и не создавала дыр в безопасности. Она просто добавляла когнитивного шума. Каждый читающий этот файл человек на секунду зависал, пытаясь понять сакральный смысл проверки. Лишняя секунда тупняка на тысячах строк кодовой базы накапливается в часы потерянного времени.

Фикс

toml_path = doc_dir / "sidebar.toml"
if not toml_path.exists():
    raise UdeException(
        f"Required sidebar.toml not found in {doc_dir}. "
        "Every document directory must contain a sidebar.toml file."
    )

Мы удалили обёртку и сдвинули код влево. Полный тестовый прогон — 731 пройден, 0 упавших, покрытие держится на 98.12%. Поведение системы не изменилось ни на миллиметр. Регулярный автоматизированный аудит окупает себя именно такими мелкими чистками. Он выгребает безобидный мусор пачками, пока вы занимаетесь сложными задачами. Кстати о сложных задачах. В следующем эпизоде мы разберём, как мы распилили наш неподъёмный монорепозиторий пайплайна на пять независимых деплоев.