Skip to content

docs: ADR-031 § Adoption Criteria threshold + Bundle CR-RL stale cleanup - #193

Merged
aloekun merged 2 commits into
masterfrom
adr-031-thresholds-cr-rl-cleanup
Jun 3, 2026
Merged

docs: ADR-031 § Adoption Criteria threshold + Bundle CR-RL stale cleanup#193
aloekun merged 2 commits into
masterfrom
adr-031-thresholds-cr-rl-cleanup

Conversation

@aloekun

@aloekun aloekun commented Jun 3, 2026

Copy link
Copy Markdown
Owner

Summary

Commit 構成

commit 内容
1 (93de4f0) docs(adr-031): § Adoption Criteria threshold 追加 + ADR-039 cross-ref (PR #192 T3-#5)
2 (6dfb730) docs(todo): Bundle CR-RL stale entry cleanup (順位 167/168/169 — PR #185 で land 済)

post-merge-feedback Tier 3 #6 (ADR-039 audit) 部分吸収

本 PR の ADR-031 amendment では ADR-039 § Bounded lifetime 3 値判定 (採用/却下/継続) の 具体化例 として § 採用判定の閾値 を位置付け、双方向 link を確立した。これにより post-merge-feedback #192 Tier 3 #6 (ADR-039 audit) の主要価値の一部を本 amendment で吸収する。

Test plan

  • markdownlint clean (両 commit で確認)
  • pre-push simplicity-review APPROVE (3m 7s)
  • pre-push security check ✅ (docs-only per ADR-035)
  • jj diff vs master 確認 (+19 -130、net -111)
  • post-PR CodeRabbit review 通過確認
  • CI 緑確認

Summary by CodeRabbit

  • Documentation
    • アーキテクチャ決定記録(ADR)を更新し、採用判定の基準と条件を明確化しました。
    • 内部タスク管理ドキュメントを整理し、完了済みのタスクを削除しました。

aloekun added 2 commits June 3, 2026 12:45
#192 T3-#5)

PR #192 post-merge-feedback Tier 3 #5 採用。Phase E land 時 § 採用判定の根拠 は
観測値の記録のみで「閾値」が暗黙だった。5 閾値 (採用率 ≥ 40% / wall-clock ≤ 10 分 /
FP ≤ 5% / context 圧迫なし / systemic 検出力) を ADR-031 inline で永続記録、
将来 trial ADR の採用判定で参照可能化。

ADR-039 § 関連 にも back-link を追加し双方向 link 形成、§ Bounded lifetime の
3 値判定 (採用 / 却下 / 継続) の具体化例として参照可能。Tier 3 #6 (ADR-039 audit)
の価値も部分吸収。
…で land 済)

PR #185 (commit 7f8b613) で Bundle CR-RL の実装 3 件は全て land 済:
- 順位 167: RATE_LIMIT_MARKERS multi-variant 配列化 (main.rs:261)
- 順位 168: 新 format fixture 3 variant (full / minutes-only / mixed)
- 順位 169: ADR-018 lines 185-186 multi-variant 表記 + ADR-034 § 既知 format 一覧 + § 検出 logic 更新手順

todo9.md / todo-summary.md の stale entry を削除して in-progress を反映。
memory feedback_verify_task_not_already_done の本来用途 (= 既 land 済タスクを
stale entry 削除に再目的化) を実適用。
@coderabbitai

coderabbitai Bot commented Jun 3, 2026

Copy link
Copy Markdown

Review Change Stack

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: dca9b825-ee66-4094-b7f2-3bf2ed3dd98b

📥 Commits

Reviewing files that changed from the base of the PR and between 8c71029 and 6dfb730.

📒 Files selected for processing (4)
  • docs/adr/adr-031-weekly-review-pipeline.md
  • docs/adr/adr-039-experimental-feature-standard-pattern.md
  • docs/todo-summary.md
  • docs/todo9.md
💤 Files with no reviewable changes (2)
  • docs/todo9.md
  • docs/todo-summary.md

📝 Walkthrough

Walkthrough

ADR-031にweekly review pipeline採用判定の5つの閾値を追加し、採用判定条件と継続/却下扱いを明文化。ADR-039でADR-031を本採用昇格と記載し、採用判定の5閾値をbounded lifetimeの具体化例として参照可能に。CR rate-limit関連の完了済みTODOエントリを削除。

Changes

ADR-031 採用判定基準定義とドキュメント整合化

Layer / File(s) Summary
ADR採用判定基準と相互参照の追加
docs/adr/adr-031-weekly-review-pipeline.md, docs/adr/adr-039-experimental-feature-standard-pattern.md
ADR-031に採用率・wall-clock・false positive・context window圧迫・systemic finding検出力の5閾値を定義し、全達成時の採用判定、未達時の継続(1回まで)/却下対応、および暫定値の将来改訂可能性を明記。ADR-039はADR-031の状態を「試験運用」から「承認済み(本採用昇格:2026-06-01)」に更新し、5閾値をbounded lifetime判定の具体化例として参照可能にした。
完了済みタスク項目の削除
docs/todo-summary.md, docs/todo9.md
推奨実行順序サマリーから順位167〜169の3エントリ(RATE_LIMIT_MARKER形式対応・統合テスト・ADR同期戦略)を削除、順位170以降を繰り上げ。todo9.mdからPR #182の rate-limit検出更新関連の126行のTODOセクションを削除。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

  • aloekun/claude-code-hook-test#81: ADR-031の初版提案PR。本PRはADR-031に採用判定基準の5閾値を追加し、同一ドキュメントを拡張。

  • aloekun/claude-code-hook-test#182: Weekly review pipelineのPhase B実装PR。本PRで削除されたTODO項目はPR #182のrate-limit検出実装に対応する完了タスク

  • aloekun/claude-code-hook-test#192: ADR-031とADR-039の同じファイルセットを修正し、ADR-031採用促進のドキュメント部分の関連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タイトルは、ADR-031への採用基準閾値の追加と、CR-RLの古い項目の削除という2つの主要な変更を正確に反映している。
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

Comment @coderabbitai help to get the list of available commands and usage tips.

@aloekun
aloekun merged commit f59a2f7 into master Jun 3, 2026
1 check passed
@aloekun
aloekun deleted the adr-031-thresholds-cr-rl-cleanup branch June 3, 2026 05:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant