-
Notifications
You must be signed in to change notification settings - Fork 6.9k
feat: Groq provider support #4943
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
🦋 Changeset detectedLatest commit: e25ce84 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
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 |
Coverage ReportExtension CoverageBase branch: 46% PR branch: 47% ✅ Coverage increased or remained the same Webview CoverageBase branch: 13% PR branch: 13% ✅ Coverage increased or remained the same Overall Assessment✅ Test coverage has been maintained or improved Last updated: 2025-07-16T02:50:03.858437 |
bc266bc
to
beb1467
Compare
7a83d1a
to
593cdfb
Compare
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.
Tested, and it works (very fast might i add)
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.
One question and two requested changes
@celestial-vault Thanks for the callouts I resolved all your suggested changes. |
@celestial-vault Review is welcome. |
d73898d
to
ab0266b
Compare
ab0266b
to
e25ce84
Compare
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Matt Rubens <mrubens@users.noreply.github.com>
Thanks to @edwin-truthsearch-io for the PR #3898
This is a followup and a rebase to that PR that works with the current code
Implement feature request #565 to support Groq provider. This includes adding proper support for the moonshot kimi k2 model in Groq, which now actually works (no reasoning pollution).
You can find the documentation here
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
Pre-flight Checklist
npm test
) and code is formatted and linted (npm run format && npm run lint
)npm run changeset
(required for user-facing changes)Screenshots
Additional Notes
Important
Adds support for Groq provider, including API handling, state management, and UI integration, with a new
GroqHandler
and model management features.GroqHandler
insrc/api/providers/groq.ts
for Groq API interactions, supporting model family detection and message creation.refreshGroqModels
inrefreshGroqModels.ts
to fetch and cache Groq models.initializeWebview.ts
to refresh Groq models on webview initialization.groqApiKey
,groqModelId
, andgroqModelInfo
to state management instate.ts
andstate-keys.ts
.ExtensionStateContext.tsx
to manage Groq models and state.GroqProvider
andGroqModelPicker
inApiOptions.tsx
andGroqProvider.tsx
for UI integration.providerUtils.ts
to normalize Groq API configuration.api.ts
andapi-configuration-conversion.ts
.README.md
to include Groq in the list of supported providers.This description was created by
for e25ce84. You can customize this summary. It will automatically update as commits are pushed.