Skip to content

feat: add per-model compatibility overrides - #21

Merged
AviBackToBlack merged 17 commits into
mainfrom
feat/v0.3-model-overrides
Sep 14, 2026
Merged

AviBackToBlack merged 17 commits into
mainfrom
feat/v0.3-model-overrides

Conversation

@AviBackToBlack

@AviBackToBlack AviBackToBlack commented Sep 14, 2026 •

Copy link
Copy Markdown
Owner

Summary

Implement the second production slice of the v0.3 model-selection/override contract from #19.

This PR:

  • adds strict exact-keyed model_overrides configuration
  • validates a closed whitelist of capability booleans, token limits, supported OpenAI parameters, and reasoning effort sets
  • rejects unknown fields, invalid values, selector-uncovered targets, and contradictory override tables
  • applies overrides only after deterministic model-group aggregation, preserving exact-template identity and deployment evidence
  • retains original aggregate evidence beside configured and effective override values in explain
  • applies dependency closure after overrides (supports_reasoning=false disables reasoning transport/efforts; supports_function_calling=false prevents parallel calls)
  • allows explicit context overrides to unblock otherwise-unsynthesizable foreign multi-deployment groups
  • keeps exact Codex context fields authoritative even when max_input_tokens evidence is overridden
  • keeps foreign Codex web search disabled even when effective web-search evidence is true
  • repairs generated-field provenance so configured assertions are not misreported as LiteLLM observations
  • preserves strict=false semantics when a selector-covered override target is temporarily absent from the current LiteLLM payload
  • documents model_globs and model_overrides in README/config.example now that both v0.3 slices are implemented

Tests

Regression coverage includes config validation/normalization, selector-covered targets, contradiction handling, exact capability restoration, exact reasoning restriction including medium/high levels, reasoning/function dependency closure, foreign context blocker replacement, foreign web-search hard-disable, foreign verbosity transport, exact context ownership, CLI explain original/effective evidence, and strict=false absent-target behavior.

Refs #5


Devin Review

Copilot AI lite review requested due to automatic review settings September 14, 2026 11:16

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Copilot AI review requested due to automatic review settings September 14, 2026 11:17

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Copilot AI review requested due to automatic review settings September 14, 2026 11:18

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Devin Review found 4 potential issues.

Devin Review

Comment thread src/litellm_codex_models/group_mapping.py
Comment thread src/litellm_codex_models/overrides.py
Comment thread src/litellm_codex_models/group_mapping.py Outdated
Copilot AI review requested due to automatic review settings September 14, 2026 11:22

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Copilot AI review requested due to automatic review settings September 14, 2026 11:23

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Copilot AI review requested due to automatic review settings September 14, 2026 11:24

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@AviBackToBlack
AviBackToBlack merged commit 8c050d1 into main Sep 14, 2026
8 checks passed
@AviBackToBlack
AviBackToBlack deleted the feat/v0.3-model-overrides branch September 14, 2026 11:29
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.

2 participants