Skip to content

Feat(19)-Fixed minor bug#69

Merged
jae9104 merged 1 commit intomainfrom
Feat(19)-StructuralCorrections
Apr 19, 2026
Merged

Feat(19)-Fixed minor bug#69
jae9104 merged 1 commit intomainfrom
Feat(19)-StructuralCorrections

Conversation

@jae9104
Copy link
Copy Markdown
Collaborator

@jae9104 jae9104 commented Apr 19, 2026

No description provided.

Copilot AI review requested due to automatic review settings April 19, 2026 16:23
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the Google Gemini model endpoint used by AiAnalysisService to address a minor issue in AI analysis generation.

Changes:

  • Switched the Gemini generateContent model in the hard-coded API URL from gemini-3.1-flash-lite to gemini-2.5-flash.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

private final IRankHistoryService rankHistoryService;

private static final String GEMINI_API_URL = "https://generativelanguage.googleapis.com/v1beta/models/gemini-3.1-flash-lite:generateContent";
private static final String GEMINI_API_URL = "https://generativelanguage.googleapis.com/v1beta/models/gemini-2.5-flash:generateContent";
Copy link

Copilot AI Apr 19, 2026

Choose a reason for hiding this comment

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

The Gemini endpoint/model is hard-coded in a constant. Since model names can change and may differ across environments, consider moving the model (or full URL) to configuration (e.g., application.properties via @value) alongside the API key so updates don’t require a redeploy and can be validated per environment.

Copilot uses AI. Check for mistakes.
@jae9104 jae9104 merged commit 7150824 into main Apr 19, 2026
13 checks passed
@jae9104 jae9104 deleted the Feat(19)-StructuralCorrections branch April 19, 2026 17:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants