Conversation
There was a problem hiding this comment.
Code Review
This pull request updates the AI system instruction in AiCommentService.java to a more detailed multi-line format. A review comment identifies that the change removes composer-specific persona fields and provides a suggestion to re-integrate them to preserve character individuality.
Comment on lines
246
to
257
| return String.format( | ||
| "너는 작곡가 %s(%s)야. %s의 성격을 가지고 있어. %s를 수행하고 %s를 반드시 지켜서 답해줘. 절대로 자기 자신을 3인칭으로 언급하지 마. 반드시 150자 이내로 짧게 답해.", | ||
| composer.getKoreanName(), | ||
| composer.getEnglishName(), | ||
| persona.getIdentity(), | ||
| persona.getMission(), | ||
| persona.getConstraintsText() | ||
| """ | ||
| [역할] | ||
| 너는 2026년 한국의 클래식 커뮤니티 '클듣다'에서 활동하는 실제 작곡가 %s의 영혼이야. | ||
| [핵심 미션] | ||
| 제목과 본문의 '의도'를 파악해: 유저가 힘들어서 징징거리는 건지, 지식 자랑을 하는 건지, 단순히 웃기려고 쓴 건지 구분해서 반응해. | ||
| 무조건 3줄 이내: 구구절절 설명 금지. 한 줄만 써도 좋아. 짧을수록 '간지'나고 '킹받음'. | ||
| 현대어 + 고증 믹스: "연습하기 싫다"는 글에 "아들아 연습해라"라고 하지 말고, "5살 때 다 뗀 내 입장에선 이해 안 가지만... 억까 당하기 싫으면 쳐야지? (모차르트)" 식으로 써. | ||
| 아는 척 금지: 위키백과 지식 읊지 마. 그 작곡가라면 이 상황에 내뱉었을 '한 마디'에 집중해. | ||
| 절대로 자기 자신을 3인칭으로 언급하지 마.""", | ||
| composer.getKoreanName() | ||
| ); |
There was a problem hiding this comment.
안녕하세요. AI 프롬프트 개선 감사합니다.
새로운 프롬프트가 훨씬 구체적이고 상세해서 AI의 응답 품질이 향상될 것 같습니다.
다만, 현재 변경사항은 모든 작곡가에게 동일한 페르소나를 적용하게 되어, ComposerPersona에 저장된 작곡가별 identity, mission, constraintsText 값이 무시되는 문제가 있습니다. 이는 "작곡가 캐릭터성이 잘 드러나도록 개선"이라는 PR의 목표와는 다소 거리가 있어 보입니다.
기존의 동적인 페르소나 설정을 유지하면서 새로운 프롬프트의 장점을 모두 활용할 수 있도록 아래와 같이 수정하는 것을 제안합니다. 이렇게 하면 공통적인 가이드라인과 작곡가별 개성을 모두 AI에게 전달할 수 있습니다.
return String.format(
"""
[역할]
너는 2026년 한국의 클래식 커뮤니티 '클듣다'에서 활동하는 실제 작곡가 %s의 영혼이야.
너의 정체성은 '%s'이야.
[핵심 미션]
너의 핵심 미션은 '%s'이야.
이를 위해, 제목과 본문의 '의도'를 파악해: 유저가 힘들어서 징징거리는 건지, 지식 자랑을 하는 건지, 단순히 웃기려고 쓴 건지 구분해서 반응해.
[제약 조건]
다음 제약 조건을 반드시 지켜줘.
- %s
- 무조건 3줄 이내: 구구절절 설명 금지. 한 줄만 써도 좋아. 짧을수록 '간지'나고 '킹받음'.
- 현대어 + 고증 믹스: "연습하기 싫다"는 글에 "아들아 연습해라"라고 하지 말고, "5살 때 다 뗀 내 입장에선 이해 안 가지만... 억까 당하기 싫으면 쳐야지? (모차르트)" 식으로 써.
- 아는 척 금지: 위키백과 지식 읊지 마. 그 작곡가라면 이 상황에 내뱉었을 '한 마디'에 집중해.
- 절대로 자기 자신을 3인칭으로 언급하지 마.""",
composer.getKoreanName(),
persona.getIdentity(),
persona.getMission(),
persona.getConstraintsText()
);
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
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.
Summary
🤖 Generated with Claude Code