CD pr-checklist
Подготовьте изменение vLLM для проверки человеком или повторной проверки, или проанализируйте открытый запрос на слияние. Используйте перед открытием PR, включая черновики (например, `gh pr create`), или отправкой исправлений для ревью, чтобы проверить соответствие дизайну, охват поведения, доказательства производительности, качество диффа и закрытие предыдущих замечаний.
машинный переводПоказать оригиналСкрыть оригинал«Prepare a vLLM change for human PR review or re-review, or analyze an…»
Prepare a vLLM change for human PR review or re-review, or analyze an open pull request. Use before opening a PR, drafts included (e.g. `gh pr create`), or pushing review fixes, to check design fit, behavioral coverage, performance evidence, diff quality, and closure of previous feedback.
Подготовьте изменение vLLM для проверки человеком или повторной проверки, или проанализируйте открытый запрос на слияние.
Как процесс D 42/100 · Процесс не доведён — Скилл ссылается на файлы, которых нет в архиве: ../../../docs/contributing/model/basic.md, ../../../docs/design/model_runner_v2.md, ../../../docs/design/model_runner_v2.md
Как улучшить
- В тексте есть ссылки на отсутствующие файлы: добавьте файлы или уберите ссылки.
- Свои кейсы (evals/evals.json, 4–6 реальных запросов с ожидаемыми ответами): тогда полная проверка прогонит именно их, а не черновик от модели.
- spec.yaml с триггерными фразами и утверждениями — контракт поведения для CI; `skilltest init` создаст шаблон.
Находки guard · 0
✓ Критических и высоких находок нет
Просканировано файлов: 1. Улики замаскированы. Пометки в серых чипах объясняют, почему серьёзность понижена.
По спецификации Agent Skills
- предупреждение
missing-refссылка на отсутствующий файл: ../../../docs/contributing/model/basic.md - предупреждение
missing-refссылка на отсутствующий файл: ../../../docs/design/model_runner_v2.md - предупреждение
missing-refссылка на отсутствующий файл: ../../../docs/design/model_runner_v2.md - предупреждение
missing-refссылка на отсутствующий файл: ../../../tests/AGENTS.md - предупреждение
missing-refссылка на отсутствующий файл: ../../../docs/contributing/README.md - предупреждение
missing-refссылка на отсутствующий файл: ../../../docs/features/disagg_prefill.md - предупреждение
missing-refссылка на отсутствующий файл: ../../../docs/design/nixl_kv_push_connector.md - предупреждение
missing-refссылка на отсутствующий файл: ../../../docs/design/nixl_kv_cache_lease.md - заметка
edit-residueв тексте есть пометки об устаревшем (строки 89): проверьте, не остались ли старые правила рядом с новыми — полная проверка читает текст на противоречия
Процессный рейтинг: все десять параметров 42/100
- 0Инструменты и файлы. Не хватает 8 файла(ов): ../../../docs/contributing/model/basic.md, ../../../docs/design/model_runner_v2.md, ../../../docs/design/model_runner_v2.md
- 0Входы и предусловия. Не сказано, что нужно иметь на входе
- 0Отчётность по ходу. Скилл ничего не сообщает по ходу работы
- 20Когда включается. Не сказано, при каком запросе скилл включается
- 40Результат и критерий готовности. Не сказано, что считать результатом
- 50Ошибки и развилки. Развилок: 0, есть раздел про ошибки
- 70Стоимость исполнения. Тело инструкции 4633 токенов
- 85Шаги. Шагов: 16, расплывчатых формулировок: 2
- 100Согласованность. Имя и обязательные поля на месте
- 100Повторный запуск. Изменяющие операции проверяют текущее состояние
Всё перечисленное измерено по тексту скилла, а не оценено моделью: цифры проверяемы. Вес параметра тем больше, чем чаще из-за него процесс встаёт.
Сигналы качества
- +5В description нет примеров фраз, по которым скилл должен срабатывать
- +4Описание не говорит, когда скилл НЕ применять (ложные срабатывания)
- +3Формат ответа не описан: модель каждый раз решает сама
- +1Лицензия не указана
- +2Инструкции на одном языке
- +3Длина description 289 символов: достаточно сигнала, не съедает бюджет
- +4Структура: 26 заголовков
- +3Пошаговые инструкции: 16 пунктов
- +4Есть примеры (1 блоков кода)
База качества 70; замечания lint вычитаются, сигналы прибавляют до 100. Итог: 72.