- Зачем:
- предыдущая запись предлагала написать тест сверки копий NO_PROXY, хотя
tests/test_proxy_env_sync.py делает ровно это с момента появления копий.
- Что:
- запись переписана: дублирование под охраной, но на Windows проверка
падает с ModuleNotFoundError: fcntl, потому что грузит proxy-client
через runpy.
- задача заменена на «дать тесту читать константу без исполнения модуля».
- сохранён нерешённый вопрос про замещение no_proxy_extra: эта ветка не
покрыта тестом вообще.
- Проверка:
- python tests/test_proxy_env_sync.py -v — 1 ok, 1 ERROR (fcntl), то есть
поведение, описанное в записи.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
- Зачем:
- машинный конфиг, собранный копированием примера, молча выкидывает
из обхода .concept-software.ru и Gitea, а обрыв всплывает как TLS-ошибка.
- Что:
- в README уточнено, что no_proxy_extra замещает DEFAULT_NO_PROXY_EXTRA
целиком, а не дополняет его.
- предупреждение продублировано в обоих примерах конфига, рядом с полем.
- в TODO.md записано дублирование списка в пяти windows-файлах и
отсутствие теста синхронности с источником.
- Проверка:
- bash tests/test_proxy_launchers.sh — 8 проверок пройдено.
- configparser читает оба примера, значение no_proxy_extra не изменилось.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
- Зачем:
- документация должна отражать только действующие правила и планы.
- Что:
- из AGENTS.md удалены устаревшие разделы о Git и общих требованиях.
- из TODO.md удален прежний список задач.
- Проверка:
- git diff --cached --check.
- Зачем:
- новые команды должны быть в справочнике, а непроверенная 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>
Dmitriy DementievandClaude Opus 5
ddadmin
Dmitry Dementiev