From 0c500e1ec659e88b7b638380e338c4f2f7985308 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Tue, 24 Feb 2026 18:44:20 +0000 Subject: [PATCH] fix(ocr): address code review: add model fallback default, remove unused variable Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com> --- app/utils/text_quality.py | 2 +- tests/test_text_quality.py | 1 - 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/app/utils/text_quality.py b/app/utils/text_quality.py index 3901633b..e2200092 100644 --- a/app/utils/text_quality.py +++ b/app/utils/text_quality.py @@ -261,7 +261,7 @@ def check_text_quality(text: str, text_source: TextSource) -> TextQualityResult: response_text: Optional[str] = None try: provider = get_ai_provider() - model = settings.ai_model or settings.openai_model + model = settings.ai_model or settings.openai_model or "gpt-4o-mini" response_text = provider.chat_completion( messages=[ { diff --git a/tests/test_text_quality.py b/tests/test_text_quality.py index aa151c1d..fc2338c8 100644 --- a/tests/test_text_quality.py +++ b/tests/test_text_quality.py @@ -401,7 +401,6 @@ class TestCheckTextQualityAI: assert len(captured_prompts) == 1 # The prompt should NOT contain more than _TEXT_SAMPLE_MAX_CHARS "a"s - count_a = captured_prompts[0].count("a" * 100) assert "a" * (_TEXT_SAMPLE_MAX_CHARS + 1) not in captured_prompts[0]