feat(providers): add Hyperbolic preset - #751
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (3)
📝 WalkthroughWalkthroughChangesHyperbolic provider integration
Estimated code review effort: 3 (Moderate) | ~20 minutes Sequence Diagram(s)sequenceDiagram
participant OcxConfig
participant buildModelsRequest
participant HyperbolicAPI
participant gatherRoutedModels
participant routeModel
OcxConfig->>buildModelsRequest: Build /models request
buildModelsRequest->>HyperbolicAPI: GET /v1/models with Bearer key
HyperbolicAPI-->>gatherRoutedModels: Return model list
gatherRoutedModels-->>routeModel: Preserve native model IDs
routeModel-->>OcxConfig: Build /chat/completions request
Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 3 | ❌ 2❌ Failed checks (2 warnings)
✅ Passed checks (3 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
NEEDS-CHANGES — the code is ready, the evidence package is not. The implementation shape is right and the tests are the kind I like to see on a preset. The blocker is Going through it: Chutes (
Hyperbolic (
One thing worth knowing before you invest more: Hyperbolic is already in What happens next: complete the evidence for whichever provider you want canonical — the endpoint and |
lidge-jun
left a comment
There was a problem hiding this comment.
Formalizing the disposition so it shows on your dashboard.
Blocking on the canonical-preset evidence gate in MAINTAINERS.md:33-42, not on the code — the implementation and tests are in good shape. Missing for both providers: terms of service and operating legal entity, a named maintenance owner, and a citable verification date. Chutes additionally needs routing authorization as an aggregator, and Hyperbolic needs authenticated GET /v1/models documentation since the entry declares liveModels.
Note that Hyperbolic already has an inert row at src/providers/free-directory.ts:127, which is the preferred landing spot when the evidence is incomplete.
Full per-item walkthrough in my earlier comment.
414e033 to
d5dd390
Compare
|
Addressed the canonical-preset evidence gate in
Validation: focused provider/contract/catalog suite 200 pass / 0 fail, typecheck passed, privacy scan passed, and the docs build produced 146 pages. The PR description records the local Bun full-suite non-exit separately; the rebased cross-platform CI is authoritative for the complete suite. Please re-review the requested changes and perform the explicit security review required for this credential-destination addition. |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@docs-site/src/content/docs/ja/guides/providers.md`:
- Around line 119-121: Update the earlier Japanese OAuth preset count in the
OAuth section to match the catalog’s 7 presets, ensuring both references
consistently reflect the registry count.
- Line 136: Correct GitHub Copilot’s provider classification by removing it from
the API-key tables or moving it to the OAuth catalogs, preserving the existing
OAuth description. Apply the same correction in
docs-site/src/content/docs/ja/guides/providers.md:136-136,
docs-site/src/content/docs/ko/guides/providers.md:136-136, and
docs-site/src/content/docs/zh-cn/guides/providers.md:128-128.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro Plus
Run ID: f68186fe-2ebe-40dc-b9d9-9e90f75cf288
📒 Files selected for processing (14)
docs-site/src/content/docs/getting-started/quickstart.mddocs-site/src/content/docs/guides/providers.mddocs-site/src/content/docs/ja/getting-started/quickstart.mddocs-site/src/content/docs/ja/guides/providers.mddocs-site/src/content/docs/ko/getting-started/quickstart.mddocs-site/src/content/docs/ko/guides/providers.mddocs-site/src/content/docs/ru/getting-started/quickstart.mddocs-site/src/content/docs/ru/guides/providers.mddocs-site/src/content/docs/zh-cn/getting-started/quickstart.mddocs-site/src/content/docs/zh-cn/guides/providers.mdsrc/providers/registry.tstests/fixtures/hyperbolic-models.jsontests/hyperbolic-provider.test.tstests/provider-registry-parity.test.ts
Relates to #572.
This is one review-sized provider addition under the umbrella issue. It intentionally does not close #572.
What changed
hyperbolicas a collision-safe API-key preset for CLI init, dashboard setup, key login, and provider workspaces/v1transport with registry-owned, bounded/modelsdiscoveryChutes was removed from this PR because its aggregator authorization is not yet established by a citable public source. It was not added to
free-directory.ts: Chutes' official pricing currently says it has no free tier, so placing it in the free-provider directory would be inaccurate. Chutes remains usable through the custom OpenAI-compatible provider flow and can return as a separate canonical-preset PR once the authorization evidence is public.Canonical-preset evidence
GET /v1/modelscontract/v1/modelshost and client authentication bindingMaintenance owner: @olddonkey. Breakage should be reported through an opencodex GitHub issue tagging @olddonkey. I will update the preset when its base URL, authentication, or catalog contract changes, or demote it from the canonical registry if its evidence no longer supports the credential destination.
Contributor affiliation: none; this contribution is not submitted on behalf of Hyperbolic.
Safety and scope
config.jsonor the key-login mapValidation
bun test tests/hyperbolic-provider.test.ts tests/provider-registry-parity.test.ts tests/provider-model-discovery-contract.test.ts tests/codex-catalog.test.ts tests/slug-codec.test.ts tests/provider-payload.test.ts— 200 pass, 0 failbun run typecheckbun run privacy:scancd docs-site && bun install --frozen-lockfile && bun run build— 146 pages builtLocal full-suite note:
bun run testemitted no failure before the Bun 1.3.14 isolated runner stopped producing output while remaining CPU-active; it was terminated after ten minutes. The focused suite above covers the changed provider, derivation, discovery contract, catalog, slug, and payload surfaces. GitHub's rebased cross-platform CI remains authoritative for the complete suite.Summary by CodeRabbit