gfp-mcp 0.2.1__py3-none-any.whl → 0.2.4__py3-none-any.whl

{gfp_mcp-0.2.1.dist-info → gfp_mcp-0.2.4.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: gfp-mcp
-Version: 0.2.1
+Version: 0.2.4
 Summary: Model Context Protocol (MCP) server for GDSFactory+ photonic IC design
 Author: GDSFactory+ Team
 License: MIT
@@ -33,13 +33,15 @@ Requires-Dist: pytest>=7.0.0; extra == "dev"
 Requires-Dist: pytest-cov>=4.0.0; extra == "dev"
 Requires-Dist: pytest-asyncio>=0.21.0; extra == "dev"
 Requires-Dist: ruff>=0.1.0; extra == "dev"
+Requires-Dist: bump-my-version>=0.26.0; extra == "dev"
+Requires-Dist: build>=1.4.0; extra == "dev"
 Dynamic: license-file
 
 # GDSFactory+ MCP Server
 
 [![PyPI version](https://img.shields.io/pypi/v/gfp-mcp.svg)](https://pypi.org/project/gfp-mcp/)
 [![Python versions](https://img.shields.io/pypi/pyversions/gfp-mcp.svg)](https://pypi.org/project/gfp-mcp/)
-[![Tests](https://github.com/doplaydo/gfp-mcp/workflows/Tests/badge.svg)](https://github.com/doplaydo/gfp-mcp/actions)
+[![Tests](https://github.com/doplaydo/gfp-mcp/actions/workflows/test.yml/badge.svg)](https://github.com/doplaydo/gfp-mcp/actions)
 [![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](https://opensource.org/licenses/MIT)
 
 Model Context Protocol (MCP) server for GDSFactory+ that enables AI assistants like Claude to design and build photonic integrated circuits.
@@ -57,15 +59,32 @@ This MCP server connects AI assistants to [GDSFactory+](https://gdsfactory.com),
 
 ### 2. Install the MCP Server
 
+**With uv (recommended):**
+
+If you don't have `uv` installed, see the [uv installation guide](https://docs.astral.sh/uv/getting-started/installation/).
+
 ```bash
-pip install gfp-mcp
+uv tool install gfp-mcp
 ```
 
-Or with uv:
+<details>
+<summary>Ephemeral approach</summary>
+
+Run without installing:
 
 ```bash
 uvx --from gfp-mcp gfp-mcp-serve
 ```
+</details>
+
+<details>
+<summary><strong>Alternative: pip install</strong></summary>
+
+```bash
+pip install gfp-mcp
+```
+
+</details>
 
 ### 3. Connect to Your AI Assistant
 
@@ -143,14 +162,12 @@ Try these commands with your AI assistant:
 
 ## Available Tools
 
-- **build_cell** - Build a single GDS cell by name
-- **build_cells** - Build multiple GDS cells in batch
+- **build_cells** - Build one or more GDS cells by name (pass a list, can be single-item)
 - **list_cells** - List all available photonic components
 - **get_cell_info** - Get detailed component metadata
-- **download_gds** - Download built GDS files
 - **list_projects** - List all running GDSFactory+ server instances
 - **get_project_info** - Get detailed information about a specific project
-- **check_drc** - Run Design Rule Check verification
+- **check_drc** - Run Design Rule Check verification (returns structured format with all violations including simplified location data for LLM-friendly troubleshooting)
 - **check_connectivity** - Run connectivity verification
 - **check_lvs** - Run Layout vs. Schematic verification
 

gfp_mcp-0.2.4.dist-info/RECORD

@@ -0,0 +1,14 @@
+gfp_mcp-0.2.4.dist-info/licenses/LICENSE,sha256=ixSuHdKKXzNJw_eTgAxHzaCNIds8k48hytA_eJgA8gQ,225
+mcp_standalone/__init__.py,sha256=1-BT202aWn5Uwt-5bDHyGtBw3ObSKhK9ATzQlIXiJdw,1069
+mcp_standalone/client.py,sha256=LWO1emsiUa4Dg9yXH0FO7-LHV2ngxOov_acwH7JnVFo,9902
+mcp_standalone/config.py,sha256=bxJXYioVhx5FnS_dzvIEyVjaQbC91s6Pkl0tCRms3Ig,1225
+mcp_standalone/mappings.py,sha256=LFjo8Q5olO3LQM4Jy4h6xa4CD3bMWlyG_RtfDJGvQx4,15928
+mcp_standalone/registry.py,sha256=ozDrMPksWoyKAkNd9MH2g8FRoN0XEQKWnLwZsgDxWLc,6127
+mcp_standalone/resources.py,sha256=upY93XVemc5ezx1p_YML57O6HYVahCs2RsYXn4Dw_X0,3517
+mcp_standalone/server.py,sha256=FSjGSjwpe_D2idMrRrAMTq9pnwBToK2vZ1VtzPIMyeg,8788
+mcp_standalone/tools.py,sha256=qQAI6-Xb0sWUTJyYz5a47830cerbjwO5euIRhDmBRUk,16468
+gfp_mcp-0.2.4.dist-info/METADATA,sha256=v99aAe-JU0aQ7pmzICvHhoGLaKv4GoG4F9fNkxJHbyw,7369
+gfp_mcp-0.2.4.dist-info/WHEEL,sha256=wUyA8OaulRlbfwMtmQsvNngGrxQHAvkKcvRmdizlJi0,92
+gfp_mcp-0.2.4.dist-info/entry_points.txt,sha256=mgyus9dsB_8mjgnztuHNPqzPi-7HcPg1iYzfM5NMIjk,61
+gfp_mcp-0.2.4.dist-info/top_level.txt,sha256=g2hRJHoDDPNtrNdXR70T7FR9Ev6DTRJiGW7ZvlvnXMc,15
+gfp_mcp-0.2.4.dist-info/RECORD,,

mcp_standalone/__init__.py

@@ -39,4 +39,4 @@ __all__ = [
     "get_resource_content",
 ]
 
-__version__ = "0.2.1"
+__version__ = "0.2.4"

mcp_standalone/client.py

@@ -43,6 +43,21 @@ class FastAPIClient:
         self._client: httpx.AsyncClient | None = None
         self._registry = ServerRegistry()
 
+    def _has_available_servers(self) -> bool:
+        """Check if any servers are available in the registry."""
+        return len(self._registry.list_servers()) > 0
+
+    def _get_default_server_url(self) -> str | None:
+        """Get the first available server URL from registry if no base_url configured."""
+        if self.base_url:
+            return self.base_url
+
+        servers = self._registry.list_servers()
+        if servers:
+            return f"http://localhost:{servers[0].port}"
+
+        return None
+
     async def __aenter__(self) -> Self:
         """Enter async context."""
         await self.start()
@@ -55,12 +70,17 @@ class FastAPIClient:
     async def start(self) -> None:
         """Start the HTTP client with connection pooling."""
         if self._client is None:
+            base_url = self.base_url or "http://localhost"
+
             self._client = httpx.AsyncClient(
-                base_url=self.base_url,
+                base_url=base_url,
                 timeout=httpx.Timeout(self.timeout),
                 limits=httpx.Limits(max_keepalive_connections=5, max_connections=10),
             )
-            logger.debug("HTTP client started with base URL: %s", self.base_url)
+            logger.debug(
+                "HTTP client started with base URL: %s (resolved per-request if needed)",
+                self.base_url or "from registry",
+            )
 
     async def close(self) -> None:
         """Close the HTTP client."""
@@ -79,22 +99,46 @@ class FastAPIClient:
             Base URL for the request
 
         Raises:
-            ValueError: If project not found in registry
+            ValueError: If base URL cannot be resolved
         """
-        # If no project specified, use default base_url
-        if project is None:
+        if project is not None:
+            server_info = self._registry.get_server_by_project(project)
+            if server_info is None:
+                available = self._registry.list_servers()
+                if available:
+                    project_list = ", ".join([s.project_name for s in available[:3]])
+                    if len(available) > 3:
+                        project_list += f", ... and {len(available) - 3} more"
+                    msg = (
+                        f"Project '{project}' not found in registry. "
+                        f"Available projects: {project_list}. "
+                        "Use list_projects tool to see all running servers."
+                    )
+                else:
+                    msg = (
+                        f"Project '{project}' not found. No GDSFactory+ servers are running. "
+                        "Please open a GDSFactory+ project in VSCode with the extension installed."
+                    )
+                raise ValueError(msg)
+
+            return f"http://localhost:{server_info.port}"
+
+        if self.base_url:
             return self.base_url
 
-        # Look up project in registry
-        server_info = self._registry.get_server_by_project(project)
-        if server_info is None:
-            msg = (
-                f"Project '{project}' not found in registry. "
-                "Make sure the server is running for this project."
+        default_url = self._get_default_server_url()
+        if default_url:
+            logger.info(
+                "No project specified, using first available server: %s", default_url
             )
-            raise ValueError(msg)
+            return default_url
 
-        return f"http://localhost:{server_info.port}"
+        msg = (
+            "No project specified and no GDSFactory+ servers are running. "
+            "Either: (1) Start a server by opening a GDSFactory+ project in VSCode, "
+            "(2) Specify a project parameter, or (3) Set GFP_API_URL environment variable."
+        )
+        raise ValueError(msg)
 
     async def request(
         self,
@@ -125,7 +169,6 @@ class FastAPIClient:
         if self._client is None:
             await self.start()
 
-        # Resolve the base URL for this request
         base_url = self._resolve_base_url(project)
 
         last_error = None
@@ -143,7 +186,6 @@ class FastAPIClient:
                     base_url,
                 )
 
-                # Build full URL with the resolved base URL
                 full_url = f"{base_url}{path}"
 
                 response = await self._client.request(  # type: ignore[union-attr]
@@ -155,7 +197,6 @@ class FastAPIClient:
                 )
                 response.raise_for_status()
 
-                # Try to parse JSON, fall back to text
                 try:
                     return response.json()
                 except (ValueError, TypeError):
@@ -165,19 +206,16 @@ class FastAPIClient:
                 last_error = e
                 logger.warning("Request failed (attempt %d): %s", attempt + 1, e)
 
-                # Don't retry on client errors (4xx)
                 if (
                     isinstance(e, httpx.HTTPStatusError)
                     and 400 <= e.response.status_code < 500
                 ):
                     raise
 
-                # Exponential backoff for retries
                 if attempt < MCPConfig.MAX_RETRIES - 1:
                     await asyncio.sleep(backoff)
                     backoff *= 2
 
-        # All retries failed
         logger.error("All %d attempts failed", MCPConfig.MAX_RETRIES)
         raise last_error  # type: ignore[misc]
 

mcp_standalone/config.py

@@ -15,34 +15,28 @@ class MCPConfig:
     that proxies requests to the FastAPI backend.
     """
 
-    # FastAPI base URL (default: http://localhost:8787)
-    # This default is primarily for backward compatibility.
-    # The MCP server automatically discovers running servers via the registry.
-    API_URL: Final[str] = os.getenv("GFP_API_URL", "http://localhost:8787")
+    API_URL: Final[str | None] = os.getenv("GFP_API_URL")
 
-    # Timeout for tool calls in seconds (default: 300 = 5 minutes)
     TIMEOUT: Final[int] = int(os.getenv("GFP_MCP_TIMEOUT", "300"))
 
-    # Enable debug logging
     DEBUG: Final[bool] = os.getenv("GFP_MCP_DEBUG", "false").lower() in (
         "true",
         "1",
         "yes",
     )
 
-    # Retry configuration
     MAX_RETRIES: Final[int] = 3
     RETRY_BACKOFF: Final[float] = 0.5  # Initial backoff in seconds
 
     @classmethod
-    def get_api_url(cls, override: str | None = None) -> str:
+    def get_api_url(cls, override: str | None = None) -> str | None:
         """Get the FastAPI base URL.
 
         Args:
             override: Optional URL to override the environment variable
 
         Returns:
-            The API base URL
+            The API base URL or None if not configured
         """
         return override or cls.API_URL
 

mcp_standalone/mappings.py

@@ -6,6 +6,7 @@ requests to the FastAPI backend, and how responses are transformed back.
 
 from __future__ import annotations
 
+import xml.etree.ElementTree as ET
 from collections.abc import Callable
 from typing import Any
 
@@ -43,24 +44,6 @@ class EndpointMapping:
         self.response_transformer = response_transformer or (lambda x: x)
 
 
-def _build_cell_request(args: dict[str, Any]) -> dict[str, Any]:
-    """Transform build_cell MCP args to FastAPI params.
-
-    Args:
-        args: MCP tool arguments
-
-    Returns:
-        Dict with 'params' key for query parameters
-    """
-    return {
-        "params": {
-            "name": args["name"],
-            "with_metadata": args.get("with_metadata", True),
-            "register": args.get("register", True),
-        }
-    }
-
-
 def _build_cells_request(args: dict[str, Any]) -> dict[str, Any]:
     """Transform build_cells MCP args to FastAPI params.
 
@@ -105,58 +88,272 @@ def _get_cell_info_request(args: dict[str, Any]) -> dict[str, Any]:
     return {"params": {"name": args["name"]}}
 
 
-def _download_gds_request(args: dict[str, Any]) -> dict[str, Any]:
-    """Transform download_gds MCP args to FastAPI path.
+def _check_drc_request(args: dict[str, Any]) -> dict[str, Any]:
+    """Transform check_drc MCP args to FastAPI params.
 
     Args:
         args: MCP tool arguments
 
     Returns:
-        Dict with modified 'path' for the endpoint
+        Dict with 'json_data' key for request body
     """
-    path = args["path"]
-    # The path template in FastAPI is /api/download/{path:path}.gds
-    # We need to construct the full path
-    return {"path": f"/api/download/{path}.gds"}
+    json_data: dict[str, Any] = {"path": args["path"]}
 
+    if "pdk" in args and args["pdk"]:
+        json_data["pdk"] = args["pdk"]
+    if "process" in args and args["process"]:
+        json_data["process"] = args["process"]
+    if "timeout" in args and args["timeout"]:
+        json_data["timeout"] = args["timeout"]
+    if "host" in args and args["host"]:
+        json_data["host"] = args["host"]
 
-def _download_gds_response(response: Any) -> dict[str, Any]:
-    """Transform download_gds response to MCP format.
+    return {"json_data": json_data}
+
+
+def _check_drc_response(response: Any) -> dict[str, Any]:
+    """Transform check_drc response to LLM-friendly format.
+
+    Parses KLayout RDB XML and extracts actionable violation summary
+    without polygon coordinates or verbose metadata. Computes simplified
+    location data (bounding box, centroid, size) from polygon coordinates
+    to reduce token usage by ~82% while preserving all violation details.
 
     Args:
-        response: FastAPI response (file path or error)
+        response: FastAPI response (XML string or dict with 'content' key)
 
     Returns:
-        Formatted response with file path
+        Structured summary with all violations including location data,
+        or error dict if parsing fails
     """
-    # If response is a string (text), it's likely the file content or path
-    if isinstance(response, str):
-        return {"status": "success", "message": response}
-    return response
+    try:
+        if isinstance(response, dict) and "content" in response:
+            xml_str = response["content"]
+        elif isinstance(response, str):
+            xml_str = response
+        else:
+            return {
+                "error": f"Unexpected response type: {type(response).__name__}",
+                "suggestion": "Expected XML string or dict with 'content' key",
+            }
+    except Exception as e:
+        return {
+            "error": f"Failed to extract XML from response: {e}",
+            "response_preview": str(response)[:500],
+        }
 
+    try:
+        root = ET.fromstring(xml_str)
+    except ET.ParseError as e:
+        return {
+            "error": f"Failed to parse DRC XML: {e}",
+            "raw_preview": xml_str[:500],
+            "suggestion": "Check if DRC server returned valid XML",
+        }
 
-def _check_drc_request(args: dict[str, Any]) -> dict[str, Any]:
-    """Transform check_drc MCP args to FastAPI params.
+    categories_map = {}
+    for category in root.findall(".//categories/category"):
+        name_elem = category.find("name")
+        desc_elem = category.find("description")
+        if name_elem is not None and name_elem.text:
+            categories_map[name_elem.text] = (
+                desc_elem.text
+                if desc_elem is not None and desc_elem.text
+                else name_elem.text
+            )
+
+    cells = []
+    for cell in root.findall(".//cells/cell"):
+        name_elem = cell.find("name")
+        if name_elem is not None and name_elem.text:
+            cells.append(name_elem.text)
+
+    violations = []
+    violation_counts: dict[str, int] = {}
+
+    for item in root.findall(".//items/item"):
+        category_elem = item.find("category")
+        cell_elem = item.find("cell")
+        comment_elem = item.find("comment")
+        values_elem = item.find("values")
+
+        if category_elem is None or category_elem.text is None:
+            continue
+
+        category = category_elem.text
+        cell = cell_elem.text if cell_elem is not None and cell_elem.text else "unknown"
+        description = (
+            comment_elem.text
+            if comment_elem is not None and comment_elem.text
+            else categories_map.get(category, category)
+        )
+
+        location = _parse_polygon_location(values_elem)
+
+        violation = {
+            "category": category,
+            "cell": cell,
+            "description": description,
+        }
+
+        if location is not None:
+            violation["location"] = location
+        elif values_elem is not None:
+            violation["location_warning"] = "Could not parse coordinates"
+
+        violations.append(violation)
+
+        violation_counts[category] = violation_counts.get(category, 0) + 1
+
+    total_violations = len(violations)
+    status = "PASSED" if total_violations == 0 else "FAILED"
+
+    summary = {
+        "total_violations": total_violations,
+        "total_categories": len(violation_counts),
+        "cells_checked": cells,
+        "status": status,
+    }
+
+    if total_violations == 0:
+        summary["message"] = "No DRC violations found"
+
+    violations_by_category = [
+        {
+            "category": cat,
+            "description": categories_map.get(cat, cat),
+            "count": count,
+        }
+        for cat, count in sorted(
+            violation_counts.items(),
+            key=lambda x: x[1],
+            reverse=True,
+        )
+    ]
+
+    recommendations = _generate_drc_recommendations(
+        total_violations,
+        violations_by_category,
+        cells,
+    )
+
+    return {
+        "summary": summary,
+        "violations_by_category": violations_by_category,
+        "violations": violations,
+        "recommendations": recommendations,
+    }
+
+
+def _parse_polygon_location(values_elem: ET.Element | None) -> dict[str, Any] | None:
+    """Parse polygon coordinates and compute location data.
 
     Args:
-        args: MCP tool arguments
+        values_elem: XML element containing polygon coordinates
 
     Returns:
-        Dict with 'json_data' key for request body
+        Dict with bbox, centroid, and size, or None if parsing fails
     """
-    json_data: dict[str, Any] = {"path": args["path"]}
+    if values_elem is None:
+        return None
+
+    value_elem = values_elem.find("value")
+    if value_elem is None or value_elem.text is None:
+        return None
+
+    value_text = value_elem.text
+    if "polygon:" not in value_text:
+        return None
+
+    try:
+        coords_str = value_text.replace("polygon:", "").strip().strip("()")
+        if not coords_str:
+            return None
+
+        pairs = coords_str.split(";")
+        coords = []
+        for pair in pairs:
+            if "," not in pair:
+                continue
+            x_str, y_str = pair.split(",", 1)
+            coords.append((float(x_str), float(y_str)))
+
+        if not coords:
+            return None
+
+        xs = [x for x, y in coords]
+        ys = [y for x, y in coords]
+
+        bbox = {
+            "min_x": round(min(xs), 3),
+            "min_y": round(min(ys), 3),
+            "max_x": round(max(xs), 3),
+            "max_y": round(max(ys), 3),
+        }
 
-    # Add optional parameters if provided
-    if "pdk" in args and args["pdk"]:
-        json_data["pdk"] = args["pdk"]
-    if "process" in args and args["process"]:
-        json_data["process"] = args["process"]
-    if "timeout" in args and args["timeout"]:
-        json_data["timeout"] = args["timeout"]
-    if "host" in args and args["host"]:
-        json_data["host"] = args["host"]
+        centroid = {
+            "x": round(sum(xs) / len(xs), 3),
+            "y": round(sum(ys) / len(ys), 3),
+        }
+
+        size = {
+            "width": round(bbox["max_x"] - bbox["min_x"], 3),
+            "height": round(bbox["max_y"] - bbox["min_y"], 3),
+        }
+
+        return {
+            "bbox": bbox,
+            "centroid": centroid,
+            "size": size,
+        }
+
+    except (ValueError, IndexError):
+        return None
 
-    return {"json_data": json_data}
+
+def _generate_drc_recommendations(
+    total_violations: int,
+    violations_by_category: list[dict[str, Any]],
+    cells: list[str],
+) -> list[str]:
+    """Generate actionable recommendations based on DRC results.
+
+    Args:
+        total_violations: Total number of violations
+        violations_by_category: List of violation categories with counts
+        cells: List of cells checked
+
+    Returns:
+        List of recommendation strings
+    """
+    if total_violations == 0:
+        return ["Design passes all DRC checks"]
+
+    recommendations = []
+
+    if violations_by_category:
+        top_category = violations_by_category[0]
+        if top_category["count"] > 10:
+            recommendations.append(
+                f"Focus on {top_category['category']} violations "
+                f"({top_category['count']:,} occurrences)"
+            )
+        elif top_category["count"] > 1:
+            recommendations.append(
+                f"Check {top_category['category']} rules in {', '.join(cells)}"
+            )
+
+    if len(violations_by_category) > 1:
+        recommendations.append(
+            f"Review {len(violations_by_category)} different DRC rule categories"
+        )
+
+    if total_violations > 100:
+        recommendations.append(
+            "Consider fixing systematic issues first to reduce violation count"
+        )
+
+    return recommendations
 
 
 def _check_connectivity_request(args: dict[str, Any]) -> dict[str, Any]:
@@ -230,7 +427,6 @@ def _generate_bbox_request(args: dict[str, Any]) -> dict[str, Any]:
     """
     json_data: dict[str, Any] = {"path": args["path"]}
 
-    # Add optional parameters if provided
     if "outpath" in args and args["outpath"]:
         json_data["outpath"] = args["outpath"]
     if "layers_to_keep" in args and args["layers_to_keep"]:
@@ -255,23 +451,13 @@ def _freeze_cell_request(args: dict[str, Any]) -> dict[str, Any]:
     cell_name = args["cell_name"]
     kwargs = args.get("kwargs", {})
 
-    # The freeze endpoint expects a JSON string in the body.
-    # httpx with json= will JSON-encode the dict and send it as the body,
-    # which FastAPI's Body() will read as a string.
     return {
         "path": f"/freeze/{cell_name}",
         "json_data": kwargs,  # httpx will JSON-encode this to a string
     }
 
 
-# Tool name -> Endpoint mapping
 TOOL_MAPPINGS: dict[str, EndpointMapping] = {
-    # Phase 1: Core Building Tools
-    "build_cell": EndpointMapping(
-        method="GET",
-        path="/api/build-cell",
-        request_transformer=_build_cell_request,
-    ),
     "build_cells": EndpointMapping(
         method="POST",
         path="/api/build-cells",
@@ -287,17 +473,11 @@ TOOL_MAPPINGS: dict[str, EndpointMapping] = {
         path="/api/cell-info",
         request_transformer=_get_cell_info_request,
     ),
-    "download_gds": EndpointMapping(
-        method="GET",
-        path="/api/download/{path}.gds",
-        request_transformer=_download_gds_request,
-        response_transformer=_download_gds_response,
-    ),
-    # Phase 2: Verification Tools
     "check_drc": EndpointMapping(
         method="POST",
         path="/api/check-drc",
         request_transformer=_check_drc_request,
+        response_transformer=_check_drc_response,
     ),
     "check_connectivity": EndpointMapping(
         method="POST",
@@ -309,7 +489,6 @@ TOOL_MAPPINGS: dict[str, EndpointMapping] = {
         path="/api/check-lvs",
         request_transformer=_check_lvs_request,
     ),
-    # Phase 4: Simulation & Advanced Tools
     "simulate_component": EndpointMapping(
         method="GET",
         path="/api/simulate",

mcp_standalone/registry.py

@@ -102,8 +102,6 @@ class ServerInfo:
             If psutil is not available, always returns True
         """
         if not HAS_PSUTIL:
-            # Without psutil, assume the process is alive
-            # The registry cleanup is handled by gdsfactoryplus
             return True
 
         try:
@@ -163,7 +161,6 @@ class ServerRegistry:
 
         server_info = ServerInfo.from_dict(data["servers"][port_key])
 
-        # Check if process is still alive (if psutil is available)
         if HAS_PSUTIL and not server_info.is_alive():
             return None
 
@@ -203,7 +200,6 @@ class ServerRegistry:
         for server_data in data["servers"].values():
             server_info = ServerInfo.from_dict(server_data)
 
-            # Include all servers if psutil is not available or include_dead is True
             if not HAS_PSUTIL or include_dead or server_info.is_alive():
                 servers.append(server_info)
 

mcp_standalone/resources.py

@@ -15,7 +15,6 @@ __all__ = [
 ]
 
 
-# Define available resources
 RESOURCES: list[Resource] = [
     Resource(
         uri="instructions://build_custom_cell",
@@ -30,7 +29,6 @@ RESOURCES: list[Resource] = [
 ]
 
 
-# Resource content
 RESOURCE_CONTENT = {
     "instructions://build_custom_cell": """---
 Workflow: Creating Custom Components in GDSFactory+ Projects
@@ -71,7 +69,6 @@ def custom_component_name(
     Returns:
         Component description
     \"\"\"
-    # Implementation using PDK functions
     c = pdk_function(parameters...)
     return c
 

mcp_standalone/server.py

@@ -40,10 +40,8 @@ def create_server(api_url: str | None = None) -> Server:
     Returns:
         Configured MCP Server instance
     """
-    # Create server instance
     server = Server("gdsfactoryplus")
 
-    # Create HTTP client for FastAPI backend
     client = FastAPIClient(api_url)
 
     @server.list_tools()
@@ -83,7 +81,6 @@ def create_server(api_url: str | None = None) -> Server:
         """
         logger.info("Resource requested: %s", uri)
 
-        # Get resource content
         content = get_resource_content(uri)
         if content is None:
             error_msg = f"Unknown resource URI: {uri}"
@@ -106,7 +103,6 @@ def create_server(api_url: str | None = None) -> Server:
         logger.info("Tool called: %s", name)
         logger.debug("Arguments: %s", arguments)
 
-        # Handle special tools that don't require HTTP requests
         if name == "list_projects":
             try:
                 projects = client.list_projects()
@@ -138,7 +134,6 @@ def create_server(api_url: str | None = None) -> Server:
                 logger.exception(error_msg)
                 return [TextContent(type="text", text=json.dumps({"error": error_msg}))]
 
-        # Get the endpoint mapping
         mapping = get_mapping(name)
         if mapping is None:
             error_msg = f"Unknown tool: {name}"
@@ -146,21 +141,17 @@ def create_server(api_url: str | None = None) -> Server:
             return [TextContent(type="text", text=json.dumps({"error": error_msg}))]
 
         try:
-            # Extract project parameter (optional)
             project = arguments.get("project")
 
-            # Transform MCP arguments to HTTP request parameters
             transformed = transform_request(name, arguments)
             logger.debug("Transformed request: %s", transformed)
 
-            # Extract request parameters
             method = mapping.method
             path = transformed.get("path", mapping.path)
             params = transformed.get("params")
             json_data = transformed.get("json_data")
             data = transformed.get("data")
 
-            # Make HTTP request to FastAPI backend (with optional project routing)
             response = await client.request(
                 method=method,
                 path=path,
@@ -170,11 +161,9 @@ def create_server(api_url: str | None = None) -> Server:
                 project=project,
             )
 
-            # Transform response to MCP format
             result = transform_response(name, response)
             logger.debug("Tool result: %s", result)
 
-            # Return as text content
             return [
                 TextContent(
                     type="text",
@@ -192,7 +181,6 @@ def create_server(api_url: str | None = None) -> Server:
                 )
             ]
 
-    # Store client reference for cleanup
     server._http_client = client  # type: ignore[attr-defined]  # noqa: SLF001
 
     return server
@@ -215,10 +203,15 @@ def _log_server_discovery(client: FastAPIClient) -> None:
                 logger.info("  ... and %d more", len(projects) - 3)
         else:
             logger.warning(
-                "No GDSFactory+ servers found. Start a server with: "
-                "gfp serve --port 8787"
+                "No GDSFactory+ servers found. To start a server:\n"
+                "  1. Open VSCode with the GDSFactory+ extension installed\n"
+                "  2. Open a GDSFactory+ project folder\n"
+                "  3. The extension will automatically start and register the server"
             )
             logger.info("Registry location: %s", get_registry_path())
+            logger.info(
+                "Alternatively, set GFP_API_URL environment variable to connect to a specific server"
+            )
     except Exception as e:
         logger.warning("Could not read server registry: %s", e)
         logger.info(
@@ -233,7 +226,6 @@ async def run_server(api_url: str | None = None) -> None:
     Args:
         api_url: Optional FastAPI base URL (default from config)
     """
-    # Configure logging
     log_level = logging.DEBUG if MCPConfig.DEBUG else logging.INFO
     logging.basicConfig(
         level=log_level,
@@ -241,21 +233,18 @@ async def run_server(api_url: str | None = None) -> None:
     )
 
     logger.info("Starting GDSFactory+ MCP server")
-    logger.info("FastAPI base URL: %s", MCPConfig.get_api_url(api_url))
+    base_url_info = MCPConfig.get_api_url(api_url) or "registry-based routing"
+    logger.info("Base URL: %s", base_url_info)
     logger.info("Timeout: %ds", MCPConfig.get_timeout())
 
-    # Create server
     server = create_server(api_url)
     client: FastAPIClient = server._http_client  # type: ignore[attr-defined]  # noqa: SLF001
 
     try:
-        # Start HTTP client
         await client.start()
 
-        # Discover available servers
         _log_server_discovery(client)
 
-        # Run server with STDIO transport
         async with stdio_server() as (read_stream, write_stream):
             logger.info("MCP server ready (STDIO transport)")
             await server.run(
@@ -270,7 +259,6 @@ async def run_server(api_url: str | None = None) -> None:
         logger.exception("MCP server error")
         raise
     finally:
-        # Cleanup
         await client.close()
         logger.info("MCP server stopped")
 

mcp_standalone/tools.py

@@ -15,7 +15,6 @@ __all__ = [
 ]
 
 
-# Project discovery tools (for multi-project support)
 PROJECT_TOOLS: list[Tool] = [
     Tool(
         name="list_projects",
@@ -52,14 +51,13 @@ PROJECT_TOOLS: list[Tool] = [
     ),
 ]
 
-# Standard optional project parameter for all tools
 PROJECT_PARAM_SCHEMA = {
     "project": {
         "type": "string",
         "description": (
             "Optional project name or path to route this request to a specific "
-            "server. If not provided, routes to the default server. "
-            "Use list_projects to see available projects."
+            "server. If not provided, uses the first available server from the registry "
+            "or GFP_API_URL if set. Use list_projects to see available projects."
         ),
     },
 }
@@ -80,48 +78,13 @@ def _add_project_param(schema: dict) -> dict:
     return schema
 
 
-# Phase 1: Core Building Tools (5 tools)
 CORE_TOOLS: list[Tool] = [
-    Tool(
-        name="build_cell",
-        description=(
-            "Build a single GDS cell by name. This creates the physical layout "
-            "file (.gds) for a photonic component. The build happens in the "
-            "background and the GDS file will be saved to the project build "
-            "directory."
-        ),
-        inputSchema=_add_project_param(
-            {
-                "type": "object",
-                "properties": {
-                    "name": {
-                        "type": "string",
-                        "description": "Name of the cell/component to build",
-                    },
-                    "with_metadata": {
-                        "type": "boolean",
-                        "description": (
-                            "Include metadata in the GDS file (default: true)"
-                        ),
-                        "default": True,
-                    },
-                    "register": {
-                        "type": "boolean",
-                        "description": (
-                            "Re-register the cell in the KLayout cache (default: true)"
-                        ),
-                        "default": True,
-                    },
-                },
-                "required": ["name"],
-            }
-        ),
-    ),
     Tool(
         name="build_cells",
         description=(
-            "Build multiple GDS cells by name in one operation. This is more "
-            "efficient than building cells one at a time. All cells are built "
+            "Build one or more GDS cells by name. This creates the physical layout "
+            "files (.gds) for photonic components. Pass a list of cell names to build. "
+            "For a single cell, pass a list with one element. All cells are built "
             "in the background and saved to the project build directory."
         ),
         inputSchema=_add_project_param(
@@ -131,7 +94,7 @@ CORE_TOOLS: list[Tool] = [
                     "names": {
                         "type": "array",
                         "items": {"type": "string"},
-                        "description": "List of cell/component names to build",
+                        "description": "List of cell/component names to build (can be a single-item list)",
                     },
                     "with_metadata": {
                         "type": "boolean",
@@ -186,32 +149,8 @@ CORE_TOOLS: list[Tool] = [
             }
         ),
     ),
-    Tool(
-        name="download_gds",
-        description=(
-            "Download a GDS file from the project build directory. Returns the "
-            "file path to the downloaded GDS file. The file must have been "
-            "previously built using build_cell or build_cells."
-        ),
-        inputSchema=_add_project_param(
-            {
-                "type": "object",
-                "properties": {
-                    "path": {
-                        "type": "string",
-                        "description": (
-                            "Relative path to the GDS file (without .gds extension). "
-                            "For example, 'mzi' or 'components/coupler'"
-                        ),
-                    },
-                },
-                "required": ["path"],
-            }
-        ),
-    ),
 ]
 
-# Phase 2: Verification Tools
 VERIFICATION_TOOLS: list[Tool] = [
     Tool(
         name="check_drc",
@@ -328,10 +267,8 @@ VERIFICATION_TOOLS: list[Tool] = [
     ),
 ]
 
-# Phase 3: SPICE Workflow Tools (to be implemented)
 SPICE_TOOLS: list[Tool] = []
 
-# Phase 4: Simulation & Advanced Tools
 ADVANCED_TOOLS: list[Tool] = [
     Tool(
         name="simulate_component",
@@ -496,7 +433,6 @@ ADVANCED_TOOLS: list[Tool] = [
     ),
 ]
 
-# All tools (Phase 1 + Project tools)
 TOOLS: list[Tool] = [
     *PROJECT_TOOLS,  # Project discovery tools (always available)
     *CORE_TOOLS,

gfp_mcp-0.2.1.dist-info/RECORD

@@ -1,14 +0,0 @@
-gfp_mcp-0.2.1.dist-info/licenses/LICENSE,sha256=ixSuHdKKXzNJw_eTgAxHzaCNIds8k48hytA_eJgA8gQ,225
-mcp_standalone/__init__.py,sha256=B6JB7U_vEMARO87RzNDesfSZdGgD1U-RC2GkL2ijOCA,1069
-mcp_standalone/client.py,sha256=vS_mw3frp5dQr2s_uFbPH-cF4k98rOJFGZLIz1FOU7A,8371
-mcp_standalone/config.py,sha256=1B00PLrKOz96c62lkhIgrraF-HWZ015uoPZ7okCcuKk,1525
-mcp_standalone/mappings.py,sha256=2J2DEZzXTdmi6VmrtCeI1cg-2T7tYHkytRGrZJ62zog,10583
-mcp_standalone/registry.py,sha256=1E61UalVot8HUS3cALjM7ejYB0qR6tI5QbQSZZeQe7Y,6401
-mcp_standalone/resources.py,sha256=iMkYIyTxLWwWE0NLxprLabGFYbPnUaIbgwwtbvZ2av0,3606
-mcp_standalone/server.py,sha256=jDCns5Cgb5JZahbx8pjCCDqryT-RiLCOFD5FIpgr3OA,9149
-mcp_standalone/tools.py,sha256=totbebwVzqOej1TjmY6lOZ7raSPFIwGWmWfOFVN3IyE,18734
-gfp_mcp-0.2.1.dist-info/METADATA,sha256=16kFH1_7TvI11tZnuXXrdTyQ6os5s7I5E7mTvSdRfVA,6858
-gfp_mcp-0.2.1.dist-info/WHEEL,sha256=wUyA8OaulRlbfwMtmQsvNngGrxQHAvkKcvRmdizlJi0,92
-gfp_mcp-0.2.1.dist-info/entry_points.txt,sha256=mgyus9dsB_8mjgnztuHNPqzPi-7HcPg1iYzfM5NMIjk,61
-gfp_mcp-0.2.1.dist-info/top_level.txt,sha256=g2hRJHoDDPNtrNdXR70T7FR9Ev6DTRJiGW7ZvlvnXMc,15
-gfp_mcp-0.2.1.dist-info/RECORD,,