Skip to content

Develop#441

Merged
stemmmm merged 98 commits into
mainfrom
develop
Apr 21, 2026
Merged

Develop#441
stemmmm merged 98 commits into
mainfrom
develop

Conversation

@stemmmm
Copy link
Copy Markdown
Contributor

@stemmmm stemmmm commented Apr 21, 2026

Summary by CodeRabbit

릴리스 노트

  • Chores
    • 배포 워크플로우를 직접 푸시 트리거 기반으로 변경하여 효율성 개선
    • 자동화된 테스트 검증 단계 추가로 배포 안정성 강화
    • CI/CD 파이프라인 구성 최적화

참고 자료

노트

stemmmm and others added 30 commits December 12, 2025 12:23
- 인터뷰 질문, 미션, 데이팅 시험 관련 기본 데이터 마이그레이션 추가
- Flyway 설정 로직에 비활성화 처리 조건 추가
- Hibernate DDL 설정 방식 변경 및 스키마 생성 스크립트 설정 추가

#377
# Conflicts:
#	src/main/java/deepple/deepple/common/config/FlywayConfig.java
- k6 테스트 설정 파일 및 API 테스트 스크립트 추가
- 테스트 환경 변수 및 부하 프로파일 설정
- README에 k6 관련 문서 및 실행 방법 추가

#394
- 성능 테스트용 SQL 데이터 생성 스크립트 추가
- 회원, 소개, 좋아요, 매칭, 알림 데이터 포함
- 데이터 검증 및 README 파일 추가

#394
- FCM 제외 테스트 로직 `sendWithoutPush` 추가
- k6 부하 테스트 스크립트 구현
- 테스트용 SQL 데이터 초기화/생성 로직 추가

#394
* [Refac] : When Match Request or Response, Don't Check Primary Contatct Type

* [fix] : Change BaseURL

* [Fix] : Using Enum
* [Refac] : When Match Request or Response, Don't Check Primary Contatct Type

* [fix] : Change BaseURL

* [Fix] : Using Enum

* [Refac] Change BizgoRequestBody
* [Refac] : When Match Request or Response, Don't Check Primary Contatct Type

* [fix] : Change BaseURL

* [Fix] : Using Enum

* [Refac] Change BizgoRequestBody

* [Fix] : Use Record for serialization
* [Refac] : When Match Request or Response, Don't Check Primary Contatct Type

* [fix] : Change BaseURL

* [Fix] : Using Enum

* [Refac] Change BizgoRequestBody

* [Fix] : Use Record for serialization

* [Refac] : Add Logging for checking.
- 데이터 접근 레이어 분리 (Reader/Writer) 및 저장 로직 개선
- FCM 예외 처리 로깅 추가
- 알림 템플릿 정보 전송 방식 변경
- k6 부하 테스트 시 최대 VU 증가
- notification_preferences 테이블에 인덱스 추가
- 테스트 코드 리팩토링 및 명확한 실패 상태 로깅 적용

#394
- CircuitBreaker 상태 전환, 실패율 초과, 느린응답률 초과 이벤트에 대한 로그 등록
- 이벤트 리스너 등록 로직 추가 및 @slf4j 어노테이션 적용
- 알림 테이블에 복합 인덱스 추가 (receiverId, deletedAt, id)
- 데이터베이스 및 엔티티 매핑 수정

#394
- Bizgo API 응답에서 에러 상태 코드 및 본문 로깅 추가
- 예외 발생 시 디버깅 용이하도록 개선

#409
stemmmm and others added 27 commits February 24, 2026 23:00
- 리프레시 토큰 예외 처리 추가 (누락, 만료, 유효하지 않음)
- 토큰 갱신 API 구현 및 토큰 유효성 확인 로직 변경
- AuthService와 관련 테스트 간소화
- TokenPairResponse 및 TokenRefreshResponse 도입
- StatusType에 만료된 액세스 토큰 상태 추가
- 불필요한 토큰 재발급 코드 제거
- 인증 필터에서 리프레시 토큰 처리 로직 삭제

#435
# Conflicts:
#	src/main/java/deepple/deepple/auth/presentation/filter/TokenFilter.java
- 리프레시 토큰 처리 및 재발급 관련 코드 제거
- `TokenFilter`의 불필요한 의존성 삭제
- 인증 로직 단순화 및 불필요한 메서드 제거
- Google Play 클라이언트 및 API 검증 로직 추가
- Google Play 서비스 계정 인증 구성
- Google Play 결제 영수증 검증 및 이벤트 처리 로직 구현
- Google Play 관련 예외, DTO, 설정, 테스트 코드 추가

#438
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 21, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 1f937096-b4c1-40e6-92c0-cc98dc133f2c

📥 Commits

Reviewing files that changed from the base of the PR and between 499274d and 69baabe.

📒 Files selected for processing (2)
  • .github/workflows/cd-dev.yml
  • .github/workflows/ci.yml

Walkthrough

CD 워크플로우(cd-dev.yml)는 트리거 메커니즘을 workflow_run에서 직접 push 이벤트로 변경하고, docs/metadata/env/license 파일만 변경된 경우를 건너뛰는 paths-ignore 필터를 추가했습니다. 새로운 test job을 도입하여 Gradle wrapper 검증, JDK 21 설정, 테스트 실행, 실패 시 artifact 업로드를 수행합니다. deploy job은 test job에 대한 의존성을 추가하고 Docker 이미지 태깅 및 SSM 파라미터를 github.sha를 기반으로 변경했습니다. 한편, CI 워크플로우(ci.yml)는 push 트리거를 main 브랜치로 제한하여 develop 브랜치를 제거했습니다.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch develop

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

Comment @coderabbitai help to get the list of available commands and usage tips.

@stemmmm stemmmm merged commit e53494e into main Apr 21, 2026
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants