Когда это нужноWhen to reach for itКоли це потрібно
Проход перед слиянием или любой случай, когда нужно, чтобы кто-то посмотрел вашу правку. Для одной короткой строки на замечание есть /caveman-review, а /critique - более полный проход.A pre-merge pass, or whenever you want someone to look over your change. For one terse line per finding use /caveman-review; /critique is the fuller pass.Прохід перед злиттям або будь-який випадок, коли потрібно, щоб хтось подивився вашу правку. Для одного короткого рядка на зауваження є /caveman-review, а /critique - повніший прохід.
ТребованияRequirementsВимоги
- Где доступноAvailable inДе доступно
- Каждая загрузка kitEvery download of the kitКожне завантаження kit
- К чему обращаетсяWhat it touchesДо чого звертається
- Читает файлы проектаReads the files of your projectЧитає файли проєкту
- Работает сWorks withПрацює з
- Сделано для Claude Code; другие агенты адаптируютBuilt for Claude Code; other agents adapt itЗроблено для Claude Code; інші агенти адаптують
- Где выполняются шагиWhere the steps happenДе виконуються кроки
- Чат вашего агента и ваши собственные файлы. Ни то ни другое не интерфейс этого сайта, поэтому шаги описаны текстом, а не скриншотами.Your agent's chat and your own files. Neither is an interface of this site, so the steps are text and not screenshots.Чат вашого агента і ваші власні файли. Ні те, ні інше не є інтерфейсом цього сайту, тому кроки описано текстом, а не скриншотами.
- Где это лежит в kitWhere it lives in the kitДе це лежить у kit
.claude/skills/critique/SKILL.md
Прежде чем начатьBefore you startПерш ніж почати
Ложное замечание стоит столько же, сколько пропуск, и звучит громче: после нескольких таких настоящий дефект оказывается одной строкой среди ложных, и ревью перестают читать. Поэтому агент сначала подтверждает или опровергает собственные замечания высокой серьёзности, а если до оцениваемого не добраться, это замечание «could not verify», а не дефект.A false finding costs as much as a miss and is louder: after a handful of them, a real defect is one row among false ones and the review stops being read. So the agent confirms or refutes its own high-severity findings first, and when it cannot reach the thing it is judging, the finding is "could not verify", not a defect.Хибне зауваження коштує стільки ж, скільки пропуск, і звучить гучніше: після кількох таких справжній дефект стає одним рядком серед хибних, і рев'ю перестають читати. Тому агент спершу підтверджує або спростовує власні зауваження високої серйозності, а якщо до оцінюваного не дістатися, це зауваження «could not verify», а не дефект.
ШагиStepsКроки
- Запустите
/critiqueдля текущего diff или сузьте область:/critique src/payments/для пути,/critique correctness only, чтобы ограничить измерения. Аргументом может быть также файл, diff, PR или тема.Run/critiquefor the current diff, or scope it:/critique src/payments/for a path,/critique correctness onlyto restrict the dimensions. The argument can also be a file, a diff, a PR or a topic.Запустіть/critiqueдля поточного diff або звузьте область:/critique src/payments/для шляху,/critique correctness only, щоб обмежити виміри. Аргументом може бути також файл, diff, PR чи тема. - Агент определяет изменённую поверхность и читает diff и достаточно окружающего кода, чтобы оценить его. Строки в отрыве от окружения он не рецензирует.The agent identifies the changed surface and reads the diff plus enough surrounding code to judge it. Lines are not reviewed in isolation.Агент визначає змінену поверхню й читає diff та достатньо навколишнього коду, щоб його оцінити. Рядки поза оточенням він не рецензує.
- Он проверяет по порядку корректность (ошибки логики, сдвиг на единицу, неверные условия, гонки, обработка null, пути ошибок, утечки ресурсов, нарушенные инварианты) и безопасность данных (инъекции, непроверенный ввод, секреты в коде, небезопасная десериализация, разрушительные операции без защиты, безопасность миграций).It checks, in order, correctness (logic bugs, off-by-one, wrong conditionals, races, null handling, error paths, resource leaks, broken invariants) and security and data safety (injection, unvalidated input, secrets in code, unsafe deserialization, destructive operations without guards, migration safety).Він перевіряє за порядком коректність (помилки логіки, зсув на одиницю, хибні умови, гонки, обробка null, шляхи помилок, витоки ресурсів, порушені інваріанти) та безпеку даних (ін'єкції, неперевірений ввід, секрети в коді, небезпечна десеріалізація, руйнівні операції без захисту, безпека міграцій).
- Затем архитектура для слоя кода (направление зависимостей, тонкие точки входа, логика в нужном слое), тесты (покрытие нового пути и его сбоев, слабые проверки), антимусор (семь ищущихся по grep шаблонов в
docs/CODE_QUALITY.md) и ясность (имена, мёртвый код, комментарии, объясняющие почему, бюджет размера файла).Then architecture for the code layer (dependency direction, thin entry points, logic in the right layer), tests (coverage for the new path and its failure modes, weak assertions), anti-slop (the seven greppable patterns indocs/CODE_QUALITY.md) and clarity (naming, dead code, comments that say why, the file-size budget).Потім архітектура для шару коду (напрям залежностей, тонкі точки входу, логіка в потрібному шарі), тести (покриття нового шляху та його збоїв, слабкі перевірки), антисміття (сім шаблонів, що шукаються за grep, уdocs/CODE_QUALITY.md) і ясність (імена, мертвий код, коментарі, що пояснюють чому, бюджет розміру файлу). - Каждое замечание сверяется с кодом, номера строк и символы не выдумываются, и записывается как
severity·file:line· проблема · конкретная правка. Серьёзность от высшей к низшей:bug>risk>arch>test>nit>q.Each finding is checked against the code, never an invented line number or symbol, and written asseverity·file:line· the problem · a concrete fix. Severities, highest first:bug>risk>arch>test>nit>q.Кожне зауваження звіряється з кодом, номери рядків і символи не вигадуються, і записується якseverity·file:line· проблема · конкретна правка. Серйозність від вищої до нижчої:bug>risk>arch>test>nit>q. - Перед отчётом он подтверждает или опровергает собственные замечания высокой серьёзности. Если существенного нет, он так и говорит и называет остаточные риски, например непротестированный путь или непокрытый крайний случай, вместо того чтобы раздувать отчёт.Before reporting, it confirms or refutes its own high-severity findings. If it finds nothing material, it says so and names the residual risks, such as an untested path or an uncovered edge case, instead of padding.Перед звітом він підтверджує або спростовує власні зауваження високої серйозності. Якщо суттєвого немає, він так і каже та називає залишкові ризики, наприклад непротестований шлях чи непокритий крайній випадок, замість того щоб роздувати звіт.
Что вы должны увидетьWhat you should seeЩо ви маєте побачити
Замечания, сгруппированные по серьёзности, самые серьёзные первыми; короткая строка о том, что в порядке, только если она несёт информацию (например, что миграция обратима и защищена); и одна итоговая строка с единственным самым важным, что нужно решить до слияния.Findings grouped by severity, highest first; a short line on what is solid, only if it carries information (for example, that a migration is reversible and guarded); and one closing line with the single most important thing to address before merge.Зауваження, згруповані за серйозністю, найсерйозніші першими; короткий рядок про те, що в порядку, лише якщо він несе інформацію (наприклад, що міграція зворотна й захищена); і один підсумковий рядок із єдиним найважливішим, що треба розв'язати до злиття.
СоветыTipsПоради
- Для документа, набора данных или договора измерения отображаются один в один: корректность - держатся ли утверждения и сходятся ли цифры; безопасность - что утекает или необратимо; архитектура - на своём ли месте это лежит; тесты - что поймает ошибку в следующий раз; антимусор - общий список в
docs/PROJECT_SHAPES.md. См. формы проектов.For a document, a dataset or a contract the dimensions map one for one: correctness is do the claims hold and the numbers reconcile; security, what leaks or cannot be undone; architecture, does it belong where it was put; tests, what would catch an error next time; anti-slop, the generic list indocs/PROJECT_SHAPES.md. See project shapes.Для документа, набору даних чи договору виміри відображаються один в один: коректність - чи тримаються твердження й чи сходяться цифри; безпека - що витікає або незворотне; архітектура - чи на своєму місці це лежить; тести - що впіймає помилку наступного разу; антисміття - загальний перелік уdocs/PROJECT_SHAPES.md. Див. форми проєктів. - Семь шаблонов, на которых держится измерение «против мусора», ищутся по grep, так что вы можете сами просмотреть diff на их наличие. См. как не дать сгенерированному коду гнить.The seven patterns behind the anti-slop dimension are greppable, so you can scan a diff for them yourself. See keep generated code from rotting.Сім шаблонів, на яких тримається вимір «проти сміття», шукаються за grep, тож ви можете самі переглянути diff на їхню наявність. Див. як не дати згенерованому коду гнити.
- Команда названа
/critique, а не review, намеренно: в Claude Code команда review - псевдоним встроенной команды code-review, а навык проекта псевдоним не заменяет. Пользуйтесь любой: они различаются областью, а эта не зависит от стека и градуирует замечания по серьёзности.The command is named/critiqueand not review on purpose: in Claude Code the review command is an alias of the built-in code-review command, and a project skill never replaces an alias. Use either; they differ in scope, and this one is stack-neutral and graded by severity.Команду названо/critique, а не review, навмисно: у Claude Code команда review - псевдонім вбудованої команди code-review, а навичка проєкту псевдонім не замінює. Користуйтеся будь-якою: вони різняться областю, а ця не залежить від стеку й градуює зауваження за серйозністю.
Связанные функцииRelated functionsПов'язані функції
- Получить ревью изменений: одна строка на замечаниеReview a diff with one terse line per findingОтримати рев'ю змін: один рядок на зауваження
/caveman-reviewпроверяет текущий diff или названную цель и выводит замечания по одной строке: файл, строка, метка (bug,risk,nit,q), проблема, исправление./caveman-reviewreviews the current diff, or what you name, and prints findings first, one line each: file, line, a label (bug,risk,nit,q), problem and fix./caveman-reviewперевіряє поточний diff або названу ціль і виводить зауваження по одному рядку: файл, рядок, мітка (bug,risk,nit,q), проблема, виправлення. - Не дать сгенерированному коду сгнитьKeep generated code from rottingНе дати згенерованому коду згнитиСемь шаблонов, которые выглядят нормально и тихо гноят кодовую базу: не пишите их, отмечайте с первого взгляда и блокируйте только новые нарушения.Seven patterns that look fine and quietly rot a codebase: do not write them, flag them on sight, and gate only new violations.Сім шаблонів, що виглядають нормально й тихо гноять кодову базу: не пишіть їх, позначайте з першого погляду й блокуйте лише нові порушення.
- Подогнать метод под работу, которая не кодFit the method to work that is not codeПідігнати метод під роботу, що не є кодомСопоставьте слова метода с вашим видом работы, найдите проверку, которая у вас уже есть, и удалите правила слоя кода, которым нечего применять.Map the method's nouns to your kind of work, find the check you already have, and delete the code-layer rules that have nothing to apply to.Зіставте слова методу з вашим видом роботи, знайдіть перевірку, яка у вас уже є, і видаліть правила шару коду, яким нічого застосовувати.
- Доказать, что правка работает, прежде чем объявлять её готовойProve a change works before calling it doneДовести, що правка працює, перш ніж оголошувати її готовоюВозьмите самую дешёвую проверку, которая доказывает именно эту правку, запишите ожидаемое и фактическое и называйте шаг готовым, только прочитав результат.Pick the lowest check that proves this change, write down expected and actual, and call the step done only after you have read the result.Візьміть найдешевшу перевірку, що доводить саме цю правку, запишіть очікуване й фактичне та називайте крок готовим, лише прочитавши результат.
- Запустить изменение и проверить, что оно правда работаетRun a change and see whether it actually worksЗапустити зміну й перевірити, що вона справді працюєЗапустить и посмотреть: проверка заканчивается одной строкой, PASS, DEFECT, COULD NOT VERIFY или NOT APPLICABLE, с путём к свидетельствам.Run the thing and observe it: the check ends in one line, PASS, DEFECT, COULD NOT VERIFY or NOT APPLICABLE, with the path to its evidence.Запустити й подивитися: перевірка закінчується одним рядком, PASS, DEFECT, COULD NOT VERIFY або NOT APPLICABLE, зі шляхом до свідчень.