fix: 코드 오류 점검 #278

yongjun0511 · 2025-10-27T14:32:31Z

#️⃣ Issue Number

fix: 명세서 작업중 코드 점검 #277

📝 요약(Summary)

코드 오류를 점검했습니다.

💬 공유사항 to 리뷰어

✅ PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

커밋 메시지 컨벤션에 맞게 작성했습니다.
변경 사항에 대한 테스트를 했습니다.(버그 수정/기능에 대한 테스트).

gemini-code-assist · 2025-10-27T14:33:05Z

Summary of Changes

Hello @yongjun0511, I'm Gemini Code Assist¹! 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은 주로 여러 컨트롤러에 걸쳐 @ApiResponses 어노테이션을 조정하여 API 문서(Swagger/OpenAPI)를 개선하는 데 중점을 둡니다. 또한 SliceService에서 예외 유형과 관련된 사소한 버그 수정도 포함되어 있습니다. 전반적인 목표는 API 오류 응답 및 문서의 명확성과 정확성을 향상시키는 것입니다.

Highlights

API 문서 개선 (OAuthController): Apple 로그인 엔드포인트에서 불필요한 404 '존재하지 않는 유저' @ApiResponse를 제거하여 API 문서의 정확성을 높였습니다.
API 문서 개선 (ReportController): 리뷰 신고 엔드포인트에 404 '존재하지 않는 리뷰'에 대한 새로운 @ApiResponse를 추가하여, 존재하지 않는 리뷰에 대한 신고 시 발생할 수 있는 오류를 명확히 문서화했습니다.
서비스 로직 오류 수정 (SliceService): SliceService에서 존재하지 않는 Meal에 대한 예외 발생 시 NOT_FOUND_MENU 대신 NOT_FOUND_MEAL을 사용하도록 수정하여, 더 정확한 오류 처리를 보장합니다. 또한 BaseResponseStatus 임포트 방식을 개선했습니다.
API 문서 개선 (UserController): 닉네임 유효성 검사 및 업데이트 엔드포인트에 다양한 닉네임 관련 유효성 검사 실패 및 중복 오류에 대한 상세한 404 @ApiResponses를 추가하여, API 사용자가 발생 가능한 모든 오류 상황을 명확히 이해할 수 있도록 했습니다.

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.

API 문서 다듬고, 오류 명확히, 코드 개선, 부드러운 호출 기다려.

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. ↩

gemini-code-assist

Code Review

이번 PR은 코드 오류 점검을 통해 API 명세(Swagger)를 개선하고 일부 로직 오류를 수정하는 내용을 담고 있습니다. 전반적으로 코드의 정확성과 명확성을 높이는 좋은 변경이라고 생각합니다.

다만, 몇 가지 개선점을 발견하여 리뷰 코멘트를 남겼습니다.

SliceService에서 스타일 가이드에 위배되는 와일드카드 import가 사용되었습니다.
UserController의 닉네임 유효성 검사 및 수정 API에서 Swagger 문서의 오류 응답 코드가 실제 반환되는 값(400, 409)과 다르게 404로 잘못 기재되어 있습니다.

자세한 내용은 각 파일의 코멘트를 확인해주세요.

gemini-code-assist · 2025-10-27T14:34:57Z

src/main/java/ssu/eatssu/domain/user/presentation/UserController.java

+            @ApiResponse(responseCode = "404", description = "숫자로만 이루어진 닉네임", content = @Content(schema = @Schema(implementation = BaseResponse.class))),
+            @ApiResponse(responseCode = "404", description = "연속된 공백을 포함하는 닉네임", content = @Content(schema = @Schema(implementation = BaseResponse.class))),
+            @ApiResponse(responseCode = "404", description = "연속된 하이폰을 사용하는 닉네임", content = @Content(schema = @Schema(implementation = BaseResponse.class))),
+            @ApiResponse(responseCode = "404", description = "첫 글자가 한글,영문,숫자가 아닌 경우", content = @Content(schema = @Schema(implementation = BaseResponse.class))),
+            @ApiResponse(responseCode = "404", description = "마지막 글자가 한글,영문,숫자가 아닌 경우", content = @Content(schema = @Schema(implementation = BaseResponse.class))),
+            @ApiResponse(responseCode = "404", description = "한글,영문,숫자,공백,하이폰(-)이외의 문자를 쓴 경우", content = @Content(schema = @Schema(implementation = BaseResponse.class))),
+            @ApiResponse(responseCode = "404", description = "닉네임이 1자 이상 16이하가 아닌 경우", content = @Content(schema = @Schema(implementation = BaseResponse.class))),
+            @ApiResponse(responseCode = "404", description = "서비스명/브랜드명 단독 닉네임", content = @Content(schema = @Schema(implementation = BaseResponse.class))),
+            @ApiResponse(responseCode = "404", description = "관리자로 혼동될 수 있는 닉네임", content = @Content(schema = @Schema(implementation = BaseResponse.class))),
+            @ApiResponse(responseCode = "404", description = "욕설/비속어가 이름에 포함되는 경우", content = @Content(schema = @Schema(implementation = BaseResponse.class)))


닉네임 유효성 검사 실패 시의 HTTP 상태 코드가 잘못 지정되었습니다. BaseResponseStatus에 정의된 바에 따르면, 닉네임 관련 유효성 오류는 400 Bad Request를 반환해야 합니다. 하지만 Swagger 문서에는 404 Not Found로 명시되어 있습니다. API 명세의 정확성을 위해 responseCode를 "400"으로 수정해야 합니다.

Suggested change

@ApiResponse(responseCode = "404", description = "숫자로만 이루어진 닉네임", content = @Content(schema = @Schema(implementation = BaseResponse.class))),

@ApiResponse(responseCode = "404", description = "연속된 공백을 포함하는 닉네임", content = @Content(schema = @Schema(implementation = BaseResponse.class))),

@ApiResponse(responseCode = "404", description = "연속된 하이폰을 사용하는 닉네임", content = @Content(schema = @Schema(implementation = BaseResponse.class))),

@ApiResponse(responseCode = "404", description = "첫 글자가 한글,영문,숫자가 아닌 경우", content = @Content(schema = @Schema(implementation = BaseResponse.class))),

@ApiResponse(responseCode = "404", description = "마지막 글자가 한글,영문,숫자가 아닌 경우", content = @Content(schema = @Schema(implementation = BaseResponse.class))),

@ApiResponse(responseCode = "404", description = "한글,영문,숫자,공백,하이폰(-)이외의 문자를 쓴 경우", content = @Content(schema = @Schema(implementation = BaseResponse.class))),

@ApiResponse(responseCode = "404", description = "닉네임이 1자 이상 16이하가 아닌 경우", content = @Content(schema = @Schema(implementation = BaseResponse.class))),

@ApiResponse(responseCode = "404", description = "서비스명/브랜드명 단독 닉네임", content = @Content(schema = @Schema(implementation = BaseResponse.class))),

@ApiResponse(responseCode = "404", description = "관리자로 혼동될 수 있는 닉네임", content = @Content(schema = @Schema(implementation = BaseResponse.class))),

@ApiResponse(responseCode = "404", description = "욕설/비속어가 이름에 포함되는 경우", content = @Content(schema = @Schema(implementation = BaseResponse.class)))

@ApiResponse(responseCode = "400", description = "숫자로만 이루어진 닉네임", content = @Content(schema = @Schema(implementation = BaseResponse.class))),

@ApiResponse(responseCode = "400", description = "연속된 공백을 포함하는 닉네임", content = @Content(schema = @Schema(implementation = BaseResponse.class))),

@ApiResponse(responseCode = "400", description = "연속된 하이폰을 사용하는 닉네임", content = @Content(schema = @Schema(implementation = BaseResponse.class))),

@ApiResponse(responseCode = "400", description = "첫 글자가 한글,영문,숫자가 아닌 경우", content = @Content(schema = @Schema(implementation = BaseResponse.class))),

@ApiResponse(responseCode = "400", description = "마지막 글자가 한글,영문,숫자가 아닌 경우", content = @Content(schema = @Schema(implementation = BaseResponse.class))),

@ApiResponse(responseCode = "400", description = "한글,영문,숫자,공백,하이폰(-)이외의 문자를 쓴 경우", content = @Content(schema = @Schema(implementation = BaseResponse.class))),

@ApiResponse(responseCode = "400", description = "닉네임이 1자 이상 16이하가 아닌 경우", content = @Content(schema = @Schema(implementation = BaseResponse.class))),

@ApiResponse(responseCode = "400", description = "서비스명/브랜드명 단독 닉네임", content = @Content(schema = @Schema(implementation = BaseResponse.class))),

@ApiResponse(responseCode = "400", description = "관리자로 혼동될 수 있는 닉네임", content = @Content(schema = @Schema(implementation = BaseResponse.class))),

@ApiResponse(responseCode = "400", description = "욕설/비속어가 이름에 포함되는 경우", content = @Content(schema = @Schema(implementation = BaseResponse.class)))

gemini-code-assist · 2025-10-27T14:34:58Z

src/main/java/ssu/eatssu/domain/user/presentation/UserController.java

