Skip to content

refactor/41 - temp: feign 임시 처리 - #42

Merged
ji-circle merged 1 commit into
devfrom
refactor/41-feign-reserve
May 19, 2026
Merged

ji-circle merged 1 commit into
devfrom
refactor/41-feign-reserve

Conversation

@ji-circle

@ji-circle ji-circle commented May 19, 2026

Copy link
Copy Markdown
Contributor

📝 작업 내용

이번 PR에서 작업한 내용을 설명해주세요.

  • feign 임시 처리

🚀 주요 변경 사항

완료한 이슈 번호
Close #41
관련된 이슈 번호 (닫고 싶지 않은 경우)
Related to #

✅ 자체 체크리스트 (필수)

  • ./gradlew build 실행 결과 정상 (인증샷 첨부)
  • 팀 내 컨벤션 준수 및 불필요한 로그, import 제거
  • 중요한 변경 사항이 팀에 공유되었는지

📸 테스트 인증샷

빌드 결과 및 IntelliJ HTTP Client 실행 화면을 여기에 첨부해 주세요.

💬 리뷰어 전달사항 (선택)

특별히 봐주었으면 하는 부분이나 논의가 필요한 점을 적어주세요.

  • 추후 수정 예정입니다


📎 참고 자료

관련 문서, 레퍼런스 링크 등이 있다면 여기에 첨부해주세요.

Summary by CodeRabbit

릴리스 노트

  • 기타
    • 예약 유효성 검증이 성능 테스트 환경뿐 아니라 프로덕션 환경에서도 활성화되었습니다.

Review Change Stack

@ji-circle ji-circle added this to the 8. 기타 milestone May 19, 2026
@ji-circle ji-circle added enhancement New feature or request question Further information is requested labels May 19, 2026
@ji-circle ji-circle linked an issue May 19, 2026 that may be closed by this pull request
1 task
@coderabbitai

coderabbitai Bot commented May 19, 2026

Copy link
Copy Markdown
📝 Walkthrough

전체 요약

PerfReservationValidationAdapter 클래스의 Spring @Profile 어노테이션이 "perf"에서 {"perf", "prod"}로 확장되어 perf와 prod 환경 모두에서 해당 어댑터가 빈으로 등록될 수 있도록 수정되었습니다.

변경 사항

어댑터 프로필 범위 확장

레이어 / 파일 요약
프로필 활성화 조건 확장
src/main/java/com/michelet/order/infrastructure/adapter/PerfReservationValidationAdapter.java
@Profile 어노테이션이 "perf"에서 {"perf", "prod"}로 변경되어 prod 환경에서도 예약 검증 어댑터가 활성화됩니다.

예상 코드 리뷰 난이도

🎯 1 (Trivial) | ⏱️ ~2분

제안 검토자

  • Sehi55
  • qldo
  • Jinyoung-Kim96

마무리 시

🐰 프로필이 확장되어
perf도, prod도 함께
어댑터는 깨어나고
예약은 검증되고
모두 행복해진다! 🎉

🚥 Pre-merge checks | ✅ 2 | ❌ 3

❌ Failed checks (1 warning, 2 inconclusive)

Check name Status Explanation Resolution
Out of Scope Changes check ⚠️ Warning PerfReservationValidationAdapter의 @Profile 범위 확장은 이슈 #41의 'feign 연동 수정'과 직접적인 관련성이 없어 범위 외 변경으로 보입니다. 변경 사항이 이슈 #41의 범위에 포함되는지 확인하고, feign 연동 수정과의 관련성을 명시하세요.
Title check ❓ Inconclusive PR 제목이 'feign 임시 처리'로 변경 사항과 부분적으로 관련있지만, 실제 변경은 PerfReservationValidationAdapter의 @Profile 범위 확장으로 feign과 직접적인 관련성이 명확하지 않습니다. PR 제목을 변경 사항을 정확히 반영하도록 수정하세요. 예: 'PerfReservationValidationAdapter의 프로필 범위를 prod 환경으로 확장' 또는 더 구체적인 설명 필요.
Linked Issues check ❓ Inconclusive 연결된 이슈 #41은 'feign 연동 수정'을 목표로 하지만, 실제 변경 사항은 PerfReservationValidationAdapter의 @Profile 어노테이션 변경으로 feign 연동과의 직접적인 관련성이 불명확합니다. PR 변경 사항이 feign 연동 수정과 어떻게 연결되는지 명확히 문서화하거나, 변경 목적을 재검토하세요.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ 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 refactor/41-feign-reserve

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

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

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In
`@src/main/java/com/michelet/order/infrastructure/adapter/PerfReservationValidationAdapter.java`:
- Line 13: PerfReservationValidationAdapter is annotated with `@Profile`({"perf",
"prod"}) which causes it to override DefaultReservationValidationAdapter in prod
and disable real validation; remove "prod" from the `@Profile` on
PerfReservationValidationAdapter or alternatively make
DefaultReservationValidationAdapter take precedence (e.g. remove its negated
profile, mark it `@Primary` or use `@ConditionalOnMissingBean`) so that in prod the
real validation bean (DefaultReservationValidationAdapter) is registered instead
of the perf stub; locate the annotations on PerfReservationValidationAdapter and
DefaultReservationValidationAdapter and adjust the profiles/bean precedence
accordingly.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: e43b7cfd-cea3-49db-898f-3128eeb156f1

📥 Commits

Reviewing files that changed from the base of the PR and between e25ab3c and 37a5aa5.

📒 Files selected for processing (1)
  • src/main/java/com/michelet/order/infrastructure/adapter/PerfReservationValidationAdapter.java

@ji-circle
ji-circle merged commit bc1d0ee into dev May 19, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request question Further information is requested

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[REFACTOR] feign 연동 수정

1 participant