BD code-review-inspector
Автоматизированный обзор кода с проверкой на наличие ошибок, проблем безопасности, лучших практик, проблем с производительностью и стиля кода
машинный переводПоказать оригиналСкрыть оригинал«Automated code review checking for bugs, security issues, best practic…»
Automated code review checking for bugs, security issues, best practices, performance problems, and code style
Автоматизированный обзор кода с проверкой на наличие ошибок, проблем безопасности, лучших практик, проблем с производительностью и стиля кода
Как процесс D 47/100 · Процесс не доведён — слабые места: результат и критерий готовности, когда включается, входы и предусловия
Чем это грозит
Находки средней серьёзности: скорее всего скилл честный, но прочитайте, что именно насторожило сканер.
Ниже описан худший случай для этой категории. Здесь находка средней серьёзности: guard увидел признак, но не доказательство.
Скилл содержит команды, которые удаляют файлы, переписывают диски или исполняют код, скачанный из сети. Агент может выполнить их без вопроса, если считает, что так требует инструкция.
Замените разрушительные команды на безопасные аналоги с подтверждением, ограничьте область действия конкретной папкой и не тяните код через curl | bash: укажите версию и контрольную сумму.
Как улучшить
- Тело SKILL.md длиннее 5 000 токенов: вынесите справочные детали в references/ и подключайте по необходимости.
- Свои кейсы (evals/evals.json, 4–6 реальных запросов с ожидаемыми ответами): тогда полная проверка прогонит именно их, а не черновик от модели.
- spec.yaml с триггерными фразами и утверждениями — контракт поведения для CI; `skilltest init` создаст шаблон.
Находки guard · 2
✓ Критических и высоких находок нет
Средние и низкие: 2
-
средняя Опасные команды
cmd-destructive-fsCODE_REVIEW_CHECKLIST.md:52Разрушительная команда для файловой системы (корень / домашняя папка / диск) (комментарий в коде; документация security-скилла)os.system(f'cat {filename}') # Attack: file=test.txt; rm -rf /комментарийsecurity-скилл -
низкая Опасные команды
cmd-eval-dynamicCODE_REVIEW_CHECKLIST.md:52Динамическое выполнение кода из декодированного/недоверенного ввода (документация security-скилла)os.system(f'cat {filename}') # Attack: file=test.txt; rm -rf /security-скилл
Просканировано файлов: 3. Улики замаскированы. Пометки в серых чипах объясняют, почему серьёзность понижена.
По спецификации Agent Skills
- предупреждение
body-longтело SKILL.md ≈ 5693 токенов (рекомендуется < 5000); вынесите детали в references/
Процессный рейтинг: все десять параметров 47/100
- 0Результат и критерий готовности. Не сказано, что считать результатом
- 0Входы и предусловия. Не сказано, что нужно иметь на входе
- 0Отчётность по ходу. Скилл ничего не сообщает по ходу работы
- 20Когда включается. Не сказано, при каком запросе скилл включается
- 30Повторный запуск. Изменяющих операций: 8, без проверки текущего состояния
- 50Ошибки и развилки. Развилок: 0, есть раздел про ошибки
- 60Инструменты и файлы. Используются инструменты (web, python), но во frontmatter они не объявлены
- 70Стоимость исполнения. Тело инструкции 5693 токенов
- 100Шаги. Шагов: 155
- 100Согласованность. Имя и обязательные поля на месте
- medium Правила безопасности и запреты внутри скилла: их место в системном промпте, здесь они не защищают
- low Разделов верхнего уровня: 13. Похоже на несколько доменов в одном скилле
Всё перечисленное измерено по тексту скилла, а не оценено моделью: цифры проверяемы. Вес параметра тем больше, чем чаще из-за него процесс встаёт.
Сигналы качества
- +5В description нет примеров фраз, по которым скилл должен срабатывать
- +4Описание не говорит, когда скилл НЕ применять (ложные срабатывания)
- +3Длина description 110: рекомендуется 120–800 символов
- +3Формат ответа не описан: модель каждый раз решает сама
- -223 эмодзи в инструкциях: шум для модели
- +1Лицензия не указана
- +2Инструкции на одном языке
- +4Структура: 45 заголовков
- +3Пошаговые инструкции: 155 пунктов
- +4Есть примеры (48 блоков кода)
База качества 70; замечания lint вычитаются, сигналы прибавляют до 100. Итог: 69.