Skip to content

fix(llm): allow default_query/custom_headers for Azure Kimi#1223

Open
kingdomseed wants to merge 5 commits intoMoonshotAI:mainfrom
kingdomseed:fix/azure-kimi-default-query
Open

fix(llm): allow default_query/custom_headers for Azure Kimi#1223
kingdomseed wants to merge 5 commits intoMoonshotAI:mainfrom
kingdomseed:fix/azure-kimi-default-query

Conversation

@kingdomseed
Copy link

@kingdomseed kingdomseed commented Feb 24, 2026

Summary

  • Enable passing default_query into OpenAI client initialization for OpenAI-compatible providers.
  • Forward custom_headers to the OpenAI client.
  • Add tests verifying OpenAI legacy/responses providers receive default_query and custom_headers.

Motivation

This enables Kimi K2.5 deployments on Azure by allowing the required fixed query parameter (e.g., api-version) and any required headers to be configured without introducing new provider types or large changes. The scope is focused on Kimi K2.5 on Azure deployments (OpenAI‑compatible chat completions).

Testing

  • make prepare (passed)
  • prek run --all-files (passed)
  • uv run pytest -q (passed: 497 passed, 26 skipped)

Related


Open with Devin

Copilot AI review requested due to automatic review settings February 24, 2026 09:02
Copy link
Contributor

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 2 additional findings.

Open in Devin Review

This comment was marked as resolved.

@kingdomseed
Copy link
Author

Tightened default_query to JSON-serializable scalars to avoid config persistence failures.

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