Skip to content

Merge dev into main - #233

Merged
SinnoLn merged 1 commit into
mainfrom
dev
Aug 21, 2026
Merged

Merge dev into main#233
SinnoLn merged 1 commit into
mainfrom
dev

Conversation

@SinnoLn

@SinnoLn SinnoLn commented Aug 21, 2026

Copy link
Copy Markdown
Member

No description provided.

* feat(auth): 탈퇴 후 30일 이내 재로그인 시 계정 자동 복구

탈퇴 즉시 재가입이 불가능해 이메일이 영구 봉인되는 문제를 완화하기 위해,
탈퇴 유예기간(30일)을 두고 그 안에 재로그인하면 계정을 조용히 복구한다.

- User: 유예기간 판정/복구 로직을 도메인 메서드로 추가 (deletedAt만 복원,
  세션 버전은 탈퇴 시점에 이미 무효화됐으므로 추가로 올리지 않음)
- 로컬/구글 로그인(이미 연동된 계정)은 30일 이내면 자동 복구 후 정상 로그인 처리
- 신규 가입/미연동 구글 인증 시도는 계속 차단하되, 30일 이내는 로그인을
  유도하는 AUTH-014(ACCOUNT_DORMANT)로, 초과는 기존 AUTH-013 그대로 응답

* docs: edit docs

* docs: update docs

* fix(auth): fix stale withdrawal tests and missing AUTH-014 in google auth docs

* docs: doc update
@SinnoLn SinnoLn self-assigned this Aug 21, 2026
@coderabbitai

coderabbitai Bot commented Aug 21, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@SinnoLn, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 47 minutes

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

How can I continue?

Wait for the limit to reset, then comment @coderabbitai review or push new commits to the PR.

An organization admin can change what happens after included review limits in Billing.

How do review limits work?

CodeRabbit enforces per-developer PR review limits within each organization.

For paid Pro and Pro+ reviews, CodeRabbit uses a developer's included PR review attempts over the past 7 days to set the current hourly allowance. At typical activity levels, the full plan allowance applies. Higher sustained activity can lower the allowance until earlier attempts leave the 7-day window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: e2191ea6-07a4-4972-976e-49d0561e0af7

📥 Commits

Reviewing files that changed from the base of the PR and between 53d5347 and 44ee768.

📒 Files selected for processing (8)
  • src/main/java/chaeso/zip/server/auth/application/AuthServiceImpl.java
  • src/main/java/chaeso/zip/server/auth/application/AuthSessionService.java
  • src/main/java/chaeso/zip/server/auth/domain/AuthErrorCode.java
  • src/main/java/chaeso/zip/server/auth/presentation/AuthApiDocs.java
  • src/main/java/chaeso/zip/server/user/domain/User.java
  • src/test/java/chaeso/zip/server/auth/application/AuthServiceTest.java
  • src/test/java/chaeso/zip/server/user/domain/UserTest.java
  • src/test/java/chaeso/zip/server/user/presentation/UserWithdrawalIntegrationTest.java

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.

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.92%. Comparing base (b6ce5b7) to head (44ee768).
⚠️ Report is 7 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #233      +/-   ##
============================================
+ Coverage     91.87%   91.92%   +0.04%     
- Complexity      782      790       +8     
============================================
  Files           131      131              
  Lines          2363     2377      +14     
  Branches        203      204       +1     
============================================
+ Hits           2171     2185      +14     
  Misses          129      129              
  Partials         63       63              
Files with missing lines Coverage Δ
...o/zip/server/auth/application/AuthServiceImpl.java 96.36% <100.00%> (+0.06%) ⬆️
...ip/server/auth/application/AuthSessionService.java 100.00% <100.00%> (ø)
...a/chaeso/zip/server/auth/domain/AuthErrorCode.java 100.00% <100.00%> (ø)
.../main/java/chaeso/zip/server/user/domain/User.java 93.75% <100.00%> (+1.15%) ⬆️
Components Coverage Δ
auth 95.66% <100.00%> (+0.07%) ⬆️
channel 82.53% <ø> (ø)
onboarding 89.00% <ø> (ø)
estimation 94.11% <ø> (ø)
performance 91.66% <ø> (ø)
simulation 91.82% <ø> (ø)
user 96.96% <100.00%> (+0.24%) ⬆️

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@SinnoLn
SinnoLn merged commit 2d52dfb into main Aug 21, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants