Skip to content

fix(src): PR上限(max_committed_prs_per_run等)をグローバルに適用#56

Merged
HappyOnigiri merged 2 commits intomainfrom
fix/global-pr-limits
Mar 11, 2026
Merged

fix(src): PR上限(max_committed_prs_per_run等)をグローバルに適用#56
HappyOnigiri merged 2 commits intomainfrom
fix/global-pr-limits

Conversation

@HappyOnigiri
Copy link
Owner

@HappyOnigiri HappyOnigiri commented Mar 11, 2026

各リポジトリごとにリセットされていたPR処理上限を、1回の実行全体で共有するように修正しました。これにより、複数のリポジトリを跨ぐ際にも上限が正しく機能します。

Summary by CodeRabbit

リリースノート

  • 新機能

    • 複数リポジトリにわたるPRの追跡機能を強化しました。
    • 複数の実行にわたるPR状態の一貫性が向上しました。
  • 改善

    • PR管理の内部構造を最適化し、クロスリポジトリでの正確なPRアカウンティングを実現しました。

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 11, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: cb343d01-d752-42ad-938e-6bc739b245ec

📥 Commits

Reviewing files that changed from the base of the PR and between bd06a51 and d48bc04.

📒 Files selected for processing (2)
  • src/auto_fixer.py
  • tests/test_auto_fixer.py

📝 Walkthrough

Walkthrough

src/auto_fixer.py のグローバル PR トラッキング機能を拡張し、複数リポジトリ間で PR の状態を追跡するために、process_repo シグネチャに 3 つの新しいオプションパラメータを追加。PR 追跡を単一の整数セットから (repo, pr_number) タプルセットに変更し、テスト呼び出しを更新。

Changes

Cohort / File(s) Summary
Auto Fixer PR Tracking
src/auto_fixer.py
グローバル PR トラッキング用の 3 つの新しいパラメータ (global_modified_prs, global_committed_prs, global_claude_prs) を process_repo に追加。PR トラッキング対象を単一リポジトリの整数から複数リポジトリ対応の (repo, pr_number) タプルに変更。main() で初期化し全 process_repo 呼び出しに伝搬。
Test Updates
tests/test_auto_fixer.py
process_repo 呼び出しサイトを更新し、新しいグローバル PR トラッキングパラメータを反映。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (2 warnings)

Check name Status Explanation Resolution
Description check ⚠️ Warning プルリクエスト説明はテンプレートの必須セクション(Summary、Related issues、Changes、Testing、Checklist)の構造を満たしていません。 テンプレートに従い、Summary、Related issues、Changes、Testing、Checklistの各セクションを追加してください。
Docstring Coverage ⚠️ Warning Docstring coverage is 50.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (1 passed)
Check name Status Explanation
Title check ✅ Passed タイトルは変更内容の主要な部分(PR上限をグローバルに適用)を正確に反映しており、簡潔で具体的です。

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/global-pr-limits

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

@HappyOnigiri HappyOnigiri added the refix:running Refix is currently processing review fixes. label Mar 11, 2026
グローバルPR上限追跡(global_*_prs)とauto_resume_run_stateの
両機能を共存させる形でコンフリクトを解消する
@HappyOnigiri
Copy link
Owner Author

@coderabbitai resume

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 11, 2026

✅ Actions performed

Reviews resumed.

@HappyOnigiri HappyOnigiri merged commit 1fdd959 into main Mar 11, 2026
2 checks passed
@HappyOnigiri HappyOnigiri deleted the fix/global-pr-limits branch March 11, 2026 21:52
@coderabbitai coderabbitai bot mentioned this pull request Mar 11, 2026
4 tasks
@HappyOnigiri HappyOnigiri removed the refix:running Refix is currently processing review fixes. label Mar 11, 2026
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