[Feat/#244] 기간 반복 일정 삭제 범위 제한

[hyuneey2]

📌 관련 이슈번호

(Closes 키워드가 있어야 PR이 머지되었을 때 이슈가 자동으로 닫힌다)

• Closes [Feat] 기간 반복 일정 삭제 범위 ALL로 제한 #244

📌 PR 유형

어떤 변경 사항이 있나요?

• 새 기능 추가
• 버그 수정
• 리팩토링

📌 PR 요약

기간 반복 일정(PERIOD) 삭제 시 단일 삭제를 제한하고, 전체 삭제만 가능하도록 수정했습니다.

📌 작업 세부 내용

1. PERIOD 타입의 scope=SINGLE 삭제 요청 차단
2. scope=ALL 요청 시 동일 반복 그룹 일정 전체 삭제

📸 스크린샷 (선택)

🔗 참고 자료

[coderabbitai]

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: 55ec81a0-5c2d-45b4-aa51-7eaf965586cf

📥 Commits

Reviewing files that changed from the base of the PR and between efabd3f and cb41aa7.

📒 Files selected for processing (4)

• src/schedules/schedule.controller.ts
• src/schedules/schedule.dto.ts
• src/schedules/schedule.service.spec.ts
• src/schedules/schedule.service.ts

---

Walkthrough

기간 반복 일정은 scope=ALL로만 삭제할 수 있도록 서비스 로직을 변경했습니다. 잠금된 일정 ID를 기준으로 삭제하며, 관련 테스트와 Swagger 설명을 추가했습니다.

Changes

기간 반복 일정 삭제

Layer / File(s)	Summary
삭제 규칙과 검증 src/schedules/schedule.service.ts, src/schedules/schedule.service.spec.ts	기간 반복 일정의 SINGLE 삭제를 거부합니다. ALL 삭제는 잠금된 일정 ID를 사용합니다. 반복 그룹 잠금과 삭제 결과를 테스트합니다.
삭제 API 문서 반영 src/schedules/schedule.controller.ts, src/schedules/schedule.dto.ts	Swagger 설명에 PERIOD 일정의 scope=ALL 제한을 추가합니다. 400 응답에 scope 검증 오류와 SINGLE 삭제 오류를 문서화합니다.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Possibly related PRs

• TEAM-DAILOG/BE#53: deleteSchedule, 삭제 범위 DTO, 일정 컨트롤러의 초기 구현과 직접 연결됩니다.
• TEAM-DAILOG/BE#156: schedule.service.ts의 반복 일정 처리와 관련되지만, 월간·연간 일정 생성 기능을 다룹니다.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	제목이 PERIOD 일정의 삭제 범위를 제한하는 주요 변경 사항을 명확하게 설명합니다.
Description check	✅ Passed	설명이 PERIOD 일정의 SINGLE 삭제 차단과 ALL 삭제 허용이라는 변경 사항을 설명합니다.
Linked Issues check	✅ Passed	구현이 이슈 #244의 PERIOD 일정 삭제 범위를 ALL로 제한하는 요구 사항을 충족합니다.
Out of Scope Changes check	✅ Passed	Swagger 문서, 서비스 로직, 테스트 변경이 모두 linked issue의 삭제 범위 제한과 직접 관련됩니다.
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 feat/#244

---

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