Skip to content

PR 봇 스레드 팀 노출·이슈 임베드 억제·머지 스레드 글쓰기 허용 + 기존 스레드 백필 - #703

Merged
sevineleven merged 2 commits into
devfrom
chore/701-discord-thread-ux
Jul 7, 2026
Merged

PR 봇 스레드 팀 노출·이슈 임베드 억제·머지 스레드 글쓰기 허용 + 기존 스레드 백필#703
sevineleven merged 2 commits into
devfrom
chore/701-discord-thread-ux

Conversation

@sevineleven

Copy link
Copy Markdown
Collaborator

Situation

  • 클라(fe-pr) 봇에 반영한 스레드 UX 개선을 서버(be-pr) 봇에도 맞춰 두 봇 동작을 일치시킨다.
  • 현재 서버 봇의 불편 세 가지:
    • 공개 스레드가 작성자만 사이드바에 떠서 팀이 남의 PR 스레드를 못 본다.
    • 연관 이슈 링크가 큰 임베드 카드로 부모 카드 본문(작업 요약 등)을 아래로 밀어낸다.
    • 머지 시 스레드를 locked 로 잠가 이후 글을 못 쓴다.
  • 더해, 과거 find_meta 버그(PR 봇 find_meta 가 리뷰 코멘트를 metadata 로 오인하던 버그 수정 #674 이전, 파일명을 언급한 리뷰 코멘트를 잡아 message_id 가 비던 문제)로 close 정리가 스킵된 PR 들의 스레드가 안 닫힌 채 남아 있다.

Task

  • 위 세 UX 를 클라와 동일하게 맞춘다.
  • 이미 쌓인 기존 스레드를 현재 정책으로 한 번에 통일한다.

Action

상시 동작 (봇)

  • 팀 노출: 스레드 생성 직후 팀원 전원을 스레드 멤버로 추가한다. 공개 스레드는 멤버만 사이드바에 떠서, 멘션 없이 조용히 목록에만 올려 팀 전원이 모든 PR 스레드를 보게 한다.
  • 이슈 임베드 억제: 연관 이슈 링크 URL 을 <> 로 감싸 큰 임베드 카드를 없앤다. 클릭은 유지되고 PR 타이틀 카드는 그대로 뜬다.
  • 머지 스레드 글쓰기 허용: 닫을 때 archived 만 두고 locked 를 뺀다. 목록에선 접혀 닫힌 상태지만, 글을 쓰면 다시 열려 논의가 가능하다.

일회성 백필 (수동 실행)

backfill-threads 잡을 workflow_dispatch 로 추가한다. Actions 에서 Run workflow 로 한 번 돌리면 기존 스레드를 아래처럼 통일한다.

PR 상태 스레드 처리
열림 unarchive + 팀원 전원 추가
머지됨 이름에 ✅ + archived (닫힘, 잠금 안 함)
머지 없이 닫힘 이름에 🗑️ + archived
  • 과거 find_meta 버그로 안 닫힌 스레드까지 소급 정리한다. 멱등이라 재실행해도 안전하다.

Result

  • 팀 전원이 모든 PR 스레드를 사이드바에서 본다. 이슈 카드가 부모 카드 본문을 밀어내지 않는다. 머지 후에도 스레드에 글을 쓸 수 있다.
  • 백필은 최근 100개 PR(열림 + 닫힘) 대상이다. Discord rate limit 으로 일부가 실패하면 멱등이므로 재실행으로 채운다.
  • 클라(fe-pr)는 팀 노출과 이슈 임베드 억제는 이미 반영됐고, 머지 스레드 글쓰기 허용(locked 제거)은 별도 후속으로 반영한다.

연관 이슈

클라(fe-pr) 봇에 반영한 스레드 UX 개선을 서버 봇에도 맞춘다.

- 스레드 생성 직후 팀원 전원을 thread-members 로 추가 — 공개 스레드는 멤버만 사이드바에
  떠서 팀이 남의 PR 스레드를 못 보던 문제 해소(멘션과 달리 조용한 목록 추가)
- 연관 이슈 링크 URL 을 <> 로 감싸 Discord 임베드 카드 억제(카드가 부모 카드 본문을 밀어냄)
- 머지 스레드는 archived 만 유지하고 locked 제거 — 닫히되(목록 접힘) 글은 쓸 수 있게
- backfill-threads 잡(workflow_dispatch): 기존 스레드 일괄 통일 — 열린 PR 팀원 추가,
  머지 PR ✅+닫힘, 미머지 🗑️+닫힘. 과거 find_meta 버그로 안 닫힌 스레드까지 소급 정리(멱등)
@sevineleven sevineleven added the chore 그 외 (빌드·deps·ci·도구·설정·잡일) label Jul 7, 2026
@sevineleven sevineleven self-assigned this Jul 7, 2026
@coderabbitai

coderabbitai Bot commented Jul 7, 2026

Copy link
Copy Markdown

Warning

Review limit reached

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

Next review available in: 20 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: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

Run ID: c5b6cfe4-ef03-4dbe-95d4-57b1bf8a472c

📥 Commits

Reviewing files that changed from the base of the PR and between 96bc1d8 and c149bab.

📒 Files selected for processing (1)
  • .github/workflows/discord-pr-bot.yml
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/701-discord-thread-ux

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

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

@github-actions

github-actions Bot commented Jul 7, 2026

Copy link
Copy Markdown

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

@sevineleven
sevineleven merged commit 1418180 into dev Jul 7, 2026
7 checks passed
@sevineleven
sevineleven deleted the chore/701-discord-thread-ux branch July 7, 2026 17:24
m-a-king added a commit that referenced this pull request Jul 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore 그 외 (빌드·deps·ci·도구·설정·잡일)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PR 봇 Discord 스레드 UX 개선 (팀 노출·이슈 임베드 억제·머지 스레드 글쓰기 허용)

1 participant