Skip to content

feat(providers): register GPT-5.6 models (sol, terra, luna) - #598

Open
chinmay1819 wants to merge 5 commits into
NVIDIA:mainfrom
chinmay1819:main
Open

chinmay1819 wants to merge 5 commits into
NVIDIA:mainfrom
chinmay1819:main

Conversation

@chinmay1819

Copy link
Copy Markdown

What

Adds gpt-5.6-sol, gpt-5.6-terra and gpt-5.6-luna to the OpenAI provider's
bundled registry (src/skillspector/providers/openai/model_registry.yaml) and
to the root model_registry.yaml.

Why

These models already worked through SKILLSPECTOR_MODEL, but because they
weren't in the registry, SkillSpector:

  • fell back to a 128K context window, so large files were split into far more
    LLM chunks than needed
  • logged a "not found in model_registry.yaml" warning on every scan
  • refused to start when SKILLSPECTOR_STRICT_MODEL_VALIDATION=true

How

Registry entries only, no code changes. All three models get the values from
OpenAI's model docs:

Model context_length max_output_tokens
gpt-5.6-sol 1,050,000 128,000
gpt-5.6-terra 1,050,000 128,000
gpt-5.6-luna 1,050,000 128,000
  • Sol caps input at 922K tokens. SkillSpector budgets 75% of the context
    window (~787K), which stays under that cap, so no extra field is needed.
  • Prices appear only as YAML comments with the source link. SkillSpector
    still never computes cost, as described in docs/INFERENCE_USAGE.md.
  • DEFAULT_MODEL stays gpt-5.4.

Testing

  • test_metadata_known_model now checks the context window and max output
    tokens for all three models.
  • make test-unit, make lint and make format-check all pass.
  • A --no-llm scan with strict model validation starts cleanly for each model.

chinmay1819 and others added 5 commits September 21, 2026 14:23
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>
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant