작업 보고서 서비스와 컨트롤러를 새로운 에러 핸들링 및
로깅 시스템으로 업그레이드하여 코드 품질 및 유지보수성 향상
주요 변경사항:
services/dailyWorkReportService.js:
- 새로운 커스텀 에러 클래스 적용
* ValidationError: 유효성 검증 실패
* NotFoundError: 리소스를 찾을 수 없음
* DatabaseError: 데이터베이스 오류
- console.log → logger 유틸리티로 전환
* 구조화된 로깅 (context 포함)
* 로그 레벨 분리 (info, warn, error)
* 파일 로깅 지원
- 상세한 에러 컨텍스트 제공
* 필수 필드, 받은 값, 유효 범위 등
* 디버깅 및 문제 해결 용이성 향상
controllers/dailyWorkReportController.js:
- 새로운 에러 클래스 import
- asyncHandler 미들웨어 통일
- createDailyWorkReport 함수 간소화
* try-catch 제거 (asyncHandler가 처리)
* 표준 JSON 응답 포맷 사용
개선 효과:
- 에러 메시지 명확성 향상
- 로그 분석 및 모니터링 용이
- 일관된 에러 처리 패턴
- 테스트 가능성 향상
- 프로덕션 환경 파일 로깅 지원
파일 통계:
- 2개 파일 수정
- +115 -65 (net +50 lines)
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
- monthly_worker_status 조회 시 GROUP BY로 중복 데이터 합산
- 작업보고서 삭제 권한을 그룹장 이상으로 제한 (admin, system, group_leader)
- 중복 데이터 정리를 위한 마이그레이션 SQL 추가 (009_fix_duplicate_monthly_status.sql)
- synology_deployment 버전에도 동일 수정 적용
✨ 새로운 기능:
- 작업 분석 페이지 구현 (기간별, 프로젝트별, 작업자별, 오류별)
- 개별 분석 실행 버튼으로 API 부하 최적화
- 연차/휴무 집계 방식 개선 (주말 제외, 작업내용 통합)
- 프로젝트 관리 시스템 (활성화/비활성화)
- 작업자 관리 시스템 (CRUD 기능)
- 코드 관리 시스템 (작업유형, 작업상태, 오류유형)
🎨 UI/UX 개선:
- 기간별 작업 현황을 테이블 형태로 변경
- 작업자별 rowspan 그룹화로 가독성 향상
- 연차/휴무 프로젝트 하단 배치 및 시각적 구분
- 기간 확정 시스템으로 사용자 경험 개선
- 반응형 디자인 적용
🔧 기술적 개선:
- Rate Limiting 제거 (내부 시스템 최적화)
- 주말 연차/휴무 자동 제외 로직
- 작업공수 계산 정확도 향상
- 데이터베이스 마이그레이션 추가
- API 엔드포인트 확장 및 최적화
🐛 버그 수정:
- projectSelect 요소 참조 오류 해결
- 차트 높이 무한 증가 문제 해결
- 날짜 표시 형식 단순화
- 작업보고서 저장 validation 오류 수정
- 새로운 에러 처리 시스템 적용:
* asyncHandler로 비동기 함수 래핑
* ApiError 클래스로 유효성 검사 에러 처리
* handleDatabaseError로 DB 에러 표준화
- 새로운 응답 포맷터 적용:
* res.created() - 작업보고서 생성 성공 응답
* res.success() - 기여자별 요약 조회 응답
- 유효성 검사 시스템 적용:
* validateSchema로 스키마 기반 검증
* schemas.createDailyWorkReport 스키마 사용
- 함수별 개선사항:
* createDailyWorkReport: 스키마 검증 및 생성 응답 포맷터
* getContributorsSummary: Promise 래핑 및 성공 응답 포맷터
- 참고: 파일이 크므로 핵심 함수들만 우선 적용
추후 나머지 함수들도 점진적으로 리팩토링 예정
- npm audit fix로 tar-fs, brace-expansion 취약점 해결
- analysisRoutes.js에서 authMiddleware import 오류 수정
- dailyWorkReportController.js에 누락된 getAccumulatedReports 함수 추가
- JWT 인증 시스템 정상 작동 확인 완료
- dailyWorkReportController의 생성 로직을 Service와 Model로 분리
- Service: 데이터 유효성 검사 등 비즈니스 로직 담당
- Model: 트랜잭션을 사용한 DB 쿼리 담당
- Controller: HTTP 요청/응답 처리만 담당하도록 단순화
