Conversation
기존 코드는 답변 제출 시 현재 팀 멤버 수와 DONE 답변 수를 == 비교했는데, 회고 진행 중 멤버가 탈퇴/합류하면 수치가 맞지 않아 AI 분석이 영원히 트리거되지 않는 버그가 있었다. - Retrospect 엔티티에 targetMemberCount 필드 추가 (회고 생성 시점 멤버 수 스냅샷) - 회고 생성 시 현재 스페이스 멤버 수를 targetMemberCount로 저장 - 분석 트리거 조건을 == 에서 >= 로 변경하고 targetMemberCount 기준으로 비교 - null인 기존 데이터는 현재 팀 멤버 수로 폴백 (기존 동작 유지) 배포 전 DB 마이그레이션 필요: ALTER TABLE retrospect ADD COLUMN target_member_count INT NULL; Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
PROCEEDING 상태 회고의 totalCount를 현재 팀원 수 대신 생성 시점에 스냅샷된 targetMemberCount 기준으로 표시. null인 경우(기존 데이터)는 현재 팀원 수로 폴백. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
팀 분석 후 팀 실행목표(TEAM)를 생성하는 것과 동일하게, 개인 분석 후 각 멤버의 IMPROVEMENT 항목을 PERSONAL 타입 실행목표로 자동 저장하도록 추가. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
DONE 상태 회고에서 getTeamMemberCountBefore(deadline)가 멤버 가입일 기준으로 필터링해 0이 되는 문제 수정. targetMemberCount가 있으면 상태와 무관하게 항상 우선 사용하고, null인 기존 데이터만 이전 로직으로 폴백. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
답변이 0개인 상태에서 분석을 요청하면 AI 할루시네이션이 발생하는 문제 수정. DONE 답변이 없으면 OpenAI 호출 없이 analysisStatus만 DONE으로 변경. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
closeRetrospect 호출 시 DONE 답변이 하나도 없으면 400 에러를 반환하여 할루시네이션 방지. RetrospectExceptionType에 NO_ANSWERS_TO_CLOSE 추가. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
deadline 초과로 자동 마감되는 회고에 DONE 답변이 없으면 AIAnalyzeStartEvent를 발행하지 않고 analysisStatus를 DONE으로 처리. AnswerRepository에 existsByRetrospectIdAndAnswerStatus 추가. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
imgUrl DB 저장 방식 대신 EmojiCode enum(LEC01~LEC10)으로 서버-클라이언트 간 소통하고, 이미지 리소스는 클라이언트가 관리. 요청 시 emojiCode(String) 전달, 응답에 emojiCode + description 포함. LEC01 대단해 / LEC02 완벽해 / LEC03 최고야 / LEC04 역시 LEC05 고생했어 / LEC06 기대중 / LEC07 괜찮아 / LEC08 성장했다 LEC09 화이팅 / LEC10 할 수 있다 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
imgUrl → emojiCode/description 변경 반영. 이모지 코드 목록(LEC01~LEC10) 및 변경된 요청/응답 스펙 업데이트. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
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: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
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.