apunkt/plano
1
0
Fork 0
mirror of https://github.com/katanemo/plano.git synced 2026-07-11 16:12:13 +02:00
Code Issues Projects Releases Packages Wiki Activity

fix(request_adapter): ensure normalization of ChatGPT responses request

Browse source
This commit is contained in:
Tom Stoffer 2026-05-25 21:28:06 +12:00
parent 18779d493e
commit 2656af89bf
2 changed files with 7 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

6
crates/hermesllm/src/providers/request.rs
View file

@ -1025,7 +1025,7 @@ mod tests {
input: InputParam::Text("Hello, Codex!".to_string()), input: InputParam::Text("Hello, Codex!".to_string()),
temperature: None, temperature: None,
max_output_tokens: Some(8192), max_output_tokens: Some(8192),
stream: Some(false), stream: None,
metadata: None, metadata: None,
tools: None, tools: None,
tool_choice: None, tool_choice: None,
@ -1051,7 +1051,9 @@ mod tests {
let upstream_api = SupportedUpstreamAPIs::OpenAIResponsesAPI(Responses); let upstream_api = SupportedUpstreamAPIs::OpenAIResponsesAPI(Responses);
let mut request = ProviderRequestType::ResponsesAPIRequest(responses_req); let mut request = ProviderRequestType::ResponsesAPIRequest(responses_req);
request.normalize_for_upstream(ProviderId::ChatGPT, &upstream_api); request
.normalize_for_upstream(ProviderId::ChatGPT, &upstream_api)
.expect("ChatGPT responses request should normalize");
match request { match request {
ProviderRequestType::ResponsesAPIRequest(req) => { ProviderRequestType::ResponsesAPIRequest(req) => {

4
crates/hermesllm/src/providers/request_adapter.rs
View file

@ -613,7 +613,9 @@ mod tests {
let mut request = ProviderRequestType::ResponsesAPIRequest(req); let mut request = ProviderRequestType::ResponsesAPIRequest(req);
// normalize_for_upstream sets store=false, stream=true, wraps input in Items // normalize_for_upstream sets store=false, stream=true, wraps input in Items
request.normalize_for_upstream(ProviderId::ChatGPT, &upstream_api); request
.normalize_for_upstream(ProviderId::ChatGPT, &upstream_api)
.expect("ChatGPT responses request should normalize");
// serialize_for_upstream then renames max_output_tokens and flattens content // serialize_for_upstream then renames max_output_tokens and flattens content
let bytes = serialize_for_upstream(&request, ProviderId::ChatGPT).unwrap(); let bytes = serialize_for_upstream(&request, ProviderId::ChatGPT).unwrap();