docs(todo): PR #83 post-merge-feedback の Tier 3 finding を採用 (T3-1 新規 + T3-2 即時 / T3-3 却下)

[aloekun]

Summary

• PR docs(todo): PR #82 post-merge-feedback の Tier 1 / Tier 3 finding をタスク化 + ファイルサイズ調整 #83 post-merge-feedback report (.claude/feedback-reports/83.md) の Tier 3 finding 3 件を評価し、2 件採用 + 1 件却下
• T3-1 (却下済み変更の再適用防止グローバルルール明文化): 新規タスクとして docs/todo2.md に追加 (Tier 1 順位 3/13)
• T3-2 (todo.md / todo2.md 運用ルール明文化): docs/todo.md 冒頭の運用ルール blockquote を即時強化
• T3-3 (Markdown-only PR review skip): ADR-032 既存タスクと重複のため却下
• 副次的に推奨実行順序サマリー表を 12 件 → 13 件に再採番、関連する cross-reference を全て更新

採用と却下の判断

#	内容	採否	配置先
T3-1	却下済み変更の再適用防止グローバルルール (~/.claude/CLAUDE.md or coding-style.md)	✅ 採用 (新規タスク化)	docs/todo2.md (Tier 1 順位 3/13)
T3-2	docs/todo.md / docs/todo2.md の役割分担運用ルール明文化	✅ 即時実施	docs/todo.md 冒頭 blockquote 強化
T3-3	Markdown-only PR の review facet skip guard	❌ 却下	ADR-032 (順位 9-11/13) と完全重複のため吸収

T3-1 (新規タスク) の動機

本セッション (2026-04-27) で T3-combined を todo2.md に移動する Edit 中、AI が無意識に直前にユーザーが却下した CodeRabbit MD040 finding (``` → ```text) を付随適用してしまった事象が発生。即座に revert したが、構造的に防ぐルールがあれば防げる事案。post-merge-feedback (PR #83) が Tier 3 として独立に提案。

ルール文 (案):

却下済み変更の再適用禁止: セッション内でユーザーが明示的に却下した変更 (CodeRabbit findings の reject、design choice の reject など) は、後続の Edit / Write 操作で 付随適用してはならない。AI が「整合性のため」「ついでに」「ベストプラクティスだから」として再適用するのも禁止。例外: ユーザーが新たに同セッションで明示的に採用を表明した場合のみ。

順位 1 (git-workflow.md) / 順位 2 (ブランチ保護方針) / 順位 3 (本タスク) はすべて ~/.claude/ 配下のグローバルルール群のため 並列実施可能。

T3-2 (即時実施) の内容

todo.md 冒頭の運用ルール blockquote を強化:

• docs/todo.md: 既存タスクの編集・完了削除専用 (~50KB 閾値内維持)
• docs/todo2.md: 新規タスクの追加先
• 例外: 既存 todo.md タスクと 同一ファイル / 同一コンポーネント を編集する密結合タスクは todo.md に追加可

本コミット自身がこのルールに従って T3-1 を todo2.md に追加することで、ルールの自己適用を実証。

副次的な整合性更新

• 推奨実行順序サマリー表: 12 件 → 13 件、全順位再採番
• 既存 5 タスクの 実行優先度 line: /12 → /13 更新、cross-reference 順位調整
• todo2.md ADR-032 sub-phase listing: /10 → /13 に統一 (前回 renumbering 漏れを併せて修正)
• 戦略コメント: 「順位 1-2 はグローバルルール」→「順位 1-3 はグローバルルール」

ファイルサイズ管理

ファイル	編集前	編集後	閾値 (50KB)
docs/todo.md	49KB	49KB (微増)	✅ 内
docs/todo2.md	29KB	33KB	✅ 余裕あり

T3-1 task entry を todo.md ではなく todo2.md に追加することで todo.md の閾値超過を回避。

Test plan

• docs/todo.md 冒頭の運用ルール blockquote 強化を確認 (T3-2 反映)
• docs/todo.md 推奨実行順序サマリー表が 13 件に拡張、新規 T3-1 が順位 3 に挿入されていることを確認
• docs/todo2.md 末尾に T3-1 (却下済み変更の再適用防止グローバルルール) タスクが追加されていることを確認
• 既存 5 タスクの 実行優先度 行が /13 に更新されていることを確認
• todo2.md ADR-032 sub-phase listing が /13 に統一されていることを確認 (Phase pre 5/13, Phase α 6/13, Phase broken-link 8/13, Phase β 9/13, Phase γ 10/13, Phase δ 11/13)
• todo.md ファイルサイズが 50KB 以内に収まっていることを確認

関連

• 補完対象: PR docs(todo): PR #82 post-merge-feedback の Tier 1 / Tier 3 finding をタスク化 + ファイルサイズ調整 #83 post-merge-feedback report
• 自己実証: T3-2 のルールが本コミット内で自己適用される (T3-1 が todo2.md に配置されている)

Summary by CodeRabbit

Documentation

• タスク管理プロセスの運用ルールを恒久化し、タスク分類の基準を明確化しました
• 優先度ランキング体系を統一し、各タスクの実行順序情報を整備
• 却下済みの変更が再度適用されることを防止する新ルールを導入

[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: 5f631a93-efa2-403b-bf5a-2a8ed95aadf9

📥 Commits

Reviewing files that changed from the base of the PR and between e4749b8 and d6d7645.

📒 Files selected for processing (2)

• docs/todo.md
• docs/todo2.md

---

📝 Walkthrough

Walkthrough

PR #83の反映として、docs/todo.mdの運用ルールを恒久化し、既存タスク編集と新規タスク追加を分離。あわせてdocs/todo2.mdに新規グローバルルール（却下済み変更の再適用防止）を追加し、全体の実行優先度の採番を12/12から13/13へ更新。

Changes

Cohort / File(s)	Summary
Task Documentation Updates docs/todo.md, docs/todo2.md	実行優先度採番スキームを12/12から13/13へ更新。todo.mdの運用ルール（既存タスクのみ編集、新規は原則todo2へ）を恒久化。todo2.mdに新規Tier-1グローバルルール（PR #83 T3-1：ユーザーが却下した変更の再適用防止）を追加し、詳細な実装指示と完了チェックリストを記載。各タスクの実行優先度ランク数値と依存関係参照を全体的に再採番。

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• docs(todo): PR #81 post-merge-feedback の知見反映 + ADR-032 (docs-only fast path) タスク化 #82: 本PRで更新するdocs/todo.mdとdocs/todo2.mdのタスク優先度採番が、PR #82で導入されたADR-032とtodo分割構成の直接的な継続更新であるため関連。
• docs(adr): ADR-030 を起案 — 決定論的 post-merge-feedback (Phase A) #75: 本PRで追加されるグローバルルールがCLAUDE.md（~/.claude/CLAUDE.md）を参照し、PR #75で導入されたADR-030との関連性があるため。
• docs(todo): 現在進行中タスクの棚卸しと pre-push-review 調査結果を追記 #43: 本PRで実施されるtodo/todo2運用ルールの恒久化が、PR #43で導入されたタスク状態管理ルールの延長線上にあるため関連。

🚥 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 #83 のフィードバックに基づく Tier 3 の指摘を反映させた内容で、新規タスク追加（T3-1）、即時対応（T3-2）、却下判定（T3-3）を明確に示しており、主要な変更内容を正確に要約している。
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.}