fix(opencode): resolve Perplexity Sonar 400 error - enforce message alternation #9418
+174
−36
Conversation
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
…ation - Add ensureMessageAlternation() to merge consecutive user messages - Add mergeMessageContent() for safe content combination - Apply fix only to Perplexity providers (perplexity providerID or @ai-sdk/perplexity npm) - Preserve full backward compatibility with all other providers - Add comprehensive test case to prevent regression - Addresses: 'After the (optional) system message(s), user and assistant roles should be alternating' Fixes issue where tools returning attachments created consecutive user messages, causing Perplexity API to reject requests with 400 Bad Request error.
Contributor
|
The following comment was made by an LLM, it may be inaccurate: Based on my search results, I found one potentially related PR: Related PR Found:
This PR might be related as it also addresses the issue of consecutive user messages, which is the core problem your fix addresses for Perplexity Sonar. However, it appears to be an older PR that may have addressed a similar issue differently. The other results (Perplexity websearch integration and TUI layout system) are unrelated to your message alternation fix. |
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.
Problem
Perplexity Sonar models reject requests with 400 error when consecutive user messages are sent (violates alternation requirement).
Solution
Merge consecutive user messages for Perplexity providers only. Preserves all content with
\n\nseparator.Testing
Fixes #9416