fix: include api.moonshot.cn in public API temperature override (#12745)
kimi-k2.5 on api.moonshot.cn/v1 rejects temperature=0.6 with HTTP 400, same as api.moonshot.ai. The public API check now matches both domains.
This commit is contained in:
@@ -155,7 +155,7 @@ def _fixed_temperature_for_model(
|
|||||||
|
|
||||||
# Public Moonshot API has a stricter contract for some models than the
|
# Public Moonshot API has a stricter contract for some models than the
|
||||||
# Coding Plan endpoint — check it first so it wins on conflict.
|
# Coding Plan endpoint — check it first so it wins on conflict.
|
||||||
if base_url and "api.moonshot.ai" in base_url.lower():
|
if base_url and ("api.moonshot.ai" in base_url.lower() or "api.moonshot.cn" in base_url.lower()):
|
||||||
public = _KIMI_PUBLIC_API_OVERRIDES.get(bare)
|
public = _KIMI_PUBLIC_API_OVERRIDES.get(bare)
|
||||||
if public is not None:
|
if public is not None:
|
||||||
logger.debug(
|
logger.debug(
|
||||||
|
|||||||
@@ -845,6 +845,8 @@ class TestKimiForCodingTemperature:
|
|||||||
"https://api.moonshot.ai/v1",
|
"https://api.moonshot.ai/v1",
|
||||||
"https://api.moonshot.ai/v1/",
|
"https://api.moonshot.ai/v1/",
|
||||||
"https://API.MOONSHOT.AI/v1",
|
"https://API.MOONSHOT.AI/v1",
|
||||||
|
"https://api.moonshot.cn/v1",
|
||||||
|
"https://api.moonshot.cn/v1/",
|
||||||
],
|
],
|
||||||
)
|
)
|
||||||
def test_kimi_k2_5_public_api_forces_temperature_1(self, base_url):
|
def test_kimi_k2_5_public_api_forces_temperature_1(self, base_url):
|
||||||
|
|||||||
Reference in New Issue
Block a user