CC thunder-deep-review
Проверяет pull request, diff или ветку на соответствие внутренним правилам Thunderbolt (без any, type-over-interface, стрелочных функций, const/раннего возврата, без скрытия ошибок или защитного кода, useReducer вместо 3+ useState, дисциплина useEffect, разделение кода маршрута, мягкие удаления, независимая от интеграции логика) и 80 архитектурным инвариантам. Выходит за рамки поиска ошибок, выявляя ошибки архитектуры/уровня абстракции, недокументированное намерение, отклонение от конвенций, раздувание/цепочку поставок и читаемость. Используйте при проверке PR/diff/ветки, когда пользователь говорит review, code-review, deep-review или "review before merge". Только чтение кода: сообщает о находках, никогда не редактирует код и не публикует в PR.
машинный переводПоказать оригиналСкрыть оригинал«Reviews a pull request, diff, or branch against Thunderbolt's house ru…»
Reviews a pull request, diff, or branch against Thunderbolt's house rules (no any, type-over-interface, arrow fns, const/early-return, no error-swallowing or defensive code, useReducer at 3+ useState, useEffect discipline, route code-splitting, soft-deletes, integration-agnostic logic) and the 80 architecture invariants. Goes beyond bug-finding to catch architecture/abstraction-layer mistakes, undocumented intent, convention drift, bloat/supply-chain, and readability. Use when reviewing a PR/diff/branch, when the user says review, code-review, deep-review, or "review before merge". Read-only on the code: reports findings, never edits code or posts to the PR.
Проверяет pull request, diff или ветку на соответствие внутренним правилам Thunderbolt (без any, type-over-interface, стрелочных функций, const/раннего…
Как процесс C 64/100 · Есть пробелы — слабые места: ошибки и развилки, повторный запуск, отчётность по ходу
Как улучшить
- Тело SKILL.md длиннее 5 000 токенов: вынесите справочные детали в references/ и подключайте по необходимости.
- Свои кейсы (evals/evals.json, 4–6 реальных запросов с ожидаемыми ответами): тогда полная проверка прогонит именно их, а не черновик от модели.
- spec.yaml с триггерными фразами и утверждениями — контракт поведения для CI; `skilltest init` создаст шаблон.
Находки guard · 0
✓ Критических и высоких находок нет
Просканировано файлов: 8. Улики замаскированы. Пометки в серых чипах объясняют, почему серьёзность понижена.
По спецификации Agent Skills
- предупреждение
frontmatter-yamlSKILL.md: фронтматтер не разбирается как YAML (YAML parse error: Nested mappings are not allowed in compact mappings at line 2, column 14: description: Reviews a pull request, diff, or branch against Thunderbolt's hous… ^ ); поля прочитаны построчно. Обычная причина — двоеточие в незакавыченном значении - предупреждение
body-longтело SKILL.md ≈ 7815 токенов (рекомендуется < 5000); вынесите детали в references/
Процессный рейтинг: все десять параметров 64/100
- 0Ошибки и развилки. Линейный процесс без обработки сбоев
- 0Отчётность по ходу. Скилл ничего не сообщает по ходу работы
- 30Повторный запуск. Изменяющих операций: 32, без проверки текущего состояния
- 60Инструменты и файлы. Используются инструменты (read, web), но во frontmatter они не объявлены
- 60Результат и критерий готовности. Формат результата описан, критерия завершения нет
- 70Когда включается. Сказано, когда применять, но не сказано, когда не стоит
- 70Входы и предусловия. Входные данные и предусловия перечислены
- 70Стоимость исполнения. Тело инструкции 7815 токенов
- 100Шаги. Шагов: 65
- 100Согласованность. Имя и обязательные поля на месте
- medium Правила безопасности и запреты внутри скилла: их место в системном промпте, здесь они не защищают
Всё перечисленное измерено по тексту скилла, а не оценено моделью: цифры проверяемы. Вес параметра тем больше, чем чаще из-за него процесс встаёт.
Сигналы качества
- +5В description нет примеров фраз, по которым скилл должен срабатывать
- +4Описание не говорит, когда скилл НЕ применять (ложные срабатывания)
- +1Лицензия не указана
- +2Инструкции на одном языке
- +3Длина description 666 символов: достаточно сигнала, не съедает бюджет
- +4Структура: 21 заголовков
- +3Пошаговые инструкции: 65 пунктов
- +3Формат ответа описан явно
- +4Есть примеры (1 блоков кода)
- +4Справочные файлы упоминаются в инструкциях (6 из 6)
База качества 70; замечания lint вычитаются, сигналы прибавляют до 100. Итог: 71.