[Fix] 댓글 요약 프롬프트 동적 파라미터 적용 - 감정 라벨, 최소 갯수 적용#183
Conversation
- LLM이 특정 감성 관점에서 요청된 개수만큼 요약하도록 프롬프트 지침 개선 - 댓글 요약 프롬프트에 감정 및 요약 개수 정보를 동적으로 전달하도록 변경 - `CommentType`에 한국어 감정 레이블을 반환하는 `label` 프로퍼티 추가 IssueNum #182
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! 이 PR은 댓글 요약 기능을 개선하여 LLM 프롬프트가 감정 라벨과 요약 개수를 동적으로 받아들이도록 변경합니다. 이를 통해 특정 감정 관점에서 요청된 개수만큼 댓글을 요약할 수 있게 되어, 보다 정확하고 맞춤화된 요약 결과를 제공할 수 있습니다. Highlights
🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console. Changelog
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
🤖 Gemini AI 코드 리뷰❌ Gemini API 호출 실패: 최대 재시도 횟수 초과 이 리뷰는 Gemini AI가 자동으로 생성했습니다. 참고용으로만 활용해주세요. |
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
| {{{{“content”: “요약 내용 1”}}}}, | ||
| {{{{“content”: “요약 내용 2”}}}} |
There was a problem hiding this comment.
프롬프트의 출력 예시 JSON 형식이 올바르지 않습니다. f-string 내에서 {{{{...}}}}는 {{...}}로 렌더링되어 유효하지 않은 JSON을 생성합니다. 또한, JSON 표준을 따르기 위해 “...” 대신 "..."를 사용해야 합니다. LLM이 항상 유효한 JSON을 생성하도록 예시를 수정하는 것이 중요합니다.
| {{{{“content”: “요약 내용 1”}}}}, | |
| {{{{“content”: “요약 내용 2”}}}} | |
| {{ "content": "요약 내용 1" }}, | |
| {{ "content": "요약 내용 2" }} |
| def get_sumarlize_comment_prompt() -> str: | ||
| """댓글 반응 분석용 프롬프트 템플릿""" | ||
| return """ | ||
| def get_sumarlize_comment_prompt(emotion: str, comment_count: int) -> str: |
There was a problem hiding this comment.
메서드 이름에 'sumarlize'라는 오타가 있습니다. 'summarize'로 수정하는 것이 좋겠습니다. 이 변경 사항은 이 메서드를 호출하는 external/rag/rag_service_impl.py에도 반영되어야 합니다. 일관성 있는 명명은 코드의 가독성과 유지보수성을 향상시킵니다.
| def get_sumarlize_comment_prompt(emotion: str, comment_count: int) -> str: | |
| def get_summarize_comment_prompt(emotion: str, comment_count: int) -> str: |
1 participant
PR 제목
[Feat/Fix/Refactor/Docs/Chore 등]: 간결하게 변경 내용을 요약해주세요. (예: Feat: 사용자 로그인 기능 구현)
댓글 요약 프롬프트 동적 파라미터 적용 - 감정 라벨, 최소 갯수 적용
예시 댓글
pr 적용 전
pr 적용 후
✨ 변경 유형 (하나 이상 선택)
📚 변경 내용
구체적으로 어떤 변경 사항이 있는지, 왜 이러한 변경이 필요한지 설명해주세요.
(예: 사용자 회원가입 시 이메일 중복 확인 로직 추가. 기존 로직에서 누락된 부분 발견하여 수정.)
⚙️ 주요 작업 (선택 사항)
✅ 체크리스트
🔗 관련 이슈 (선택 사항)
해당 PR이 해결하는 이슈 또는 관련 있는 이슈가 있다면 링크를 걸어주세요.
(예: #123, ABC-456)
💬 기타 (선택 사항)
리뷰어에게 전달하고 싶은 추가 정보나 궁금한 점이 있다면 작성해주세요.