docs(adr): ADR-041 Test Isolation Patterns + 順位 78 placeholder 化 (順位 139 land)

[aloekun]

Summary

• 順位 139 (PR test(cli-pr-monitor): Bundle B — early-return guard 単独検証 test + code-review.md checklist 追記 (順位 83 + 84) #168 post-merge-feedback Tier 3 fix(hooks): stop-quality のパイプデッドロックを修正 #2 採用) を land — ADR-041 (Test Isolation Patterns for Multi-Condition Guards) を新規作成し、PR feat(cli-pr-monitor): cli-finding-classifier 統合 + Finding C strict (ADR-038 Phase 5) #120 W-001 + PR test(cli-pr-monitor): Bundle B — early-return guard 単独検証 test + code-review.md checklist 追記 (順位 83 + 84) #168 sentinel pattern 実装の 2 PR 横断 history を project codify。
• ~/.claude/rules/common/code-review.md checklist (順位 84 land 済 global rule) を project-level rationale + 具体実装例 (poll.rs 2 variant) + history で補完する layer として位置づけ。
• 順位 135 codified placeholder policy の dogfood: 当初 ADR-NNN (採番未確定) で entry 登録 → 本 PR で ADR-041 確定取得 → 順位 78 (旧 ADR-041 予約) を ADR-NNN に再 placeholder 化 (= 同一 entry が ADR-038 → 041 → NNN の 3 段番号変更を経た実証)。
• 2 commit 構成:
  • Commit 1: PR test(cli-pr-monitor): Bundle B — early-return guard 単独検証 test + code-review.md checklist 追記 (順位 83 + 84) #168 post-merge-feedback 反映 (順位 139 entry 新規追加、placeholder 状態)
  • Commit 2: ADR-041 land + 順位 78 番号 placeholder 化 + multi-point sync (4 ファイル横断)

Background

PR #120 W-001 で enrich_with_classifier_skips_when_disabled テストが OR-guard (if !config.enabled || state.findings.is_empty() { return; }) の責務混在 = vacuous assertion 問題 (classified_findings が空のまま → 早期 return 由来か他経路由来か判別不能) を抱えていた。PR #168 で sentinel pattern + 直交 precondition setup により構造的解決。本 PR は、2 PR 横断で観測された pattern を project ADR として永続化する。

global checklist (code-review.md 順位 84) は要件を 1 行で示すが、本 ADR は問題の具体例・設計原則・実装例 (poll.rs 2 variant の抜粋コード)・適用範囲・既存資料との関係を記述し、将来の複合 guard test 実装者が独立して参照できる reference layer になる。

Files changed

File	Nature
docs/adr/adr-041-test-isolation-patterns.md	新規 (137 行) — PR #120 W-001 history + sentinel pattern + 2 variant 実装例 + 適用範囲
CLAUDE.md	ADR-041 link を Architecture Decisions list に追加 (+1 行)
docs/todo-summary.md	順位 78 行 placeholder 化 / 順位 139 行 ADR-041 確定 / PR #168 post-merge-feedback explanation block update
docs/todo5.md	順位 78 詳細 entry 内の ADR-041 reference 7 箇所を ADR-NNN に書き戻し + 番号 history コメント追加
docs/todo8.md	順位 139 entry を ADR-041 確定形に変更 + 順位 135 entry の動機/参照を 3 段 (038→041→NNN) history に update

差分規模: +170 / -25 = 145 net 行 (docs-only PR)。

Test plan

• markdownlint: 全ファイル 0 error (PostToolUse hook で連続 verify 済)
• Cross-file reference 整合: 順位 78 / 順位 139 / 順位 135 で番号 history 一貫
• fabricated URL なし (grep github.com/aloekun docs/adr/ で no match、advisor 指摘の 2 箇所を plain inline code に修正済)
• ADR の cross-link target は全て実在 path (~/.claude/rules/common/code-review.md は plain code text、src/cli-pr-monitor/src/stages/poll.rs は実在 file)
• Multi-point synchronization: ADR-041 番号確定に伴う 4 ファイル横断の placeholder 整理を同一 commit で実施 (code-review.md § Multi-point synchronization の実例 dogfood)
• pre-push-review: APPROVE (1 iteration、docs-only PR として scope 適用外項目を正しく除外)

Notes

• 順位 139 entry (todo8.md) は本 PR land 後の post-merge-feedback サイクルで削除予定 (memory: feedback_todo_no_history、マージ → 即削除)。
• 順位 78 (Rust timestamp arithmetic safety) は今後 land 時 PR で空き番号を取得する運用に統一 (順位 135 codified policy)。
• ADR-041 ステータスは「試験運用 (2026-05-22)」で開始、後続 PR で pattern 改良があれば supersede 余地を残す (直近 ADR-029〜ADR-040 の慣習に整合)。

Summary by CodeRabbit

• Documentation
  • 「ADR-041: Test Isolation Patterns for Multi-Condition Guards」を新規追加し、sentinelパターンや直交な前提設定のテスト設計例、適用範囲、Pros/Consを明記しました
  • ADR一覧にADR-041を追記して参照を確定化しました
  • TODO/優先度サマリーや運用メモを更新し、ADR番号のプレースホルダ運用と再割当に関する注記を反映しました

[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: 0f7fed9f-f688-4408-ae93-8d85d8306d25

📥 Commits

Reviewing files that changed from the base of the PR and between 67a514e and 35b6109.

📒 Files selected for processing (1)

• docs/todo8.md

---

📝 Walkthrough

Walkthrough

複合ガード関数のテスト分離パターンを ADR-041 として新規定義し、CLAUDE.md に登録。todo ドキュメント群の ADR 参照を統一し、ADR 番号の hardcode 回避と land 時の採番確定運用方針を明文化しました。

Changes

ADR-041 定義と登録

Layer / File(s)	Summary
ADR-041 ドキュメント作成と CLAUDE.md 登録 docs/adr/adr-041-test-isolation-patterns.md, CLAUDE.md	enrich_with_classifier を例に、sentinel pre-population と OR-guard の直交 precondition assertion を ADR として定義。Pros/Cons・適用範囲・関連資料を記載し、CLAUDE.md のアーキテクチャ決定リストに ADR-041 を追加登録。

ADR 番号付与プロセスと参照整備

Layer / File(s)	Summary
todo-summary.md における ADR-041 反映と placeholder 運用 docs/todo-summary.md	順位 78 を ADR-NNN プレースホルダへ置換（land 時に確定する運用）。順位 139 に PR #168 の採用として ADR-041 を追記。post-merge-feedback として確定手順と artifact relocation の未文書化ギャップを追記。
todo5.md と todo8.md における ADR-NNN placeholder 方針確立 docs/todo5.md, docs/todo8.md	ADR-041 参照を ADR-NNN に統一し、ADR-NNN (採番未確定、land 時に確定) プレースホルダ方針を明文化。todo8.md に本 PR による ADR-041 確定取得の記録を追加。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• aloekun/claude-code-hook-test#158: todo ドキュメント群での ADR-NNN プレースホルダ方針および参照更新と関連。
• aloekun/claude-code-hook-test#133: docs/todo-summary.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	プルリクエストのタイトルはADR-041テスト分離パターンの追加と順位78のプレースホルダ化という、変更内容の主要な側面を正確に反映している。
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: 1

🤖 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/todo8.md`:
- Around line 358-369: The checklist heading "#### 作業計画 (本 PR で完了)" is
inconsistent with the unchecked item "- [ ] PR description で
`docs/adr/adr-041-test-isolation-patterns.md` への link ..." — either mark that
checkbox as completed ("- [x]") if the PR description already contains the
link/summary, or change the heading to something like "作業計画 (本 PR での進捗)" to
reflect mixed completion; update the line containing the unchecked item or the
heading accordingly in docs/todo8.md so the checklist state matches the header.

🪄 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: 12320979-1f14-4726-8509-ef4990ababe3

📥 Commits

Reviewing files that changed from the base of the PR and between 08a9ce3 and 67a514e.

📒 Files selected for processing (5)

• CLAUDE.md
• docs/adr/adr-041-test-isolation-patterns.md
• docs/todo-summary.md
• docs/todo5.md
• docs/todo8.md

[aloekun]

@coderabbitai review

[coderabbitai]

✅ Actions performed

Review triggered.

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.