BF solo-review
Финальная проверка кода и контроль качества — запускайте тесты, проверяйте покрытие, аудируйте безопасность, проверяйте критерии приемки по спецификации и генерируйте отчет, готовый к выпуску. Используйте, когда пользователь говорит «проверить код», «проверка качества», «готово к выпуску», «финальная проверка» или после завершения /deploy. НЕ используйте для планирования (используйте /plan) или сборки (используйте /build).
машинный переводПоказать оригиналСкрыть оригинал«Final code review and quality gate — run tests, check coverage, audit…»
Final code review and quality gate — run tests, check coverage, audit security, verify acceptance criteria from spec, and generate ship-ready report. Use when user says "review code", "quality check", "is it ready to ship", "final review", or after /deploy completes. Do NOT use for planning (use /plan) or building (use /build).
Финальная проверка кода и контроль качества — запускайте тесты, проверяйте покрытие, аудируйте безопасность, проверяйте критерии приемки по спецификации и…
Как процесс F 61/100 · Не запустится — Скилл ссылается на файлы, которых нет в архиве: templates/stacks/{stack}.yaml, templates/principles/dev-principles.md
Чем это грозит
Находки средней серьёзности: скорее всего скилл честный, но прочитайте, что именно насторожило сканер.
Ниже описан худший случай для этой категории. Здесь находка средней серьёзности: guard увидел признак, но не доказательство.
Скилл просит больше прав, чем нужно для задачи: широкий доступ к инструментам, секретные переменные окружения, бинарные файлы. Каждое лишнее право расширяет ущерб при ошибке или взломе.
Сузьте allowed-tools и список переменных до минимума, замените бинарники на исходники или скрипты, которые можно прочитать.
Как улучшить
- Тело SKILL.md длиннее 5 000 токенов: вынесите справочные детали в references/ и подключайте по необходимости.
- В тексте есть ссылки на отсутствующие файлы: добавьте файлы или уберите ссылки.
- Свои кейсы (evals/evals.json, 4–6 реальных запросов с ожидаемыми ответами): тогда полная проверка прогонит именно их, а не черновик от модели.
- spec.yaml с триггерными фразами и утверждениями — контракт поведения для CI; `skilltest init` создаст шаблон.
Находки guard · 1
✓ Критических и высоких находок нет
Средние и низкие: 1
-
средняя Широкие права
meta-broad-allowed-toolsSKILL.md:1Заранее разрешены широкие инструменты: Bashallowed-tools: Read Grep Bash Glob Write Edit mcp__solograph__session_search mcp__solograph__project_code_search mcp__solograph__codegraph_query mcp__solograph__codegraph_explain mcp__solograph__web_s
Просканировано файлов: 2. Улики замаскированы. Пометки в серых чипах объясняют, почему серьёзность понижена.
По спецификации Agent Skills
- предупреждение
body-longтело SKILL.md ≈ 5454 токенов (рекомендуется < 5000); вынесите детали в references/ - предупреждение
missing-refссылка на отсутствующий файл: templates/stacks/{stack}.yaml - предупреждение
missing-refссылка на отсутствующий файл: templates/principles/dev-principles.md
Процессный рейтинг: все десять параметров 61/100
- 0Инструменты и файлы. Не хватает 2 файла(ов): templates/stacks/{stack}.yaml, templates/principles/dev-principles.md
- 0Входы и предусловия. Не сказано, что нужно иметь на входе
- 50Когда включается. Не сказано, при каком запросе скилл включается
- 70Стоимость исполнения. Тело инструкции 5454 токенов
- 85Шаги. Шагов: 162, расплывчатых формулировок: 1
- 100Результат и критерий готовности. Формат результата и критерий готовности описаны
- 100Ошибки и развилки. Развилок: 1, есть раздел про ошибки
- 100Согласованность. Имя и обязательные поля на месте
- 100Повторный запуск. Изменяющие операции проверяют текущее состояние
- 100Отчётность по ходу. Скилл сообщает о ходе работы
- medium Правила безопасности и запреты внутри скилла: их место в системном промпте, здесь они не защищают
- low Разделов верхнего уровня: 12. Похоже на несколько доменов в одном скилле
- low Ответ описан самодельной разметкой (3 тегов): типизированный вызов надёжнее
Всё перечисленное измерено по тексту скилла, а не оценено моделью: цифры проверяемы. Вес параметра тем больше, чем чаще из-за него процесс встаёт.
Сигналы качества
- -5В тексте остались TODO / заглушки
- +2Инструкции на одном языке
- +5В description 4 примера фраз-триггеров в кавычках
- +4Описание говорит, когда скилл НЕ применять
- +3Длина description 329 символов: достаточно сигнала, не съедает бюджет
- +4Структура: 35 заголовков
- +3Пошаговые инструкции: 162 пунктов
- +3Формат ответа описан явно
- +4Есть примеры (21 блоков кода)
- +1Лицензия указана
База качества 70; замечания lint вычитаются, сигналы прибавляют до 100. Итог: 73.