Skip to content

Comments

[Hotfix] 연관분석 AI 프롬프트 수정#223

Merged
yjhss merged 3 commits intodevelopfrom
feat/analysis-108-hourly-chart
Feb 12, 2026
Merged

[Hotfix] 연관분석 AI 프롬프트 수정#223
yjhss merged 3 commits intodevelopfrom
feat/analysis-108-hourly-chart

Conversation

@yjhss
Copy link
Contributor

@yjhss yjhss commented Feb 12, 2026

🔗 관련 이슈

관련된 이슈 번호를 적어주세요.

closes #

📌 작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요.

프론트 측에서 감정이 영어로 나와서 어색하다는 피드백 요청을 받아 수정

/analysis/association/infra/AiAnalysisPromptBuilder 프롬프트 수정

sb.append("반드시 한국어만 사용할 것.:\n");
sb.append("감정을 언급할 경우, GREED는 '탐욕', CONFIDENCE는 '확신', REGRET는 '후회', ANXIETY는 '불안', CALM은 '평온'으로 매핑하여 출력하세요.\n");

🧪 테스트 결과

Postman 스크린샷, 테스트 통과 여부 등을 첨부해주세요.

image

📸 스크린샷 (선택)

필요시 스크린샷을 첨부해주세요.

📎 참고 사항 (선택)

리뷰어에게 전달할 내용이 있다면 작성해주세요.

Summary by CodeRabbit

Summary by CodeRabbit

릴리스 노트

  • Documentation

    • API 명세서 설명 정리 — 특정 주가 데이터 API 언급 삭제 및 문서 간결화
  • Chores

    • AI 분석 프롬프트 개선 — 한국어 전용 강제화 및 금지 표현 범위 조정
    • 감정 매핑 추가 — 감정 레이블의 한국어 대응어 도입
    • 출력 형식 제약 강화 — 문장 수·종결표현·후행 개행 등 포맷 규칙 명확화

@coderabbitai
Copy link

coderabbitai bot commented Feb 12, 2026

Walkthrough

API 명세의 일일 차트 설명에서 특정 주가 데이터 API 관련 문구를 삭제하고, AI 분석 프롬프트 문구를 한국어 전용으로 명시하고 금지 용어·감정 매핑·줄바꿈·형식 제약을 수정함(행동·패턴 표현과 문장 포맷 조정 포함).

Changes

Cohort / File(s) Summary
API 명세 문서 업데이트
src/main/java/com/umc/finly/domain/analysis/association/controller/AssociationAnalysisApiSpecification.java
일일 차트 설명에서 특정 주가 데이터 API 사용 관련 문구(문서 주석) 삭제.
AI 프롬프트 지침 수정
src/main/java/com/umc/finly/domain/analysis/association/infra/AiAnalysisPromptBuilder.java
한국어 전용 명시 추가, 금지 용어 목록 및 표현 변경(예: 심리 상태/투자 행동 패턴 관련 표현 조정), 감정 매핑 문구 추가(탐욕/확신/후회/불안/평온), 출력 형식·줄바꿈 제약 및 마지막 줄에 개행 추가.

Possibly related PRs

Suggested reviewers

  • dosp74
  • na311ng
  • jangsh7
🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed 제목이 PR의 주요 변경 사항(AI 프롬프트 수정)을 명확하게 설명하고 있습니다.
Description check ✅ Passed 필수 섹션인 관련 이슈, 작업 내용, 테스트 결과가 포함되어 있으며 수정 사항이 명확히 설명되어 있습니다.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/analysis-108-hourly-chart

No actionable comments were generated in the recent review. 🎉


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 and usage tips.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

@yjhss yjhss added ♻️Refactor 리팩토링 🚨Hotfix 비상 비상 비상! labels Feb 12, 2026
@yjhss yjhss self-assigned this Feb 12, 2026
@yjhss yjhss merged commit e24344d into develop Feb 12, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🚨Hotfix 비상 비상 비상! ♻️Refactor 리팩토링

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants