Repository navigation
feat(shared): add UberLLM proxy preset - #551
Open
PSA-Team-source wants to merge 1 commit into
Open
PSA-Team-source wants to merge 1 commit into
PSA-Team-source wants to merge 1 commit into
Conversation
Contributor
There was a problem hiding this comment.
Findings
No issues found.
- The new preset is schema-valid: all five fields (
id,label,provider,baseUrl,notes) are present inpackages/shared/src/proxy-presets.ts:59-65, matching theProxyPresetschema. ProxyPresetIdandProxyPresetIdSchemaderive from thePROXY_PRESETSarray, souberllmis automatically included in the enum and covered by the existing "all preset ids are valid ProxyPresetIdSchema values" test (packages/shared/src/proxy-presets.test.ts:18-22).- Base URL
https://api.uberllm.dev/v1ends in/v1, consistent with the OpenRouter (/api/v1) and Requesty (/v1) presets, so no canonicalization mismatch is expected. - The id assertion added at
packages/shared/src/proxy-presets.test.ts:38follows the existing test pattern, and the changeset correctly targets@open-codesign/sharedas a patch. - No new dependency, so the ≤30-dep budget and MIT-compatibility constraints are unaffected; BYOK is preserved since this is a user-keyed OpenAI-compatible preset.
Questions
- None.
Summary
Review mode: initial
This is a small, self-contained data addition: one preset entry, one id assertion, one changeset. It is consistent with how the Opper/Requesty presets were added and does not touch provider runtime, persistence, or release/distribution paths. No blocking or non-blocking code issues were found.
Residual risks (not blocking):
- The live behavior of
https://api.uberllm.dev/v1(model listing,complete()auth/wire) was verified only by the author; this run did not independently exercise the endpoint. If the upstream changes host/path, the preset would silently point at a dead endpoint, same as any relay preset. - There is no explicit assertion pinning the UberLLM
baseUrl/provider; a future edit could change them without a test failing. This is optional given the risk profile and existing test style. - The PR author discloses working on UberLLM; that is a maintainer disclosure decision, not a code finding.
Testing
- Not run (automation). Suggested optional regression test if desired: assert
PROXY_PRESETS.find(p => p.id === 'uberllm')?.baseUrl === 'https://api.uberllm.dev/v1', mirroring theofficial-openai/litellmassertions. Fullpnpm test/typecheckwill run in CI.
Open-CoDesign Bot
This branch has not been deployed
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.
Summary
Adds UberLLM as a proxy preset, next to the OpenRouter, Requesty and Opper presets. UberLLM is an OpenAI-compatible gateway: base URL
https://api.uberllm.dev/v1,GET /modelsfor model discovery, model ids likedeepseek/deepseek-v4-flashoranthropic/claude-sonnet-4.6. Prepaid, pay-as-you-go keys; prices are usually below list price and shown per model at https://uberllm.dev/models. Same three files as the Opper preset in #529 (and Requesty in #397). The base URL already ends in/v1, socanonicalBaseUrlkeeps it as is.I work on UberLLM.
Type of change
Linked issue
None.
Checklist
pnpm lint && pnpm typecheck && pnpm testpasses locally (ran the touched package only, see below; CI will run the full set)pnpm changeset) if user-visibleTests
packages/shared:vitest run23 files, 397 tests passed;tsc --noEmitclean;biome checkon the two changed files clean.uberllmpreset, randetectWireFromBaseUrl→openai-chat,canonicalBaseUrl→https://api.uberllm.dev/v1,modelsEndpointUrl→GET https://api.uberllm.dev/v1/modelsreturned 200 with 263 models, thencomplete()from@open-codesign/providerswithdeepseek/deepseek-v4-flashreturned "pong". (Throwaway test file, not committed.)