Навык проверки кода из репозитория agent-skills Адди Османи: перед слиянием (или при проверке кода, написанного другими агентами или людьми) агент оценивает изменения по пяти критериям — корректность, читаемость, архитектура, безопасность и производительность. Сначала проверяются тесты, затем реализация; каждое замечание маркируется уровнем серьезности (Critical / обязательно к исправлению / Nit / Optional / FYI), а в конце выносится решение об одобрении или запросе на доработку.
Количество установок skills.sh 79294 (+975 / 24 ч)
Аудит безопасности: не проводился
- Сканирование еще не выполнено
Обоснование
- Процесс состоит из 5 шагов: сначала выясняется цель изменений, затем проверяются тесты, реализация анализируется по файлам с учетом пяти критериев, проблемы классифицируются по уровню серьезности, а затем сверяются с описанием проверки автора. Перед составлением отчета ложные срабатывания исключаются путем проверки вызывающего кода, защитных механизмов и тестов; отсутствие проблем является допустимым результатом, выдумывать проблемы нельзя.
- Эффективность тестов проверяется экспериментально: одно из условий в изменениях инвертируется, после чего запускаются тесты, а затем файл восстанавливается из копии. Если тесты все еще проходят, это указывает на отсутствие необходимых тест-кейсов, которые нужно указать.
- Приведены ориентиры по объему изменений (около 100 строк подходят для одной проверки, около 1000 строк следует разбить) и четыре способа разбиения. При обновлении зависимостей необходимо читать журналы изменений, обновлять только один пакет за раз и проверять различия в lock-файлах.
- Если обнаружен неиспользуемый код, сначала составляется список, затем спрашивается разрешение на удаление; самовольное удаление запрещено. При выявлении структурных проблем одновременно предлагаются конкретные варианты рефакторинга.
- Репозиторий распространяется под лицензией MIT; в том же репозитории есть еще 24 навыков. На skills.sh за последние 24 часов было выполнено 975 установок, всего накоплено 79294. За последние 8 недель еженедельное число установок выросло с 3503 до 5359 (что отражает только уровень интереса).
Использование
- Команда установки появится после успешного прохождения аудита; навык активируется при запросе «проверить этот PR / diff», когда функциональность завершена и готова к слиянию, или когда нужно оценить код, написанный другим агентом.
- Не требуется учетная запись или внешние сервисы; шаг «инвертирование условия для запуска тестов» требует возможности локального запуска тестов проекта.
Риски
- В этом выпуске аудит не проводился, установлен фиксированный рейтинг C, команда установки не отображается. Файл SKILL.md представляет собой преимущественно чек-лист проверки; единственное действие, изменяющее файлы, — «временная инверсия одного условия для запуска тестов с последующим восстановлением из копии», что соответствует назначению навыка, однако необходимо убедиться, что файл был восстановлен. На skills.sh статус аудита Gen Agent Trust Hub и Socket — Pass, а Snyk — Warn.
- Разделы по безопасности и производительности ссылаются на навыки security-and-hardening и performance-optimization из того же репозитория, а также на файлы чек-листов в корне репозитория; при установке только этого навыка эти ссылки могут быть недоступны.
- Критерии одобрения довольно мягкие: изменение одобряется, если оно в целом улучшает состояние кодовой базы. Такие командные нормы, как «ответ в течение одного рабочего дня», мало применимы к индивидуальным проектам.