Skip to content

Conversation

@yongjun0511
Copy link
Collaborator

No description provided.

JjungminLee and others added 22 commits September 23, 2025 21:49
* fix: statistics 공통 메뉴 나오는 문제 해결

* fix: list api 공통 메뉴 안나오게 수정

* feat: /v2/review/meals에서 mealID 추출 시 valid menu만 추출하게끔 변경

* feat: 내리뷰 조회 api dto id,name 형태로 변경

* feat: list dto에 id,name,like / statistics dto에 id, name추가

* refactor: gemini 리뷰 반영 및 menuNames -> menuList로 수정
* feat: 자장밥 필터링 안되게 추가

* refactor: /v2/reviews/my swagger json 깨지는 문제 수정

* refactor: valid-for-review swagger menulist 수정

* refactor: /list/menu mainRating->rating, 맛/양 rating 필드 삭제

* chore: 코드 포맷팅

* refactor: mainRating->rating으로 필드 변경

* /list/menu에 id,name,isLike추가

* fix: gemini 리뷰 반영
* chore: DS 파일 git ignore

* refactor: 만료된 제휴가 보이지 않도록 수정

* feat: 24시간 이내에 댓글 신고 횟수 제한

* refactor: 스웨거 문서 업데이트

* chore: CI/CD test

* chore: CI/CD test

* fix: 공백 제거

* fix: CI/CD 수정 완료

* fix: gemini 수정 사항 반영

* fix: writtenAt 필드 수정

---------

Co-authored-by: 나용준 <[email protected]>
@yongjun0511 yongjun0511 self-assigned this Sep 30, 2025
@gemini-code-assist
Copy link

Summary of Changes

Hello @yongjun0511, 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!

이 Pull Request는 v2.0.8 릴리즈를 위한 변경 사항을 포함하며, 주로 애플리케이션의 로깅 인프라를 대폭 개선하고 리뷰 시스템의 데이터 구조와 로직을 정교화하는 데 중점을 두었습니다. 또한, 비동기 처리 기능을 도입하고 빌드 설정을 업데이트하여 전반적인 시스템 안정성과 개발 편의성을 향상시키는 것을 목표로 합니다.

Highlights

  • 로깅 시스템 개선: 요청 ID(MDC)를 활용한 상세 로깅, 민감 정보 마스킹을 위한 @LogMask 어노테이션 도입, 그리고 비동기 이벤트 기반 로깅 시스템을 구축하여 애플리케이션의 가시성과 디버깅 효율성을 크게 향상시켰습니다.
  • 리뷰 시스템 구조 개선: 식단 및 메뉴 리뷰 응답 DTO를 MenuIdNameDtoMenuIdNameLikeDto와 같은 새로운 구조로 리팩토링하여 메뉴 정보를 더 체계적으로 제공합니다. 또한, 리뷰 평점 필드명을 mainRating에서 rating으로 통일하고, 메뉴 필터링 로직을 개선했습니다.
  • 리뷰 신고 기능 강화: 동일한 리뷰에 대해 24시간 이내 중복 신고를 방지하는 로직을 추가하여 신고 시스템의 오용을 줄이고, 신고 생성 시 로깅 이벤트를 발행하도록 업데이트했습니다.
  • 비동기 처리 활성화: @EnableAsync를 통해 애플리케이션 전반에 걸쳐 비동기 처리를 활성화하여, 특히 로깅과 같은 백그라운드 작업의 성능을 최적화했습니다.
  • 빌드 및 환경 설정 변경: build.gradle-parameters 컴파일러 인자를 추가하여 런타임 시 메서드 파라미터 이름을 유지하도록 했으며, 로컬 환경 설정 파일인 application-local.yml을 제거했습니다.
Ignored Files
  • Ignored by pattern: .github/workflows/** (1)
    • .github/workflows/deploy.yml
Using Gemini Code Assist

The 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 /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help /gemini help Displays a list of available commands.

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 .gemini/ folder in the base of the repository. Detailed instructions can be found here.

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.


코드의 흐름, 로그에 담으니, 요청마다 ID, 길을 밝히네. 민감한 정보, 별로 가리니, 보안과 기록, 함께 이루네. 새로운 버전, 빛을 발하리.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

Copy link

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

이번 PR은 포괄적인 로깅 시스템을 도입하고 여러 DTO를 리팩토링하여 API 응답을 개선했습니다. AOP를 통한 요청/응답 로깅, 트랜잭션을 인지하는 이벤트 기반 로깅, MDC를 사용한 요청 추적 등 새로운 로깅 인프라는 유지보수성과 디버깅 편의성을 크게 향상시킵니다. DTO 리팩토링은 API를 더욱 구조적이고 유익하게 만듭니다. 또한, 중복 신고 방지 및 메뉴 필터링 로직 개선과 같은 여러 소소한 개선 사항과 버그 수정도 포함되어 있습니다. 전반적으로 애플리케이션의 기반을 크게 개선하는 고품질의 PR입니다. 코드 단순화를 위한 한 가지 작은 제안 사항이 있습니다.

Comment on lines +42 to +44
List<Menu> menus = meal.getMealMenus().stream()
.map(MealMenu::getMenu)
.toList();

Choose a reason for hiding this comment

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

medium

Meal 엔티티에 이번 PR에서 getMenus() 메소드가 추가되었습니다. 이 메소드를 활용하여 코드를 더 간결하게 만들 수 있습니다. 스트림을 직접 사용하여 메뉴 목록을 가져오는 대신 meal.getMenus()를 호출하면 캡슐화가 개선되고 코드의 가독성이 높아집니다.

        List<Menu> menus = meal.getMenus();

@yongjun0511 yongjun0511 merged commit 3555353 into main Sep 30, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants