[CHORE] 베드락 플로우 검증

[unifolio0]

closed #

작업 내용

스크린샷

참고 사항

[coderabbitai]

Caution

Review failed

The pull request is closed.

Summary by CodeRabbit

릴리스 노트

• Chores
  • 개발 워크플로우 자동화 설정을 최적화했습니다.
  • 내부 구성 관리를 간소화하고 서비스 안정성을 개선했습니다.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

Walkthrough

GitHub 워크플로우 트리거를 develop 브랜치만 포함하도록 축소하고, ResumeBasedQuestionBedrockService에서 구성을 상수로 인라인화하면서 조건부 흐름을 제거하여 항상 Bedrock 흐름을 먼저 시도하도록 변경했습니다.

Changes

Cohort / File(s)	변경 사항 요약
CI/CD 구성 \.github/workflows/cd-api-dev\.yml	워크플로우 트리거를 develop 브랜치로만 제한 (feat/#274 브랜치 제거)
Bedrock 서비스 리팩토링 api/src/main/java/com/samhap/kokomen/interview/external/ResumeBasedQuestionBedrockService\.java	flowId 및 flowAliasId를 private 상수로 변경하여 외부 주입 제거; isBedrockFlowConfigured() 조건 제거하고 항상 Bedrock 흐름을 우선 시도하도록 수정

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 깃허브 흐름은 깔끔하게,
상수로 묶인 설정들,
Bedrock이 먼저 나서고,
조건문은 물러나고—
코드는 더욱 명확해져라! ✨

✨ Finishing touches

• 📝 Generate docstrings

---

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

[gemini-code-assist]

Summary of Changes

Hello @unifolio0, I'm Gemini Code Assist¹! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

이 PR은 ResumeBasedQuestionBedrockService 내 Bedrock 플로우 구성을 리팩토링합니다. 기존에 @Value 어노테이션을 통해 주입받던 플로우 ID를 정적 상수로 하드코딩하여 서비스의 복잡성을 줄이고, Bedrock 플로우 구성 여부를 확인하던 조건부 로직을 제거했습니다. 이는 코드의 명확성을 높이고 설정 관리를 단순화합니다.

Highlights

• Bedrock 플로우 ID 설정 방식 변경: ResumeBasedQuestionBedrockService에서 Bedrock 플로우 ID와 Alias ID를 @Value 어노테이션을 통해 주입받는 대신, 정적(static final) 상수로 직접 정의하도록 변경되었습니다.
• 생성자 및 필드 간소화: 플로우 ID와 Alias ID가 정적 상수로 변경됨에 따라, 서비스의 생성자에서 해당 값들을 매개변수로 받던 부분이 제거되었고, 관련 인스턴스 필드도 삭제되었습니다.
• 조건부 로직 제거: Bedrock 플로우가 설정되었는지 확인하던 isBedrockFlowConfigured() 메서드와 이를 기반으로 GPT 또는 Bedrock을 선택적으로 호출하던 조건부 로직이 제거되었습니다. 이제 Bedrock 플로우가 항상 사용되는 것으로 가정합니다.
• InvokeFlowRequest 빌드 방식 업데이트: InvokeFlowRequest를 생성할 때, 기존의 인스턴스 필드 대신 새로 정의된 정적 상수 FLOW_ID와 FLOW_ALIAS_ID를 직접 사용하도록 수정되었습니다.

🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console.

Ignored Files

• Ignored by pattern: .github/workflows/** (1)
  • .github/workflows/cd-api-dev.yml

Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature	Command	Description
Code Review	/gemini review	Performs a code review for the current pull request in its current state.
Pull Request Summary	/gemini summary	Provides a summary of the current pull request in its current state.
Comment	@gemini-code-assist	Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help	/gemini help	Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

---

값은 이제 고정, 흐름은 명확해졌네, 코드 더욱 깔끔.

Footnotes

1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution. ↩

[gemini-code-assist]

Code Review

이 PR은 Bedrock Flow ID와 Alias ID를 기존의 @Value를 통한 프로퍼티 주입 방식에서 클래스 내 상수로 하드코딩하는 방식으로 변경했습니다. 이로 인해 설정 유무를 확인하던 로직도 함께 제거되었습니다.

리뷰 결과, 설정 값을 하드코딩하는 것은 여러 환경(개발, 운영 등)에서 유연하게 대처하기 어렵게 만들고 유지보수성을 저해하므로 중대한(critical) 이슈로 판단됩니다. 따라서, 원래대로 외부 설정 파일에서 값을 주입받는 방식으로 되돌리는 것을 권장합니다. 이는 애플리케이션의 유연성과 확장성을 위해 필수적입니다.

[gemini-code-assist]

베드락 Flow ID와 Alias ID를 상수로 하드코딩하면 각 환경(개발, 프로덕션 등)에 따라 다른 값을 사용하기 어렵습니다. 이로 인해 관련 로직(isBedrockFlowConfigured 등)이 삭제되었는데, 이는 유연성을 해치는 변경입니다.

이 값들은 외부 설정(application.yml)에서 @Value를 통해 주입받도록 하는 것이 좋습니다. 이렇게 하면 코드 변경 없이 환경별로 다른 Flow를 사용하도록 유연하게 설정할 수 있습니다.

이전 구현처럼 @Value를 사용해 flowId와 flowAliasId를 주입받고, 해당 값들이 설정되지 않았을 때 GPT로 fallback하는 로직을 복원하는 것을 강력히 권장합니다. 만약 테스트를 위한 임시 변경이었다면, 이 PR이 병합되기 전에 원래 코드로 복구해야 합니다.