feat: add support for openai models#27
Open
aayushpurplesectors wants to merge 1 commit intobug0inc:mainfrom
Open
feat: add support for openai models#27aayushpurplesectors wants to merge 1 commit intobug0inc:mainfrom
aayushpurplesectors wants to merge 1 commit intobug0inc:mainfrom
Conversation
Contributor
|
Hi @aayushpurplesectors sorry about the delay here, but I am reviewing it now. Do you mind resolving the merge conflict please? |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This PR adds support for integrating direct OpenAI models into Passmark. It follows the existing architectural patterns set by the direct Anthropic and Google integrations. Users can now pass models using the
openai/prefix (e.g.,openai/gpt-4o) directly to configuration.What Changed
createOpenAIinsrc/models.ts._openailazy singleton insrc/models.tswith graceful missing-key error handling mapping toConfigurationError.switch caseinresolveModel()to correctly route"openai"string prefixes.README.mdto document the newOPENAI_API_KEYenvironment variable.CHANGELOG.mdunder the[Unreleased]section detailing the change.Why It's Needed
Although users can use Vercel AI Gateway or OpenRouter for OpenAI routing, native support for
@ai-sdk/openaienables developers bypassing those systems to utilize models like GPT-4o or o3-mini cleanly and directly using theirOPENAI_API_KEY.Checklist
npm run buildlocallyREADME.md)CHANGELOG.mdnotes to the[Unreleased]section