Skip to content

fix(openai): handle compatible stream tool calls#85

Open
kevinn142857 wants to merge 1 commit into
OpenBMB:mainfrom
kevinn142857:codex/openai-stream-tool-call-compat
Open

fix(openai): handle compatible stream tool calls#85
kevinn142857 wants to merge 1 commit into
OpenBMB:mainfrom
kevinn142857:codex/openai-stream-tool-call-compat

Conversation

@kevinn142857
Copy link
Copy Markdown

Summary

  • accept OpenAI-compatible stream tool call variants with top-level name/function_name/tool_name and arguments/input fields
  • reject streamed tool calls that still lack a function name with a clear provider error instead of surfacing Tool does not exist.
  • add stream parser tests for standard OpenAI chunks, compatible variants, and missing names

Tests

  • npm run build
  • node --test --test-force-exit --test-timeout 60000 "dist/tests/**/*.test.js"

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.

1 participant