Skip to content

Fix: Preserve index field in tool call responses#203

Merged
ashinzekene merged 1 commit intopoe-platform:mainfrom
ashinzekene:fix/tool-call-index-preservation
Dec 8, 2025
Merged

Fix: Preserve index field in tool call responses#203
ashinzekene merged 1 commit intopoe-platform:mainfrom
ashinzekene:fix/tool-call-index-preservation

Conversation

@ashinzekene
Copy link
Copy Markdown
Contributor

Description

Fixed an issue where tool call responses had index=None instead of preserving the index from the incoming message, which was inconsistent with other partial responses.

Changes Made

  • Fixed _stream_request_with_tools to preserve the index field when yielding tool call responses
  • Updated tool call response to pass through index field from incoming message (matching behavior of other partial responses)
  • Added test case to verify index preservation in tool call responses
  • Bumped version to 0.0.82

Testing Done

  • Verified fix works correctly with manual test
  • Existing tool call tests continue to pass
  • Added new test case test_stream_request_with_tools_index_preserved to verify index preservation

Version

  • Updated to version 0.0.82

Breaking Changes

  • None

Checklist

  • Tests added
  • Documentation updated (not needed for this bug fix)
  • Version bumped
  • Changes tested locally

@ashinzekene ashinzekene requested a review from a team as a code owner December 4, 2025 23:29
@ashinzekene ashinzekene force-pushed the fix/tool-call-index-preservation branch 2 times, most recently from 7cb04da to 8caab12 Compare December 4, 2025 23:54
@ashinzekene ashinzekene force-pushed the fix/tool-call-index-preservation branch from 8caab12 to 4c65373 Compare December 5, 2025 00:05
@ashinzekene ashinzekene merged commit 995b9dd into poe-platform:main Dec 8, 2025
14 checks passed
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