القائمة
Независимый аудит результата любого скилла по его чек-листу. Используй после завершения скилла для проверки качества output. Не используй для аудита самих SKILL.md — для этого /skill-audit.
التثبيت باستخدام Codex أو Claude انسخ هذا Prompt والصقه في Codex أو Claude أو مساعد آخر ليراجع صفحة Skill ويثبّتها لك.
استنادا إلى تصنيف SOC المهني
Generate Production-Ready API tests in Kotlin (JUnit5, Allure) from specifications.
Independent Quality Gatekeeper — represents end user perspective, last line of defense before merge.
Audit documentation quality — size, structure, duplicates, SSOT violations.
Generate qa_agent.md — AI job description with QA culture, principles and anti-patterns.
Initialize CLAUDE.md for a new QA project with stack, protocols and conventions.
Generate new skills with interactive workflow, checkpoints and iterative refinement.
| name | output-review |
| description | Независимый аудит результата любого скилла по его чек-листу. Используй после завершения скилла для проверки качества output. Не используй для аудита самих SKILL.md — для этого /skill-audit. |
| allowed-tools | Read Write Edit Glob Grep Bash(./gradlew*) Bash(wc*) |
| agent | agents/auditor.md |
| context | fork |
Проверяет OUTPUT скилла по чек-листам из SKILL.md целевого скилла. Независимая оценка — не тот же AI-контекст, который генерировал результат.
Прочитай .claude/qa_agent.md и .claude/agents/auditor.md.
SKILL COMPLETE)| Параметр | Обязательность | Описание |
|---|---|---|
| skill-name | Опционально | Имя скилла (/output-review api-tests). Если не указано — ищет SKILL COMPLETE в чате |
Structured Output Priority: Весь analysis идёт в артефакт (MD/HTML), не в чат.
Chat output (ограничения):
📊 Полный отчёт: {path} + открыть файлIterative steps: Не выводить прогресс по каждому файлу. Checkpoint только при:
Tools first:
Post-Check: Inline перед SKILL COMPLETE (5-7 строк checklist), не отдельный файл.
Фазы 1-6: Silent. Фаза 7: Сохранить полный отчёт в файл + краткая сводка в чат (max 5 строк).
Цель: Определить какой скилл аудитировать.
/output-review {skill-name} → использовать егоSKILL COMPLETE: /{skill-name} в контексте чата.claude/skills/{skill-name}/SKILL.md — файл должен существоватьЕсли скилл не найден → STOP:
❌ Скилл /{skill-name} не найден в .claude/skills/
Доступные: [список из Glob]
Цель: Извлечь все проверки из SKILL.md целевого скилла.
.claude/skills/{skill-name}/SKILL.md## / ###):
Self-Check / Definition of DoneQuality GatesPost-*Check (Post-Check, Post-Compilation Check, Post-Audit Check)BANNED / ЗАПРЕЩЕНОCompilation Gatereferences/ → Glob references/*.md → читать, искать чек-листы- [ ] и каждый нумерованный пункт из BANNEDВывод перед продолжением: таблица | # | Группа | Пункт | + строка Всего: N проверок.
Цель: Найти артефакты (файлы), созданные скиллом.
Приоритет поиска:
Артефакты: → парсить пути файловДля каждого найденного файла — проверить существование через Glob.
Если артефакты не найдены:
⚠️ Артефакты не обнаружены. Оценка будет проведена по контексту чата.
Цель: Независимо проверить каждый пункт чек-листа.
Для каждого пункта из Фазы 2:
| Вердикт | Значение |
|---|---|
| ✅ PASS | Полностью соответствует |
| ❌ FAIL | Не соответствует — с доказательством |
| ⚠️ PARTIAL | Частично соответствует — описать что не так |
| ⏭️ SKIP | Неприменимо к данному контексту |
Правила:
Thread.sleep, shouldBe, Map<String, Any>)Пример FAIL: ❌ FAIL: assertEquals без message — файл:строка — Найдено: X / Ожидалось: Y
Цель: Проверить артефакты на анти-паттерны из agents/sdet.md.
ls .claude/qa-antipatterns/ — получить список anti-patternsThread.sleep, Map<String, Any>, PII)Если скилл отсутствует в таблице Anti-Patterns → пропустить фазу.
| Вердикт | Значение |
|---|---|
| ✅ CLEAN | Сигнатура не найдена |
| ❌ FOUND | Сигнатура найдена — файл:строка |
Цель: Проверки, общие для всех скиллов.
АртефактыCompilationUpstreamCoverageЦель: Сформировать итоговый отчёт и сохранить в файл audit/output-review_{skill-name}_{YYYY-MM-DD}.md.
Если файл с таким именем уже существует — добавить суффикс _2, _3 и т.д.
## Output Review Report: /{skill-name}
### Self-Check / Definition of Done
| # | Пункт | Вердикт | Комментарий |
|---|-------|---------|-------------|
| 1 | Архитектура | ✅ PASS | config/, requests/, helpers/ (main) + tests (test) — присутствуют |
| 2 | assertions с message | ❌ FAIL | assertEquals без message в RegistrationApiTests.kt:45 |
### BANNED
| # | Правило | Вердикт | Комментарий |
|---|---------|---------|-------------|
| 1 | Thread.sleep() | ✅ PASS | Не найдено |
### Anti-Patterns (agents/sdet.md)
| # | Паттерн | Вердикт | Комментарий |
|---|---------|---------|-------------|
| 1 | PII в коде | ✅ CLEAN | @gmail.com не найден |
### Universal Checks
| # | Проверка | Вердикт | Комментарий |
|---|----------|---------|-------------|
| 1 | SKILL COMPLETE блок | ✅ PASS | Присутствует |
| 2 | 5 полей формата | ⚠️ PARTIAL | Отсутствует Coverage |
Score = PASS / (PASS + FAIL) × 100
Формула: {N_pass} + {N_partial}×0.5 / ({N_pass} + {N_partial}×0.5 + {N_fail}) × 100
SKIP не учитываются.
Пример:
Scorecard: 12 PASS + 1 PARTIAL×0.5 / (12 + 0.5 + 2) × 100 = 86%
Если скилл уже выполнил Post-Check в чате — сравнить:
Пункт X: скилл → ✅, self-review → ❌ (причина)Конкретные действия для устранения каждого FAIL в артефакте:
### Рекомендации по артефакту
1. **[FAIL]** assertEquals без message в RegistrationApiTests.kt:45
→ Добавить message: `assertEquals(200, response.code, "Registration should return 200")`
2. **[PARTIAL]** Coverage не указан в SKILL COMPLETE
→ Добавить строку `├─ Coverage: X/Y тестов`
Цель: На основе найденных FAIL и PARTIAL — предложить конкретные правки в .claude/skills/{skill-name}/SKILL.md, чтобы следующий запуск скилла не воспроизводил те же ошибки.
Логика: каждый FAIL/PARTIAL — симптом недостающего или нечёткого правила в скилле.
### Рекомендации по улучшению /{skill-name}
| # | Проблема в артефакте | Причина (правило отсутствует) | Рекомендуемое правило в SKILL.md |
|---|---------------------|-------------------------------|----------------------------------|
| 1 | Два HTTP-кода в одном Expected Result (`201 ИЛИ 400`) | Нет явного требования атомарности | Добавить в Protocol: "Каждый сценарий содержит ровно 1 Expected Result. Запрещено `X ИЛИ Y` в одной строке." |
| 2 | L10N сценарии отсутствуют | Нет явного Coverage Matrix для L10N | Добавить в Coverage Matrix: `L10N: {EMOJI_STRING}, {SPECIAL_CHARS}` как обязательные типы, если спека не запрещает явно. |
Правила генерации рекомендаций:
---
## Ограничения
- Полный отчёт сохранять в `audit/output-review_{skill-name}_{YYYY-MM-DD}.md`, краткая сводка (5 строк) — в чат
- Compilation Gate: максимум 1 попытка (не исправлять код, только зафиксировать результат)
- Не исправлять артефакты — только документировать findings
- Если артефакт слишком большой (>500 строк) — проверять по ключевым паттернам, не читать целиком
---
## Post-Review Check (inline перед SKILL COMPLETE)
- [ ] Все извлечённые пункты чек-листа проверены?
- [ ] Каждый FAIL содержит конкретное доказательство (файл:строка)?
- [ ] BANNED-пункты проверены через Grep, а не визуально?
- [ ] Scorecard рассчитан с показом формулы (числитель/знаменатель)?
- [ ] Нет ложных FAIL (контекст каждого finding перепроверен)?
- [ ] Полный отчёт сохранён в `audit/output-review_{skill-name}_{YYYY-MM-DD}.md`?
**Если нашёл ошибку в аудите → исправь.**
---
### Завершение
После Post-Review Check — напечатай блок `SKILL COMPLETE` (формат в qa_agent.md § Skill Completion Protocol).