Skip to content

fix: include assistant tool call content#282

Open
C1-BA-B1-F3 wants to merge 1 commit into
OpenBMB:mainfrom
C1-BA-B1-F3:fix/issue-263
Open

fix: include assistant tool call content#282
C1-BA-B1-F3 wants to merge 1 commit into
OpenBMB:mainfrom
C1-BA-B1-F3:fix/issue-263

Conversation

@C1-BA-B1-F3

Copy link
Copy Markdown

Summary

  • Ensure OpenAI assistant messages with tool_calls include content, defaulting to an empty string when no text is present.
  • Add a focused OpenAI request serialization regression test.

Closes #263

Tests

  • npx tsc -p tsconfig.json --noEmit
  • node --import tsx --test tests/model/providers/openai/request.test.ts

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug(model): OpenAI-compatible streaming fails after tool calls when assistant content is omitted

2 participants