Skip to content

[FIX] api 경로 수정#8

Merged
CHOOSLA merged 1 commit into
devfrom
SPM-78
Oct 14, 2025
Merged

[FIX] api 경로 수정#8
CHOOSLA merged 1 commit into
devfrom
SPM-78

Conversation

@taemin3
Copy link
Copy Markdown
Contributor

@taemin3 taemin3 commented Oct 14, 2025

📝 Summary

api 경로 수정

🙏 Question & PR point

📬 Reference

Summary by CodeRabbit

  • Refactor
    • 헬스체크 API의 기본 경로에서 버전 프리픽스를 제거했습니다. 기존 /api/v1 경로가 제거되어 엔드포인트가 상위 경로로 노출됩니다. 클라이언트는 새 경로로 호출해야 합니다.
  • Chores
    • 데이터베이스 컬럼 매핑을 정비하여 소재 이름 필드가 명시적 컬럼명과 매핑되도록 개선했습니다.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Oct 14, 2025

Walkthrough

HealthCheckController의 클래스 수준 @RequestMapping("/api/v1")가 제거되어 컨트롤러의 기본 경로가 변경되었습니다. Material 엔티티의 name 필드에 @Column(name = "material_name")가 추가되어 컬럼 매핑이 명시되었습니다.

Changes

Cohort / File(s) Change Summary
컨트롤러 경로 매핑 변경
src/main/java/com/sampoom/material/api/health/HealthCheckController.java
클래스 수준 @RequestMapping("/api/v1") 제거. 기존 @GetMapping 메서드들의 상대 경로는 동일하나, 컨트롤러의 기본 경로 접두사가 사라짐.
엔티티 컬럼 매핑 보강
src/main/java/com/sampoom/material/api/material/entity/Material.java
String name 필드에 @Column(name = "material_name") 추가로 DB 컬럼명 명시.

Sequence Diagram(s)

sequenceDiagram
    participant Client as Client
    participant Router as Spring Router
    participant HC as HealthCheckController

    note over Router: 클래스 수준 /api/v1 기본 경로 제거됨
    Client->>Router: GET /health (예시)
    alt 매핑 존재
        Router->>HC: 핸들러 메서드 호출
        HC-->>Router: 200 OK
        Router-->>Client: 응답 반환
    else 잘못된 경로
        Router-->>Client: 404 Not Found
    end
Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

(\/)

( •
•) 토끼 말해요, 경로가 살짝 이동했죠—

`>🍃 이름엔 표지가, material_name으로 또렷이 보여요.

탁— 문지방 한 칸, API는 더 가까이,

딱— 라벨 한 줄, 엔티티는 또렷이.

깡총깡총, 깔끔해진 길 따라 응답으로 고고!

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description Check ⚠️ Warning PR 설명이 템플릿의 Related Issue 섹션을 누락했고 이슈 닫기 정보가 빠져 있으며 Question & PR point가 비어 있어 필요한 정보를 충분히 제공하지 않고 있습니다. ## 📣 Related Issue 섹션을 추가해 연관된 이슈 번호와 close 구문을 명시하고 Question & PR point에 검토 시점에 알아야 할 사항이나 질문을 기입하거나 불필요하면 해당 섹션을 제거하며 요약을 좀 더 상세히 작성하세요.
✅ Passed checks (2 passed)
Check name Status Explanation
Title Check ✅ Passed 제목은 API 경로 수정 작업을 명확히 반영하고 있으나 Material 엔티티의 컬럼 매핑 추가라는 변경사항은 언급하지 않아 변경 내역 전체를 요약하지는 못하고 있습니다.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch SPM-78

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 8723d71 and 7c69234.

📒 Files selected for processing (2)
  • src/main/java/com/sampoom/material/api/health/HealthCheckController.java (0 hunks)
  • src/main/java/com/sampoom/material/api/material/entity/Material.java (1 hunks)
💤 Files with no reviewable changes (1)
  • src/main/java/com/sampoom/material/api/health/HealthCheckController.java
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: call-reusable-labeler / labeler
🔇 Additional comments (1)
src/main/java/com/sampoom/material/api/material/entity/Material.java (1)

22-23: 데이터베이스 마이그레이션 스크립트 확인 필요
현재 리포지토리에서 material_name 컬럼 변경을 반영한 마이그레이션 파일이 발견되지 않습니다. src/main/resources/db/migration 또는 Liquibase changelog 위치에서 관련 스크립트를 직접 확인해 주세요.


Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@vivivim vivivim left a comment

Choose a reason for hiding this comment

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

잘했어요

Copy link
Copy Markdown
Member

@Sangyoon98 Sangyoon98 left a comment

Choose a reason for hiding this comment

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

승인

Copy link
Copy Markdown
Member

@CHOOSLA CHOOSLA left a comment

Choose a reason for hiding this comment

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

승인!

@33Auto-Bot 33Auto-Bot added the ready-to-merge 3명 이상의 리뷰어에게 승인되어 병합 준비가 완료된 PR label Oct 14, 2025
@CHOOSLA CHOOSLA merged commit efdb048 into dev Oct 14, 2025
8 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge 3명 이상의 리뷰어에게 승인되어 병합 준비가 완료된 PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants