[Release] v1.0.0 - #118
Merged
Merged
[Release] v1.0.0#118
Conversation
- GET /api/v1/home: 사용자 정보 + 주간 스트립 + 오늘 부글/생활 기록 + 연속 기록일수(streak) 조회 - weeklyPattern은 리포트/가이드 도메인 산출 데이터에 의존 — 소스 확정 전까지 null 반환 - StubAuthGuard / @currentuser() / PrismaService 유닛테스트 모킹 설정 포함 (feat/#16/calendar와 동일 내용 — develop에서 독립 브랜치로 분기해 공통 기반 중복 추가, 두 브랜치 모두 develop에 머지되면 내용이 같아 충돌 없이 정리됨)
- 캘린더 날짜 경계/키를 UTC → KST(Asia/Seoul) 기준으로 수정 (새벽 0~9시 기록이 전날로 잘못 분류되던 버그) - StubAuthGuard: 잘못된 x-user-id 헤더가 500 대신 400을 반환하도록 방어 - CurrentUser 데코레이터: request.user 없을 때 401을 던지도록 방어 - takenTime 필드 의미(1/2/3) 주석 추가 - 컨트롤러 예외 전파 테스트 추가 - docs 코드블록 언어 태그(http) 추가 (markdownlint)
feat/#16/calendar에서 CodeRabbit이 지적한 문제들이 동일 패턴으로 복사돼 있어 미리 반영함 (PR 리뷰 전 선조치). - 홈 날짜 경계/키를 UTC → KST(Asia/Seoul) 기준으로 수정 (새벽 0~9시 기록이 전날로 잘못 분류되던 버그, weekStrip 요일 계산 포함) - StubAuthGuard: 잘못된 x-user-id 헤더가 500 대신 400을 반환하도록 방어 - CurrentUser 데코레이터: request.user 없을 때 401을 던지도록 방어 - 컨트롤러 예외 전파 테스트 추가
- StubAuthGuard: NODE_ENV=production이면 무조건 실패(fail-closed)하도록 방어 (실서비스 라우트에 임시 인증이 그대로 노출되는 위험 차단) + 유닛테스트 추가 - home.service: NotFoundException → BusinessException(HomeErrorCode) 사용으로 통일 - home.service: 오늘/이번 주/streak가 각각 하던 boogleRecord 조회(3회)를 겹치는 범위 특성을 이용해 1회 조회 후 메모리에서 파생하도록 통합 - home.service: joinedDays가 0 이하로 계산되지 않도록 최소 1로 방어 - home-query.dto: date를 YYYY-MM-DD 형식으로만 제한 (datetime 형식 거부) + 검증 테스트 추가 - 컨트롤러 라우트 레벨 테스트 추가 (ValidationPipe 400 검증) - docs: weeklyPattern 예시를 실제 동작(null)에 맞게 수정, has_bowl 오타 수정
CodeRabbit 지적사항 중 400은 이전 커밋에서 반영했으나 401(인증 실패) 경로는 누락되어 있었음. 가드 없이 @currentuser()만 적용한 테스트 전용 컨트롤러로 request.user가 없을 때 401을 반환하는지 검증.
- 날짜 범위 조회를 반개방 구간(lt 다음날 시작)으로 통일 (23:59:59.999 고정 소수초로 자르던 방식은 그보다 정밀한 소수초 기록을 누락할 수 있어 제거) - streak 400일 상한을 의도된 제품 계약으로 문서화 (무제한 페이지 조회는 초기 단계에 과함 — 필요해지면 상수 조정/페이지 조회로 교체) - stub-auth.guard.spec: NODE_ENV가 원래 undefined였을 때 문자열 "undefined"로 남는 복원 버그 수정 - docs: 코드펜스 앞뒤 빈 줄 추가(MD031), 404/MEMBER_NOT_FOUND를 공통 에러 표에 추가
- GET /api/v1/notifications: 알림 목록 + 안읽음 개수(unreadCount) 조회 (alarm_map ⨝ alarm, N101~N105 전부 이 API 하나로 커버) - category(W/R/P) → linkTo(GUIDE_WARNING/HOME/REPORT) 코드 매핑 - alarm_id가 null인 행(원본 알람 누락)은 목록에서 방어적으로 제외 - StubAuthGuard / @currentuser() / PrismaService 유닛테스트 모킹 포함 (공통 기반) - 알림 API 명세 + 생성(트리거) 규칙 문서 추가 (docs/api/notification-api.md)
- notification.service: 목록 조회에 상한(take=100) 추가, unreadCount는 별도 count 쿼리로 정확히 계산(목록 상한과 무관하게 뱃지 정확도 유지) - notification.controller.spec: getHttpServer() App 타입 좁혀서 no-unsafe-argument 경고 제거 (INestApplication<SupertestApp>) - notification.service.spec: regDate가 null인 행 제외 테스트, take/count 쿼리 호출 검증 테스트 추가
- PR #8 병합으로 develop에 확정된 schema.prisma를 기준으로 충돌 해결
- feat/#11/auth-user 머지로 JWT 기반 인증(AuthService, JwtAuthGuard, CurrentUser) 도입
- LifeRecordController/FoodController: 임시 x-user-id 헤더 인증 제거,
- 각 엔드포인트에 실제 에러코드 기준 응답 문서(@ApiOkResponse 등) 및 사용법 설명(@apioperation description) 추가
- package.json: postinstall에 prisma generate 추가 (생성 클라이언트 부재로 인한 CI lint/typecheck 오류 방지)
[Feat] 캘린더 API 구현
Feat: 알림 API 구현
[Feat] 홈 API 구현
Gitkeep 정리 및 servie 파일의 type들을 DTO 파일로 리펙토링
변경된 캘린더 API PR의 전역 Express Request Type을 적용하였음. Branch 분기가 나누어져 있어 최신 Origin 브랜치의 전역 설절 파일만 가져와 사용하였음. Merge시 Diff로 잡히지 않아 충돌 발생 가능성이 낮다고 예상됨
홈에 이어 나머지 담당 API도 Swagger 문서화를 완료한다. 기존에는 엔드포인트만 노출되고 설명·응답 스키마가 없어 프론트가 응답 형태를 확인할 수 없었다. - 응답 DTO를 interface → class로 전환하고 @ApiProperty로 필드 문서화 (calendar, calendar-daily, notification, push-token, notification-settings) - 각 컨트롤러에 @apitags·@apioperation·ApiSuccessResponse·에러 응답 추가 - 쿼리 DTO(year/month/date)에 @ApiProperty 추가 - notification-response.dto의 런타임 상수·헬퍼(NOTIFICATION_TYPES, toNotificationType, NOTIFICATION_LINK_TO)는 그대로 유지 Swagger 문서 생성 결과 10개 엔드포인트 전부 태그·스키마·에러 코드 노출 확인. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
[Docs] 홈·캘린더·알림·푸시 API Swagger 스키마 정리
[Docs] API Swagger 문서 보강
…wagger [Docs] 인증 및 계정 관리 API Swagger 문서 보강
프론트 푸시 수신 확인용. 알림 5종 중 4종은 배치(cron)나 이벤트 트리거라
임의 시점에 확인할 수 없어, 로그인 사용자 본인에게 즉시 발송하는 수단을 만든다.
- POST /api/v1/notifications/test { type } — 5종 중 하나를 본인에게 발송
- 실제 발송 경로(NotificationDispatchService.dispatch)를 그대로 사용해
인앱 알림 생성·설정 게이트·FCM payload가 운영과 동일하다(트리거만 수동).
- 템플릿 필수 파라미터가 있는 유형(WARNING·STREAK)은 테스트용 기본값을 채운다.
- 운영에서 아무나 호출하지 못하도록 NOTIFICATION_TEST_ENABLED=true인 환경에서만
동작하고, 그 외에는 403(NOTIFICATION_TEST_DISABLED)을 반환한다.
- 남에게는 보낼 수 없다(대상은 항상 JWT의 본인).
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
월간 리포트 조회시 사용자 유형 코드와 조건, 설명을 변경하였다.
리포트 조회시 날짜 검증, 유형 조건 반올림 문제, 미래 시간 요청 문제를 해결하였다.
[Fix]: 월간 리포트 사용자 유형 코드 및 조건, 설명 변경
CodeRabbit 리뷰 반영: dispatch는 설정이 N이거나 발송이 실패해도 예외 없이
끝나는데, 테스트 API가 무조건 sent:true를 반환해 "보냈다는데 안 온다"는
혼동을 줄 수 있었다. 수신 확인용 API라 결과가 정확해야 한다.
- NotificationDispatchService.dispatch가 DispatchResult
({ notificationId, pushSent })를 반환하도록 계약 변경
· 설정 N으로 건너뜀 / 발송 실패 → pushSent:false
- 테스트 발송 응답을 { type, notificationId, pushSent }로 변경하고
Swagger 설명에 pushSent=false의 의미(설정 N·토큰 없음·발송 실패) 명시
- 반환 계약 검증 테스트 추가
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
[Feat] 알림 테스트 발송 API 추가
홈의 USER_TYPE_LABEL이 리포트가 저장하는 코드 체계와 달라, 유형이 있는 사용자에게 전혀 다른 유형명이 표시되고 있었다. 기준: prisma/schema.prisma의 user_type 주석 및 report.service의 월간 유형 산출 R 규칙형 / C 변비경향형 / W 묽은변경향형 / L 생활영향형 / I 불규칙형 / N 유형 분석 중 - W가 매핑에 없어 묽은변경향형이 라벨 없이 null로 내려갔다. - L(생활영향형)을 '묽은변경향형', I(불규칙형)을 '생활영향형'으로 잘못 표시했다. - 존재하지 않는 코드 U를 '불규칙형'으로 두고 있었다. - N의 라벨도 '기록부족형' → '유형 분석 중'으로 통일했다. 6개 코드 전부와 계약 밖 코드(null 폴백) 테스트를 추가해 재발을 막는다. home-response.dto의 Swagger 설명도 동일하게 정정. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
[Fix] 홈 월간 유형 라벨이 실제 코드와 어긋나던 문제 수정
주간/월간 리포트 조회 시 캐싱된 기록 먼저 조회 후 기록이 없으면 boogle_record, life_record에서 하나씩 조회하여 기록 저장 구현
weekly_record, monthly_record 데이터의 정합성을 위해 부글 기록/생활 기록 수정/삭제/복구 시 캐시된 스냅샷을 삭제하고 리포트 조회 시 캐시 미스매칭으로 인한 재생성 구현
스냅샷 저장이 실패해도 오류 반환이 아닌 원본 기록을 반환한다. 캐시 히트/기록 부족 관련 테스트를 추가하였다.
[Feat] swagger 응답 추가
record 모듈의 응답 시 타입 고정되는 문제 수정
[feat] 주간/월간 리포트 조회 시 기록 스냅샷 저장, 부글/생활 기록 수정/삭제/복구 시 캐시된 스냅샷 삭제
|
Important Review skippedToo many files! This PR contains 244 files, which is 144 over the limit of 100. To get a review, reduce the PR to 100 files or fewer by splitting it into smaller PRs or changing its base branch. Upgrade to a paid plan to raise the limit. This review couldn't start because sufficient usage credits or metered capacity aren't available. Add credits or update usage-based reviews in the billing tab, then retry. ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (19)
📒 Files selected for processing (244)
You can disable this status message by setting the 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. Comment |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📑 이슈 번호
✨️ 작업 내용
작업 내용을 간략히 설명해주세요.
💭 코멘트
코드 리뷰가 필요한 부분이나 궁금한 점을 자유롭게 남겨주세요!
📸 구현 결과
구현한 기능이 모두 결과물에 포함되도록 자유롭게 첨부해주세요.