test: add e2e regression test for issue #190 (streamObject flush error)#373
Merged
robert-j-y merged 3 commits intomainfrom Jan 28, 2026
Merged
Conversation
Co-Authored-By: Robert Yeakel <robert.yeakel@openrouter.ai>
- Remove sparse array unit test from src/chat/index.test.ts - Add e2e test using streamObject + jsonSchema (matches issue exactly) - Test verifies streamObject works without TypeError in flush function Co-Authored-By: Robert Yeakel <robert.yeakel@openrouter.ai>
robert-j-y
changed the title
Keep only known facts from the issue report Co-Authored-By: Robert Yeakel <robert.yeakel@openrouter.ai>
robert-j-y
deleted the
devin/1769563160-issue-190-sparse-toolcalls-regression
branch
kesavan-byte
pushed a commit
to osm-API/ai-sdk-provider
that referenced
this pull request
Feb 13, 2026
…ect flush error) (OpenRouterTeam#373) * test: add regression test for issue OpenRouterTeam#190 (sparse tool call indices) Co-Authored-By: Robert Yeakel <robert.yeakel@openrouter.ai> * test: replace unit test with e2e regression test for issue OpenRouterTeam#190 - Remove sparse array unit test from src/chat/index.test.ts - Add e2e test using streamObject + jsonSchema (matches issue exactly) - Test verifies streamObject works without TypeError in flush function Co-Authored-By: Robert Yeakel <robert.yeakel@openrouter.ai> * test: remove speculative root cause from issue OpenRouterTeam#190 test comments Keep only known facts from the issue report Co-Authored-By: Robert Yeakel <robert.yeakel@openrouter.ai> --------- Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Adds an e2e regression test for GitHub issue #190, which reported a
TypeError: Cannot read properties of undefined (reading 'sent')when usingstreamObjectwith certain models.Test approach: Uses
streamObjectwithjsonSchemato match the exact reproduction case from the issue. The test verifies:Updates since last revision
Checklist
pnpm stylecheckandpnpm typecheckpnpm testand all tests passChangeset
pnpm changesetto create a changeset fileHuman Review Checklist
streamObject+jsonSchema)Link to Devin run: https://app.devin.ai/sessions/7f31d2bedf5e414d82b79125604de72b
Requested by: Robert Yeakel (@robert-j-y)