SKILLEMALL.ai

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).

LeoYeAI/openclaw-master-skills Claude Code автор: LeoYeAI MIT 2 файла тело ≈ 5 454 токенов Открыть источникgithub.com проанализирован 28 ч назад

Финальная проверка кода и контроль качества — запускайте тесты, проверяйте покрытие, аудируйте безопасность, проверяйте критерии приемки по спецификации и…

Как процесс F 61/100 · Не запустится — Скилл ссылается на файлы, которых нет в архиве: templates/stacks/{stack}.yaml, templates/principles/dev-principles.md

АнализаторРазработкаИнфраструктуратип и темы размечены автоматически по тексту скилла
JSON
Технический рейтинг
B
86/100
безопасность, качество, тесты
Безопасность 60%
95
Качество 40%
73
Прогон на моделях
не было
Процессный рейтинг
F
61/100
Не запустится
Скилл ссылается на файлы, которых нет в архиве: templates/stacks/{stack}.yaml, templates/principles/dev-principles.md
Инструменты и файлы вес 18
0
Входы и предусловия вес 11
0
Когда включается вес 12
50
три самых слабых из десяти параметров · все десять

Чем это грозит

Находки средней серьёзности: скорее всего скилл честный, но прочитайте, что именно насторожило сканер.

Широкие права средняя серьёзность

Ниже описан худший случай для этой категории. Здесь находка средней серьёзности: guard увидел признак, но не доказательство.

Если установить

Скилл просит больше прав, чем нужно для задачи: широкий доступ к инструментам, секретные переменные окружения, бинарные файлы. Каждое лишнее право расширяет ущерб при ошибке или взломе.

Автору

Сузьте allowed-tools и список переменных до минимума, замените бинарники на исходники или скрипты, которые можно прочитать.

Как улучшить

  1. Тело SKILL.md длиннее 5 000 токенов: вынесите справочные детали в references/ и подключайте по необходимости.
  2. В тексте есть ссылки на отсутствующие файлы: добавьте файлы или уберите ссылки.
Для прогона на моделях — необязательно
  • Свои кейсы (evals/evals.json, 4–6 реальных запросов с ожидаемыми ответами): тогда полная проверка прогонит именно их, а не черновик от модели.
  • spec.yaml с триггерными фразами и утверждениями — контракт поведения для CI; `skilltest init` создаст шаблон.

Находки guard · 1

✓ Критических и высоких находок нет

Средние и низкие: 1
  • средняя Широкие права meta-broad-allowed-tools SKILL.md:1
    Заранее разрешены широкие инструменты: Bash
    allowed-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

Не запустится. Скилл ссылается на файлы, которых нет в архиве: templates/stacks/{stack}.yaml, templates/principles/dev-principles.md
  • 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.