Repository navigation
release/1.0.2 - #33
Conversation
PR #28 리뷰 반영: 티켓 일일 충전 설명의 '지연 지급'을 '최초 요청 시점에 충전'으로 수정해 lazy 충전 의미를 명확히 함.
POST /v1/player/init의 생성·로드 겸용과 직업 불일치 409 모호함을 GET /v1/player(로드) + POST /v1/player(생성)로 분리하는 재설계 스펙.
플레이어 엔티티는 있으나 resource/stage/session이 누락된 데이터 정합성 위반에서 NotFoundException(404) 대신 InvalidOperationException(500)을 던진다. 404는 클라이언트가 POST /v1/player 생성을 시도하게 만들어 409와 교착 상태에 빠뜨린다. (PR #29 리뷰 반영)
[Required]만으로는 미정의 정수 값(예: 99)이 모델 검증을 통과해 도메인 영역으로 유입될 수 있어, [EnumDataType(typeof(JobType))]으로 정의된 JobType 값만 허용하도록 보강.
…player-load-create refactor: 플레이어 init을 GET 로드 + POST 생성으로 분리
…endency-updates chore: 의존성 업데이트 및 SQLite 취약점 패치
EF Core에서 AddAsync는 HiLo 등 특수 비동기 값 생성기에만 필요하며, 일반적인 경우 동기 Add 사용이 권장된다. 최근 Dungeon 리포지토리 방향과 일치. Refs: PR #31 review (gemini-code-assist)
…utorial-domain feat: Tutorial 도메인 추가 (완료 처리·목록 조회 API)
…eon-type-progress feat: 던전 타입별 진행도 및 최고 기록 추적 기능 추가
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request introduces several major architectural improvements and new features. It refactors the player initialization flow by splitting the combined player initialization endpoint into separate GET (load) and POST (create) endpoints, backed by new dedicated services. It also introduces a new Tutorial domain to track and persist player tutorial completion status, and decouples dungeon progress from the main player stage by introducing a dedicated PlayerDungeonProgress entity to track independent highest cleared stages and high scores. Additionally, tracking is enabled on several repositories by removing AsNoTracking, production Swagger basic authentication is removed, and comprehensive unit tests and documentation are added. No review comments were provided for evaluation.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
PR #33 병합 충돌 해결. main 핫픽스(IRandomProvider 주입 + 던전 보상 트랜잭션)와 release/1.0.2의 던전 진행도 기능(IPlayerDungeonProgressRepository)이 던전 서비스 생성자에서 충돌. - WeaponDungeonService: 두 의존성 모두 주입, 스크롤 재화 갱신을 진행도 저장과 함께 단일 트랜잭션 내부에서 처리(release 버전 채택 — main의 트랜잭션 의도 포함) - WeaponDungeonServiceTests/BossDungeonServiceTests: progressRepo·randomProvider 파라미터 병합, 생성자 호출 인자 순서를 프로덕션 시그니처에 맞춤
1 participant
수정내역
비고
빌드 성공, 테스트 187개 통과.
이 PR이 main에 머지되면 prod CD가 v1.0.2 태그를 생성하고 배포합니다.