Skip to content

refact: AppDbContext constructor 파라미터명 통일 (#92) - #95

Open
jinseon901 wants to merge 1 commit into
developfrom
refact/#92-db-context
Open

refact: AppDbContext constructor 파라미터명 통일 (#92)#95
jinseon901 wants to merge 1 commit into
developfrom
refact/#92-db-context

Conversation

@jinseon901

@jinseon901 jinseon901 commented Jul 25, 2026

Copy link
Copy Markdown
Contributor

#️⃣연관된 이슈

close #92

📌작업 내용

db repository의 AppDbContext 파라미터명 통일

🧪 테스트

  • 로컬에서 동작 확인

@jinseon901
jinseon901 requested a review from hyorim-jo July 25, 2026 05:34
@jinseon901 jinseon901 self-assigned this Jul 25, 2026
@jinseon901 jinseon901 added the ♻️ refactor 코드 개선 (기능 변경 없음) label Jul 25, 2026
@coderabbitai

coderabbitai Bot commented Jul 25, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: a0afa637-83ad-4742-972a-0b3a934f3bb8

📥 Commits

Reviewing files that changed from the base of the PR and between ede8a66 and 6742e2b.

📒 Files selected for processing (7)
  • Speakat.Infrastructure/Persistence/Repositories/GameSessionRepository.cs
  • Speakat.Infrastructure/Persistence/Repositories/UserCalendarRepository.cs
  • Speakat.Infrastructure/Persistence/Repositories/UserProfileRepository.cs
  • Speakat.Infrastructure/Persistence/Repositories/UserSettingsRepository.cs
  • Speakat.Infrastructure/Persistence/Repositories/UserStageRepository.cs
  • Speakat.Infrastructure/Persistence/Repositories/UserStatsRepository.cs
  • Speakat.Infrastructure/Persistence/Repositories/UserStreakRepository.cs

📝 Walkthrough

Summary by CodeRabbit

  • 개선 사항
    • 데이터 조회 및 저장 처리의 내부 참조 방식을 일관되게 정리했습니다.
    • 게임 세션, 프로필, 설정, 단계, 통계 및 연속 기록 관련 기능의 기존 동작과 결과는 그대로 유지됩니다.

Walkthrough

모든 대상 리포지토리에서 주입된 AppDbContext 매개변수와 EF Core 조회·저장 호출의 참조명을 db에서 context로 변경했습니다. 메서드 시그니처의 동작과 데이터 처리 흐름은 유지되었습니다.

Changes

리포지토리 컨텍스트 참조 리팩터링

Layer / File(s) Summary
게임 세션 처리 컨텍스트 참조 변경
Speakat.Infrastructure/Persistence/Repositories/GameSessionRepository.cs
세션 생성, 포기, 완료 처리의 조회·추가·저장 호출을 context 기준으로 변경했습니다.
사용자 데이터 변경 컨텍스트 참조 변경
Speakat.Infrastructure/Persistence/Repositories/UserProfileRepository.cs, Speakat.Infrastructure/Persistence/Repositories/UserSettingsRepository.cs, Speakat.Infrastructure/Persistence/Repositories/UserStageRepository.cs
프로필, 설정, 스테이지 관련 조회 및 SaveChangesAsync() 호출을 context 기준으로 변경했습니다.
사용자 데이터 조회 컨텍스트 참조 변경
Speakat.Infrastructure/Persistence/Repositories/UserCalendarRepository.cs, Speakat.Infrastructure/Persistence/Repositories/UserStatsRepository.cs, Speakat.Infrastructure/Persistence/Repositories/UserStreakRepository.cs
캘린더, 통계, 스트릭 조회의 EF Core 쿼리 대상을 context 기준으로 변경했습니다.

Estimated code review effort: 1 (Trivial) | ~3 minutes

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (2 warnings)

Check name Status Explanation Resolution
Linked Issues check ⚠️ Warning 이슈 #92의 DbContext(options) 통일 요구와 달리 실제 변경은 여러 저장소의 db를 context로 바꾼 리팩터링입니다. linked issue의 목표에 맞게 AppDbContext 생성자/options 관련 변경을 반영하거나, 이슈 범위를 현재 리팩터링에 맞게 재정의하세요.
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.
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed 제목은 파라미터명 통일 리팩터링을 요약하고 있어 변경 내용과 관련성이 높습니다.
Description check ✅ Passed 설명이 db repository의 AppDbContext 파라미터명 통일로 변경 범위를 잘 설명합니다.
Out of Scope Changes check ✅ Passed 모든 변경이 저장소 내 컨텍스트 변수명 통일에만 해당하며 별도의 무관한 수정은 보이지 않습니다.

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

Comment @coderabbitai help to get the list of available commands.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

♻️ refactor 코드 개선 (기능 변경 없음)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Refactor] DbContext(options)로 통일

1 participant