Skip to content

server-fix : accept text tool outputs in responses API#25073

Open
jesco-absolut wants to merge 1 commit into
ggml-org:masterfrom
jesco-absolut:fix-23557-responses-tool-output
Open

server-fix : accept text tool outputs in responses API#25073
jesco-absolut wants to merge 1 commit into
ggml-org:masterfrom
jesco-absolut:fix-23557-responses-tool-output

Conversation

@jesco-absolut

@jesco-absolut jesco-absolut commented Jun 27, 2026

Copy link
Copy Markdown

Overview

This PR fixes:

  1. Issue Responses API: function_call_output.output validation throws on valid client input #23557
  2. the function for call_output was too strict
  3. Only accepted type : "input text"
  4. Normalized to chat completions type : "text"

Also added a regression test in tests/test-chat.cpp which covers arrays containing output_text and text. You can run validation run by using uvx cmake --build build --target test-chat

Requirements

  • I have read and agree with the contributing guidelines
  • AI usage disclosure: : YES - used for local validation assistance; I manually reviewed the final changes.

@jesco-absolut jesco-absolut requested review from a team and pwilkin as code owners June 27, 2026 04:49
@github-actions github-actions Bot added testing Everything test related server labels Jun 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

server testing Everything test related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant