Commit Graph
7 Commits
Author SHA1 Message Date
ddadminandClaude Opus 4.6 76e6b293c8 refactor(skills): оптимизирован codex-review для Plan Mode и resume
- Зачем:
  - в 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>
2026-04-03 12:26:46 +03:00
ddadminandClaude Opus 4.6 55079efe46 refactor(skills): codex-review передаёт список файлов вместо diff
- Зачем:
  - 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>
2026-04-02 00:07:57 +03:00
ddadminandClaude Opus 4.6 c7e981604e fix(skills): убраны permission prompts из codex-review
- Зачем:
  - каждый раунд ревью требовал 3-5 ручных подтверждений из-за $(), > и 2>&1 в bash-командах.
- Что:
  - шаг 2: Session ID генерируется Claude без bash, вместо $(date).
  - шаг 3: git diff → stdout + Write tool вместо > redirect; добавлен preflight через --shortstat.
  - шаг 7: resume stdout + Write tool вместо > 2>&1; дифференцированная обработка exit code.
  - добавлено пояснение о плейсхолдерах ${REVIEW_ID}/${CODEX_SESSION_ID}.
  - TODO.md: задача отмечена как выполненная.
- Проверка:
  - /codex-review code на текущем diff — 0 permission prompts за 3 раунда.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-03-31 23:33:36 +03:00
ddadminandClaude Opus 4.6 3c690bdb12 fix(skills): ужесточены permissions и добавлен TODO для codex-review
- Зачем:
  - 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>
2026-03-31 10:59:26 +03:00
ddadminandClaude Opus 4.6 6190fbaf08 fix(skills): улучшена надёжность и UX скилла codex-review
- Зачем:
  - скилл генерировал множество промежуточных bash-команд, каждая из которых
    требовала ручного подтверждения пользователя.
  - poll-loop (while/sleep) мог зависнуть навсегда при сбое Codex.
- Что:
  - добавлены permissions для типовых команд скилла (timeout, rm/Write для /tmp).
  - uuidgen заменён на date +%s (меньше промптов).
  - codex exec обёрнут в timeout 600 (10 мин) вместо poll-loop.
  - resume оставлен как primary, свежий exec — fallback.
  - добавлен захват session ID в Шаг 4.
- Проверка:
  - /codex-review при следующем использовании.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-03-31 10:38:07 +03:00
ddadminandClaude Opus 4.6 bdccd889fe refactor(skills): переработан скилл codex-review
- Зачем:
  - скилл работал только с планами; нужна поддержка ревью кода и автодетект режима.
- Что:
  - добавлен автодетект режима: plan, code, code-vs-plan.
  - три отдельных промпта для каждого режима ревью.
  - модель обновлена на gpt-5.4, reasoning effort = high.
  - исправлен сбор diff (условный, не перезаписывающий).
  - инструкции переведены на русский, промпты к Codex оставлены на английском.
  - добавлен вывод пути к файлу плана для открытия в IDE.
  - явные правила парсинга аргументов (model:..., reasoning level, путь к файлу).
- Проверка:
  - /codex-review в Claude Code.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-03-31 09:40:41 +03:00
ddadmin 9b05b4ab82 chore(skills): добавлен скилл codex-review
- Зачем:
  - нужен инструмент для итеративного ревью планов реализации через OpenAI Codex CLI.
- Что:
  - добавлен SKILL.md для скилла codex-review: цикл до 5 раундов, Claude отправляет план → Codex рецензирует → Claude правит → повторяет до VERDICT: APPROVED.
- Проверка:
  - вызвать `/codex-review` во время или после plan mode.
2026-03-30 23:42:22 +03:00