Feat(#64): RabbitMQ 이벤트에 evidenceCards 필드 추가 - #75
Conversation
REST 응답(SmishingAnalysisResponse.evidence)에는 이미 노출되던 증거 카드가 RabbitMQ AnalysisResultPayload에는 매핑되지 않아 Spring에 전달되지 않고 있었다. Spring PR #102(머지·배포됨)에서 합의한 대로 필드를 구현한다. - EvidenceCardDetail 스키마 추가 (category/title/description) - AnalysisResultPayload 최상위에 evidenceCards로 연결 - institutionMatch와 달리 규칙/URL/텍스트 트랙을 모두 아우르는 결과라 ruleAnalysis 하위가 아닌 최상위 (Spring과 합의된 위치) - result_factory.py에서 result.evidence(EvidenceItem 리스트)를 매핑, category는 enum .value로 변환 Spring이 이미 반영·배포된 상태(PR #102)라 배포 순서 조건 충족.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (3)
📝 WalkthroughWalkthroughThe RabbitMQ result payload now includes serialized evidence cards. A new Pydantic schema defines each card, and the result factory maps evidence items or returns an empty list. Tests cover successful and failed executions. ChangesEvidence card payload support
Estimated code review effort: 2 (Simple) | ~10 minutes Merge Risk: ⚪ Minimal · up to RabbitMQ 분석 결과에 evidenceCards를 추가하고 기존 증거 카드 데이터를 매핑하는 localized change이며, 관련 테스트가 통과했습니다. 현재 머지를 막는 actionable risk는 없습니다. Possibly related issues
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 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 |
Summary
evidence)가 RabbitMQAnalysisResultPayload에는 매핑되지 않아 Spring에 전달되지 않고 있던 문제 해결evidenceCards를AnalysisResultPayload최상위에 추가 (institutionMatch와 달리 규칙/URL/텍스트 트랙을 모두 아우르므로ruleAnalysis하위가 아님)EvidenceCardDetail(category, title, description)스키마,result_factory.py에서result.evidence매핑 (category는 enum →.value)Spring이 이미 반영·배포된 상태라 배포 순서 조건(Spring 먼저) 충족 — 이 PR은 머지 후 바로 배포해도 됩니다.
Refs #64
Test plan
pytest tests/infrastructure/rabbitmq/(91 passed)pytest(599 passed, 3 deselected)build_evidence()→_evidence_cards()수동 통합 확인Summary by CodeRabbit