CC yotta-code-quality
Парный ревьюер качества кода: двенадцать рисков угасания, основанных на книгах (R1–R6, T1–T6), плюс проверки безопасности релиза и UX первого отображения. Результаты всегда используют Железный закон (Симптом → Источник → Следствие → Средство) и оценку здоровья (review-index) от 0 до 100. Триггеры, когда: пользователь просит проверить код/PR/diff, «any issues», «ready to merge», запахи, рефакторинг, технический долг, качество тестов, покрытие или состояние архитектуры; или говорит «结对评审»/«发版前扫一眼»/ yotta-code-quality. НЕ триггерить для: нового проекта «how do I write X» без кода, чистых вопросов по синтаксису или вопросов по инструментам/фреймворкам без общего кода.
машинный переводПоказать оригиналСкрыть оригинал«Pair-style code quality reviewer: twelve book-grounded decay risks (R1…»
Pair-style code quality reviewer: twelve book-grounded decay risks (R1–R6, T1–T6) plus release-safety and first-paint UX checks. Findings always use Iron Law (Symptom → Source → Consequence → Remedy) and a 0–100 review-index Health Score. Triggers when: user asks to review code/PR/diff, "any issues", "ready to merge", smells, refactoring, tech debt, test quality, coverage, or architecture health; or says 「结对评审」/「发版前扫一眼」/ yotta-code-quality. Do NOT trigger for: greenfield "how do I write X" with no code, pure syntax questions, or tool/framework questions with no shared code.
Парный ревьюер качества кода: двенадцать рисков угасания, основанных на книгах (R1–R6, T1–T6), плюс проверки безопасности релиза и UX первого отображения.
Как процесс C 60/100 · Есть пробелы — слабые места: результат и критерий готовности, входы и предусловия, повторный запуск
Чем это грозит
Скилл содержит фрагменты, которые в чужих руках стоят денег или данных. Ниже, что рискует потерять тот, кто установит, и что должен сделать автор.
Скилл содержит команды, которые удаляют файлы, переписывают диски или исполняют код, скачанный из сети. Агент может выполнить их без вопроса, если считает, что так требует инструкция.
Замените разрушительные команды на безопасные аналоги с подтверждением, ограничьте область действия конкретной папкой и не тяните код через curl | bash: укажите версию и контрольную сумму.
Как улучшить
- Разберите находки высокой серьёзности: каждая стоит 18 баллов безопасности. Если это ложное срабатывание, добавьте код правила в guard.allow в spec.yaml.
- Свои кейсы (evals/evals.json, 4–6 реальных запросов с ожидаемыми ответами): тогда полная проверка прогонит именно их, а не черновик от модели.
- spec.yaml с триггерными фразами и утверждениями — контракт поведения для CI; `skilltest init` создаст шаблон.
Находки guard · 3
-
высокая Опасные команды
cmd-destructive-fsREADME.md:138Разрушительная команда для файловой системы (корень / домашняя папка / диск) (комментарий в коде)│ └── hooks.json # optional: PreToolUse hooks for rm -rf / git push --force
комментарий -
высокая Опасные команды
cmd-destructive-fsREADME.zh-CN.md:151Разрушительная команда для файловой системы (корень / домашняя папка / диск) (комментарий в коде)│ └── hooks.json # 可选：PreToolUse 拦截 rm -rf / git push --force
комментарий
Средние и низкие: 1
-
средняя Опасные команды
cmd-pipe-to-shellreferences/editorial-extensions.md:16Скачивание и запуск удалённого кода с неизвестного хоста (pipe в shell) (определение детектора / чёрного списка)- `curl | sh` or equivalent in docs/scripts aimed at end users.
детектор
Просканировано файлов: 20. Улики замаскированы. Пометки в серых чипах объясняют, почему серьёзность понижена.
По спецификации Agent Skills
✓ По спецификации Agent Skills замечаний нет
Процессный рейтинг: все десять параметров 60/100
- 0Результат и критерий готовности. Не сказано, что считать результатом
- 0Входы и предусловия. Не сказано, что нужно иметь на входе
- 0Отчётность по ходу. Скилл ничего не сообщает по ходу работы
- 30Повторный запуск. Изменяющих операций: 1, без проверки текущего состояния
- 60Ошибки и развилки. Развилок: 2
- 70Когда включается. Сказано, когда применять, но не сказано, когда не стоит
- 85Шаги. Шагов: 18, расплывчатых формулировок: 3
- 100Инструменты и файлы. Внешние инструменты не нужны
- 100Согласованность. Имя и обязательные поля на месте
- 100Стоимость исполнения. Тело инструкции 1484 токенов
- medium Правила безопасности и запреты внутри скилла: их место в системном промпте, здесь они не защищают
Всё перечисленное измерено по тексту скилла, а не оценено моделью: цифры проверяемы. Вес параметра тем больше, чем чаще из-за него процесс встаёт.
Сигналы качества
- +4Описание не говорит, когда скилл НЕ применять (ложные срабатывания)
- +3Формат ответа не описан: модель каждый раз решает сама
- +2Инструкции на одном языке
- +5В description 3 примера фраз-триггеров в кавычках
- +3Длина description 580 символов: достаточно сигнала, не съедает бюджет
- +4Структура: 13 заголовков
- +3Пошаговые инструкции: 18 пунктов
- +4Есть примеры (1 блоков кода)
- +4Справочные файлы упоминаются в инструкциях (10 из 10)
- +1Лицензия указана
База качества 70; замечания lint вычитаются, сигналы прибавляют до 100. Итог: 94.