스크랩 동시 생성 시 유니크 제약 위반이 500으로 새던 문제 수정

[unam98]

작업 배경

• 백엔드 채용 준비 과정에서 "동시성을 다뤄본 경험"을 이력서에 구체적으로 녹이기 위해, Runnect 기존 코드에서 유사한 동시성 취약점이 더 있는지 점검하다가 발견함.
• 스크랩 생성(ScrapService.createAndDeleteScrap)이 "기존 스크랩 조회 → 없으면 저장" 순서(TOCTOU)로 동작해, 더블탭이나 다중 기기로 동시 요청이 겹치면 둘 다 "스크랩 없음"을 보고 각자 저장을 시도할 수 있음. (user_id, public_course_id) 유니크 제약이 이미 DB 레벨에서 경합을 막고 있었지만, 그 예외를 서비스가 잡지 않아 그대로 500 + 불필요한 Slack/Sentry 알림으로 이어짐.
• 같은 프로젝트의 HealthService엔 이미 동일 패턴(try/catch DataIntegrityViolationException → ConflictException 409)이 적용돼 있었는데, ScrapService엔 일관되게 적용돼 있지 않았음 — 새 기법 도입이 아니라 기존 컨벤션을 놓친 곳을 찾아 맞추는 작업.

변경 사항

영역	내용
common/constant/ErrorStatus.java	ALREADY_EXIST_SCRAP_EXCEPTION(409) 추가
scrap/service/ScrapService.java	scrapRepository.save()를 try/catch로 감싸 DataIntegrityViolationException 발생 시 ConflictException으로 변환 (HealthService와 동일 패턴)

영향 범위

• 스크랩 생성 시 동시 요청 경합 상황에 한정. 정상 경로(경합 없음) 동작은 변화 없음.
• 새로운 동시성 제어 기법(락, 재시도 등)을 도입하지 않음 — DB 유니크 제약이 이미 원자적으로 보장하던 것을, 애플리케이션이 그 결과(예외)를 우아하게 처리하도록만 고침. 오버엔지니어링 없이 최소 변경으로 해결.
• 런타임 영향: 경합이 실제로 발생하는 극히 드문 경우에만 응답 코드가 500 → 409로 바뀜. 그 외 영향 없음.

검증 매트릭스

영향 범위	테스트 코드
동시 스크랩 요청 경합 시 409(ConflictException)로 처리됨 (실제 로컬 Postgres, 2스레드로 재현)	• 동시에_같은_코스를_스크랩하면_한쪽은_ConflictException으로_처리된다

Test Plan

• 수정 전: 같은 재현 테스트로 DataIntegrityViolationException이 서비스 밖으로 그대로 새는 것을 먼저 확인
• 수정 후: 같은 재현 테스트가 ConflictException으로 처리됨을 확인, 3연속 실행으로 플레이키니스 점검
• ./gradlew test 전체 통과 (262개 테스트, 실패/에러 0건)
• 로컬 테스트 데이터 정리 확인(잔존 row 0건)

🤖 Generated with Claude Code

[coderabbitai]

Warning

Review limit reached

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

Next review available in: 59 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: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 5e7afa24-aec8-491e-bd52-2637c412400d

📥 Commits

Reviewing files that changed from the base of the PR and between 0a8a26e and 4cff35f.

📒 Files selected for processing (3)

• src/main/java/org/runnect/server/common/constant/ErrorStatus.java
• src/main/java/org/runnect/server/scrap/service/ScrapService.java
• src/test/java/org/runnect/server/scrap/ScrapConcurrencyTest.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

• X
• Mastodon
• Reddit
• LinkedIn

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