-
Notifications
You must be signed in to change notification settings - Fork 0
feat: API-013 회고 상세 조회에 currentUserStatus 필드 추가 #114
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: dev
Are you sure you want to change the base?
Conversation
## 변경 사항 - Phase 3: phase-3-code-fix.md → phase-3-ai-diagnostic.md로 변경 - Phase 4: phase-4-pr-creation.md → phase-4-issue-automation.md로 변경 - Phase 5: phase-5-auto-fix-pr.md 신규 추가 - overview.md: 새 Phase 구조 반영 - 각 문서 간 참조 링크 수정 ## Phase 구조 1. Event Trigger - 이벤트 수신 및 트리거 2. Issue Analysis - 이슈 분석 및 브랜치 생성 3. AI Diagnostic - Claude API 기반 진단 4. Issue Automation - GitHub Issue 자동 생성 5. Auto-Fix & PR - 자동 수정 및 PR 생성 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- RetrospectDetailResponse에 currentUserStatus 필드 추가 - CurrentUserStatus enum 정의 (NOT_PARTICIPATED/DRAFT/SUBMITTED/ANALYZED) - member_retro 테이블에서 현재 사용자의 참여 상태 조회 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
📝 WalkthroughWalkthroughA new Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
🧪 Generate unit tests (beta)
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. Comment |
PR Check PassedAll validation checks have passed:
This comment was automatically generated by the PR Check workflow. |
| /// 현재 사용자의 회고 참여 상태 | ||
| #[derive(Debug, Serialize, ToSchema)] | ||
| #[serde(rename_all = "SCREAMING_SNAKE_CASE")] | ||
| pub enum CurrentUserStatus { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[Suggestion] CurrentUserStatus enum이 main.rs의 #[derive(OpenApi)] schemas 목록에 등록되어 있지 않습니다. RetrospectDetailResponse가 이 타입을 참조하므로, Swagger UI에서 enum 값이 제대로 표시되지 않을 수 있습니다.
// main.rs schemas에 추가 필요:
CurrentUserStatus,| /// 해당 회고의 질문 리스트 (index 기준 오름차순 정렬, 최대 5개) | ||
| pub questions: Vec<RetrospectQuestionItem>, | ||
| /// 현재 요청 사용자의 회고 참여 상태 | ||
| pub current_user_status: CurrentUserStatus, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[Suggestion] API 스펙 문서(docs/api-specs/013-retrospect-detail.md)에 currentUserStatus 필드가 아직 추가되지 않았습니다. 응답 필드 목록과 예시 JSON에 새 필드를 반영해주세요.
| currentUserStatus | string (Enum) | 현재 요청 사용자의 회고 참여 상태 (NOT_PARTICIPATED / DRAFT / SUBMITTED / ANALYZED) |
catturtle123
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
코드 리뷰 요약
✅ 잘된 점
CurrentUserStatusenum이SCREAMING_SNAKE_CASEserde 규칙으로 잘 정의되어 있고,RetrospectStatus엔티티와 일관된 매핑이 깔끔합니다- 기존
member_retros데이터를 재활용하여 추가 DB 쿼리 없이 상태를 판단하는 효율적인 구현입니다 unwrap_or(CurrentUserStatus::NotParticipated)로 참여하지 않은 사용자 케이스를 안전하게 처리했습니다- 테스트에서
Submitted,NotParticipated,Draft등 다양한 상태를 검증하고 있습니다
📝 개선 제안
- OpenAPI 스키마 등록 누락:
CurrentUserStatusenum이main.rs의#[derive(OpenApi)]schemas 목록에 등록되지 않았습니다. Swagger UI에서 정상적으로 표시되려면 추가가 필요합니다 - API 스펙 문서 미반영:
docs/api-specs/013-retrospect-detail.md에currentUserStatus필드가 반영되지 않았습니다. 응답 예시와 필드 목록 업데이트가 필요합니다
ℹ️ 참고
- PR에
docs/ai-automation/문서 변경이 대량 포함되어 있는데 (+2196 -1324), 이 변경들은currentUserStatus기능과 무관합니다. 별도 PR로 분리하면 리뷰가 더 수월할 것 같습니다
📁 변경 파일: 12개 (+2240 -1326)
📊 리뷰 요약
🟡 개선 제안: 2건
⚪ 참고: 1건
총 2개 인라인 코멘트 작성됨
🤖 Reviewed by Claude Code
Summary
currentUserStatus필드 추가NOT_PARTICIPATED/DRAFT/SUBMITTED/ANALYZEDRetrospectDetailResponseDTO 및 서비스 로직 수정Changed Files
codes/server/src/domain/retrospect/dto.rs-currentUserStatus필드 및 enum 추가codes/server/src/domain/retrospect/service.rs- 사용자 상태 조회 로직 추가Test plan
cargo test통과cargo clippy -- -D warnings통과/health체크currentUserStatus필드 확인🤖 Generated with Claude Code
Summary by CodeRabbit