Skip to content

refactor: normalize AI request messages#2403

Open
MGMCN wants to merge 1 commit into
The-PR-Agent:mainfrom
MGMCN:refactor/normalize-ai-request-messages
Open

refactor: normalize AI request messages#2403
MGMCN wants to merge 1 commit into
The-PR-Agent:mainfrom
MGMCN:refactor/normalize-ai-request-messages

Conversation

@MGMCN
Copy link
Copy Markdown

@MGMCN MGMCN commented May 19, 2026

Normalize AI request messages before constructing common kwargs to support user-message-only models and remove duplicated request parameter definitions.

@qodo-free-for-open-source-projects
Copy link
Copy Markdown
Contributor

Review Summary by Qodo

Consolidate AI request kwargs construction after message normalization

✨ Enhancement

Grey Divider

Walkthroughs

Description
• Consolidate duplicated kwargs construction logic
• Support user-message-only models with normalized messages
• Remove conditional branching for cleaner code structure
Diagram
flowchart LR
  A["Check model type"] --> B["Normalize messages<br/>for target model"]
  B --> C["Build kwargs once<br/>with normalized messages"]
  D["Remove duplicate<br/>kwargs definitions"] -.-> C
Loading

Grey Divider

File Changes

1. pr_agent/algo/ai_handlers/litellm_ai_handler.py Refactoring +3/-9

Unify kwargs construction after message normalization

• Removed conditional branching that duplicated kwargs construction
• Moved kwargs building after message normalization logic
• Unified single kwargs definition for all model types
• Added clarifying comment about message normalization timing

pr_agent/algo/ai_handlers/litellm_ai_handler.py


Grey Divider

Qodo Logo

@qodo-free-for-open-source-projects
Copy link
Copy Markdown
Contributor

qodo-free-for-open-source-projects Bot commented May 19, 2026

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

Qodo Logo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant