Skip to content

Nightly OSS: gemini-3-flash-preview openai compatible tool call error#3178

Open
xodn348 wants to merge 1 commit intoopenai:mainfrom
xodn348:nightly/2806-gemini-3-flash-preview-openai-compatible-tool-ca
Open

Nightly OSS: gemini-3-flash-preview openai compatible tool call error#3178
xodn348 wants to merge 1 commit intoopenai:mainfrom
xodn348:nightly/2806-gemini-3-flash-preview-openai-compatible-tool-ca

Conversation

@xodn348
Copy link
Copy Markdown

@xodn348 xodn348 commented Apr 29, 2026

Automated nightly Codex contribution for #2806.

Verification:

  • See the uploaded nightly artifacts for the Codex final report.
  • See this workflow run for exact commands and logs.

This PR is generated by xodn348/oss-nightly-control and is opened ready for maintainer review when the local nightly verification step succeeds.

… tool call error

Nightly Codex produced a focused contribution for openai#2806.

Constraint: Automated nightly run; keep changes small and reviewable.

Confidence: medium

Scope-risk: narrow

Tested: See uploaded nightly artifacts and workflow logs.

Not-tested: Maintainer CI beyond this workflow.
@xodn348 xodn348 requested a review from a team as a code owner April 29, 2026 15:24
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 46bd5d59e9

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +784 to +785
if index is None:
return fallback_index
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Derive missing tool-call index from stream state

When tool_call_delta.index is None, this helper falls back to the delta’s position within the current chunk, but many OpenAI-compatible streams emit one tool-call delta per chunk, so that fallback is repeatedly 0. In responses that contain multiple tool calls with missing indices, later tool-call argument chunks get merged into tool_calls[0], causing incorrect tool_calls.function.arguments.* events and a corrupted final tool-call snapshot instead of preserving separate calls.

Useful? React with 👍 / 👎.

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