refactor : Session url open and Cohort value adding

[BlackBean99]

Summary

• closed 비로그인시 1차 url open #424

Tasks

• 해당 PR에 포함된 작업을 작성해주세요.
• 해당 PR에 포함된 작업을 작성해주세요.

ETC

Screenshot

Summary by CodeRabbit

릴리스 노트

• 신기능

  • 코호트 응답에 코호트 ID 정보 추가

• 버그 수정

  • 코호트 조회 실패 시 더 정확한 예외 처리 추가

• 보안

  • 세션 조회 엔드포인트를 공개 접근 가능으로 변경
  • 코호트 최신 정보 엔드포인트 공개 접근 허용

[coderabbitai]

Caution

Review failed

Pull request was closed or merged during review

개요

비로그인 사용자를 위해 코호트 및 세션 관련 엔드포인트를 공개 접근으로 변경하고, 코호트 응답에 cohortId를 추가하며, 서비스 계층에서 코호트 조회 로직을 통합하는 리팩토링을 진행합니다.

변경 사항

변경 사항 / 파일	요약
코호트 서비스 계층 application/src/main/kotlin/core/application/cohort/application/service/CohortQueryService.kt	새로운 getLatestCohort() 메서드를 추가하여 코호트 조회 로직을 캐싱합니다. getLatestCohortId()와 getLatestCohortValue()를 리팩토링하여 이 새로운 헬퍼 메서드를 사용하도록 변경하고, getCohort(value) 메서드에서 null 반환 대신 예외를 발생시키도록 개선합니다.
코호트 API 계층 application/src/main/kotlin/core/application/cohort/presentation/controller/CohortController.kt, application/src/main/kotlin/core/application/cohort/presentation/response/CohortNumberResponse.kt	latestCohort 엔드포인트에 @PreAuthorize("permitAll()") 주석을 추가하여 공개 접근을 허용합니다. 응답 데이터 클래스에 cohortId: Long 속성을 추가하고, 컨트롤러에서 getLatestCohort()를 호출하여 새로운 응답 구조를 매핑합니다.
세션 보안 설정 application/src/main/kotlin/core/application/session/presentation/controller/SessionQueryController.kt	getNextSession() 및 getAllSessions() 엔드포인트의 보안 주석을 @PreAuthorize("hasAuthority('read:session')")에서 @PreAuthorize("permitAll()")로 변경하여 인증 없이 접근 가능하도록 설정합니다.

추정 코드 리뷰 노력

🎯 3 (Moderate) | ⏱️ ~25 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 3

❌ Failed checks (2 warnings, 1 inconclusive)

Check name	Status	Explanation	Resolution
Out of Scope Changes check	⚠️ Warning	CohortNumberResponse의 cohortId 추가, CohortQueryService의 새 메서드 도입, 세션/코호트 엔드포인트 보안 변경이 이슈 #424의 명시적 범위를 벗어남. 세션 URL 개방의 목표와만 부분적으로 연관.	CohortQueryService 리팩토링과 CohortNumberResponse API 변경이 이슈 #424의 범위와 무관해 보임. 이러한 변경사항의 필요성을 이슈에서 명확히 하거나 별도 이슈로 분리하세요.
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.
Linked Issues check	❓ Inconclusive	링크된 이슈 #424는 '비로그인시 1차 url open' 목표만 제시하며 기술적 세부사항이 부족. 변경사항(세션/코호트 엔드포인트 보안 변경, 응답 구조 수정)이 이슈의 광범위한 의도와 일치하지만, 구체적 요구사항 부재로 완전한 검증 불가.	이슈 #424에 구체적 요구사항, 영향받는 엔드포인트, 보안 정책 변경사항을 명시하여 명확한 구현 기준을 제공하세요.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	PR 제목이 변경사항의 주요 내용과 부분적으로 관련됨. 'Session url open'은 세션 엔드포인트의 보안 제한 제거를 반영하고, 'Cohort value adding'은 cohortId 추가를 반영하지만, 실제 주요 변경사항(응답 객체의 API 변경, 보안 주석 변경)을 명확하게 설명하지 못함.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch refactor/#424

📝 Coding Plan

• Generate coding plan for human review comments

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

Tip

CodeRabbit can approve the review once all CodeRabbit's comments are resolved.

Enable the reviews.request_changes_workflow setting to automatically approve the review once all CodeRabbit's comments are resolved.