feat: Bundle CR-RL — CR rate-limit detection を新フォーマット対応に拡張 (順位 167-169) + 174

[aloekun]

Summary

Bundle CR-RL = RATE_LIMIT_MARKER 旧フォーマット固定値が CR の新フォーマット (<!-- rate limited by coderabbit.ai --> HTML マーカー + ## Review limit reached heading + More reviews will be available in N minutes wait time) を検出できず、cli-pr-monitor の RateLimitOutcome::Parked 経路 (ADR-018 設計) が silent regression で完全無効化されていた critical bug を 3 層で構造的に修正。

実体観測根拠:

• PR feat(takt): 週次レビューパイプライン Phase B — workflow + 4 facets 追加 (ADR-031) #182 セッション: 30+ 分間 rate-limit 未検出で polling 浪費 (本セッション内 detection bug の発見契機)
• PR feat: 週次レビュー Phase C — /weekly-review skill + SessionStart hook reminder + facet P-1 fix (ADR-031) #184 セッション: 36 分 rate-limit + 12 分待機 + retry trigger が pragmatic に機能した実例

加えて PR #184 post-merge-feedback で採用された T3-2 (ADR-039 checklist 拡張) を todo 化する commit を同 PR に bundle (ユーザー判断 2026-05-29「既存変更同 PR 許容、commit のみ分離」)。

含まれる commits (4 件)

1. docs(todo): 順位 174 (PR #184 T3-2 採用)

PR #184 post-merge-feedback T3-#2 採用: ADR-039 experimental feature lifecycle checklist 拡張 task を todo 化。新規 feature 追加時の config schema ↔ feature flag default OFF ↔ docs / config example ↔ test coverage の 4 点整合 mechanical checklist。PR #184 M-2 (weekly_review_reminder の enabled = true 違反) は checklist があれば self-review で捕捉可能だった。

T3-1 (Feedback adoption pattern ADR) は不採用 (analyzer rubric と user 判断の完全一致を formalize するのは構造的に困難、memory feedback_no_unenforced_rules 延長線上の原則)。

2. feat(rate-limit): 順位 167 (T1-#1) 機械強制層

src/check-ci-coderabbit/src/main.rs の rate-limit detection を CR 新フォーマット対応に拡張:

• RATE_LIMIT_MARKER: &str (旧 marker 単一固定) → RATE_LIMIT_MARKERS: &[&str] multi-variant 配列化
  • 旧 marker: "Rate limit exceeded" (backward compat 維持)
  • 新 marker: "rate limited by coderabbit.ai" (HTML コメントマーカー、PR feat(takt): 週次レビューパイプライン Phase B — workflow + 4 facets 追加 (ADR-031) #182/feat: 週次レビュー Phase C — /weekly-review skill + SessionStart hook reminder + facet P-1 fix (ADR-031) #184 で実観測)
• is_rate_limit_comment() を multi-variant check に refactor
• extract_wait_time() を 2 helper 関数 (extract_old_format_wait_time / extract_new_format_wait_time) に分離して新フォーマット regex を追加
  • 旧: Please wait N minutes? and M seconds? (4 variants、3 既存 fixtures が backward compat 保証)
  • 新: More reviews will be available in N minutes? and M seconds? (2 variants、PR feat(takt): 週次レビューパイプライン Phase B — workflow + 4 facets 追加 (ADR-031) #182/feat: 週次レビュー Phase C — /weekly-review skill + SessionStart hook reminder + facet P-1 fix (ADR-031) #184 で実観測)
• parse_rate_limit docstring を multi-variant 前提に update

注: ADR-039 experimental feature pattern は本タスクには不適用 (rate-limit detection は ADR-018 で既に決定済の standard feature であり experimental ではない)。

3. test(rate-limit): 順位 168 (T2-#1) test 層

#[cfg(test)] mod に 3 新規 fixture を追加 (旧 6 fixture は backward compat 維持):

• rate_limit_detected_from_new_format_with_html_marker_and_full_wait_time — HTML マーカー + 標準 wait time
• rate_limit_detected_from_new_format_with_minutes_only — 短縮形 (seconds 省略)
• rate_limit_picks_latest_when_mixed_old_and_new_formats — 旧 + 新混在で時系列最新が選ばれる

memory feedback_test_dry_antipattern 適用 (各 fixture 独立 setup、共通 helper 化しない)。

4. docs(adr): 順位 169 (T3-#1) ADR 永続 reference 層

• ADR-034 に新 section「既知 CR rate-limit format 一覧」+「検出 logic 更新手順 (6 step)」追加
  • 既知 format 表 (発見時期 / body marker / wait time regex の 3 列、format 追加で append しやすい構造)
  • 検出 logic 更新の標準手順を明文化 (= 将来 CR が format 変更した場合の同型 silent regression を構造的予防)
  • HTML マーカー優先の理由を明示 (UI 文言より stable な可能性)
• ADR-018 lines 185-186 の rate-limit 表 description を「順位 167 で multi-variant 対応済、詳細は ADR-034 § 既知 CR rate-limit format 一覧 参照」に書き換え

検証結果

• cargo build (release): clean
• cargo test -p check-ci-coderabbit: 82 tests pass (79 既存 + 3 新規 fixtures)
• cargo clippy -p check-ci-coderabbit -- -D warnings: clean
• pre-push-review (2m 36s): simplicity-review APPROVED + security-review APPROVED (No blocking anomalies detected)

メタ評価

本 PR は PR #182 セッションで dogfood 実体観測した structural finding (CR rate-limit marker drift) を 3 commit 分離で構造修正した実例。Phase B (PR #182) で確立した weekly-review pipeline が「rule では検出できない rate-limit detection の format drift」を実体観測経由で拾い、Bundle CR-RL として系統的に修正できた = self-reinforcing detection loop の中期成果。

format 更新手順の codify (ADR-034) により、本 PR land 後の 次回 CR format 変更時 は同型の 30+ 分 polling 浪費を回避可能になる構造化が確立。

Test plan

• cargo test -p check-ci-coderabbit で 82 tests pass (旧 6 + 新 3 fixtures + その他)
• cargo clippy clean
• markdownlint clean (ADR-034 + ADR-018)
• pre-push-review APPROVED
• CodeRabbit が本 PR 自身を新 format detection 対象として review (= self-reference test、本 PR 内で CR が rate-limit に遭遇した場合 fix 後の挙動を実観測)
• CI green 確認後 merge 判断
• (post-merge) 次の rate-limit 観測時に新 detection 経路で RateLimitOutcome::Parked が機能することを実観測

🤖 Generated with Claude Code

Summary by CodeRabbit

リリースノート

• New Features

  • CodeRabbit のレート制限検出が複数フォーマットに対応しました。従来の検出方式に加え、新しいフォーマットも自動的に認識し、待機時間をより正確に抽出できるようになりました。

• Documentation

  • レート制限検出とモニタリングに関する設計ドキュメントを更新しました。実装状態の詳細を追加し、フォーマット変更への対応手順を明記しました。

[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: 286cb19f-4f24-4a8c-af88-0eff74f4309e

📥 Commits

Reviewing files that changed from the base of the PR and between 3f00f87 and 4743c5a.

📒 Files selected for processing (5)

• docs/adr/adr-018-pr-monitor-takt-migration.md
• docs/adr/adr-034-coderabbit-auto-monitoring.md
• docs/todo-summary.md
• docs/todo9.md
• src/check-ci-coderabbit/src/main.rs

---

📝 Walkthrough

Walkthrough

PR は CodeRabbit の rate-limit コメント検出を複数フォーマット対応へ拡張し、待機時間抽出を旧・新フォーマットの専用関数に分割した改善と、新規 experimental feature 追加時の 4 点整合チェックリストを追加した。

Changes

Rate-limit detection multi-format support

Layer / File(s)	Summary
Rate-limit detection strategy documentation docs/adr/adr-034-coderabbit-auto-monitoring.md, docs/adr/adr-018-pr-monitor-takt-migration.md	ADR-034 で walkthrough comment の body/updated_at を直接 polling する方式へ移行。多段階の RATE_LIMIT_MARKERS と regex により旧・新フォーマットを判定し、format drift 時の marker・regex・fixture・ADR 表更新の標準手順を明記。ADR-018 の transient failure table を RATE_LIMIT_MARKERS + reset_time 判定に更新。
Multi-format rate-limit marker and wait-time extraction src/check-ci-coderabbit/src/main.rs	RATE_LIMIT_MARKERS を複数マーカー対応へ変更し、待機時間抽出を extract_old_format_wait_time（Please wait ...）と extract_new_format_wait_time（More reviews will be available in ...）に分割。共通関数 extract_wait_time が両フォーマットをフォールバック実装。
Multi-format detection and mixed-format tests src/check-ci-coderabbit/src/main.rs	新フォーマット（HTML マーカー付き）検出テストと、旧・新混在時に最新コメント採用により wait 値を決定することを検証。

Experimental feature lifecycle checklist

Layer / File(s)	Summary
ADR-039 experimental feature checklist expansion docs/todo9.md	新規 feature 追加時に schema・feature flag・docs example・disabled test の 4 点整合を self-review で機械的に確認する ADR-039 チェックリスト定義。PR #184 の実例、grandfathered case、作業計画、完了基準を記載。
Checklist reference in execution order summary docs/todo-summary.md	推奨実行順序サマリー表に ADR-039 checklist 拡張タスク（順位 174）を参照行として追加。

Possibly related PRs

• aloekun/claude-code-hook-test#102: cli-pr-monitor の rate-limit skip logic が rate-limit comment 認識に依存し、本 PR で check-ci-coderabbit の rate-limit marker 検出・待機時間抽出が改善されることで連携。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

🚥 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 タイトルは、CR rate-limit detection の新フォーマット対応という主要な変更を明確に説明しており、ドキュメント更新や TODO 追加も含む複数の関連タスクを参照している。
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
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.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

---

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