-
Notifications
You must be signed in to change notification settings - Fork 512
added aimlapi as provider #1752
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
base: main
Are you sure you want to change the base?
Conversation
@OctavianTheI is attempting to deploy a commit to the assistant-ui Team on Vercel. A member of the Team first needs to authorize it. |
LGTM 👍 |
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.
PR Summary
This PR integrates AI/ML API as a new provider across documentation and components, ensuring consistency with the existing naming conventions and assets.
• Updated apps/docs/content/docs/runtimes/ai-sdk/rsc.mdx to include AI/ML API among supported providers.
• Modified README.md to list AI/ML API in the supported model providers section.
• Added a new model entry and corresponding AIML API SVG asset in apps/docs/components/shadcn/ModelPicker.tsx.
• Updated use-chat.mdx and use-chat-hook.mdx to reflect AI/ML API integration.
• Enhanced apps/docs/content/docs/getting-started.mdx with AI/ML API setup instructions, noting a potential syntax issue with unquoted environment variable identifiers.
💡 (1/5) You can manually trigger the bot by mentioning @greptileai in a comment!
7 file(s) reviewed, 1 comment(s)
Edit PR Review Bot Settings | Greptile
</Tabs> | ||
|
||
Define environment variables: | ||
|
||
<Tabs id="provider" items={["OpenAI", "Anthropic", "Azure", "AWS", "Gemini", "GCP", "Groq", "Fireworks", "Cohere", "Ollama", "Chrome AI"]}> | ||
<Tabs id="provider" items={["OpenAI", "Anthropic", "Azure", "AWS", "Gemini", "GCP", "Groq", "Fireworks", "Cohere", "Ollama", "Chrome AI", AI/ML API]}> |
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.
syntax: Wrap 'AI/ML API' in quotes in the Tabs items for environment variables.
<Tabs id="provider" items={["OpenAI", "Anthropic", "Azure", "AWS", "Gemini", "GCP", "Groq", "Fireworks", "Cohere", "Ollama", "Chrome AI", AI/ML API]}> | |
<Tabs id="provider" items={["OpenAI", "Anthropic", "Azure", "AWS", "Gemini", "GCP", "Groq", "Fireworks", "Cohere", "Ollama", "Chrome AI", "AI/ML API"]}> |
📝 Documentation updates detected! You can review documentation updates here |
</Tabs> | ||
|
||
Define environment variables: | ||
|
||
<Tabs id="provider" items={["OpenAI", "Anthropic", "Azure", "AWS", "Gemini", "GCP", "Groq", "Fireworks", "Cohere", "Ollama", "Chrome AI"]}> | ||
<Tabs id="provider" items={["OpenAI", "Anthropic", "Azure", "AWS", "Gemini", "GCP", "Groq", "Fireworks", "Cohere", "Ollama", "Chrome AI", AI/ML API]}> |
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.
Wrap 'AI/ML API' in quotes in the Tabs items array to maintain valid syntax.
<Tabs id="provider" items={["OpenAI", "Anthropic", "Azure", "AWS", "Gemini", "GCP", "Groq", "Fireworks", "Cohere", "Ollama", "Chrome AI", AI/ML API]}> | |
<Tabs id="provider" items={["OpenAI", "Anthropic", "Azure", "AWS", "Gemini", "GCP", "Groq", "Fireworks", "Cohere", "Ollama", "Chrome AI", "AI/ML API"]}> |
WalkthroughThe update incorporates the "AI/ML API" integration into the assistant-ui library. Across documentation and code components, references to supported model providers have been revised to include this new API alongside existing services. In the ModelPicker component, a new model entry, "Llama 3.3 70B Instruct Turbo," is added with its corresponding icon. The changes extend to the getting-started documentation, where terminal commands, API endpoint implementations, and environment variable setups now account for the "AI/ML API" provider. Additionally, various sections in the runtime and hook documentation have been updated to reflect the expanded list of integrations. No modifications were made to the declarations of exported or public entities. 📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
🔇 Additional comments (5)
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
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.
Actionable comments posted: 1
📜 Review details
Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
Plan: Pro
⛔ Files ignored due to path filters (1)
apps/docs/assets/providers/aimlapi.svg
is excluded by!**/*.svg
📒 Files selected for processing (7)
README.md
(1 hunks)apps/docs/components/shadcn/ModelPicker.tsx
(2 hunks)apps/docs/content/docs/getting-started.mdx
(4 hunks)apps/docs/content/docs/runtimes/ai-sdk/rsc.mdx
(1 hunks)apps/docs/content/docs/runtimes/ai-sdk/use-chat-hook.mdx
(1 hunks)apps/docs/content/docs/runtimes/ai-sdk/use-chat.mdx
(1 hunks)packages/react/README.md
(1 hunks)
🔇 Additional comments (10)
apps/docs/content/docs/runtimes/ai-sdk/use-chat.mdx (1)
8-8
: Documentation updated with new provider support.The Vercel AI SDK UI's
useChat
hook documentation now includes "AI/ML API" in the list of supported integrations.apps/docs/content/docs/runtimes/ai-sdk/rsc.mdx (1)
8-8
: Documentation updated with new provider support.The Vercel AI SDK RSC integration documentation now includes "AI/ML API" in the supported integration list, maintaining consistency with other documentation changes.
packages/react/README.md (1)
26-26
: Documentation updated with new provider support.The packages/react README now includes "AI/ML API" in the list of supported model providers, keeping documentation consistent with the main README.
README.md (1)
26-26
: Documentation updated with new provider support.The main README now includes "AI/ML API" in the supported model providers list, reflecting the new integration.
apps/docs/components/shadcn/ModelPicker.tsx (2)
7-7
: New provider icon import added correctly.The import for the AI/ML API provider icon has been added appropriately.
52-56
:✅ Verification successful
New model added to the ModelPicker component.
The Llama 3.3 70B Instruct Turbo model has been successfully added to the ModelPicker with the correct provider icon.
🏁 Script executed:
#!/bin/bash # Verify that the aimlapi.svg file exists fd -t f "aimlapi.svg" assets/providersLength of output: 142
Please verify the location of the aimlapi provider icon.
The new "Llama 3.3 70B Instruct Turbo" model has been added to the ModelPicker as intended. However, our initial verification using the pathassets/providers
resulted in an error, suggesting that the directory might be incorrect or that the file exists elsewhere. Kindly ensure that the fileaimlapi.svg
is present in the correct location (or use a repository-wide search, such as the script above, to confirm its presence).
🏁 Script executed:
#!/bin/bash # Search for the aimlapi.svg file across the entire repository without restricting to a specific path. fd -t f "aimlapi.svg"Length of output: 182
New model integration verified in ModelPicker
- The "Llama 3.3 70B Instruct Turbo" entry is correctly added in the ModelPicker component.
- Verification confirmed that the
aimlapi.svg
file is present atapps/docs/assets/providers/aimlapi.svg
, ensuring that the correct provider icon is in use.apps/docs/content/docs/runtimes/ai-sdk/use-chat-hook.mdx (1)
8-8
: Inclusion of AI/ML API in the integrations list
The updated sentence now includes "AI/ML API" alongside the other supported providers. This update aligns with the PR objectives and is clearly communicated.apps/docs/content/docs/getting-started.mdx (3)
73-73
: Updated Provider Tabs: Added "AI/ML API"
The Tabs component’sitems
array now includes"AI/ML API"
along with the other providers. This change ensures that users see the new integration option in the installation instructions.
119-121
: "AI/ML API" Terminal Installation Command
The Terminal command block for the "AI/ML API" tab installs the same package (@ai-sdk/openai
) as used for OpenAI. This appears intentional—using the underlying OpenAI adapter with a different configuration. Just verify that this reuse is on purpose for the AIML API integration.
315-334
: API Endpoint Implementation for "AI/ML API"
The new API endpoint code block correctly configures the AIML API integration usingcreateOpenAI
with a custombaseURL
and the environment variableAIMLAPI_API_KEY
. The model identifier"accounts/aimlapi/models/meta-llama/Llama-3.3-70B-Instruct-Turbo"
is clearly specified.
Consider adding an inline comment (if desired) to highlight that this is the new integration for clarity in future maintenance.
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
|
||
Define environment variables: | ||
|
||
<Tabs id="provider" items={["OpenAI", "Anthropic", "Azure", "AWS", "Gemini", "GCP", "Groq", "Fireworks", "Cohere", "Ollama", "Chrome AI"]}> | ||
|
||
```sh title="/.env.local" tab="OpenAI" | ||
<Tabs id="provider" items={["OpenAI", "Anthropic", "Azure", "AWS", "Gemini", "GCP", "Groq", "Fireworks", "Cohere", "Ollama", "Chrome AI", "AI/ML API"]}> |
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.
Fixed a syntax error in the Tabs
component where the AI/ML API
item was missing quotes, which would cause a parsing error.
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.
fixed this ☝️
Pull request summary
|
hi, guys! |
Sup!
I'm from the integration team of AI/ML API
We'd like to join your amazing app as one of the providers.
We got 200+ models, including stuff by Openai, Qwen models, Llama's etc - and with no region lock, which is a pretty good benefit for some folks.
Anyways - please check my PR, and let me know if anything needs to be adjusted :)
Important
Added AI/ML API as a new provider, updating documentation and components to support its integration.
ModelPicker.tsx
with modelLlama 3.3 70B Instruct Turbo
.README.md
andpackages/react/README.md
to include AI/ML API in the list of supported providers.getting-started.mdx
to include installation and setup instructions for AI/ML API.rsc.mdx
,use-chat-hook.mdx
, anduse-chat.mdx
to list AI/ML API as a supported integration.getting-started.mdx
.This description was created by
for 387e35e. It will automatically update as commits are pushed.