Skip to content

feat(session): 세션 조회 API 구현 - #115

Merged
exijn merged 2 commits into
developfrom
feat/110-session-query
Sep 28, 2026
Merged

exijn merged 2 commits into
developfrom
feat/110-session-query

Conversation

@exijn

@exijn exijn commented Sep 28, 2026

Copy link
Copy Markdown
Member

배경

현재 Session 생성·복구·token 검증·inactivity expiration 기반은 존재하지만 Session 조회 API와 mode/mission read state가 구현되어 있지 않다.

Closes #110

변경 내용

  • GET /api/v1/sessions/{sessionId} 구현
  • 기존 SessionAccessVerifier 재사용
  • 성공한 token 인증 요청의 touch 결과를 응답에 반영
  • ExperienceMode(MISSION, FREE)와 nullable missionId 추가
  • SessionStatus를 expiredAt에서 read-time 파생
  • 기존 poppy.session.inactivity-timeout 기반 expiresAt 계산
  • V19 nullable migration 추가
  • mode 선택 전 Session은 mode = null, missionId = null 반환
  • API Response DTO와 integration test 추가

mode 변경 write API와 Simulation Pass invalidation 정책은 이번 Issue 범위에서 제외했다.

검증

  • ./gradlew.bat test — 329 tests passed
  • ./gradlew.bat build — passed
  • 신규 테스트 — 정상 조회, nullable state, persisted mode/missionId, 401/404/410, configured expiration deadline

로컬 PostgreSQL Testcontainers의 client limit 때문에 검증 시 SPRING_DATASOURCE_HIKARI_MAXIMUM_POOL_SIZE=7을 실행 환경 변수로만 지정했다. repository 설정에는 반영하지 않았다.

@github-actions github-actions Bot added domain: session session 도메인 변경 test 테스트 관련 작업 labels Sep 28, 2026

@inryeok-bot inryeok-bot Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

리뷰 결과

변경된 12개 파일을 검토했으며, 수정이 필요한 문제 1개를 발견했습니다.

심각도 개수
Critical 0
High 1
Medium 0
Low 0

주요 내용

  • HIGH · TRANSACTION — 읽기 전용 트랜잭션 안에서 세션 활동 시각을 갱신합니다

검토한 head: 2d7889d5fa43

@inryeok-bot inryeok-bot Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

리뷰 결과

변경된 12개 파일을 검토했으며, 수정이 필요한 문제를 찾지 못했습니다.

심각도 개수
Critical 0
High 0
Medium 0
Low 0

완료

변경 사항을 검토했으며 게시할 인라인 코멘트가 없습니다.

검토한 head: 99ff06489c3e

@exijn
exijn merged commit 3951e51 into develop Sep 28, 2026
3 checks passed
@exijn
exijn deleted the feat/110-session-query branch September 28, 2026 09:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

domain: session session 도메인 변경 test 테스트 관련 작업

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant