docs(todo): 順位264 追加 — pre-push review-diff を jj diff --git 化 (PR #256 post-merge-feedback Tier1 #1 採用)

[aloekun]

概要

PR #256 の post-merge-feedback Tier1 #1 で採用された改善タスクを todo に登録する（実装は後日）。

pre-push review の .takt/review-diff.txt は jj diff -r @（jj デフォルト形式）で生成され、色を落とすと削除が - マーカー無しになり、LLM レビュアーが「削除」を「追加」と誤読しうる。実際 PR #256 で todo 25行の削除を simplicity-review が false positive REJECT し、レビュー約19分を浪費した。jj diff --git -r @ への切替で解消する（config + template の2箇所）。

変更内容

• docs/todo13.md — 順位264 の詳細エントリ追加（動機・設計決定案・作業計画・完了基準）
• docs/todo-summary.md — 順位264 行追加（🔧 Tier 2 / Effort S）

他6提案（todo-consistency linter 新設 / ADR 化 / 啓発ルール等）は post-merge-feedback で over-engineering として却下推奨のため登録しない。

補足

本 PR 作成時の pre-push simplicity-review が本エントリの ADR-033 違反（本文中の順位番号直書き）とテンプレート不整合を2件検出 → takt fix が自動修正済み。

Summary by CodeRabbit

• Documentation
  • 実行順序サマリーに新しいタスクを追加し、差分表示の扱いに関する改善案を追記しました。
  • TODO一覧に、差分の表示形式を見直すための項目と、確認ポイント・完了条件を追加しました。

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 054c67ab-f181-4c51-8dfc-2666e447bfaa

📥 Commits

Reviewing files that changed from the base of the PR and between 6f6758e and b0810e1.

📒 Files selected for processing (2)

• docs/todo-summary.md
• docs/todo13.md

---

📝 Walkthrough

Walkthrough

pre-push時の review-diff.txt 生成方式を jj diff から jj diff --git に切り替える計画を記載したTODOエントリが docs/todo13.md に追加され、docs/todo-summary.md の推奨実行順序サマリーテーブルに対応する行(順位264, Tier 2)が追加された。

Changes

TODOドキュメント追加

Layer / File(s)	Summary
pre-push diff形式変更のTODO追加 docs/todo13.md, docs/todo-summary.md	jj diff --git -r @ への切替を計画するTODOエントリ(設計・検証観点・完了基準)を追加し、サマリーテーブルにTier 2・順位264の行を追加した。

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

• aloekun/claude-code-hook-test#133: 同じ docs/todo-summary.md の「推奨実行順序サマリー」テーブル構造を対象としており、本PRはそこに新しいTier 2行を追加している。

🚥 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	順位264のTODO追加と、pre-push review-diff を jj diff --git 化する主変更を正確に要約しています。
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.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch rank264-review-diff-git-format

---

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.}

[aloekun]

@coderabbitai review

[coderabbitai]

✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.