fix: cover o1 max-completion token fallback

Maintainer edit: keep the GPT-5/o-series fallback on slug-boundary matching so unrelated model names are not caught by substring checks, and include o1 alongside o3/o4 because it is also an o-series chat model.

nanobot/providers/openai_compat_provider.py

@@ -94,10 +94,10 @@ def _model_slug(model_name: str) -> str:
 
 
 def _requires_max_completion_tokens(model_name: str) -> bool:
-    """Return True for models that reject ``max_tokens`` (GPT-5 family, o3/o4)."""
+    """Return True for models that reject ``max_tokens`` (GPT-5 family, o-series)."""
     slug = _model_slug(model_name)
     return "gpt-5" in slug or any(
-        slug == p or slug.startswith((p + "-", p + ".")) for p in ("o3", "o4")
+        slug == p or slug.startswith((p + "-", p + ".")) for p in ("o1", "o3", "o4")
     )
 
 

tests/providers/test_litellm_kwargs.py

@@ -933,8 +933,12 @@ def test_openai_compat_build_kwargs_uses_gpt5_safe_parameters() -> None:
     ("model_name", "expected_key"),
     [
         ("gpt-5.4", "max_completion_tokens"),
+        ("o1-mini", "max_completion_tokens"),
         ("o3-mini", "max_completion_tokens"),
+        ("o4-mini", "max_completion_tokens"),
         ("gpt-4", "max_tokens"),
+        ("foo3-mini", "max_tokens"),
+        ("foo4-mini", "max_tokens"),
     ],
 )
 def test_openai_compat_build_kwargs_max_completion_tokens_by_model_name(