[FIX] 분석 재요청 차단 + AI 멘토 질문 제한 로직 제거

[p1001q]

📍 개요

동일 playing 재분석 차단 로직 제거 + AI 멘토 질문 횟수 제한 임시 해제

⛓️‍💥 관련 이슈

• Closes 🐛 BugFix - 분석 재요청 차단 + AI 멘토 질문 횟수 제한 로직 제거 #183

---

🛠️ 작업 내용

• AnalysisService.createAnalysis에서 기존 분석 존재 여부로 새 요청을 막던 검증 로직 제거 (playing:analysis는 1:N이라 재분석 가능해야 함)
• MentorChatSession.startGenerating의 questionCount >= 3 제한 체크 제거 (시연 목적 임시 조치)
• 위 변경에 맞춰 관련 테스트(AnalysisServiceTest, MentorQuestionServiceTest) 정리

---

🔥 리뷰 요청 사항

리뷰어가 중점적으로 확인해주었으면 하는 내용을 작성해주세요.

• 최소 수정 범위로 진행해서 existsByPlayingIdAndStatusIn, ANALYSIS_ALREADY_IN_PROGRESS, MENTOR_QUESTION_LIMIT_EXCEEDED 등 이제 호출부가 없어진 코드는 일부러 남겨뒀습니다(후속 정리 예정) — 이대로 머지해도 괜찮을지 확인 부탁드려요.
• AI 멘토 질문 제한 해제는 시연용 임시 조치라, 시연 이후 플랜별 제한 정책이 확정되면 별도 이슈로 재작업이 필요합니다.

---

✅ 체크리스트

• 코드 컨벤션을 준수했습니다.
• 불필요한 코드 및 import를 제거했습니다. (import는 정리했으나, 위 리뷰 요청 사항의 미사용 코드는 의도적으로 보류)
• 예외 처리를 적용했습니다.
• 테스트를 완료했습니다.
• 관련 Issue를 연결했습니다.

---

📎 참고 사항

• 로컬에서 ./gradlew compileJava compileTestJava 통과, 전체 테스트 437/437 통과 확인

Summary by CodeRabbit

• 변경 사항
  • 동일한 연습에 대해 기존 분석 상태와 관계없이 분석 요청을 생성할 수 있습니다.
  • 멘토 세션 생성 시 질문 수 3개 제한이 제거되었습니다.
  • 변경된 분석 생성 및 멘토 질문 처리 동작에 맞춰 관련 테스트를 정리했습니다.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 9d330557-1a33-43f6-8e23-589b9a9f533e

📥 Commits

Reviewing files that changed from the base of the PR and between 40ecf2d and 424ff7e.

📒 Files selected for processing (4)

• src/main/java/com/mr/domain/analysis/service/AnalysisService.java
• src/main/java/com/mr/domain/mentor/entity/MentorChatSession.java
• src/test/java/com/mr/domain/analysis/service/AnalysisServiceTest.java
• src/test/java/com/mr/domain/mentor/service/MentorQuestionServiceTest.java

💤 Files with no reviewable changes (4)

• src/main/java/com/mr/domain/mentor/entity/MentorChatSession.java
• src/test/java/com/mr/domain/mentor/service/MentorQuestionServiceTest.java
• src/test/java/com/mr/domain/analysis/service/AnalysisServiceTest.java
• src/main/java/com/mr/domain/analysis/service/AnalysisService.java

---

📝 Walkthrough

Walkthrough

분석 생성 시 기존 분석 상태를 확인하지 않도록 변경했습니다. 멘토 세션의 질문 수 제한을 제거했습니다. 두 변경에 맞춰 관련 테스트의 스텁, 검증, 제한 초과 테스트를 삭제했습니다.

Changes

분석 및 멘토 제한 정책 변경

Layer / File(s)	Summary
분석 재요청 정책 변경 src/main/java/com/mr/domain/analysis/service/AnalysisService.java, src/test/java/com/mr/domain/analysis/service/AnalysisServiceTest.java	분석 생성 시 PENDING, PROCESSING, COMPLETED 상태의 기존 분석을 확인하는 검사를 제거했습니다. 성공 테스트에서 관련 스텁과 호출 검증을 삭제했습니다.
멘토 질문 제한 해제 src/main/java/com/mr/domain/mentor/entity/MentorChatSession.java, src/test/java/com/mr/domain/mentor/service/MentorQuestionServiceTest.java	startGenerating의 질문 수 3회 제한을 제거했습니다. 질문 한도 초과를 검증하던 테스트를 삭제했습니다.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

• Musereview/BE#84: AnalysisService의 분석 중복 검사와 생성 테스트 변경이 연결됩니다.
• Musereview/BE#119: MentorChatSession의 멘토 질문 횟수 제한 변경이 연결됩니다.

Poem

분석은 다시 흐르고
질문의 벽은 잠시 내려가네
대기 상태도, 완료 상태도
새 요청을 막지 않네
멘토의 토큰, 계속 발급되네 ✨

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	제목이 분석 재요청 차단 제거와 AI 멘토 질문 제한 제거라는 두 가지 주요 변경을 정확히 요약합니다.
Linked Issues check	✅ Passed	분석 재요청 차단 제거와 AI 멘토 질문 제한 임시 해제를 구현했으며, 관련 테스트도 요구사항에 맞게 정리했습니다.
Out of Scope Changes check	✅ Passed	변경 사항은 연결된 이슈의 구현 및 관련 테스트 정리 범위에 포함되며, 별도의 범위 외 변경은 확인되지 않습니다.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ 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 fix/#183-remove-limit-checks

---

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.}

[ownue]

필요한 부분만 최소한으로 잘 수정하셨네요!! 수고하셨습니다 🫰🫰😍😁😁

[rkdehdrbs7885-oss]

수고하셨습니다!