Repository navigation
feat(sync): add update-only Novita AI catalog sync - #9250
Merged
Merged
Conversation
Contributor
|
No actionable findings. |
Contributor
|
No actionable findings. |
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
Minimal automation follow-up to the merged catalog audit in #9243. Adds an hourly
novita-aitarget to the existing sync runner rather than a separate workflow or a broad catalog rewrite.GET https://api.novita.ai/openai/v1/models.skipCreates: truekeeps new models out of the update-only sync PRs. The sync does not fabricate canonical metadata or reasoning controls; ordinary existing price/limit updates use the normal auto-merge policy without any Novita-specific exception.No catalog data, model-ID loader changes, or shared auto-merge policy changes are included.
Authentication
NOVITA_AI_API_KEY, forwarded by the hourly workflow.NOVITA_AI_MODELS_DEV_KEYtakes precedence when set, otherwise the canonical CI variable is used.Pricing and safety
cost.tiers; reject incomplete/duplicate tier boundaries, empty inventories, duplicate IDs, and malformed prices. Model IDs remain opaque nonempty strings; file-write safety is handled by the existing runner.[]placeholder.Sources
Validation
bun validate— passed.bun models:sync novita-ai --dry-run --no-issues— 0 created, 0 updated, 0 removed, 84 unchanged; the four unlisted embedding/reranking entries were retained.git diff --check— passed.tsc --noEmitremains blocked by existing errors in unrelated providers/tests and the unchanged shared metadata code; the new Novita files had no diagnostics in the last check.