Feature/attendance api integration fix

[ochanhyeok]

Summary by CodeRabbit

릴리스 노트

• 새로운 기능

  • 출석 라운드 관리 기능 추가 - 세션별 시간 기반 출석 체크인 지원
  • GPS 위치 기반 출석 검증 시스템 구현
  • 익명 사용자 출석 체크인 지원
  • OAuth 로그인 개선 - 자동 리디렉션 및 토큰 관리
  • 세션 위치 정보 업데이트 기능 추가

• 개선사항

  • 출석 관리 페이지 UI 개선 - 실시간 API 기반 세션 및 라운드 로드
  • 로그인/회원가입 페이지 UX 개선 - 실제 API 통합 및 에러 메시지 표시
  • 사용자 인증 상태 관리 강화

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Caution

Review failed

The pull request is closed.

코드 리뷰 요약

Walkthrough

출석 관리 시스템에 라운드(Round) 개념을 도입합니다. 라운드 엔티티, 컨트롤러, 서비스, 저장소, DTO를 추가하고 출석 엔티티를 확장합니다. 세션 기능을 단순화하고 OAuth 리다이렉트 및 JWT 처리를 개선합니다. 프론트엔드 UI 및 API 클라이언트를 업데이트합니다.

Changes

Cohort / File(s)	변경 요약
라운드 엔티티 및 저장소 backend/src/main/java/org/sejongisc/backend/attendance/entity/AttendanceRound.java, RoundStatus.java backend/src/main/java/org/sejongisc/backend/attendance/repository/AttendanceRoundRepository.java	새로운 라운드 엔티티 추가 (roundId, roundDate, startTime, allowedMinutes, 상태 계산 로직). RoundStatus 열거형 정의 (UPCOMING, ACTIVE, CLOSED). 라운드 조회 및 통계 기능을 위한 저장소 메서드 추가
라운드 DTO backend/src/main/java/org/sejongisc/backend/attendance/dto/AttendanceRoundRequest.java, AttendanceRoundResponse.java	라운드 생성/업데이트 요청 DTO 및 응답 DTO 추가. 유효성 검증 및 OpenAPI 스키마 주석 포함
라운드 컨트롤러 및 서비스 backend/src/main/java/org/sejongisc/backend/attendance/controller/AttendanceRoundController.java backend/src/main/java/org/sejongisc/backend/attendance/service/AttendanceRoundService.java	라운드 CRUD 및 라운드 기반 체크인 엔드포인트 추가. JWT 기반 사용자 ID 추출 및 역할 기반 접근 제어 포함
체크인 DTO 및 기능 backend/src/main/java/org/sejongisc/backend/attendance/dto/AttendanceCheckInRequest.java, AttendanceCheckInResponse.java	라운드 기반 체크인 요청/응답 DTO 추가. 위치(위도/경도) 및 익명 사용자 지원
출석 엔티티 및 저장소 확장 backend/src/main/java/org/sejongisc/backend/attendance/entity/Attendance.java backend/src/main/java/org/sejongisc/backend/attendance/repository/AttendanceRepository.java	라운드 연관 관계 추가, 익명 사용자명 필드 추가, 사용자 연관을 nullable로 변경. 라운드별 출석 조회 메서드 추가
출석 응답 DTO backend/src/main/java/org/sejongisc/backend/attendance/dto/AttendanceResponse.java	attendanceRoundId 필드 추가
세션 기능 수정 backend/src/main/java/org/sejongisc/backend/attendance/controller/AttendanceSessionController.java backend/src/main/java/org/sejongisc/backend/attendance/service/AttendanceSessionService.java backend/src/main/java/org/sejongisc/backend/attendance/entity/AttendanceSession.java	코드/태그/가시성 기반 조회 엔드포인트 제거. 세션 위치 업데이트 엔드포인트 추가. 라운드 관계 추가. tag 및 visibility 필드 제거
세션 위치 업데이트 backend/src/main/java/org/sejongisc/backend/attendance/dto/SessionLocationUpdateRequest.java, AttendanceSessionResponse.java	세션 위치 업데이트 요청 DTO 추가. 응답 DTO에 LocationInfo 중첩 클래스 추가
저장소 정리 backend/src/main/java/org/sejongisc/backend/attendance/repository/AttendanceSessionRepository.java backend/src/main/java/org/sejongisc/backend/attendance/entity/SessionVisibility.java	태그/가시성 기반 저장소 메서드 제거. SessionVisibility 열거형 삭제
서비스 확장 backend/src/main/java/org/sejongisc/backend/attendance/service/AttendanceService.java	라운드 기반 체크인 및 라운드별 출석 조회 메서드 추가. 위치 검증 및 익명 사용자 처리 로직 추가
인증 및 JWT backend/src/main/java/org/sejongisc/backend/auth/controller/AuthController.java backend/src/main/java/org/sejongisc/backend/auth/service/LoginServiceImpl.java backend/src/main/java/org/sejongisc/backend/common/auth/jwt/JwtProvider.java	OAuth 리다이렉트 응답 처리 변경 (302 리다이렉트로 토큰 전달). 로그아웃 로깅 개선. getUserIdFromToken에서 uid 클레임 사용으로 변경
보안 정책 backend/src/main/java/org/sejongisc/backend/user/util/PasswordPolicyValidator.java	암호 정책에서 대문자 요구사항 제거
테스트 코드 추가 backend/src/test/java/org/sejongisc/backend/attendance/...	라운드 체크인 검증, 통합 테스트, 라운드 컨트롤러 테스트, 라운드 서비스 테스트, 세션 위치 업데이트 테스트 추가
테스트 코드 제거 backend/src/test/java/org/sejongisc/backend/attendance/controller/AttendanceSessionControllerTest.java backend/src/test/java/org/sejongisc/backend/attendance/service/AttendanceSessionServiceTest.java	기존 세션 컨트롤러/서비스 테스트 삭제
테스트 보안 설정 backend/src/test/java/org/sejongisc/backend/attendance/controller/TestSecurityConfig.java	JwtParser 모의 객체 추가. 라운드 엔드포인트 인증 규칙 추가
템플릿 서비스 backend/src/main/java/org/sejongisc/backend/template/service/TemplateService.java backend/src/test/java/org/sejongisc/backend/template/service/TemplateServiceTest.java, controller/TemplateControllerTest.java	findById 메서드에 userId 매개변수 추가
설정 파일 .claude/settings.local.json backend/src/main/resources/application.yml	Claude 설정 및 Spring 프로필 활성 상태 추가
프론트엔드 OAuth 및 인증 frontend/src/App.jsx frontend/src/pages/Home.jsx frontend/src/components/login/LoginForm.jsx	OAuth 콜백 처리 추가. 로그인 상태 관리 추가. 로그아웃 기능 구현. OAuth 소셜 로그인 핸들러 추가
프론트엔드 회원가입 frontend/src/components/signup/SignUpForm.jsx	비동기 API 기반 회원가입 흐름으로 변경. 에러 메시지 및 로딩 상태 추가
프론트엔드 출석 관리 frontend/src/pages/Attendance.jsx frontend/src/pages/AttendanceManage.jsx frontend/src/components/attendancemanage/SessionSettingCard.jsx frontend/src/components/attendancemanage/RoundAddingCard.jsx, RoundAddingCard.module.css frontend/src/components/attendancemanage/AttendanceManagementCard.jsx	동적 세션/라운드 로드 및 API 기반 체크인 추가. 위치 기반 체크인 구현. 라운드 추가 카드 컴포넌트 추가. 세션 설정 카드 단순화
프론트엔드 API 클라이언트 frontend/src/utils/attendanceApi.js	세션, 라운드, 체크인 관련 API 메서드 통합. Bearer 토큰 인증 및 에러 처리 포함
빌드 설정 frontend/vite.config.js	개발 서버 포트를 3000으로, 호스트를 localhost로 설정

Sequence Diagram(s)

sequenceDiagram
    actor User
    participant Frontend
    participant Backend
    participant Repo

    User->>Frontend: OAuth 콜백 클릭
    Frontend->>Frontend: 쿼리 스트링에서 토큰 파싱
    Frontend->>Frontend: localStorage에 토큰 저장
    Frontend->>Frontend: 홈으로 리다이렉트

    User->>Frontend: 세션 선택 후 라운드 확인
    Frontend->>Backend: GET /api/attendance/sessions
    Backend->>Repo: 세션 조회
    Repo-->>Backend: 세션 목록
    Backend-->>Frontend: 세션 응답
    Frontend->>Frontend: UI 렌더링

    User->>Frontend: 라운드 체크인 시도
    Frontend->>Frontend: 위치 수집
    Frontend->>Backend: POST /api/attendance/rounds/{roundId}/check-in
    Backend->>Backend: 날짜/시간 검증
    Backend->>Backend: 위치 검증
    Backend->>Repo: 출석 기록 저장
    Repo-->>Backend: 저장 완료
    Backend-->>Frontend: 체크인 응답 (성공/실패)
    Frontend->>User: 결과 메시지 표시

Loading

sequenceDiagram
    participant Admin
    participant Frontend
    participant Backend
    participant Repo

    Admin->>Frontend: 세션 생성
    Frontend->>Backend: POST /api/attendance/sessions
    Backend->>Repo: 세션 저장
    Repo-->>Backend: 생성된 세션

    Admin->>Frontend: 라운드 추가
    Frontend->>Backend: POST /api/attendance/sessions/{sessionId}/rounds
    Backend->>Backend: 라운드 상태 계산
    Backend->>Repo: 라운드 저장
    Repo-->>Backend: 생성된 라운드
    Backend-->>Frontend: 라운드 응답

    Admin->>Frontend: 세션 위치 업데이트
    Frontend->>Backend: PUT /api/attendance/sessions/{sessionId}/location
    Backend->>Backend: 위도/경도 검증
    Backend->>Repo: 세션 위치 업데이트
    Repo-->>Backend: 업데이트 완료
    Backend-->>Frontend: 응답

