-
-
Notifications
You must be signed in to change notification settings - Fork 114
[Platform] Add Perplexity #388
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
2c703eb to
c3452f3
Compare
|
Looks a bit like you started off with an old base branch? |
c3452f3 to
9a26b07
Compare
9a26b07 to
0cc04c4
Compare
|
While working on Perplexity specific features integration, I'm wondering how could we handle Related Perplexity API reference: https://docs.perplexity.ai/api-reference/chat-completions-post#response-search-results It's very specific to Perplexity platform, as they perform real web search internally (without leveraging MCP tools). I see 2 possible solutions:
I'd prefer the 1st one. WDYT @chr-hertel @OskarStark |
80fae92 to
6e9b56d
Compare
|
Let's go with Metadata first and see if there are similar things in other platforms before creating a new kind of result. It is still text right, but they add the search_results and citations to provide some extra information to the text that was generated, right? |
|
Yes I would prefer the metadata option too in the first version |
|
It looks like this: |
c4af78a to
fe54779
Compare
bac56dc to
0595bc0
Compare
OskarStark
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, but we are lacking tests
|
From a functional point of view this looks good: After addressing Oskar's comments we're good to merge here - finally Perplexity - thanks already @welcoMattic! |
0595bc0 to
b33bedc
Compare
|
@OskarStark @chr-hertel comments addressed, tests added, ready for a final review and merge! |
|
Just some minor, afterwards good to merge |
b33bedc to
b3bd2b2
Compare
cc7f142 to
f431dcc
Compare
f431dcc to
5e7f2f2
Compare
|
Thanks Mathieu for working on this feature, this is much appreciated. |

This PR adds Perplexity as a supported Platform.
Models are: sonar, sonar-pro, sonar-reasoning, sonar-reasoning-pro, and sonar-deep-research
API is similar to OpenAI API, with additional features I will take care of in coming days.
Perplexity docs: