Skip to content

Fix invalid input_text type when sending model messages back to OpenAI Responses API#184

Merged
felixarntz merged 3 commits intotrunkfrom
fix/182-openai-invalid-model-text-type
Jan 26, 2026
Merged

Fix invalid input_text type when sending model messages back to OpenAI Responses API#184
felixarntz merged 3 commits intotrunkfrom
fix/182-openai-invalid-model-text-type

Conversation

@felixarntz
Copy link
Copy Markdown
Member

Fixes #182

Supersedes #183

I've added a basic integration test for multi-turn messages, for the other providers too, just to make sure it works and is covered.

You can verify that this fixes the bug by temporarily changing the output_text string in my changed line back to input_text - this would lead to the original behavior before this PR, and you'll get the exact exception that #182 mentions.

@felixarntz felixarntz added this to the 0.4.1 milestone Jan 25, 2026
@felixarntz felixarntz added the [Type] Bug An existing feature does not function as intended label Jan 25, 2026
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jan 25, 2026

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: felixarntz <flixos90@git.wordpress.org>
Co-authored-by: JasonTheAdams <jason_the_adams@git.wordpress.org>
Co-authored-by: jmarx <jmarx75@git.wordpress.org>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

Copy link
Copy Markdown
Member

@JasonTheAdams JasonTheAdams left a comment

Choose a reason for hiding this comment

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

Great catch! I'm feeling better and better about these integration tests.

@felixarntz felixarntz merged commit 1c05280 into trunk Jan 26, 2026
6 checks passed
@felixarntz felixarntz deleted the fix/182-openai-invalid-model-text-type branch January 26, 2026 19:52
@jmarx
Copy link
Copy Markdown

jmarx commented Jan 26, 2026

@felixarntz Thanks! I actually have not had a chance to test your version. But I assume it works :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Type] Bug An existing feature does not function as intended

3 participants