docs: rank 8 handoff + PR #177 post-merge-feedback adoption

[aloekun]

Summary

順位 8 (週次レビュー Phase B) 着手用 handoff 資料の作成と、PR #177 post-merge-feedback で採用判定された 2 件の follow-up タスクを docs に登録する docs-only PR。

Commits

change_id	description
oozvoyxs	docs(handoff): 順位 8 (週次レビュー Phase B) 着手用引き継ぎ資料を作成
npzmsvvv	docs(todo): PR #177 post-merge-feedback 採用 2 件を todo9.md / summary table に登録

Details

1. 引き継ぎ資料 (docs/handoff-rank-8-weekly-review-phase-b.md)

別セッション継続用の試験運用 ephemeral 計画書。順位 8 Phase B (週次レビューパイプラインの takt workflow + facets) 着手に必要な context を集約:

• ゴール / scope / 7 観点責務 mapping (2026-05-26 ユーザー合意)
• 依存タスク現状 (順位 136 land 済、順位 20 / 38 / 95 / 96)
• Phase B 実装計画 (persona / facets / workflow / 検証)
• 適用すべき memory rule / Auto mode pattern
• retirement 条件 (Phase B land で役割終了)

2. PR #177 post-merge-feedback 採用 2 件

順位 136 (working copy staleness 検出 hook) の post-merge-feedback で採用判定された 2 件を docs/todo9.md + docs/todo-summary.md に登録:

• 順位 161 (T2 feat(hooks): 設定駆動型アーキテクチャに移行し配布自動化を実装 #1): Subprocess timeout+kill lifecycle 検証テスト追加 — PR feat(hooks): 順位 136 — working copy staleness 検出 hook 2 段構え (案 A SessionStart + 案 B PreToolUse) #177 Major fix(hooks): stop-quality のパイプデッドロックを修正 #2 fix の回帰テスト
• 順位 162 (T2 fix(hooks): stop-quality のパイプデッドロックを修正 #2): fail-closed error path (Option::None) 個別テスト追加 — PR feat(hooks): 順位 136 — working copy staleness 検出 hook 2 段構え (案 A SessionStart + 案 B PreToolUse) #177 Major feat(hooks): 設定駆動型アーキテクチャに移行し配布自動化を実装 #1 fix の回帰テスト

Diff stats

docs/handoff-rank-8-weekly-review-phase-b.md | 166 ++++++++++++++++++++
docs/todo-summary.md                         |   2 +
docs/todo9.md                                |  87 ++++++++++++
3 files changed, 255 insertions(+), 0 deletions(-)

Test plan

• docs-only 変更につき機能テスト不要
• pnpm push の pre-push-review が APPROVED で完了 (3m12s)
• markdownlint clean
• cli-docs-lint (本 PR 直後の PR 2 で導入予定) の preamble + cross-ref 検査も clean

Next PR

本 PR 後続として cli-docs-lint binary (順位 95 + 96 統合実装) を別 PR で land 予定。

Summary by CodeRabbit

リリースノート

• Documentation
  • 開発チームの週次レビュープロセスに関する運用計画資料を追加しました。
  • 回帰テスト追加のためのタスク管理ドキュメントを更新しました。

[coderabbitai]

Warning

Review limit reached

@aloekun, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 39 minutes and 50 seconds. Learn how PR review limits work.

Your organization has run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: f42417b0-2016-4d84-9446-7985310fb841

📥 Commits

Reviewing files that changed from the base of the PR and between e47fe72 and 9cbc6a1.

📒 Files selected for processing (2)

• docs/handoff-rank-8-weekly-review-phase-b.md
• docs/todo9.md

📝 Walkthrough

概要

Phase B（週次レビューパイプライン）の引き継ぎ用 ephemeral 計画資料 1 件と、PR #177 関連の回帰テスト追加タスク 2 件の詳細計画を 3 つのドキュメントに追加した。Phase B のゴール・MVP 構成・責務分担・工程・運用制約を統合的に定義し、次段階の実行に向けた参照資料を整備した。

変更内容

Phase B 引き継ぎ計画と運用ガイドライン

レイヤー / ファイル	概要
Phase B 計画の基盤とゴール定義 docs/handoff-rank-8-weekly-review-phase-b.md (行 1-52)	資料の位置付け（ephemeral 試験運用、Phase B〜C 完了で役割終了、永続価値の移管先）、MVP 構成（3 facets + aggregate-weekly）、並列統合方針、7 観点の責務分担表（担当 facet・prompt 重点・Phase B 対象外範囲）、依存タスク状態と推奨実装順序を定義した。
Phase B 工程・運用制約・実行パターン docs/handoff-rank-8-weekly-review-phase-b.md (行 53-166)	Phase B 工程表（persona/3 facet/aggregate/workflow 新規作成、takt dry-run、PR 対象範囲）、Phase C/Phase D〜E への委譲線引き、ユーザー判断記録の設計パターン（トリガー・承認フロー・失敗ポリシー）、重要制約（memory rule）、auto mode + preference パターン、推奨初手・想定 PR diff 規模・retire 条件を記載した。

PR #177 回帰テスト追加タスク計画

レイヤー / ファイル	概要
PR #177 タスク一覧と詳細計画 docs/todo-summary.md (行 79-80), docs/todo9.md (行 577-662)	推奨実行順序サマリー表に PR #177 由来の新規タスク 2 件（T2-#1: run_jj_with_timeout の timeout→kill→cleanup 回帰テスト、T2-#2: check_todo_staleness/build_todo_staleness_message の Option::None fail-closed エラーパス unit test）を追加し、各タスクの対象関数・テストケース詳細・失敗時の想定・完了基準を記載した。

評価基準

🎯 2 (Simple) | ⏱️ ~10 分

関連する可能性のある PR

• aloekun/claude-code-hook-test#81: Phase B 引き継ぎ計画は ADR-031 Phase A（週次レビューワークフロー・hook・skill 構造の定義）の直接的な後続資料として、Phase B 実装スコープを具体化している。
• aloekun/claude-code-hook-test#133: 本 PR と PR #133 はいずれも docs/todo-summary.md を修正対象としており、PR #133 はテーブル再構成・アンカー更新、本 PR はタスク行追加として同一ドキュメント表面に対する操作が重複する可能性がある。

🚥 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タイトルは、ランク8のハンドオフドキュメント追加とPR #177のフォローアップタスク登録という、チェンジセット全体の主要な変更を正確に要約している。
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]

Actionable comments posted: 2

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

Inline comments:
In `@docs/handoff-rank-8-weekly-review-phase-b.md`:
- Line 3: 該当行のリンク先が誤って `adr/`
になっているので、docs/handoff-rank-8-weekly-review-phase-b.md 内の「docs-governance.md §
Retirement Workflow」リンクを実際のファイル（docs-governance.md）へ直接差し替え、必要なら該当セクションのアンカー（例:
`#retirement-workflow`）を付けてリンク先が正しく遷移するように修正してください。該当テキストまたは現在の 'adr/'
リンクを検索して、リンク先 URL を "./docs-governance.md"（または正しい相対パス＋セクションアンカー）に更新してください。

In `@docs/todo9.md`:
- Around line 625-626: The docs claim the code was changed to use
behind.map_or(true, |n| n > 0) but the actual implementation in
src/hooks-pre-tool-validate/src/main.rs (referencing functions
check_todo_staleness and build_todo_staleness_message and the behind variable)
still uses behind.is_none() { return None; } and behind.unwrap_or(0); update
either the docs or the code: either (A) amend the documentation snippet to match
the current behavior (explicitly note the unwrap_or/None early-return) or (B)
change the implementation to the intended fail-closed behavior by replacing the
unwrap_or/early-return logic with behind.map_or(true, |n| n > 0) inside
check_todo_staleness/build_todo_staleness_message and add/adjust tests
accordingly; ensure the comment near the security gate (Option-return + jj call)
is updated to reflect whether tests or implementation were changed.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 582e8c7a-94c8-48cc-b7b8-bde45003dba9

📥 Commits

Reviewing files that changed from the base of the PR and between d8ff101 and e47fe72.

📒 Files selected for processing (3)

• docs/handoff-rank-8-weekly-review-phase-b.md
• docs/todo-summary.md
• docs/todo9.md