feat: harden article summary timeout and fallback

src/summary_mcp/server.py

@@ -172,7 +172,7 @@ def generate_article_summaries(
     selected_ids: list[str],
     output_dir: str | None = None,
     max_retries: int = 2,
-    timeout_seconds: float = 60.0,
+    timeout_seconds: float = 120.0,
     llm_api_key: str | None = None,
     llm_model: str | None = None,
     llm_api_url: str | None = None,

src/summary_mcp/workflows/article_summary.py

@@ -4,6 +4,8 @@ from dataclasses import dataclass
 from pathlib import Path
 from typing import Iterable, Mapping, Sequence
 
+import httpx
+
 from summary_mcp.core.summary_loop import run_loop_payload
 from summary_mcp.validators.article_summary import validate_article_summary_payload
 
@@ -18,7 +20,7 @@ DEFAULT_PROMPT_PATH = OUTPUT_ROOT / "prompts" / "article-summary-prompt.txt"
 class ArticleSummaryConfig:
     prompt_path: Path = DEFAULT_PROMPT_PATH
     max_retries: int = 2
-    timeout_seconds: float = 60.0
+    timeout_seconds: float = 120.0
     llm_api_key: str | None = None
     llm_model: str | None = None
     llm_api_url: str | None = None
@@ -187,17 +189,45 @@ def summarize_selected_articles(
             "warnings": warnings,
         }
 
-        summary_exit_code, summary_payload, _ = run_loop_payload(
-            extracted_payload=extracted_payload,
-            prompt_path=cfg.prompt_path,
-            max_retries=cfg.max_retries,
-            timeout_seconds=cfg.timeout_seconds,
-            api_key=resolved_api_key,
-            model=resolved_model,
-            api_url=resolved_api_url,
-            output_path=None,
-            validator=validate_article_summary_payload,
+        summary_exit_code = 1
+        summary_payload = None
+
+        try:
+            summary_exit_code, summary_payload, _ = run_loop_payload(
+                extracted_payload=extracted_payload,
+                prompt_path=cfg.prompt_path,
+                max_retries=cfg.max_retries,
+                timeout_seconds=cfg.timeout_seconds,
+                api_key=resolved_api_key,
+                model=resolved_model,
+                api_url=resolved_api_url,
+                output_path=None,
+                validator=validate_article_summary_payload,
+            )
+        except httpx.TimeoutException:
+            summary_exit_code = 1
+            summary_payload = None
+
+        should_fallback = (
+            (summary_exit_code != 0 or summary_payload is None)
+            and resolved_model is not None
+            and resolved_model == (cfg.llm_model or model or resolved_model)
+            and resolved_model.startswith("deepseek")
         )
+
+        if should_fallback:
+            summary_exit_code, summary_payload, _ = run_loop_payload(
+                extracted_payload=extracted_payload,
+                prompt_path=cfg.prompt_path,
+                max_retries=cfg.max_retries,
+                timeout_seconds=cfg.timeout_seconds,
+                api_key=None,
+                model=None,
+                api_url=None,
+                output_path=None,
+                validator=validate_article_summary_payload,
+            )
+
         if summary_exit_code != 0 or summary_payload is None:
             continue