git:20261008.46339a7 to git:20261012.4b1b6a8

---

name: review-pr

description: 'Run a quality review on an open pull request (by PR number or linked TASK), externally via codex or internally via a clean-context self-review, and post the verdict. Use when PM mentions "review PR", "PR review", "review pull request", "quality review", "review this pr", "сделай ревью pr", or any request to evaluate an open PR before merge. Trigger liberally — under-triggering lets PRs land without a second look; over-triggering is recoverable (PM can ignore the review comments).'

argument-hint: "[PR# or TASK-XXX] [self]"

cli_requires: "task_tool, codex_cli"

fallback: self

---

# /polisade:review-pr [PR# or TASK-XXX] [self] — PR Quality Review (external CLI or self)

Независимый quality review Pull Request. По умолчанию — через внешний reviewer CLI (OpenAI Codex, `gpt-5.3-codex`). С флагом `self` — через CLI текущего агента в отдельном процессе (чистый контекст).

> **Флаг `self`** — для случаев, когда доступна подписка только на один агент. Ревью проводится тем же CLI, но в изолированном процессе.

**Цикл:** `implement → PR → review-pr → fix замечаний → merge`

## Использование

```

/polisade:review-pr 42 # Review PR #42 через reviewer CLI

/polisade:review-pr TASK-001 # Найти PR для TASK-001

/polisade:review-pr # Review PR текущей ветки

/polisade:review-pr 42 self # Review PR #42 через текущий агент

/polisade:review-pr TASK-001 self # PR для TASK + self-review

/polisade:review-pr self # Текущая ветка + self-review

```

## Архитектура

```

/polisade:review-pr 42

|

v

+-----------------------------------------+

| ОСНОВНОЙ АГЕНТ |

| 1. Определить PR# и TASK-ID |

| 2. Запустить Review субагент |

+-----------------+-----------------------+

v

+-----------------------------------------+

| СУБАГЕНТ (general-purpose) |

| 1. Pre-fetch: pr-diff + pr-view (vcs.py)|

| 2. Bash: codex exec --full-auto |

| (или CLI текущего агента при self) |

| Ревьюер получает: |

| - diff и PR description в промпте |

| - Читает TASK, parent, гайдлайны |

| - Анализирует код/тесты |

| 3. pr-comment: публикация ревью в PR |

| Возвращает ревью с score и findings |

+-----------------+-----------------------+

v

+-----------------------------------------+

| ОСНОВНОЙ АГЕНТ |

| Score >= 8 → merge |

| Score < 8 → |

| Improvement субагент → fix |

| → Re-review (макс. 2 итерации) |

| → After 2 iterations: STOP + waiting_pm |

+-----------------------------------------+

```

**Anti-loop safety:** Максимум 2 итерации (review+improve). После 2-й — STOP, ждём PM.

<!-- polisade:nav-canon POINTER — навигационный канон клиента: grep-капсула

LOCALIZE в skills/implement/SKILL.md §1.8 (дом канона). Ревьюер навигирует

ВОКРУГ диффа (design conformance) тем же детерминированным grep-протоколом;

указателя. MCP-нав-протокол вырезан в V3-P2: клиент навигируется grep-протоколом и ни к какому внешнему движку не обращается.

Maintainer-note. -->

> **Навигация ревьюера (design conformance).** Ревью смотрит код **вокруг**

> правимых символов, а не только дифф: для правимых символов — прицельный read

> определения плюс 1–2 бюджетных `grep -rn "<symbol>"` вокруг символов диффа

> (точки использования). Свободный grep-обход всего проекта не нужен — это

> тот же детерминированный протокол, что LOCALIZE в `/polisade:implement`.

## Алгоритм

### 1. Определить PR, TASK-ID и режим

<!-- polisade:exec-denied CAPSULE BEGIN -->

> ⛔ **Если вызов скрипта на этом шаге ты СДЕЛАЛ, и он отклонён или не запустился** (`Command references protected path` / `Install directory is read-protected` / `Filesystem Guard` / `the tool's default permission is 'deny'`, отказ песочницы, ненулевой exit без вывода) — **STOP**. Вызова не было — отказа нет: сначала выполни команду; этот блок описывает её вывод и сам ответом не служит.

> Процитируй отказ дословно. НЕ пересказывай по исходнику, что скрипт «сделал бы»; НЕ собирай dry-run вручную; НЕ переходи к apply/push/pr-create.

> НЕ транскрибируй скрипт (прочитать → записать копию в `/tmp` или в проект → запустить копию): копия не байт-идентична — уезжают классы символов в regex, форма возврата функций, пропадают целые функции — и молча меняется набор применённых изменений.

> Отказ инструмента — это отказ, а не результат. Доложи PM дословный текст отказа и назови следствие: без рабочего вызова скрипта этот шаг выполнить нечем. Причину не придумывай — её называет только сам текст отказа.

<!-- polisade:exec-denied CAPSULE END -->

Из аргументов извлечь:

- **PR/TASK-ID**: число (`42`) или `TASK-XXX` если указано

- **Режим**: если слово `self` присутствует в аргументах — форсировать `self`. Иначе — спросить единый OPS-011 helper:

```bash

caps=$(${POLISADE_PYTHON:-python3} {plugin_root}/scripts/polisade_cli_caps.py detect)

mode=$(${POLISADE_PYTHON:-python3} -c 'import json,sys; print(json.loads(sys.argv[1])["reviewer"]["mode"])' "$caps")

# OPS-007: surface codex-impersonator warnings so a foreign

# `codex` binary in PATH is never silently ignored.

warning=$(${POLISADE_PYTHON:-python3} -c 'import json,sys; print(json.loads(sys.argv[1])["reviewer"].get("warning") or "")' "$caps")

[ -n "$warning" ] && echo "⚠ $warning"

# mode: "codex" | "self" | "blocked" | "off"

```

- `mode == "codex"` → использовать `codex exec` (раздел ниже).

- `mode == "self"` → использовать CLI текущего агента (таблица ниже).

- `mode == "blocked"` → STOP с диагностикой (`reviewer.reason`).

- `mode == "off"` → STOP; reviewer отключён в settings, PM делает ревью руками.

Никакого повторного `which codex` / `which {own_cli}` — helper уже видит окружение и возвращает правильный режим для любого target CLI (Qwen/GigaCode → self без явного ветвления).

- Если аргумент — число (напр. `42`) → PR #42, TASK-ID из PR body

- Если аргумент — `TASK-XXX` → найти PR по ветке/коммитам этой TASK

- Если нет аргумента (кроме `self`) → определить по текущей ветке:

<!-- polisade:exec-denied CAPSULE BEGIN -->

> ⛔ **Если вызов скрипта на этом шаге ты СДЕЛАЛ, и он отклонён или не запустился** (`Command references protected path` / `Install directory is read-protected` / `Filesystem Guard` / `the tool's default permission is 'deny'`, отказ песочницы, ненулевой exit без вывода) — **STOP**. Вызова не было — отказа нет: сначала выполни команду; этот блок описывает её вывод и сам ответом не служит.

> Процитируй отказ дословно. НЕ пересказывай по исходнику, что скрипт «сделал бы»; НЕ собирай dry-run вручную; НЕ переходи к apply/push/pr-create.

> НЕ транскрибируй скрипт (прочитать → записать копию в `/tmp` или в проект → запустить копию): копия не байт-идентична — уезжают классы символов в regex, форма возврата функций, пропадают целые функции — и молча меняется набор применённых изменений.

> Отказ инструмента — это отказ, а не результат. Доложи PM дословный текст отказа и назови следствие: без рабочего вызова скрипта этот шаг выполнить нечем. Причину не придумывай — её называет только сам текст отказа.

<!-- polisade:exec-denied CAPSULE END -->

```bash

${POLISADE_PYTHON:-python3} {plugin_root}/scripts/polisade_vcs.py pr-list --head "$(git branch --show-current)" --format json --project-root "${POLISADE_WORK_DIR:-.}" | jq -r '.[0].number // empty'

```

- Если PR не найден → ошибка:

```

═══════════════════════════════════════════

PR QUALITY REVIEW — ОШИБКА

═══════════════════════════════════════════

PR не найден.

Возможные причины:

- Ветка не запушена

- PR не создан

-> Создай PR: /polisade:continue (автоматически) или /polisade:pr — проверь статус

═══════════════════════════════════════════

```

Номер PR известен — получи описание PR и готовую шапку отчёта ОДНИМ

вызовом хелпера (подставь номер PR вместо `${PR_NUM}`; JSON — в файл,

строка шапки — на stderr):

```bash

mkdir -p ${POLISADE_WORK_DIR:-.}/.polisade/tmp

${POLISADE_PYTHON:-python3} {plugin_root}/scripts/polisade_vcs.py pr-view ${PR_NUM} --fields title,body,files --format json --project-root "${POLISADE_WORK_DIR:-.}" > ${POLISADE_WORK_DIR:-.}/.polisade/tmp/pr-desc.json

```

<!-- polisade:exec-denied CAPSULE BEGIN -->

> ⛔ **Если вызов скрипта на этом шаге ты СДЕЛАЛ, и он отклонён или не запустился** (`Command references protected path` / `Install directory is read-protected` / `Filesystem Guard` / `the tool's default permission is 'deny'`, отказ песочницы, ненулевой exit без вывода) — **STOP**. Вызова не было — отказа нет: сначала выполни команду; этот блок описывает её вывод и сам ответом не служит.

> Процитируй отказ дословно. НЕ пересказывай по исходнику, что скрипт «сделал бы»; НЕ собирай dry-run вручную; НЕ переходи к apply/push/pr-create.

> НЕ транскрибируй скрипт (прочитать → записать копию в `/tmp` или в проект → запустить копию): копия не байт-идентична — уезжают классы символов в regex, форма возврата функций, пропадают целые функции — и молча меняется набор применённых изменений.

> Отказ инструмента — это отказ, а не результат. Доложи PM дословный текст отказа и назови следствие: без рабочего вызова скрипта этот шаг выполнить нечем. Причину не придумывай — её называет только сам текст отказа.

<!-- polisade:exec-denied CAPSULE END -->

**Шапка отчёта — готовая строка инструмента.** stderr этой команды — одна

строка вида `PR #<номер> «<заголовок>» · <ветка> → <база> · файлов: <число>`:

