Explorar el Código

移除项目列表的url

Lu Xianghui hace 1 mes
padre
commit
be5c7b8622

+ 1 - 4
data_collector_mcp/common_server.py

@@ -17,8 +17,7 @@ from .mcp_app import mcp
     title="Project List",
     description=(
         "List enabled 汇采 projects available to this MCP service, including "
-        "project_key, project_name, base_url, and data_collector_base_url. "
-        "Call this first to choose project_key."
+        "project_key and project_name. Call this first to choose project_key."
     ),
     tags={"project", "list"},
 )
@@ -28,8 +27,6 @@ def project_list() -> dict[str, Any]:
         {
             "project_key": item["project_key"],
             "project_name": item["project_name"],
-            "base_url": item["base_url"],
-            "data_collector_base_url": item["data_collector_base_url"],
         }
         for item in projects
         if item["enabled"]

+ 1 - 2
data_collector_mcp/mcp_app.py

@@ -7,8 +7,7 @@ from .tool_logging import ToolCallLoggingMiddleware
 
 SERVER_INSTRUCTIONS = (
     "Data collector tools. Use project.list first to choose a project_key. "
-    "base_url is used for login and gateway test APIs. data_collector_base_url "
-    "is required for collector management APIs and never falls back to base_url. "
+    "Gateway and collector API endpoints are resolved internally by project_key. "
     "For Modbus, gateway word_byte_order values ABCD/BADC/CDAB/DCBA must be "
     "mapped to collector byte_order/word_order before creating a device. "
     "For BACnet/IP, use bacnet.point_search first when object_type/object_id are unknown."

+ 3 - 4
tests/test_server_tools.py

@@ -86,19 +86,18 @@ class ServerToolTests(unittest.TestCase):
                     {
                         "project_key": "a-dev",
                         "project_name": "Dev",
-                        "base_url": "http://gateway.dev",
-                        "data_collector_base_url": "http://collector.dev",
                     },
                     {
                         "project_key": "z-prod",
                         "project_name": "Prod",
-                        "base_url": "http://gateway.prod",
-                        "data_collector_base_url": "http://collector.prod",
                     },
                 ],
                 "total": 2,
             },
         )
+        for project in result["projects"]:
+            self.assertNotIn("base_url", project)
+            self.assertNotIn("data_collector_base_url", project)
 
     def test_common_device_tools_forward_to_api(self) -> None:
         with patch("data_collector_mcp.common_server.api_list_devices", return_value={"state": 0}) as list_devices: