Код-ревью, которое объясняет причину и предлагает исправление.
ArchPilot смотрит только изменённый код, комментирует точную строку, объясняет почему и предлагает фикс. Существующий долг остаётся за пределами вашего PR, а исправленный комментарий закрывается сам.
Шесть ситуаций, которые снова и снова возникают в этой роли, и что AppArch делает с каждой из них.
01
Линтер считает стиль и не видит архитектуру
Пайплайн зелёный, все точки с запятой на месте, а контроллер по-прежнему ходит прямо в базу данных.
Что делает AppArch
AppArch смотрит на связи между модулями: интерфейс, обращающийся к базе напрямую, пропущенный слой, циклическая зависимость. А не на пропущенную точку с запятой.
Комментарии ревью называют проблему, но не решение
Комментарий, который говорит только «неправильно», оставляет гадать, как правильно, и ревью уходит на третий круг.
Что делает AppArch
Каждый комментарий ArchPilot содержит обоснование и предлагаемое исправление на языке вашего рабочего пространства, английском или турецком. Не согласны? Ответьте в треде: тон коллеги, а не бота.
Запушьте исправление опечатки, и бот снова публикует те же двенадцать комментариев. К четвергу их уже никто не читает.
Что делает AppArch
У находок есть ключ: одна и та же не публикуется дважды, а комментарий к исправленному нарушению закрывается сам. Merge request остаётся чистым, а вы возвращаетесь к коду.
Изменение в две строки и сорок комментариев к старому коду
Тронете legacy-файл, и инструмент проверяет его целиком, будто вы написали каждую строку сегодня утром.
Что делает AppArch
ArchPilot проверяет только строки, которые меняет ваш merge request. Старый долг по-прежнему учитывается, но в отчёте ArchScan, где ему и место, а не в вашем ревью.
Инструмент ошибается в одном случае и повторяет это при каждом запуске, пока люди не перестают читать всё, что он пишет.
Что делает AppArch
Администратор рабочего пространства может закрыть находку как ложное срабатывание, принятый риск или «не исправлять» с указанием причины. Она остаётся закрытой в следующих запусках, а запись хранит, кто и почему её закрыл.
«Нарушает принцип слоёв». Какой принцип, почему он важен здесь и каким должен быть код вместо этого?
Что делает AppArch
У каждого из 1000+ правил каталога есть своя страница документации с обоснованием, в один клик от комментария. Справочник Arch объясняет паттерны, стоящие за ними.
ArchPilotРевью merge request во времени: доля прошедших, средняя оценка и направление тренда.SmartScanКаждая точка входа прослежена до базы данных, найденные по пути риски сгруппированы по категориям.Здоровье кодаГде на самом деле сидит долг: методы высокого риска, файлы с наибольшим числом находок и насколько отстают зависимости.
Мы используем cookie, чтобы улучшить ваш опыт, анализировать трафик и измерять эффективность нашей рекламы. Аналитические и рекламные cookie включаются только с вашего согласия. Политика конфиденциальности