From f2eb468c464ef929dc7990211b832615b30be750 Mon Sep 17 00:00:00 2001 From: chrislangston Date: Sat, 31 Jan 2026 10:51:04 -0500 Subject: [PATCH] fix: add env-provided DEFAULT_MODEL to VALID_MODELS for validation consistency When ANTHROPIC_DEFAULT_OPUS_MODEL env var is set to a custom model ID, that model was not present in VALID_MODELS (derived from AVAILABLE_MODELS), causing potential validation failures in server/schemas.py validators. This fix dynamically appends the env-provided DEFAULT_MODEL to VALID_MODELS when set, ensuring validators accept the runtime default. The merge is idempotent (only adds if missing) and doesn't alter AVAILABLE_MODELS semantics. Addresses CodeRabbit review feedback on PR #147. --- registry.py | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/registry.py b/registry.py index 2e67f3c..f53e8bd 100644 --- a/registry.py +++ b/registry.py @@ -41,6 +41,11 @@ VALID_MODELS = [m["id"] for m in AVAILABLE_MODELS] # Default model and settings # Respect ANTHROPIC_DEFAULT_OPUS_MODEL env var for Foundry/custom deployments DEFAULT_MODEL = os.getenv("ANTHROPIC_DEFAULT_OPUS_MODEL", "claude-opus-4-5-20251101") + +# Ensure env-provided DEFAULT_MODEL is in VALID_MODELS for validation consistency +# (idempotent: only adds if missing, doesn't alter AVAILABLE_MODELS semantics) +if DEFAULT_MODEL not in VALID_MODELS: + VALID_MODELS.append(DEFAULT_MODEL) DEFAULT_YOLO_MODE = False # SQLite connection settings