Skip to content

[Fix/#222] Gemini thinkingLevel을 minimal로 낮춰 응답 지연·타임아웃 완화 - #223

Merged
tnals0924 merged 1 commit into
mainfrom
fix/#222-gemini-thinking-level-minimal
Jul 17, 2026
Merged

[Fix/#222] Gemini thinkingLevel을 minimal로 낮춰 응답 지연·타임아웃 완화#223
tnals0924 merged 1 commit into
mainfrom
fix/#222-gemini-thinking-level-minimal

Conversation

@tnals0924

Copy link
Copy Markdown
Contributor

#️⃣연관된 이슈

📝작업 내용

가끔 발생하는 GeminiRequestException: GenAiIOException: Failed to execute HTTP request. (60초 타임아웃성 실패) 빈도를 줄이기 위해 Gemini 3.x 사고 깊이를 lowminimal로 낮췄습니다.

  • GeminiProperties.thinkingLevel 기본값 minimal로 변경
  • application-infrastructure-client.ymlthinking-level 기본값 minimal로 변경
  • .env.exampleGEMINI_THINKING_LEVEL minimal로 동기화

💬리뷰 요구사항(선택)

  • 운영 서버 환경변수에 GEMINI_THINKING_LEVEL이 명시돼 있으면 그 값이 우선하므로, 배포 환경도 minimal로 바꾸거나 환경변수를 제거해 yml 기본값을 타도록 해야 합니다.

@coderabbitai

coderabbitai Bot commented Jul 17, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 9b57e6b3-a1ec-485a-9d38-247844b8e057

📥 Commits

Reviewing files that changed from the base of the PR and between 4b69c1f and 1cf4575.

📒 Files selected for processing (3)
  • .env.example
  • infrastructure/client/src/main/kotlin/kr/dongchimi/client/ai/GeminiProperties.kt
  • infrastructure/client/src/main/resources/application-infrastructure-client.yml

📝 Walkthrough

변경 사항

  • Gemini 3.x의 기본 thinkingLevellow에서 minimal로 변경했습니다.
  • AI 인프라 클라이언트의 프로퍼티, 애플리케이션 설정, 환경변수 예시를 동기화했습니다.
  • 응답 지연과 60초 타임아웃성 실패를 완화합니다.

Walkthrough

Gemini 사고 수준의 기본값을 low에서 minimal로 변경하고, Kotlin 속성·YAML 설정·환경 변수 예시를 동일하게 맞췄습니다.

Changes

Gemini 설정 기본값 동기화

Layer / File(s) Summary
Gemini 기본값 동기화
infrastructure/client/src/main/kotlin/.../GeminiProperties.kt, infrastructure/client/src/main/resources/application-infrastructure-client.yml, .env.example
thinkingLevel, gemini.thinking-level, GEMINI_THINKING_LEVEL의 기본값을 minimal로 변경했습니다.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

Suggested reviewers: imddoy

🚥 Pre-merge checks | ✅ 7
✅ Passed checks (7 passed)
Check name Status Explanation
Title check ✅ Passed 제목이 #222와 변경 내용(Gemini thinkingLevel을 minimal로 낮춤)을 정확히 반영합니다.
Description check ✅ Passed 필수 섹션인 연관 이슈와 작업 내용이 있고, Closes #222 및 리뷰 요구사항도 포함되어 있습니다.
Linked Issues check ✅ Passed #222의 요구사항인 세 기본값 minimal 동기화를 모두 반영했습니다.
Out of Scope Changes check ✅ Passed 요구 범위를 벗어난 변경은 보이지 않으며 설정값 동기화만 수행했습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
마이그레이션-엔티티 쌍 확인 ✅ Passed 변경 파일은 설정 3개뿐이며 JPA Entity나 Flyway 마이그레이션 SQL 변경은 없습니다.
레이어 의존성 방향 확인 ✅ Passed 변경은 .env.example과 Gemini 설정 파일뿐이라 Service 클래스의 Repository/JpaRepository 직접 주입 변경이 없습니다.

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.

@tnals0924
tnals0924 merged commit d686d78 into main Jul 17, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Gemini thinkingLevel을 minimal로 낮춰 응답 지연·타임아웃 완화

1 participant