fromis_9/docs/refactoring.md

Backend Refactoring Plan

백엔드 코드 품질 개선을 위한 리팩토링 계획서

작업 목록

1단계: 설정 통합 (config 정리) ✅ 완료

• 카테고리 ID 상수 통합 (CATEGORY_IDS)
• 매직 넘버 설정 파일로 이동 (추후)
• JWT 기본값 제거 (추후)

수정된 파일:

• src/config/index.js - CATEGORY_IDS 상수 추가
• src/routes/admin/youtube.js - config에서 import
• src/routes/admin/x.js - config에서 import
• src/routes/schedules/index.js - 하드코딩된 2, 3, 8 → 상수로 변경

---

2단계: N+1 쿼리 최적화 ✅ 완료

• 앨범 목록 조회 시 트랙 한 번에 조회로 변경
• 스케줄 멤버 조회 - 이미 최적화됨 (확인 완료)

수정된 파일:

• src/routes/albums/index.js - GET /api/albums에서 트랙 조회 최적화
  • 변경 전: 앨범마다 트랙 조회 (N+1 쿼리)
  • 변경 후: 모든 트랙을 한 번에 조회 후 JavaScript에서 그룹화 (2 쿼리)

---

3단계: 서비스 레이어 분리 ✅ 완료

• src/services/album.js 생성 - 앨범 관련 비즈니스 로직
• src/services/schedule.js 생성 - 스케줄 관련 비즈니스 로직
• 라우트에서 서비스 호출로 변경

생성된 파일:

• src/services/album.js - getAlbumDetails, getAlbumsWithTracks
• src/services/schedule.js - getMonthlySchedules, getUpcomingSchedules

수정된 파일:

• src/routes/albums/index.js - 서비스 import 및 사용
• src/routes/schedules/index.js - 서비스 import 및 기존 함수 제거 (240줄 감소)

---

4단계: 에러 처리 통일 ✅ 완료

• 에러 응답 유틸리티 생성 (src/utils/error.js)
• reply.status() → reply.code() 통일
• console.error → fastify.log.error 변경 (추후)

생성된 파일:

• src/utils/error.js - sendError, badRequest, unauthorized, notFound, conflict, serverError

수정된 파일:

• src/routes/auth.js - reply.status → reply.code
• src/routes/stats/index.js - reply.status → reply.code
• src/routes/members/index.js - reply.status → reply.code

---

5단계: 중복 코드 제거 ✅ 완료

• 스케줄러 상태 업데이트 로직 통합 (handleSyncResult 함수)
• youtube/index.js 하드코딩된 카테고리 ID → config 사용
• 멤버 이름 매핑 유틸리티 (기존 코드가 충분히 분리됨)
• 이미지 업로드 공통 함수 (기존 코드가 충분히 분리됨)

수정된 파일:

• src/plugins/scheduler.js - handleSyncResult 함수 추출로 중복 제거
• src/services/youtube/index.js - CATEGORY_IDS.YOUTUBE 사용

---

진행 상황

단계	작업	상태
1단계	설정 통합	✅ 완료
2단계	N+1 쿼리 최적화	✅ 완료
3단계	서비스 레이어 분리	✅ 완료
4단계	에러 처리 통일	✅ 완료
5단계	중복 코드 제거	✅ 완료

---

참고사항

• 각 단계별로 테스트 후 다음 단계 진행
• 기존 API 응답 형식은 유지
• 프론트엔드 수정 불필요하도록 진행