- Зачем:
- новые команды должны быть в справочнике, а непроверенная Windows-часть — зафиксирована как задача.
- Что:
- README: абзац про pclaude/pcodex и строки в таблицах bash/PowerShell-функций.
- TODO: добавлен ручной чеклист проверки PowerShell-части на Windows.
- Проверка:
- git show --stat HEAD.
- Зачем:
- codex-review дублировал adversarial-review с менее проработанными промптами,
а Codex CLI регулярно путался и пытался вызывать сам себя.
- Что:
- удалён скилл codex-review (SKILL.md).
- из TODO.md убраны завершённые задачи, актуальные перефокусированы на adversarial-review.
- в settings.local.json паттерны codex-review-* расширены до codex-* для adversarial-review.
- Проверка:
- git diff HEAD~1 --stat.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Зачем:
- в Plan Mode ревью плана вызывало ~5 permission prompts (git, Write, rm),
хотя для плана git не нужен, а план уже является файлом.
- Что:
- добавлен fast path: явный `plan` или Plan Mode пропускают git-проверки
- base branch определяется лениво (только для code/code-vs-plan)
- resume упрощён через `2>/dev/null` (stderr → /dev/null, stdout = чистое ревью)
- cleanup пропускается в Plan Mode, best-effort вне его
- удалены deprecated permissions claude-diff-* из settings.local.json
- Проверка:
- `/codex-review plan` в Plan Mode — 0 permission prompts
- `/codex-review code` вне Plan Mode — работает как раньше
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Зачем:
- зафиксировать известные проблемы и план выделения скилла в отдельный репо.
- Что:
- добавлен TODO: resume переключает sandbox/reasoning (разобраться с плагином).
- добавлен TODO: pipe в определении base branch.
- добавлен TODO: отдельный репозиторий + статья на Хабр (высокий приоритет).
- Проверка:
- cat TODO.md.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Зачем:
- официальный плагин OpenAI может содержать идеи для улучшения скилла codex-review.
- Что:
- добавлен пункт TODO со ссылкой на openai/codex-plugin-cc.
- Проверка:
- cat TODO.md.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Зачем:
- Codex имеет доступ к репо в read-only sandbox и может сам читать diff — сбор в temp-файл избыточен.
- Что:
- шаг 1: автодетект дополнен проверкой branch-изменений (BASE_BRANCH...HEAD).
- шаг 2: добавлено определение base branch через origin/HEAD + fallback main/master.
- шаг 3: diff заменён на --name-only списки (unstaged+staged, branch как fallback).
- шаг 4: промпты передают список файлов и git-команды вместо <diff-path>.
- убраны temp-файлы claude-diff-*, preflight --shortstat, часть cleanup.
- TODO.md: задача отмечена как выполненная.
- Проверка:
- /codex-review code — одобрено Codex за 3 раунда, 0 permission prompts.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Зачем:
- Codex и так имеет доступ к репо в read-only sandbox, сбор diff избыточен.
- Что:
- добавлен пункт TODO: передавать список файлов вместо diff.
- Проверка:
- cat TODO.md.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Зачем:
- планы должны жить в TODO.md, AGENTS.md ссылается на него.
- Что:
- перенесён план по codex-review из AGENTS.md в TODO.md.
- в AGENTS.md добавлена ссылка на TODO.md.
- Проверка:
- cat TODO.md, cat AGENTS.md.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Зачем:
- Codex-ревью выявило permission bypass через Bash(timeout *) и
сохраняющиеся safety prompts на $() и > в bash-командах.
- Что:
- permission сужен до Bash(timeout 600 codex exec *), rm-паттерны конкретизированы.
- REVIEW_ID использует $RANDOM для энтропии, resume пишет в файл вместо pipe.
- добавлен TODO в AGENTS.md: план замены bash-паттернов на нативные инструменты.
- Проверка:
- /codex-review при следующем использовании.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Dmitry Dementiev
ddadminandClaude Opus 4.6