From 8ac351407ef8c00b3ab8f0be3a944ba921052a39 Mon Sep 17 00:00:00 2001
From: AgentArcLab <19233945+AgentArcLab@users.noreply.github.com>
Date: Tue, 12 May 2026 18:49:58 -0700
Subject: [PATCH] fix(agent): clear stale config context_length on model switch

When switching models via /model, AIAgent._config_context_length was
never cleared, so the new model inherited the previous model's context
window instead of auto-detecting the correct one via
get_model_context_length().

Clear _config_context_length to None before the runtime field swap so
the full resolution chain (custom_providers per-model, endpoint probe,
models.dev, etc.) is re-evaluated for the newly selected model.

Closes #21509
---
 run_agent.py | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/run_agent.py b/run_agent.py
index eb9cc94f3a9..7c6c62cc9a3 100644
--- a/run_agent.py
+++ b/run_agent.py
@@ -2647,6 +2647,11 @@ class AIAgent:
         old_model = self.model
         old_provider = self.provider
 
+        # Clear the per-config context_length override so the new model's
+        # actual context window is resolved via get_model_context_length()
+        # instead of inheriting the stale value from the previous model.
+        self._config_context_length = None
+
         # ── Swap core runtime fields ──
         self.model = new_model
         self.provider = new_provider
@@ -8824,6 +8829,11 @@ class AIAgent:
                 fb_api_mode = "bedrock_converse"
 
             old_model = self.model
+
+            # Clear the per-config context_length override so the fallback
+            # model's actual context window is resolved instead of inheriting
+            # the stale value from the previous model.  See #22387.
+            self._config_context_length = None
             self.model = fb_model
             self.provider = fb_provider
             self.base_url = fb_base_url