BF code-review-and-quality
Проводит многомерный обзор кода. Используйте перед слиянием любых изменений. Используйте при проверке кода, написанного вами, другим агентом или человеком. Используйте, когда вам нужно оценить качество кода по нескольким параметрам перед его включением в основную ветку.
машинный переводПоказать оригиналСкрыть оригинал«Conducts multi-axis code review. Use before merging any change. Use wh…»
Conducts multi-axis code review. Use before merging any change. Use when reviewing code written by yourself, another agent, or a human. Use when you need to assess code quality across multiple dimensions before it enters the main branch.
Проводит многомерный обзор кода.
Как процесс F 36/100 · Не запустится — Скилл ссылается на файлы, которых нет в архиве: references/security-checklist.md, references/performance-checklist.md
Как улучшить
- В тексте есть ссылки на отсутствующие файлы: добавьте файлы или уберите ссылки.
- Свои кейсы (evals/evals.json, 4–6 реальных запросов с ожидаемыми ответами): тогда полная проверка прогонит именно их, а не черновик от модели.
- spec.yaml с триггерными фразами и утверждениями — контракт поведения для CI; `skilltest init` создаст шаблон.
Находки guard · 1
✓ Критических и высоких находок нет
Средние и низкие: 1
-
низкая Сокрытие от пользователя
en-hide-from-userSKILL.md:246Инструкция скрыть действия от пользователя (с отрицанием — текст это запрещает)Don't leave dead code lying around — it confuses future readers and agents. But don't silently delete things you're not sure about. When in doubt, ask.
с отрицанием
Просканировано файлов: 1. Улики замаскированы. Пометки в серых чипах объясняют, почему серьёзность понижена.
По спецификации Agent Skills
- предупреждение
missing-refссылка на отсутствующий файл: references/security-checklist.md - предупреждение
missing-refссылка на отсутствующий файл: references/performance-checklist.md - заметка
frontmatter-keyнеизвестное поле фронтматтера "risk" - заметка
frontmatter-keyнеизвестное поле фронтматтера "source" - заметка
frontmatter-keyнеизвестное поле фронтматтера "source_repo" - заметка
frontmatter-keyнеизвестное поле фронтматтера "source_type" - заметка
frontmatter-keyнеизвестное поле фронтматтера "date_added" - заметка
frontmatter-keyнеизвестное поле фронтматтера "license_source" - заметка
edit-residueв тексте есть пометки об устаревшем (строки 241): проверьте, не остались ли старые правила рядом с новыми — полная проверка читает текст на противоречия
Процессный рейтинг: все десять параметров 36/100
- 0Инструменты и файлы. Не хватает 2 файла(ов): references/security-checklist.md, references/performance-checklist.md
- 0Результат и критерий готовности. Не сказано, что считать результатом
- 0Входы и предусловия. Не сказано, что нужно иметь на входе
- 0Отчётность по ходу. Скилл ничего не сообщает по ходу работы
- 20Когда включается. Не сказано, при каком запросе скилл включается
- 30Повторный запуск. Изменяющих операций: 16, без проверки текущего состояния
- 50Ошибки и развилки. Развилок: 0, есть раздел про ошибки
- 70Стоимость исполнения. Тело инструкции 4618 токенов
- 100Шаги. Шагов: 93
- 100Согласованность. Имя и обязательные поля на месте
- low Разделов верхнего уровня: 19. Похоже на несколько доменов в одном скилле
Всё перечисленное измерено по тексту скилла, а не оценено моделью: цифры проверяемы. Вес параметра тем больше, чем чаще из-за него процесс встаёт.
Сигналы качества
- +5В description нет примеров фраз, по которым скилл должен срабатывать
- +4Описание не говорит, когда скилл НЕ применять (ложные срабатывания)
- +3Формат ответа не описан: модель каждый раз решает сама
- +2Инструкции на одном языке
- +3Длина description 237 символов: достаточно сигнала, не съедает бюджет
- +4Структура: 30 заголовков
- +3Пошаговые инструкции: 93 пунктов
- +4Есть примеры (9 блоков кода)
- +1Лицензия указана
База качества 70; замечания lint вычитаются, сигналы прибавляют до 100. Итог: 73.