Merged
Conversation
…Runtime pattern - Add fallback chain with priority strategy + retry logic to demo API route - Add fallback toggle UI in sidebar showing chain order - Update docs: createTogetherAI + createRuntime as primary pattern, fallback chain section, cleaned up model list - Add REST test script for all models - Bump llm-sdk to 2.1.9 Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…dler, bump v2.5.0 The post-message:end done handler was correctly skipping server-side assistant messages (to avoid duplicate tool cards), but was still inserting the corresponding tool result messages (role: "tool"). These orphaned tool messages caused: 1. Duplicate "Reading from" skill cards during streaming 2. 400 API errors on follow-up requests (OpenAI rejects role:"tool" without a preceding tool_calls assistant message) Fix: skip all tool result messages in the done handler since server-side results are already represented in metadata.toolExecutions on the finalized assistant message. Also bumps all package versions to 2.5.0 for release. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
ProviderId type includes "yourgpt-server" but the envVarNames Record<ProviderId, string> was missing it, causing a TS build error. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This was an internal-only provider that proxied to a local yourgpt-server-demo (which is gitignored). Removing it to avoid type mismatches and keep the playground clean for public use. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Rohitjoshi9023
approved these changes
Apr 15, 2026
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
2 participants
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.
Summary
createRuntime+CopilotProviderpatternBug Fix Details
The
donehandler inAbstractChatwas correctly skipping server-side assistant messages (to avoid duplicate tool cards), but was still inserting the corresponding tool result messages (role: "tool"). These orphaned messages caused:400 Invalid parametererrors on follow-up requests (OpenAI rejectsrole: "tool"without a precedingtool_callsmessage)Test plan
🤖 Generated with Claude Code