nocfo-cli 1.4.0__tar.gz → 1.4.2__tar.gz

{nocfo_cli-1.4.0 → nocfo_cli-1.4.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: nocfo-cli
-Version: 1.4.0
+Version: 1.4.2
 Summary: NoCFO CLI, MCP server, and Cursor skill toolkit.
 License: MIT
 License-File: LICENSE

{nocfo_cli-1.4.0 → nocfo_cli-1.4.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "poetry.core.masonry.api"
 
 [tool.poetry]
 name = "nocfo-cli"
-version = "1.4.0"
+version = "1.4.2"
 description = "NoCFO CLI, MCP server, and Cursor skill toolkit."
 authors = ["NoCFO"]
 readme = "README.md"

{nocfo_cli-1.4.0 → nocfo_cli-1.4.2}/src/nocfo_toolkit/mcp/contract_validation.py

@@ -13,6 +13,7 @@ from nocfo_toolkit.mcp.server import (
     MCP_OPENAPI_ROUTE_MAPS,
     apply_mcp_operation_metadata,
     apply_mcp_namespace_names,
+    restore_openapi_output_schema,
 )
 from nocfo_toolkit.openapi import filter_mcp_spec
 
@@ -35,6 +36,7 @@ class MCPContractValidationResult:
 def _apply_component_mcp_metadata(route: Any, component: Any) -> None:
     apply_mcp_namespace_names(route, component)
     apply_mcp_operation_metadata(route, component)
+    restore_openapi_output_schema(route, component)
 
 
 def _get_base_url(openapi_spec: dict[str, Any]) -> str:
@@ -146,7 +148,7 @@ def _build_result(
 def validate_openapi_mcp_contract(
     openapi_spec: dict[str, Any],
     *,
-    validate_output: bool = True,
+    validate_output: bool = False,
 ) -> MCPContractValidationResult:
     """Validate that MCP-tagged operations map cleanly to FastMCP components.
 
@@ -186,7 +188,7 @@ def validate_openapi_mcp_contract(
 def assert_openapi_mcp_contract_valid(
     openapi_spec: dict[str, Any],
     *,
-    validate_output: bool = True,
+    validate_output: bool = False,
 ) -> MCPContractValidationResult:
     """Assert MCP contract validity and return the validation result."""
     result = validate_openapi_mcp_contract(

{nocfo_cli-1.4.0 → nocfo_cli-1.4.2}/src/nocfo_toolkit/mcp/server.py

@@ -32,6 +32,8 @@ from nocfo_toolkit.openapi import (
     load_openapi_spec,
 )
 
+from fastmcp.utilities.openapi import extract_output_schema_from_responses
+
 if TYPE_CHECKING:
     from fastmcp import FastMCP
 
@@ -64,26 +66,16 @@ def _normalize_mcp_namespace_token(value: str) -> str:
 def build_mcp_component_name(
     operation_id: str, extensions: dict[str, Any] | None
 ) -> str:
-    """Build MCP name ``<x-mcp-namespace>_<operation_id_with_dots_as_underscores>``.
+    """Build MCP component name from ``operation_id``.
 
-    Namespace is read from the OpenAPI operation extension ``x-mcp-namespace`` (set in
-    the backend via ``mcp_extend_schema``). If missing, ``operation_id`` is
-    returned unchanged.
+    Namespace extension values are preserved in metadata but are not used for
+    display-name generation to keep backend ``operationId`` as the single source
+    of truth.
     """
+    del extensions
     if not operation_id or not str(operation_id).strip():
         return operation_id
-    raw = (extensions or {}).get(X_MCP_NAMESPACE)
-    if not isinstance(raw, str) or not raw.strip():
-        return operation_id
-    ns = _normalize_mcp_namespace_token(raw)
-    if not ns:
-        return operation_id
-    # Avoid duplicating the namespace if operation_id already starts with it.
-    # Example: operation_id "invoicing.contact.create" with ns "invoicing"
-    # should become "invoicing_contact_create", not "invoicing_invoicing_contact_create".
-    if operation_id.startswith(f"{ns}."):
-        operation_id = operation_id[len(ns) + 1 :]
-    return f"{ns}_{operation_id.replace('.', '_')}"
+    return operation_id.replace(".", "_")
 
 
 def _resource_uri_with_name(uri: str | AnyUrl, display_name: str) -> AnyUrl:
@@ -133,6 +125,29 @@ def apply_mcp_operation_metadata(route: Any, component: Any) -> None:
     component.meta = meta
 
 
+def restore_openapi_output_schema(route: Any, component: Any) -> None:
+    """Restore the real OpenAPI output schema on tools for agent-facing metadata.
+
+    When ``validate_output=False`` FastMCP replaces the output schema with a
+    permissive fallback so that runtime responses are never rejected.  This
+    function re-derives the accurate schema from the route and writes it back
+    onto the tool so that ``tools/list`` still exposes precise type information
+    to the agent — without enforcing it at call time.
+    """
+    if not isinstance(component, Tool):
+        return
+    responses = getattr(route, "responses", None)
+    if responses is None:
+        return
+    real_schema = extract_output_schema_from_responses(
+        responses,
+        getattr(route, "response_schemas", None),
+        getattr(route, "openapi_version", None),
+    )
+    if real_schema is not None:
+        component.output_schema = real_schema
+
+
 def _get_server_instructions(openapi_spec: dict[str, Any]) -> str | None:
     """Read backend-owned MCP server instructions from OpenAPI root extensions."""
     instructions = openapi_spec.get(X_NOCFO_MCP_SERVER_INSTRUCTIONS)
@@ -229,6 +244,7 @@ def create_server(
     def component_mapper(route: Any, component: Any) -> None:
         apply_mcp_namespace_names(route, component)
         apply_mcp_operation_metadata(route, component)
+        restore_openapi_output_schema(route, component)
         if opts.auth_mode == "oauth" and isinstance(
             component, (Tool, OpenAPIResource, OpenAPIResourceTemplate)
         ):
@@ -246,7 +262,7 @@ def create_server(
         middleware=[MCPToolErrorMiddleware()],
         route_maps=MCP_OPENAPI_ROUTE_MAPS,
         mcp_component_fn=component_mapper,
-        validate_output=True,
+        validate_output=False,
     )