Добавить настраиваемый и документный промпт для review

[dapi]

Контекст

Сейчас review-стадия использует встроенную инструкцию Code-Converge для schema-constrained Codex review. Нужен отдельный режим ревью документации: он должен проверять документы, изменённые в текущей ветке, с точки зрения системной инженерии и правил Memory Bank.

Результат

Пользователь может выбрать инструкцию для review-стадии одним из следующих способов:

• передать путь к Markdown-файлу с промптом;
• передать имя промпта, чтобы утилита загрузила .code-converge/<NAME>.md;
• одним аргументом выбрать встроенный дефолтный промпт документного ревью вместо обычного code review;
• отдельной командой или аргументом записать встроенный дефолтный промпт в .code-converge/default.md, чтобы затем адаптировать его в проекте.

Встроенный промпт должен поручать агенту проверить изменённые в ветке документы на:

• консистентность и отсутствие противоречий;
• отсутствие незакрытых существенных вопросов;
• соответствие принципам Memory Bank.

Границы

• Это не замена существующего code review по умолчанию: новый режим включается явно.
• Изменение затрагивает CLI/config contract и должно быть реализовано в Feature Flow, с feature package в memory-bank/features/.
• Вопросы публичного контракта, которые нельзя выводить из запроса, должны быть решены в feature package: точные имена флагов/команды, приоритет источников, допустимость путей вне repository root, поведение при конфликте выбора и правила перезаписи существующего .code-converge/default.md.

Acceptance criteria

• Пользователь может передать путь к файлу промпта для review.
• Пользователь может выбрать именованный промпт, загружаемый из .code-converge/<NAME>.md.
• Пользователь может явно выбрать встроенный промпт документного ревью одним аргументом.
• Встроенный промпт ограничивает анализ изменёнными в текущей ветке документами и покрывает консистентность, непротиворечивость, незакрытые вопросы и Memory Bank principles.
• Имеется явный CLI-механизм экспорта встроенного промпта в .code-converge/default.md для пользовательского редактирования.
• Ошибки чтения/разрешения промпта и конфликты выбора диагностируются предсказуемо, без неявного fallback.
• README описывает итоговый CLI/config contract и примеры использования.
• Feature package содержит routing rationale, выбранный validation profile и проверяемые evidence/тесты; реализация проходит применимые repository checks.

Routing

Feature Flow. Задача меняет внешний CLI/config contract и добавляет новый наблюдаемый режим работы; Small Change не применим.

[dapi]

FPF decision record

Prompt

Как сделать документное ревью явным, настраиваемым и безопасным для существующего workflow?

Контекст и инварианты

• Обычный code review остаётся поведением по умолчанию.
• Результат любого review должен оставаться совместим с существующей строгой JSON-схемой, классификацией finding и циклом исправления.
• Пользовательский выбор не должен зависеть от неявного поиска или конфигурационной precedence.
• Документный режим должен обозревать тот же merge-base-to-worktree snapshot, что и обычный review, но только свой тип файлов.

Альтернативы

Вопрос	Альтернативы	Решение
Выбор промпта	один неоднозначный аргумент; отдельные флаги	отдельные взаимоисключающие флаги: --review-prompt-file <path>, --review-prompt <name>, --document-review
Именованный промпт	произвольный путь; project-local имя	--review-prompt <name> принимает только безопасное имя без path separators и суффикса; читает .code-converge/<name>.md
Default	всегда встроенный; всегда файл; файл с fallback	--document-review использует .code-converge/default.md, если он существует, иначе встроенный шаблон
Экспорт	молча перезаписывать; безопасный bootstrap	подкоманда code-converge init-document-review-prompt [--force]: создаёт .code-converge/default.md; существующий файл не заменяется без --force
Scope	все файлы; неформальный «документы»	все изменённые Markdown-файлы (.md) в review snapshot; memory-bank/prompts/** исключён согласно repository policy
Формат находок	новая схема; общая схема	общая текущая JSON-схема: finding ссылается на Markdown-файл и строковый диапазон; user prompt дополняет обязательную инструкцию/schema, а не заменяет её

FPF reasoning

Bounded contexts разделяют обычный code review, document-review criteria и механизм выполнения workflow: режим меняет предмет анализа, но не транспорт результата, budget или публикацию. Из альтернатив отдельные флаги выигрывают по точности и диагностируемости; fallback default.md → built-in даёт изменяемый project template без обязательного bootstrap. Явная Markdown-область не даёт «документам» дрейфовать по смыслу. Запрет неявной перезаписи сохраняет пользовательские изменения.

Дедукции / acceptance additions

• Одновременная передача любых двух из трёх review-флагов — configuration error с exit 2.
• Несуществующий/нечитаемый файл, недопустимое имя или ошибка создания prompt file — configuration/operational error с exit 2; fallback применяется только для отсутствующего .code-converge/default.md в --document-review.
• Если в document-review snapshot нет изменённых Markdown-файлов, review завершается clean без вызова Codex и с явным диагностическим/structured событием scope-empty.
• --document-review является явным ограничением scope: изменения не-Markdown файлов не анализируются этим review; для code review пользователь запускает обычный режим.
• Требуются table-driven tests для всех источников, конфликтов, path/name validation, fallback, --force, Markdown filtering и сохранения strict-schema failure semantics; README должен документировать команды и это scope limitation.

Assurance

Это selected design hypothesis с высокой уверенностью в compatibility и safety: он основан на текущем file-backed prompt pattern и строгой review-схеме. Реализация должна подтвердить его детерминированными тестами и independent convergence review в Feature Flow.

[dapi]

FPF addendum — prompt для исправления документных findings

Решение

Добавить опцию --document-fix-prompt-file <path>.

• Опция допустима только вместе с --document-review.
• Она передаёт содержимое указанного Markdown-файла в стадию исправления findings документного ревью.
• Если опция не задана, используется встроенный документный fix prompt: исправить только подтверждённые findings в изменённых Markdown-документах, сохраняя их intent, согласованность и принципы Memory Bank.
• В документном режиме --document-fix-prompt-file и существующий --fix-prompt-file взаимоисключающие. Обычный --fix-prompt-file продолжает обслуживать только стандартный code-review workflow.
• Нечитаемый/несуществующий путь — configuration error с exit 2; файл не ищется неявно и не имеет fallback, кроме встроенного prompt при полном отсутствии опции.

Почему

Review prompt задаёт критерии выявления, а fix prompt — действие по устранению findings; это разные bounded contexts и их нельзя связывать одним неявным источником. Отдельный флаг сохраняет обратную совместимость и даёт однозначное правило выбора.

Дополнительные acceptance criteria

• Документный режим запускает fix stage с явно выбранным --document-fix-prompt-file либо со встроенным документным fix prompt.
• Обычный workflow и его --fix-prompt-file не меняются.
• Конфликт двух fix-prompt флагов, отсутствие --document-review и ошибки чтения имеют детерминированную диагностику и exit 2.
• Тесты проверяют resolved prompt и передачу полного document-review report в fix stage; README содержит пример.

[dapi]

Feature Flow package: (routing, validation profile, scope and verification contract).

[dapi]

Feature Flow package: memory-bank/features/FT-042/brief.md (routing, validation profile, scope and verification contract).