Conversation
Summary of ChangesHello @Jin409, 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은 응원톡 저장 시 기존에 적용되던 욕설 필터링 기능을 제거하고, 해당 기능을 Deprecated 처리하는 변경사항을 포함합니다. 이는 스포츠 경기 특성상 욕설이 포함된 코멘트가 발생할 수 있음을 고려하여, 향후 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
Activity
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
|
There was a problem hiding this comment.
Code Review
안녕하세요. 응원톡 저장 시 필터링을 제거하는 변경사항 잘 보았습니다. LLM을 이용한 마스킹 처리로 정책을 변경하면서 기존 필터링 로직을 제거하고, 관련 코드를 @Deprecated 처리하신 점이 인상 깊습니다. 덕분에 추후 기능 복원이 용이해 보이네요.
코드 리뷰에서는 주로 테스트 코드의 유지보수성에 초점을 맞추었습니다. 비활성화된 테스트들을 새로운 정책에 맞게 수정하여 현재 로직을 더 명확하게 검증하도록 제안했습니다. 또한, 더 이상 사용되지 않는 Spring 컴포넌트를 컨텍스트에서 제외하여 불필요한 리소스 사용을 막는 작은 개선점도 포함했습니다.
자세한 내용은 각 파일의 주석을 참고해주세요.
|
|
||
| } | ||
|
|
||
| @Disabled(value = "욕설 필터링 기능을 LLM 으로 대체해 스킵") |
|
고생하셨습니다! |
TAEW00KIM
left a comment
There was a problem hiding this comment.
간단한 리팩토링이라 따로 크게 볼 건 없네요 ㅎㅎ 고생하셨습니다 저도 슬슬 일에 참여해볼게요!
📝 구현 내용
as-is
to-be
제거의 배경