-
Notifications
You must be signed in to change notification settings - Fork 133
feat: add vercel ai sdk apify mcp integration #2021
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
|
Preview for this PR was built for commit |
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.
Thanks, looks good to me. I think @TC-MO will handle the English and polish it a bit.
IMO I would go out with that to create and create PR at Vercel and we can polish it later
Co-authored-by: Michał Olender <[email protected]>
Co-authored-by: Michał Olender <[email protected]>
Co-authored-by: Michał Olender <[email protected]>
Co-authored-by: Michał Olender <[email protected]>
Co-authored-by: Michał Olender <[email protected]>
|
Preview for this PR was built for commit |
|
Preview for this PR was built for commit |
1 similar comment
|
Preview for this PR was built for commit |
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, one note or rather more of a question
|
|
||
| :::tip Single token | ||
|
|
||
| By using the [Apify OpenRouter](https://apify.com/apify/openrouter) you don't need to provide a separate API key for OpenRouter or any other LLM provider. Only your Apify token is needed. All token costs go to your Apify account. |
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.
I know I changed requested this change, but I'm not sure 100% about that last sentence.
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.
I think it does the job, we can improve that later on 👍
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.
Yeah, that is true, well then, merge away it;s all good from my side
related to #1872