Loading

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~60 분

특별 주의 사항:

• 라운드 기능 전체 스택: 새로운 엔티티, 저장소, 서비스, 컨트롤러, DTO가 일관되게 추가되었으며 비즈니스 로직 검증 필요

  • AttendanceRound 엔티티의 상태 계산 로직 (calculateCurrentStatus, isCheckInAvailable)
  • AttendanceRoundService의 CRUD 및 트랜잭션 처리
  • AttendanceService.checkInByRound의 위치 검증 및 상태 결정 로직

• 출석 엔티티 변경: 사용자 관계를 nullable로 변경하고 라운드 관계 추가 — 기존 코드와의 호환성 검증 필요

• 인증 흐름 변경: OAuth 리다이렉트 응답 처리 및 JWT userId 추출 방식 변경 — 보안 영향 검토 필수

• 세션 기능 축소: 태그/가시성/코드 기반 조회 제거, SessionVisibility 열거형 삭제 — 의존하는 기능이 있는지 전체 확인 필요

• 프론트엔드 API 통합: attendanceApi.js의 Bearer 토큰 처리 및 에러 핸들링 검증

• 테스트 커버리지: 여러 통합 테스트 클래스 추가, 기존 테스트 삭제 — 제거된 테스트의 커버리지 이관 확인

Possibly related PRs

• #92 [BE] SISC1-204 [FEAT] 출석 세션 기능 개선 #97: AttendanceRoundController, AttendanceRound, 라운드 DTO/저장소/서비스 및 관련 테스트의 동일 컴포넌트 변경
• [BE] SISC1-219 [FIX] : 출석세션 관련 불필요한 응답필드 수정 #126: AttendanceSessionResponse 필드 제거/변경 (tag, startsAt, code, endsAt 등) 직접 관련
• [BE] SISC1-221 [FIX] api 응답 개선 #138: SessionVisibility 열거형 및 tag/visibility 필드 제거 동일 변경

Suggested reviewers

• discipline24
• DongEun02
• Kosw6
• gxuoo

Poem

🐰 라운드 시스템이 한 걸음 더,
세션에 날개를 달았네!
위도 경도로 자리 잡고,
OAuth 흐름도 부드럽게,
출석의 모든 순간을 담았어! ✨

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feature/attendance-api-integration-fix

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 5e4416b and 6e8e3ad.

📒 Files selected for processing (53)

• .claude/settings.local.json (1 hunks)
• backend/ATTENDANCE_CHECKIN_GUIDE.md (1 hunks)
• backend/src/main/java/org/sejongisc/backend/attendance/controller/AttendanceRoundController.java (1 hunks)
• backend/src/main/java/org/sejongisc/backend/attendance/controller/AttendanceSessionController.java (3 hunks)
• backend/src/main/java/org/sejongisc/backend/attendance/dto/AttendanceCheckInRequest.java (1 hunks)
• backend/src/main/java/org/sejongisc/backend/attendance/dto/AttendanceCheckInResponse.java (1 hunks)
• backend/src/main/java/org/sejongisc/backend/attendance/dto/AttendanceResponse.java (1 hunks)
• backend/src/main/java/org/sejongisc/backend/attendance/dto/AttendanceRoundRequest.java (1 hunks)
• backend/src/main/java/org/sejongisc/backend/attendance/dto/AttendanceRoundResponse.java (1 hunks)
• backend/src/main/java/org/sejongisc/backend/attendance/dto/AttendanceSessionRequest.java (1 hunks)
• backend/src/main/java/org/sejongisc/backend/attendance/dto/AttendanceSessionResponse.java (3 hunks)
• backend/src/main/java/org/sejongisc/backend/attendance/dto/SessionLocationUpdateRequest.java (1 hunks)
• backend/src/main/java/org/sejongisc/backend/attendance/entity/Attendance.java (2 hunks)
• backend/src/main/java/org/sejongisc/backend/attendance/entity/AttendanceRound.java (1 hunks)
• backend/src/main/java/org/sejongisc/backend/attendance/entity/AttendanceSession.java (1 hunks)
• backend/src/main/java/org/sejongisc/backend/attendance/entity/RoundStatus.java (1 hunks)
• backend/src/main/java/org/sejongisc/backend/attendance/entity/SessionVisibility.java (0 hunks)
• backend/src/main/java/org/sejongisc/backend/attendance/repository/AttendanceRepository.java (1 hunks)
• backend/src/main/java/org/sejongisc/backend/attendance/repository/AttendanceRoundRepository.java (1 hunks)
• backend/src/main/java/org/sejongisc/backend/attendance/repository/AttendanceSessionRepository.java (0 hunks)
• backend/src/main/java/org/sejongisc/backend/attendance/service/AttendanceRoundService.java (1 hunks)
• backend/src/main/java/org/sejongisc/backend/attendance/service/AttendanceService.java (5 hunks)
• backend/src/main/java/org/sejongisc/backend/attendance/service/AttendanceSessionService.java (4 hunks)
• backend/src/main/java/org/sejongisc/backend/auth/controller/AuthController.java (3 hunks)
• backend/src/main/java/org/sejongisc/backend/auth/service/LoginServiceImpl.java (1 hunks)
• backend/src/main/java/org/sejongisc/backend/common/auth/jwt/JwtProvider.java (1 hunks)
• backend/src/main/java/org/sejongisc/backend/user/util/PasswordPolicyValidator.java (1 hunks)
• backend/src/main/resources/application.yml (1 hunks)
• backend/src/test/java/org/sejongisc/backend/attendance/AttendanceCheckInTimeValidationTest.java (1 hunks)
• backend/src/test/java/org/sejongisc/backend/attendance/AttendanceSystemComprehensiveTest.java (1 hunks)
• backend/src/test/java/org/sejongisc/backend/attendance/AttendanceWorkflowIntegrationTest.java (1 hunks)
• backend/src/test/java/org/sejongisc/backend/attendance/controller/AttendanceRoundControllerTest.java (1 hunks)
• backend/src/test/java/org/sejongisc/backend/attendance/controller/AttendanceSessionControllerTest.java (0 hunks)
• backend/src/test/java/org/sejongisc/backend/attendance/controller/TestSecurityConfig.java (2 hunks)
• backend/src/test/java/org/sejongisc/backend/attendance/service/AttendanceRoundCheckInTest.java (1 hunks)
• backend/src/test/java/org/sejongisc/backend/attendance/service/AttendanceRoundServiceTest.java (1 hunks)
• backend/src/test/java/org/sejongisc/backend/attendance/service/AttendanceServiceTest.java (0 hunks)
• backend/src/test/java/org/sejongisc/backend/attendance/service/AttendanceSessionServiceTest.java (0 hunks)
• backend/src/test/java/org/sejongisc/backend/attendance/service/SessionLocationUpdateTest.java (1 hunks)
• backend/src/test/java/org/sejongisc/backend/template/controller/TemplateControllerTest.java (1 hunks)
• backend/src/test/java/org/sejongisc/backend/template/service/TemplateServiceTest.java (2 hunks)
• frontend/src/App.jsx (2 hunks)
• frontend/src/components/attendancemanage/AttendanceManagementCard.jsx (1 hunks)
• frontend/src/components/attendancemanage/RoundAddingCard.jsx (1 hunks)
• frontend/src/components/attendancemanage/RoundAddingCard.module.css (1 hunks)
• frontend/src/components/attendancemanage/SessionSettingCard.jsx (2 hunks)
• frontend/src/components/login/LoginForm.jsx (4 hunks)
• frontend/src/components/signup/SignUpForm.jsx (3 hunks)
• frontend/src/pages/Attendance.jsx (1 hunks)
• frontend/src/pages/AttendanceManage.jsx (4 hunks)
• frontend/src/pages/Home.jsx (1 hunks)
• frontend/src/utils/attendanceApi.js (1 hunks)
• frontend/vite.config.js (1 hunks)

---

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