Conversation
개요AI 분석 프롬프트 생성기의 음성 지침을 수정했습니다. 기존 한 줄의 친화적 톤 지침을 더 상세한 가이드라인으로 교체하여, 대화체 톤과 한국어 정중한 어미(요/해요/네요) 사용, 다양한 문장 종결, 구어체 표현을 강조합니다. 변경 사항
관련 PR
제안된 검토자
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches
🧪 Generate unit tests (beta)
Tip Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord. 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 |
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
src/main/java/com/umc/finly/domain/analysis/association/infra/AiAnalysisPromptBuilder.java (1)
47-53: 페르소나 정의를 프롬프트 앞부분으로 이동 권장Line 47의 역할 정의(
"당신은 친근하고 다정한 금융 상담가입니다.")가 모든 요구사항(Lines 34–46) 이후에 위치하고 있습니다. LLM 프롬프트 설계에서는 역할/페르소나를 프롬프트 맨 앞에 배치해야 이후 지시사항 전체에 걸쳐 일관된 톤이 적용됩니다. 현재 구조에서는 요구사항 파싱 후 역할이 오버라이드되는 형태라 효과가 약할 수 있습니다.또한 Line 51의
"## 말투 가이드라인 (예시):"마크다운 헤더(##)는 나머지 프롬프트가 순수 평문(plain text)으로 구성된 것과 형식이 불일치합니다. 모델이##를 리터럴 텍스트로 인식할 수 있으므로 일반 텍스트 헤더로 통일하는 것이 좋습니다.♻️ 구조 개선 제안 (페르소나 선행 배치 + 마크다운 헤더 제거)
sb.append("[분석 데이터]\n"); + // 역할 정의를 프롬프트 최상단에 배치 + sb.append("당신은 친근하고 다정한 금융 상담가입니다.\n"); sb.append("- 공포지수: ").append(...).append("\n"); // ... (기존 데이터 및 요구사항 라인 유지) sb.append("'사용자', '투자자' 지칭 금지, 반드시 존댓말을 사용하세요.\n"); - sb.append("당신은 친근하고 다정한 금융 상담가입니다.\n"); sb.append("딱딱한 '~습니다' 체 대신, 이웃과 대화하듯 자연스럽고 부드러운 '~요', '~해요', '~네요' 체를 사용하세요.\n"); sb.append("문장 끝이 너무 반복되지 않도록 '~인 것 같아요', '~이 좋겠네요' 처럼 다양한 종결 어미를 섞어주세요.\n"); sb.append("군더더기 없는 문어체보다는 대화하듯 매끄러운 구어체를 지향하세요.\n"); - sb.append("## 말투 가이드라인 (예시):\n"); + sb.append("말투 가이드라인 (예시):\n"); sb.append("변경 전: ...\n"); sb.append("변경 후: ...\n");
| sb.append("군더더기 없는 문어체보다는 대화하듯 매끄러운 구어체를 지향하세요.\n"); | ||
| sb.append("## 말투 가이드라인 (예시):\n"); | ||
| sb.append("변경 전: 현재 공포지수가 낮고 매수확신도가 높아 탐욕과 확신이 느껴집니다. 이러한 긍정적인 감정은 매수 활동을 촉진할 수 있지만, 지나친 탐욕은 위험을 초래할 수 있습니다. 안정성을 위해 과도한 감정에 휘둘리지 않고 신중한 결정을 내리는 것이 좋습니다.\n"); | ||
| sb.append("변경 후: 현재 공포지수가 낮고 매수확신도가 높아 탐욕과 학신이 느껴지네요. 이러한 긍정적인 감정은 매수 활동을 촉진할 수 있지만, 지나친 탐욕은 위험을 초래할 수 있어요. 안정성을 위해 과도한 감정에 휘둘리지 않고 신중한 결정을 내리는 것이 좋아요.\n"); |
There was a problem hiding this comment.
"학신이" → "확신이" 오타 수정 필요
"변경 후" 예시 문장에 오타가 있습니다. "탐욕과 학신이 느껴지네요" → "탐욕과 확신이 느껴지네요". 이 예시는 LLM에게 직접 전달되는 프롬프트 문자열이며, 프롬프트 내 다른 곳에서 CONFIDENCE → '확신'으로 명시적으로 매핑하도록 지시한 것과 불일치하여 모델 출력 품질에 영향을 줄 수 있습니다.
🐛 수정 제안
- sb.append("변경 후: 현재 공포지수가 낮고 매수확신도가 높아 탐욕과 학신이 느껴지네요. ...\n");
+ sb.append("변경 후: 현재 공포지수가 낮고 매수확신도가 높아 탐욕과 확신이 느껴지네요. ...\n");📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| sb.append("변경 후: 현재 공포지수가 낮고 매수확신도가 높아 탐욕과 학신이 느껴지네요. 이러한 긍정적인 감정은 매수 활동을 촉진할 수 있지만, 지나친 탐욕은 위험을 초래할 수 있어요. 안정성을 위해 과도한 감정에 휘둘리지 않고 신중한 결정을 내리는 것이 좋아요.\n"); | |
| sb.append("변경 후: 현재 공포지수가 낮고 매수확신도가 높아 탐욕과 확신이 느껴지네요. 이러한 긍정적인 감정은 매수 활동을 촉진할 수 있지만, 지나친 탐욕은 위험을 초래할 수 있어요. 안정성을 위해 과도한 감정에 휘둘리지 않고 신중한 결정을 내리는 것이 좋아요.\n"); |
🔗 관련 이슈
closes #
📌 작업 내용
PM님 요청으로 연관분석 AI 말투 변경 (~했습니다 -> ~해요)
🧪 테스트 결과
📸 스크린샷 (선택)
📎 참고 사항 (선택)
Summary by CodeRabbit