verifiers 0.2.1.dev41__tar.gz → 0.2.1.dev42__tar.gz

{verifiers-0.2.1.dev41 → verifiers-0.2.1.dev42}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: verifiers
-Version: 0.2.1.dev41
+Version: 0.2.1.dev42
 Summary: Verifiers: Environments for LLM Reinforcement Learning
 Project-URL: Homepage, https://github.com/primeintellect-ai/verifiers
 Project-URL: Documentation, https://github.com/primeintellect-ai/verifiers

{verifiers-0.2.1.dev41 → verifiers-0.2.1.dev42}/verifiers/v1/harnesses/default/program.py

@@ -198,7 +198,8 @@ async def connect_mcp(stack: AsyncExitStack, config: dict) -> tuple[list[dict],
         session = await stack.enter_async_context(ClientSession(read, write))
         await session.initialize()
         for tool in (await session.list_tools()).tools:
-            full = f"{name}_{tool.name}"
+            # A server named "" (TOOL_PREFIX = None) advertises its tools bare.
+            full = f"{name}_{tool.name}" if name else tool.name
             tool_schemas.append(
                 {
                     "type": "function",

{verifiers-0.2.1.dev41 → verifiers-0.2.1.dev42}/verifiers/v1/harnesses/null/program.py

@@ -24,7 +24,9 @@ async def chat(
 
 async def connect_mcp(stack: AsyncExitStack, config: dict) -> tuple[list[dict], dict]:
     """Connect to each configured MCP server (a streamable-HTTP `url`); return
-    (tool schemas, dispatch mapping `<server>_<tool>` -> (session, raw tool name))."""
+    (tool schemas, dispatch mapping advertised name -> (session, raw tool name)). Tools are
+    advertised as `<server>_<tool>`; a server named `""` (TOOL_PREFIX = None) advertises its
+    tools bare, so names must be unique across the rollout's servers."""
     from mcp import ClientSession
     from mcp.client.streamable_http import (
         create_mcp_http_client,
@@ -43,7 +45,11 @@ async def connect_mcp(stack: AsyncExitStack, config: dict) -> tuple[list[dict],
         session = await stack.enter_async_context(ClientSession(read, write))
         await session.initialize()
         for tool in (await session.list_tools()).tools:
-            full = f"{name}_{tool.name}"
+            full = f"{name}_{tool.name}" if name else tool.name
+            if full in dispatch:
+                raise ValueError(
+                    f"duplicate tool name {full!r} across servers; keep qualified names"
+                )
             tool_schemas.append(
                 {
                     "type": "function",

{verifiers-0.2.1.dev41 → verifiers-0.2.1.dev42}/verifiers/v1/mcp/server.py

@@ -113,8 +113,10 @@ def _import_ref(ref: str) -> object:
 
 
 class ServerBase(Generic[ConfigT, StateT]):
-    # The empty value falls back to the snake-cased class name.
-    TOOL_PREFIX: ClassVar[str] = ""
+    TOOL_PREFIX: ClassVar[str | None] = ""
+    """The empty value falls back to the snake-cased class name. None advertises the server's
+    tools bare (no `<server>_` prefix); name collisions across servers are then the taskset
+    author's concern."""
 
     EXTRAS: ClassVar[tuple[str, ...]] = ()
     """Package extras the server's module needs, applied at sandbox install."""
@@ -214,6 +216,8 @@ class ServerBase(Generic[ConfigT, StateT]):
 
     @property
     def server_name(self) -> str:
+        if self.TOOL_PREFIX is None:
+            return ""
         return self.TOOL_PREFIX or "".join(
             ("_" + c.lower() if c.isupper() else c) for c in type(self).__name__
         ).lstrip("_")