Conversation
* feat: 디스코드 webhook 메시지 편집 기능 추가 검수 결과를 표시하려면 이미 전송한 알림 메시지를 수정할 수 있어야 하므로 editMessage 를 추가한다. - webhook 은 자신이 보낸 메시지를 봇 권한 없이 편집할 수 있다. - SimpleClientHttpRequestFactory 는 PATCH 를 지원하지 않으므로 webhook 전송용 팩토리를 JDK HttpClient 기반으로 교체한다. - webhook url 에 쿼리스트링이 붙어 있을 수 있어 경로 뒤에 이어붙이지 않는다. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> * refactor: DiscordNotifier 설정 값을 생성자로 주입 필드 주입은 테스트에서 인스턴스를 직접 만들 수 없어 리플렉션을 요구한다. 동작은 그대로 두고 주입 방식만 바꾼다. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> * fix: 성적 승인 시 디스코드 이모지 요청 실패로 발생하는 500 에러 해결 봇이 알림 채널에 초대되어 있지 않아 반응 요청이 403 Missing Access 를 받고, 예외가 검수 트랜잭션 밖으로 전파되어 롤백되면서 500 이 반환되었다. 봇을 초대하지 않기로 하여 표시 방식을 webhook 메시지 편집으로 바꾼다. - 검수 시점 데이터로 본문을 다시 만들어 마커와 함께 덮어쓴다. 본문을 새로 만들므로 재검수해도 마커가 누적되지 않는다. - notify 와 동일하게 webhook url 이 없거나 local 이면 아무것도 하지 않는다. - 편집 대상 메시지가 삭제된 경우(404)는 재시도해도 성공하지 않으므로 검수를 실패시키지 않는다. 그 외 실패는 기존과 같이 전파한다. - 더 이상 호출하지 않는 봇 반응 클라이언트를 제거한다. Fixes #840 Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> * refactor: DiscordReactionEmoji 를 DiscordReviewMarker 로 개명 반응(reaction)이 아니라 알림 메시지에 붙이는 마커이므로 이름을 동작에 맞춘다. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> * refactor: 사용하지 않는 디스코드 봇 RestTemplate 제거 봇 반응 클라이언트가 사라져 discordBotRestTemplate 을 주입받는 곳이 없다. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
봇을 초대하지 않아 반응(reaction)을 사용할 수 없어 알림 본문에 결과를 표시하는데, 이모지보다 문구가 결과를 분명하게 전달한다. Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
whqtker
requested review from
Gyuhyeok99,
Hexeong,
lsy1307,
sukangpunch and
wibaek
as code owners
August 29, 2026 11:39
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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. Comment |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.