SKILLEMALL.ai
GitHub ↗ EN Войти

AC github-review-pr

Проверяет или повторно проверяет один PR от контрибьютора — включая явно названный закрытый PR, который рассматривается повторно — или ограниченный набор всех открытых PR от контрибьюторов от новейшего к старейшему, для мейнтейнера репозитория GitHub против текущей базовой ветки. Обрабатывает смещение базы, разрывы истории, загрязненные ветки, владение, курирование, замену и ремонт или слияние, обусловленное проверкой, с использованием неизменяемых снимков Git, результатов трехстороннего слияния, изолированной проекции вклада, проверок, тестов и отчетности, ориентированной на результаты. Использовать для URL или номера PR, «main изменился, проверьте снова», «проверьте все открытые PR от новейшего к старейшему», «примените наши принципы мейнтейнера», «можем ли мы это слить и исправить остальное сами?», или готовность к слиянию. Не использовать для общего CRUD GitHub, аудита всего репозитория, диагностики только CI, аудита различий только безопасности, непротолкнутых локальных различий, простого ответа на существующие комментарии к проверке или слияния без свежей проверки.

машинный переводПоказать оригиналСкрыть оригинал«Reviews or re-reviews one contributor pull request—including an explic…»

Reviews or re-reviews one contributor pull request—including an explicitly named closed PR being reconsidered—or a bounded newest-to-oldest sweep of all open contributor PRs, for a GitHub repository maintainer against the current base branch. Handles base drift, history discontinuities, polluted branches, ownership, curation, supersession, and review-conditioned repair or landing using immutable Git snapshots, three-way merge results, isolated contribution projection, checks, tests, and findings-first reporting. Use for a PR URL or number, "main changed, review again", "review all open PRs newest to oldest", "apply our maintainer principles", "can we merge this and fix the rest ourselves?", or merge readiness. Do not use for general GitHub CRUD, repository-wide audits, CI-only diagnosis, security-only diff audits, unpushed local diffs, merely addressing existing review comments, or merging without a fresh review.

daymade/claude-code-skills Claude Code автор: daymade 4 файла тело ≈ 6 565 токенов Открыть источникgithub.com↗ проанализирован 2 ч назад

Проверяет или повторно проверяет один PR от контрибьютора — включая явно названный закрытый PR, который рассматривается повторно — или ограниченный набор всех…

Как процесс C 59/100 · Есть пробелы — слабые места: результат и критерий готовности, входы и предусловия

АнализаторGitHubРазработкатип и темы размечены автоматически по тексту скилла
Запускается в: Claude / Claude Code ChatGPT / Codex Gemini CLI Cursor GitHub Copilot / VS Code Qwen Code Kimi Code CLI Z.ai (GLM) DeepSeek (Deep Code) Mistral Vibe LangChain (deepagents) OpenClaw Goose OpenCode
JSON
Технический рейтинг
A
94/100
безопасность, качество, тесты
Безопасность 60%
100
Качество 40%
84
Прогон на моделях
не было
Процессный рейтинг
C
59/100
Есть пробелы
Результат и критерий готовности вес 14
0
Входы и предусловия вес 11
0
Ошибки и развилки вес 10
50
три самых слабых из десяти параметров · все десять

Как улучшить

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

Находки guard · 0

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

Просканировано файлов: 3. Улики замаскированы. Пометки в серых чипах объясняют, почему серьёзность понижена.

По спецификации Agent Skills

  • предупреждение body-long тело SKILL.md ≈ 6565 токенов (рекомендуется < 5000); вынесите детали в references/
  • заметка edit-residue в тексте есть пометки об устаревшем (строки 374, 421): проверьте, не остались ли старые правила рядом с новыми — полная проверка читает текст на противоречия

Процессный рейтинг: все десять параметров 59/100

  • 0Результат и критерий готовности. Не сказано, что считать результатом
  • 0Входы и предусловия. Не сказано, что нужно иметь на входе
  • 50Ошибки и развилки. Развилок: 0, есть раздел про ошибки
  • 60Инструменты и файлы. Используются инструменты (web), но во frontmatter они не объявлены
  • 70Стоимость исполнения. Тело инструкции 6565 токенов
  • 85Шаги. Шагов: 50, расплывчатых формулировок: 1
  • 100Когда включается. Сказано, когда применять и когда не применять
  • 100Согласованность. Имя и обязательные поля на месте
  • 100Повторный запуск. Изменяющие операции проверяют текущее состояние
  • 100Отчётность по ходу. Скилл сообщает о ходе работы
  • medium Правила безопасности и запреты внутри скилла: их место в системном промпте, здесь они не защищают
  • medium Тестов 1, и все положительные: нет ни одного случая «должен отказать» или «должен переспросить»

Всё перечисленное измерено по тексту скилла, а не оценено моделью: цифры проверяемы. Вес параметра тем больше, чем чаще из-за него процесс встаёт.

Сигналы качества

  • +3Длина description 926: рекомендуется 120–800 символов
  • +3Формат ответа не описан: модель каждый раз решает сама
  • +1Лицензия не указана
  • +2Инструкции на одном языке
  • +5В description 4 примера фраз-триггеров в кавычках
  • +4Описание говорит, когда скилл НЕ применять
  • +4Структура: 18 заголовков
  • +3Пошаговые инструкции: 50 пунктов
  • +4Есть примеры (15 блоков кода)
  • +4Справочные файлы упоминаются в инструкциях (2 из 2)

База качества 70; замечания lint вычитаются, сигналы прибавляют до 100. Итог: 84.