+            @ApiResponse(responseCode = "404", description = "숫자로만 이루어진 닉네임", content = @Content(schema = @Schema(implementation = BaseResponse.class))),
+            @ApiResponse(responseCode = "404", description = "연속된 공백을 포함하는 닉네임", content = @Content(schema = @Schema(implementation = BaseResponse.class))),
+            @ApiResponse(responseCode = "404", description = "연속된 하이폰을 사용하는 닉네임", content = @Content(schema = @Schema(implementation = BaseResponse.class))),
+            @ApiResponse(responseCode = "404", description = "첫 글자가 한글,영문,숫자가 아닌 경우", content = @Content(schema = @Schema(implementation = BaseResponse.class))),
+            @ApiResponse(responseCode = "404", description = "마지막 글자가 한글,영문,숫자가 아닌 경우", content = @Content(schema = @Schema(implementation = BaseResponse.class))),
+            @ApiResponse(responseCode = "404", description = "한글,영문,숫자,공백,하이폰(-)이외의 문자를 쓴 경우", content = @Content(schema = @Schema(implementation = BaseResponse.class))),
+            @ApiResponse(responseCode = "404", description = "닉네임이 1자 이상 16이하가 아닌 경우", content = @Content(schema = @Schema(implementation = BaseResponse.class))),
+            @ApiResponse(responseCode = "404", description = "서비스명/브랜드명 단독 닉네임", content = @Content(schema = @Schema(implementation = BaseResponse.class))),
+            @ApiResponse(responseCode = "404", description = "관리자로 혼동될 수 있는 닉네임", content = @Content(schema = @Schema(implementation = BaseResponse.class))),
+            @ApiResponse(responseCode = "404", description = "욕설/비속어가 이름에 포함되는 경우", content = @Content(schema = @Schema(implementation = BaseResponse.class))),
+            @ApiResponse(responseCode = "404", description = "중복된 닉네임", content = @Content(schema = @Schema(implementation = BaseResponse.class)))


닉네임 수정 API의 오류 응답 상태 코드가 잘못 지정되었습니다.

닉네임 유효성 검사 실패 시에는 400 Bad Request를 반환해야 합니다.

중복된 닉네임의 경우 BaseResponseStatus.DUPLICATE_NICKNAME에 따라 409 Conflict를 반환해야 합니다.

현재 Swagger 문서에는 모두 404 Not Found로 명시되어 있어 API 명세가 부정확합니다. 올바른 상태 코드로 수정해주세요.

Suggested change

@ApiResponse(responseCode = "404", description = "숫자로만 이루어진 닉네임", content = @Content(schema = @Schema(implementation = BaseResponse.class))),

@ApiResponse(responseCode = "404", description = "연속된 공백을 포함하는 닉네임", content = @Content(schema = @Schema(implementation = BaseResponse.class))),

@ApiResponse(responseCode = "404", description = "연속된 하이폰을 사용하는 닉네임", content = @Content(schema = @Schema(implementation = BaseResponse.class))),

@ApiResponse(responseCode = "404", description = "첫 글자가 한글,영문,숫자가 아닌 경우", content = @Content(schema = @Schema(implementation = BaseResponse.class))),

@ApiResponse(responseCode = "404", description = "마지막 글자가 한글,영문,숫자가 아닌 경우", content = @Content(schema = @Schema(implementation = BaseResponse.class))),

@ApiResponse(responseCode = "404", description = "한글,영문,숫자,공백,하이폰(-)이외의 문자를 쓴 경우", content = @Content(schema = @Schema(implementation = BaseResponse.class))),

@ApiResponse(responseCode = "404", description = "닉네임이 1자 이상 16이하가 아닌 경우", content = @Content(schema = @Schema(implementation = BaseResponse.class))),

@ApiResponse(responseCode = "404", description = "서비스명/브랜드명 단독 닉네임", content = @Content(schema = @Schema(implementation = BaseResponse.class))),

@ApiResponse(responseCode = "404", description = "관리자로 혼동될 수 있는 닉네임", content = @Content(schema = @Schema(implementation = BaseResponse.class))),

@ApiResponse(responseCode = "404", description = "욕설/비속어가 이름에 포함되는 경우", content = @Content(schema = @Schema(implementation = BaseResponse.class))),

@ApiResponse(responseCode = "404", description = "중복된 닉네임", content = @Content(schema = @Schema(implementation = BaseResponse.class)))

@ApiResponse(responseCode = "400", description = "숫자로만 이루어진 닉네임", content = @Content(schema = @Schema(implementation = BaseResponse.class))),

@ApiResponse(responseCode = "400", description = "연속된 공백을 포함하는 닉네임", content = @Content(schema = @Schema(implementation = BaseResponse.class))),

