원클릭으로 Manus에서 모든 스킬 실행

external-review

스타1

포크0

업데이트2026년 7월 1일 21:29

Внешнее ревью PR через Codex CLI ChatGPT subscription (Sprint Pipeline v3.9 Mode A primary). Две полноразмерные модели последовательно — gpt-5.5 (reasoning, primary) + gpt-5.3-codex (code); для doc-only PR Reviewer B (gpt-5.3-codex) пропускается автоматически (§1.5) — single-model gpt-5.5. Fallback модели — gpt-5.4. Fallback режимы — A-legacy (openai-review.mjs Platform API), C (Claude adversarial degraded) и D (manual emergency). Используй `/external-review <PR_NUMBER>`.

설치

Codex 또는 Claude로 설치 이 Prompt를 복사해 Codex, Claude 또는 다른 어시스턴트에 붙여 넣으면 Skill 페이지를 검토하고 설치를 진행할 수 있습니다.

Manus에서 실행

출처

komleff

komleff/u2

GitHub 저장소 열기 Creator 저장소 보기

다운로드

Manus에서 실행

External Review — кросс-модельное ревью через Codex CLI ChatGPT subscription

Автоматизация Sprint Final (Фаза 3 sprint-pr-cycle). Две полноразмерные модели разных архитектур проверяют PR по всем 4 аспектам, максимизируя adversarial diversity (для doc-only scope Reviewer B пропускается — single-model gpt-5.5, §1.5; Mode A остаётся Mode A, не degradation). Основной путь — Codex CLI с ChatGPT subscription quota (Sprint Pipeline v3.9, ADR 3.27); при его недоступности — Mode A-legacy (Node.js native через openai-review.mjs на Platform API) или degraded режимы C/D. Sprint Final не блокируется ни в одном режиме, но помечается честно.

Источник подхода: Adversarial Code Review для Claude Code

Backend evolution: v3.6 — openai-review.mjs Platform API primary; v3.9 — Codex CLI subscription primary, openai-review.mjs сохраняется как Mode A-legacy. Решение: .agents/PIPELINE_ADR.md ADR 3.27 (cost-driven: $0/вызов в пределах квоты vs $20-22/вызов на Platform API). Decision keep both reviewers подтверждён данными big-heroes/surprise-arena: 22.7% unique-to-Codex Critical/High findings (см. docs/research/2026-05-05-codex-overlap-analysis.md, U2-275). U2-native анализ 2026-05-19 уточняет invocation policy: GPT-5.3-Codex остаётся Reviewer B для Critical/Sprint Final/runtime/tooling PR, но не вызывается по умолчанию для Light/doc-only/landing проходов (docs/research/2026-05-19-gpt55-vs-gpt53-codex-u2-review-effectiveness.md).

Контекст

Прочитай перед началом:

.agents/AGENT_ROLES.md секция «3. Reviewer» — формат вердикта (4 аспекта).
.memory-bank/activeContext.md — текущее состояние проекта.
.agents/CODEX_AUTH.md §8 — ChatGPT subscription path setup и validated invocation pattern.
.agents/PIPELINE_ADR.md ADR 3.27 — design rationale для Codex CLI subscription backend.
docs/research/2026-05-19-gpt55-vs-gpt53-codex-u2-review-effectiveness.md — U2-specific effectiveness stats and Reviewer B invocation policy.
docs/archive/pipeline/2026-05-05-u2-rhm-external-review-codex-migration.md — архивный handoff-документ миграции v3.9 (план завершён) с Section A-K описывающим backend migration. Для деталей runtime allowlist Mode A-legacy и endpoint dispatch — см. .claude/tools/openai-review.mjs (--help + sources) и .agents/CODEX_AUTH.md §1-§7 (Platform API setup).

Аргументы

PR_NUMBER — номер PR для ревью (обязательный).
--legacy — explicit override на Mode A-legacy (openai-review.mjs Platform API), bypass Mode A primary (Codex CLI subscription) даже если subscription доступна. Use case: self-modification PRs которые меняют .claude/skills/*, .agents/*, .codex/, или AGENTS.md — Mode A primary имеет known trust boundary gap для них (Beads U2-me5, P1; см. Шаг 3.1 KNOWN LIMITATION warning).
--manual — explicit Mode D ручной fallback (per Шаг 2 mode table).
--doc — ручной override/страховка, allow-list, не enumerate-negative: форсирует skip Reviewer B (single-model gpt-5.5) только когда HAS_EXECUTABLE = "gh_diff_failed_fail_closed" — авто-классификатор (Шаг 1.5) реально недоступен, gh pr diff упал. Любой другой scope отказывает: реальный executable/governance/config, unknown_extension_fail_closed (диф прочитан, классификатор доступен, конкретный файл просто не опознан как known-harmless), и любой прочий non-md scope (в т.ч. known-harmless non-md вроде .png/.txt/.csv — диф не строго .md, значит не «doc-only» в терминах этого флага). НЕ меняет floor (floor всегда по фактическому scope). На чистом doc-only diff флаг не нужен — авто-детект (Шаг 1.5) уже скипает Reviewer B.

Argument parsing (выполняется первым в Шаге 1):

PR_NUMBER=""
FORCE_LEGACY=0
FORCE_MANUAL=0
FORCE_DOC=0
for arg in "$@"; do
  case "$arg" in
    --legacy) FORCE_LEGACY=1 ;;
    --manual) FORCE_MANUAL=1 ;;
    --doc) FORCE_DOC=1 ;;
    [0-9]*) PR_NUMBER="$arg" ;;
    *) echo "ВНИМАНИЕ: неизвестный аргумент '$arg' игнорируется." >&2 ;;
  esac
done
if [ -z "$PR_NUMBER" ]; then
  echo "СТОП: PR_NUMBER обязателен. Usage: /external-review <PR_NUMBER> [--legacy|--manual|--doc]" >&2
  exit 1
fi

Шаг 1: Пререквизиты

1.1 Проверка чистоты рабочего дерева

Проверяется ПЕРЕД любыми операциями (git fetch, ref creation), чтобы избежать конфликтов и потери локальных изменений. После trust boundary fix (Шаг 7.6 Plan A) gh pr checkout больше не выполняется — pipeline остаётся в trusted base, но clean tree всё равно требуется для предсказуемого git fetch и операций с локальными refs.

if [ -n "$(git status --porcelain)" ]; then
  echo "СТОП: рабочее дерево не чистое. Закоммить или stash изменения."
  exit 1
fi

1.2 Получение PR-метаданных + fetch PR head как trusted ref (БЕЗ checkout)

Trust boundary (Шаг 7.6 Plan A): tool НЕ запускается из ветки PR в Mode A-legacy. Раньше gh pr checkout переключал cwd в untrusted ветку, и node openai-review.mjs в ней мог быть подменён PR-автором с доступом к $OPENAI_API_KEY. Сейчас pipeline остаётся в trusted base/default branch, PR код только читается через diff с fetched ref.

В Mode A primary (Codex CLI subscription) trust boundary не критичен: Codex CLI — глобальная установка (@openai/codex npm install -g), бинарник в $AppData\Roaming\npm\codex.cmd или /usr/local/bin/codex, не из cwd. PR-автор не может подменить codex binary через PR. Однако clean tree всё равно требуется для предсказуемого git fetch и git checkout --detach (если используется --base invocation).

BASE_BRANCH=$(gh pr view <PR_NUMBER> --json baseRefName --jq '.baseRefName')
STATE=$(gh pr view <PR_NUMBER> --json state --jq '.state')

if [ "$STATE" != "OPEN" ]; then echo "СТОП: PR не открыт"; exit 1; fi

# Fetch PR head как локальный ref (БЕЗ checkout — cwd остаётся в trusted base).
# refspec `pull/<PR>/head` — это GitHub-специфичный ref для PR head независимо от source repo.
PR_REF="u2-pr-<PR_NUMBER>"
git fetch origin "+pull/<PR_NUMBER>/head:refs/heads/${PR_REF}"

BASE_BRANCH используется далее в Mode A через codex review --base "origin/$BASE_BRANCH" (после detached checkout PR_REF) или codex review --commit "$HEAD_COMMIT" (single-commit path). В Mode A-legacy — openai-review.mjs --base "$BASE_BRANCH" --head "$PR_REF". PR_REF для всех режимов одинаков.

1.3 Проверка что ветка запушена

if ! git rev-parse --abbrev-ref '@{u}' >/dev/null 2>&1; then
  echo "СТОП: upstream не настроен. Выполни: git push -u origin $(git branch --show-current)"
  exit 1
fi

UNPUSHED=$(git log @{u}..HEAD --oneline)
if [ -n "$UNPUSHED" ]; then
  echo "СТОП: есть незапушенные коммиты. Выполни git push."
  exit 1
fi

1.4 Pre-flight: проверка backend availability, фиксация HEAD_COMMIT

Двухступенчатый check (v3.9): сначала проверяем Codex CLI ChatGPT subscription (primary), затем — openai-review.mjs Platform API (Mode A-legacy fallback). Если оба недоступны → degraded path (Mode C/D).

# Фиксация HEAD_COMMIT сразу — используется далее в именах артефактов (Шаг 3.1)
# и в commit binding публикации (Шаг 5.3). Единое значение для всего прохода ревью.
HEAD_COMMIT=$(timeout 10 gh pr view <PR_NUMBER> --json headRefOid --jq '.headRefOid')
if [[ -z "$HEAD_COMMIT" || "$HEAD_COMMIT" == "null" || ! "$HEAD_COMMIT" =~ ^[0-9a-fA-F]{40}$ ]]; then
  echo "СТОП: не удалось получить валидный HEAD commit для PR <PR_NUMBER>" >&2
  exit 1
fi

# Trust boundary verify: убедиться что cwd = trusted base (default branch или
# другая базовая ветка), НЕ ветка PR. Защищает Mode A-legacy (openai-review.mjs из cwd).
# Mode A primary не страдает от cwd-substitution (codex CLI — глобальный binary), но
# clean trusted base всё равно нужен для idempotent git операций.
CURRENT_BRANCH=$(git branch --show-current)
if [ "$CURRENT_BRANCH" != "$BASE_BRANCH" ]; then
  echo "СТОП: external-review должен запускаться из trusted base ('$BASE_BRANCH'), текущая ветка '$CURRENT_BRANCH'." >&2
  echo "Сначала: git switch '$BASE_BRANCH' && git pull" >&2
  exit 1
fi

MODE_A_AVAILABLE=0
MODE_PRIMARY=""
MODE_A_LEGACY_AVAILABLE=0

# --- Operator override: --manual флаг force Mode D (manual emergency) ---
# Bypass всю automation chain, route напрямую в Mode D. Use case: incident response,
# forced risk-acceptance runs, или когда automated paths сломаны.
if [ "$FORCE_MANUAL" = "1" ]; then
  echo "Operator override --manual: bypass automation, force Mode D (manual emergency)."
  MODE_A_AVAILABLE=0
  MODE="D"
  # Skip всю availability detection — оператор вручную провёл review external tool'ом.
# --- Operator override: --legacy флаг bypass Mode A primary ---
# Для self-modification PRs которые меняют .claude/skills/*, .agents/*, .codex/,
# AGENTS.md — Mode A primary имеет fundamental trust boundary gap (Beads U2-me5).
# Operator-aware override force routing на Mode A-legacy (openai-review.mjs runs from
# trusted base, читает diff через --head refspec без cwd contents loading).
elif [ "$FORCE_LEGACY" = "1" ]; then
  echo "Operator override --legacy: bypass Mode A primary, force Mode A-legacy."
  # Skip Codex CLI check полностью, идём сразу к Mode A-legacy availability check ниже.
else
  # --- Step 1: Codex CLI ChatGPT subscription (primary) ---
  # Бинарник `codex` должен быть в PATH (npm install -g @openai/codex@latest, ≥0.128.0).
  # `codex login status` возвращает строку "Logged in using ChatGPT" если subscription active.
  # Profile [profiles.review] из ~/.codex/config.toml должен существовать (см. CODEX_AUTH.md §8.4).
  if command -v codex >/dev/null 2>&1; then
    CODEX_LOGIN_STATUS=$(timeout 10 codex login status 2>&1 || true)
    if echo "$CODEX_LOGIN_STATUS" | grep -q "Logged in using ChatGPT"; then
      # Profile validation (per external review iter 4+5 findings): убедиться что
      # [profiles.review] существует и загружается. Без этого Mode A primary будет
      # помечен available, но invocation в Шаге 3.1 упадёт с "profile not found" вместо
      # graceful fallback на Mode A-legacy.
      #
      # Iter 5 finding: `codex -p review --version` exit 0 даже когда profile missing
      # (--version не загружает profile). Только `exec` действительно loads profile config.
      # Используем `exec` only (no OR) для точной валидации.
      if timeout 10 codex -p review exec "echo profile-ok" >/dev/null 2>&1; then
        MODE_A_AVAILABLE=1
        MODE_PRIMARY="chatgpt"
        echo "Mode A primary: Codex CLI ChatGPT subscription + profile [review] доступны."
      else
        echo "ВНИМАНИЕ: Codex CLI logged in, но profile [profiles.review] не загружается."
        echo "Проверь ~/.codex/config.toml (см. CODEX_AUTH.md §8.4)."
        echo "Проверка fallback на Mode A-legacy (openai-review.mjs Platform API)..."
      fi
    else
      echo "ВНИМАНИЕ: Codex CLI установлен, но subscription path недоступен. Status: $CODEX_LOGIN_STATUS"
      echo "Проверка fallback на Mode A-legacy (openai-review.mjs Platform API)..."
    fi
  else
    echo "ВНИМАНИЕ: codex CLI не в PATH. Проверка Mode A-legacy fallback..."
  fi
fi

# --- Step 2: Mode A-legacy fallback (openai-review.mjs Platform API) ---
# Запускается только если Mode A primary недоступен. Установка openai SDK
# и валидация $OPENAI_API_KEY через --ping.
if [ "$MODE_A_AVAILABLE" = "0" ]; then
  if [ ! -d .claude/tools/node_modules ]; then
    # npm ci (не install): строго по lockfile, не пачкает рабочее дерево, воспроизводимо.
    # --ignore-scripts оставлен как defence-in-depth (даже trusted package может иметь
    # неожиданные lifecycle hooks).
    if ! (cd .claude/tools && npm ci --ignore-scripts); then
      echo "ВНИМАНИЕ: npm ci упал в .claude/tools/ — Mode A-legacy недоступен."
    fi
  fi

  # Ранняя проверка валидности $OPENAI_API_KEY (<2 сек, AC6 плана) — только если SDK установлен.
  if [ -d .claude/tools/node_modules ]; then
    if node .claude/tools/openai-review.mjs --ping; then
      MODE_A_AVAILABLE=1
      MODE_PRIMARY="legacy"
      MODE_A_LEGACY_AVAILABLE=1
      echo "Mode A-legacy: openai-review.mjs Platform API доступен (Codex CLI subscription unavailable, используем fallback)."
    else
      echo "ВНИМАНИЕ: Mode A-legacy недоступен (невалидный ключ / rate limit / network)."
      echo "Скрипт перейдёт в degraded-режим C. Детали ошибки — на stderr выше."
    fi
  fi
fi

# Если оба primary и legacy недоступны → MODE_A_AVAILABLE=0 → fallback C/D.
if [ "$MODE_A_AVAILABLE" = "0" ]; then
  echo "ВНИМАНИЕ: Mode A primary (Codex CLI subscription) и Mode A-legacy (openai-review.mjs) оба недоступны."
fi

Что детектируется на каждом шаге:

Codex CLI primary check fail: бинарник не в PATH; codex login status не возвращает Logged in using ChatGPT (logged out, refresh token expired, OAuth broken); profile review не зарегистрирован в ~/.codex/config.toml (косвенно через invocation в Шаге 3.1 — здесь не проверяем).
Mode A-legacy fail: npm ci не отработал; --ping exit ≠ 0 (невалидный ключ, rate limit, network); подробности через stderr скрипта (см. .claude/tools/README.md таблицу exit codes).

1.5 Pre-flight scope classification (единый источник для skip-политики и floor/cap)

Почему в pre-flight, а не в §3.5.2: Reviewer B (§3.1) вызывается ДО convergence-детекта (§3.5), поэтому чтобы применить doc-only политику (research 2026-05-19: для DOC_ONLY не звать gpt-5.3-codex) на ПЕРВОЙ итерации, scope должен быть известен здесь. Это единственный источник классификации — §3.5.2 переиспользует эти же переменные (single source of truth, защита от drift regex-паттернов).

DIFF_FILES=$(gh pr diff "$PR_NUMBER" --name-only 2>&1)
GH_EXIT=$?

if [ "$GH_EXIT" -ne 0 ] || [ -z "$DIFF_FILES" ]; then
  # Fallback ДО fail-closed sentinel (iter-6 external review finding, PR #492): PR_REF уже
  # fetch'нут локально в Шаге 1.2 (`git fetch origin +pull/<N>/head:refs/heads/u2-pr-<N>`),
  # поэтому при сбое именно gh API (rate limit / network / auth) файл-лист часто доступен
  # локально без обращения к GitHub API. Уменьшает случаи, когда PM вынужден полагаться
  # на ручное суждение через --doc: если fallback дал реальный список — классификатор
  # отрабатывает НОРМАЛЬНО (не sentinel), --doc в этом раунде не нужен вовсе.
  echo "ВНИМАНИЕ: gh pr diff упал (exit=$GH_EXIT) или пустой список. Пробую fallback: локальный git diff по уже fetch'нутому PR_REF."
  # Освежаем base ref перед диффом (та же практика, что в §3.1) — без этого локальный
  # origin/$BASE_BRANCH может быть устаревшим относительно текущего состояния GitHub,
  # и fallback-дифф разойдётся с тем, что реально видит gh. Сбой самого fetch — не фатален
  # (git diff ниже использует что есть локально), не блокирует fallback.
  git fetch origin "+refs/heads/$BASE_BRANCH:refs/remotes/origin/$BASE_BRANCH" 2>&1 || true
  DIFF_FILES=$(git diff --name-only "origin/$BASE_BRANCH...$PR_REF" 2>&1)
  GIT_EXIT=$?
  if [ "$GIT_EXIT" -ne 0 ] || [ -z "$DIFF_FILES" ]; then
    echo "ВНИМАНИЕ: локальный git diff тоже упал (exit=$GIT_EXIT) или пуст. Fail-closed: HAS_EXECUTABLE, CAP=20, SKIP_REVIEWER_B=0."
    EXEC_REAL=""                          # реального совпадения нет — поднятие по неопределённости
    HAS_EXECUTABLE="gh_diff_failed_fail_closed"
    HAS_CONFIG=""
    DOC_ONLY="fail_closed"
    CAP=20
  else
    echo "Fallback успешен: DIFF_FILES получен через локальный git diff (gh API недоступен, но локальный ref рабочий)."
  fi
fi

if [ -n "${HAS_EXECUTABLE:-}" ] && [ "$HAS_EXECUTABLE" = "gh_diff_failed_fail_closed" ]; then
  : # оба источника (gh API и локальный git) недоступны — sentinel уже выставлен выше, классификация ниже не нужна
else
  # EXEC_REAL — РЕАЛЬНОЕ совпадение executable/governance-путей (код, ^.claude/, ^.agents/, ^.codex/,
  # AGENTS.md/CLAUDE.md, .gitignore/.gitattributes). Тот же набор, что detection --legacy в Шаге 1.4.
  EXEC_REAL=$(echo "$DIFF_FILES" | grep -E '\.(py|ts|cs|js|jsx|tsx|mjs|cjs|mts|sh|ps1|bat|go|rs|java|kt|rb|php|swift|html|css|svg)$|(^|/)Dockerfile$|^\.claude/|^\.agents/|^\.codex/|(^|/)AGENTS\.md$|(^|/)CLAUDE\.md$|^\.gitignore$|^\.gitattributes$' | head -1)
  HAS_EXECUTABLE="$EXEC_REAL"

  # Fail-closed: unknown extension (вне known-harmless whitelist) → escalate в executable (sentinel, НЕ EXEC_REAL).
  KNOWN_HARMLESS=$(echo "$DIFF_FILES" | grep -E '\.(md|json|yaml|yml|toml|txt|csv|tsv|png|jpg|jpeg|gif|webp|ico)$|^\.gitignore$|^\.gitattributes$' | wc -l)
  TOTAL_FILES=$(echo "$DIFF_FILES" | wc -l)
  if [ "$KNOWN_HARMLESS" -lt "$TOTAL_FILES" ] && [ -z "$HAS_EXECUTABLE" ]; then
    echo "ВНИМАНИЕ: файлы вне known-harmless whitelist. Fail-closed: escalate в executable scope."
    HAS_EXECUTABLE="unknown_extension_fail_closed"
  fi

  # Config: остальные .json/.yaml/.yml/.toml (кроме .claude/settings.json — оно governance).
  HAS_CONFIG=$(echo "$DIFF_FILES" | grep -vE '^\.claude/settings\.json$' | grep -E '\.(json|yaml|yml|toml)$' | head -1)
  # DOC_ONLY: инвертирована — НЕПУСТО ⟺ есть не-.md файл; ПУСТО ⟺ все файлы .md.
  DOC_ONLY=$(echo "$DIFF_FILES" | grep -vE '\.md$' | head -1)

  if [ -n "$HAS_EXECUTABLE" ]; then
    CAP=20
  elif [ -n "$HAS_CONFIG" ]; then
    CAP=12
  elif [ -z "$DOC_ONLY" ]; then
    CAP=7
  else
    CAP=12  # mixed default
  fi
fi
echo "Scope: EXEC_REAL='${EXEC_REAL:-}' HAS_EXECUTABLE='${HAS_EXECUTABLE:-}' HAS_CONFIG='${HAS_CONFIG:-}' DOC_ONLY_nonmd='${DOC_ONLY:-}' CAP=$CAP"

# --- Reviewer B skip policy (research 2026-05-19) ---
# ⚠️ ИНВАРИАНТ: DOC_ONLY инвертирована (см. выше) — skip требует ПУСТОГО DOC_ONLY: только [ -z ], НИКОГДА [ -n ].
# Governance-.md (^.claude/, ^.agents/, ^.codex/, AGENTS.md, CLAUDE.md) уже в HAS_EXECUTABLE по пути —
# поэтому self-modification PR НЕ проходит в skip-ветку (получает обе модели).
SKIP_REVIEWER_B=0
SKIPPED_REVIEWER_B=0  # явная инициализация каждого прогона — защита от stale-значения из предыдущего /external-review в той же shell-сессии
if [ -z "$HAS_EXECUTABLE" ] && [ -z "$HAS_CONFIG" ] && [ -z "$DOC_ONLY" ]; then
  SKIP_REVIEWER_B=1
  echo "AUTO doc-only scope: Reviewer B (gpt-5.3-codex) пропущен по политике research 2026-05-19 (только gpt-5.5)."
fi
# --doc — УЗКИЙ override, ALLOW-LIST (не enumerate-negative): единственный легитимный сценарий —
# классификатор реально недоступен из-за сбоя самого `gh pr diff`
# (HAS_EXECUTABLE="gh_diff_failed_fail_closed"). Только это разрешено — ЛЮБОЙ другой scope
# (EXEC_REAL, HAS_CONFIG, unknown_extension_fail_closed, известные-harmless non-md типа
# .png/.txt/.csv, mixed doc+non-md) отказывает. Allow-list вместо enumerate-negative закрывает
# и уже известные, и ещё не найденные варианты того же класса «диф не является строго .md,
# но --doc всё равно скипает» (iter-3 High + iter-5 Medium: обе находки — enumerate-negative
# пропускал ровно то, что не было явно перечислено). Governance/self-mod PR через --doc НЕ скипается.
if [ "$FORCE_DOC" = "1" ]; then
  if [ "$HAS_EXECUTABLE" = "gh_diff_failed_fail_closed" ]; then
    SKIP_REVIEWER_B=1
    echo "--doc принят: снята неопределённость сбоя gh pr diff, Reviewer B пропущен (только gpt-5.5)."
  else
    echo "ОТКАЗ: --doc применяется только когда единственная причина эскалации — сбой самого gh pr diff (gh_diff_failed_fail_closed). Текущий scope: EXEC_REAL='${EXEC_REAL:-}' HAS_CONFIG='${HAS_CONFIG:-}' HAS_EXECUTABLE='${HAS_EXECUTABLE:-}' DOC_ONLY_nonmd='${DOC_ONLY:-}'. Reviewer B вызывается."
  fi
fi

⚠️ Skip ≠ понижение floor. SKIP_REVIEWER_B управляет лишь тем, звать ли gpt-5.3-codex. Floor итераций (§3.5.1) — строго по scope diff (fail-closed: HAS_EXECUTABLE → floor 5 всегда, даже если Reviewer B не отработал). --doc НЕ опускает floor. Инвариант «floor следует scope, не доступности модели» сохранён.

Для doc-only scope Mode A остаётся Mode A (не новый режим и не degradation) — просто Reviewer B не вызывается.

Шаг 2: Определение режима работы

Пять активных режимов: четыре initial-detected (A primary, A-legacy, C, D) выбираются на pre-flight в Шаге 1.4 + один runtime-derived (A-hybrid) emerges из частичного отказа Mode A primary в Шаге 3.1.2. Mode A primary — Codex CLI ChatGPT subscription. Mode A-legacy — openai-review.mjs Platform API как automatic fallback при недоступности subscription. Mode A-hybrid — Reviewer A через subscription + Reviewer B через Platform API при недоступности gpt-5.3-codex на subscription tier. Mode C/D — degraded paths.

Режим A primary: Codex CLI ChatGPT subscription (основной путь, v3.9)

Если MODE_A_AVAILABLE=1 И MODE_PRIMARY="chatgpt" из Шага 1.4:

Reviewer A (primary): codex -p review -c sandbox_mode="danger-full-access" review --commit "$HEAD_COMMIT" — модель gpt-5.5 берётся из [profiles.review] (см. .agents/CODEX_AUTH.md §8.4). Reasoning высокий, MCP_DOCKER подавлен через mcp_servers = {} table-replace в profile.
Reviewer B: codex -p review -c sandbox_mode="danger-full-access" -c model="gpt-5.3-codex" review --commit "$HEAD_COMMIT" — -c model="gpt-5.3-codex" перебивает default из profile. Если gpt-5.3-codex недоступна через subscription tier (выясняется по exit code и stderr CLI; типичная ошибка model X not available on this tier) — Reviewer B автоматически переключается на Mode A-legacy через openai-review.mjs --model gpt-5.3-codex (см. Шаг 3.1.2 hybrid fallback).
Запуск sequential, НЕ parallel — два concurrent codex вызова конфликтуют через shared ~/.codex/auth.json lock. Sequential добавляет ~30-60s wall-clock, но stable.
Quota burn: ChatGPT subscription quota — fixed monthly ($25 Plus / $100 Pro / $30/seat Business). $0/вызов в пределах квоты. Quota check — UI: chatgpt.com Settings → Subscriptions → Usage (нет CLI API).
На 429 quota exhausted: ручной switch второго аккаунта через .claude/tools/codex-account-switch.ps1 (Use-CodexAccount work-business), затем retry. Если оба исчерпаны → fallback на Mode A-legacy (см. Шаг 3.1.2).
В отчёте: — Reviewer (GPT-5.5 via Codex CLI subscription) и — Reviewer (GPT-5.3-Codex via Codex CLI subscription). Для doc-only scope (Шаг 1.5 SKIP_REVIEWER_B=1) Reviewer B не вызывается — только строка Reviewer A (gpt-5.5); Mode A остаётся Mode A, не degradation.

Codex CLI выводит markdown report на stdout. Exit codes: 0 OK, ≠0 runtime/auth/model error.

Режим A-legacy: Node.js native через OpenAI SDK (auto-fallback v3.6 path)

Если MODE_A_AVAILABLE=1 И MODE_PRIMARY="legacy" из Шага 1.4 (Codex CLI subscription недоступен, Platform API доступен):

Reviewer A: node .claude/tools/openai-review.mjs --model gpt-5.5 --base "$BASE_BRANCH" --head "$PR_REF" — reasoning-специализация (/v1/chat/completions с reasoning_effort: "high"). Fallback: --model gpt-5.4 если 5.5 недоступна.
Reviewer B: node .claude/tools/openai-review.mjs --model gpt-5.3-codex --base "$BASE_BRANCH" --head "$PR_REF" — code-специализация (/v1/responses с reasoning.effort: "high").
Две полноразмерные модели разных архитектур = максимальная adversarial diversity. Платформа: OpenAI Platform API (pay-per-token, $OPENAI_API_KEY).
В отчёте: — Reviewer (GPT-5.5 Platform API) и — Reviewer (GPT-5.3-Codex Platform API) + label ⚠️ Mode A-legacy fallback — primary v3.9 path (Codex CLI subscription) недоступен.

Скрипт возвращает raw markdown на stdout (4 аспекта + вердикт APPROVED | CHANGES_REQUESTED | ESCALATION). Exit codes: 0 OK, 1 runtime/API, 2 валидация, 3 nothing to review — см. .claude/tools/README.md таблицу.

Режим C: Все Mode A варианты недоступны (degraded, Claude adversarial)

Если MODE_A_AVAILABLE=0 (Codex CLI subscription и Platform API оба недоступны), а оператор явно не выбрал ручной fallback:

Два прохода Claude-субагента против одного и того же PR:
- Проход 1 (стандартный): Reviewer в обычном режиме, промпт как в Standard tier.
- Проход 2 (adversarial): Reviewer с промптом «Ты adversarial reviewer. Твоя задача — найти то, что первый проход пропустил. Ищи baseline-слабые места: необработанные edge-cases, скрытые дубликаты, некорректные инварианты».
⚠️ В отчёте обязательна метка: «⚠️ Degraded mode с имитацией adversarial diversity. Не является cross-model review».
Атрибуция: — Reviewer (Claude Opus 4.7, standard) и — Reviewer (Claude Opus 4.7, adversarial).

Режим D: Автоматические пути недоступны (manual emergency)

Если все Mode A варианты недоступны И оператор требует ручного fallback (например, gpt-5.4 через VS Code GitHub Copilot Agent):

Оператор вручную прогоняет PR через внешний инструмент.
PM собирает вывод в текстовом виде и публикует с меткой: «⚠️ Manual emergency mode. Adversarial diversity и воспроизводимость снижены».
Атрибуция — по фактическому источнику (— Reviewer (GPT-5.4 via Copilot Agent)).

Таблица выбора режима

Режим	Условие запуска	Reviewer A	Reviewer B
A (primary, v3.9)	`codex login status` → `Logged in using ChatGPT` И profile `[profiles.review]` загружается	gpt-5.5 (Codex CLI subscription)	gpt-5.3-codex (Codex CLI subscription, может fallback на Platform API при tier limit)
A-legacy	Codex CLI недоступен И `openai-review.mjs --ping` → exit 0	gpt-5.5 Platform API (или gpt-5.4 fallback)	gpt-5.3-codex Platform API
C	Все Mode A варианты недоступны, оператор не указал ручной fallback	Claude Opus 4.7 standard	Claude Opus 4.7 adversarial
D	Оператор явно указал ручной fallback (`--manual` или чат-команда)	По факту (Copilot Agent / др.)	— (single reviewer)

В любом режиме каждый проверяющий проходит все 4 аспекта. Различается источник ревью и атрибуция.

Doc-only scope (Шаг 1.5 SKIP_REVIEWER_B=1): в режимах A / A-legacy Reviewer B (gpt-5.3-codex) не вызывается — single-model gpt-5.5. Это под-вариант того же режима (MODE не меняется), не шестой режим и не degradation; Reviewer A проходит все 4 аспекта.

Шаг 3: Запуск проверяющих

3.1 Режим A primary — через Codex CLI subscription (v3.9, основной путь)

Sequential, НЕ parallel. Два concurrent codex вызова могут конфликтовать через shared ~/.codex/auth.json (refresh token rotation, OAuth state). Sequential добавляет ~30-60s wall-clock, но даёт stable result.

Default invocation = --base "origin/$BASE_BRANCH" после git checkout --detach $PR_REF (per external review iter 1 finding). Это покрывает PR полностью независимо от количества коммитов. --commit "$HEAD_COMMIT" — opt-in для smoke/single-commit cases (см. 3.1.x альтернатива ниже). Sprint Final review должен видеть весь diff PR'а, не только последний commit.

⚠️ KNOWN LIMITATION (Beads U2-me5, P1): Mode A primary имеет fundamental trust boundary gap для self-modification PRs — PRs которые меняют .claude/skills/*, .agents/*, .codex/, или AGENTS.md. Codex CLI loads these files из cwd regardless of git ref selection через --base/--commit, так что detached checkout в PR head exposes reviewer к PR-controlled agent configuration. PR-author мог бы подменить AGENTS.md в своём PR и biased/suppressed Codex review behavior.

Mitigation до fix U2-me5: для self-modification PRs operator должен explicitly override на Mode A-legacy через /external-review <PR_NUMBER> --legacy (manual flag). Mode A-legacy через openai-review.mjs runs from trusted base, читает diff через --head refspec без cwd contents loading — trust boundary preserved.

Detection: gh pr diff <PR_NUMBER> --name-only | grep -qE '^(\.claude/|\.agents/|\.codex/|AGENTS\.md)' → если true, рекомендуется Mode A-legacy. PR #191 (этот sprint) сам является self-modification PR — iter 1-3 strictly executed под partial trust compromise; mitigating factor что operator-authored PR не имеет malicious intent + reviewers continued surfacing real issues каждой iter (compromise не achieved practically).

# Создать каталог для артефактов (если ещё нет):
# .review-responses/ исключён из git (см. .gitignore) — raw outputs не коммитятся.
mkdir -p .review-responses

# Refresh base ref ПЕРЕД detached checkout — защита от stale `origin/$BASE_BRANCH`
# (per external review iter 2 finding). Если base branch продвинулся на GitHub
# с последнего fetch — без этого Mode A primary review against устаревший remote-tracking
# ref, possibly missing conflicts с current base или surfacing already-merged code.
# `+refs/heads/$BASE_BRANCH:refs/remotes/origin/$BASE_BRANCH` — explicit refspec
# (устойчиво к пользовательским remote.origin.fetch).
git fetch origin "+refs/heads/$BASE_BRANCH:refs/remotes/origin/$BASE_BRANCH"

# Detached checkout PR head — НЕ branch switch (trust boundary preserved для git refs).
# Codex CLI читает diff origin/$BASE_BRANCH..HEAD из git, поэтому detached HEAD = PR head
# даёт правильный diff scope. Codex CLI binary — глобальный (не из cwd), trust boundary
# preserved для primary path. Однако cwd contents = PR head state — это критично для
# любого fallback который запускает скрипты из cwd (см. Шаг 3.1.2 hybrid fallback —
# обязательный switch обратно в trusted base перед exec'ом openai-review.mjs).
git checkout --detach "$PR_REF"

# Reviewer A: gpt-5.5 (default из [profiles.review])
# Validated invocation pattern (CODEX_AUTH.md §8.3 + ADR 3.27):
# `-p review` активирует profile (model=gpt-5.5, reasoning=high, MCP_DOCKER suppressed).
# `-c sandbox_mode=...` обязателен — profile-level sandbox_mode игнорируется (Codex CLI 0.128.0 quirk).
# `review --base "origin/$BASE_BRANCH"` покрывает весь PR diff (multi-commit safe).
codex -p review -c sandbox_mode="danger-full-access" review --base "origin/$BASE_BRANCH" \
  > .review-responses/mode-a-reviewer-a-"$HEAD_COMMIT".md \
  2> .review-responses/mode-a-reviewer-a-"$HEAD_COMMIT".err
EXIT_A=$?

if [ "$EXIT_A" -ne 0 ]; then
  echo "ВНИМАНИЕ: Reviewer A (Codex CLI gpt-5.5) упал с exit $EXIT_A. stderr: .review-responses/mode-a-reviewer-a-$HEAD_COMMIT.err"
  # На 429: ручной operator action — `Use-CodexAccount work-business` (см. CODEX_AUTH.md §8.5).
  # Эту проверку реализуй inline: если grep -q '429\|quota' в .err → подсказка в stderr.
fi

# Reviewer B: gpt-5.3-codex через -c model override.
# Пропускается для doc-only scope (SKIP_REVIEWER_B=1 из Шага 1.5, research 2026-05-19).
if [ "$SKIP_REVIEWER_B" = "1" ]; then
  echo "doc-only scope: Reviewer B (gpt-5.3-codex) пропущен по политике research 2026-05-19; только gpt-5.5."
  EXIT_B=0            # штатный skip, НЕ ошибка → hybrid fallback (§3.1.2) НЕ активируется
  SKIPPED_REVIEWER_B=1
else
  SKIPPED_REVIEWER_B=0  # defensive-сброс: реальный вызов Reviewer B, не skip (защита от stale-значения предыдущего прогона)
  # `-c model="gpt-5.3-codex"` перебивает default из profile.
  # Если model недоступна на subscription tier → Codex CLI вернёт error на stderr и exit ≠ 0.
  codex -p review -c sandbox_mode="danger-full-access" -c model="gpt-5.3-codex" review --base "origin/$BASE_BRANCH" \
    > .review-responses/mode-a-reviewer-b-"$HEAD_COMMIT".md \
    2> .review-responses/mode-a-reviewer-b-"$HEAD_COMMIT".err
  EXIT_B=$?

  if [ "$EXIT_B" -ne 0 ]; then
    echo "ВНИМАНИЕ: Reviewer B (Codex CLI gpt-5.3-codex) упал с exit $EXIT_B."
    echo "Возможная причина: gpt-5.3-codex не доступна на текущем ChatGPT subscription tier."
    echo "→ Hybrid fallback на Mode A-legacy для Reviewer B (см. Шаг 3.1.2)."
  fi
fi

Альтернативный invocation для smoke/single-commit cases:

# Single-commit review — для smoke-тестов или fixup-only PRs.
# НЕ используй для Sprint Final multi-commit PR — пропустит earlier commits.
codex -p review -c sandbox_mode="danger-full-access" review --commit "$HEAD_COMMIT" \
  > .review-responses/mode-a-reviewer-a-"$HEAD_COMMIT".md \
  2> .review-responses/mode-a-reviewer-a-"$HEAD_COMMIT".err
# (Reviewer B аналогично с -c model="gpt-5.3-codex" --commit "$HEAD_COMMIT")

Выбор между --base + checkout --detach (default) и --commit (alternative): для PR review (multi-commit или single-commit одинаково корректно) — --base. Для smoke-тестов одного коммита вне PR контекста — --commit. Если есть сомнение — используй --base (multi-commit safe).

3.1.2 Hybrid fallback: Reviewer B → Mode A-legacy (если codex CLI не даёт gpt-5.3-codex)

Размещён ДО cleanup (3.1.3) per external review iter 1 finding — hybrid fallback использует --head "$PR_REF", поэтому ref должен быть жив до завершения fallback.

Если Reviewer A через Codex CLI subscription отработал, но Reviewer B упал из-за недоступности gpt-5.3-codex на subscription tier — переключи только Reviewer B на Platform API:

# Условие активации hybrid fallback:
# 1. EXIT_A == 0 (Reviewer A через Codex subscription удался)
# 2. EXIT_B != 0 И stderr содержит "model" + "not available|not supported|tier"
# 3. MODE_A_LEGACY_AVAILABLE=1 ИЛИ openai-review.mjs --ping проходит
HYBRID_FALLBACK_NEEDED=0
HYBRID_FALLBACK_FAILED=0  # для partial-report маркировки если npm ci упал
if [ "${SKIPPED_REVIEWER_B:-0}" != "1" ] && [ "$EXIT_A" = "0" ] && [ "$EXIT_B" -ne 0 ]; then
  if grep -qE 'model.*(not available|not supported|tier)' ".review-responses/mode-a-reviewer-b-$HEAD_COMMIT.err"; then
    HYBRID_FALLBACK_NEEDED=1
  fi
fi

if [ "$HYBRID_FALLBACK_NEEDED" = "1" ]; then
  echo "Hybrid fallback: Reviewer B → openai-review.mjs Platform API (gpt-5.3-codex недоступна на subscription tier)."

  # КРИТИЧНО (per external review iter 2 CRITICAL finding): SWITCH BACK В TRUSTED BASE
  # перед exec'ом openai-review.mjs. После Шага 3.1 cwd находится в detached HEAD = PR head,
  # т.е. в untrusted PR contents. Запуск `node .claude/tools/openai-review.mjs` оттуда
  # позволил бы PR-автору подменить скрипт и получить execution с $OPENAI_API_KEY.
  # Это rollback Trust boundary fix Шаг 7.6 Plan A. PR_REF остаётся жив и используется
  # через `--head "$PR_REF"` (refspec), а скрипт читается из trusted base checkout.
  git checkout "$BASE_BRANCH"

  # Ленивая установка — npm ci только когда нужен fallback (минимизируем первый-вход overhead).
  if [ ! -d .claude/tools/node_modules ]; then
    if ! (cd .claude/tools && npm ci --ignore-scripts); then
      echo "ОШИБКА: hybrid fallback недоступен — npm ci упал. Partial report публикуется с Reviewer B unavailable."
      HYBRID_FALLBACK_FAILED=1
    fi
  fi

  if [ "$HYBRID_FALLBACK_FAILED" = "0" ] && [ -d .claude/tools/node_modules ]; then
    # Запуск из trusted base (cwd = $BASE_BRANCH after switch выше). PR код читается
    # только через diff origin/$BASE_BRANCH..$PR_REF — refspec, не cwd contents.
    node .claude/tools/openai-review.mjs --model gpt-5.3-codex --base "$BASE_BRANCH" --head "$PR_REF" \
      > .review-responses/mode-a-reviewer-b-"$HEAD_COMMIT".md \
      2> .review-responses/mode-a-reviewer-b-"$HEAD_COMMIT".err
    EXIT_B=$?
    if [ "$EXIT_B" = "0" ]; then
      MODE="A-hybrid"  # Reviewer A на subscription, Reviewer B на Platform API
      echo "Hybrid fallback успешен."
    else
      echo "ВНИМАНИЕ: hybrid fallback exec упал — exit_B=$EXIT_B. Partial report с Reviewer B unavailable."
      HYBRID_FALLBACK_FAILED=1
    fi
  fi

  # Если hybrid fallback failed (npm ci broken OR exec failure) — partial report.
  # MODE остаётся "A-hybrid" (валиден для case statement в Шаге 5.3),
  # MODEL_B_NAME явно помечается как unavailable + warning label injected при публикации.
  if [ "$HYBRID_FALLBACK_FAILED" = "1" ]; then
    MODE="A-hybrid"
    MODEL_B_NAME="(unavailable — Mode A-legacy fallback failed)"
    # В Шаге 5.3 publish добавь warning label:
    # ⚠️ Reviewer B unavailable — partial report (Reviewer A через subscription отработал,
    # Reviewer B fallback на Platform API не удался: npm ci broken или exec failure)
  fi
fi

3.1.3 Cleanup PR_REF после ревью (важно для idempotency)

Размещён ПОСЛЕ hybrid fallback (3.1.2) per external review iter 1 finding — fallback использует --head "$PR_REF", ref должен быть удалён только когда все reviewer paths завершены.

После того как оба reviewer'а (включая возможный hybrid fallback) отработали — возврат в trusted base + удаление локального ref:

# Возврат в trusted base branch (был git checkout --detach в Шаге 3.1).
git checkout "$BASE_BRANCH"

# Удалить локальный ref u2-pr-<PR_NUMBER> — идемпотентно (флаг -f игнорирует отсутствие).
git update-ref -d "refs/heads/${PR_REF}" || true

# Альтернатива (если ref был как обычная ветка):
# git branch -D "$PR_REF" 2>/dev/null || true

Без cleanup при повторном /external-review для того же PR git fetch обновит ref на актуальный head — это не баг, но ref'ы накапливаются. Cleanup гигиеничен.

В отчёте (Шаг 5.3) Mode A-hybrid помечается как ⚠️ Reviewer B на Mode A-legacy fallback (gpt-5.3-codex недоступна через subscription).

3.1.4 Mode A-legacy (full Platform API path, v3.6 baseline)

Если из Шага 1.4 MODE_PRIMARY="legacy" (Codex CLI вообще недоступен) — оба reviewer'а через openai-review.mjs. Pattern идентичен v3.6:

# Trust boundary: --head "$PR_REF" указывает PR head как fetched ref,
# tool делает diff origin/${BASE_BRANCH}...${PR_REF} БЕЗ переключения cwd.
# Reviewer A: gpt-5.5 через /v1/chat/completions
node .claude/tools/openai-review.mjs --model gpt-5.5 --base "$BASE_BRANCH" --head "$PR_REF" \
  > .review-responses/mode-a-reviewer-a-"$HEAD_COMMIT".md \
  2> .review-responses/mode-a-reviewer-a-"$HEAD_COMMIT".err &
PID_A=$!

# Reviewer B: gpt-5.3-codex через /v1/responses.
# Пропускается для doc-only scope (SKIP_REVIEWER_B=1 из Шага 1.5) — не жечь Platform-токены.
# Те же side-effects, что в §3.1 primary: EXIT_B=0 + SKIPPED_REVIEWER_B=1 (иначе §5.3 валидатор упадёт exit 1).
if [ "$SKIP_REVIEWER_B" = "1" ]; then
  echo "doc-only scope: Reviewer B (gpt-5.3-codex Platform API) пропущен по политике research 2026-05-19; только gpt-5.5."
  wait "$PID_A"; EXIT_A=$?
  EXIT_B=0
  SKIPPED_REVIEWER_B=1
else
  SKIPPED_REVIEWER_B=0  # defensive-сброс: реальный вызов Reviewer B, не skip (защита от stale-значения предыдущего прогона)
  node .claude/tools/openai-review.mjs --model gpt-5.3-codex --base "$BASE_BRANCH" --head "$PR_REF" \
    > .review-responses/mode-a-reviewer-b-"$HEAD_COMMIT".md \
    2> .review-responses/mode-a-reviewer-b-"$HEAD_COMMIT".err &
  PID_B=$!

  wait "$PID_A"; EXIT_A=$?
  wait "$PID_B"; EXIT_B=$?
fi

В Mode A-legacy parallel допустим (openai-review.mjs instances independent). На Codex CLI primary — sequential обязателен (см. 3.1).

Exit 3 (nothing to review) — не ошибка, штатный сигнал пустого diff'а.

openai-review.mjs сам обеспечивает:

Явный git fetch origin +refs/heads/<base>:refs/remotes/origin/<base> (устойчиво к пользовательским remote.origin.fetch).
git diff --no-color --no-ext-diff (детерминированный diff независимо от git config пользователя).
Runtime allowlist (три полноразмерные модели — gpt-5.5 primary, gpt-5.4 fallback, gpt-5.3-codex; отказ при любой другой).
Диспатч endpoint по модели (chat.completions vs responses).
Классификацию ошибок API (401/403 ключ, 429 rate limit, 400 oversized, 5xx сервер, network).
Таймауты: 180 сек для review (reasoning high может отрабатывать долго); 2 сек для --ping без retry — строгий wall-clock бюджет AC6 плана для fail-fast на проблемах сети/ключа.

Формат вывода — готовый markdown в обоих режимах (A primary и A-legacy): system prompt (для openai-review.mjs) или Codex CLI native review задают жёсткий формат ## Вердикт: ... + ### Архитектура: ... + 3 других аспекта.

3.2 Режим C — Claude adversarial degraded

Запусти два Claude-субагента последовательно на том же PR.

Проход 1 (стандартный):

Ты Reviewer. Прочитай .agents/AGENT_ROLES.md секция "3. Reviewer".
Режим: Standard, все 4 аспекта.
Задача: проверь PR #<PR_NUMBER>. Верни structured findings PM.

Проход 2 (adversarial):

Ты adversarial Reviewer. Прочитай .agents/AGENT_ROLES.md секция "3. Reviewer".
Контекст: первый проход этого PR уже прошёл. Твоя задача — найти то, что первый проход пропустил.
Фокус: baseline-слабые места, необработанные edge-cases, скрытые дубликаты, некорректные инварианты, маловероятные, но критичные ошибки.
Задача: проверь PR #<PR_NUMBER>. Верни structured findings PM.

Сохрани findings обоих проходов для collapsible блоков. Обязательно добавь в итоговый отчёт метку «⚠️ Degraded mode с имитацией adversarial diversity. Не является cross-model review».

3.3 Режим D — manual emergency

Оператор вручную прогоняет PR через внешний инструмент (VS Code GitHub Copilot Agent или аналог) и передаёт PM текстовый вывод. PM:

Принимает вывод как есть.
Маппит на 4 аспекта.
Публикует с меткой «⚠️ Manual emergency mode. Adversarial diversity и воспроизводимость снижены».
Атрибуция — по тому, кого оператор использовал.

Шаг 3.5: Convergence detection и iteration cap (ADR 3.24)

⛔ Stateless модели не сходятся естественно. GPT-5.5/5.3-Codex каждую iter перечитывают весь diff и могут indefinitely surfacing replays + Low refinements. Без cap — infinite polish loop, нарушает цель пайплайна (облегчение труда оператора).

3.5.1 Convergence detection (preferred path)

PM объявляет APPROVED если выполнены ВСЕ условия (порог итераций — scope-aware, см. таблицу ниже):

Streak подряд с 0 новых Critical/High findings (новый = не в triage предыдущих iter, не replay уже-deferred);
только replays уже-deferred + Low refinements того же класса;
AND total external iter ≥ floor для scope/типа прохода;
AND каждое выявленное Low/Med/High finding имеет явный triage-статус (fix now / defer to Beads с ID / reject with rationale). Сохраняет инвариант #5 — никаких «висящих» finding'ов без owner.

Floor итераций и streak (scope-aware):

Scope определяется единым fail-closed классификатором из Шага 1.5 (DOC_ONLY / HAS_CONFIG / HAS_EXECUTABLE по gh pr diff --name-only; §3.5.2 soft cap переиспользует его). Floor keys на scope диффа, НЕ на том, какой ревьюер фактически отработал.

Scope diff (классификатор §1.5)	Streak clean APPROVED	Floor (total iter)
`DOC_ONLY` — product/spec/gdd `*.md`, НЕ governance	1	1
`HAS_CONFIG` — mixed (doc + config без executable)	2	3
`HAS_EXECUTABLE` — executable / governance / весь self-modification surface (`^.claude/` любой файл, `^.agents/`, `^.codex/`, `AGENTS.md`/`CLAUDE.md`, server/client код; + любой unknown-extension по fail-closed)	3	5

Fail-closed (обязательно): при неоднозначности scope, ошибке/недоступности классификатора или unknown-extension — floor 5 / streak 3 (как CAP=20 в §3.5.2). HAS_EXECUTABLE (governance/code) даёт floor 5 всегда — даже если в degraded-режиме (Mode C / A-hybrid / tier-limit) Reviewer B не отработал: floor следует scope, а не доступности модели. Понижать floor для governance из-за случайного fallback'а запрещено.

⚠️ Ярлык «doc-only» = строго DOC_ONLY из единого bash-классификатора §1.5 (§3.5.2 его переиспользует). Любой файл под ^.claude/, ^.agents/, ^.codex/, а также AGENTS.md/CLAUDE.md — governance (HAS_EXECUTABLE), не doc-only, даже с расширением .md (его .md-расширение не делает его harmless — классификатор ловит по пути, т.к. fail-closed по extension governance-.md не отсекает). «Все файлы .md» ≠ doc-only. Этот самый скилл (правка SKILL.md + PIPELINE_ADR.md) — HAS_EXECUTABLE, floor 5.

Повторный / landing проход. «Неизменный HEAD» = HEAD совпадает с коммитом предыдущего external review-pass (PM проверяет: новых коммитов с того pass нет). Floor 1 / streak 1 для повторного / landing прохода применяется только если gh pr diff <предыдущий-full-coverage-HEAD>..HEAD пуст или попадает в DOC_ONLY. Если landing-коммит несёт HAS_CONFIG / HAS_EXECUTABLE дельту — floor по scope этой дельты (вплоть до 5). Повторный проход не «обнуляет» floor сам по себе.

Почему floor 5 / streak 3 для HAS_EXECUTABLE: там работает cross-model diversity (два ревьюера разных архитектур — gpt-5.5 + gpt-5.3-codex) и выше плотность багов, нужно больше iter для покрытия. Для DOC_ONLY Reviewer B по политике (research 2026-05-19) не вызывается (вшито: §1.5 SKIP_REVIEWER_B + условный вызов §3.1/§3.1.4; ручной флаг --doc) → diversity-аргумент floor 5 не действует, а один stateless-ревьюер на неизменном тексте выдаёт противоречивые вердикты (APPROVED↔CHANGES_REQUESTED на идентичных байтах — «infinite polish loop» из преамбулы §3.5), жгущие GPT-квоту без новой информации. Эмпирика 5-7 iter — про нетривиальные code-PR.

⚠️ Только для DOC_ONLY scope — разрешение конфликтующих вердиктов на неизменном HEAD (формальная процедура, не усмотрение). «Неизменный HEAD» = HEAD == коммит предыдущего external review-pass. Если после APPROVED на том же HEAD приходит CHANGES_REQUESTED, PM не вправе объявить его «шумом» декларативно. PM обязан построчно сопоставить каждый finding диссентящего прохода ровно с одной категорией и приложить таблицу сопоставления в PR-комментарии:

replay — finding уже присутствует в triage-таблице предыдущей итерации на этом же HEAD; PM указывает проверяемую ссылку (iteration N, строка #M). Без конкретной ссылки категория не засчитывается.

вне скоупа PR — PM присваивает finding статус reject with rationale (как любому finding); обоснование аудируемо.

новый finding — не покрыт (1) и (2).

Converged-вердикт на DOC_ONLY объявляется только если КАЖДЫЙ finding диссентящего прохода попал в (1) replay-с-ссылкой или (2) reject-с-обоснованием — т.е. ни одного нового. Достаточно одного finding категории (3) — это не шум: он триажится по обычным правилам (fix now + повторный APPROVED / defer с Beads-ID / reject с обоснованием), и convergence «по шуму» не объявляется. Так субъективная фиксация convergence заменена на проверяемое доказательство (ссылка на прежний triage или явный reject), а ошибочно-пропущенный реальный finding не может быть отброшен без следа.

Для HAS_CONFIG / HAS_EXECUTABLE это послабление не действует вовсе: любой CHANGES_REQUESTED требует fix + повторного APPROVED по обычным правилам.

Инвариант floor↔gate: floor — recipe экономии итераций (когда ПЕРЕСТАТЬ гонять ревью), не gate готовности к merge. Merge-readiness определяется исключительно hard-gate'ом /finalize-pr, который не читает счётчик итераций / floor: для Sprint Final он требует external review-pass с APPROVED на HEAD независимо от floor. Пониженный floor не может создать ложный merge-ready. Не связывать floor с hard-gate'ом в будущих правках.

В v3.7 convergence/cap enforcement — PM-side manual recipe, не автоматический runtime gate: PM считает streak, scope, CAP и triage evidence в PR comment. Автоматизация этого правила — отдельное улучшение (--prev-head / diff-narrowing и fixtures), не текущее поведение скилла.

3.5.2 Soft iteration cap (safety valve, не hard ceiling)

Если convergence не detected — soft cap по scope:

Scope diff (по `gh pr diff <N> --name-only`)	Soft cap
Doc-only (только product/spec/gdd `*.md`, не governance — см. ниже)	7
Mixed (doc + config без executable: `.json`, `.yaml`)	12
Executable / governance / security-sensitive infrastructure — весь self-modification surface: `^.claude/` (любой файл: hooks, tools, skills вкл. не-SKILL.md, agents, commands, rules, settings.json), `^.agents/` (doctrine/ADR/governance, .md и не-.md), `^.codex/`, `AGENTS.md`/`CLAUDE.md` (корень или вложенные), server/client code, `.gitignore`/`.gitattributes`	20

Cap — soft. При hit cap PM эскалирует к оператору binary вопросом (см. §3.5.3), НЕ автоматически APPROVE. Если оператор хочет ещё iter — продолжаем. Cap — только trigger для дискуссии оператору, не для финального решения.

Определение scope выполнено в Шаге 1.5 (единый fail-closed классификатор — single source of truth). §3.5.2 НЕ пересчитывает scope, а переиспользует CAP / HAS_EXECUTABLE / HAS_CONFIG / DOC_ONLY оттуда (иначе два экземпляра grep-паттернов разойдутся при будущей правке — drift). Fail-closed правила (пустой diff / ошибка gh pr diff / unknown extension → CAP=20) — там же.

# Scope и CAP уже вычислены в Шаге 1.5 (единый классификатор). Здесь только используем результат —
# НЕ пересчитываем (защита от drift regex-паттернов).
if [ -z "${CAP:-}" ]; then
  echo "ВНИМАНИЕ: CAP не задан (Шаг 1.5 не выполнен?). Fail-closed: CAP=20."
  CAP=20
fi
echo "External review iteration cap для scope: $CAP"

3.5.3 Binary escalation при hit cap (без оценки findings)

Когда iter == cap и convergence ещё не detected — PM публикует бинарный вопрос в PR comment, не проси оператора оценить findings (нарушает Zero Trust to Human Tech Skills):

## ⏸ External review hit iteration cap (iter <N>/<CAP>)

Cross-model adversarial coverage достигнута. Оставшиеся findings:
- <X> новых Med refinements (детали в collapsible ниже)
- <Y> replays уже-deferred (см. <BEADS_ID>)
- 0 новых Critical/High

PM-рекомендация: **(a) APPROVE с deferred bundle**.

**Operator decision:**
- (a) Approve — finalize PR с deferred bundle <BEADS_ID> (рекомендация)
- (b) Continue iter <N+1> — ожидаемо ещё refinements того же класса, не блокеры

Жду явный (a) или (b) в PR comment или чате.

Оператор выбирает «принять текущее качество» vs «полировать дальше», без необходимости evaluating findings — это требование Zero Trust директивы.

3.5.4 Когда cap не применим

Tier ≠ Sprint Final. Cap для external review только; internal review цикл не имеет cap (Critical-tier требует 2 iter, остальные — 1).
Operator override. Оператор может явно потребовать продолжение iter ≥ cap+1 (например при подозрении на missed Critical в новом коде после cap'а).
Regression detected. Если iter N+1 нашёл NEW Critical/High который раньше не surfac'ился — cap reset, fix-cycle продолжается до полной convergence по scope-aware streak §3.5.1 (для своего scope). Регрессия, по сути, эскалирует scope: NEW Critical/High означает, что дифф уже не «тривиально doc-only», поэтому такой проход трактуется не ниже HAS_CONFIG (streak ≥ 2), а для HAS_EXECUTABLE — полные streak 3 / floor 5. Снижать streak ниже scope-значения нельзя — иначе post-regression покрытие слабее initial.

Источник: ADR 3.24 + U2 PR #186 retrospective.

Шаг 4: Copilot Re-Review

Запроси re-review от GitHub Copilot (auto-reviewer):

REPO=$(gh repo view --json nameWithOwner -q '.nameWithOwner')
gh api "repos/$REPO/pulls/<PR_NUMBER>/requested_reviewers" \
  --method POST \
  -f 'reviewers[]=copilot-pull-request-reviewer[bot]' \
  && echo "Copilot: re-review requested" \
  || echo "Copilot: request failed — может потребоваться ручной запуск"

Шаг 5: Консолидация и публикация

5.1 Имена моделей (атрибуция)

Подставь реальные имена из шага 2 — не захардкоженные:

Режим	MODEL_A_NAME	MODEL_B_NAME	Метка
A (primary, v3.9)	`GPT-5.5 via Codex CLI subscription`	`GPT-5.3-Codex via Codex CLI subscription`	—
A-hybrid	`GPT-5.5 via Codex CLI subscription`	`GPT-5.3-Codex via Platform API (openai-review.mjs)`	`⚠️ Reviewer B на Mode A-legacy fallback — gpt-5.3-codex недоступна через subscription`
A-legacy (v3.6 baseline)	`GPT-5.5 via Platform API (openai-review.mjs)` (или `GPT-5.4` если 5.5 недоступна)	`GPT-5.3-Codex via Platform API (openai-review.mjs)`	`⚠️ Mode A-legacy fallback — primary v3.9 path (Codex CLI subscription) недоступен`
C	`Claude Opus 4.7 (standard)`	`Claude Opus 4.7 (adversarial)`	`⚠️ Degraded mode с имитацией adversarial diversity. Не является cross-model review`
D	По факту (например, `GPT-5.4 via Copilot Agent`)	— (single reviewer)	`⚠️ Manual emergency mode. Adversarial diversity и воспроизводимость снижены`

Doc-only skip (под-вариант Mode A / A-legacy, НЕ новый режим): при SKIPPED_REVIEWER_B=1 (Шаг 1.5) Reviewer B (gpt-5.3-codex) не вызывался. MODE остаётся A/A-legacy; MODEL_B_NAME = — (пропущен: doc-only policy research 2026-05-19); секция Reviewer B в шаблоне §5.3 суппрессируется. Это не degradation — Reviewer A (gpt-5.5) прошёл все 4 аспекта.

5.2 Защита findings — raw output в collapsible

Инвариант 6 (PM не искажает findings). PM имеет право структурировать по 4 аспектам и убирать дубликаты, но не перефразировать или смягчать формулировки. Чтобы оператор мог проверить, raw-вывод каждого проверяющего публикуется целиком в <details> блоке отдельно от консолидированного отчёта.

5.3 Шаблон комментария

⚠️ Commit binding — без ** и с META JSON. /finalize-pr ищет маркер через regex Commit:\s*\?ИЛИ JSON"commit": "". Шаблон Commit:(bold markdown) **не** матчится (междуCommit:и хэшем идёт**, не whitespace) — это drift, найденный внешним ревью (round 12). Используй простой Commit: ` и добавь HTML META с JSON — дублирование на случай, если кто-то изменит markdown-форматирование.

HEAD_COMMIT уже зафиксирован в шаге 1.4 — повторно не вычисляем.

Публикация отчёта (quoted heredoc с high-entropy делимитером + bash parameter expansion, чтобы body-содержимое не проходило shell-расширения). Перед запуском команды сгенерируй fresh suffix (openssl rand -hex 8), замени <RAND> в обеих строках делимитера и проверь подготавливаемое тело: выбранный делимитер не должен встречаться как отдельная строка.

BODY=$(cat <<'GH_BODY_<RAND>'
## Внешнее ревью (Sprint Final) — Режим: __MODE__

Commit: `__HEAD_COMMIT__`

<!-- {"reviewer": "__MODEL_A_NAME__ + __MODEL_B_NAME__", "reviewer_a": "__MODEL_A_NAME__", "reviewer_b": "__MODEL_B_NAME__", "commit": "__HEAD_COMMIT__", "kind": "external", "mode": "__MODE__", "iteration": __ITERATION__} -->

<!--
Для режима A primary не добавляй warning-метки — оставь этот блок как есть (в комментарии).
Для режима A-hybrid: вынеси только строку `⚠️ Reviewer B на Mode A-legacy fallback ...`.
Для режима A-legacy: вынеси только строку `⚠️ Mode A-legacy fallback — primary v3.9 path ...`.
Для режима C: вынеси только строку `⚠️ Degraded mode ...`.
Для режима D: вынеси только строку `⚠️ Manual emergency mode ...`.
Не добавляй несколько строк одновременно. Дефолт — безопасный (нет меток).
⚠️ Reviewer B на Mode A-legacy fallback — <описание из таблицы 5.1>
⚠️ Mode A-legacy fallback — <описание из таблицы 5.1>
⚠️ Degraded mode — <описание из таблицы 5.1>
⚠️ Manual emergency mode — <описание из таблицы 5.1>
-->

### Findings (обязательная таблица для /finalize-pr фазы 2 triage)

> Если вердикт APPROVED без замечаний — оставь таблицу с единственной строкой `| — | — | нет замечаний | — | — | — |`. Пустая таблица недопустима: `/finalize-pr` отличает «нет findings» от «парсинг сломался».

| # | Severity | Заголовок | Файл:строка | Статус | Beads ID / Обоснование |
|---|----------|-----------|-------------|--------|------------------------|
| 1 | CRITICAL | ... | path:N | fix now | — |
| 2 | WARNING | ... | path:N | defer to Beads | bd-xyz-123 |
| 3 | INFO | ... | path:N | reject with rationale | <обоснование> |

### Консолидация (PM)

#### Reviewer A: __MODEL_A_NAME__

##### Архитектура: [OK / ISSUE]
[обоснование — дословно из findings, не перефразировать]

##### Безопасность: [OK / ISSUE]
[обоснование]

##### Качество: [OK / ISSUE]
[обоснование]

##### Гигиена кода: [OK / ISSUE]
[обоснование]

**Вердикт:** [APPROVED / CHANGES_REQUESTED / ESCALATION]
— Reviewer (__MODEL_A_NAME__)

---

__REVIEWER_B_SECTION__

---

### Raw output (для аудита оператором)

<details>
<summary>Reviewer A: __MODEL_A_NAME__ — raw</summary>

<pre><code>
<вставить raw-вывод Reviewer A целиком, без редактуры>
</code></pre>

</details>

__REVIEWER_B_RAW__

---

### Copilot
[re-review requested / auto-triggered / unavailable]

### Итоговый вердикт: [APPROVED / CHANGES_REQUESTED / ESCALATION]
CRITICAL: N, WARNING: N

— PM (Claude Opus 4.7), по результатам внешнего ревью
GH_BODY_<RAND>
)
# Вычисление обязательных полей публикации.
# MODE из шага 2 (A / A-hybrid / A-legacy / C / D); MODEL_A_NAME и MODEL_B_NAME — из таблицы 5.1.
# ITERATION — порядковый номер прохода внешнего ревью.
if [ -z "${MODE:-}" ]; then
  echo "MODE не задан. Установи переменную MODE (A / A-hybrid / A-legacy / C / D)." >&2
  exit 1
fi
case "$MODE" in
  A|A-hybrid|A-legacy|C|D) ;;
  *)
    echo "MODE='$MODE' invalid. Допустимые значения: A / A-hybrid / A-legacy / C / D." >&2
    exit 1
    ;;
esac
if [ -z "${MODEL_A_NAME:-}" ]; then
  echo "MODEL_A_NAME не задан. См. таблицу 5.1." >&2
  exit 1
fi
# doc-only skip: Reviewer B штатно не вызывался — БЕЗУСЛОВНО перезаписываем skip-меткой,
# даже если MODEL_B_NAME уже задан по таблице §5.1 (иначе META JSON соврёт, что B отработал).
if [ "${SKIPPED_REVIEWER_B:-0}" = "1" ]; then
  MODEL_B_NAME="— (пропущен: doc-only policy research 2026-05-19)"
fi
if [ -z "${MODEL_B_NAME:-}" ]; then
  # В режиме D Reviewer B отсутствует — допустимо "—".
  # В режимах A / A-hybrid / A-legacy / C без skip Reviewer B обязателен — молчаливый дефолт маскирует ошибку конфигурации.
  if [ "$MODE" = "D" ]; then
    MODEL_B_NAME="—"
  else
    echo "MODEL_B_NAME не задан для режима $MODE, где Reviewer B обязателен. См. таблицу 5.1." >&2
    exit 1
  fi
fi
ITERATION="${ITERATION:-1}"

# Условная (механическая) сборка секции Reviewer B: при doc-only skip — одна честная строка
# без 4 аспектов и raw-details; иначе полный skeleton (плейсхолдеры __MODEL_B_NAME__ заменяются ниже).
if [ "${SKIPPED_REVIEWER_B:-0}" = "1" ]; then
  REVIEWER_B_SECTION="> Reviewer B (gpt-5.3-codex) пропущен — doc-only policy (research 2026-05-19). Single-model gpt-5.5; все 4 аспекта покрыты Reviewer A."
  REVIEWER_B_RAW=""
elif [ "$MODE" = "D" ]; then
  REVIEWER_B_SECTION="> Reviewer B отсутствует — Mode D (manual emergency, single reviewer)."
  REVIEWER_B_RAW=""
else
  REVIEWER_B_SECTION=$(cat <<'RB_SECTION'
#### Reviewer B: __MODEL_B_NAME__

##### Архитектура: [OK / ISSUE]
[обоснование]

##### Безопасность: [OK / ISSUE]
[обоснование]

##### Качество: [OK / ISSUE]
[обоснование]

##### Гигиена кода: [OK / ISSUE]
[обоснование]

**Вердикт:** [APPROVED / CHANGES_REQUESTED / ESCALATION]
— Reviewer (__MODEL_B_NAME__)
RB_SECTION
)
  REVIEWER_B_RAW=$(cat <<'RB_RAW'
<details>
<summary>Reviewer B: __MODEL_B_NAME__ — raw</summary>

<pre><code>
<вставить raw-вывод Reviewer B целиком, без редактуры>
</code></pre>

</details>
RB_RAW
)
fi

# Порядок важен: сначала вставляем блоки Reviewer B (могут нести __MODEL_B_NAME__),
# затем подставляем имена — иначе __MODEL_B_NAME__ внутри вставленного блока не заменится.
BODY="${BODY//__REVIEWER_B_SECTION__/$REVIEWER_B_SECTION}"
BODY="${BODY//__REVIEWER_B_RAW__/$REVIEWER_B_RAW}"
BODY="${BODY//__HEAD_COMMIT__/$HEAD_COMMIT}"
BODY="${BODY//__MODEL_A_NAME__/$MODEL_A_NAME}"
BODY="${BODY//__MODEL_B_NAME__/$MODEL_B_NAME}"
BODY="${BODY//__MODE__/$MODE}"
BODY="${BODY//__ITERATION__/$ITERATION}"
gh pr comment <PR_NUMBER> --body "$BODY"

⚠️ Raw output публикуется без редактуры содержимого, но с HTML-escaping перед вставкой в <pre><code>: заменить & → &, < → <, > → >. Без этого attacker-controlled diff может вернуть модели строку </code></pre> и сломать структуру комментария (закрывает Security ISSUE Reviewer A на 8e7dce6).

Пример экранирования (bash):
RAW_A_ESC=$(printf '%s' "$RAW_A" | sed 's/&/\&amp;/g; s/</\&lt;/g; s/>/\&gt;/g')
# далее используй $RAW_A_ESC внутри <pre><code>...</code></pre> вместо $RAW_A
Если raw слишком длинный — собрать как artifact и приложить ссылкой, но не сокращать пересказом.

Шаг 6: Pre-Chat Gate

Инвариант: review-pass НЕ завершён, пока отчёт не опубликован в PR.

Перед любым сообщением оператору проверь:

gh pr comment выполнился успешно.
Ссылка на комментарий зафиксирована (если доступна).
Только после этого сообщай оператору результат.

⛔ Чат-резюме без PR comment не завершает review-pass.

Шаг 7: Обработка ошибок

7.1 Mode A primary (Codex CLI subscription) errors

Ситуация	Признак	Действие
`429` quota exhausted	stderr содержит `429` или `quota`	Manual swap аккаунта: `Use-CodexAccount work-business` (см. CODEX_AUTH.md §8.5). Retry. Если оба исчерпаны — fallback на Mode A-legacy через перезапуск skill (auto-detected в Шаге 1.4 при перезапуске)
`oauth token exchange failed`	stderr содержит `oauth`	Refresh token истёк. Re-login: `codex logout && codex login`, затем restore из backup при необходимости. Retry
`model X not available on this tier` (Reviewer B)	EXIT_B ≠ 0, stderr содержит `model` + `not available\|tier`	Auto hybrid fallback (Шаг 3.1.2) — Reviewer B через openai-review.mjs Platform API. Mode помечается `A-hybrid`
Profile `[profiles.review]` не найден	stderr содержит `profile` + `not found`	Operator должен добавить profile в `~/.codex/config.toml` (CODEX_AUTH.md §8.4). Skill exit 1 — невозможно continue без profile
Empty output / pipe error	Empty stdout file	Retry × 1; если снова пусто — переход в Mode A-legacy или публикация частичного отчёта с меткой

7.2 Mode A-legacy (openai-review.mjs Platform API) errors

Ситуация	Действие
`openai-review.mjs` exit 1 (локальная или API-ошибка)	Retry × 1; если повторно падает — переход в Mode C или D. Классификация ошибки — в stderr скрипта
`openai-review.mjs` exit 2 (валидация)	Исправить аргументы/модель. См. `--help`
`openai-review.mjs` exit 3 (nothing to review)	Нормальный сигнал — `HEAD == origin/<base>`. Ревью не запускается

7.3 Общие правила для всех режимов

Ситуация	Действие
Пустой вывод от модели	Retry × 1; если снова пусто — переход в Mode A-legacy / Mode C/D или публикация частичного отчёта
Оба проверяющих упали	Эскалация оператору, публикация частичного отчёта с меткой
Один проверяющий упал	Публикация отчёта с пометкой об упавшем проверяющем
Rate limit на всех повторах	Пометить как unavailable, переход вниз по chain (A → A-hybrid → A-legacy → C → D)

Никогда не пропускай ошибку молча. Частичный отчёт лучше, чем отсутствие отчёта.

Шаг 8: Итеративные исправления

Если итоговый вердикт CHANGES_REQUESTED:

PM передаёт CRITICAL и WARNING findings Developer-субагенту.
Developer исправляет → git push.
PM запрашивает Copilot re-review (шаг 4).
PM повторно запускает /external-review <PR_NUMBER> на новом HEAD.
Каждый повторный запуск создаёт новый комментарий (не редактирует старый) — для audit trail.

name	external-review
description	Внешнее ревью PR через Codex CLI ChatGPT subscription (Sprint Pipeline v3.9 Mode A primary). Две полноразмерные модели последовательно — gpt-5.5 (reasoning, primary) + gpt-5.3-codex (code); для doc-only PR Reviewer B (gpt-5.3-codex) пропускается автоматически (§1.5) — single-model gpt-5.5. Fallback модели — gpt-5.4. Fallback режимы — A-legacy (openai-review.mjs Platform API), C (Claude adversarial degraded) и D (manual emergency). Используй `/external-review <PR_NUMBER>`.
user-invocable	true

External Review — кросс-модельное ревью через Codex CLI ChatGPT subscription

Источник подхода: Adversarial Code Review для Claude Code

Backend evolution: v3.6 — openai-review.mjs Platform API primary; v3.9 — Codex CLI subscription primary, openai-review.mjs сохраняется как Mode A-legacy. Решение: .agents/PIPELINE_ADR.md ADR 3.27 (cost-driven: $0/вызов в пределах квоты vs $20-22/вызов на Platform API). Decision keep both reviewers подтверждён данными big-heroes/surprise-arena: 22.7% unique-to-Codex Critical/High findings (см. docs/research/2026-05-05-codex-overlap-analysis.md, U2-275). U2-native анализ 2026-05-19 уточняет invocation policy: GPT-5.3-Codex остаётся Reviewer B для Critical/Sprint Final/runtime/tooling PR, но не вызывается по умолчанию для Light/doc-only/landing проходов (docs/research/2026-05-19-gpt55-vs-gpt53-codex-u2-review-effectiveness.md).

Контекст

Прочитай перед началом:

.agents/AGENT_ROLES.md секция «3. Reviewer» — формат вердикта (4 аспекта).
.memory-bank/activeContext.md — текущее состояние проекта.
.agents/CODEX_AUTH.md §8 — ChatGPT subscription path setup и validated invocation pattern.
.agents/PIPELINE_ADR.md ADR 3.27 — design rationale для Codex CLI subscription backend.
docs/research/2026-05-19-gpt55-vs-gpt53-codex-u2-review-effectiveness.md — U2-specific effectiveness stats and Reviewer B invocation policy.
docs/archive/pipeline/2026-05-05-u2-rhm-external-review-codex-migration.md — архивный handoff-документ миграции v3.9 (план завершён) с Section A-K описывающим backend migration. Для деталей runtime allowlist Mode A-legacy и endpoint dispatch — см. .claude/tools/openai-review.mjs (--help + sources) и .agents/CODEX_AUTH.md §1-§7 (Platform API setup).

Аргументы

PR_NUMBER — номер PR для ревью (обязательный).
--legacy — explicit override на Mode A-legacy (openai-review.mjs Platform API), bypass Mode A primary (Codex CLI subscription) даже если subscription доступна. Use case: self-modification PRs которые меняют .claude/skills/*, .agents/*, .codex/, или AGENTS.md — Mode A primary имеет known trust boundary gap для них (Beads U2-me5, P1; см. Шаг 3.1 KNOWN LIMITATION warning).
--manual — explicit Mode D ручной fallback (per Шаг 2 mode table).
--doc — ручной override/страховка, allow-list, не enumerate-negative: форсирует skip Reviewer B (single-model gpt-5.5) только когда HAS_EXECUTABLE = "gh_diff_failed_fail_closed" — авто-классификатор (Шаг 1.5) реально недоступен, gh pr diff упал. Любой другой scope отказывает: реальный executable/governance/config, unknown_extension_fail_closed (диф прочитан, классификатор доступен, конкретный файл просто не опознан как known-harmless), и любой прочий non-md scope (в т.ч. known-harmless non-md вроде .png/.txt/.csv — диф не строго .md, значит не «doc-only» в терминах этого флага). НЕ меняет floor (floor всегда по фактическому scope). На чистом doc-only diff флаг не нужен — авто-детект (Шаг 1.5) уже скипает Reviewer B.

Argument parsing (выполняется первым в Шаге 1):

PR_NUMBER=""
FORCE_LEGACY=0
FORCE_MANUAL=0
FORCE_DOC=0
for arg in "$@"; do
  case "$arg" in
    --legacy) FORCE_LEGACY=1 ;;
    --manual) FORCE_MANUAL=1 ;;
    --doc) FORCE_DOC=1 ;;
    [0-9]*) PR_NUMBER="$arg" ;;
    *) echo "ВНИМАНИЕ: неизвестный аргумент '$arg' игнорируется." >&2 ;;
  esac
done
if [ -z "$PR_NUMBER" ]; then
  echo "СТОП: PR_NUMBER обязателен. Usage: /external-review <PR_NUMBER> [--legacy|--manual|--doc]" >&2
  exit 1
fi

Шаг 1: Пререквизиты

1.1 Проверка чистоты рабочего дерева

Проверяется ПЕРЕД любыми операциями (git fetch, ref creation), чтобы избежать конфликтов и потери локальных изменений. После trust boundary fix (Шаг 7.6 Plan A) gh pr checkout больше не выполняется — pipeline остаётся в trusted base, но clean tree всё равно требуется для предсказуемого git fetch и операций с локальными refs.

if [ -n "$(git status --porcelain)" ]; then
  echo "СТОП: рабочее дерево не чистое. Закоммить или stash изменения."
  exit 1
fi

1.2 Получение PR-метаданных + fetch PR head как trusted ref (БЕЗ checkout)

Trust boundary (Шаг 7.6 Plan A): tool НЕ запускается из ветки PR в Mode A-legacy. Раньше gh pr checkout переключал cwd в untrusted ветку, и node openai-review.mjs в ней мог быть подменён PR-автором с доступом к $OPENAI_API_KEY. Сейчас pipeline остаётся в trusted base/default branch, PR код только читается через diff с fetched ref.

В Mode A primary (Codex CLI subscription) trust boundary не критичен: Codex CLI — глобальная установка (@openai/codex npm install -g), бинарник в $AppData\Roaming\npm\codex.cmd или /usr/local/bin/codex, не из cwd. PR-автор не может подменить codex binary через PR. Однако clean tree всё равно требуется для предсказуемого git fetch и git checkout --detach (если используется --base invocation).

BASE_BRANCH=$(gh pr view <PR_NUMBER> --json baseRefName --jq '.baseRefName')
STATE=$(gh pr view <PR_NUMBER> --json state --jq '.state')

if [ "$STATE" != "OPEN" ]; then echo "СТОП: PR не открыт"; exit 1; fi

# Fetch PR head как локальный ref (БЕЗ checkout — cwd остаётся в trusted base).
# refspec `pull/<PR>/head` — это GitHub-специфичный ref для PR head независимо от source repo.
PR_REF="u2-pr-<PR_NUMBER>"
git fetch origin "+pull/<PR_NUMBER>/head:refs/heads/${PR_REF}"

1.3 Проверка что ветка запушена

if ! git rev-parse --abbrev-ref '@{u}' >/dev/null 2>&1; then
  echo "СТОП: upstream не настроен. Выполни: git push -u origin $(git branch --show-current)"
  exit 1
fi

UNPUSHED=$(git log @{u}..HEAD --oneline)
if [ -n "$UNPUSHED" ]; then
  echo "СТОП: есть незапушенные коммиты. Выполни git push."
  exit 1
fi

1.4 Pre-flight: проверка backend availability, фиксация HEAD_COMMIT

Двухступенчатый check (v3.9): сначала проверяем Codex CLI ChatGPT subscription (primary), затем — openai-review.mjs Platform API (Mode A-legacy fallback). Если оба недоступны → degraded path (Mode C/D).

# Фиксация HEAD_COMMIT сразу — используется далее в именах артефактов (Шаг 3.1)
# и в commit binding публикации (Шаг 5.3). Единое значение для всего прохода ревью.
HEAD_COMMIT=$(timeout 10 gh pr view <PR_NUMBER> --json headRefOid --jq '.headRefOid')
if [[ -z "$HEAD_COMMIT" || "$HEAD_COMMIT" == "null" || ! "$HEAD_COMMIT" =~ ^[0-9a-fA-F]{40}$ ]]; then
  echo "СТОП: не удалось получить валидный HEAD commit для PR <PR_NUMBER>" >&2
  exit 1
fi

# Trust boundary verify: убедиться что cwd = trusted base (default branch или
# другая базовая ветка), НЕ ветка PR. Защищает Mode A-legacy (openai-review.mjs из cwd).
# Mode A primary не страдает от cwd-substitution (codex CLI — глобальный binary), но
# clean trusted base всё равно нужен для idempotent git операций.
CURRENT_BRANCH=$(git branch --show-current)
if [ "$CURRENT_BRANCH" != "$BASE_BRANCH" ]; then
  echo "СТОП: external-review должен запускаться из trusted base ('$BASE_BRANCH'), текущая ветка '$CURRENT_BRANCH'." >&2
  echo "Сначала: git switch '$BASE_BRANCH' && git pull" >&2
  exit 1
fi

MODE_A_AVAILABLE=0
MODE_PRIMARY=""
MODE_A_LEGACY_AVAILABLE=0

# --- Operator override: --manual флаг force Mode D (manual emergency) ---
# Bypass всю automation chain, route напрямую в Mode D. Use case: incident response,
# forced risk-acceptance runs, или когда automated paths сломаны.
if [ "$FORCE_MANUAL" = "1" ]; then
  echo "Operator override --manual: bypass automation, force Mode D (manual emergency)."
  MODE_A_AVAILABLE=0
  MODE="D"
  # Skip всю availability detection — оператор вручную провёл review external tool'ом.
# --- Operator override: --legacy флаг bypass Mode A primary ---
# Для self-modification PRs которые меняют .claude/skills/*, .agents/*, .codex/,
# AGENTS.md — Mode A primary имеет fundamental trust boundary gap (Beads U2-me5).
# Operator-aware override force routing на Mode A-legacy (openai-review.mjs runs from
# trusted base, читает diff через --head refspec без cwd contents loading).
elif [ "$FORCE_LEGACY" = "1" ]; then
  echo "Operator override --legacy: bypass Mode A primary, force Mode A-legacy."
  # Skip Codex CLI check полностью, идём сразу к Mode A-legacy availability check ниже.
else
  # --- Step 1: Codex CLI ChatGPT subscription (primary) ---
  # Бинарник `codex` должен быть в PATH (npm install -g @openai/codex@latest, ≥0.128.0).
  # `codex login status` возвращает строку "Logged in using ChatGPT" если subscription active.
  # Profile [profiles.review] из ~/.codex/config.toml должен существовать (см. CODEX_AUTH.md §8.4).
  if command -v codex >/dev/null 2>&1; then
    CODEX_LOGIN_STATUS=$(timeout 10 codex login status 2>&1 || true)
    if echo "$CODEX_LOGIN_STATUS" | grep -q "Logged in using ChatGPT"; then
      # Profile validation (per external review iter 4+5 findings): убедиться что
      # [profiles.review] существует и загружается. Без этого Mode A primary будет
      # помечен available, но invocation в Шаге 3.1 упадёт с "profile not found" вместо
      # graceful fallback на Mode A-legacy.
      #
      # Iter 5 finding: `codex -p review --version` exit 0 даже когда profile missing
      # (--version не загружает profile). Только `exec` действительно loads profile config.
      # Используем `exec` only (no OR) для точной валидации.
      if timeout 10 codex -p review exec "echo profile-ok" >/dev/null 2>&1; then
        MODE_A_AVAILABLE=1
        MODE_PRIMARY="chatgpt"
        echo "Mode A primary: Codex CLI ChatGPT subscription + profile [review] доступны."
      else
        echo "ВНИМАНИЕ: Codex CLI logged in, но profile [profiles.review] не загружается."
        echo "Проверь ~/.codex/config.toml (см. CODEX_AUTH.md §8.4)."
        echo "Проверка fallback на Mode A-legacy (openai-review.mjs Platform API)..."
      fi
    else
      echo "ВНИМАНИЕ: Codex CLI установлен, но subscription path недоступен. Status: $CODEX_LOGIN_STATUS"
      echo "Проверка fallback на Mode A-legacy (openai-review.mjs Platform API)..."
    fi
  else
    echo "ВНИМАНИЕ: codex CLI не в PATH. Проверка Mode A-legacy fallback..."
  fi
fi

# --- Step 2: Mode A-legacy fallback (openai-review.mjs Platform API) ---
# Запускается только если Mode A primary недоступен. Установка openai SDK
# и валидация $OPENAI_API_KEY через --ping.
if [ "$MODE_A_AVAILABLE" = "0" ]; then
  if [ ! -d .claude/tools/node_modules ]; then
    # npm ci (не install): строго по lockfile, не пачкает рабочее дерево, воспроизводимо.
    # --ignore-scripts оставлен как defence-in-depth (даже trusted package может иметь
    # неожиданные lifecycle hooks).
    if ! (cd .claude/tools && npm ci --ignore-scripts); then
      echo "ВНИМАНИЕ: npm ci упал в .claude/tools/ — Mode A-legacy недоступен."
    fi
  fi

  # Ранняя проверка валидности $OPENAI_API_KEY (<2 сек, AC6 плана) — только если SDK установлен.
  if [ -d .claude/tools/node_modules ]; then
    if node .claude/tools/openai-review.mjs --ping; then
      MODE_A_AVAILABLE=1
      MODE_PRIMARY="legacy"
      MODE_A_LEGACY_AVAILABLE=1
      echo "Mode A-legacy: openai-review.mjs Platform API доступен (Codex CLI subscription unavailable, используем fallback)."
    else
      echo "ВНИМАНИЕ: Mode A-legacy недоступен (невалидный ключ / rate limit / network)."
      echo "Скрипт перейдёт в degraded-режим C. Детали ошибки — на stderr выше."
    fi
  fi
fi

# Если оба primary и legacy недоступны → MODE_A_AVAILABLE=0 → fallback C/D.
if [ "$MODE_A_AVAILABLE" = "0" ]; then
  echo "ВНИМАНИЕ: Mode A primary (Codex CLI subscription) и Mode A-legacy (openai-review.mjs) оба недоступны."
fi

Что детектируется на каждом шаге:

Codex CLI primary check fail: бинарник не в PATH; codex login status не возвращает Logged in using ChatGPT (logged out, refresh token expired, OAuth broken); profile review не зарегистрирован в ~/.codex/config.toml (косвенно через invocation в Шаге 3.1 — здесь не проверяем).
Mode A-legacy fail: npm ci не отработал; --ping exit ≠ 0 (невалидный ключ, rate limit, network); подробности через stderr скрипта (см. .claude/tools/README.md таблицу exit codes).

1.5 Pre-flight scope classification (единый источник для skip-политики и floor/cap)

Почему в pre-flight, а не в §3.5.2: Reviewer B (§3.1) вызывается ДО convergence-детекта (§3.5), поэтому чтобы применить doc-only политику (research 2026-05-19: для DOC_ONLY не звать gpt-5.3-codex) на ПЕРВОЙ итерации, scope должен быть известен здесь. Это единственный источник классификации — §3.5.2 переиспользует эти же переменные (single source of truth, защита от drift regex-паттернов).

DIFF_FILES=$(gh pr diff "$PR_NUMBER" --name-only 2>&1)
GH_EXIT=$?

if [ "$GH_EXIT" -ne 0 ] || [ -z "$DIFF_FILES" ]; then
  # Fallback ДО fail-closed sentinel (iter-6 external review finding, PR #492): PR_REF уже
  # fetch'нут локально в Шаге 1.2 (`git fetch origin +pull/<N>/head:refs/heads/u2-pr-<N>`),
  # поэтому при сбое именно gh API (rate limit / network / auth) файл-лист часто доступен
  # локально без обращения к GitHub API. Уменьшает случаи, когда PM вынужден полагаться
  # на ручное суждение через --doc: если fallback дал реальный список — классификатор
  # отрабатывает НОРМАЛЬНО (не sentinel), --doc в этом раунде не нужен вовсе.
  echo "ВНИМАНИЕ: gh pr diff упал (exit=$GH_EXIT) или пустой список. Пробую fallback: локальный git diff по уже fetch'нутому PR_REF."
  # Освежаем base ref перед диффом (та же практика, что в §3.1) — без этого локальный
  # origin/$BASE_BRANCH может быть устаревшим относительно текущего состояния GitHub,
  # и fallback-дифф разойдётся с тем, что реально видит gh. Сбой самого fetch — не фатален
  # (git diff ниже использует что есть локально), не блокирует fallback.
  git fetch origin "+refs/heads/$BASE_BRANCH:refs/remotes/origin/$BASE_BRANCH" 2>&1 || true
  DIFF_FILES=$(git diff --name-only "origin/$BASE_BRANCH...$PR_REF" 2>&1)
  GIT_EXIT=$?
  if [ "$GIT_EXIT" -ne 0 ] || [ -z "$DIFF_FILES" ]; then
    echo "ВНИМАНИЕ: локальный git diff тоже упал (exit=$GIT_EXIT) или пуст. Fail-closed: HAS_EXECUTABLE, CAP=20, SKIP_REVIEWER_B=0."
    EXEC_REAL=""                          # реального совпадения нет — поднятие по неопределённости
    HAS_EXECUTABLE="gh_diff_failed_fail_closed"
    HAS_CONFIG=""
    DOC_ONLY="fail_closed"
    CAP=20
  else
    echo "Fallback успешен: DIFF_FILES получен через локальный git diff (gh API недоступен, но локальный ref рабочий)."
  fi
fi

if [ -n "${HAS_EXECUTABLE:-}" ] && [ "$HAS_EXECUTABLE" = "gh_diff_failed_fail_closed" ]; then
  : # оба источника (gh API и локальный git) недоступны — sentinel уже выставлен выше, классификация ниже не нужна
else
  # EXEC_REAL — РЕАЛЬНОЕ совпадение executable/governance-путей (код, ^.claude/, ^.agents/, ^.codex/,
  # AGENTS.md/CLAUDE.md, .gitignore/.gitattributes). Тот же набор, что detection --legacy в Шаге 1.4.
  EXEC_REAL=$(echo "$DIFF_FILES" | grep -E '\.(py|ts|cs|js|jsx|tsx|mjs|cjs|mts|sh|ps1|bat|go|rs|java|kt|rb|php|swift|html|css|svg)$|(^|/)Dockerfile$|^\.claude/|^\.agents/|^\.codex/|(^|/)AGENTS\.md$|(^|/)CLAUDE\.md$|^\.gitignore$|^\.gitattributes$' | head -1)
  HAS_EXECUTABLE="$EXEC_REAL"

  # Fail-closed: unknown extension (вне known-harmless whitelist) → escalate в executable (sentinel, НЕ EXEC_REAL).
  KNOWN_HARMLESS=$(echo "$DIFF_FILES" | grep -E '\.(md|json|yaml|yml|toml|txt|csv|tsv|png|jpg|jpeg|gif|webp|ico)$|^\.gitignore$|^\.gitattributes$' | wc -l)
  TOTAL_FILES=$(echo "$DIFF_FILES" | wc -l)
  if [ "$KNOWN_HARMLESS" -lt "$TOTAL_FILES" ] && [ -z "$HAS_EXECUTABLE" ]; then
    echo "ВНИМАНИЕ: файлы вне known-harmless whitelist. Fail-closed: escalate в executable scope."
    HAS_EXECUTABLE="unknown_extension_fail_closed"
  fi

  # Config: остальные .json/.yaml/.yml/.toml (кроме .claude/settings.json — оно governance).
  HAS_CONFIG=$(echo "$DIFF_FILES" | grep -vE '^\.claude/settings\.json$' | grep -E '\.(json|yaml|yml|toml)$' | head -1)
  # DOC_ONLY: инвертирована — НЕПУСТО ⟺ есть не-.md файл; ПУСТО ⟺ все файлы .md.
  DOC_ONLY=$(echo "$DIFF_FILES" | grep -vE '\.md$' | head -1)

  if [ -n "$HAS_EXECUTABLE" ]; then
    CAP=20
  elif [ -n "$HAS_CONFIG" ]; then
    CAP=12
  elif [ -z "$DOC_ONLY" ]; then
    CAP=7
  else
    CAP=12  # mixed default
  fi
fi
echo "Scope: EXEC_REAL='${EXEC_REAL:-}' HAS_EXECUTABLE='${HAS_EXECUTABLE:-}' HAS_CONFIG='${HAS_CONFIG:-}' DOC_ONLY_nonmd='${DOC_ONLY:-}' CAP=$CAP"

# --- Reviewer B skip policy (research 2026-05-19) ---
# ⚠️ ИНВАРИАНТ: DOC_ONLY инвертирована (см. выше) — skip требует ПУСТОГО DOC_ONLY: только [ -z ], НИКОГДА [ -n ].
# Governance-.md (^.claude/, ^.agents/, ^.codex/, AGENTS.md, CLAUDE.md) уже в HAS_EXECUTABLE по пути —
# поэтому self-modification PR НЕ проходит в skip-ветку (получает обе модели).
SKIP_REVIEWER_B=0
SKIPPED_REVIEWER_B=0  # явная инициализация каждого прогона — защита от stale-значения из предыдущего /external-review в той же shell-сессии
if [ -z "$HAS_EXECUTABLE" ] && [ -z "$HAS_CONFIG" ] && [ -z "$DOC_ONLY" ]; then
  SKIP_REVIEWER_B=1
  echo "AUTO doc-only scope: Reviewer B (gpt-5.3-codex) пропущен по политике research 2026-05-19 (только gpt-5.5)."
fi
# --doc — УЗКИЙ override, ALLOW-LIST (не enumerate-negative): единственный легитимный сценарий —
# классификатор реально недоступен из-за сбоя самого `gh pr diff`
# (HAS_EXECUTABLE="gh_diff_failed_fail_closed"). Только это разрешено — ЛЮБОЙ другой scope
# (EXEC_REAL, HAS_CONFIG, unknown_extension_fail_closed, известные-harmless non-md типа
# .png/.txt/.csv, mixed doc+non-md) отказывает. Allow-list вместо enumerate-negative закрывает
# и уже известные, и ещё не найденные варианты того же класса «диф не является строго .md,
# но --doc всё равно скипает» (iter-3 High + iter-5 Medium: обе находки — enumerate-negative
# пропускал ровно то, что не было явно перечислено). Governance/self-mod PR через --doc НЕ скипается.
if [ "$FORCE_DOC" = "1" ]; then
  if [ "$HAS_EXECUTABLE" = "gh_diff_failed_fail_closed" ]; then
    SKIP_REVIEWER_B=1
    echo "--doc принят: снята неопределённость сбоя gh pr diff, Reviewer B пропущен (только gpt-5.5)."
  else
    echo "ОТКАЗ: --doc применяется только когда единственная причина эскалации — сбой самого gh pr diff (gh_diff_failed_fail_closed). Текущий scope: EXEC_REAL='${EXEC_REAL:-}' HAS_CONFIG='${HAS_CONFIG:-}' HAS_EXECUTABLE='${HAS_EXECUTABLE:-}' DOC_ONLY_nonmd='${DOC_ONLY:-}'. Reviewer B вызывается."
  fi
fi

⚠️ Skip ≠ понижение floor. SKIP_REVIEWER_B управляет лишь тем, звать ли gpt-5.3-codex. Floor итераций (§3.5.1) — строго по scope diff (fail-closed: HAS_EXECUTABLE → floor 5 всегда, даже если Reviewer B не отработал). --doc НЕ опускает floor. Инвариант «floor следует scope, не доступности модели» сохранён.

Для doc-only scope Mode A остаётся Mode A (не новый режим и не degradation) — просто Reviewer B не вызывается.

Шаг 2: Определение режима работы

Режим A primary: Codex CLI ChatGPT subscription (основной путь, v3.9)

Если MODE_A_AVAILABLE=1 И MODE_PRIMARY="chatgpt" из Шага 1.4:

Reviewer A (primary): codex -p review -c sandbox_mode="danger-full-access" review --commit "$HEAD_COMMIT" — модель gpt-5.5 берётся из [profiles.review] (см. .agents/CODEX_AUTH.md §8.4). Reasoning высокий, MCP_DOCKER подавлен через mcp_servers = {} table-replace в profile.
Reviewer B: codex -p review -c sandbox_mode="danger-full-access" -c model="gpt-5.3-codex" review --commit "$HEAD_COMMIT" — -c model="gpt-5.3-codex" перебивает default из profile. Если gpt-5.3-codex недоступна через subscription tier (выясняется по exit code и stderr CLI; типичная ошибка model X not available on this tier) — Reviewer B автоматически переключается на Mode A-legacy через openai-review.mjs --model gpt-5.3-codex (см. Шаг 3.1.2 hybrid fallback).
Запуск sequential, НЕ parallel — два concurrent codex вызова конфликтуют через shared ~/.codex/auth.json lock. Sequential добавляет ~30-60s wall-clock, но stable.
Quota burn: ChatGPT subscription quota — fixed monthly ($25 Plus / $100 Pro / $30/seat Business). $0/вызов в пределах квоты. Quota check — UI: chatgpt.com Settings → Subscriptions → Usage (нет CLI API).
На 429 quota exhausted: ручной switch второго аккаунта через .claude/tools/codex-account-switch.ps1 (Use-CodexAccount work-business), затем retry. Если оба исчерпаны → fallback на Mode A-legacy (см. Шаг 3.1.2).
В отчёте: — Reviewer (GPT-5.5 via Codex CLI subscription) и — Reviewer (GPT-5.3-Codex via Codex CLI subscription). Для doc-only scope (Шаг 1.5 SKIP_REVIEWER_B=1) Reviewer B не вызывается — только строка Reviewer A (gpt-5.5); Mode A остаётся Mode A, не degradation.

Codex CLI выводит markdown report на stdout. Exit codes: 0 OK, ≠0 runtime/auth/model error.

Режим A-legacy: Node.js native через OpenAI SDK (auto-fallback v3.6 path)

Если MODE_A_AVAILABLE=1 И MODE_PRIMARY="legacy" из Шага 1.4 (Codex CLI subscription недоступен, Platform API доступен):

Reviewer A: node .claude/tools/openai-review.mjs --model gpt-5.5 --base "$BASE_BRANCH" --head "$PR_REF" — reasoning-специализация (/v1/chat/completions с reasoning_effort: "high"). Fallback: --model gpt-5.4 если 5.5 недоступна.
Reviewer B: node .claude/tools/openai-review.mjs --model gpt-5.3-codex --base "$BASE_BRANCH" --head "$PR_REF" — code-специализация (/v1/responses с reasoning.effort: "high").
Две полноразмерные модели разных архитектур = максимальная adversarial diversity. Платформа: OpenAI Platform API (pay-per-token, $OPENAI_API_KEY).
В отчёте: — Reviewer (GPT-5.5 Platform API) и — Reviewer (GPT-5.3-Codex Platform API) + label ⚠️ Mode A-legacy fallback — primary v3.9 path (Codex CLI subscription) недоступен.

Режим C: Все Mode A варианты недоступны (degraded, Claude adversarial)

Если MODE_A_AVAILABLE=0 (Codex CLI subscription и Platform API оба недоступны), а оператор явно не выбрал ручной fallback:

Два прохода Claude-субагента против одного и того же PR:
- Проход 1 (стандартный): Reviewer в обычном режиме, промпт как в Standard tier.
- Проход 2 (adversarial): Reviewer с промптом «Ты adversarial reviewer. Твоя задача — найти то, что первый проход пропустил. Ищи baseline-слабые места: необработанные edge-cases, скрытые дубликаты, некорректные инварианты».
⚠️ В отчёте обязательна метка: «⚠️ Degraded mode с имитацией adversarial diversity. Не является cross-model review».
Атрибуция: — Reviewer (Claude Opus 4.7, standard) и — Reviewer (Claude Opus 4.7, adversarial).

Режим D: Автоматические пути недоступны (manual emergency)

Если все Mode A варианты недоступны И оператор требует ручного fallback (например, gpt-5.4 через VS Code GitHub Copilot Agent):

Оператор вручную прогоняет PR через внешний инструмент.
PM собирает вывод в текстовом виде и публикует с меткой: «⚠️ Manual emergency mode. Adversarial diversity и воспроизводимость снижены».
Атрибуция — по фактическому источнику (— Reviewer (GPT-5.4 via Copilot Agent)).

Таблица выбора режима

Режим	Условие запуска	Reviewer A	Reviewer B
A (primary, v3.9)	`codex login status` → `Logged in using ChatGPT` И profile `[profiles.review]` загружается	gpt-5.5 (Codex CLI subscription)	gpt-5.3-codex (Codex CLI subscription, может fallback на Platform API при tier limit)
A-legacy	Codex CLI недоступен И `openai-review.mjs --ping` → exit 0	gpt-5.5 Platform API (или gpt-5.4 fallback)	gpt-5.3-codex Platform API
C	Все Mode A варианты недоступны, оператор не указал ручной fallback	Claude Opus 4.7 standard	Claude Opus 4.7 adversarial
D	Оператор явно указал ручной fallback (`--manual` или чат-команда)	По факту (Copilot Agent / др.)	— (single reviewer)

В любом режиме каждый проверяющий проходит все 4 аспекта. Различается источник ревью и атрибуция.

Doc-only scope (Шаг 1.5 SKIP_REVIEWER_B=1): в режимах A / A-legacy Reviewer B (gpt-5.3-codex) не вызывается — single-model gpt-5.5. Это под-вариант того же режима (MODE не меняется), не шестой режим и не degradation; Reviewer A проходит все 4 аспекта.

Шаг 3: Запуск проверяющих

3.1 Режим A primary — через Codex CLI subscription (v3.9, основной путь)

Sequential, НЕ parallel. Два concurrent codex вызова могут конфликтовать через shared ~/.codex/auth.json (refresh token rotation, OAuth state). Sequential добавляет ~30-60s wall-clock, но даёт stable result.

Default invocation = --base "origin/$BASE_BRANCH" после git checkout --detach $PR_REF (per external review iter 1 finding). Это покрывает PR полностью независимо от количества коммитов. --commit "$HEAD_COMMIT" — opt-in для smoke/single-commit cases (см. 3.1.x альтернатива ниже). Sprint Final review должен видеть весь diff PR'а, не только последний commit.

⚠️ KNOWN LIMITATION (Beads U2-me5, P1): Mode A primary имеет fundamental trust boundary gap для self-modification PRs — PRs которые меняют .claude/skills/*, .agents/*, .codex/, или AGENTS.md. Codex CLI loads these files из cwd regardless of git ref selection через --base/--commit, так что detached checkout в PR head exposes reviewer к PR-controlled agent configuration. PR-author мог бы подменить AGENTS.md в своём PR и biased/suppressed Codex review behavior.

Mitigation до fix U2-me5: для self-modification PRs operator должен explicitly override на Mode A-legacy через /external-review <PR_NUMBER> --legacy (manual flag). Mode A-legacy через openai-review.mjs runs from trusted base, читает diff через --head refspec без cwd contents loading — trust boundary preserved.

Detection: gh pr diff <PR_NUMBER> --name-only | grep -qE '^(\.claude/|\.agents/|\.codex/|AGENTS\.md)' → если true, рекомендуется Mode A-legacy. PR #191 (этот sprint) сам является self-modification PR — iter 1-3 strictly executed под partial trust compromise; mitigating factor что operator-authored PR не имеет malicious intent + reviewers continued surfacing real issues каждой iter (compromise не achieved practically).

# Создать каталог для артефактов (если ещё нет):
# .review-responses/ исключён из git (см. .gitignore) — raw outputs не коммитятся.
mkdir -p .review-responses

# Refresh base ref ПЕРЕД detached checkout — защита от stale `origin/$BASE_BRANCH`
# (per external review iter 2 finding). Если base branch продвинулся на GitHub
# с последнего fetch — без этого Mode A primary review against устаревший remote-tracking
# ref, possibly missing conflicts с current base или surfacing already-merged code.
# `+refs/heads/$BASE_BRANCH:refs/remotes/origin/$BASE_BRANCH` — explicit refspec
# (устойчиво к пользовательским remote.origin.fetch).
git fetch origin "+refs/heads/$BASE_BRANCH:refs/remotes/origin/$BASE_BRANCH"

# Detached checkout PR head — НЕ branch switch (trust boundary preserved для git refs).
# Codex CLI читает diff origin/$BASE_BRANCH..HEAD из git, поэтому detached HEAD = PR head
# даёт правильный diff scope. Codex CLI binary — глобальный (не из cwd), trust boundary
# preserved для primary path. Однако cwd contents = PR head state — это критично для
# любого fallback который запускает скрипты из cwd (см. Шаг 3.1.2 hybrid fallback —
# обязательный switch обратно в trusted base перед exec'ом openai-review.mjs).
git checkout --detach "$PR_REF"

# Reviewer A: gpt-5.5 (default из [profiles.review])
# Validated invocation pattern (CODEX_AUTH.md §8.3 + ADR 3.27):
# `-p review` активирует profile (model=gpt-5.5, reasoning=high, MCP_DOCKER suppressed).
# `-c sandbox_mode=...` обязателен — profile-level sandbox_mode игнорируется (Codex CLI 0.128.0 quirk).
# `review --base "origin/$BASE_BRANCH"` покрывает весь PR diff (multi-commit safe).
codex -p review -c sandbox_mode="danger-full-access" review --base "origin/$BASE_BRANCH" \
  > .review-responses/mode-a-reviewer-a-"$HEAD_COMMIT".md \
  2> .review-responses/mode-a-reviewer-a-"$HEAD_COMMIT".err
EXIT_A=$?

if [ "$EXIT_A" -ne 0 ]; then
  echo "ВНИМАНИЕ: Reviewer A (Codex CLI gpt-5.5) упал с exit $EXIT_A. stderr: .review-responses/mode-a-reviewer-a-$HEAD_COMMIT.err"
  # На 429: ручной operator action — `Use-CodexAccount work-business` (см. CODEX_AUTH.md §8.5).
  # Эту проверку реализуй inline: если grep -q '429\|quota' в .err → подсказка в stderr.
fi

# Reviewer B: gpt-5.3-codex через -c model override.
# Пропускается для doc-only scope (SKIP_REVIEWER_B=1 из Шага 1.5, research 2026-05-19).
if [ "$SKIP_REVIEWER_B" = "1" ]; then
  echo "doc-only scope: Reviewer B (gpt-5.3-codex) пропущен по политике research 2026-05-19; только gpt-5.5."
  EXIT_B=0            # штатный skip, НЕ ошибка → hybrid fallback (§3.1.2) НЕ активируется
  SKIPPED_REVIEWER_B=1
else
  SKIPPED_REVIEWER_B=0  # defensive-сброс: реальный вызов Reviewer B, не skip (защита от stale-значения предыдущего прогона)
  # `-c model="gpt-5.3-codex"` перебивает default из profile.
  # Если model недоступна на subscription tier → Codex CLI вернёт error на stderr и exit ≠ 0.
  codex -p review -c sandbox_mode="danger-full-access" -c model="gpt-5.3-codex" review --base "origin/$BASE_BRANCH" \
    > .review-responses/mode-a-reviewer-b-"$HEAD_COMMIT".md \
    2> .review-responses/mode-a-reviewer-b-"$HEAD_COMMIT".err
  EXIT_B=$?

  if [ "$EXIT_B" -ne 0 ]; then
    echo "ВНИМАНИЕ: Reviewer B (Codex CLI gpt-5.3-codex) упал с exit $EXIT_B."
    echo "Возможная причина: gpt-5.3-codex не доступна на текущем ChatGPT subscription tier."
    echo "→ Hybrid fallback на Mode A-legacy для Reviewer B (см. Шаг 3.1.2)."
  fi
fi

Альтернативный invocation для smoke/single-commit cases:

# Single-commit review — для smoke-тестов или fixup-only PRs.
# НЕ используй для Sprint Final multi-commit PR — пропустит earlier commits.
codex -p review -c sandbox_mode="danger-full-access" review --commit "$HEAD_COMMIT" \
  > .review-responses/mode-a-reviewer-a-"$HEAD_COMMIT".md \
  2> .review-responses/mode-a-reviewer-a-"$HEAD_COMMIT".err
# (Reviewer B аналогично с -c model="gpt-5.3-codex" --commit "$HEAD_COMMIT")

3.1.2 Hybrid fallback: Reviewer B → Mode A-legacy (если codex CLI не даёт gpt-5.3-codex)

Размещён ДО cleanup (3.1.3) per external review iter 1 finding — hybrid fallback использует --head "$PR_REF", поэтому ref должен быть жив до завершения fallback.

# Условие активации hybrid fallback:
# 1. EXIT_A == 0 (Reviewer A через Codex subscription удался)
# 2. EXIT_B != 0 И stderr содержит "model" + "not available|not supported|tier"
# 3. MODE_A_LEGACY_AVAILABLE=1 ИЛИ openai-review.mjs --ping проходит
HYBRID_FALLBACK_NEEDED=0
HYBRID_FALLBACK_FAILED=0  # для partial-report маркировки если npm ci упал
if [ "${SKIPPED_REVIEWER_B:-0}" != "1" ] && [ "$EXIT_A" = "0" ] && [ "$EXIT_B" -ne 0 ]; then
  if grep -qE 'model.*(not available|not supported|tier)' ".review-responses/mode-a-reviewer-b-$HEAD_COMMIT.err"; then
    HYBRID_FALLBACK_NEEDED=1
  fi
fi

if [ "$HYBRID_FALLBACK_NEEDED" = "1" ]; then
  echo "Hybrid fallback: Reviewer B → openai-review.mjs Platform API (gpt-5.3-codex недоступна на subscription tier)."

  # КРИТИЧНО (per external review iter 2 CRITICAL finding): SWITCH BACK В TRUSTED BASE
  # перед exec'ом openai-review.mjs. После Шага 3.1 cwd находится в detached HEAD = PR head,
  # т.е. в untrusted PR contents. Запуск `node .claude/tools/openai-review.mjs` оттуда
  # позволил бы PR-автору подменить скрипт и получить execution с $OPENAI_API_KEY.
  # Это rollback Trust boundary fix Шаг 7.6 Plan A. PR_REF остаётся жив и используется
  # через `--head "$PR_REF"` (refspec), а скрипт читается из trusted base checkout.
  git checkout "$BASE_BRANCH"

  # Ленивая установка — npm ci только когда нужен fallback (минимизируем первый-вход overhead).
  if [ ! -d .claude/tools/node_modules ]; then
    if ! (cd .claude/tools && npm ci --ignore-scripts); then
      echo "ОШИБКА: hybrid fallback недоступен — npm ci упал. Partial report публикуется с Reviewer B unavailable."
      HYBRID_FALLBACK_FAILED=1
    fi
  fi

  if [ "$HYBRID_FALLBACK_FAILED" = "0" ] && [ -d .claude/tools/node_modules ]; then
    # Запуск из trusted base (cwd = $BASE_BRANCH after switch выше). PR код читается
    # только через diff origin/$BASE_BRANCH..$PR_REF — refspec, не cwd contents.
    node .claude/tools/openai-review.mjs --model gpt-5.3-codex --base "$BASE_BRANCH" --head "$PR_REF" \
      > .review-responses/mode-a-reviewer-b-"$HEAD_COMMIT".md \
      2> .review-responses/mode-a-reviewer-b-"$HEAD_COMMIT".err
    EXIT_B=$?
    if [ "$EXIT_B" = "0" ]; then
      MODE="A-hybrid"  # Reviewer A на subscription, Reviewer B на Platform API
      echo "Hybrid fallback успешен."
    else
      echo "ВНИМАНИЕ: hybrid fallback exec упал — exit_B=$EXIT_B. Partial report с Reviewer B unavailable."
      HYBRID_FALLBACK_FAILED=1
    fi
  fi

  # Если hybrid fallback failed (npm ci broken OR exec failure) — partial report.
  # MODE остаётся "A-hybrid" (валиден для case statement в Шаге 5.3),
  # MODEL_B_NAME явно помечается как unavailable + warning label injected при публикации.
  if [ "$HYBRID_FALLBACK_FAILED" = "1" ]; then
    MODE="A-hybrid"
    MODEL_B_NAME="(unavailable — Mode A-legacy fallback failed)"
    # В Шаге 5.3 publish добавь warning label:
    # ⚠️ Reviewer B unavailable — partial report (Reviewer A через subscription отработал,
    # Reviewer B fallback на Platform API не удался: npm ci broken или exec failure)
  fi
fi

3.1.3 Cleanup PR_REF после ревью (важно для idempotency)

Размещён ПОСЛЕ hybrid fallback (3.1.2) per external review iter 1 finding — fallback использует --head "$PR_REF", ref должен быть удалён только когда все reviewer paths завершены.

После того как оба reviewer'а (включая возможный hybrid fallback) отработали — возврат в trusted base + удаление локального ref:

# Возврат в trusted base branch (был git checkout --detach в Шаге 3.1).
git checkout "$BASE_BRANCH"

# Удалить локальный ref u2-pr-<PR_NUMBER> — идемпотентно (флаг -f игнорирует отсутствие).
git update-ref -d "refs/heads/${PR_REF}" || true

# Альтернатива (если ref был как обычная ветка):
# git branch -D "$PR_REF" 2>/dev/null || true

В отчёте (Шаг 5.3) Mode A-hybrid помечается как ⚠️ Reviewer B на Mode A-legacy fallback (gpt-5.3-codex недоступна через subscription).

3.1.4 Mode A-legacy (full Platform API path, v3.6 baseline)

Если из Шага 1.4 MODE_PRIMARY="legacy" (Codex CLI вообще недоступен) — оба reviewer'а через openai-review.mjs. Pattern идентичен v3.6:

# Trust boundary: --head "$PR_REF" указывает PR head как fetched ref,
# tool делает diff origin/${BASE_BRANCH}...${PR_REF} БЕЗ переключения cwd.
# Reviewer A: gpt-5.5 через /v1/chat/completions
node .claude/tools/openai-review.mjs --model gpt-5.5 --base "$BASE_BRANCH" --head "$PR_REF" \
  > .review-responses/mode-a-reviewer-a-"$HEAD_COMMIT".md \
  2> .review-responses/mode-a-reviewer-a-"$HEAD_COMMIT".err &
PID_A=$!

# Reviewer B: gpt-5.3-codex через /v1/responses.
# Пропускается для doc-only scope (SKIP_REVIEWER_B=1 из Шага 1.5) — не жечь Platform-токены.
# Те же side-effects, что в §3.1 primary: EXIT_B=0 + SKIPPED_REVIEWER_B=1 (иначе §5.3 валидатор упадёт exit 1).
if [ "$SKIP_REVIEWER_B" = "1" ]; then
  echo "doc-only scope: Reviewer B (gpt-5.3-codex Platform API) пропущен по политике research 2026-05-19; только gpt-5.5."
  wait "$PID_A"; EXIT_A=$?
  EXIT_B=0
  SKIPPED_REVIEWER_B=1
else
  SKIPPED_REVIEWER_B=0  # defensive-сброс: реальный вызов Reviewer B, не skip (защита от stale-значения предыдущего прогона)
  node .claude/tools/openai-review.mjs --model gpt-5.3-codex --base "$BASE_BRANCH" --head "$PR_REF" \
    > .review-responses/mode-a-reviewer-b-"$HEAD_COMMIT".md \
    2> .review-responses/mode-a-reviewer-b-"$HEAD_COMMIT".err &
  PID_B=$!

  wait "$PID_A"; EXIT_A=$?
  wait "$PID_B"; EXIT_B=$?
fi

В Mode A-legacy parallel допустим (openai-review.mjs instances independent). На Codex CLI primary — sequential обязателен (см. 3.1).

Exit 3 (nothing to review) — не ошибка, штатный сигнал пустого diff'а.

openai-review.mjs сам обеспечивает:

Явный git fetch origin +refs/heads/<base>:refs/remotes/origin/<base> (устойчиво к пользовательским remote.origin.fetch).
git diff --no-color --no-ext-diff (детерминированный diff независимо от git config пользователя).
Runtime allowlist (три полноразмерные модели — gpt-5.5 primary, gpt-5.4 fallback, gpt-5.3-codex; отказ при любой другой).
Диспатч endpoint по модели (chat.completions vs responses).
Классификацию ошибок API (401/403 ключ, 429 rate limit, 400 oversized, 5xx сервер, network).
Таймауты: 180 сек для review (reasoning high может отрабатывать долго); 2 сек для --ping без retry — строгий wall-clock бюджет AC6 плана для fail-fast на проблемах сети/ключа.

3.2 Режим C — Claude adversarial degraded

Запусти два Claude-субагента последовательно на том же PR.

Проход 1 (стандартный):

Ты Reviewer. Прочитай .agents/AGENT_ROLES.md секция "3. Reviewer".
Режим: Standard, все 4 аспекта.
Задача: проверь PR #<PR_NUMBER>. Верни structured findings PM.

Проход 2 (adversarial):

Ты adversarial Reviewer. Прочитай .agents/AGENT_ROLES.md секция "3. Reviewer".
Контекст: первый проход этого PR уже прошёл. Твоя задача — найти то, что первый проход пропустил.
Фокус: baseline-слабые места, необработанные edge-cases, скрытые дубликаты, некорректные инварианты, маловероятные, но критичные ошибки.
Задача: проверь PR #<PR_NUMBER>. Верни structured findings PM.

3.3 Режим D — manual emergency

Принимает вывод как есть.
Маппит на 4 аспекта.
Публикует с меткой «⚠️ Manual emergency mode. Adversarial diversity и воспроизводимость снижены».
Атрибуция — по тому, кого оператор использовал.

Шаг 3.5: Convergence detection и iteration cap (ADR 3.24)

⛔ Stateless модели не сходятся естественно. GPT-5.5/5.3-Codex каждую iter перечитывают весь diff и могут indefinitely surfacing replays + Low refinements. Без cap — infinite polish loop, нарушает цель пайплайна (облегчение труда оператора).

3.5.1 Convergence detection (preferred path)

PM объявляет APPROVED если выполнены ВСЕ условия (порог итераций — scope-aware, см. таблицу ниже):

Streak подряд с 0 новых Critical/High findings (новый = не в triage предыдущих iter, не replay уже-deferred);
только replays уже-deferred + Low refinements того же класса;
AND total external iter ≥ floor для scope/типа прохода;
AND каждое выявленное Low/Med/High finding имеет явный triage-статус (fix now / defer to Beads с ID / reject with rationale). Сохраняет инвариант #5 — никаких «висящих» finding'ов без owner.

Floor итераций и streak (scope-aware):

Scope diff (классификатор §1.5)	Streak clean APPROVED	Floor (total iter)
`DOC_ONLY` — product/spec/gdd `*.md`, НЕ governance	1	1
`HAS_CONFIG` — mixed (doc + config без executable)	2	3
`HAS_EXECUTABLE` — executable / governance / весь self-modification surface (`^.claude/` любой файл, `^.agents/`, `^.codex/`, `AGENTS.md`/`CLAUDE.md`, server/client код; + любой unknown-extension по fail-closed)	3	5

⚠️ Ярлык «doc-only» = строго DOC_ONLY из единого bash-классификатора §1.5 (§3.5.2 его переиспользует). Любой файл под ^.claude/, ^.agents/, ^.codex/, а также AGENTS.md/CLAUDE.md — governance (HAS_EXECUTABLE), не doc-only, даже с расширением .md (его .md-расширение не делает его harmless — классификатор ловит по пути, т.к. fail-closed по extension governance-.md не отсекает). «Все файлы .md» ≠ doc-only. Этот самый скилл (правка SKILL.md + PIPELINE_ADR.md) — HAS_EXECUTABLE, floor 5.

⚠️ Только для DOC_ONLY scope — разрешение конфликтующих вердиктов на неизменном HEAD (формальная процедура, не усмотрение). «Неизменный HEAD» = HEAD == коммит предыдущего external review-pass. Если после APPROVED на том же HEAD приходит CHANGES_REQUESTED, PM не вправе объявить его «шумом» декларативно. PM обязан построчно сопоставить каждый finding диссентящего прохода ровно с одной категорией и приложить таблицу сопоставления в PR-комментарии:

replay — finding уже присутствует в triage-таблице предыдущей итерации на этом же HEAD; PM указывает проверяемую ссылку (iteration N, строка #M). Без конкретной ссылки категория не засчитывается.

вне скоупа PR — PM присваивает finding статус reject with rationale (как любому finding); обоснование аудируемо.

новый finding — не покрыт (1) и (2).

Converged-вердикт на DOC_ONLY объявляется только если КАЖДЫЙ finding диссентящего прохода попал в (1) replay-с-ссылкой или (2) reject-с-обоснованием — т.е. ни одного нового. Достаточно одного finding категории (3) — это не шум: он триажится по обычным правилам (fix now + повторный APPROVED / defer с Beads-ID / reject с обоснованием), и convergence «по шуму» не объявляется. Так субъективная фиксация convergence заменена на проверяемое доказательство (ссылка на прежний triage или явный reject), а ошибочно-пропущенный реальный finding не может быть отброшен без следа.

Для HAS_CONFIG / HAS_EXECUTABLE это послабление не действует вовсе: любой CHANGES_REQUESTED требует fix + повторного APPROVED по обычным правилам.

Инвариант floor↔gate: floor — recipe экономии итераций (когда ПЕРЕСТАТЬ гонять ревью), не gate готовности к merge. Merge-readiness определяется исключительно hard-gate'ом /finalize-pr, который не читает счётчик итераций / floor: для Sprint Final он требует external review-pass с APPROVED на HEAD независимо от floor. Пониженный floor не может создать ложный merge-ready. Не связывать floor с hard-gate'ом в будущих правках.

3.5.2 Soft iteration cap (safety valve, не hard ceiling)

Если convergence не detected — soft cap по scope:

Scope diff (по `gh pr diff <N> --name-only`)	Soft cap
Doc-only (только product/spec/gdd `*.md`, не governance — см. ниже)	7
Mixed (doc + config без executable: `.json`, `.yaml`)	12
Executable / governance / security-sensitive infrastructure — весь self-modification surface: `^.claude/` (любой файл: hooks, tools, skills вкл. не-SKILL.md, agents, commands, rules, settings.json), `^.agents/` (doctrine/ADR/governance, .md и не-.md), `^.codex/`, `AGENTS.md`/`CLAUDE.md` (корень или вложенные), server/client code, `.gitignore`/`.gitattributes`	20

# Scope и CAP уже вычислены в Шаге 1.5 (единый классификатор). Здесь только используем результат —
# НЕ пересчитываем (защита от drift regex-паттернов).
if [ -z "${CAP:-}" ]; then
  echo "ВНИМАНИЕ: CAP не задан (Шаг 1.5 не выполнен?). Fail-closed: CAP=20."
  CAP=20
fi
echo "External review iteration cap для scope: $CAP"

3.5.3 Binary escalation при hit cap (без оценки findings)

## ⏸ External review hit iteration cap (iter <N>/<CAP>)

Cross-model adversarial coverage достигнута. Оставшиеся findings:
- <X> новых Med refinements (детали в collapsible ниже)
- <Y> replays уже-deferred (см. <BEADS_ID>)
- 0 новых Critical/High

PM-рекомендация: **(a) APPROVE с deferred bundle**.

**Operator decision:**
- (a) Approve — finalize PR с deferred bundle <BEADS_ID> (рекомендация)
- (b) Continue iter <N+1> — ожидаемо ещё refinements того же класса, не блокеры

Жду явный (a) или (b) в PR comment или чате.

3.5.4 Когда cap не применим

Tier ≠ Sprint Final. Cap для external review только; internal review цикл не имеет cap (Critical-tier требует 2 iter, остальные — 1).
Operator override. Оператор может явно потребовать продолжение iter ≥ cap+1 (например при подозрении на missed Critical в новом коде после cap'а).
Regression detected. Если iter N+1 нашёл NEW Critical/High который раньше не surfac'ился — cap reset, fix-cycle продолжается до полной convergence по scope-aware streak §3.5.1 (для своего scope). Регрессия, по сути, эскалирует scope: NEW Critical/High означает, что дифф уже не «тривиально doc-only», поэтому такой проход трактуется не ниже HAS_CONFIG (streak ≥ 2), а для HAS_EXECUTABLE — полные streak 3 / floor 5. Снижать streak ниже scope-значения нельзя — иначе post-regression покрытие слабее initial.

Источник: ADR 3.24 + U2 PR #186 retrospective.

Шаг 4: Copilot Re-Review

Запроси re-review от GitHub Copilot (auto-reviewer):

REPO=$(gh repo view --json nameWithOwner -q '.nameWithOwner')
gh api "repos/$REPO/pulls/<PR_NUMBER>/requested_reviewers" \
  --method POST \
  -f 'reviewers[]=copilot-pull-request-reviewer[bot]' \
  && echo "Copilot: re-review requested" \
  || echo "Copilot: request failed — может потребоваться ручной запуск"

Шаг 5: Консолидация и публикация

5.1 Имена моделей (атрибуция)

Подставь реальные имена из шага 2 — не захардкоженные:

Режим	MODEL_A_NAME	MODEL_B_NAME	Метка
A (primary, v3.9)	`GPT-5.5 via Codex CLI subscription`	`GPT-5.3-Codex via Codex CLI subscription`	—
A-hybrid	`GPT-5.5 via Codex CLI subscription`	`GPT-5.3-Codex via Platform API (openai-review.mjs)`	`⚠️ Reviewer B на Mode A-legacy fallback — gpt-5.3-codex недоступна через subscription`
A-legacy (v3.6 baseline)	`GPT-5.5 via Platform API (openai-review.mjs)` (или `GPT-5.4` если 5.5 недоступна)	`GPT-5.3-Codex via Platform API (openai-review.mjs)`	`⚠️ Mode A-legacy fallback — primary v3.9 path (Codex CLI subscription) недоступен`
C	`Claude Opus 4.7 (standard)`	`Claude Opus 4.7 (adversarial)`	`⚠️ Degraded mode с имитацией adversarial diversity. Не является cross-model review`
D	По факту (например, `GPT-5.4 via Copilot Agent`)	— (single reviewer)	`⚠️ Manual emergency mode. Adversarial diversity и воспроизводимость снижены`

Doc-only skip (под-вариант Mode A / A-legacy, НЕ новый режим): при SKIPPED_REVIEWER_B=1 (Шаг 1.5) Reviewer B (gpt-5.3-codex) не вызывался. MODE остаётся A/A-legacy; MODEL_B_NAME = — (пропущен: doc-only policy research 2026-05-19); секция Reviewer B в шаблоне §5.3 суппрессируется. Это не degradation — Reviewer A (gpt-5.5) прошёл все 4 аспекта.

5.2 Защита findings — raw output в collapsible

Инвариант 6 (PM не искажает findings). PM имеет право структурировать по 4 аспектам и убирать дубликаты, но не перефразировать или смягчать формулировки. Чтобы оператор мог проверить, raw-вывод каждого проверяющего публикуется целиком в <details> блоке отдельно от консолидированного отчёта.

5.3 Шаблон комментария

⚠️ Commit binding — без ** и с META JSON. /finalize-pr ищет маркер через regex Commit:\s*\?ИЛИ JSON"commit": "". Шаблон Commit:(bold markdown) **не** матчится (междуCommit:и хэшем идёт**, не whitespace) — это drift, найденный внешним ревью (round 12). Используй простой Commit: ` и добавь HTML META с JSON — дублирование на случай, если кто-то изменит markdown-форматирование.

HEAD_COMMIT уже зафиксирован в шаге 1.4 — повторно не вычисляем.

BODY=$(cat <<'GH_BODY_<RAND>'
## Внешнее ревью (Sprint Final) — Режим: __MODE__

Commit: `__HEAD_COMMIT__`

<!-- {"reviewer": "__MODEL_A_NAME__ + __MODEL_B_NAME__", "reviewer_a": "__MODEL_A_NAME__", "reviewer_b": "__MODEL_B_NAME__", "commit": "__HEAD_COMMIT__", "kind": "external", "mode": "__MODE__", "iteration": __ITERATION__} -->

<!--
Для режима A primary не добавляй warning-метки — оставь этот блок как есть (в комментарии).
Для режима A-hybrid: вынеси только строку `⚠️ Reviewer B на Mode A-legacy fallback ...`.
Для режима A-legacy: вынеси только строку `⚠️ Mode A-legacy fallback — primary v3.9 path ...`.
Для режима C: вынеси только строку `⚠️ Degraded mode ...`.
Для режима D: вынеси только строку `⚠️ Manual emergency mode ...`.
Не добавляй несколько строк одновременно. Дефолт — безопасный (нет меток).
⚠️ Reviewer B на Mode A-legacy fallback — <описание из таблицы 5.1>
⚠️ Mode A-legacy fallback — <описание из таблицы 5.1>
⚠️ Degraded mode — <описание из таблицы 5.1>
⚠️ Manual emergency mode — <описание из таблицы 5.1>
-->

### Findings (обязательная таблица для /finalize-pr фазы 2 triage)

> Если вердикт APPROVED без замечаний — оставь таблицу с единственной строкой `| — | — | нет замечаний | — | — | — |`. Пустая таблица недопустима: `/finalize-pr` отличает «нет findings» от «парсинг сломался».

| # | Severity | Заголовок | Файл:строка | Статус | Beads ID / Обоснование |
|---|----------|-----------|-------------|--------|------------------------|
| 1 | CRITICAL | ... | path:N | fix now | — |
| 2 | WARNING | ... | path:N | defer to Beads | bd-xyz-123 |
| 3 | INFO | ... | path:N | reject with rationale | <обоснование> |

### Консолидация (PM)

#### Reviewer A: __MODEL_A_NAME__

##### Архитектура: [OK / ISSUE]
[обоснование — дословно из findings, не перефразировать]

##### Безопасность: [OK / ISSUE]
[обоснование]

##### Качество: [OK / ISSUE]
[обоснование]

##### Гигиена кода: [OK / ISSUE]
[обоснование]

**Вердикт:** [APPROVED / CHANGES_REQUESTED / ESCALATION]
— Reviewer (__MODEL_A_NAME__)

---

__REVIEWER_B_SECTION__

---

### Raw output (для аудита оператором)

<details>
<summary>Reviewer A: __MODEL_A_NAME__ — raw</summary>

<pre><code>
<вставить raw-вывод Reviewer A целиком, без редактуры>
</code></pre>

</details>

__REVIEWER_B_RAW__

---

### Copilot
[re-review requested / auto-triggered / unavailable]

### Итоговый вердикт: [APPROVED / CHANGES_REQUESTED / ESCALATION]
CRITICAL: N, WARNING: N

— PM (Claude Opus 4.7), по результатам внешнего ревью
GH_BODY_<RAND>
)
# Вычисление обязательных полей публикации.
# MODE из шага 2 (A / A-hybrid / A-legacy / C / D); MODEL_A_NAME и MODEL_B_NAME — из таблицы 5.1.
# ITERATION — порядковый номер прохода внешнего ревью.
if [ -z "${MODE:-}" ]; then
  echo "MODE не задан. Установи переменную MODE (A / A-hybrid / A-legacy / C / D)." >&2
  exit 1
fi
case "$MODE" in
  A|A-hybrid|A-legacy|C|D) ;;
  *)
    echo "MODE='$MODE' invalid. Допустимые значения: A / A-hybrid / A-legacy / C / D." >&2
    exit 1
    ;;
esac
if [ -z "${MODEL_A_NAME:-}" ]; then
  echo "MODEL_A_NAME не задан. См. таблицу 5.1." >&2
  exit 1
fi
# doc-only skip: Reviewer B штатно не вызывался — БЕЗУСЛОВНО перезаписываем skip-меткой,
# даже если MODEL_B_NAME уже задан по таблице §5.1 (иначе META JSON соврёт, что B отработал).
if [ "${SKIPPED_REVIEWER_B:-0}" = "1" ]; then
  MODEL_B_NAME="— (пропущен: doc-only policy research 2026-05-19)"
fi
if [ -z "${MODEL_B_NAME:-}" ]; then
  # В режиме D Reviewer B отсутствует — допустимо "—".
  # В режимах A / A-hybrid / A-legacy / C без skip Reviewer B обязателен — молчаливый дефолт маскирует ошибку конфигурации.
  if [ "$MODE" = "D" ]; then
    MODEL_B_NAME="—"
  else
    echo "MODEL_B_NAME не задан для режима $MODE, где Reviewer B обязателен. См. таблицу 5.1." >&2
    exit 1
  fi
fi
ITERATION="${ITERATION:-1}"

# Условная (механическая) сборка секции Reviewer B: при doc-only skip — одна честная строка
# без 4 аспектов и raw-details; иначе полный skeleton (плейсхолдеры __MODEL_B_NAME__ заменяются ниже).
if [ "${SKIPPED_REVIEWER_B:-0}" = "1" ]; then
  REVIEWER_B_SECTION="> Reviewer B (gpt-5.3-codex) пропущен — doc-only policy (research 2026-05-19). Single-model gpt-5.5; все 4 аспекта покрыты Reviewer A."
  REVIEWER_B_RAW=""
elif [ "$MODE" = "D" ]; then
  REVIEWER_B_SECTION="> Reviewer B отсутствует — Mode D (manual emergency, single reviewer)."
  REVIEWER_B_RAW=""
else
  REVIEWER_B_SECTION=$(cat <<'RB_SECTION'
#### Reviewer B: __MODEL_B_NAME__

##### Архитектура: [OK / ISSUE]
[обоснование]

##### Безопасность: [OK / ISSUE]
[обоснование]

##### Качество: [OK / ISSUE]
[обоснование]

##### Гигиена кода: [OK / ISSUE]
[обоснование]

**Вердикт:** [APPROVED / CHANGES_REQUESTED / ESCALATION]
— Reviewer (__MODEL_B_NAME__)
RB_SECTION
)
  REVIEWER_B_RAW=$(cat <<'RB_RAW'
<details>
<summary>Reviewer B: __MODEL_B_NAME__ — raw</summary>

<pre><code>
<вставить raw-вывод Reviewer B целиком, без редактуры>
</code></pre>

</details>
RB_RAW
)
fi

# Порядок важен: сначала вставляем блоки Reviewer B (могут нести __MODEL_B_NAME__),
# затем подставляем имена — иначе __MODEL_B_NAME__ внутри вставленного блока не заменится.
BODY="${BODY//__REVIEWER_B_SECTION__/$REVIEWER_B_SECTION}"
BODY="${BODY//__REVIEWER_B_RAW__/$REVIEWER_B_RAW}"
BODY="${BODY//__HEAD_COMMIT__/$HEAD_COMMIT}"
BODY="${BODY//__MODEL_A_NAME__/$MODEL_A_NAME}"
BODY="${BODY//__MODEL_B_NAME__/$MODEL_B_NAME}"
BODY="${BODY//__MODE__/$MODE}"
BODY="${BODY//__ITERATION__/$ITERATION}"
gh pr comment <PR_NUMBER> --body "$BODY"

⚠️ Raw output публикуется без редактуры содержимого, но с HTML-escaping перед вставкой в <pre><code>: заменить & → &, < → <, > → >. Без этого attacker-controlled diff может вернуть модели строку </code></pre> и сломать структуру комментария (закрывает Security ISSUE Reviewer A на 8e7dce6).

Пример экранирования (bash):
RAW_A_ESC=$(printf '%s' "$RAW_A" | sed 's/&/\&amp;/g; s/</\&lt;/g; s/>/\&gt;/g')
# далее используй $RAW_A_ESC внутри <pre><code>...</code></pre> вместо $RAW_A
Если raw слишком длинный — собрать как artifact и приложить ссылкой, но не сокращать пересказом.

Шаг 6: Pre-Chat Gate

Инвариант: review-pass НЕ завершён, пока отчёт не опубликован в PR.

Перед любым сообщением оператору проверь:

gh pr comment выполнился успешно.
Ссылка на комментарий зафиксирована (если доступна).
Только после этого сообщай оператору результат.

⛔ Чат-резюме без PR comment не завершает review-pass.

Шаг 7: Обработка ошибок

7.1 Mode A primary (Codex CLI subscription) errors

Ситуация	Признак	Действие
`429` quota exhausted	stderr содержит `429` или `quota`	Manual swap аккаунта: `Use-CodexAccount work-business` (см. CODEX_AUTH.md §8.5). Retry. Если оба исчерпаны — fallback на Mode A-legacy через перезапуск skill (auto-detected в Шаге 1.4 при перезапуске)
`oauth token exchange failed`	stderr содержит `oauth`	Refresh token истёк. Re-login: `codex logout && codex login`, затем restore из backup при необходимости. Retry
`model X not available on this tier` (Reviewer B)	EXIT_B ≠ 0, stderr содержит `model` + `not available\|tier`	Auto hybrid fallback (Шаг 3.1.2) — Reviewer B через openai-review.mjs Platform API. Mode помечается `A-hybrid`
Profile `[profiles.review]` не найден	stderr содержит `profile` + `not found`	Operator должен добавить profile в `~/.codex/config.toml` (CODEX_AUTH.md §8.4). Skill exit 1 — невозможно continue без profile
Empty output / pipe error	Empty stdout file	Retry × 1; если снова пусто — переход в Mode A-legacy или публикация частичного отчёта с меткой

7.2 Mode A-legacy (openai-review.mjs Platform API) errors

Ситуация	Действие
`openai-review.mjs` exit 1 (локальная или API-ошибка)	Retry × 1; если повторно падает — переход в Mode C или D. Классификация ошибки — в stderr скрипта
`openai-review.mjs` exit 2 (валидация)	Исправить аргументы/модель. См. `--help`
`openai-review.mjs` exit 3 (nothing to review)	Нормальный сигнал — `HEAD == origin/<base>`. Ревью не запускается

7.3 Общие правила для всех режимов

Ситуация	Действие
Пустой вывод от модели	Retry × 1; если снова пусто — переход в Mode A-legacy / Mode C/D или публикация частичного отчёта
Оба проверяющих упали	Эскалация оператору, публикация частичного отчёта с меткой
Один проверяющий упал	Публикация отчёта с пометкой об упавшем проверяющем
Rate limit на всех повторах	Пометить как unavailable, переход вниз по chain (A → A-hybrid → A-legacy → C → D)

Никогда не пропускай ошибку молча. Частичный отчёт лучше, чем отсутствие отчёта.

Шаг 8: Итеративные исправления

Если итоговый вердикт CHANGES_REQUESTED:

PM передаёт CRITICAL и WARNING findings Developer-субагенту.
Developer исправляет → git push.
PM запрашивает Copilot re-review (шаг 4).
PM повторно запускает /external-review <PR_NUMBER> на новом HEAD.
Каждый повторный запуск создаёт новый комментарий (не редактирует старый) — для audit trail.

external-review

이 저장소의 다른 Skills

External Review — кросс-модельное ревью через Codex CLI ChatGPT subscription

Контекст

Аргументы

Шаг 1: Пререквизиты

1.1 Проверка чистоты рабочего дерева

1.2 Получение PR-метаданных + fetch PR head как trusted ref (БЕЗ checkout)

1.3 Проверка что ветка запушена

1.4 Pre-flight: проверка backend availability, фиксация HEAD_COMMIT

1.5 Pre-flight scope classification (единый источник для skip-политики и floor/cap)

Шаг 2: Определение режима работы

Режим A primary: Codex CLI ChatGPT subscription (основной путь, v3.9)

Режим A-legacy: Node.js native через OpenAI SDK (auto-fallback v3.6 path)

Режим C: Все Mode A варианты недоступны (degraded, Claude adversarial)

Режим D: Автоматические пути недоступны (manual emergency)

Таблица выбора режима

Шаг 3: Запуск проверяющих

3.1 Режим A primary — через Codex CLI subscription (v3.9, основной путь)

3.1.2 Hybrid fallback: Reviewer B → Mode A-legacy (если codex CLI не даёт gpt-5.3-codex)

3.1.3 Cleanup PR_REF после ревью (важно для idempotency)

3.1.4 Mode A-legacy (full Platform API path, v3.6 baseline)

3.2 Режим C — Claude adversarial degraded

3.3 Режим D — manual emergency

Шаг 3.5: Convergence detection и iteration cap (ADR 3.24)

3.5.1 Convergence detection (preferred path)

3.5.2 Soft iteration cap (safety valve, не hard ceiling)

3.5.3 Binary escalation при hit cap (без оценки findings)

3.5.4 Когда cap не применим

Шаг 4: Copilot Re-Review

Шаг 5: Консолидация и публикация

5.1 Имена моделей (атрибуция)

5.2 Защита findings — raw output в collapsible

5.3 Шаблон комментария

Шаг 6: Pre-Chat Gate

Шаг 7: Обработка ошибок

7.1 Mode A primary (Codex CLI subscription) errors

7.2 Mode A-legacy (openai-review.mjs Platform API) errors

7.3 Общие правила для всех режимов

Шаг 8: Итеративные исправления

External Review — кросс-модельное ревью через Codex CLI ChatGPT subscription

Контекст

Аргументы

Шаг 1: Пререквизиты

1.1 Проверка чистоты рабочего дерева

1.2 Получение PR-метаданных + fetch PR head как trusted ref (БЕЗ checkout)

1.3 Проверка что ветка запушена

1.4 Pre-flight: проверка backend availability, фиксация HEAD_COMMIT

1.5 Pre-flight scope classification (единый источник для skip-политики и floor/cap)

Шаг 2: Определение режима работы

Режим A primary: Codex CLI ChatGPT subscription (основной путь, v3.9)

Режим A-legacy: Node.js native через OpenAI SDK (auto-fallback v3.6 path)

Режим C: Все Mode A варианты недоступны (degraded, Claude adversarial)

Режим D: Автоматические пути недоступны (manual emergency)

Таблица выбора режима

Шаг 3: Запуск проверяющих

3.1 Режим A primary — через Codex CLI subscription (v3.9, основной путь)

3.1.2 Hybrid fallback: Reviewer B → Mode A-legacy (если codex CLI не даёт gpt-5.3-codex)

3.1.3 Cleanup PR_REF после ревью (важно для idempotency)

3.1.4 Mode A-legacy (full Platform API path, v3.6 baseline)

3.2 Режим C — Claude adversarial degraded

3.3 Режим D — manual emergency

Шаг 3.5: Convergence detection и iteration cap (ADR 3.24)

3.5.1 Convergence detection (preferred path)

3.5.2 Soft iteration cap (safety valve, не hard ceiling)

3.5.3 Binary escalation при hit cap (без оценки findings)

3.5.4 Когда cap не применим

Шаг 4: Copilot Re-Review

Шаг 5: Консолидация и публикация

5.1 Имена моделей (атрибуция)

5.2 Защита findings — raw output в collapsible

5.3 Шаблон комментария

Шаг 6: Pre-Chat Gate

Шаг 7: Обработка ошибок

7.1 Mode A primary (Codex CLI subscription) errors

7.2 Mode A-legacy (openai-review.mjs Platform API) errors

7.3 Общие правила для всех режимов

Шаг 8: Итеративные исправления

이 저장소의 다른 Skills