fix: address CodeRabbit feedback and CI failure

- Mock check_magentic_requirements in tests to run without OPENAI_API_KEY
- Update stale "biorxiv" comment in models.py per CodeRabbit suggestion

CI was failing because MagenticOrchestrator requires OPENAI_API_KEY,
which exists locally in .env but not in CI environment.

src/utils/models.py

@@ -5,7 +5,7 @@ from typing import Any, ClassVar, Literal
 
 from pydantic import BaseModel, Field
 
-# Centralized source type - add new sources here (e.g., "biorxiv" in Phase 11)
+# Centralized source type - add new sources here (e.g., new databases)
 SourceName = Literal["pubmed", "clinicaltrials", "europepmc", "preprint"]
 
 

tests/unit/test_magentic_fix.py

@@ -2,6 +2,7 @@
 
 from unittest.mock import MagicMock, patch
 
+import pytest
 from agent_framework import MagenticFinalResultEvent
 
 from src.orchestrator_magentic import MagenticOrchestrator
@@ -31,10 +32,17 @@ class MockChatMessage:
         return "<ChatMessage object at 0xMOCK>"
 
 
+@pytest.fixture
+def mock_magentic_requirements():
+    """Mock the API key check so tests run in CI without OPENAI_API_KEY."""
+    with patch("src.orchestrator_magentic.check_magentic_requirements"):
+        yield
+
+
 class TestMagenticFixes:
     """Tests for the Magentic mode fixes."""
 
-    def test_process_event_extracts_text_correctly(self) -> None:
+    def test_process_event_extracts_text_correctly(self, mock_magentic_requirements) -> None:
         """
         Test that _process_event correctly extracts text from a ChatMessage.
 
@@ -54,7 +62,7 @@ class TestMagenticFixes:
         assert result_event.type == "complete"
         assert result_event.message == "Final Report Content"
 
-    def test_max_rounds_configuration(self) -> None:
+    def test_max_rounds_configuration(self, mock_magentic_requirements) -> None:
         """Test that max_rounds is correctly passed to the orchestrator."""
         orchestrator = MagenticOrchestrator(max_rounds=25)
         assert orchestrator._max_rounds == 25