fix: ensure correct agent URL path parsing and registration in Starlette app

Signed-off-by: Shingo OKAWA <[email protected]>

Author: ognis1205

pyproject.toml

@@ -62,6 +62,11 @@ exclude = [
 vcs = "git"
 style = "pep440"
 
+[tool.uv.workspace]
+members = [
+    "examples/apicatalog",
+]
+
 [dependency-groups]
 dev = [
     "datamodel-code-generator>=0.30.0",

src/a2a/server/apps/__init__.py

@@ -1,6 +1,10 @@
 """HTTP application components for the A2A server."""
 
-from a2a.server.apps.starlette_app import A2AStarletteApplication
+from a2a.server.apps.starlette_app import A2AStarletteApplication, A2AStarletteRouteBuilder, A2AStarletteBuilder
 
 
-__all__ = ['A2AStarletteApplication']
+__all__ = [
+    'A2AStarletteApplication',
+    'A2AStarletteRouteBuilder',
+    'A2AStarletteBuilder',
+]

src/a2a/server/apps/starlette_app.py

@@ -809,6 +809,36 @@ def build(self) -> list[Route]:
         return routes
 
 
+def _join_url(base: str, *paths: str) -> str:
+    """Joins a base URL with one or more URL path fragments into a normalized absolute URL.
+
+    This method ensures that redundant slashes are removed between path segments,
+    and that the resulting URL is correctly formatted.
+
+    Args:
+        base: The base URL.
+        *paths: One or more URL fragments to append to the base path.
+
+    Returns:
+        A well-formed absolute URL with the joined path components.
+    """
+    parsed = urlparse(base)
+    clean_paths = [p.strip('/') for p in paths]
+    joined_path = posixpath.join(parsed.path.rstrip('/'), *clean_paths)
+    return urlunparse(parsed._replace(path='/' + joined_path))
+
+def _get_path_from_url(url: str) -> str:
+    """Extracts and returns the path component from a full URL.
+
+    Args:
+        url: The full URL string.
+
+    Returns:
+        The path component of the URL.
+    """
+    path = urlparse(url).path
+    return path if path else '/'
+
 class A2AStarletteBuilder:
     """Builder class for assembling a Starlette application with A2A protocol routes.
 
@@ -833,25 +863,6 @@ def __init__(self):
         self._mounts: list[Mount] = []
         self._catalog_links: list[AgentLinkContext] = []
 
-    @staticmethod
-    def _join_url(base: str, *paths: str) -> str:
-        """Joins a base URL with one or more URL path fragments into a normalized absolute URL.
-
-        This method ensures that redundant slashes are removed between path segments,
-        and that the resulting URL is correctly formatted.
-
-        Args:
-            base: The base URL.
-            *paths: One or more URL fragments to append to the base path.
-
-        Returns:
-            A well-formed absolute URL with the joined path components.
-        """
-        parsed = urlparse(base)
-        clean_paths = [p.strip('/') for p in paths]
-        joined_path = posixpath.join(parsed.path.rstrip('/'), *clean_paths)
-        return urlunparse(parsed._replace(path='/' + joined_path))
-
     async def _handle_get_api_catalog(self, request: Request) -> JSONResponse:
         """Handles GET requests for the AgentCatalog endpoint.
 
@@ -866,14 +877,14 @@ async def _handle_get_api_catalog(self, request: Request) -> JSONResponse:
 
     def mount(
         self,
-        path: str,
         route_builder: A2AStarletteRouteBuilder,
     ) -> Self:
         """Mounts routes generated by the given builder and adds metadata to the agent catalog.
 
         Raises:
             ValueError: If a mount for the given path already exists.
         """
+        path = _get_path_from_url(route_builder.agent_card.url)
         if any(
             posixpath.normpath(mount.path) == posixpath.normpath(path)
             for mount in self._mounts