Chore: kafka 통신을 위한 orderId 필드 추가

[caminobelllo]

💡 PR 작업 내용

• 기능 관련 작업
• 버그 수정
• 코드 개선 및 수정
• 문서 작성
• 배포
• 브랜치
• 기타 (아래에 자세한 내용 기입해 주세요)

📋 세부 작업 내용

• CashRequestDto / CashResponseDTO에 각각 orderId 필드를 추가하였습니다.
• 필드 추가로 인해 kafka consumer 요청 시에 반영하였습니다.
• 로그 트레이싱을 위한 zipkin 의존성을 추가하였습니다.

📸 작업 화면 (스크린샷)

💬 리뷰 요구사항(선택)

⚠️ PR하기 전에 확인해 주세요.

• 로컬테스트를 진행하셨나요?
• 머지할 브랜치를 확인하셨나요?
• 관련 label을 선택하셨나요?

⭐ 관련 이슈 번호 [#이슈번호]

• Feat: core-service 포인트 결제 관련 기능 #34

🖇️ 기타

Summary by CodeRabbit

릴리스 노트

• 새로운 기능
  • 결제 서비스에 tracing/모니터링(분산 추적) 기능 추가
  • 결제 요청 및 응답에 주문 ID(orderId) 포함 — 내부 처리 및 이벤트에 주문 ID가 전달되어 추적성이 향상됨

[coderabbitai]

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

orderId 필드를 CashRequestDTO/CashResponseDTO에 추가하고, Kafka 소비자에서 결과 전송 시 해당 필드를 포함하도록 업데이트했습니다. build.gradle에서 validation 의존성 위치를 변경하고 Micrometer/Zipkin 추적 의존성을 추가했습니다.

Changes

Cohort / File(s)	Summary
DTO 변경 on-premise/core-payment-service/src/main/java/com/fisa/core_payment_service/dto/CashRequestDTO.java, on-premise/core-payment-service/src/main/java/com/fisa/core_payment_service/dto/CashResponseDTO.java	orderId 필드 추가. Lombok 생성자/게터 등 시그니처가 변경되어 생성자 호출 순서에 영향 있음.
Kafka 소비자 업데이트 on-premise/core-payment-service/src/main/java/com/fisa/core_payment_service/service/KafkaConsumerService.java	withdraw 흐름의 성공/실패 응답 생성 시 CashResponseDTO 생성자에 orderId를 첫 인자로 포함하도록 수정(전송 페이로드 변경).
빌드 의존성 변경 on-premise/core-payment-service/build.gradle	기본 dependencies 블록에서 org.springframework.boot:spring-boot-starter-validation 제거 후, kafka/secondary 블록에 동일 validation 의존성으로 재배치. 또한 io.micrometer:micrometer-tracing-bridge-brave 및 io.zipkin.reporter2:zipkin-reporter-brave 추적 관련 의존성 추가.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• Chore: kafka 통신을 위한 orderId 필드 추가 #46: DTO 필드 추가 및 build.gradle의 validation/추적 의존성 관련 변경이 동일하게 적용되어 있음.
• Feat: 포인트 결제 관련 core payment 로직 추가 #39: CashRequestDTO/CashResponseDTO 및 Kafka 컨슈머/프로듀서 메시지 형식 변경과 직접적인 겹침이 있음.
• Refactor: core 에서 들어오는 kafka Listener test push 수정 #38: core-to-point Kafka 메시지에 orderId 추가 및 전파 로직과 유사한 변경을 포함함.

Poem

작은 ID 하나가 흐름을 이어가네 ✨
요청에 새겨진 orderId 한 줄,
Kafka를 타고 지나며 흔적을 남기고,
Zipkin은 그 길을 조용히 비추네 🔍
서비스의 이야기, 한 줄로 연결되다 🚀

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	PR 제목이 변경사항의 핵심을 명확하게 반영하고 있습니다. orderId 필드 추가와 Kafka 통신 개선이라는 주요 변경사항을 간결하게 표현하고 있습니다.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feature/core-payment-point

---

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

[coderabbitai]

Actionable comments posted: 1

🤖 Fix all issues with AI agents

In `@on-premise/core-payment-service/build.gradle`:
- Around line 47-49: Remove the duplicate dependency declaration for
org.springframework.boot:spring-boot-starter-actuator (the second implementation
line) so only the single declaration remains; locate the repeated implementation
'org.springframework.boot:spring-boot-starter-actuator' entry in the
build.gradle and delete it, leaving the other dependencies
(io.micrometer:micrometer-tracing-bridge-brave and
io.zipkin.reporter2:zipkin-reporter-brave) intact.