From f48183177799513c204854462059bb51cdd20fa3 Mon Sep 17 00:00:00 2001 From: Dmitry Dementiev Date: Tue, 4 Aug 2026 00:16:02 +0300 Subject: [PATCH] =?UTF-8?q?fix(skills):=20=D0=BF=D0=B5=D1=81=D0=BE=D1=87?= =?UTF-8?q?=D0=BD=D0=B8=D1=86=D0=B0=20=D1=80=D0=B5=D0=B2=D1=8C=D1=8E=D0=B5?= =?UTF-8?q?=D1=80=D0=B0=20=D0=B7=D0=B0=D0=B4=D0=B0=D0=BD=D0=B0=20=D0=B7?= =?UTF-8?q?=D0=BD=D0=B0=D1=87=D0=B5=D0=BD=D0=B8=D0=B5=D0=BC,=20=D0=B0=20?= =?UTF-8?q?=D0=BD=D0=B5=20=D0=B7=D0=B0=D0=BF=D1=80=D0=B5=D1=82=D0=BE=D0=BC?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Зачем: - запрет «read-only ревьюеру не режь» стоял в двух местах и всё равно был нарушен при сборке команды для холодного ревью этой же сессии: отступление обосновали тем, что у ревью документа руки не нужны. Третья копия запрета делу не помогла бы — нужен не запрет, а отсутствие выбора. - Что: - правило переписано значением: песочница ревьюера — workspace-write, как у исполнителя; линии различает свежесть треда и мандат, а не привилегия. Оговорка про руки ушла внутрь обоснования, и там же сказано, что ревью документа не исключение. - добавлена строка, что ветка та же для любого слепого ревью — кода, постановки, документа, самого скилла: рецепт копируют целиком, линзу задают файлом линии. Повод собирать команду руками снят. - Проверка: - рецепт ветки ревьюера содержит -s workspace-write и штатную связку approval_policy/approvals_reviewer; прочитан целиком после правки. Co-Authored-By: Claude Opus 5 --- .../claude-subagent-playbook/references/codex-exec.md | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/agents/.agents/skills/claude-subagent-playbook/references/codex-exec.md b/agents/.agents/skills/claude-subagent-playbook/references/codex-exec.md index 1edba21..c5d8d60 100644 --- a/agents/.agents/skills/claude-subagent-playbook/references/codex-exec.md +++ b/agents/.agents/skills/claude-subagent-playbook/references/codex-exec.md @@ -159,13 +159,18 @@ cat "$D/review-prompt-code.md" | codex exec --json --strict-config \ > "$D/review-impl-stdout.jsonl" 2> "$D/review-impl-stderr.txt" ``` +- **Ветка та же для любого слепого ревью** — кода, постановки, документа, + самого скилла: рецепт копируй целиком, линзу задавай файлом линии, а не + пересборкой команды. - **Свежим тредом**, не тем, что писал код. Вызов тот же, что у исполнителя: запись + авто-аппрув. - Эффорт `high` — дефолт ревью; ниже опускай только по явной просьбе пользователя. -- Read-only-песочницей ревьюера не режь: она отнимает у него руки — тесты, - проверку гипотез, поиск, документацию, первоисточники. Защита — мандат в - промпте, готовым файлом +- **Песочница у ревьюера та же, что у исполнителя — `workspace-write`.** + Линии различает свежесть треда и мандат, а не привилегия: read-only отнял + бы руки — тесты, проверку гипотез, поиск, документацию, первоисточники, — и + ревью документа не исключение, там руками проверяют команды и рецепты. + Защита — мандат в промпте, готовым файлом [prompts/review-boilerplate.md](prompts/review-boilerplate.md) (сборка — [prompt-templates.md](prompt-templates.md)); Codex держит его надёжно (практика пользователя, 2026-07-10). Дешёвая подстраховка: