PR 봇 스레드를 팀 전원이 볼 수 있게 (스레드 멤버 자동 추가)

[sevineleven]

작업 요약

• 스레드 생성 직후 팀원 전원을 thread-members 로 추가해 모든 PR 스레드를 팀 전원 사이드바에 노출
• 이 변경 이전에 열린 스레드용 backfill-thread-members 수동 백필 잡 추가 (workflow_dispatch)
• 연관 이슈 링크의 Discord 임베드 카드 억제 (카드가 부모 카드 본문을 밀어내지 않게)

Situation

• Discord 봇이 PR마다 여는 스레드는 공개 스레드지만, 부모 카드가 작성자만 @멘션해서 작성자만 자기 PR 스레드가 사이드바에 뜬다. 남의 PR 스레드는 사이드바에 안 떠(링크로만 접근) 팀이 리뷰 논의를 놓친다.

Task

• 팀 전원이 모든 PR 스레드를 사이드바에서 보게 한다. 새로 열리는 스레드뿐 아니라 이미 열려 있는 스레드도 소급 적용한다.

Action

• 스레드 멤버 자동 추가: 스레드 생성 직후 DISCORD_USER_MAP의 팀원 전원을 thread-members 로 추가한다. 멘션과 달리 시끄러운 핑 없이 조용히 목록에만 올린다.
• 기존 스레드 백필: backfill-thread-members 잡을 추가한다. workflow_dispatch로 수동 실행하면 열린 PR들의 메타 주석에서 thread_id를 읽어 팀원을 일괄 추가한다.
• 트리거 가드: workflow_dispatch 추가에 맞춰 notify-discord를 github.event_name == 'pull_request'로 가드해, 수동 실행 시 알림 잡은 돌지 않게 한다.

Result

• 새 PR: 생성 즉시 팀 전원 사이드바에 스레드가 뜬다.
• 기존 열린 PR: Actions 탭 → "Discord PR Bot" → Run workflow 로 한 번 실행하면 일괄 노출된다.
• 한계: 백필은 열린 PR 대상이다. 장기 미활동으로 archived 된 스레드는 멤버 추가만으로 사이드바에 복귀하지 않을 수 있다(열린 PR은 대개 활성이라 무해).
• 서버(be-pr) 봇도 같은 구조라 이후 동일 반영이 필요하다.

---

연관 이슈

• close PR 봇 스레드를 팀 전원이 볼 수 있게 (스레드 멤버 자동 추가) #312

Summary by CodeRabbit

• New Features

  • 수동 실행 가능한 워크플로 지원이 추가되었습니다.
  • PR 알림 스레드에 팀원이 자동으로 추가되도록 개선되었습니다.
  • 기존 공개 PR 스레드에 팀원을 일괄 추가하는 백필 작업이 추가되었습니다.

• Bug Fixes

  • PR 이벤트가 아닐 때 알림 작업이 실행되지 않도록 제한되었습니다.
  • 스레드 정보가 없거나 찾을 수 없는 PR은 안전하게 건너뛰도록 처리되었습니다.

---

Updates

연관 이슈 링크 임베드 억제 (4ffaef7)

부모 카드의 연관 이슈 링크가 GitHub 임베드 카드로 크게 떠서 본문(작업 요약 등)을 아래로 밀어냈다. 이슈 URL 을 <> 로 감싸 임베드만 억제하고 클릭 링크는 유지한다. PR 타이틀 링크는 그대로 둬 PR 카드는 계속 뜬다.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
depromeet	Ready	Preview, Comment	Jul 7, 2026 4:58pm

[github-actions]

Discord 스레드 연동용 메타데이터입니다. discord-pr-bot 워크플로가 자동 생성하며, 수정·삭제하면 PR 과 Discord 알림 연동이 끊깁니다.

[coderabbitai]

Warning

Review limit reached

@sevineleven, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 42 minutes

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 95d38eba-a5f5-497b-8081-b97c77d0d19a

📥 Commits

Reviewing files that changed from the base of the PR and between e43f601 and 3e3648a.

📒 Files selected for processing (1)

• .github/workflows/discord-pr-bot.yml

📝 Walkthrough

Walkthrough

Discord PR 봇 워크플로에 workflow_dispatch 수동 트리거가 추가되고, notify-discord 잡은 pull_request 이벤트에서만 실행되도록 제한되었다. 새 스레드 생성 시 DISCORD_USER_MAP 팀원을 스레드 멤버로 자동 추가하며, 기존 공개 스레드를 백필하는 backfill-thread-members 잡이 신설되었다.

Changes

Discord 스레드 멤버 자동 추가

Layer / File(s)	Summary
트리거 및 실행 조건 변경 .github/workflows/discord-pr-bot.yml	workflow_dispatch 트리거가 추가되고 notify-discord 잡에 pull_request 이벤트 조건이 걸렸다.
신규 스레드 멤버 추가 .github/workflows/discord-pr-bot.yml	스레드 생성 직후 DISCORD_USER_MAP 사용자 전원을 thread-members API로 추가하는 로직이 추가되었다(오류 발생 시에도 흐름 지속).
기존 스레드 백필 잡 .github/workflows/discord-pr-bot.yml	workflow_dispatch 전용 backfill-thread-members 잡이 추가되어 열린 PR의 메타 주석에서 thread_id를 추출하고 팀원 전원을 스레드 멤버로 추가한다. 메타나 thread_id가 없으면 해당 PR은 건너뛴다.

Estimated code review effort: 2 (Simple) | ~10 minutes

Sequence Diagram(s)

<visible_sequence_diagrams>

Sequence Diagram(s)

</visible_sequence_diagrams>

Possibly related PRs

• depromeet/PIKI-Client#308: 동일한 discord-pr-bot.yml 워크플로의 스레드/메시지 처리 및 DISCORD_USER_MAP 사용 로직과 관련됨.
• depromeet/PIKI-Client#310: 동일 파일에서 PR 메타 주석의 thread_id 탐색 로직(find_meta)과 스레드 처리 흐름을 함께 조정함.

Suggested reviewers: ychany, iOdiO89

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	새 스레드에 팀원을 자동 추가하고, 기존 열린 스레드를 수동 백필하는 요구사항을 충족합니다.
Out of Scope Changes check	✅ Passed	워크플로 수동 실행과 알림 조건 변경은 백필 지원을 위한 범위 내 변경입니다.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	제목이 PR 봇 스레드에 팀원을 자동 추가해 전체 팀이 보이게 하는 핵심 변경을 정확히 요약합니다.

---

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]

🧹 Nitpick comments (2)

.github/workflows/discord-pr-bot.yml (2)

568-568: 🎯 Functional Correctness | 🔵 Trivial | ⚡ Quick win

열린 PR이 100개를 넘으면 백필에서 누락됩니다.

per_page=100 은 GitHub pulls API의 상한이라 100개를 초과하는 열린 PR은 조용히 빠집니다. 일회성 수동 백필이라도 완전성을 위해 --paginate 사용을 고려하세요.

♻️ 페이지네이션 적용 예시

-          PRS=$(gh api "repos/${{ github.repository }}/pulls?state=open&per_page=100" --jq '.[].number')
+          PRS=$(gh api --paginate "repos/${{ github.repository }}/pulls?state=open&per_page=100" --jq '.[].number')

🤖 Prompt for 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.

In @.github/workflows/discord-pr-bot.yml at line 568, The PR backfill step in
the GitHub workflow only fetches the first page of open pull requests, so PRs
beyond 100 are silently missed. Update the `gh api` call used to populate `PRS`
in the workflow to paginate through all open pulls (for example by using
`--paginate`), keeping the existing `gh api`/`PRS` flow intact so the backfill
covers every open PR.

---

277-284: 🩺 Stability & Availability | 🔵 Trivial | ⚡ Quick win

스레드 멤버 추가 루프에도 429 재시도가 필요합니다. PUT /channels/{id}/thread-members/{uid} 도 Discord REST rate limit 대상이라, 지금처럼 응답을 무시하면 일부 팀원이 스레드 사이드바에 누락될 수 있습니다. reaction_api처럼 retry_after 기준 재시도(또는 짧은 지연)와 실패 로그를 넣어 주세요.

🤖 Prompt for 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.

In @.github/workflows/discord-pr-bot.yml around lines 277 - 284, The
thread-member add loop currently ignores Discord rate-limit responses, so some
users may fail to be added silently. Update the loop that calls the thread
membership PUT request to handle 429s the same way as reaction_api: inspect the
response, retry after the returned retry_after (or apply a short backoff), and
log failures when retries are exhausted. Keep the change localized to the thread
membership logic that iterates over DISCORD_USER_MAP and calls the
thread-members endpoint.

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

Nitpick comments:
In @.github/workflows/discord-pr-bot.yml:
- Line 568: The PR backfill step in the GitHub workflow only fetches the first
page of open pull requests, so PRs beyond 100 are silently missed. Update the
`gh api` call used to populate `PRS` in the workflow to paginate through all
open pulls (for example by using `--paginate`), keeping the existing `gh
api`/`PRS` flow intact so the backfill covers every open PR.
- Around line 277-284: The thread-member add loop currently ignores Discord
rate-limit responses, so some users may fail to be added silently. Update the
loop that calls the thread membership PUT request to handle 429s the same way as
reaction_api: inspect the response, retry after the returned retry_after (or
apply a short backoff), and log failures when retries are exhausted. Keep the
change localized to the thread membership logic that iterates over
DISCORD_USER_MAP and calls the thread-members endpoint.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 41ee531c-af18-428d-ae57-b93f425eee49

📥 Commits

Reviewing files that changed from the base of the PR and between dad6817 and e43f601.

📒 Files selected for processing (1)

• .github/workflows/discord-pr-bot.yml