docs(adr-039): kill-switch 診断メッセージ網羅原則 + 新規 feature self-review checklist 追記

[aloekun]

概要

docs/claude-code-web-tasks.md の採用タスクから 順位 164 / 174（いずれも ADR-039 の docs-only 追記、同一ファイル編集のため同 PR 推奨）を実装。

複数 task を 1 PR に束ねた理由: 両タスクとも編集対象が docs/adr/adr-039-experimental-feature-standard-pattern.md で、タスク表でも「同一 ADR-039 編集」「同 PR 推奨」と明記されているため。

変更内容

順位 164 — § 決定 2 (Kill-switch) に診断メッセージ網羅原則を追記 (PR #179 T3-#1)

• kill-switch 発動時の診断メッセージは判定関数 (is_*_value 等) が受理する全 value variant を反映する原則を codify
• 判定ロジックが複数値 (1 / true / TRUE / True) を受理するのにメッセージが 1 値固定だと spec-impl drift になる点を、PR feat(cli-docs-lint): preamble + cross-ref validator in push-runner quality_gate (ranks 95+96 land) #179 の CLI_DOCS_LINT_DISABLE 実例とともに明文化

順位 174 — 「新規 experimental feature 追加時の self-review checklist」section 新設 (PR #184 T3-#2)

• config schema / feature flag default OFF / docs・config example / test coverage の 4 点整合 を mechanical に確認する手順を追加
• PR feat: 週次レビュー Phase C — /weekly-review skill + SessionStart hook reminder + facet P-1 fix (ADR-031) #184 CR Major M-2 (weekly_review_reminder の enabled = true が opt-in 違反) を OK/NG 実例として inline cite
• 既存 grandfathered case ([session_start.staleness]) は scope 外である旨を補足

着手フローに伴う台帳更新 (step 3-4)

• docs/claude-code-web-tasks.md の 164 / 174 行を削除
• docs/todo-summary.md の 164 / 174 行を削除
• docs/todo9.md の詳細エントリ 2 件を削除

検証

• 純 docs 追記のみ。Rust ビルド / Windows hook 実行は成功条件外 (Web scope 準拠)
• markdownlint (.markdownlint-cli2.jsonc 有効ルール = MD028 / MD040 / MD058) に抵触する要素 (blockquote 内空行 / 言語なし code fence / 表前後の空行欠落) は追加していない

https://claude.ai/code/session_01N4EC63wSspZRjuWAsTLbTB

---

Generated by Claude Code

Summary by CodeRabbit

Documentation

• ADR-039（試験運用標準パターン）ドキュメントを更新し、kill-switch 診断メッセージの標準方針と新規 experimental feature 追加時のセルフレビューチェックリストを整備しました。
• 内部タスク管理ドキュメントを整理し、完了済みのタスク項目を削除しました。

[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: e97515bc-f440-4d5d-a36c-1aed1c70d399

📥 Commits

Reviewing files that changed from the base of the PR and between 32d3269 and 170819b.

📒 Files selected for processing (4)

• docs/adr/adr-039-experimental-feature-standard-pattern.md
• docs/claude-code-web-tasks.md
• docs/todo-summary.md
• docs/todo9.md

💤 Files with no reviewable changes (3)

• docs/claude-code-web-tasks.md
• docs/todo9.md
• docs/todo-summary.md

---

📝 Walkthrough

Walkthrough

PR #189 は ADR-039 の試験運用標準パターンドキュメントを拡張し、kill-switch 診断メッセージが判定ロジックの全値バリアントを反映する原則と、新規 experimental feature 追加時のセルフレビューチェックリスト（構成・フラグデフォルト・設定例・無効状態テスト）を追記します。同時に対応する完了済みタスク 2 件を複数のトラッキングドキュメントから削除します。

Changes

ADR-039 ドキュメント更新と完了タスク削除

Layer / File(s)	Summary
ADR-039 ドキュメント拡張：診断メッセージ原則とチェックリスト docs/adr/adr-039-experimental-feature-standard-pattern.md	Kill-switch 発動時の診断メッセージが判定関数の全 value variant を反映する方針（固定文字列の spec-impl drift を避け、列挙または動的表示で対応）を追記。新規 experimental feature 追加時のセルフレビューチェックリストとして、構成スキーマ、フィーチャーフラグデフォルト OFF、リポジトリ構成例 enabled=false、無効状態スキップテストの4点を機械的に確認する手順を追加。加えて opt-in 契約違反の具体例と既存 grandfathered case の適用外範囲を明記。
完了済み ADR-039 タスクをトラッキングドキュメントから削除 docs/claude-code-web-tasks.md, docs/todo-summary.md, docs/todo9.md	kill-switch 診断メッセージ網羅原則追記と experimental feature チェックリスト拡張に関する 2 件の完了済み TODO エントリ（PR #179 T3-#1、PR #184 T3-#2）を「採用タスク」テーブルおよび推奨実行順序サマリーから削除し、タスク追跡状況を更新。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• aloekun/claude-code-hook-test#139: ADR-039「試験運用機能の標準パターン」ドキュメント導入と相互リンク整備を行い、本 PR で kill-switch 診断受理値原則と experimental feature セルフレビューチェックリストの追加拡張および対応 TODO 削除を実施する関連 PR。
• aloekun/claude-code-hook-test#133: 本 PR で ADR-039 行をトラッキングドキュメントから削除する一方、当該 PR は todo-summary.md と todo.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	プルリクエストのタイトルは、変更内容の主要な2つのポイント（kill-switch 診断メッセージ網羅原則と新規 feature self-review checklist）を正確かつ簡潔に反映しており、実装内容と完全に一致している。
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.}