[FEAT] 피드백 타이머 초기화 시 0분 0초로 변경

[jaeml06]

🚩 연관 이슈

closed #398

📝 작업 내용

피드백 타이머 초기화 시 0분 0초로 변경

2025-11-13.2.19.36.mov

FeedbackTimerPage에서 선언되어있던 불필요한 useEffect제거

기존 defaultTime과 time의 초기 세팅을 하는 useEffect를 useFeedbackTimer에서 초기상태로 변경하였습니다.

🏞️ 스크린샷 (선택)

없음

🗣️ 리뷰 요구사항 (선택)

없음

Summary by CodeRabbit

릴리스 노트

• Refactor
  • 타이머 초기화 로직을 내부적으로 개선하여 코드 구조를 간소화했습니다.
  • 타이머 리셋 동작을 고정된 초기값 기반으로 통일하여 예측 가능성을 높였습니다.

[coderabbitai]

Walkthrough

피드백 타이머의 초기화 로직을 페이지 수준에서 훅 수준으로 이동했습니다. 페이지의 마운트 시점에 타이머 상태를 설정하는 useEffect를 제거하고, 훅 내부에서 INITIAL_TIME을 0으로 고정하여 초기화 및 리셋 동작을 일관되게 처리합니다.

Changes

Cohort / File(s)	Change Summary
타이머 페이지 정리 src/page/TimerPage/FeedbackTimerPage.tsx	마운트 시 타이머 상태를 초기화하는 useEffect 제거, INITIAL_TIME 상수 제거, setTimer/setDefaultTimer 추출 제거
훅 내부 초기화 로직 강화 src/page/TimerPage/hooks/useFeedbackTimer.ts	INITIAL_TIME = 0 상수 추가, 초기 타이머 상태를 null에서 INITIAL_TIME으로 변경, resetTimer가 defaultTimer 대신 INITIAL_TIME으로 리셋하도록 변경, resetTimer 의존성을 [defaultTimer]에서 []로 변경

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

• 주의 사항:
  • resetTimer 함수의 의존성 제거([])로 인한 의도적 동작 변경 확인 필요
  • 기존 코드에서 defaultTimer에 의존하던 리셋 동작이 고정값으로 변경되었으므로 사이드 이펙트 검증 필요
  • 페이지 수준 초기화 제거가 다른 타이머 페이지 컴포넌트에 영향을 미치지 않는지 확인

Possibly related PRs

• [FEAT] 피드백 타이머 구현 #361: FeedbackTimerPage와 useFeedbackTimer 구현을 모두 수정하므로 직접 연관 (페이지 레벨 초기화 제거 및 훅 내부 로직 재구성)
• [FIX] 버그 수정 및 사소한 디자인 개선 #385: useFeedbackTimer.ts의 adjustTime 로직을 수정하므로 같은 훅 모듈에 영향을 미침

Suggested labels

feat

Suggested reviewers

• useon
• i-meant-to-be

Poem

🐰 타이머가 춤을 춤, 영 0에서
초기화의 마법이 떨어진다네,
훅이 품으니 정갈하고 깔끔하게 ✨
페이지는 한발 물러나 휴식을 취하고
피드백의 시간, 이제 온전해졌네 ⏱️

Pre-merge checks and finishing touches

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	PR 제목은 피드백 타이머 초기화 시 0분 0초로 변경하는 주요 변경사항을 명확하게 요약하고 있으며, 코드 변경 내용과 직접적으로 일치합니다.
Linked Issues check	✅ Passed	PR 변경사항이 연관 이슈 #398의 요구사항을 충족합니다. 피드백 타이머 초기화 시 0분 0초로 설정하도록 변경했으며, 초기화 로직을 useFeedbackTimer 훅으로 중앙화했습니다.
Out of Scope Changes check	✅ Passed	모든 변경사항이 이슈 #398의 요구사항 범위 내에 있습니다. FeedbackTimerPage와 useFeedbackTimer 모두 피드백 타이머 초기화 로직 개선에만 집중하고 있습니다.
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.

✨ 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 feat/#398

---

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

[useon]

확인했습니다 ~!! 감사해요 치코 !!!