Fix mutable default previous_run parameters across assistants

peak_assistant/able_assistant/__init__.py

@@ -20,7 +20,7 @@
 #
 # SPDX-License-Identifier: MIT
 
-from typing import List
+from typing import List, Optional
 
 from autogen_core.models import UserMessage, SystemMessage
 
@@ -32,7 +32,7 @@ async def able_table(
     research_document: str,
     local_data_document: str,
     local_context: str,
-    previous_run: list[SystemMessage | UserMessage] = list(),
+    previous_run: Optional[list[SystemMessage | UserMessage]] = None,
 ) -> str:
     """
     Generate a PEAK ABLE table based on the given hypothesis and research document.

peak_assistant/data_assistant/__init__.py

@@ -20,6 +20,8 @@
 #
 # SPDX-License-Identifier: MIT
 
+from typing import Optional
+
 from autogen_agentchat.base import TaskResult
 from autogen_agentchat.messages import TextMessage
 from autogen_agentchat.agents import AssistantAgent
@@ -38,7 +40,7 @@ async def identify_data_sources(
     able_info: str,
     local_context: str,
     verbose: bool = False,
-    previous_run: list = list(),
+    previous_run: Optional[list] = None,
     mcp_server_group: str = "data_discovery",
     msg_preprocess_callback=None,
     msg_preprocess_kwargs=None,

peak_assistant/hypothesis_assistant/hypothesis_refiner_cli.py

@@ -25,7 +25,7 @@
 import os
 import sys
 import argparse
-from typing import List
+from typing import List, Optional
 from dotenv import load_dotenv
 import asyncio
 
@@ -51,7 +51,7 @@ async def refiner(
     research_document: str,
     local_data_document: str,
     verbose: bool = False,
-    previous_run: list = list(),
+    previous_run: Optional[list] = None,
     msg_preprocess_callback=None,
     msg_preprocess_kwargs=None,
     msg_postprocess_callback=None,

peak_assistant/planning_assistant/__init__.py

@@ -20,6 +20,8 @@
 #
 # SPDX-License-Identifier: MIT
 
+from typing import Optional
+
 from autogen_agentchat.messages import TextMessage
 from autogen_agentchat.agents import AssistantAgent
 from autogen_agentchat.teams import RoundRobinGroupChat
@@ -38,7 +40,7 @@ async def plan_hunt(
     data_discovery: str,
     local_context: str,
     verbose: bool = False,
-    previous_run: list = list(),
+    previous_run: Optional[list] = None,
     msg_preprocess_callback=None,
     msg_preprocess_kwargs=None,
     msg_postprocess_callback=None,

peak_assistant/research_assistant/__init__.py

@@ -39,7 +39,7 @@ async def researcher(
     technique: str,
     local_context: str,
     verbose: bool = False,
-    previous_run: list = list(),
+    previous_run: Optional[list] = None,
     mcp_server_group_external: str = "research-external",
     user_id: Optional[str] = None,
     msg_preprocess_callback=None,
@@ -397,7 +397,7 @@ async def local_data_searcher(
     local_context: str,
     research_document: str,
     verbose: bool = False,
-    previous_run: list = list(),
+    previous_run: Optional[list] = None,
     mcp_server_group_local_data: str = "local-data-search",
     user_id: Optional[str] = None,
     msg_preprocess_callback=None,