smartapi-mcp 0.3.0__tar.gz → 0.3.2__tar.gz

{smartapi_mcp-0.3.0 → smartapi_mcp-0.3.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: smartapi-mcp
-Version: 0.3.0
+Version: 0.3.2
 Summary: Create MCP servers for one or multiple APIs registered in SmartAPI registry
 Author-email: BioThings Team <help@biothings.io>
 Maintainer-email: BioThings Team <help@biothings.io>
@@ -18,6 +18,7 @@ Classifier: Programming Language :: Python :: 3.10
 Classifier: Programming Language :: Python :: 3.11
 Classifier: Programming Language :: Python :: 3.12
 Classifier: Programming Language :: Python :: 3.13
+Classifier: Programming Language :: Python :: 3.14
 Classifier: Topic :: Software Development :: Libraries :: Python Modules
 Classifier: Topic :: Internet :: WWW/HTTP :: HTTP Servers
 Classifier: Topic :: Scientific/Engineering :: Bio-Informatics

{smartapi_mcp-0.3.0 → smartapi_mcp-0.3.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "smartapi-mcp"
-version = "0.3.0"
+version = "0.3.2"
 description = "Create MCP servers for one or multiple APIs registered in SmartAPI registry"
 readme = "README.md"
 license = "Apache-2.0"
@@ -24,6 +24,7 @@ classifiers = [
     "Programming Language :: Python :: 3.11",
     "Programming Language :: Python :: 3.12",
     "Programming Language :: Python :: 3.13",
+    "Programming Language :: Python :: 3.14",
     "Topic :: Software Development :: Libraries :: Python Modules",
     "Topic :: Internet :: WWW/HTTP :: HTTP Servers",
     "Topic :: Scientific/Engineering :: Bio-Informatics",

{smartapi_mcp-0.3.0 → smartapi_mcp-0.3.2}/smartapi_mcp/__init__.py

@@ -4,7 +4,7 @@ SmartAPI MCP Server Package
 Create MCP servers for one or multiple APIs registered in SmartAPI registry.
 """
 
-__version__ = "0.3.0"
+__version__ = "0.3.2"
 __author__ = "BioThings Team"
 __email__ = "help@biothings.io"
 

{smartapi_mcp-0.3.0 → smartapi_mcp-0.3.2}/smartapi_mcp/cli.py

@@ -27,47 +27,63 @@ def main():
         help=(
             "A predefined set of SmartAPI APIs to include. One of: "
             "'biothings_core' (5 core BioThings APIs), 'biothings_test' "
-            "(core + SemmedDB), or 'biothings_all' (all BioThings APIs)."
+            "(core + SemmedDB), or 'biothings_all' (all BioThings APIs). "
+            "[env: SMARTAPI_API_SET]"
         ),
     )
     parser.add_argument(
         "--smartapi_id",
-        help="Pass a single SmartAPI (id) to create a MCP server.",
+        help=("Pass a single SmartAPI (id) to create a MCP server. [env: SMARTAPI_ID]"),
     )
     parser.add_argument(
         "--smartapi_ids",
-        help="Pass a list of SmartAPIs (comma-separated ids) to create a MCP server.",
+        help=(
+            "Pass a list of SmartAPIs (comma-separated ids) to create a MCP "
+            "server. [env: SMARTAPI_IDS]"
+        ),
     )
     parser.add_argument(
         "--smartapi_q",
         help=(
             "A SmartAPI registry search query selecting which APIs to include, "
-            "e.g. 'tags.name:biothings'."
+            "e.g. 'tags.name:biothings'. [env: SMARTAPI_Q]"
         ),
     )
     parser.add_argument(
         "--smartapi_exclude_ids",
         help=(
-            "Exclude a list of SmartAPIs (comma-separated ids) to create a MCP server."
+            "Exclude a list of SmartAPIs (comma-separated ids) to create a MCP "
+            "server. [env: SMARTAPI_EXCLUDE_IDS]"
         ),
     )
     parser.add_argument(
         "--host",
-        help="The host address for the MCP server in HTTP mode. Default is localhost.",
+        help=(
+            "The host address for the MCP server in HTTP mode. Default is "
+            "localhost. [env: SERVER_HOST]"
+        ),
     )
     parser.add_argument(
         "--port",
         type=int,
         default=8000,
-        help="The http port for the MCP server in HTTP mode. Default is 8000.",
+        help=(
+            "The http port for the MCP server in HTTP mode. Default is 8000. "
+            "[env: SERVER_PORT]"
+        ),
     )
     parser.add_argument(
         "--transport",
-        help="The transport mode for the MCP server, either stdio (default) or http.",
+        help=(
+            "The transport mode for the MCP server, either stdio (default) or "
+            "http. [env: SERVER_TRANSPORT]"
+        ),
     )
     parser.add_argument(
         "--server_name",
-        help='The name of the MCP server, default is "smartapi_mcp".',
+        help=(
+            'The name of the MCP server, default is "smartapi_mcp". [env: SERVER_NAME]'
+        ),
     )
     parser.add_argument(
         "--facade",
@@ -79,7 +95,8 @@ def main():
             "non-BioThings APIs in the set are added as per-API tools (hybrid). "
             "'auto' (default): use the facade once there are enough BioThings "
             "APIs (see --facade-threshold). 'on': always use it for BioThings "
-            "APIs. 'off': always emit faithful per-API tools for every API."
+            "APIs. 'off': always emit faithful per-API tools for every API. "
+            "[env: SMARTAPI_FACADE]"
         ),
     )
     parser.add_argument(
@@ -88,7 +105,7 @@ def main():
         default=10,
         help=(
             "Number of BioThings APIs in the set at which 'auto' switches to the "
-            "facade (default: 10)."
+            "facade (default: 10). [env: FACADE_THRESHOLD]"
         ),
     )
     parser.add_argument(
@@ -97,7 +114,8 @@ def main():
         help=(
             "Inspect BioThings specs and serve any API that has non-standard "
             "endpoints (e.g. SemmedDB's /query/ngd) with faithful per-API tools "
-            "instead of the facade. Slower startup (downloads specs upfront)."
+            "instead of the facade. Slower startup (downloads specs upfront). "
+            "[env: FACADE_STRICT]"
         ),
     )
     parser.add_argument(

{smartapi_mcp-0.3.0 → smartapi_mcp-0.3.2}/smartapi_mcp/server.py

@@ -4,6 +4,7 @@ SmartAPI MCP Server
 Main MCP server implementation for SmartAPI integration.
 """
 
+import hashlib
 import re
 
 from awslabs.openapi_mcp_server import logger
@@ -23,6 +24,15 @@ from .smartapi import (
     smartapi_spec_url,
 )
 
+# Cap names at 64 characters. The MCP spec (SEP-986) recommends 1-64 chars for
+# *tool* names as a SHOULD, but the limit is enforced as a hard error by the
+# model APIs: both Anthropic (FrontendRemoteMcpToolDefinition.name; 400 on
+# longer names) and OpenAI (^[a-zA-Z0-9_-]{1,64}$) reject names over 64 chars.
+# So prefixed per-API names must be truncated to fit. The spec does not define a
+# length for *prompt* names, but we cap them too as a harmless safeguard against
+# clients that reuse the tool-name validator.
+MAX_TOOL_NAME_LEN = 64
+
 
 async def get_mcp_server(smartapi_id: str) -> FastMCP:
     config = Config(
@@ -35,6 +45,33 @@ async def get_mcp_server(smartapi_id: str) -> FastMCP:
     return await create_mcp_server_async(config)
 
 
+def _fit_name(name: str, used: set[str]) -> str:
+    """Return a unique name no longer than :data:`MAX_TOOL_NAME_LEN` chars.
+
+    Used for both tool and prompt names. Names within the limit (and not already
+    in ``used``) are returned unchanged. Longer or colliding names are truncated
+    and given a short hash suffix derived from the *full* name, so the result
+    stays deterministic and collision-free (two different long names hash
+    differently).
+    """
+    if len(name) <= MAX_TOOL_NAME_LEN and name not in used:
+        return name
+
+    digest = hashlib.sha1(name.encode()).hexdigest()[:6]  # noqa: S324 - non-crypto
+    suffix = f"_{digest}"
+    truncated = name[: MAX_TOOL_NAME_LEN - len(suffix)].rstrip("_") + suffix
+    # Guard against the (unlikely) case where the truncated form still collides.
+    while truncated in used:
+        digest = hashlib.sha1((name + digest).encode()).hexdigest()[:6]  # noqa: S324
+        suffix = f"_{digest}"
+        truncated = name[: MAX_TOOL_NAME_LEN - len(suffix)].rstrip("_") + suffix
+    logger.debug(
+        f"Name '{name}' exceeds {MAX_TOOL_NAME_LEN} chars or collides; "
+        f"renamed to '{truncated}'."
+    )
+    return truncated
+
+
 async def _merge_servers_into(
     target: FastMCP, list_of_servers: list[FastMCP]
 ) -> FastMCP:
@@ -43,6 +80,11 @@ async def _merge_servers_into(
     Tool and prompt names are prefixed with the source server's (API) name to
     avoid conflicts. ``target`` is mutated in place and returned.
     """
+    # Seed with names already in the target (e.g. facade tools in the hybrid
+    # path) so merged per-API tools/prompts never collide with them. Tools and
+    # prompts have separate namespaces, so each gets its own set.
+    used_tool_names: set[str] = set(await target.get_tools())
+    used_prompt_names: set[str] = set(await target.get_prompts())
     for server in list_of_servers:
         api_name = re.sub(
             r"[^a-z0-9_-]", "_", getattr(server, "name", "unknown_api").lower()
@@ -51,8 +93,11 @@ async def _merge_servers_into(
         tools = await server.get_tools()
         if tools:
             for original_name, tool in tools.items():
-                # Rename the tool by prefixing with API name
-                tool.name = f"{api_name}_{original_name}"
+                # Rename the tool by prefixing with API name, keeping it within
+                # the 64-char limit that MCP clients enforce.
+                prefixed = f"{api_name}_{original_name}"
+                tool.name = _fit_name(prefixed, used_tool_names)
+                used_tool_names.add(tool.name)
                 target.add_tool(tool)
         else:
             err_msg = f"Server {server} does not have accessible tools."
@@ -62,8 +107,11 @@ async def _merge_servers_into(
         prompts = await server.get_prompts()
         if prompts:
             for original_name, prompt in prompts.items():
-                # Rename the prompt by prefixing with API name
-                prompt.name = f"{api_name}_{original_name}"
+                # Rename the prompt by prefixing with API name, keeping it
+                # within the 64-char limit that MCP clients enforce.
+                prefixed = f"{api_name}_{original_name}"
+                prompt.name = _fit_name(prefixed, used_prompt_names)
+                used_prompt_names.add(prompt.name)
                 target.add_prompt(prompt)
             logger.debug(f"Merged {len(prompts)} prompts from {api_name}")