KusionStack / karpor / 42347965157

66%

Build Type

push

github

Committed by web-flow

Commit Message

fix(ai): pass the configured top-p to the Azure OpenAI backend (#874)

`--ai-top-p` is ignored by the Azure OpenAI backend: `AzureAIClient`
never stores `cfg.TopP` and builds its requests (`Generate` and
`GenerateStream`) with only `Temperature`. The `openai` and `deepseek`
backends already send it.

**Fix:** store `topP` in `Configure` and build both requests through a
shared `newRequest`, which follows the OpenAI client: `top_p` is left
out when it has the API default (1), so reasoning deployments that only
accept default sampling values keep working with the default flag value.

The `huggingface` backend also ignores `--ai-top-p` and is left
unchanged here, since its API may not take the parameter in the same
way.

**Test:** `azureopenai_test.go` checks the request body sent by
`Generate`: a custom `top_p` is sent (red on main: missing) and the
default is omitted. `go test ./pkg/infra/ai/` is green.

---
This change was prepared with the help of an AI coding assistant (Claude
Code) and submitted by Alex Castillo González.

Coverage Stats

13 of 13 new or added lines in 1 file covered. (100.0%)

2 existing lines in 1 file now uncovered.

4525 of 6897 relevant lines covered (65.61%)

0.72 hits per line