Skip to content

Conversation

edwin-truthsearch-io
Copy link

@edwin-truthsearch-io edwin-truthsearch-io commented May 29, 2025

Description

Implement feature request #565 to support Groq provider. This includes adding proper support for the qwen-qwq-32b model in Groq, which now actually works (no reasoning pollution).

Test Procedure

Unit tests, some feature tests that are enabled by setting the environment variable GROQ_API_KEY.
Ran all known regression tests, and used the updated plugin in VS Code while developing.

Type of Change

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • ♻️ Refactor Changes
  • 💅 Cosmetic Changes
  • 📚 Documentation update
  • 🏃 Workflow Changes

Pre-flight Checklist

  • Changes are limited to a single feature, bugfix or chore (split larger changes into separate PRs)
  • Tests are passing (npm test) and code is formatted and linted (npm run format && npm run lint)
  • I have created a changeset using npm run changeset (required for user-facing changes)
  • I have reviewed contributor guidelines

Screenshots

image image

Additional Notes

@saoudrizwan I noted you had said this was on the todo list. Please let me know what I need to fix to get this one incorporated. Thanks!


Important

Adds support for Groq provider, including model handling, API integration, and UI updates for Groq models.

  • Behavior:
    • Adds Groq provider support, including handling for qwen-qwq-32b model.
    • Updates ApiOptions.tsx to include Groq in the provider dropdown and handle API key input.
    • Implements refreshGroqModels in refreshGroqModels.ts to fetch and cache Groq models.
  • Handlers:
    • Introduces GroqHandler in groq.ts for API interactions and model management.
    • Adds model family detection and optimal parameter settings in GroqHandler.
  • Tests:
    • Adds unit tests for GroqHandler in groq.test.ts.
    • Includes integration tests for Groq API connectivity.
  • UI:
    • Updates ChatRow.tsx to render Groq model outputs.
    • Adds Groq API key input and model refresh button in ApiOptions.tsx.

This description was created by Ellipsis for d71b79c. You can customize this summary. It will automatically update as commits are pushed.

Copy link

changeset-bot bot commented May 29, 2025

🦋 Changeset detected

Latest commit: cafe343

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
claude-dev Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@edwin-truthsearch-io edwin-truthsearch-io changed the title feat: Groq provider support (#565) feat: Groq provider support May 29, 2025
@celestial-vault
Copy link
Contributor

@edwin-truthsearch-io Isn't the existing X AI provider the same thing?

@edwin-truthsearch-io
Copy link
Author

edwin-truthsearch-io commented May 29, 2025

@celestial-vault Groq is not to be confused with Grok. Groq is known for speed. https://groq.com/. Groq has nothing to do with xAI or Grok.

@edwinpjacques
Copy link

@edwin-truthsearch-io Isn't the existing X AI provider the same thing?

No, Groq is different than Grok, completely different companies.

@edwinpjacques edwinpjacques force-pushed the groq-support branch 3 times, most recently from 03ea720 to ad02e48 Compare June 1, 2025 12:59
@edwin-truthsearch-io edwin-truthsearch-io marked this pull request as draft June 1, 2025 18:02
@edwinpjacques edwinpjacques force-pushed the groq-support branch 4 times, most recently from ac2a9c1 to d6be594 Compare June 2, 2025 12:31
@edwin-truthsearch-io edwin-truthsearch-io marked this pull request as ready for review June 2, 2025 12:33
@edwin-truthsearch-io edwin-truthsearch-io marked this pull request as draft June 2, 2025 15:57
@edwinpjacques edwinpjacques force-pushed the groq-support branch 4 times, most recently from 6163785 to d71b79c Compare June 2, 2025 21:43
@edwin-truthsearch-io edwin-truthsearch-io marked this pull request as ready for review June 2, 2025 21:52
@0xToshii
Copy link
Contributor

@edwin-truthsearch-io we are interested in adding the groq provider. we are in the middle or refactoring ApiOptions.tsx - would you be able to get groq to be structured similarly to like mistral/deepseek where the ui component is in its own file?

also are you willing to open this pr up to contributor changes. surface level i think there might be issues with the protobuf messaging which would be easier for me to fix rather than going back and forth. also i can't see the merge conflicts

@orliesaurus
Copy link

+1 for groq

@saoudrizwan
Copy link
Contributor

Thank you for the contribution 🙏 I apologize we couldn't get to your PR but we have added Groq support in the latest v3.20.2. Appreciate your patience here!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants