docs: make the review mechanism an explicit, verifiable choice - #139
Merged
Merged
Conversation
Независимая проверка запускалась через произвольный транспорт — отдельную cmux-сессию, вручную собранный codex exec или иной ad hoc механизм. Результат приходилось вручную извлекать из логов, а review contract зависел от локального UI и orchestration setup. Разделено по слоям, потому что `code-converge` — конкретный внешний инструмент, а `template/memory-bank/` уходит в произвольные downstream-проекты, где его нет: - в generic-слой (`flows/testing-policy.md`) добавлены требования к любому механизму: structured verdict, fail closed, review-only и разделение автора и проверяющего. Terminal и UI-оркестраторы объявлены средством размещения вкладок, а не механизмом проверки; - в project-adaptation слой (`engineering/testing-conventions.md`) добавлен слот, где проект называет свой механизм и точные вызовы двух режимов; - собственный выбор этого репозитория — `code-converge` — зафиксирован в `AGENTS.md` вместе с командами: обычный review-режим для реализации и `--document-review --max-cycles 0` для документов и артефактов. Нулевой fix budget разделяет роли: проверяющий возвращает findings, но не правит проверяемую revision; исправляет автор, после чего запускается новая проверка новой revision — это уже требует контракт сходимости из #137. Closes #120 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01RtxeRwhVf1y8zvxUtk6xrS
dapi
added a commit
that referenced
this pull request
Sep 6, 2026
…bsystem * origin/main: docs: make the review mechanism an explicit, verifiable choice (#139) docs: add WAIT status for external events without a human gate (#138) docs: bound review convergence across delivery flows (#137) chore: pin memory-bank-cli v2.3.0 in CI (#136) refactor: make project-local memory bank a projection of the payload (#134)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #120.
Проблема
Независимая проверка запускалась через произвольный транспорт: отдельную cmux-сессию, вручную собранный
codex execили иной ad hoc механизм. Результат приходилось вручную извлекать из логов и интерпретировать, а review contract зависел от локального UI и orchestration setup. В #117 это дошло до того, что cmux использовался как механизм проверки, хотя его дело — размещать вкладки.Проектное решение: два слоя вместо одного
Issue просит назвать
code-convergeканоническим механизмом в governance. Ноcode-converge— конкретный внешний инструмент, аtemplate/memory-bank/уходит в произвольные downstream-проекты, где его нет;AGENTS.mdпрямо запрещает возвращать project-specific specialization в шаблон.Поэтому требование разделено:
flows/testing-policy.md— genericengineering/testing-conventions.md— project-adaptationAGENTS.md— этот репозиторийcode-convergeи его точные командыDownstream-проект с другим инструментом получает рабочий контракт, а не мёртвую ссылку.
Требования к механизму (generic)
Terminal и UI-оркестраторы открываются только по прямой просьбе человека.
Выбор этого репозитория
--max-cycles 0даёт проверяющему нулевой fix budget — именно это технически разделяет роли: он возвращает findings, но не правит проверяемую revision.Повторная проверка после исправлений отдельно прописывать не пришлось: контракт сходимости из #137 уже требует, чтобы изменение кода аннулировало verdict.
Проверки
lintшаблона,lint --repo-root .,doctor --profile template, валидатор priming-манифестов,git diff --check— всё зелёное. Якорь#механизм-проверкипроверен вручную:lintякоря не валидирует.🤖 Generated with Claude Code
https://claude.ai/code/session_01RtxeRwhVf1y8zvxUtk6xrS