docs(takt): post-merge-feedback report の wording 強化 — ✅ 採用 → ✅ 採用候補 で AI agent の auto-adopt 防止

[aloekun]

Summary

.takt/facets/instructions/aggregate-feedback.md (post-merge-feedback report 生成元) の wording を強化し、AI agent が analyzer 推奨を読んだだけで自動採用処理に進む anti-pattern を構造的に防止する。PR #179 land 後セッションでの実観測事例 (Claude が「✅ 採用」を auto-adopt し docs/todo*.md に無断登録) を契機にユーザー判断で wording 改訂。

Background

PR #179 直後の post-merge-feedback で Claude (本セッション) が「✅ 採用」と書かれた 2 件を ユーザー明示承認なしに docs/todo9.md + docs/todo-summary.md に自動登録するという process 違反を起こした。ユーザー指摘により revert 済。原因分析の結果、report 自体の wording (✅ 採用 = 「採用済み」と読める) が auto-adopt を invite していると判明し、source 側の wording を「採用候補」に改訂することで決定論的に防止する方針が確定。

Changes

1. Recommendation 列値の rename

旧	新
✅ 採用	✅ 採用候補
🤔 様子見	(変更なし)
❌ 却下	❌ 却下推奨

「採用」「却下」は確定形を示唆 → 「候補」「推奨」で analyzer 推奨 + ユーザー判断必要 を明示。

2. Phase 2 intro に AI agent 明示禁則 blockquote 追加

> **AI agent への明示禁則**: 本 report の生成完了 = ユーザーへの提示完了に過ぎず、
> Claude / Codex / Opencode 等の agent は `✅ 採用候補` を読んだだけで採用処理
> (`docs/todo*.md` への entry 追加 / 実装着手 / ADR 編集 等) に進んではならない。
> **必ずユーザーの明示承認 (AskUserQuestion 回答 or テキスト承認のいずれか) を待つこと**。

Phase 2 (Severity/Frequency/Effort/Adoption Risk/Recommendation を埋める段) の冒頭に配置 → agent が rubric を埋め始める前に禁則を読む構造。

3. Recommendation rubric table 強調文追加

✅ 採用候補 / ❌ 却下推奨 行に「判定は analyzer 推奨であり、ユーザー承認後に採用/却下確定」を inline で追記。

4. 「次のアクション」section 全面改訂

旧: 「✅ 採用は基本採用、🤔 様子見は dogfood トリガ次第、❌ 却下は不要」(自動採用を許容するように読める)

新: 「重要: ... ユーザー明示承認なしに採用・却下を確定してはならない。Claude / 他 AI agent は report を読んだだけで docs/todo*.md への entry 追加、実装着手、ADR 編集等を実行してはならず、必ずユーザー承認 (AskUserQuestion 回答 or テキスト承認のいずれか) を待つこと」(禁則の明示 + 承認形式の許容範囲を具体化)

5. Rationale 例 + 品質フィルタ cross-ref の表記更新

• 「採用 例:」→「採用候補 例:」
• 「却下 例:」→「却下推奨 例:」
• 品質フィルタの ❌ 却下 言及を ❌ 却下推奨 に更新

Memory rule 連動更新 (本 PR 範囲外、global 配下)

• feedback_post_merge_feedback_adoption_requires_user_approval.md: How to apply に「AskUserQuestion 回答 or テキスト承認のいずれか」明記、新 wording (採用候補/却下推奨) 反映
• feedback_coderabbit_no_actionable_merge_signal.md: 「ユーザーから OK」を「AskUserQuestion 回答 or テキスト承認のいずれかで OK」と clarify

Test plan

• markdownlint clean
• pnpm push pre-push-review (simplicity + security) APPROVED — simplicity reviewer は「removed line 169 was a pre-existing within-file contradiction; new wording aligns with the table」と semantic 検証済
• 次の post-merge-feedback 実行で新 wording が出力されることを確認 (dogfood)

観測効果 (期待)

新 report (PR #180 以降の post-merge-feedback) は:

• Recommendation 列に「✅ 採用候補」「❌ 却下推奨」を表示
• Phase 2 intro / 「次のアクション」section で AI agent に対し明示禁則
• compaction 後にコンテキストが消えても、report 自体が「analyzer 推奨であり確定判断ではない」と明示するため、Claude が誤って auto-adopt するリスクが構造的に低減 (PR feat(cli-docs-lint): preamble + cross-ref validator in push-runner quality_gate (ranks 95+96 land) #179 後事例の再発防止)

Summary by CodeRabbit

• ドキュメント
  • 推奨ラベルの表記と意味を明確化しました（「✅ 採用候補」「❌ 却下推奨」など）
  • ユーザーの明示的な承認なしで処理を進めないルールを明文化しました
  • AIエージェントの動作制限と推奨列の運用ルールを整理しました

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 7d2e9736-bf6d-45cc-98ca-b2ee7025d5ad

📥 Commits

Reviewing files that changed from the base of the PR and between 91f2939 and ad4cce0.

📒 Files selected for processing (1)

• .takt/facets/instructions/aggregate-feedback.md

---

📝 Walkthrough

Walkthrough

フィードバック集約手順書が更新され、Tier 1→2→3 の並び要件が明示され、Recommendation ラベルが「採用」から「採用候補」等に改められた。AI agent の実行禁則とユーザー承認に基づく運用ルールが追加された。

Changes

フィードバック集約ルール更新

Layer / File(s)	Summary
基本ルール定義：Tier 並び要件と実行禁則 .takt/facets/instructions/aggregate-feedback.md	Phase 1 で最終リスト作成時の Tier 順序ルールが明示され、Phase 2 で AI agent が報告書読了だけで採用/却下処理に進むことを禁ずる条項が追加された。
Recommendation ラベル体系の更新 .takt/facets/instructions/aggregate-feedback.md	Recommendation rubric の選択肢が「✅採用」→「✅採用候補」、「❌却下」→「❌却下推奨」に改められ、Rationale 例文と Tier 1 テンプレート例が合わせて更新された。
次のアクション節での運用ルール明文化 .takt/facets/instructions/aggregate-feedback.md	ユーザー承認なしの処理を禁止する重要注意が追加され、各 Recommendation 状態の運用方法、docs/todo\*.md 登録の要件、Claude 単独却下の禁止が明文化された。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• aloekun/claude-code-hook-test#107: 同じドキュメントの Recommendation 列の意味と運用ルール、rubric 体系の更新を扱う直接関連の PR。
• aloekun/claude-code-hook-test#84: 本 PR で追加された「ユーザー承認なしで docs/todo\.md 追加や実装着手を行わない」実行禁則と、retrived PR での docs/todo\.md 運用ルール変更が直結している。

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	PR タイトルは、AI agent の自動採用を防止するため Recommendation ラベルを「✅ 採用」から「✅ 採用候補」に変更する主要な目的を明確に述べており、変更内容を正確に反映している。
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}