[feat] 다정이 제거, 슬픔이 추가, 기쁨이에게 다정이 성격 추가 - #133
Merged
Merged
Conversation
Test Coverage
|
WalkthroughWARM 감정을 제거하고 SADNESS 감정을 추가했습니다. 프롬프트 캐릭터 매핑, 관리 화면, API 문서, 데이터베이스 마이그레이션, 골든 파일과 관련 테스트를 갱신했습니다. Changes감정 캐릭터 전환
Estimated code review effort: 3 (Moderate) | ~20 minutes 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches 💡 1📝 Generate docstrings 💡
🧪 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 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
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
`@src/main/resources/db/migration/V26__replace_dajeong_with_seulpeum_prompts.sql`:
- Around line 1-16: In
src/main/resources/db/migration/V26__replace_dajeong_with_seulpeum_prompts.sql
lines 1-16, add idempotent data cleanup for every schema-confirmed emotion
column in messages, cards, conversations, and generation_log, converting
persisted 'WARM' values to 'SADNESS' before normal reads occur, alongside the
existing prompt update. In
src/main/kotlin/com/nexters/gamss/emotion/domain/EmotionType.kt lines 4-14, no
direct change is required because the migration must make removal of WARM safe
for existing and newly initialized databases.
In
`@src/test/kotlin/com/nexters/gamss/card/controller/dto/CardCalendarResponseTest.kt`:
- Line 52: Update the test around CardCalendarResponseTest to assert that the
first grouped result’s emotions equals listOf("SADNESS"), alongside the existing
date assertion. This must verify SADNESS serialization and date grouping through
CardCalendarResponse.emotions.
🪄 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: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro Plus
Run ID: 41af65fa-525e-4ed6-b842-6d9cc92398af
📒 Files selected for processing (18)
admin-web/src/pages/dashboard/usage-section.tsxadmin-web/src/pages/prompt-playground.tsxsrc/main/kotlin/com/nexters/gamss/admin/controller/dto/PromptPreviewRequest.ktsrc/main/kotlin/com/nexters/gamss/conversation/controller/dto/MessageResponse.ktsrc/main/kotlin/com/nexters/gamss/emotion/domain/EmotionType.ktsrc/main/kotlin/com/nexters/gamss/llm/prompt/PromptCharacterId.ktsrc/main/resources/db/migration/V26__replace_dajeong_with_seulpeum_prompts.sqlsrc/test/kotlin/com/nexters/gamss/card/controller/dto/CardCalendarResponseTest.ktsrc/test/kotlin/com/nexters/gamss/conversation/controller/ConversationControllerIntegrationTest.ktsrc/test/kotlin/com/nexters/gamss/conversation/domain/MessageTest.ktsrc/test/kotlin/com/nexters/gamss/conversation/service/CommentGenerationServiceTest.ktsrc/test/kotlin/com/nexters/gamss/conversation/service/MessageThreadOrderTest.ktsrc/test/kotlin/com/nexters/gamss/llm/parsing/CommentFeedValidatorTest.ktsrc/test/kotlin/com/nexters/gamss/llm/prompt/PromptCharacterIdTest.ktsrc/test/kotlin/com/nexters/gamss/llm/settings/PromptSeedIntegrationTest.ktsrc/test/resources/prompts/CARD.default.txtsrc/test/resources/prompts/COMMENT.default.txtsrc/test/resources/prompts/COMMON.default.txt
kite707
changed the title
theminjunchoi
approved these changes
Aug 12, 2026
theminjunchoi
left a comment
theminjunchoi
left a comment
Collaborator
There was a problem hiding this 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
2 participants
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.
🔗 연관 이슈
📌 개요
감정 캐릭터에서 다정이를 빼고 슬픔이를 넣습니다. 다정이가 맡던 "챙기는" 포지션은 기쁨이 보이스 카드가 흡수하고, 슬픔이는 조언 대신 "그럴 만했다"를 먼저 주는 역할을 맡습니다.
🔧 주요 변경사항
EmotionType·PromptCharacterId에서WARM/dajeong을 제거하고SADNESS/seulpeum을 추가했습니다. 캐릭터는 계속 6종입니다.V26을 추가했습니다. COMMON은 유저-캐릭터 경계 규칙의 이름 목록, 기쁨이 보이스 카드, 다정이 자리의 슬픔이 보이스 카드가 바뀌고, COMMENT는 진영 분류 한 줄, CARD는 대변 방식 한 줄이 바뀝니다.🌐 API · DB 영향
MessageResponse.emotionType의 허용값에서WARM이 빠지고SADNESS가 들어갑니다.excludeCharacters도 같습니다.V26—llm_settings의 COMMON·COMMENT·CARD 프롬프트를 UPDATE하고, 각 타입의 리비전을max+1로 채번해 함께 남깁니다.V27—messages·cards·conversations·generation_log를 비웁니다.emotion = 'WARM'으로 저장된 카드·메시지는 enum 매핑에 실패합니다. 아직 실사용자 배포 전이라 보존할 데이터가 없다고 판단해 상수를 남기지 않고, 남은 행은V27로 정리합니다.V27을 수동 SQL이 아니라 마이그레이션으로 넣은 이유는 순서 때문입니다. 정리가 배포보다 늦으면 그 사이에 새 코드가WARM행을 읽다가 500이 나는데, Flyway는 앱이 트래픽을 받기 전에 실행되므로 그 구간 자체가 생기지 않습니다. 다정이 행만 골라 지우면replies_to_message_id·root_message_id가 사라진 메시지를 가리켜 스레드가 끊기므로 네 테이블을 통째로 비웁니다. 서로를 참조하는 컬럼이 이 네 테이블 안에만 있어 끊어진 참조는 남지 않고, 새로 만든 환경에서는 대상 행이 없어 아무 일도 하지 않습니다.되돌릴 수 없는 작업이므로 prod 배포 전에 덤프를 뜹니다.
💬 리뷰 포인트
처음에는
selectable플래그로 다정이를 "저장된 값으로는 유효하되 새로 뽑히지는 않는" 상태로 두고 신규 생성에서만 제외했는데, 배포 전이라 지켜야 할 과거 레코드가 없다는 게 확인돼서 제거했습니다. 그 장치를 두면 "캐릭터 전체"가entries(저장·번역 가능)와SELECTABLE(신규 생성 가능)로 쪼개집니다. 이후에 누가entries를 무심코 쓰면 다정이가 다시 생성 후보에 들어가는데, 예외도 나지 않고 조용히 그렇게 됩니다. 이 판단이 맞는지 봐주세요.프롬프트를 백오피스 수동 저장이 아니라 마이그레이션으로 넣은 것도 봐주시면 좋겠습니다. 프롬프트의 단일 원본은 DB지만, V23 시드에 다정이가 그대로 남아 있어 새로 만든 환경은 슬픔이 보이스 카드가 없는 상태가 됩니다. 이때 LLM이 임의로 지어내고 파싱은 성공하므로, 에러 없이 생성 품질만 떨어집니다. 각 환경의 리비전이 전부 v1이라(아무도 편집한 적 없음) 조건 없는 UPDATE로 덮어쓸 편집분도 없습니다.