@ApiResponse(responseCode = "400", description = "연속된 하이폰을 사용하는 닉네임", content = @Content(schema = @Schema(implementation = BaseResponse.class))),

@ApiResponse(responseCode = "400", description = "첫 글자가 한글,영문,숫자가 아닌 경우", content = @Content(schema = @Schema(implementation = BaseResponse.class))),

@ApiResponse(responseCode = "400", description = "마지막 글자가 한글,영문,숫자가 아닌 경우", content = @Content(schema = @Schema(implementation = BaseResponse.class))),

@ApiResponse(responseCode = "400", description = "한글,영문,숫자,공백,하이폰(-)이외의 문자를 쓴 경우", content = @Content(schema = @Schema(implementation = BaseResponse.class))),

@ApiResponse(responseCode = "400", description = "닉네임이 1자 이상 16이하가 아닌 경우", content = @Content(schema = @Schema(implementation = BaseResponse.class))),

@ApiResponse(responseCode = "400", description = "서비스명/브랜드명 단독 닉네임", content = @Content(schema = @Schema(implementation = BaseResponse.class))),

@ApiResponse(responseCode = "400", description = "관리자로 혼동될 수 있는 닉네임", content = @Content(schema = @Schema(implementation = BaseResponse.class))),

@ApiResponse(responseCode = "400", description = "욕설/비속어가 이름에 포함되는 경우", content = @Content(schema = @Schema(implementation = BaseResponse.class))),

@ApiResponse(responseCode = "409", description = "중복된 닉네임", content = @Content(schema = @Schema(implementation = BaseResponse.class)))

gemini-code-assist · 2025-10-27T14:34:58Z

src/main/java/ssu/eatssu/domain/slice/service/SliceService.java


-import static ssu.eatssu.global.handler.response.BaseResponseStatus.NOT_FOUND_MENU;
-import static ssu.eatssu.global.handler.response.BaseResponseStatus.NOT_FOUND_USER;
+import static ssu.eatssu.global.handler.response.BaseResponseStatus.*;


와일드카드(*)를 사용한 import는 리포지토리 스타일 가이드에 위배됩니다.¹ NOT_FOUND_MEAL이 추가되었으므로, 필요한 BaseResponseStatus만 명시적으로 import 하도록 수정해주세요.

Suggested change

import static ssu.eatssu.global.handler.response.BaseResponseStatus.*;

import static ssu.eatssu.global.handler.response.BaseResponseStatus.NOT_FOUND_MEAL;

import static ssu.eatssu.global.handler.response.BaseResponseStatus.NOT_FOUND_MENU;

import static ssu.eatssu.global.handler.response.BaseResponseStatus.NOT_FOUND_USER;

Style Guide References

Footnotes

라이브러리를 import할 때 와일드카드를 사용하지 않아야 합니다. ↩

fix: 수정 사항 반영

dc2a0e7

yongjun0511 self-assigned this Oct 27, 2025

yongjun0511 added the fix 고치기 label Oct 27, 2025

yongjun0511 linked an issue Oct 27, 2025 that may be closed by this pull request

fix: 명세서 작업중 코드 점검 #277

Closed

gemini-code-assist bot reviewed Oct 27, 2025

View reviewed changes

fix: 제미나이 수정 사항 반영

6ecbf24

yongjun0511 merged commit 319451d into develop Oct 29, 2025

yongjun0511 deleted the fix/#277-code-check branch October 29, 2025 12:00

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

fix: 코드 오류 점검 #278

fix: 코드 오류 점검 #278

Uh oh!

yongjun0511 commented Oct 27, 2025

Uh oh!

gemini-code-assist bot commented Oct 27, 2025

Uh oh!

gemini-code-assist bot left a comment

Uh oh!

gemini-code-assist bot Oct 27, 2025

Uh oh!

gemini-code-assist bot Oct 27, 2025

Uh oh!

gemini-code-assist bot Oct 27, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

fix: 코드 오류 점검 #278

fix: 코드 오류 점검 #278

Uh oh!

Conversation

yongjun0511 commented Oct 27, 2025

#️⃣ Issue Number

📝 요약(Summary)

💬 공유사항 to 리뷰어

✅ PR Checklist

Uh oh!

gemini-code-assist bot commented Oct 27, 2025

Summary of Changes

Highlights

Footnotes

Uh oh!

gemini-code-assist bot left a comment

Choose a reason for hiding this comment

Code Review

Uh oh!

gemini-code-assist bot Oct 27, 2025

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist bot Oct 27, 2025

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist bot Oct 27, 2025

Choose a reason for hiding this comment

Style Guide References

Footnotes

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants