docs: 문의 도메인 정책 가이드 추가#576
Conversation
- 문의 API가 저장 없이 이벤트를 발행하고 Slack 알림으로 이어지는 경계를 문서화했습니다. - 공개 API 검증, 원문 이벤트 발행, AFTER_COMMIT Slack 위임 정책을 분리해 채팅 문의방 정책과 혼동되지 않도록 했습니다. - 문서에서 고정한 이벤트 발행과 Slack 리스너 위임 경로를 단위 테스트로 보강했습니다.
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 52 minutes and 9 seconds. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (3)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Pull request overview
문의(inquiry) 도메인의 정책/경계(공개 API, 이벤트 발행, 트랜잭션 커밋 이후 Slack 알림 위임)를 AGENTS 가이드로 문서화하고, 핵심 실행 경로(이벤트 payload 원문 유지, Slack 서비스 위임)를 단위 테스트로 고정하는 PR입니다.
Changes:
inquiry도메인 정책 가이드 문서(AGENTS.md) 추가InquiryService.submitInquiry가 입력 content를 원문 그대로InquirySubmittedEvent로 발행하는 단위 테스트 추가InquirySlackListener가 이벤트 content를SlackNotificationService.notifyInquiry로 위임하는 단위 테스트 추가
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| src/main/java/gg/agit/konect/domain/inquiry/AGENTS.md | 문의 도메인의 공개 API/이벤트/Slack 후속 처리 정책과 변경 시 체크리스트를 문서화 |
| src/test/java/gg/agit/konect/unit/domain/inquiry/service/InquiryServiceTest.java | 이벤트 발행 시 content 원문 유지(트리밍/마스킹 없음)를 단위 테스트로 고정 |
| src/test/java/gg/agit/konect/unit/infrastructure/slack/listener/InquirySlackListenerTest.java | Slack 리스너가 이벤트 content를 알림 서비스로 위임하는 경로를 단위 테스트로 고정 |
🧪 JaCoCo Coverage Report (Changed Files)
|
🔍 개요
🚀 주요 변경 내용
src/main/java/gg/agit/konect/domain/inquiry/AGENTS.md추가InquiryService가 문의 내용을 원문 그대로InquirySubmittedEvent로 발행하는 단위 테스트 추가InquirySlackListener가 이벤트 content를 Slack 알림 서비스에 위임하는 단위 테스트 추가💬 참고 사항
CI=true ./gradlew test --tests 'gg.agit.konect.integration.domain.inquiry.*' --tests 'gg.agit.konect.unit.domain.inquiry.*' --tests 'gg.agit.konect.unit.infrastructure.slack.listener.InquirySlackListenerTest'\n* 확인:git diff --check,./gradlew checkstyleMain\n*./gradlew checkstyleTest는 기존 테스트 파일의 120자 초과 13건으로 실패하며, 이번 변경 파일에서는 위반이 발생하지 않았습니다.✅ Checklist (완료 조건)