코드 품질 개선 리팩토링

수용된 개선사항

1. setTimeout(300) 제거

• 문제: 네트워크 응답과 무관한 인위적 300ms 지연으로 UX 저하
• 해결: 불필요한 지연 제거
• 효과: 페이지 로딩 0.3초 단축

2. Non-null assertion(!) 제거

• 문제: mypage.ts에서 DB 작업 실패 시 안전하지 않은 return bookmark!; 패턴
• 해결: 명시적 에러 처리로 변경
• 효과: 런타임 오류 방지

3. 테스트 코드 중복 제거

• 문제: guards.test.ts에서 walk 함수 3회 중복 정의
• 해결: 파일 상단 헬퍼로 추출
• 효과: 테스트 코드 유지보수성 향상

불수용된 제안

• 커스텀 훅 분리: 현재 컴포넌트 150줄 미만으로 과잉 설계
• Repository 패턴: 함수 10개 미만으로 추상화 이점 적음
• 전략 패턴: 현재 switch 문이 명확하고 확장 빈도 낮음