aiagents4pharma 1.27.2__py3-none-any.whl → 1.29.0__py3-none-any.whl

aiagents4pharma/talk2scholars/configs/tools/search/default.yaml

@@ -9,8 +9,8 @@ api_fields:
   - "authors"
   - "citationCount"
   - "url"
+  - "externalIds"
 # Commented fields that could be added later if needed
-# - "externalIds"
 #  - "publicationTypes"
 #  - "openAccessPdf"
 

aiagents4pharma/talk2scholars/configs/tools/single_paper_recommendation/default.yaml

@@ -9,8 +9,8 @@ api_fields:
   - "authors"
   - "citationCount"
   - "url"
+  - "externalIds"
 # Commented fields that could be added later if needed
-# - "externalIds"
 #  - "publicationTypes"
 #  - "openAccessPdf"
 

aiagents4pharma/talk2scholars/configs/tools/zotero_read/default.yaml

@@ -12,4 +12,45 @@ search_params:
 # Item Types and Limit
 zotero:
   max_limit: 100
-  filter_item_types: ["journalArticle", "conferencePaper", "preprint"]
+  filter_item_types:
+    [
+      "Artwork",
+      "Audio Recording",
+      "Bill",
+      "Blog Post",
+      "Book",
+      "Book Section",
+      "Case",
+      "Conference Paper",
+      "Dataset",
+      "Dictionary Entry",
+      "Document",
+      "E-mail",
+      "Encyclopedia Article",
+      "Film",
+      "Forum Post",
+      "Hearing",
+      "Instant Message",
+      "Interview",
+      "Journal Article",
+      "Letter",
+      "Magazine Article",
+      "Manuscript",
+      "Map",
+      "Newspaper Article",
+      "Patent",
+      "Podcast",
+      "Preprint",
+      "Presentation",
+      "Radio Broadcast",
+      "Report",
+      "Software",
+      "Standard",
+      "Statute",
+      "Thesis",
+      "TV Broadcast",
+      "Video Recording",
+      "Web Page",
+    ]
+
+  filter_excluded_types: ["attachment", "note", "annotation"]

aiagents4pharma/talk2scholars/configs/tools/zotero_write/__inti__.py

@@ -0,0 +1,3 @@
+"""
+Import all the modules in the package
+"""

aiagents4pharma/talk2scholars/state/state_talk2scholars.py

@@ -60,5 +60,6 @@ class Talk2Scholars(AgentState):
     last_displayed_papers: Annotated[Dict[str, Any], replace_dict]
     papers: Annotated[Dict[str, Any], replace_dict]
     multi_papers: Annotated[Dict[str, Any], replace_dict]
+    pdf_data: Annotated[Dict[str, Any], replace_dict]
     zotero_read: Annotated[Dict[str, Any], replace_dict]
     llm_model: BaseChatModel

aiagents4pharma/talk2scholars/tests/test_main_agent.py

@@ -3,119 +3,194 @@ Unit tests for main agent functionality.
 Tests the supervisor agent's routing logic and state management.
 """
 
-# pylint: disable=redefined-outer-name
 # pylint: disable=redefined-outer-name,too-few-public-methods
-import random
-from unittest.mock import Mock, patch, MagicMock
+
+from types import SimpleNamespace
 import pytest
-from langchain_core.messages import HumanMessage, AIMessage
-from langgraph.graph import END
-from ..agents.main_agent import make_supervisor_node, get_hydra_config, get_app
-from ..state.state_talk2scholars import Talk2Scholars
+import hydra
+from langchain_core.language_models.chat_models import BaseChatModel
+from langchain_openai import ChatOpenAI
+from pydantic import Field
+from aiagents4pharma.talk2scholars.agents.main_agent import get_app
+
+# --- Dummy LLM Implementation ---
+
+
+class DummyLLM(BaseChatModel):
+    """A dummy language model implementation for testing purposes."""
+
+    model_name: str = Field(...)
+
+    def _generate(self, prompt, stop=None):
+        """Generate a response given a prompt."""
+        DummyLLM.called_prompt = prompt
+        return "dummy output"
+
+    @property
+    def _llm_type(self):
+        """Return the type of the language model."""
+        return "dummy"
+
+
+# --- Dummy Workflow and Sub-agent Functions ---
+
+
+class DummyWorkflow:
+    """A dummy workflow class that records arguments for verification."""
+
+    def __init__(self, supervisor_args=None):
+        """Initialize the workflow with the given supervisor arguments."""
+        self.supervisor_args = supervisor_args or {}
+        self.checkpointer = None
+        self.name = None
+
+    def compile(self, checkpointer, name):
+        """Compile the workflow with the given checkpointer and name."""
+        self.checkpointer = checkpointer
+        self.name = name
+        return self
+
+
+def dummy_get_app_s2(uniq_id, llm_model):
+    """Return a DummyWorkflow for the S2 agent."""
+    dummy_get_app_s2.called_uniq_id = uniq_id
+    dummy_get_app_s2.called_llm_model = llm_model
+    return DummyWorkflow(supervisor_args={"agent": "s2", "uniq_id": uniq_id})
+
+
+def dummy_get_app_zotero(uniq_id, llm_model):
+    """Return a DummyWorkflow for the Zotero agent."""
+    dummy_get_app_zotero.called_uniq_id = uniq_id
+    dummy_get_app_zotero.called_llm_model = llm_model
+    return DummyWorkflow(supervisor_args={"agent": "zotero", "uniq_id": uniq_id})
+
+
+def dummy_create_supervisor(apps, model, state_schema, **kwargs):
+    """Return a DummyWorkflow for the supervisor."""
+    dummy_create_supervisor.called_kwargs = kwargs
+    return DummyWorkflow(
+        supervisor_args={
+            "apps": apps,
+            "model": model,
+            "state_schema": state_schema,
+            **kwargs,
+        }
+    )
+
+
+# --- Dummy Hydra Configuration Setup ---
+
+
+class DummyHydraContext:
+    """A dummy context manager for mocking Hydra's initialize and compose functions."""
+
+    def __enter__(self):
+        """Return None when entering the context."""
+        return None
+
+    def __exit__(self, exc_type, exc_val, traceback):
+        """Exit function that does nothing."""
+        return None
+
+
+def dict_to_namespace(d):
+    """Convert a dictionary to a SimpleNamespace object."""
+    return SimpleNamespace(
+        **{
+            key: dict_to_namespace(val) if isinstance(val, dict) else val
+            for key, val in d.items()
+        }
+    )
+
+
+dummy_config = {
+    "agents": {
+        "talk2scholars": {"main_agent": {"system_prompt": "Dummy system prompt"}}
+    }
+}
+
+
+class DummyHydraCompose:
+    """A dummy class that returns a namespace from a dummy config dictionary."""
+
+    def __init__(self, config):
+        """Constructor that stores the dummy config."""
+        self.config = config
+
+    def __getattr__(self, item):
+        """Return a namespace from the dummy config."""
+        return dict_to_namespace(self.config.get(item, {}))
+
+
+# --- Pytest Fixtures to Patch Dependencies ---
+
+
+@pytest.fixture(autouse=True)
+def patch_hydra(monkeypatch):
+    """Patch the hydra.initialize and hydra.compose functions to return dummy objects."""
+    monkeypatch.setattr(
+        hydra, "initialize", lambda version_base, config_path: DummyHydraContext()
+    )
+    monkeypatch.setattr(
+        hydra, "compose", lambda config_name, overrides: DummyHydraCompose(dummy_config)
+    )
 
 
 @pytest.fixture(autouse=True)
-def mock_hydra():
-    """Mock Hydra configuration."""
-    with patch("hydra.initialize"), patch("hydra.compose") as mock_compose:
-        cfg_mock = MagicMock()
-        cfg_mock.agents.talk2scholars.main_agent.temperature = 0
-        cfg_mock.agents.talk2scholars.main_agent.system_prompt = "System prompt"
-        cfg_mock.agents.talk2scholars.main_agent.router_prompt = "Router prompt"
-        mock_compose.return_value = cfg_mock
-        yield mock_compose
-
-
-def test_get_app():
-    """Test the full initialization of the LangGraph application."""
-    thread_id = "test_thread"
-    mock_llm = Mock()
-    app = get_app(thread_id, mock_llm)
-    assert app is not None
-    assert "supervisor" in app.nodes
-    assert "s2_agent" in app.nodes  # Ensure nodes exist
-    assert "zotero_agent" in app.nodes
-
-
-def test_get_app_with_default_llm():
-    """Test app initialization with default LLM parameters."""
-    thread_id = "test_thread"
-    llm_mock = Mock()
-
-    # We need to explicitly pass the mock instead of patching, since the function uses
-    # ChatOpenAI as a default argument value which is evaluated at function definition time
-    app = get_app(thread_id, llm_mock)
-    assert app is not None
-    # We can only verify the app was created successfully
-
-
-def test_get_hydra_config():
-    """Test that Hydra configuration loads correctly."""
-    with patch("hydra.initialize"), patch("hydra.compose") as mock_compose:
-        cfg_mock = MagicMock()
-        cfg_mock.agents.talk2scholars.main_agent.temperature = 0
-        mock_compose.return_value = cfg_mock
-        cfg = get_hydra_config()
-        assert cfg is not None
-        assert cfg.temperature == 0
-
-
-def test_hydra_failure():
-    """Test exception handling when Hydra fails to load config."""
-    thread_id = "test_thread"
-    llm_mock = Mock()
-    with patch("hydra.initialize", side_effect=Exception("Hydra error")):
-        with pytest.raises(Exception) as exc_info:
-            get_app(thread_id, llm_model=llm_mock)
-        assert "Hydra error" in str(exc_info.value)
-
-
-def test_supervisor_node_execution():
-    """Test that the supervisor node routes correctly."""
-    mock_llm = Mock()
-    thread_id = "test_thread"
-
-    class MockRouter:
-        """Mock router class."""
-
-        next = random.choice(["s2_agent", "zotero_agent"])
-
-    with (
-        patch.object(mock_llm, "with_structured_output", return_value=mock_llm),
-        patch.object(mock_llm, "invoke", return_value=MockRouter()),
-    ):
-        supervisor_node = make_supervisor_node(mock_llm, thread_id)
-        mock_state = Talk2Scholars(messages=[HumanMessage(content="Find AI papers")])
-        result = supervisor_node(mock_state)
-
-        # Accept either "s2_agent" or "zotero_agent"
-        assert result.goto in ["s2_agent", "zotero_agent"]
-
-
-def test_supervisor_node_finish():
-    """Test that supervisor node correctly handles FINISH case."""
-    mock_llm = Mock()
-    thread_id = "test_thread"
-
-    class MockRouter:
-        """Mock router class."""
-
-        next = "FINISH"
-
-    class MockAIResponse:
-        """Mock AI response class."""
-
-        def __init__(self):
-            self.content = "Final AI Response"
-
-    with (
-        patch.object(mock_llm, "with_structured_output", return_value=mock_llm),
-        patch.object(mock_llm, "invoke", side_effect=[MockRouter(), MockAIResponse()]),
-    ):
-        supervisor_node = make_supervisor_node(mock_llm, thread_id)
-        mock_state = Talk2Scholars(messages=[HumanMessage(content="End conversation")])
-        result = supervisor_node(mock_state)
-        assert result.goto == END
-        assert "messages" in result.update
-        assert isinstance(result.update["messages"], AIMessage)
-        assert result.update["messages"].content == "Final AI Response"
+def patch_sub_agents_and_supervisor(monkeypatch):
+    """Patch the sub-agents and supervisor creation functions."""
+    monkeypatch.setattr(
+        "aiagents4pharma.talk2scholars.agents.main_agent.get_app_s2", dummy_get_app_s2
+    )
+    monkeypatch.setattr(
+        "aiagents4pharma.talk2scholars.agents.main_agent.get_app_zotero",
+        dummy_get_app_zotero,
+    )
+    monkeypatch.setattr(
+        "aiagents4pharma.talk2scholars.agents.main_agent.create_supervisor",
+        dummy_create_supervisor,
+    )
+
+
+# --- Test Cases ---
+
+
+def test_dummy_llm_generate():
+    """Test the dummy LLM's generate function."""
+    dummy = DummyLLM(model_name="test-model")
+    output = getattr(dummy, "_generate")("any prompt")
+    assert output == "dummy output"
+
+
+def test_dummy_llm_llm_type():
+    """Test the dummy LLM's _llm_type property."""
+    dummy = DummyLLM(model_name="test-model")
+    assert getattr(dummy, "_llm_type") == "dummy"
+
+
+def test_get_app_with_gpt4o_mini():
+    """
+    Test that get_app replaces a 'gpt-4o-mini' LLM with a new ChatOpenAI instance.
+    """
+    uniq_id = "test_thread"
+    dummy_llm = DummyLLM(model_name="gpt-4o-mini")
+    app = get_app(uniq_id, dummy_llm)
+
+    supervisor_args = getattr(app, "supervisor_args", {})
+    assert isinstance(supervisor_args.get("model"), ChatOpenAI)
+    assert supervisor_args.get("prompt") == "Dummy system prompt"
+    assert getattr(app, "name", "") == "Talk2Scholars_MainAgent"
+
+
+def test_get_app_with_other_model():
+    """
+    Test that get_app does not replace the LLM if its model_name is not 'gpt-4o-mini'.
+    """
+    uniq_id = "test_thread_2"
+    dummy_llm = DummyLLM(model_name="other-model")
+    app = get_app(uniq_id, dummy_llm)
+
+    supervisor_args = getattr(app, "supervisor_args", {})
+    assert supervisor_args.get("model") is dummy_llm
+    assert supervisor_args.get("prompt") == "Dummy system prompt"
+    assert getattr(app, "name", "") == "Talk2Scholars_MainAgent"

aiagents4pharma/talk2scholars/tests/test_pdf_agent.py

@@ -0,0 +1,126 @@
+"""
+Unit Tests for the PDF agent.
+"""
+
+# pylint: disable=redefined-outer-name
+from unittest import mock
+import pytest
+from langchain_core.messages import HumanMessage, AIMessage
+from ..agents.pdf_agent import get_app
+from ..state.state_talk2scholars import Talk2Scholars
+
+
+@pytest.fixture(autouse=True)
+def mock_hydra_fixture():
+    """Mock Hydra configuration to prevent external dependencies."""
+    with mock.patch("hydra.initialize"), mock.patch("hydra.compose") as mock_compose:
+        # Create a mock configuration with a pdf_agent section.
+        cfg_mock = mock.MagicMock()
+        # The pdf_agent config will be accessed as cfg.agents.talk2scholars.pdf_agent in get_app.
+        cfg_mock.agents.talk2scholars.pdf_agent.some_property = "Test prompt"
+        mock_compose.return_value = cfg_mock
+        yield mock_compose
+
+
+@pytest.fixture
+def mock_tools_fixture():
+    """Mock PDF agent tools to prevent execution of real API calls."""
+    with (
+        mock.patch(
+            "aiagents4pharma.talk2scholars.agents.pdf_agent.question_and_answer_tool"
+        ) as mock_question_and_answer_tool,
+        mock.patch(
+            "aiagents4pharma.talk2scholars.agents.pdf_agent.query_results"
+        ) as mock_query_results,
+    ):
+        mock_question_and_answer_tool.return_value = {
+            "result": "Mock Question and Answer Result"
+        }
+        mock_query_results.return_value = {"result": "Mock Query Result"}
+        yield [mock_question_and_answer_tool, mock_query_results]
+
+
+@pytest.fixture
+def mock_llm():
+    """Provide a dummy language model to pass into get_app."""
+    return mock.Mock()
+
+
+@pytest.mark.usefixtures("mock_hydra_fixture")
+def test_pdf_agent_initialization(mock_llm):
+    """Test that PDF agent initializes correctly with mock configuration."""
+    thread_id = "test_thread"
+    with mock.patch(
+        "aiagents4pharma.talk2scholars.agents.pdf_agent.create_react_agent"
+    ) as mock_create:
+        mock_create.return_value = mock.Mock()
+        app = get_app(thread_id, mock_llm)
+        assert app is not None
+        assert mock_create.called
+
+
+def test_pdf_agent_invocation(mock_llm):
+    """Test that the PDF agent processes user input and returns a valid response."""
+    thread_id = "test_thread"
+    # Create a sample state with a human message.
+    mock_state = Talk2Scholars(
+        messages=[HumanMessage(content="Extract key data from PDF")]
+    )
+    with mock.patch(
+        "aiagents4pharma.talk2scholars.agents.pdf_agent.create_react_agent"
+    ) as mock_create:
+        mock_agent = mock.Mock()
+        mock_create.return_value = mock_agent
+        # Simulate a response from the PDF agent.
+        mock_agent.invoke.return_value = {
+            "messages": [
+                AIMessage(content="PDF content extracted successfully")
+            ],
+            "pdf_data": {"page": 1, "text": "Sample PDF text"},
+        }
+        app = get_app(thread_id, mock_llm)
+        result = app.invoke(
+            mock_state,
+            config={
+                "configurable": {
+                    "thread_id": thread_id,
+                    "checkpoint_ns": "test_ns",
+                    "checkpoint_id": "test_checkpoint",
+                }
+            },
+        )
+        assert "messages" in result
+        assert "pdf_data" in result
+        assert result["pdf_data"]["page"] == 1
+
+
+def test_pdf_agent_tools_assignment(request, mock_llm):
+    """Ensure that the correct tools are assigned to the PDF agent."""
+    thread_id = "test_thread"
+    mock_tools = request.getfixturevalue("mock_tools_fixture")
+    with (
+        mock.patch(
+            "aiagents4pharma.talk2scholars.agents.pdf_agent.create_react_agent"
+        ) as mock_create,
+        mock.patch(
+            "aiagents4pharma.talk2scholars.agents.pdf_agent.ToolNode"
+        ) as mock_toolnode,
+    ):
+        mock_agent = mock.Mock()
+        mock_create.return_value = mock_agent
+        mock_tool_instance = mock.Mock()
+        # For the PDF agent, we expect two tools: question_and_answer_tool and query_results.
+        mock_tool_instance.tools = mock_tools
+        mock_toolnode.return_value = mock_tool_instance
+        get_app(thread_id, mock_llm)
+        assert mock_toolnode.called
+        assert len(mock_tool_instance.tools) == 2
+
+
+def test_pdf_agent_hydra_failure(mock_llm):
+    """Test exception handling when Hydra fails to load config for PDF agent."""
+    thread_id = "test_thread"
+    with mock.patch("hydra.initialize", side_effect=Exception("Hydra error")):
+        with pytest.raises(Exception) as exc_info:
+            get_app(thread_id, mock_llm)
+        assert "Hydra error" in str(exc_info.value)