docs: Bundle j 登録 + Phase c MVP land 反映 (PR #132 post-merge-feedback)

[aloekun]

Summary

• docs/todo6.md / docs/todo-summary.md に Bundle j 3 タスク (順位 94-96) を登録
  • 順位 94 (Tier 1, S): docs/ 内 ../docs/ 相対パストラップ検出 lint rule
  • 順位 95 (Tier 2, S): docs/todo*.md preamble file count 自動照合スクリプト
  • 順位 96 (Tier 2, M): Markdown cross-reference validator CI step
• docs/local-llm-offload-analysis.md に Phase c MVP 完了 (PR feat(cli-push-runner): Phase c MVP — pre-push lint-screen step (default OFF) #132 land) を反映
  • 実装方針変更 (takt facet → cli-push-runner stage) を §1 Phase c に明記
  • Phase c+ (Bundle i) として PR feat(cli-push-runner): Phase c MVP — pre-push lint-screen step (default OFF) #132 post-merge-feedback 採用 3 件 (順位 91/92/93) を追記
  • §8.E を MVP land 済み状態に更新
  • 再開チェックリストに Phase c smoke / Bundle i 着手手順を追加
• docs/todo-summary.md retrospective に Bundle j (PR docs(todo): split summary and todo5.md tasks to keep files under 50KB #133 post-merge-feedback サマリー) を追記

Background

• PR docs(todo): split summary and todo5.md tasks to keep files under 50KB #133 (todo5.md 50KB 分割) merge 後の post-merge-feedback で 3 件の docs/ 整合性多層検証タスク (Bundle j) を採用
• PR feat(cli-push-runner): Phase c MVP — pre-push lint-screen step (default OFF) #132 (Phase c MVP) merge 済み。Phase d 着手前に Bundle i (順位 91-93) の処理が前提
• 本 PR は ADR-035 docs-only PR 判定: 全変更ファイル docs/** 配下、executable code logic 変更なし

Test Plan

• pnpm push パイプライン (lint/test/build/rust-test) PASS
• takt pre-push-review (security-review + simplicity-review) ともに APPROVE
• ADR-035 docs-only judgment 適用 (人間レビュー軽め運用)
• Markdown link 整合性確認 ((todo6.md) リンクが docs/local-llm-offload-analysis.md から正しく解決される)

Summary by CodeRabbit

• Documentation
  • 実装進捗状況を更新（Phase b/c の達成条件と実装内容を明記）
  • 再実行手順と再開チェックリストを追記
  • ドキュメント検証用の新規タスクを追加（相対パス検査、リンク整合性確認など）

[coderabbitai]

📝 Walkthrough

概要

PR #134は、Phase c（lint screen機能）の試験運用ステータス更新、MVP実装詳細の記録、および後続タスク（Bundle j）の定義を含むドキュメント更新です。Phase b/cの達成条件を反映し、cli-push-runner Rust stageでの実装、フォールバック動作、設定方針、観測事象（mistral:7b JSON欠落）とBundle iフォローアップを詳細化。再開チェックリストと優先度リストを追加し、docs整合性多層検証（3タスク）をBundle jとして新規定義します。

変更内容

Phase c試験運用の実装記録と再開ガイド

レイヤー / ファイル	概要
ステータス更新 docs/local-llm-offload-analysis.md	Phase b/c達成条件（conditional GO、PR番号、日付）およびPhase d未着手を状態欄に記載。
MVP実装詳細化 docs/local-llm-offload-analysis.md	Phase c（§8.E）をcli-push-runner Rust stageでのMVP実装に詳細化、出力・パイプライン・設定・フォールバック・レビュー連携を記録。mistral:7b大規模diff時JSON欠落観測とBundle iフォローアップ（config検証、fixtures、coding-style anti-pattern）を追加。
提案仕様更新 docs/local-llm-offload-analysis.md	§8.E（提案1）をPR #132 MVP land済み方針に更新、scope（報告のみ・gating無・auto-fix無・default OFF）とBundle i・Phase d着手前提を明記。
再実行ガイド docs/local-llm-offload-analysis.md	別セッション再開チェックリスト（master最新化→Phase a/c反映→Ollama起動→Phase b再現→Phase c MVP smoke）と優先度リスト（Bundle i→coding-style→Phase d）を追加。
Bundle jタスク定義（概要表） docs/todo-summary.md	PR #133対応Bundle j（docs整合性検証）の3タスクを推奨実行順序表に追加（順位94〜96）、タスク詳細・工数・依存関係を記載。
Bundle j仕様説明 docs/todo-summary.md	Bundle j説明文（採用対策・Sub-PR分割推奨・却下/様子見根拠）を戦略セクションに追記。
Bundle jタスク詳細 docs/todo6.md	3タスク詳細（相対パストrap検出lint、preamble自動照合、cross-reference validator）を設計案・作業計画・完了基準付きで追加。

関連PR

• aloekun/claude-code-hook-test#133: docs分割とanchor変更で導入されたファイル構造上の整合性検証タスク（Bundle j）がPR #134で詳細定義される
• aloekun/claude-code-hook-test#122: Phase/Bundle実装ステータスと§8.Eの同一ドキュメント領域（local-llm-offload-analysis.md）を更新
• aloekun/claude-code-hook-test#124: docs/todo6.mdとtodo-summary.mdへのBundle/タスク追加で同じドキュメント構造を拡張

🎯 2 (Simple) | ⏱️ ~12 分

🚥 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	The title clearly and specifically describes the main changes: adding Bundle j tasks and reflecting Phase c MVP completion with PR #132 post-merge feedback. It is concise and directly related to the documentation updates.
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.}

[coderabbitai]

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

docs/todo-summary.md (1)

8-8: ⚠️ Potential issue | 🟡 Minor | ⚡ Quick win

更新日が最新変更と一致していません。

Line 8 の見出しが「2026-04-29 更新」のままですが、今回 2026-05-09 の内容（Bundle j）を追加しています。検索性と運用精度のため、更新日を最新化してください。

✏️ 修正案

-## 推奨実行順序サマリー (2026-04-29 更新、ADR-033 採番管理簡素化 land 後)
+## 推奨実行順序サマリー (2026-05-09 更新、ADR-033 採番管理簡素化 land 後)

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@docs/todo-summary.md` at line 8, 見出し "## 推奨実行順序サマリー (2026-04-29 更新、ADR-033
採番管理簡素化 land 後)" の更新日が古いので、今回追加した内容（Bundle j）を反映するために見出し内の日付を "2026-05-09"
に更新し、必要ならカッコ内の補足（例: "Bundle j 追加"）を追記して検索性と運用精度を保ってください。

🧹 Nitpick comments (1)

docs/local-llm-offload-analysis.md (1)

149-149: ⚡ Quick win

テスト件数の固定値は将来すぐ陳腐化しやすいです。

Line 149 の「47+ 件 pass」は保守コストが高いので、件数を外して「全件 pass」などの表現に寄せると運用が安定します。

🛠️ 文言の簡易修正案

-cargo test -p cli-push-runner                                    # 47+ 件 pass
+cargo test -p cli-push-runner                                    # テストが全件 pass することを確認

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@docs/local-llm-offload-analysis.md` at line 149,
行末の固定テスト件数表記を削除して将来の陳腐化を防いでください: "cargo test -p cli-push-runner"
の出力説明で現在使っている「47+ 件 pass」を「全件 pass」や「All tests
passed」のような汎用的な表現に置き換え、同じ行（該当テキスト）で数値に依存しない文言に修正してください。

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Outside diff comments:
In `@docs/todo-summary.md`:
- Line 8: 見出し "## 推奨実行順序サマリー (2026-04-29 更新、ADR-033 採番管理簡素化 land 後)"
の更新日が古いので、今回追加した内容（Bundle j）を反映するために見出し内の日付を "2026-05-09" に更新し、必要ならカッコ内の補足（例:
"Bundle j 追加"）を追記して検索性と運用精度を保ってください。

---

Nitpick comments:
In `@docs/local-llm-offload-analysis.md`:
- Line 149: 行末の固定テスト件数表記を削除して将来の陳腐化を防いでください: "cargo test -p cli-push-runner"
の出力説明で現在使っている「47+ 件 pass」を「全件 pass」や「All tests
passed」のような汎用的な表現に置き換え、同じ行（該当テキスト）で数値に依存しない文言に修正してください。

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 915abf1a-b11c-4e9d-bee3-d0fc673e8d10

📥 Commits

Reviewing files that changed from the base of the PR and between 4210e79 and 7d29cca.

📒 Files selected for processing (3)

• docs/local-llm-offload-analysis.md
• docs/todo-summary.md
• docs/todo6.md