Reconcile Infermatic provider catalog - #1847
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Warning Review limit reachedNext included review available in 50 minutes. View limit detailsLimit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (25)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Scanned FilesNone |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: cbb4f5d35f
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| "provider_api_model_id": "infermatic:prometheus-eval-prometheus-7b-v2.0", | ||
| "provider_model_slug": "prometheus-eval-prometheus-7b-v2.0", |
There was a problem hiding this comment.
Remove the guessed Prometheus route identifier
When catalog data is imported for model discovery, provider_model_slug is treated as an exact upstream identifier and added to the discovery allowlist (scripts/model-discovery/run.ts), but this record later admits that the exact API ID requires authenticated discovery and that this value was synthesized from a naming convention. If Infermatic exposes a different slug, the real model will not match this known route and the catalog will publish a nonexistent provider ID; retain only the canonical model/plan association until the route ID is verified.
AGENTS.md reference: packages/data/catalog/AGENTS.md:L9-L10
Useful? React with 👍 / 👎.
Summary
Reconciliation
Validation
pnpm validate:datapnpm validate:pricingpnpm validate:gatewaygit diff --checkCreated with Codex