feat(providers): register GPT-5.6 models (sol, terra, luna) - #598
Open
chinmay1819 wants to merge 5 commits into
Open
chinmay1819 wants to merge 5 commits into
chinmay1819 wants to merge 5 commits into
Conversation
Add both models to the OpenAI provider registry and the root model_registry.yaml with their 1,050,000-token context window and 128,000 max output tokens, so token budgeting uses real limits and strict model validation accepts them. Pricing is recorded as comments only; SkillSpector still does not compute cost. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Signed-off-by: chinmay1819 <chinmaydeshpande34@gmail.com>
Signed-off-by: chinmay1819 <chinmaydeshpande34@gmail.com>
Feat/gpt 5.6 terra luna
Add the GPT-5.6 flagship to the OpenAI provider registry and the root model_registry.yaml with its 1,050,000-token context window and 128,000 max output tokens. Its 922K input cap sits above the 75% input budget (~787K), so context_length alone is sufficient. Pricing is recorded as comments only. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Signed-off-by: chinmay1819 <chinmaydeshpande34@gmail.com>
feat(providers): register gpt-5.6-sol
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.
What
Adds
gpt-5.6-sol,gpt-5.6-terraandgpt-5.6-lunato the OpenAI provider'sbundled registry (
src/skillspector/providers/openai/model_registry.yaml) andto the root
model_registry.yaml.Why
These models already worked through
SKILLSPECTOR_MODEL, but because theyweren't in the registry, SkillSpector:
LLM chunks than needed
SKILLSPECTOR_STRICT_MODEL_VALIDATION=trueHow
Registry entries only, no code changes. All three models get the values from
OpenAI's model docs:
context_lengthmax_output_tokenswindow (~787K), which stays under that cap, so no extra field is needed.
still never computes cost, as described in
docs/INFERENCE_USAGE.md.DEFAULT_MODELstaysgpt-5.4.Testing
test_metadata_known_modelnow checks the context window and max outputtokens for all three models.
make test-unit,make lintandmake format-checkall pass.--no-llmscan with strict model validation starts cleanly for each model.