Skip to content

[REFACTOR] usage_limit 삭제 - #231

Merged
kimyw1018 merged 2 commits into
developfrom
refactor/#230-delete-usagelimit
Aug 13, 2026
Merged

[REFACTOR] usage_limit 삭제#231
kimyw1018 merged 2 commits into
developfrom
refactor/#230-delete-usagelimit

Conversation

@kimyw1018

@kimyw1018 kimyw1018 commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

📍 개요

mvp 기능 아닌 코드 삭제

⛓️‍💥 관련 이슈


🛠️ 작업 내용

  • 미사용 usagelimit 엔티티, 레포지토리 삭제
  • usagelimit 연관 코드 삭제

✅ 체크리스트

  • 코드 컨벤션을 준수했습니다.
  • 불필요한 코드 및 import를 제거했습니다.
  • 예외 처리를 적용했습니다.
  • 테스트를 완료했습니다.
  • 관련 Issue를 연결했습니다.

📎 참고 사항

Summary by CodeRabbit

  • 변경 사항
    • 사용자 탈퇴 처리에서 사용량 제한 데이터 정리 절차가 제거되었습니다.
    • 사용량 제한 관련 기능과 오류 안내가 더 이상 제공되지 않습니다.
    • 탈퇴 시 소셜 인증 및 기타 관련 데이터 삭제 후 계정이 삭제되는 기존 흐름은 유지됩니다.

@coderabbitai

coderabbitai Bot commented Aug 12, 2026

Copy link
Copy Markdown

Review Change Stack

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: 68a0eb79-6dbc-461b-bbd1-efd1cdd7a5df

📥 Commits

Reviewing files that changed from the base of the PR and between 18473b0 and dc61c7d.

📒 Files selected for processing (1)
  • src/main/java/com/mr/domain/user/exception/UserUsageErrorStatus.java
💤 Files with no reviewable changes (1)
  • src/main/java/com/mr/domain/user/exception/UserUsageErrorStatus.java

📝 Walkthrough

Walkthrough

UsageLimit 엔티티, 저장소, 오류 상태를 삭제했습니다. AuthService의 관련 의존성과 회원 탈퇴 시 사용량 제한 삭제 호출을 제거했습니다. SocialAuth 삭제와 사용자 삭제 흐름은 유지됩니다.

Changes

사용량 제한 제거

Layer / File(s) Summary
사용량 제한 의존성 및 탈퇴 처리 제거
src/main/java/com/mr/domain/auth/service/AuthService.java, src/main/java/com/mr/domain/user/entity/UsageLimit.java, src/main/java/com/mr/domain/user/repository/UsageLimitRepository.java, src/main/java/com/mr/domain/user/exception/UserUsageErrorStatus.java
사용량 제한 엔티티, 저장소, 오류 상태를 삭제했습니다. AuthService에서 관련 import, 필드, 의존성, 회원 탈퇴 삭제 호출을 제거했습니다. SocialAuth 삭제와 사용자 삭제는 유지됩니다.

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

Mergeability Score: ⚪ Minimal · up to dc61c

This change removes unused usage-limit code without introducing a concrete user or production impact; no actionable merge-blocking risk remains beyond normal checks.

Possibly related PRs

  • Musereview/BE#14: 삭제된 UsageLimit 엔티티와 저장소를 추가한 변경입니다.
  • Musereview/BE#61: AuthService의 회원 탈퇴 흐름에서 SocialAuth 삭제를 추가한 변경입니다.
  • Musereview/BE#115: 회원 탈퇴 시 사용량 제한 삭제 흐름을 추가한 변경입니다.

Poem

사용량 제한 코드가 사라지고
탈퇴 흐름도 짧아졌네.
SocialAuth 정리는 남아서
마지막 단계까지 수행하네.
작은 삭제, 큰 정리!

🚥 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 제목은 MVP 기능이 아닌 usageLimit 관련 코드 삭제라는 주요 변경 사항을 명확하고 간결하게 설명합니다.
Linked Issues check ✅ Passed UsageLimit 엔티티와 레포지토리 및 해당 엔티티에 종속된 UserUsageErrorStatus를 삭제하여 이슈 #230의 목표를 충족합니다.
Out of Scope Changes check ✅ Passed 삭제된 UserUsageErrorStatus는 UsageLimit 삭제 후 남은 미참조 코드이므로 PR 목표와 관련된 범위 내 변경입니다.
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 refactor/#230-delete-usagelimit

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.

@rkdehdrbs7885-oss rkdehdrbs7885-oss left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니다!

@p1001q

p1001q commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

🟡 P3src/main/java/com/mr/domain/user/exception/UserUsageErrorStatus.java (이번 PR에서 안 건드린 파일)

문제상황: 로컬에서 브랜치 전체를 grep해보니, 이 PR 적용 후 UserUsageErrorStatus를 참조하는 곳이 자기 자신(enum 선언)밖에 없습니다 — UsageLimit.java 삭제로 인해 완전히 죽은 코드가 됐는데 파일 자체는 안 지워졌습니다.

문제가 되는 이유: 이번 PR 목적이 "MVP 기능 아닌 코드 삭제"라, 딱 같은 카테고리의 잔여물입니다.

수정 방향: UserUsageErrorStatus.java도 같이 삭제하면 깔끔합니다.


참고!) DB에 남는 usage_limit 테이블 자체는 Flyway 없이
ddl-auto: validate(prod)/update(dev) 조합이라 지금 당장 문제는 안 되니 급하지 않은 상황입니다.

@p1001q p1001q left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

p1 문제는 없어서 승인드립니다! 수고하셨습니다~

Comment thread src/main/java/com/mr/domain/auth/service/AuthService.java
@kimyw1018
kimyw1018 merged commit 5ca0e51 into develop Aug 13, 2026
2 checks passed
@kimyw1018
kimyw1018 deleted the refactor/#230-delete-usagelimit branch August 14, 2026 00:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

♻️ Refactor - usageLimit 엔티티 삭제

3 participants