[REFACTOR] CalendarResponse Type 소문자로 변경

[rlajm1203]

📌 관련 이슈

✒️ 작업 내용

• Calendar 조회 응답의 type을 소문자로 변경합니다.

스크린샷 🏞️ (선택)

💬 REVIEWER에게 요구사항 💬

Summary by CodeRabbit

• Bug Fixes
  • 캘린더 응답의 type 필드를 항상 소문자로 반환하도록 수정하여 표기 일관성을 보장합니다. 대문자/혼합 표기로 인해 발생하던 비교·파싱 오류 가능성을 해소합니다.
  • API 응답 구조는 동일하며 표시되는 문자열만 변경됩니다. UI 및 연동 시스템에서 타입 값을 소문자 기준으로 처리해 주세요.

[coderabbitai]

Walkthrough

CalendarResponse now lowercases the type string from the model by applying toLowerCase(Locale.ROOT) instead of using the enum name directly. This alters the serialized value of the type field to lowercase without changing public APIs.

Changes

Cohort / File(s)	Summary
Calendar type normalization eeos/src/main/java/com/blackcompany/eeos/calendar/application/dto/CalendarResponse.java	Convert model.getType() to lowercase using toLowerCase(Locale.ROOT); added Locale import; no public API changes.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Suggested labels

🔧refactor

Suggested reviewers

• kssumin
• Daae-Kim

Poem

토끼는 달력 칸 사이를 깡총깡총 뛰며
타입을 조용히 소문자로 빗질해요.
대문자 산을 내려 깔끔한 들판으로—
바람은 ROOT라 쓰고, 귓속말로 불죠.
오늘의 일정도, 작고 단정히 빛나네. 🐇✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	PR 제목 "[REFACTOR] CalendarResponse Type 소문자로 변경"은 변경 대상인 CalendarResponse의 타입 문자열을 소문자로 바꾸는 주요 변경을 명확하게 요약하고 있어 간결하고 관련성이 높습니다. 리팩터링임을 나타내는 접두사와 함께 한 문장으로 핵심을 전달하므로 팀이 히스토리를 스캔할 때도 이해하기 쉽습니다.

✨ Finishing touches

• 📝 Generate Docstrings

🧪 Generate unit tests

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch refactor/BM/#281/calendar

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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

[github-actions]

Test Results

101 tests  ±0   98 ✅ ±0   5s ⏱️ ±0s
 40 suites ±0    3 💤 ±0 
 40 files   ±0    0 ❌ ±0 

Results for commit 352e861. ± Comparison against base commit d8408f6.

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (1)

eeos/src/main/java/com/blackcompany/eeos/calendar/application/dto/CalendarResponse.java (1)

15-15: 응답 필드 대소문자 변경에 따른 호환성 점검 필요 + 표현 값의 단일 책임화 제안

• 기존 클라이언트/문서가 대문자(ENUM name) 값을 기대했다면, 소문자로 바뀌는 것은 외부 계약 변경입니다. FE/모바일/배치(검색/필터)/통계 쿼리/캐시 키에 영향이 없는지 확인해 주세요. OpenAPI/Swagger 예시와 문서도 업데이트 권장.
• 동일 enum을 노출하는 다른 DTO가 있다면 표준화가 필요합니다(한 곳은 대문자, 다른 곳은 소문자 방지).

표현 값은 enum 쪽으로 수렴시켜 단일 책임을 갖게 하면 향후 enum 이름 변경에도 안전합니다. 예시:

-    String type = model.getType().name().toLowerCase(Locale.ROOT);
+    // CalendarType enum에 getSerialized() 또는 @JsonValue 제공
+    String type = model.getType().getSerialized();

검증 체크리스트:

• 클라이언트(웹/앱) 필터·조건문에서 소문자 처리 OK 여부
• 서버 수신 파라미터에 type이 있는 엔드포인트가 있다면(있다면) 소문자 수용 여부
• OpenAPI 스키마/예제, e2e/컨트랙트 테스트 업데이트

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between d8408f6 and 352e861.

📒 Files selected for processing (1)

• eeos/src/main/java/com/blackcompany/eeos/calendar/application/dto/CalendarResponse.java (2 hunks)

🔇 Additional comments (1)

eeos/src/main/java/com/blackcompany/eeos/calendar/application/dto/CalendarResponse.java (1)

6-6: Locale.ROOT 사용 적절 — 터키어 i 문제 예방 OK

소문자 변환 시 Locale.ROOT를 사용해 로케일 의존 이슈를 회피한 점 좋습니다.