⚡ Bolt: 에이전트 멘션 스윕 PR 조회 N+1 API 병목 해결

[seonghobae]

Outcome

Parallelize organization-wide recent-PR discovery without allowing a bounded mention sweep to wait for irrelevant repository work after reaching its dispatch limit.

Exact identity

• protected-main base snapshot: 6eb06cdd08c79a06f7b390069d4ffa49e2eb7dba;
• current source head: 93eae45d78fc56b319a7c9f9ccf600180577c196;
• source branch: bolt-fix-agent-mention-n-plus-one-11378016381235193798;
• current hosted workflows: 9 queued; queued evidence is not represented as passing;
• formal reviews: 1; unresolved review threads: 0.

Root cause and repair

The original bounded ThreadPoolExecutor implementation removed the N+1 API bottleneck and serialized repository-local failures, but the executor context manager still used shutdown(wait=True). When sweep() reached max_dispatches, returning from the consumer did not explicitly close its candidate iterator and could wait for every running or queued repository fetch.

The repaired contract:

• preserves the serial fast path for zero or one repository;
• bounds concurrent repository reads to at most ten workers;
• preserves deterministic repository-order emission and caller-thread error accounting;
• explicitly closes candidate iteration at the dispatch limit;
• shares a cancellation event with workers and checks it between page requests;
• cancels queued futures with shutdown(wait=False, cancel_futures=True);
• lets normal complete collection perform a full waited shutdown;
• documents cutoff and invalid-number behavior without exposing raw API failures.

Test-first evidence

• fail-first standalone reproduction: dispatch returned 1 while the candidate iterator remained open;
• repaired focused suite: 18 passed;
• owned module: 195/195 statements and 78/78 branches;
• public docstrings: 100%;
• Python compilation: passed.

The hosted full-repository, security, dependency, and supply-chain runs on the current head remain authoritative for integration.

Duplicate lineage

#938 independently reintroduced the same N+1 optimization from the same protected base, but lacks this cancellation contract and currently fails its quality gate. Preserve #938 as duplicate evidence and converge on this older, fully tested branch once its active Jules writer is no longer targeting it.

Acceptance

Keep merge fail-closed until all current-head workflows complete successfully, current semantic review has no valid unresolved finding, qualifying independent human approval exists, live protected-main ancestry is revalidated, and protected-main operational acceptance proves bounded early termination.

[google-labs-jules]

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!

New to Jules? Learn more at jules.google/docs.

---

For security, I will only act on instructions from the user who triggered this task.

[coderabbitai]

📝 Walkthrough

Walkthrough

에이전트 멘션 스윕이 여러 저장소의 최근 PR을 병렬 조회합니다. 단일 저장소는 순차 조회합니다. 조회 결과는 수집 후 반환하며, 페이지네이션과 저장소별 오류 격리는 유지합니다.

Changes

에이전트 멘션 스윕

Layer / File(s)	Summary
최근 PR 조회 병렬화 scripts/ci/agent_mention_sweep.py, .jules/bolt.md	여러 저장소를 최대 10개 워커로 조회합니다. 단일 저장소는 순차 처리합니다. PR 결과를 저장한 뒤 수집 순서대로 반환합니다. 페이지네이션과 저장소별 오류 격리는 유지합니다. 관련 오류 메시지와 호출 형식을 정리했습니다.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Sequence Diagram(s)

sequenceDiagram
  participant list_recent_pull_requests
  participant ThreadPoolExecutor
  participant GitHub API
  list_recent_pull_requests->>ThreadPoolExecutor: 저장소별 조회 작업 제출
  ThreadPoolExecutor->>GitHub API: 최근 PR 및 페이지 조회
  GitHub API-->>ThreadPoolExecutor: 저장소별 PR 결과
  ThreadPoolExecutor-->>list_recent_pull_requests: 수집된 결과 반환
  list_recent_pull_requests-->>list_recent_pull_requests: 수집 순서대로 PR 생성

Loading

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	Docstring coverage is 85.71% 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.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	제목은 에이전트 멘션 스윕의 PR 조회 N+1 API 병목을 병렬화로 해결하는 주요 변경 사항을 명확하게 설명합니다.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch bolt-fix-agent-mention-n-plus-one-11378016381235193798

---

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

[coderabbitai]

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@scripts/ci/agent_mention_sweep.py`:
- Line 158: fetch_repo_pulls 함수에 docstring을 추가해 다중 저장소 처리와 저장소별 오류 격리 동작을 간단히
설명하십시오.
- Around line 211-214: Update the worker flow around fetch_repo_pulls so workers
return pull-request results and exceptions instead of invoking on_error
directly; have the caller thread process those results in repository order and
invoke on_error serially. Update fetch_repo_pulls’s docstring to document this
behavior, and add tests covering concurrent errors and exact failure-counter
values.

🪄 Autofix

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: ff2c489a-6b53-443c-ad23-f99b89c68e63

📥 Commits

Reviewing files that changed from the base of the PR and between 6eb06cd and 0d6a30e.

📒 Files selected for processing (2)

• .jules/bolt.md
• scripts/ci/agent_mention_sweep.py

[seonghobae]

@opencode-agent review

Please review exact current head 93eae45d78fc56b319a7c9f9ccf600180577c196. Focus on prompt cancellation after max_dispatches, queued-future cancellation, running-worker page-boundary stopping, deterministic repository-order evidence, and preservation of caller-thread failure accounting. Do not transfer predecessor-head conclusions or treat local/automated evidence as qualifying human approval.

[seonghobae]

Superseded by #938 at exact head b6f2dba96a014e6ac6fddd776fa9a2db85e824f0 from the same protected-main tip 6eb06cdd08c79a06f7b390069d4ffa49e2eb7dba. The canonical branch now preserves deterministic repository-order emission, a zero/one-repository serial fast path, caller-thread error isolation, prompt cancellation between page requests, pending-future cancellation, explicit candidate-source close, and full current-suite/100% owned-coverage proof. All 14 hosted #938 exact-head runs, including Strix, OpenCode, Noema, CodeQL, Semgrep, OSV, SBOM, secret, Python-security, and quality gates, are terminal-success with zero unresolved review threads. Closing this duplicate preserves the requirements and history established here; merge authority for #938 remains fail-closed on two qualifying independent approvals and last-push approval.

[google-labs-jules]

Superseded by #938 at exact head b6f2dba96a014e6ac6fddd776fa9a2db85e824f0 from the same protected-main tip 6eb06cdd08c79a06f7b390069d4ffa49e2eb7dba. The canonical branch now preserves deterministic repository-order emission, a zero/one-repository serial fast path, caller-thread error isolation, prompt cancellation between page requests, pending-future cancellation, explicit candidate-source close, and full current-suite/100% owned-coverage proof. All 14 hosted #938 exact-head runs, including Strix, OpenCode, Noema, CodeQL, Semgrep, OSV, SBOM, secret, Python-security, and quality gates, are terminal-success with zero unresolved review threads. Closing this duplicate preserves the requirements and history established here; merge authority for #938 remains fail-closed on two qualifying independent approvals and last-push approval.

Understood. Acknowledging that this work is now obsolete as it has been superseded by #938, and stopping work on this task.