sawz/last30days-skill
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Revert MODEL_FALLBACK_ORDER to upstream values

Browse Source 
Model optimization (mini-first fallback, is_search_capable_model) belongs
in PR #67. This PR stays focused on endpoint/API fixes only.

Also fixes pre-existing test bug where test asserted gpt-4o was first in
MODEL_FALLBACK_ORDER when it was actually gpt-4.1.
This commit is contained in:
Jeffrey Sperling
2026-03-11 18:02:33 -07:00
parent 3e9e2f632b
commit e568ef8af9
2 changed files with 8 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tests/test_openai_reddit.py
+6 -6
View File
@@ -64,13 +64,13 @@ class TestIsModelAccessError(unittest.TestCase):
class TestModelFallbackOrder(unittest.TestCase):
"""Tests for MODEL_FALLBACK_ORDER constant."""
def test_contains_gpt41(self):
"""Fallback list should include gpt-4.1 as last resort."""
self.assertIn("gpt-4.1", MODEL_FALLBACK_ORDER)
def test_contains_gpt4o(self):
"""Fallback list should include gpt-4o."""
self.assertIn("gpt-4o", MODEL_FALLBACK_ORDER)
def test_gpt5_mini_is_first(self):
"""gpt-5-mini should be the first fallback option (cheapest with web_search support)."""
self.assertEqual(MODEL_FALLBACK_ORDER[0], "gpt-5-mini")
def test_gpt41_is_first(self):
"""gpt-4.1 should be the first fallback option."""
self.assertEqual(MODEL_FALLBACK_ORDER[0], "gpt-4.1")
if __name__ == "__main__":