AF named-persona-adversarial-review
Обзор кода через призму задокументированных философий реальных инженеров (Торвальдс, Томпсон, Кармак, Кент Бек, Джобс, Каган). Дополняет абстрактный ролевой антагонистический обзор именованными, обоснованными точками зрения. Используйте, когда результаты автоматического обзора кажутся общими, когда PR имеет архитектурное или UX-влияние, или когда автор хочет усилить код перед отправкой сверх стандартных проверок.
машинный переводПоказать оригиналСкрыть оригинал«Code review through the lens of real engineers' documented philosophie…»
Code review through the lens of real engineers' documented philosophies (Torvalds, Thompson, Carmack, Kent Beck, Jobs, Cagan). Complements abstract-role adversarial review with named, sourced perspectives. Use when automated review findings feel generic, when a PR has architectural or UX impact, or when the author wants pre-submit hardening beyond standard checks.
Обзор кода через призму задокументированных философий реальных инженеров (Торвальдс, Томпсон, Кармак, Кент Бек, Джобс, Каган).
Как процесс F 33/100 · Не запустится — Скилл ссылается на файлы, которых нет в архиве: ../adversarial-reviewer/SKILL.md, ../code-reviewer/SKILL.md, ../senior-security/SKILL.md
Как улучшить
- В тексте есть ссылки на отсутствующие файлы: добавьте файлы или уберите ссылки.
- Свои кейсы (evals/evals.json, 4–6 реальных запросов с ожидаемыми ответами): тогда полная проверка прогонит именно их, а не черновик от модели.
- spec.yaml с триггерными фразами и утверждениями — контракт поведения для CI; `skilltest init` создаст шаблон.
Находки guard · 0
✓ Критических и высоких находок нет
Просканировано файлов: 2. Улики замаскированы. Пометки в серых чипах объясняют, почему серьёзность понижена.
По спецификации Agent Skills
- предупреждение
missing-refссылка на отсутствующий файл: ../adversarial-reviewer/SKILL.md - предупреждение
missing-refссылка на отсутствующий файл: ../code-reviewer/SKILL.md - предупреждение
missing-refссылка на отсутствующий файл: ../senior-security/SKILL.md - предупреждение
missing-refссылка на отсутствующий файл: ../../../productivity/andreessen/skills/andreessen/SKILL.md
Процессный рейтинг: все десять параметров 33/100
- 0Инструменты и файлы. Не хватает 4 файла(ов): ../adversarial-reviewer/SKILL.md, ../code-reviewer/SKILL.md, ../senior-security/SKILL.md
- 0Результат и критерий готовности. Не сказано, что считать результатом
- 0Входы и предусловия. Не сказано, что нужно иметь на входе
- 0Ошибки и развилки. Линейный процесс без обработки сбоев
- 0Отчётность по ходу. Скилл ничего не сообщает по ходу работы
- 20Когда включается. Не сказано, при каком запросе скилл включается
- 30Повторный запуск. Изменяющих операций: 3, без проверки текущего состояния
- 100Шаги. Шагов: 39
- 100Согласованность. Имя и обязательные поля на месте
- 100Стоимость исполнения. Тело инструкции 2609 токенов
- medium Правила безопасности и запреты внутри скилла: их место в системном промпте, здесь они не защищают
- low Разделов верхнего уровня: 13. Похоже на несколько доменов в одном скилле
Всё перечисленное измерено по тексту скилла, а не оценено моделью: цифры проверяемы. Вес параметра тем больше, чем чаще из-за него процесс встаёт.
Сигналы качества
- +5В description нет примеров фраз, по которым скилл должен срабатывать
- +4Описание не говорит, когда скилл НЕ применять (ложные срабатывания)
- +3Формат ответа не описан: модель каждый раз решает сама
- +1Лицензия не указана
- +2Инструкции на одном языке
- +3Длина description 366 символов: достаточно сигнала, не съедает бюджет
- +4Структура: 18 заголовков
- +3Пошаговые инструкции: 39 пунктов
- +4Есть примеры (1 блоков кода)
- +4Справочные файлы упоминаются в инструкциях (1 из 1)
База качества 70; замечания lint вычитаются, сигналы прибавляют до 100. Итог: 76.