[hotfix] #118 - 캘린더 connectedAt을 사용자 zoneId 기준으로 변경

[Jy000n]

관련 이슈 🛠

• closes [hotfix] 시간 관련 응답 내려줄 때 zoneId 적용 #118

작업 내용 요약 ✏️

캘린더 연동 응답의 connectedAt을 UTC 고정 표시에서 사용자의 시간대(User.zoneId) 기준 문자열로 변경합니다. DB 저장은 기존과 동일하게 UTC로 유지하고, 응답 시점에만 사용자 기준으로 변환합니다.

주요 변경 사항 🛠️

• [Calendar] CalendarConnectResponse.connectedAt을 LocalDateTime(UTC 고정, @JsonFormat)에서 String(사용자 zoneId 기준 변환 문자열)으로 변경
• [Calendar] CalendarConnectionCommandService에 resolveConnectedAt private 메서드 추가하여 변환 처리

트러블 슈팅 ⚽️

테스트 결과 📄

캘린더 연동 시 connectedAt 시각 현재 노트북 시각과 일치하는지 확인

스크린샷 📷

캘린더 연동 시 connectedAt 시각 현재 노트북 시각과 일치하는지 확인

리뷰 요구사항 📢

📎 참고 자료 (선택)

Summary by CodeRabbit

• 개선 사항
  • 캘린더 연동 시각이 사용자의 시간대 기준으로 변환되어 표시됩니다.
  • 연동 시각 응답 형식이 날짜·시간 문자열로 제공되며, 표시 형식과 시간대 정보가 명확해졌습니다.

[coderabbitai]

Walkthrough

캘린더 연결 응답의 connectedAt 타입을 문자열로 변경하고, 저장된 UTC 시각을 사용자 ZoneId 기준으로 변환해 yyyy-MM-dd HH:mm:ss 형식으로 반환합니다.

Changes

캘린더 연결 시각 변환

Layer / File(s)	Summary
연결 시각 응답 계약 src/main/java/com/Timo/Timo/domain/calendar/dto/response/CalendarConnectResponse.java	connectedAt을 LocalDateTime에서 String으로 변경하고 사용자 시간대 기준 스키마 설명을 추가합니다.
사용자 시간대 변환 및 응답 연결 src/main/java/com/Timo/Timo/domain/calendar/service/CalendarConnectionCommandService.java	UTC 기준 연결 시각을 사용자 ZoneId로 변환하고 지정 형식의 문자열을 응답에 전달합니다.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

• Team-Timo/Timo-Server#44: CalendarConnectResponse.connectedAt과 캘린더 연결 서비스를 처음 도입한 변경입니다.

Suggested reviewers: laura-jung

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	응답 시 사용자 zoneId를 적용하고 DB 저장은 UTC로 유지한다는 이슈 요구를 충족합니다.
Out of Scope Changes check	✅ Passed	요구사항과 무관한 추가 변경은 보이지 않으며 connectedAt 응답 변환에만 집중합니다.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	핵심 변경인 캘린더 connectedAt의 사용자 zoneId 기준 변환을 정확히 요약한 제목입니다.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch hotfix/#118-calendar-apply-zoneId

---

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

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In
`@src/main/java/com/Timo/Timo/domain/calendar/service/CalendarConnectionCommandService.java`:
- Around line 59-65: Update resolveConnectedAt to handle null connectedAt and
null or invalid user.getZoneId() values without propagating ZoneId conversion or
atZone exceptions, using the method’s established fallback behavior. Extract the
yyyy-MM-dd HH:mm:ss DateTimeFormatter into a class-level constant and reuse it
instead of constructing one per invocation.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: b6187220-c2d5-47d6-83f8-2fd7e00082e6

📥 Commits

Reviewing files that changed from the base of the PR and between d0abee2 and 76fd562.

📒 Files selected for processing (2)

• src/main/java/com/Timo/Timo/domain/calendar/dto/response/CalendarConnectResponse.java
• src/main/java/com/Timo/Timo/domain/calendar/service/CalendarConnectionCommandService.java