1.7.3

[rettooo]

📝작업 내용

로그 추가

👀변경 사항

🎫 Jira Ticket

• Jira Ticket: HCR-420

#️⃣관련 이슈

• closes [HSC-420] v1.7.3 #272

[gemini-code-assist]

Code Review

이번 PR은 Kafka 메시지 페이로드에 traceId를 추가하고, 컨슈머의 로깅 체계를 개선하여 처리 단계별 실행 시간과 트레이스 정보를 기록하도록 변경했습니다. 리뷰 과정에서 RecommendationKafkaConsumer의 프로필 격리(@Profile) 누락, 인프라 레이어에서의 직접적인 리포지토리 호출 및 트랜잭션 부재, 그리고 에러 로그 내 원본 페이로드 누락 등 아키텍처 및 운영 관점의 개선 사항들이 지적되었습니다.

I am having trouble creating individual review comments. Click here to see my feedback.

src/main/java/site/holliverse/infra/kafka/consumer/RecommendationKafkaConsumer.java (28)

RecommendationKafkaConsumer 클래스에 @Profile("customer") 어노테이션이 누락되었습니다. 이 클래스는 customer 프로필에서만 활성화되는 PersonaRecommendationRepository 및 CustomerMetrics 등에 의존하고 있습니다. 프로젝트의 ECS 서비스 분리 구조(Rule 1.13-16) 및 프로필 격리 원칙(Rule 2.35)에 따라, admin 프로필 환경에서 애플리케이션 시작 시 빈 주입 실패(NoSuchBeanDefinitionException)가 발생할 위험이 큽니다.

References

1. Profile Isolation (BLOCKER): Customer/Admin 서비스 간의 빈 격리를 위해 적절한 @Profile 설정이 필수적입니다. ^(link)

src/main/java/site/holliverse/infra/kafka/consumer/RecommendationKafkaConsumer.java (105-108)

인프라 레이어의 Kafka Consumer가 직접 Repository를 호출하여 엔티티를 조회하고 저장하는 비즈니스 오케스트레이션을 수행하고 있습니다. 이는 Application Layer(UseCase)가 트랜잭션 경계와 도메인 로직 흐름을 제어해야 한다는 아키텍처 원칙(Rule 3.42)에 어긋납니다. 또한 현재 consume 메서드에 @Transactional이 적용되어 있지 않아 upsert 로직의 원자성이 보장되지 않습니다. 해당 로직을 UseCase로 이동하여 처리할 것을 권장합니다.

References

1. Application Layer (UseCase) Responsibilities: Transaction boundaries, Domain orchestration. ^(link)

src/main/java/site/holliverse/infra/kafka/consumer/RecommendationKafkaConsumer.java (139-147)

catch 블록의 에러 로그에서 원본 페이로드(payload) 기록이 누락되었습니다. 이전 코드에서는 raw={}를 통해 수신된 메시지 내용을 남겼으나, 로그 포맷 변경 과정에서 제외되었습니다. JSON 파싱 오류나 데이터 정합성 문제 발생 시 정확한 원인 파악을 위해 에러 로그에는 원본 페이로드를 포함하는 것이 좋습니다.

"[REC][trace_id={}][member_id={}] stage={} elapsed_ms={} topic={} offset={} error={} raw={}",
                    traceId,
                    memberIdForCleanup != null ? memberIdForCleanup : "unknown",
                    "consume_failed",
                    elapsedMs(consumeStartedAt),
                    topic,
                    offset,
                    e.getMessage(),
                    payload,
                    e

[github-actions]

🧪 Test Coverage Report (JaCoCo)

기준(soft, workflow는 실패 안 함): Overall line 70% / branch 50%, Changed line 80% / branch 60% · Generated: 2026-03-31 04:16 UTC

Scope	Line	Branch	Line Graph	Branch Graph	Verdict
Overall	40.6%	29.4%	████████░░░░░░░░░░░░	██████░░░░░░░░░░░░░░	⚠️⚠️
Changed	18.3%	8.1%	████░░░░░░░░░░░░░░░░	██░░░░░░░░░░░░░░░░░░	⚠️⚠️

Package line spark (sample): ▃▂▁▂▂▄▄·▇█▁▆▆▆·▆▁▁▆█▇▃▁▁▁██▁▁▃
Package branch spark (sample): ▃▁·▁▁·▁·▇··▅▆▅·▅▁▁▅▇▇▂▁·▁·█▁▁▁

📦 Package coverage (worst 10)

Rank	Package	Line	Branch	Lines	Line Graph	Branch Graph
1	site.holliverse.shared.monitoring	0.0%	N/A	76	░░░░░░░░░░░░░░░░░░░░	░░░░░░░░░░░░░░░░░░░░
2	site.holliverse.customer.application.usecase.log	0.0%	0.0%	69	░░░░░░░░░░░░░░░░░░░░	░░░░░░░░░░░░░░░░░░░░
3	site.holliverse.customer.coupon.application	0.0%	0.0%	61	░░░░░░░░░░░░░░░░░░░░	░░░░░░░░░░░░░░░░░░░░
4	site.holliverse.infra.kafka.consumer	0.0%	0.0%	44	░░░░░░░░░░░░░░░░░░░░	░░░░░░░░░░░░░░░░░░░░
5	site.holliverse.customer.integration.fastapi	0.0%	0.0%	28	░░░░░░░░░░░░░░░░░░░░	░░░░░░░░░░░░░░░░░░░░
6	site.holliverse.customer.integration.external	0.0%	0.0%	28	░░░░░░░░░░░░░░░░░░░░	░░░░░░░░░░░░░░░░░░░░
7	site.holliverse.customer.application.usecase.counsel	0.0%	0.0%	14	░░░░░░░░░░░░░░░░░░░░	░░░░░░░░░░░░░░░░░░░░
8	site.holliverse.customer.coupon.web	0.0%	0.0%	12	░░░░░░░░░░░░░░░░░░░░	░░░░░░░░░░░░░░░░░░░░
9	site.holliverse.shared.config.runtime	0.0%	N/A	11	░░░░░░░░░░░░░░░░░░░░	░░░░░░░░░░░░░░░░░░░░
10	site.holliverse.customer.web.util	0.0%	0.0%	8	░░░░░░░░░░░░░░░░░░░░	░░░░░░░░░░░░░░░░░░░░

🧨 Lowest coverage classes (worst 10)

Rank	Class	Line	Branch	Lines	Line Graph	Branch Graph
1	site.holliverse.customer.application.usecase.log.UserLogService	0.0%	0.0%	52	░░░░░░░░░░░░░░░░░░░░	░░░░░░░░░░░░░░░░░░░░
2	site.holliverse.shared.monitoring.CustomerMetrics	0.0%	N/A	50	░░░░░░░░░░░░░░░░░░░░	░░░░░░░░░░░░░░░░░░░░
3	site.holliverse.infra.kafka.consumer.RecommendationKafkaConsumer	0.0%	0.0%	44	░░░░░░░░░░░░░░░░░░░░	░░░░░░░░░░░░░░░░░░░░
4	site.holliverse.customer.application.usecase.recommendation.RecommendationKafkaConsumeUseCase	0.0%	0.0%	42	░░░░░░░░░░░░░░░░░░░░	░░░░░░░░░░░░░░░░░░░░
5	site.holliverse.admin.application.usecase.CalculateLogChurnScoreService	0.0%	0.0%	38	░░░░░░░░░░░░░░░░░░░░	░░░░░░░░░░░░░░░░░░░░
6	site.holliverse.admin.application.usecase.CalculateChurnScoreService	0.0%	0.0%	37	░░░░░░░░░░░░░░░░░░░░	░░░░░░░░░░░░░░░░░░░░
7	site.holliverse.customer.coupon.application.UseMemberCouponUseCase	0.0%	0.0%	34	░░░░░░░░░░░░░░░░░░░░	░░░░░░░░░░░░░░░░░░░░
8	site.holliverse.admin.application.usecase.ChurnRiskReason$ReasonCode	0.0%	0.0%	31	░░░░░░░░░░░░░░░░░░░░	░░░░░░░░░░░░░░░░░░░░
9	site.holliverse.customer.integration.fastapi.FastApiRecommendationClient	0.0%	0.0%	28	░░░░░░░░░░░░░░░░░░░░	░░░░░░░░░░░░░░░░░░░░
10	site.holliverse.customer.integration.external.AdminLogFeaturesClient	0.0%	0.0%	28	░░░░░░░░░░░░░░░░░░░░	░░░░░░░░░░░░░░░░░░░░

🧩 Changed files coverage breakdown (12 files)

File	Line	Branch	Lines	Verdict	Line Graph	Branch Graph
src/main/java/site/holliverse/customer/application/usecase/log/UserLogService.java	0.0%	0.0%	52	⚠️⚠️	░░░░░░░░░░░░░░░░░░░░	░░░░░░░░░░░░░░░░░░░░
src/main/java/site/holliverse/customer/application/usecase/recommendation/RecommendationKafkaConsumeUseCase.java	0.0%	0.0%	42	⚠️⚠️	░░░░░░░░░░░░░░░░░░░░	░░░░░░░░░░░░░░░░░░░░
src/main/java/site/holliverse/customer/application/usecase/recommendation/RecommendationPendingFutureRegistry.java	0.0%	N/A	5	⚠️✅	░░░░░░░░░░░░░░░░░░░░	░░░░░░░░░░░░░░░░░░░░
src/main/java/site/holliverse/infra/kafka/consumer/RecommendationKafkaConsumer.java	0.0%	0.0%	44	⚠️⚠️	░░░░░░░░░░░░░░░░░░░░	░░░░░░░░░░░░░░░░░░░░
src/main/java/site/holliverse/customer/application/usecase/recommendation/RecommendationService.java	50.6%	25.0%	81	⚠️⚠️	██████████░░░░░░░░░░	█████░░░░░░░░░░░░░░░
src/main/java/site/holliverse/admin/config/SolapiConfig.java	N/A	N/A	0	✅✅	░░░░░░░░░░░░░░░░░░░░	░░░░░░░░░░░░░░░░░░░░
src/main/java/site/holliverse/customer/config/AdminLogFeaturesConfig.java	N/A	N/A	0	✅✅	░░░░░░░░░░░░░░░░░░░░	░░░░░░░░░░░░░░░░░░░░
src/main/java/site/holliverse/customer/config/FastApiConfig.java	N/A	N/A	0	✅✅	░░░░░░░░░░░░░░░░░░░░	░░░░░░░░░░░░░░░░░░░░
src/main/java/site/holliverse/customer/integration/kafka/dto/RecommendationMessagePayload.java	N/A	N/A	0	✅✅	░░░░░░░░░░░░░░░░░░░░	░░░░░░░░░░░░░░░░░░░░
src/main/java/site/holliverse/infra/kafka/config/RecommendationKafkaConfig.java	N/A	N/A	0	✅✅	░░░░░░░░░░░░░░░░░░░░	░░░░░░░░░░░░░░░░░░░░
src/main/java/site/holliverse/shared/monitoring/OperationMonitoringAspect.java	N/A	N/A	0	✅✅	░░░░░░░░░░░░░░░░░░░░	░░░░░░░░░░░░░░░░░░░░
src/main/java/site/holliverse/shared/monitoring/http/ObservedRestTemplateInterceptor.java	N/A	N/A	0	✅✅	░░░░░░░░░░░░░░░░░░░░	░░░░░░░░░░░░░░░░░░░░

---

🔎 HTML 리포트: Actions → Artifacts → jacoco-html · XML: jacoco-xml