её собрал `pr-view` из ответа сервера (то же значение — поле `review_header`

в `pr-desc.json`). Запомни её ДОСЛОВНО — это строка шапки отчёта PM (раздел

неё: не пересказывай, не сокращай, не собирай шапку сам. Ненулевой exit —

STOP: процитируй stderr, шапку и заголовок не придумывай. TASK-ID ищи в

`title` и `body` этого же файла.

Определение TASK-ID:

1. Из PR title: `[TASK-XXX]` паттерн

2. Из PR body: поиск `TASK-XXX`

3. Из коммитов PR: `[TASK-XXX]` в сообщениях

### 1.1. Проверить, что TASK всё ещё на ревью

После определения TASK-ID и **до** запуска ревьюера выполни из корня проекта

(подставь найденный ID вместо `TASK-XXX`). Выполни именно эту команду:

не переопределяй `POLISADE_PLUGIN_ROOT`, не ищи helper через `find` и не

запускай копию из исходного checkout. Если доставленный helper недоступен,

**STOP**, даже если другой файл с тем же именем найден где-то ещё.

<!-- polisade:exec-denied CAPSULE BEGIN -->

> ⛔ **Если вызов скрипта на этом шаге ты СДЕЛАЛ, и он отклонён или не запустился** (`Command references protected path` / `Install directory is read-protected` / `Filesystem Guard` / `the tool's default permission is 'deny'`, отказ песочницы, ненулевой exit без вывода) — **STOP**. Вызова не было — отказа нет: сначала выполни команду; этот блок описывает её вывод и сам ответом не служит.

> Процитируй отказ дословно. НЕ пересказывай по исходнику, что скрипт «сделал бы»; НЕ собирай dry-run вручную; НЕ переходи к apply/push/pr-create.

> НЕ транскрибируй скрипт (прочитать → записать копию в `/tmp` или в проект → запустить копию): копия не байт-идентична — уезжают классы символов в regex, форма возврата функций, пропадают целые функции — и молча меняется набор применённых изменений.

> Отказ инструмента — это отказ, а не результат. Доложи PM дословный текст отказа и назови следствие: без рабочего вызова скрипта этот шаг выполнить нечем. Причину не придумывай — её называет только сам текст отказа.

<!-- polisade:exec-denied CAPSULE END -->

```bash

```

Только exit 0 и JSON `"status" == "allowed"` разрешают продолжить. Exit 1, ошибка

запуска или отказ Filesystem Guard → **STOP до комментария, исправлений, merge

`artifactIndex`, требует их совпадения и статус `review` или

`changes_requested`. При `cancelled`, `not_actual`, `done`, неизвестном статусе

или расхождении сначала нужен явный выбор PM: оставить закрытой, создать новую

TASK либо документировать решение о возобновлении и вернуть TASK в ревью;

после этого запусти `/polisade:sync --apply` и повтори guard. Сам вызов

`review-pr` не является решением возобновить отменённую работу.

### 2. Запустить Review субагент

<!-- polisade:silo-legacy POINTER — канон «Силос → корпус» живёт в /polisade:design -->

> **Силос ≠ корпус.** Источник правды по архитектуре — живой корпус

> `docs/architecture/`; пакет `DESIGN-NNN-<slug>/` — legacy-силос. Прочитал

> файл из силоса — скажи об этом вслух (переходное чтение). Полный канон —

> `/polisade:design`, блок «Силос → корпус»; перевод силоса на корпус —

> `${POLISADE_PYTHON:-python3} scripts/polisade_migrate_silo.py <пакет>` (dry-run по умолчанию).

```

Task tool:

subagent_type: "general-purpose"

description: "Review PR #N for TASK-XXX"

```

### 3. Обработать результат

- Парсить ИТОГО score и ВЕРДИКТ из ответа ревьюера

- Если ошибка (timeout, API, не установлен) → показать с рекомендациями

### 4. Если IMPROVE (score < 8) — Improvement субагент

```

Task tool:

subagent_type: "general-purpose"

description: "Fix PR #N based on review"

prompt: [prompt ниже]

```

Prompt для improvement субагента:

<!-- polisade:exec-denied CAPSULE BEGIN -->

> ⛔ **Если вызов скрипта на этом шаге ты СДЕЛАЛ, и он отклонён или не запустился** (`Command references protected path` / `Install directory is read-protected` / `Filesystem Guard` / `the tool's default permission is 'deny'`, отказ песочницы, ненулевой exit без вывода) — **STOP**. Вызова не было — отказа нет: сначала выполни команду; этот блок описывает её вывод и сам ответом не служит.

> Процитируй отказ дословно. НЕ пересказывай по исходнику, что скрипт «сделал бы»; НЕ собирай dry-run вручную; НЕ переходи к apply/push/pr-create.

> НЕ транскрибируй скрипт (прочитать → записать копию в `/tmp` или в проект → запустить копию): копия не байт-идентична — уезжают классы символов в regex, форма возврата функций, пропадают целые функции — и молча меняется набор применённых изменений.

> Отказ инструмента — это отказ, а не результат. Доложи PM дословный текст отказа и назови следствие: без рабочего вызова скрипта этот шаг выполнить нечем. Причину не придумывай — её называет только сам текст отказа.

<!-- polisade:exec-denied CAPSULE END -->

```

Ты получил результаты независимого quality review PR.

Твоя задача — исправить найденные проблемы.

PR ВЕТКА: {branch name}

ЗАДАЧА: {TASK-ID}

РЕЗУЛЬТАТЫ РЕВЬЮ:

{полный ответ review}

ИНСТРУКЦИИ:

1. Прочитай файлы, указанные в замечаниях (Read tool)

2. Примени ТОЛЬКО рекомендации из ревью — не добавляй лишнего

4. Сделай коммит: [{TASK-ID}] Address review feedback: {summary}

<!-- # OPS-010: это коммит вида `improvement` (OPS-010).

В этот же commit-staging бандли ЛЮБЫЕ отложенные правки frontmatter

TASK.md (`status:`) и PROJECT_STATE.json task-bucket. НЕ делай

отдельный status-only commit перед/после. НЕ пиши `lastUpdated`

в PROJECT_STATE.json — поле всегда null. -->

<!-- polisade:push-stop CAPSULE BEGIN -->

> ⛔ **Если вызов `polisade_vcs.py` на этом шаге ты СДЕЛАЛ, и он отклонён** (`Command references protected path` / `Install directory is read-protected` / `Filesystem Guard`) — **STOP до push**: ни `git-push`, ни `pr-create`, ни `pr-merge`, ни `pr-comment` больше не выполняются. Вызова не было — отказа нет: сначала выполни команду; этот блок описывает её вывод и сам ответом не служит.

> Bare `git push` запрещён (push обязан проверяться сравнением remote SHA — OPS-028); самодельные REST/curl-вызовы к Bitbucket/GitHub запрещены; helper не транскрибируется в `/tmp`.

> Доложи PM дословный текст отказа из вывода этого вызова, ветку с локальным коммитом и то, что pr-create не выполнялся, — и заверши рецепт на этом. Готовой фразы отказа здесь нет: её даёт только вывод вызова.

<!-- polisade:push-stop CAPSULE END -->

5. Push изменения через verified-helper (OPS-028 — НЕ bare `git push`):

`${POLISADE_PYTHON:-python3} {plugin_root}/scripts/polisade_vcs.py git-push --branch <branch> --project-root "${POLISADE_WORK_DIR:-.}"`

Если exit=2 — НЕ заявляй success. Верни в ответе `remote_lines` и `reason`

из JSON-вывода и пометь итерацию как failed (PM получит `waiting_pm`).

Верни список применённых исправлений.

```

### 5. Re-review (если был IMPROVE)

- Повторный запуск review (шаг 2) + публикация комментария в PR (шаг 3.1)

- Комментарий публикуется после **каждой** итерации — без исключений

- После 2-й итерации — STOP, ждём PM

```python

iterations = 0

while iterations < 2:

review = run_review(pr, task_id) # external CLI or self

post_pr_comment(pr, review, iterations + 1) # polisade_vcs.py pr-comment

iterations += 1

if review.score >= 8: # PASS

require_reviewable(task_id) # непосредственно перед merge

merge = merge_pr(pr, task_id) # pr-merge {N} … --task {TASK-ID}

if merge.exit_code != 0: # мерж НЕ подтверждён

# waiting_pm и запись отказа уже сделал сам pr-merge (JSON:

# task_status, merge_guard) — статус руками НЕ пиши

reason=f"pr-merge: {merge.reason or merge.stderr}") # дословно

STOP # мерж НЕ повторять, done НЕ писать — решает PM

break

else: # IMPROVE

require_reviewable(task_id) # до любого исправления/коммита/push

# OPS-028: commit_and_push() =

# git commit ... && ${POLISADE_PYTHON:-python3} {plugin_root}/scripts/polisade_vcs.py git-push \

# --branch <branch> --project-root "${POLISADE_WORK_DIR:-.}"

# break # НЕ продолжаем re-review, НЕ мёржим

commit_and_push()

else:

# Max iterations — STOP, ждём PM

require_reviewable(task_id) # не переписывай отмену в waiting_pm

# OPS-010: терминальный waiting_pm без следующего семантического

STOP # вернуть управление PM

```

⛔ **НЕ пиши `lastUpdated`** в PROJECT_STATE.json на любом шаге review-pr —

его пишет ТОЛЬКО `scripts/_polisade_state_io.py` из `polisade_sync.py --apply` / `polisade_migrate.py --apply` (OPS-010). Скилл, пишущий это поле сам, порождает лишний status-only коммит — ровно тот дефект, ради которого контракт видов коммитов и введён. Для времени последнего КОММИТА используй

`git log -1 --format=%cI .state/PROJECT_STATE.json`.

### 6. Merge PR

<!-- polisade:push-stop CAPSULE BEGIN -->

> ⛔ **Если вызов `polisade_vcs.py` на этом шаге ты СДЕЛАЛ, и он отклонён** (`Command references protected path` / `Install directory is read-protected` / `Filesystem Guard`) — **STOP до push**: ни `git-push`, ни `pr-create`, ни `pr-merge`, ни `pr-comment` больше не выполняются. Вызова не было — отказа нет: сначала выполни команду; этот блок описывает её вывод и сам ответом не служит.

> Bare `git push` запрещён (push обязан проверяться сравнением remote SHA — OPS-028); самодельные REST/curl-вызовы к Bitbucket/GitHub запрещены; helper не транскрибируется в `/tmp`.

> Доложи PM дословный текст отказа из вывода этого вызова, ветку с локальным коммитом и то, что pr-create не выполнялся, — и заверши рецепт на этом. Готовой фразы отказа здесь нет: её даёт только вывод вызова.

<!-- polisade:push-stop CAPSULE END -->

```bash

# Merge PR (squash and delete branch). --task: отказ инструмент сам

# записывает в состояние и сам переводит TASK в waiting_pm

```

```

Провайдер блокирует approve собственного PR!

Решение: После успешного quality review — merge напрямую (без approve).

```

**Ненулевой exit — мерж НЕ подтверждён** (сервер отклонил мерж — защита

ветки, конфликт, обязательные проверки, нет прав — или мерж прошёл, а

удаление ветки отказано; что именно, говорит причина): TASK → `waiting_pm`

с причиной дословно — это уже сделал сам `pr-merge`: отказ записан в

`.state/PROJECT_STATE.json` (`mergeRefusals`, причина дословно), задача

= `not_written` — STOP и назови PM `task_status.reason`. Мерж НЕ повторяй и

`done` НЕ пиши — решает PM: повтор того же PR инструмент отклонит сам, без

`pr-merge-ack` — команда человека: сам её НЕ вызывай.

`merge_guard.status` не `recorded` / `refused_repeat` (`not_recorded`,

`unavailable`, `state_unreadable`, `broken`) — отказ НЕ записан и повтор НЕ

### 7. Логирование в session-log

Добавь запись в `.state/session-log/{today}-{seed}.md`, где `{seed}` — поле

`seed:` ревьюируемой TASK (у PR своего семени нет; журнал принадлежит

```markdown

### PR Quality Review: PR #{N} (TASK-{ID})

- Date: {today}

- Reviewer: {REVIEWER_NAME}

- Iteration 1: {score}/10 → {PASS|IMPROVE}

- Iteration 2: {score}/10 → {PASS|IMPROVE} (если была)

- Command: /polisade:review-pr

- Result: merged | improvements_applied

```

## Формат вывода

> В режиме `self` — заменить "PR QUALITY REVIEW" на "PR REVIEW (SELF)", `Reviewer:` — на CLI текущего агента (напр. "Claude Code (self-review)").

### PASS с первой итерации

```

═══════════════════════════════════════════

PR QUALITY REVIEW: PR #{N}

═══════════════════════════════════════════

{строка шапки, которую напечатал `pr-view` в шаге 1, — дословно}

Reviewer: OpenAI Codex CLI (gpt-5.3-codex)

───────────────────────────────────────────

Iteration: 1/2

Score: 8.4/10

- Acceptance criteria: 9/10

- Полнота: 8/10

- Качество: 8/10

- Тесты: 8.5/10 (покрытие AC 9, edge cases 8, поведение vs структура 9,

надёжность 8; сканер test-smells: находок нет)

- Безопасность: 8/10

- Стендозависимые значения: N/A (дифф не трогает data-access/конфиг)

Вердикт: PASS

───────────────────────────────────────────

═══════════════════════════════════════════

```

### IMPROVE → PASS

```

═══════════════════════════════════════════

PR QUALITY REVIEW: PR #{N}

═══════════════════════════════════════════

{строка шапки, которую напечатал `pr-view` в шаге 1, — дословно}

Iteration 1: Score 6.4/10 → IMPROVE

- Полнота: 6/10 — пропущена валидация email

- Тесты: 5/10 (покрытие AC 6, edge cases 3, поведение 6, надёжность 5) —

нет тестов на границы, в test_auth.py:31 sleep(2) [сканер: TS-01]

Применено 3 исправления:

- src/auth/login.ts: добавлена валидация

- tests/auth.test.ts: edge case тесты

- src/auth/login.ts: error handling

Iteration 2: Score 8.6/10 → PASS

───────────────────────────────────────────

═══════════════════════════════════════════

```

### При STOP после 2 итераций (quality gate не пройден)

```

───────────────────────────────────────────

PR QUALITY REVIEW: PR #{N}

───────────────────────────────────────────

{строка шапки, которую напечатал `pr-view` в шаге 1, — дословно}

Iteration 1: Score 5.8/10 → IMPROVE

- Полнота: 6/10 — ...

- Тесты: 5/10 (покрытие AC 6, edge cases 4, поведение 5, надёжность 5) — ...

Iteration 2: Score 7.2/10 → IMPROVE

- Полнота: 7/10 — ...

- Тесты: 7/10 (покрытие AC 8, edge cases 6, поведение 7, надёжность 7) — ...

⛔ QUALITY GATE НЕ ПРОЙДЕН (2/2 итерации)

Последний score: 7.2/10 (порог: 8)

PR #{N} НЕ замержен

{TASK-ID} → waiting_pm

Подробный отчёт последней итерации:

{полный ответ ревьюера с рекомендациями}

Варианты для PM:

→ Ещё итерация исправлений: исправить замечания, push, /polisade:review-pr {N}

→ Ручной review и merge: /polisade:pr merge {N} --squash

→ Отклонить PR: /polisade:pr close {N}

───────────────────────────────────────────

```

### При ошибке ревьюера

```

═══════════════════════════════════════════

PR REVIEW: PR #{N} — ОШИБКА

═══════════════════════════════════════════

{текст ошибки}

Возможные причины (режим codex):

- Codex CLI не установлен (codex)

- Нет OPENAI_API_KEY

- Timeout (увеличь --timeout)

Возможные причины (режим self):

- CLI агента не установлен (claude / qwen / qwen-code)

- Не авторизован / нет API-ключа

- Timeout

-> Повтори: /polisade:review-pr {N}

═══════════════════════════════════════════

```

## Интеграция с автономным циклом

Когда вызывается из `/polisade:continue` или `/polisade:implement`:

```

1. Ревьюер (Codex CLI или CLI текущего агента в режиме `self`) получает чистый контекст

2. Субагент pre-fetch'ит diff и PR description, ревьюер читает TASK, parent, гайдлайны проекта (из корня)

3. Оценивает PR diff vs TASK requirements (независимый reviewer)

4. Если PASS → основной агент делает merge

5. Если IMPROVE → improvement субагент исправляет → re-review

6. После 2 итераций с score < 8 → STOP, waiting_pm (PM decides)

```

**Режим Codex (по умолчанию):** ревью делает OpenAI Codex CLI — другая модель, другой провайдер, полностью независимое второе мнение.

**Режим `self`:** ревью делает тот же агент, но в изолированном CLI-процессе (чистый контекст). Не полноценное "второе мнение", но независимость от текущей сессии.

## Важно

- Ревьюер запускается в автономном режиме — сам навигирует проект (Codex: `--full-auto`, self: зависит от CLI)

- Diff и PR description pre-fetch'атся субагентом и передаются в промпт ревьюера

- Improvement субагент наследует модель parent — качественное применение рекомендаций

- Максимум 2 итерации review+improve — anti-loop safety

- После 2 итераций — STOP + waiting_pm, PM решает дальнейшие действия

- PM не делает code review — это автоматизированный процесс

- GitHub не позволяет approve свой PR — merge напрямую после PASS

- Timeout 300s — достаточно для анализа PR