bohr-agent-sdk 0.1.109__tar.gz → 0.1.111__tar.gz

{bohr_agent_sdk-0.1.109 → bohr_agent_sdk-0.1.111}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: bohr-agent-sdk
-Version: 0.1.109
+Version: 0.1.111
 Summary: SDK for scientific agents
 Home-page: https://github.com/dptech-corp/bohr-agent-sdk/
 Author: DP Technology

{bohr_agent_sdk-0.1.109 → bohr_agent_sdk-0.1.111}/setup.py

@@ -9,7 +9,7 @@ with open("README.md", "r", encoding="utf-8") as fh:
 
 setup(
     name="bohr-agent-sdk",
-    version="0.1.109",
+    version="0.1.111",
     description="SDK for science agent and mcp tools",
     long_description=long_description,
     long_description_content_type="text/markdown",

{bohr_agent_sdk-0.1.109 → bohr_agent_sdk-0.1.111}/src/bohr_agent_sdk.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: bohr-agent-sdk
-Version: 0.1.109
+Version: 0.1.111
 Summary: SDK for scientific agents
 Home-page: https://github.com/dptech-corp/bohr-agent-sdk/
 Author: DP Technology

{bohr_agent_sdk-0.1.109 → bohr_agent_sdk-0.1.111}/src/dp/agent/client/mcp_client.py

@@ -125,7 +125,7 @@ class MCPClient:
 
         executor = arguments.get("executor")
         storage = arguments.get("storage")
-        res = await self.session.call_tool("submit_" + tool_name, arguments)
+        res = await self.call_tool("submit_" + tool_name, arguments)
         if res.isError:
             logger.error("Failed to submit %s: %s" % (
                 tool_name, res.content[0].text))
@@ -137,7 +137,7 @@ class MCPClient:
             logger.info(job_info["extra_info"])
 
         while True:
-            res = await self.session.call_tool("query_job_status", {
+            res = await self.call_tool("query_job_status", {
                 "job_id": job_id, "executor": executor})
             if res.isError:
                 logger.error(res.content[0].text)
@@ -148,7 +148,7 @@ class MCPClient:
                     break
             await asyncio.sleep(self.query_interval)
 
-        res = await self.session.call_tool("get_job_results", {
+        res = await self.call_tool("get_job_results", {
             "job_id": job_id, "executor": executor, "storage": storage})
         if res.isError:
             logger.error("Job %s failed: %s" % (job_id, res.content[0].text))

{bohr_agent_sdk-0.1.109 → bohr_agent_sdk-0.1.111}/src/dp/agent/server/calculation_mcp_server.py

@@ -7,7 +7,7 @@ from copy import deepcopy
 from datetime import datetime
 from pathlib import Path
 from urllib.parse import urlparse
-from typing import Any, Literal, Optional, TypedDict
+from typing import Any, Literal, Optional, TypedDict, List, Dict
 
 import mcp
 from mcp.server.fastmcp import FastMCP
@@ -133,6 +133,69 @@ def handle_input_artifacts(fn, kwargs, storage):
                 "storage_type": scheme,
                 "uri": uri,
             }
+        elif param.annotation is List[Path] or (
+            param.annotation is Optional[List[Path]] and
+                kwargs.get(name) is not None):
+            uris = kwargs[name]
+            new_paths = []
+            for uri in uris:
+                scheme, key = parse_uri(uri)
+                if scheme == storage_type:
+                    s = storage
+                else:
+                    s = storage_dict[scheme]()
+                path = s.download(key, "inputs/%s" % name)
+                new_paths.append(Path(path))
+                logger.info("Artifact %s downloaded to %s" % (
+                    uri, path))
+            kwargs[name] = new_paths
+            input_artifacts[name] = {
+                "storage_type": storage_type,
+                "uri": uris,
+            }
+        elif param.annotation is Dict[str, Path] or (
+            param.annotation is Optional[Dict[str, Path]] and
+                kwargs.get(name) is not None):
+            uris_dict = kwargs[name]
+            new_paths_dict = {}
+            for key_name, uri in uris_dict.items():
+                scheme, key = parse_uri(uri)
+                if scheme == storage_type:
+                    s = storage
+                else:
+                    s = storage_dict[scheme]()
+                path = s.download(key, f"inputs/{name}/{key_name}")
+                new_paths_dict[key_name] = Path(path)
+                logger.info("Artifact %s (key=%s) downloaded to %s" % (
+                    uri, key_name, path))
+            kwargs[name] = new_paths_dict
+            input_artifacts[name] = {
+                "storage_type": storage_type,
+                "uri": uris_dict,
+            }
+        elif param.annotation is Dict[str, List[Path]] or (
+            param.annotation is Optional[Dict[str, List[Path]]] and
+                kwargs.get(name) is not None):
+            uris_dict = kwargs[name]
+            new_paths_dict = {}
+            for key_name, uris in uris_dict.items():
+                new_paths = []
+                for uri in uris:
+                    scheme, key = parse_uri(uri)
+                    if scheme == storage_type:
+                        s = storage
+                    else:
+                        s = storage_dict[scheme]()
+                    path = s.download(key, f"inputs/{name}/{key_name}")
+                    new_paths.append(Path(path))
+                    logger.info("Artifact %s (key=%s) downloaded to %s" % (
+                        uri, key_name, path))
+                new_paths_dict[key_name] = new_paths
+            kwargs[name] = new_paths_dict
+            input_artifacts[name] = {
+                "storage_type": storage_type,
+                "uri": uris_dict,
+            }
     return kwargs, input_artifacts
 
 
@@ -152,6 +215,20 @@ def handle_output_artifacts(results, exec_id, storage):
                     "storage_type": storage_type,
                     "uri": uri,
                 }
+            elif isinstance(results[name], list) and all(isinstance(item, Path) for item in results[name]):
+                new_uris = []
+                for item in results[name]:
+                    key = storage.upload("%s/outputs/%s" % (exec_id, name),
+                                         item)
+                    uri = storage_type + "://" + key
+                    logger.info("Artifact %s uploaded to %s" % (
+                        item, uri))
+                    new_uris.append(uri)
+                results[name] = new_uris
+                output_artifacts[name] = {
+                    "storage_type": storage_type,
+                    "uri": new_uris,
+                }
     return results, output_artifacts
 
 
@@ -211,6 +288,30 @@ class CalculationMCPServer:
                     parameters.append(inspect.Parameter(
                         name=param.name, default=param.default,
                         annotation=Optional[str], kind=param.kind))
+                elif param.annotation is List[Path]:
+                    parameters.append(inspect.Parameter(
+                        name=param.name, default=param.default,
+                        annotation=List[str], kind=param.kind))
+                elif param.annotation is Optional[List[Path]]:
+                    parameters.append(inspect.Parameter(
+                        name=param.name, default=param.default,
+                        annotation=Optional[List[str]], kind=param.kind))
+                elif param.annotation is Dict[str, Path]:
+                    parameters.append(inspect.Parameter(
+                        name=param.name, default=param.default,
+                        annotation=Dict[str, str], kind=param.kind))
+                elif param.annotation is Optional[Dict[str, Path]]:
+                    parameters.append(inspect.Parameter(
+                        name=param.name, default=param.default,
+                        annotation=Optional[Dict[str, str]], kind=param.kind))
+                elif param.annotation is Dict[str, List[Path]]:
+                    parameters.append(inspect.Parameter(
+                        name=param.name, default=param.default,
+                        annotation=Dict[str, List[str]], kind=param.kind))
+                elif param.annotation is Optional[Dict[str, List[Path]]]:
+                    parameters.append(inspect.Parameter(
+                        name=param.name, default=param.default,
+                        annotation=Optional[Dict[str, List[str]]], kind=param.kind))
                 else:
                     parameters.append(param)
             for param in new_typed_signature.parameters.values():

{bohr_agent_sdk-0.1.109 → bohr_agent_sdk-0.1.111}/src/dp/agent/server/executor/base_executor.py

@@ -5,6 +5,8 @@ from collections.abc import Callable
 from typing import Any, Literal, TypedDict
 
 from mcp.server.fastmcp.server import Context
+from mcp.shared.context import RequestContext
+from starlette.requests import Request
 logger = logging.getLogger(__name__)
 
 
@@ -27,6 +29,26 @@ class BaseExecutor(ABC):
     def get_results(self, job_id: str) -> dict:
         pass
 
+    def prune_context(self, kwargs: dict):
+        for key, value in kwargs.items():
+            if isinstance(value, Context):
+                context = Context(request_context=RequestContext(
+                    request_id=value.request_context.request_id,
+                    meta=value.request_context.meta,
+                    session=None,
+                    lifespan_context=value.request_context.lifespan_context,
+                    request=Request(
+                        scope={
+                            k: v for k, v in
+                            value.request_context.request.scope.items()
+                            if k not in ["app", "router", "endpoint",
+                                         "starlette.exception_handlers"]
+                        },
+                    )
+                ))
+                kwargs[key] = context
+        return kwargs
+
     async def async_run(
         self, fn: Callable, kwargs: dict, context: Context,
         trace_id: str) -> TypedDict(

{bohr_agent_sdk-0.1.109 → bohr_agent_sdk-0.1.111}/src/dp/agent/server/executor/dispatcher_executor.py

@@ -129,12 +129,13 @@ class DispatcherExecutor(BaseExecutor):
         self.resources["envs"]["DP_AGENT_RUNNING_MODE"] = "1"
 
     def submit(self, fn, kwargs):
+        kwargs = self.prune_context(kwargs)
         script = ""
         fn_name = fn.__name__
         func_def_script, packages = get_func_def_script(fn)
         self.python_packages.extend(packages)
 
-        script += "import asyncio, jsonpickle, os\n"
+        script += "import asyncio, jsonpickle, os, shutil\n"
         script += "from pathlib import Path\n\n"
         script += "if __name__ == \"__main__\":\n"
         script += "    cwd = os.getcwd()\n"
@@ -148,11 +149,24 @@ class DispatcherExecutor(BaseExecutor):
             script += "        results = asyncio.run(%s(**kwargs))\n" % fn_name
         else:
             script += "        results = %s(**kwargs)\n" % fn_name
+            script += "        result_dir = None\n"
+            script += "        import uuid\n"
             script += "        if isinstance(results, dict):\n"
             script += "            for name in results:\n"
             script += "                if isinstance(results[name], Path):\n"
-            script += "                    results[name] = " \
-                      "results[name].absolute().relative_to(cwd)\n"
+            script += "                    if not results[name].absolute().is_relative_to(cwd):\n"
+            script += "                        if result_dir is None:\n"
+            script += "                            result_dir = Path('result_files_dir_' + str(uuid.uuid4()))\n"
+            script += "                            result_dir.mkdir(parents=True, exist_ok=True)\n"
+            script += "                        dest_path = result_dir / results[name].absolute().relative_to('/')\n"
+            script += "                        dest_path.parent.mkdir(parents=True, exist_ok=True)\n"
+            script += "                        if results[name].is_file():\n"
+            script += "                            shutil.copy2(results[name], dest_path)\n"
+            script += "                        elif results[name].is_dir():\n"
+            script += "                            shutil.copytree(results[name], dest_path, dirs_exist_ok=True)\n"
+            script += "                        results[name] = dest_path.absolute().relative_to(cwd)\n"
+            script += "                    else:\n"
+            script += "                        results[name] = results[name].absolute().relative_to(cwd)\n"
         script += "    except Exception as e:\n"
         script += "        os.chdir(cwd)\n"
         script += "        with open('err', 'w') as f:\n"

{bohr_agent_sdk-0.1.109 → bohr_agent_sdk-0.1.111}/src/dp/agent/server/executor/local_executor.py

@@ -88,6 +88,7 @@ class LocalExecutor(BaseExecutor):
         """
         self.env = env or {}
         self.dflow = dflow
+        self.workflow_id = None
 
     def set_env(self):
         old_env = {}
@@ -105,6 +106,7 @@ class LocalExecutor(BaseExecutor):
                 del os.environ[k]
 
     def submit(self, fn, kwargs):
+        kwargs = self.prune_context(kwargs)
         os.environ["DP_AGENT_RUNNING_MODE"] = "1"
         old_env = self.set_env()
         params = {"fn": fn, "kwargs": kwargs}
@@ -123,6 +125,7 @@ class LocalExecutor(BaseExecutor):
                     match_link = re.search(DFLOW_LINK_PATTERN, log)
                     if match_id and match_link:
                         wf_id = match_id.group(1)
+                        self.workflow_id = wf_id
                         wf_uid = match_id.group(2)
                         wf_link = match_link.group(1)
                         extra_info["workflow_id"] = wf_id
@@ -155,11 +158,18 @@ class LocalExecutor(BaseExecutor):
             return "Failed"
 
     def terminate(self, job_id):
+        if self.workflow_id is not None:
+            try:
+                from dflow import Workflow
+                wf = Workflow(id=self.workflow_id)
+                wf.terminate()
+            except Exception as e:
+                logger.error(f"Failed to terminate workflow: {e}")
         try:
             p = psutil.Process(int(job_id))
             p.terminate()
-        except Exception:
-            pass
+        except Exception as e:
+            logger.error(f"Failed to terminate process: {e}")
 
     def get_results(self, job_id):
         if os.path.isfile("%s.txt" % job_id):