diff --git a/TODO.md b/TODO.md index bf16b63..6b00af5 100644 --- a/TODO.md +++ b/TODO.md @@ -11,17 +11,7 @@ Связанный PR: [anomalyco/opencode#16525](https://github.com/anomalyco/opencode/pull/16525) -## ~~codex-review: убрать permission prompts~~ DONE - -Реализовано: шаги 2, 3, 7 переписаны на встроенные инструменты (Write tool, stdout capture). -См. коммит с `fix(skills): убраны permission prompts из codex-review`. - -## ~~codex-review: передавать список файлов вместо diff~~ DONE - -Реализовано: Codex получает `--name-only` список файлов и git-команды, -сам читает diff. Убраны temp-файлы `claude-diff-*`, preflight, часть cleanup. - -## codex-review: изучить официальный плагин OpenAI +## adversarial-review: изучить официальный плагин OpenAI Посмотреть [openai/codex-plugin-cc](https://github.com/openai/codex-plugin-cc) — официальный плагин для ревью с Codex. Найти идеи для улучшения скилла: @@ -30,22 +20,9 @@ Также разобраться с поведением `codex exec resume` — переключается на `workspace-write` sandbox и `xhigh` reasoning вместо запрошенных `read-only` / `high`. -## ~~codex-review: plan mode без git~~ DONE +## adversarial-review: отдельный репозиторий + статья на Хабр -Реализовано: fast path для режима `plan` — при явном аргументе `plan` или -в Claude Code Plan Mode git-проверки и определение base branch пропускаются. -Resume упрощён через `2>/dev/null` (stderr → /dev/null, stdout = чистое ревью). - -## codex-review: pipe в определении base branch - -Команда `git symbolic-ref ... | sed` содержит pipe `|`, который может -триггерить permission prompt в Claude Code. Пока не триггерит, но если начнёт — -заменить на два отдельных вызова или определять branch иначе. -Актуально только для режимов `code` и `code-vs-plan` — в `plan` git не вызывается. - -## codex-review: отдельный репозиторий + статья на Хабр - -Выделить скилл codex-review в отдельный публичный репозиторий: +Выделить скилл adversarial-review в отдельный публичный репозиторий: - README с описанием, установкой, примерами - Инструкция по подключению к Claude Code / Codex / другим AI-агентам diff --git a/agents/.agents/skills/codex-review/SKILL.md b/agents/.agents/skills/codex-review/SKILL.md deleted file mode 100644 index 3b82ca6..0000000 --- a/agents/.agents/skills/codex-review/SKILL.md +++ /dev/null @@ -1,340 +0,0 @@ ---- -name: codex-review -description: Ревью плана или кода через OpenAI Codex. Автодетект режима, итеративные правки до одобрения. -user_invocable: true ---- - -# Итеративное ревью через Codex - -Отправляет текущую работу в OpenAI Codex на adversarial-ревью. Автоматически определяет, что ревьюить: **план** или **код**. Claude правит по замечаниям Codex и переотправляет до одобрения. Максимум 5 раундов. - ---- - -## Когда вызывать - -- `/codex-review` — автодетект что ревьюить -- `/codex-review plan` — принудительно ревью плана -- `/codex-review code` — принудительно ревью кода -- `/codex-review <путь-к-файлу>` — ревью конкретного файла (аргумент содержит `/` или `.`) -- Переопределение reasoning: `/codex-review xhigh` или `/codex-review low` (одно из: `none`, `low`, `medium`, `high`, `xhigh`) -- Переопределение модели: `/codex-review model:gpt-5.3-codex` (аргумент с префиксом `model:`) - -## Инструкции - -> **Плейсхолдеры:** `${REVIEW_ID}`, `${CODEX_SESSION_ID}` и `${BASE_BRANCH}` в шагах ниже — это шаблонные плейсхолдеры, НЕ shell-переменные. Подставляй литеральные значения напрямую в каждый tool call. - -### Шаг 1: Определить режим ревью - -Определи, что ревьюить. Проверяй в порядке приоритета: - -**1. Явный аргумент** (`plan`, `code`, путь к файлу) → использовать его. - - Для `plan` → пропустить все git-проверки, перейти к шагу 2 (только REVIEW_ID). - -**2. Claude Code Plan Mode** — если в контексте есть системное сообщение "Plan mode is active" → режим = `plan`, пропустить git. В Plan Mode код не редактируется, поэтому code/code-vs-plan невозможны. - -**3. Автодетект** (без явного аргумента, вне Plan Mode): - -1. Проверь наличие изменений кода (любой непустой — значит есть): - - `git diff --name-only` — unstaged - - `git diff --cached --name-only` — staged - - `git diff --name-only ${BASE_BRANCH}...HEAD` — коммиты ветки -2. Проверь, есть ли план в текущем контексте разговора (из plan mode, задач или обсуждения). - -| Изменения кода? | План в контексте? | Режим | -|----------------|-------------------|-------| -| Нет | Да | **plan** — ревью плана | -| Да | Да | **code-vs-plan** — ревью реализации против плана | -| Да | Нет | **code** — ревью изменений кода | -| Нет | Нет | Спросить пользователя, что ревьюить | - -### Шаг 2: Сгенерировать Session ID и определить base branch - -Сгенерируй уникальный `REVIEW_ID` самостоятельно, формат: `{unix_timestamp}-{случайное_4значное_число}`. -Пример: `1711872000-4821`. **НЕ используй bash** — подставляй значение напрямую в команды следующих шагов. - -**Определение base branch (только для режимов `code` и `code-vs-plan`):** - -Для режима `plan` — пропустить определение base branch, перейти к шагу 3. - -Для остальных режимов определи base branch репозитория: - -```bash -git symbolic-ref refs/remotes/origin/HEAD 2>/dev/null | sed 's|refs/remotes/origin/||' -``` - -Если команда вернула пустой результат (remote HEAD не настроен), используй fallback: - -```bash -git rev-parse --verify main 2>/dev/null && echo main || echo master -``` - -Сохрани результат как `BASE_BRANCH` — используется в `git diff ${BASE_BRANCH}...HEAD` далее. - -### Шаг 3: Подготовить материал для ревью - -**Ревью плана:** - -- Если план уже существует как файл (в `project/`, plan file от Plan Mode, memory или где-то в репо) — использовать путь напрямую. НЕ копировать. В Claude Code Plan Mode план всегда является файлом. -- Если план только в контексте разговора (вне Plan Mode) — записать через **Write tool** в `/tmp/claude-plan-${REVIEW_ID}.md`. -- **Обязательно вывести путь к файлу плана пользователю**, чтобы он мог открыть его в IDE: - `📄 План для ревью: <путь-к-файлу>` - -**Ревью кода:** - -Собери список изменённых файлов: - -1. `git diff --name-only` — unstaged changes -2. `git diff --cached --name-only` — staged changes - -Объедини unstaged + staged (уникальные пути). Если оба пусты: - -3. `git diff --name-only ${BASE_BRANCH}...HEAD` — коммиты ветки (fallback) - -Branch берётся ТОЛЬКО когда нет локальных изменений — иначе контекст раздувается. -Для branch в промпте указывай команду `git diff ${BASE_BRANCH}...HEAD` (полный diff). - -Codex имеет доступ к репо и сам прочитает полный diff и файлы. -В промпт (шаг 4) передай список файлов и какие git diff команды запускать. - -**Много файлов (> 50):** если объединённый список превышает 50 путей, -передай в промпт только git-команды без списка файлов — Codex разберётся сам. - -Если все источники пусты — нет изменений для ревью, сообщи пользователю. - -**Ревью кода против плана:** подготовить путь к плану И собрать список изменённых файлов (как выше). - -### Шаг 4: Сформировать промпт и запустить первый раунд - -Сформируй промпт в зависимости от режима: - -**Промпт для ревью плана:** -``` -Review the implementation plan in . Focus on: -1. Correctness — will this plan achieve the stated goals? -2. Risks — what could go wrong? Edge cases? Data loss? -3. Missing steps — is anything forgotten? -4. Alternatives — is there a simpler or better approach? -5. Security — any security concerns? -``` - -**Промпт для ревью кода (<= 50 файлов):** -``` -Review the code changes in this repo. Changed files: - -<список файлов из --name-only> - -Changes include: . -Run to see the full diffs. Focus on: -1. Bugs — logic errors, off-by-one, null handling, race conditions -2. Edge cases — what inputs or states could break this? -3. Style — does the code follow existing project conventions? -4. Security — injection, credential exposure, unsafe operations -5. Tests — is the change adequately tested? -``` - -**Промпт для ревью кода (> 50 файлов):** -``` -Review the code changes in this repo. -Changes include: . -Run to see changed files and full diffs. Focus on: -1. Bugs — ... -(те же 5 пунктов) -``` - -**Промпт для ревью кода против плана:** -``` -Review the code changes in this repo against the implementation plan in . -Changed files: - -<список файлов или пусто если > 50> - -Changes include: <тип>. -Run to see the full diffs. Focus on: -1. Completeness — does the implementation cover all plan steps? -2. Deviations — where does the code differ from the plan? Are deviations justified? -3. Bugs — logic errors, edge cases, null handling -4. Security — injection, credential exposure, unsafe operations -5. Missing — what from the plan is not yet implemented? -``` - -**Все промпты заканчиваются:** -``` -Be specific and actionable. Reference file paths and line numbers where possible. - -If the work is solid and ready, end your review with exactly: VERDICT: APPROVED -If changes are needed, end with exactly: VERDICT: REVISE -``` - -**Запуск Codex — шаблон команды:** - -Флаги: -- `-m gpt-5.4` — модель (переопределяется аргументом `model:...`) -- `-c model_reasoning_effort=high` — глубина рассуждения (переопределяется аргументом `xhigh`, `low` и т.д.) -- `-s read-only` — Codex только читает, не пишет -- `-o /tmp/codex-review-${REVIEW_ID}.md` — файл для записи ответа - -```bash -timeout 600 codex exec \ - -m gpt-5.4 \ - -c model_reasoning_effort=high \ - -s read-only \ - -o /tmp/codex-review-${REVIEW_ID}.md \ - "ПРОМПТ" -``` - -**Важно:** -- Всегда оборачивай `codex exec` в `timeout 600` (10 минут). Если Codex зависнет — команда завершится с кодом 124. -- Используй параметр `timeout: 620000` в Bash tool для запаса. -- Команда **синхронная**: когда она вернулась, файл `-o` уже готов. **НЕ** используй poll-loop (`while/sleep`). -- Если exit code = 124 (таймаут) — сообщи пользователю и предложи повторить. - -**Пример для режима code** (unstaged изменения в двух файлах): - -```bash -timeout 600 codex exec \ - -m gpt-5.4 \ - -c model_reasoning_effort=high \ - -s read-only \ - -o /tmp/codex-review-${REVIEW_ID}.md \ - "Review the code changes in this repo. Changed files: - -agents/.agents/skills/codex-review/SKILL.md -TODO.md - -Changes include: unstaged changes. -Run git diff to see the full diffs. Focus on: -1. Bugs — logic errors, off-by-one, null handling, race conditions -... -If changes are needed, end with exactly: VERDICT: REVISE" -``` - -**После запуска:** найди в выводе строку `session id: ` и сохрани значение как `CODEX_SESSION_ID` — оно нужно для `resume` в последующих раундах. - -**Примечания:** -- Модель по умолчанию: `gpt-5.4` с `model_reasoning_effort=high`. Пользователь может переопределить через аргументы. -- Всегда `-s read-only` — Codex не должен писать файлы. -- `-o` для захвата вывода в файл. **НЕ** запускай в background — команда сама вернёт управление. - -### Шаг 5: Прочитать ревью и проверить вердикт - -1. Прочитать `/tmp/codex-review-${REVIEW_ID}.md` -2. Показать пользователю: - -``` -## Codex Review — Раунд N (режим: , модель: gpt-5.4) - -[Отзыв Codex] -``` - -3. Проверить вердикт: - - **VERDICT: APPROVED** → перейти к Шагу 8 (Готово) - - **VERDICT: REVISE** → перейти к Шагу 6 (Правки) - - Нет явного вердикта, но всё позитивно → считать одобренным - - Достигнут максимум (5 раундов) → перейти к Шагу 8 с пометкой - -### Шаг 6: Внести правки - -По замечаниям Codex: - -**Для ревью плана:** исправить план — адресовать каждое замечание. Обновить файл плана (или temp-файл). Показать пользователю: - -``` -### Правки (Раунд N) -- [Что изменено и почему, один пункт на замечание] -``` - -**Для ревью кода:** исправить код напрямую — редактировать файлы, запустить тесты если применимо. Показать пользователю: - -``` -### Исправления (Раунд N) -- [Что исправлено и почему, один пункт на замечание] -``` - -**Пропустить** правку, если она противоречит явным требованиям пользователя — отметить это для пользователя. - -### Шаг 7: Переотправить в Codex (Раунды 2-5) - -**Resume — основной путь.** Экономит токены и сохраняет контекст сессии Codex. Свежий `codex exec` без resume — **аварийный fallback**, расходует значительно больше токенов. Использовать только при ошибке resume. - -1. Запусти resume с подавлением stderr (`2>/dev/null`): - -```bash -timeout 600 codex exec resume ${CODEX_SESSION_ID} \ - "I've revised based on your feedback. - -Here's what I changed: -[Список правок] - -Please re-review. End with VERDICT: APPROVED or VERDICT: REVISE" 2>/dev/null -``` - -Используй `timeout: 620000` в параметрах Bash tool. - -**Почему `2>/dev/null`:** `codex exec` по дизайну разделяет потоки — progress/metadata → stderr, финальный ответ модели → stdout. Подавление stderr даёт чистый вывод без CLI-шума. Результат Bash tool = только ревью. - -2. Проверь результат по exit code: - - **exit 0** — успех. stdout содержит чистое ревью. Показать пользователю напрямую (Write в файл и Read **не нужны**). Проверить VERDICT: последняя непустая строка stdout = `VERDICT: APPROVED` или `VERDICT: REVISE`. Если вердикт отсутствует → вывод мог быть обрезан, перейти к Fallback. Далее применить обработку вердикта из Шага 5 (APPROVED → Шаг 8, REVISE → Шаг 6). - - **exit 124** — таймаут. Сообщи пользователю: "Codex не ответил за 10 минут" и предложи повторить. - - **другой exit code** — сообщи пользователю: "Resume не удался (exit code N)". Перейди к Fallback. Диагностика без stderr недоступна — не пытайся парсить stdout как ошибку. - -**Fallback** — если `resume` не сработал (сессия истекла, session ID не захвачен, ошибка): - -1. Собрать список изменённых файлов (аналогично Шагу 3). -2. Запустить свежий `codex exec -o` с описанием предыдущих раундов в промпте. - -Вернуться к **Шагу 5**. - -### Шаг 8: Итоговый результат - -**Одобрено:** -``` -## Codex Review — Итог (режим: <режим>, модель: gpt-5.4) - -**Статус:** ✅ Одобрено после N раунд(ов) - -[Итоговый отзыв] - ---- -**Проверено и одобрено Codex. Ожидает вашего решения.** -``` - -**Достигнут максимум раундов:** -``` -## Codex Review — Итог (режим: <режим>, модель: gpt-5.4) - -**Статус:** ⚠️ Достигнут максимум (5 раундов) — не полностью одобрено - -**Оставшиеся замечания:** -[Нерешённые вопросы] - ---- -**У Codex остались замечания. Просмотрите их и решите, как действовать дальше.** -``` - -### Шаг 9: Очистка - -**В Claude Code Plan Mode:** пропустить любой cleanup (включая deferred). rm вызовет permission prompt. Файлы подчистятся при следующем вызове вне Plan Mode. - -**Вне Plan Mode:** - -```bash -rm -f /tmp/claude-plan-${REVIEW_ID}.md /tmp/codex-review-${REVIEW_ID}.md -``` - -Если пользователь отклонил rm — продолжить без ошибки. - -НЕ удалять файлы планов, которые существовали до ревью (только temp-файлы, созданные этим скиллом). Старые temp-файлы от предыдущих сессий безвредны в /tmp и очистятся ОС при перезагрузке. - -## Правила - -- Claude **активно правит** по замечаниям Codex — это НЕ просто передача сообщений -- Автодетект режима ревью по контексту; аргументы пользователя имеют приоритет -- При явном аргументе `plan` или в Claude Code Plan Mode: пропускать git-проверки и определение base branch -- Resume — основной путь для повторных раундов. Свежий exec — аварийный fallback (дорогой по токенам) -- Cleanup — best-effort: в Plan Mode пропускать, при отказе продолжать без ошибки -- Предпочитать существующие файлы, не создавать лишние копии -- Всегда read-only sandbox — Codex никогда не пишет файлы -- Максимум 5 раундов для защиты от бесконечных циклов -- Показывать пользователю отзывы и правки каждого раунда -- Если Codex CLI не установлен или упал — сообщить пользователю: `npm install -g @openai/codex` -- Если правка противоречит явным требованиям пользователя — пропустить и объяснить почему diff --git a/claude/.claude/settings.local.json b/claude/.claude/settings.local.json index cd8f45b..cd22325 100644 --- a/claude/.claude/settings.local.json +++ b/claude/.claude/settings.local.json @@ -106,11 +106,11 @@ "Bash(codex *)", "Bash(timeout 600 codex exec *)", "Bash(rm -f /tmp/claude-plan-*)", - "Bash(rm -f /tmp/codex-review-*)", + "Bash(rm -f /tmp/codex-*)", "WebSearch", "mcp__claude_ai_Context7__resolve-library-id", "Write(/tmp/claude-plan-*)", - "Write(/tmp/codex-review-*)" + "Write(/tmp/codex-*)" ] }, "statusLine": {