feat(pipeline): PR 1 即時 skip バンドル (#C-3 + #A-2)

[aloekun]

Summary

docs/pipeline-token-efficiency.md の「PR 1: 即時 skip バンドル」を実装。条件検出 → パイプライン skip の単純 guard を 2 crate に追加し、rate-limit 中・doc-only PR での takt 空打ちを削減する。

• #C-3 cli-pr-monitor: rate-limit 検出時に post-pr-review takt invoke を skip (PR feat(cli-pr-monitor): rate-limit 自動検出 + 再トリガー (Phase 4 / 順位 13) #97 の state.rate_limit を流用)
• #A-2 cli-merge-pipeline: trivial PR (doc-only + 1 commit + <50 行) で post-merge-feedback を skip
• docs pipeline-token-efficiency.md: 完了済 Bundle (Y2 / Z Phase 1 / a) を履歴化 + 残作業を 3 PR 計画に再編

Commits

Commit	内容
b58153f0	docs(pipeline-token-efficiency): 完了済 Bundle 整理 + 残作業を 3 PR 計画に再編
43959d46	feat(cli-pr-monitor): rate-limit 検出時に post-pr-review takt invoke を skip (#C-3)
162f786d	feat(cli-merge-pipeline): trivial PR で post-merge-feedback を skip (#A-2)

注: docs commit と実装 commits は本来別 PR が望ましいが、ユーザー判断 (PR が混ざることを許容、暫定処置) で 1 PR にまとめている。

期待効果

• #C-3: rate-limit 頻発セッションで post-pr-review 空打ち 1-2 分/run を削減 (頻発時 4-8 分/session)
• #A-2: doc-only PR で post-merge-feedback 9 分丸ごと削減

実装の要点

#C-3 (cli-pr-monitor)

• PollResult に rate_limit: Option<RateLimitState> field を追加
• 8 つの PollResult 終了点すべてで state.rate_limit を伝播 (struct literal なので型レベルで漏れなし)
• monitor.rs で has_coderabbit_findings + collect_findings 成功後、rate_limit.is_some() 判定で takt branch を skip + log

#A-2 (cli-merge-pipeline)

• feedback.rs に PrDiffSummary / fetch_pr_diff_summary / parse_pr_diff_summary を追加
• 判定: commit_count == 1 && total_lines_changed < TRIVIAL_PR_LINE_LIMIT (=50) && all_files_are_markdown
• main.rs run_ai_step で feedback::run 前に skip 判定
• 判定失敗時は WARN log で通常 flow に fallback (誤 skip による学習機会損失を最小化)

Test plan

• cli-pr-monitor: 120 既存 test pass (PollResult への field 追加は型レベルで全箇所反映済)
• cli-merge-pipeline: 53 test pass (うち #A-2 用新規 9 件: 境界値 / multi-commit / non-md / uppercase MD / mixed / empty / missing field)
• ワークスペース全体: 600+ test 全 pass
• takt pre-push-review APPROVE (simplicity + security 両方、1 iteration / 4 分)
• merge 後 1-2 PR で誤 skip / 誤発火がないことを観測
• rate-limit 発生時に新しい skip 経路が log を残すことを次回観測

参照

• 設計: docs/pipeline-token-efficiency.md PR 1 セクション
• 前提 PR: feat(cli-pr-monitor): rate-limit 自動検出 + 再トリガー (Phase 4 / 順位 13) #97 (rate-limit 検出機構)、feat(hooks): Bundle Z Phase 1 - Rust comment lint hook + 計画書再編 #99 (Bundle Z Phase 1)

Summary by CodeRabbit

リリースノート

• 新機能

  • トリビアルPR（単一コミット・マークダウンファイルのみの変更）の自動検出機能を追加しました。対象PRはフィードバック処理がスキップされます
  • レート制限が発生した場合に、レビュー処理を自動的にスキップする機能を追加しました

• ドキュメント

  • パイプライン効率化に関する実装計画及び検証方法を更新しました

[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: a006eb30-892c-4da4-a7cd-242a164b9540

📥 Commits

Reviewing files that changed from the base of the PR and between 6cd9316 and 162f786.

📒 Files selected for processing (5)

• docs/pipeline-token-efficiency.md
• src/cli-merge-pipeline/src/feedback.rs
• src/cli-merge-pipeline/src/main.rs
• src/cli-pr-monitor/src/stages/monitor.rs
• src/cli-pr-monitor/src/stages/poll.rs

---

📝 Walkthrough

Walkthrough

PR は 3 つの独立した機能を導入します：ドキュメント化されたパイプライン改善計画、CLI マージパイプラインの trivial PR 検出ロジック、PR モニター内のレート制限状態追跡と自動リトライ機構です。

Changes

Trivial PR Detection in Merge Pipeline

Layer / File(s)	Summary
Data Shape src/cli-merge-pipeline/src/feedback.rs	PrDiffSummary 構造体を追加（コミット数、変更行数合計、Markdown のみ判定）、TRIVIAL_PR_LINE_LIMIT 定数を定義。
Detection Logic src/cli-merge-pipeline/src/feedback.rs	is_trivial() メソッド（コミット数=1、変更行数<50、全ファイルが Markdown）と fetch_pr_diff_summary()、parse_pr_diff_summary() 関数を実装。
Integration & Skip Flow src/cli-merge-pipeline/src/main.rs	run_ai_step が diff summary を取得し、trivial PR 判定時に post-merge-feedback ワークフローをスキップするロジックを追加。
Tests & Validation src/cli-merge-pipeline/src/feedback.rs	Trivial 判定の境界条件、大文字 .MD 拡張子、混在ファイル、JSON パースエラーケースのテストカバレッジを拡張。

Rate-Limit State Tracking & Auto-Retry in PR Monitor

Layer / File(s)	Summary
State Definition src/cli-pr-monitor/src/stages/poll.rs	PollResult に rate_limit: Option<RateLimitState> フィールドを追加し、ポーリングループの全終了パスで rate_limit 状態を伝播。
Retry & Dedup Logic src/cli-pr-monitor/src/stages/poll.rs	handle_rate_limit_retry ヘルパー関数を追加、スリープ予算チェック、GitHub コメント投稿、rate_limit_last_retriggered_at による重複排除、再試行カウンタ管理を実装。
Monitor Skip Behavior src/cli-pr-monitor/src/stages/monitor.rs	start_monitoring が rate_limit 状態を検出時、post-pr-review takt ワークフロー呼び出しをスキップするコンディショナル分岐を追加。
Tests & Coverage src/cli-pr-monitor/src/stages/poll.rs	Rate-limit 状態の永続化、重複排除による再試行スキップ、スリープ予算オーバーランエラー、PR 番号不在時の失敗ケースのテスト。

Pipeline Improvement Documentation

Layer / File(s)	Summary
Document Structure docs/pipeline-token-efficiency.md	完了済バンドル履歴表を追加、方針を「未実施の改善方向のみ」へ転換。
Post-Merge Feedback (#A) docs/pipeline-token-efficiency.md	#A-2（trivial PR skip: 単一コミット、Markdown のみ、±50 行以下）と #A-3（transcript タイムレンジフィルター）の仕様・期待効果・リスク、採用判定表を追加更新。
Pre-Push Review (#B) docs/pipeline-token-efficiency.md	3 層/3 フェーズ構成（決定論レイヤー → 制約付き修正 → 異常検知）へ全面再編、#B-β ではメトリクス diff 比較と ALLOWED_*_PREFIXES 参照同期要件、#B-γ では列挙義務削除と異常検知化を追加。
Post-PR Review (#C) & Response Style (#D) docs/pipeline-token-efficiency.md	#C-2（Convergence gate）と #C-3（rate-limit skip）を追加、#D ではテキスト応答簡素化ガイドラインを追記。
Overall Integration & PR Plan docs/pipeline-token-efficiency.md	PR 1～3 の依存関係別バンドル計画、実行順序、着手前チェックリスト、検証テンプレート、進捗管理表を再構成・更新。

Sequence Diagram(s)

sequenceDiagram
    participant main as run_ai_step
    participant feedback as feedback module
    participant gh as GitHub CLI
    participant parser as JSON Parser
    participant checker as post-merge workflow

    main->>feedback: fetch_pr_diff_summary(pr_number)
    feedback->>gh: gh pr view [pr_number]
    gh-->>feedback: JSON (commit_count, total_lines, files)
    feedback->>parser: parse_pr_diff_summary(JSON)
    parser-->>feedback: PrDiffSummary
    feedback-->>main: PrDiffSummary

    alt is_trivial() == true
        main->>main: log SKIP (trivial PR detected)
        main-->>main: return early
    else is_trivial() == false
        main->>checker: run post-merge-feedback
        checker-->>main: feedback result
    end

Loading

sequenceDiagram
    participant poll_loop as run_poll_loop
    participant state as PollState
    participant gh_api as GitHub API
    participant monitor as start_monitoring
    participant takt as takt workflow

    poll_loop->>state: check rate_limit_last_retriggered_at
    
    alt rate_limit.is_some() && not_already_retriggered
        poll_loop->>gh_api: post `@coderabbitai` review comment
        gh_api-->>poll_loop: comment success
        state->>state: increment rate_limit_retries
        state->>state: update rate_limit_last_retriggered_at
        poll_loop->>poll_loop: continue polling
    else max_retries exceeded or state persist fails
        poll_loop-->>monitor: PollResult { rate_limit: Some(...) }
    end

    monitor->>monitor: check poll_result.rate_limit.is_some()
    alt rate_limit is active
        monitor->>monitor: log rate-limit skip message
        monitor-->>monitor: skip takt invocation
    else rate_limit is None
        monitor->>takt: invoke takt workflow
        takt-->>monitor: takt result
    end

Loading

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~45 minutes

複数の独立した機能実装（trivial PR 検出、レート制限自動リトライ）、ロジック密度の高い状態管理（重複排除、スリープ予算チェック）、3 つの異なるシステムにまたがる変更（ドキュメント、マージパイプライン、モニター）、新規パブリック API の導入が混在しているため。

Possibly related PRs

• PR #77: cli-merge-pipeline の feedback.rs と main.rs を修正し、post-merge-feedback ワークフローの実行フローを変更する関連 PR。本 PR は trivial PR 検出ロジックでそれを拡張します。
• PR #97: cli-pr-monitor のレート制限ハンドリング（rate_limit 状態型、設定、ポーリング/モニターロジック）の観測ログ分析に基づく設計ドキュメント、本 PR が実装する自動リトライ・スキップ機構と直結。
• PR #44: cli-pr-monitor の start_monitoring 内の collect_findings 後の制御フローを変更する PR。本 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 タイトルは変更セットの主要な内容を明確に要約しており、ドキュメント更新と2つの実装機能（#C-3と#A-2）を正確に反映している。
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

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

• Generate code and open pull requests
• Plan features and break down work
• Investigate incidents and troubleshoot customer tickets together
• Automate recurring tasks and respond to alerts with triggers
• Summarize progress and report instantly

Built for teams:

• Shared memory across your entire org—no repeating context
• Per-thread sandboxes to safely plan and execute work
• Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started

---

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

_{Review rate limit: 0/1 reviews remaining, refill in 60 minutes.}

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