Reduce response size by omitting bulky fields (such as logo)

Author: r-brown

src/netlicensing_mcp/tools/__init__.py

@@ -2,6 +2,7 @@
 
 from netlicensing_mcp.tools import (
     bundles,
+    helpers,
     license_templates,
     licensees,
     licenses,
@@ -15,6 +16,7 @@
 
 __all__ = [
     "bundles",
+    "helpers",
     "license_templates",
     "licensees",
     "licenses",

src/netlicensing_mcp/tools/bundles.py

@@ -3,16 +3,17 @@
 from __future__ import annotations
 
 from netlicensing_mcp.client import nl_delete, nl_get, nl_post
+from netlicensing_mcp.tools.helpers import strip_output_fields
 
 
 async def list_bundles() -> dict:
     """List all bundles in the account."""
-    return await nl_get("/bundle")
+    return strip_output_fields(await nl_get("/bundle"))
 
 
 async def get_bundle(bundle_number: str) -> dict:
     """Get a single bundle by its number."""
-    return await nl_get(f"/bundle/{bundle_number}")
+    return strip_output_fields(await nl_get(f"/bundle/{bundle_number}"))
 
 
 async def create_bundle(
@@ -30,6 +31,9 @@ async def create_bundle(
     license_template_numbers: list of license template numbers included in the bundle.
     price: bundle price (optional).
     currency: ISO 4217 currency code (e.g. EUR, USD).
+
+    Note: 'logo' is intentionally not passed — omitting it avoids accidentally
+    clearing an existing bundle logo on the NetLicensing server.
     """
     data: dict[str, str] = {
         "number": number,
@@ -44,7 +48,7 @@ async def create_bundle(
         data["price"] = str(price)
     if currency:
         data["currency"] = currency
-    return await nl_post("/bundle", data)
+    return strip_output_fields(await nl_post("/bundle", data))
 
 
 async def update_bundle(
@@ -56,7 +60,11 @@ async def update_bundle(
     currency: str | None = None,
     description: str | None = None,
 ) -> dict:
-    """Update fields of an existing bundle."""
+    """Update fields of an existing bundle.
+
+    Note: 'logo' is intentionally not passed — omitting it preserves the
+    existing bundle logo on the NetLicensing server.
+    """
     data: dict[str, str] = {}
     if name is not None:
         data["name"] = name
@@ -70,7 +78,7 @@ async def update_bundle(
         data["currency"] = currency
     if description is not None:
         data["description"] = description
-    return await nl_post(f"/bundle/{bundle_number}", data)
+    return strip_output_fields(await nl_post(f"/bundle/{bundle_number}", data))
 
 
 async def delete_bundle(bundle_number: str, force_cascade: bool = False) -> str:

src/netlicensing_mcp/tools/helpers.py

@@ -0,0 +1,37 @@
+"""Shared helpers for MCP tool output post-processing."""
+
+from __future__ import annotations
+
+from typing import Any
+
+# Fields that carry large binary/base64 blobs and should be stripped from all
+# tool outputs to keep MCP responses concise.  They are intentionally NOT sent
+# in create/update requests either — omitting them preserves the existing value
+# on the NetLicensing server side.
+STRIP_OUTPUT_FIELDS: frozenset[str] = frozenset({"logo"})
+
+
+def strip_output_fields(data: Any, fields: frozenset[str] = STRIP_OUTPUT_FIELDS) -> Any:
+    """Recursively remove large/binary fields from a NetLicensing API response.
+
+    Handles two common patterns:
+    - Plain dict keys  (e.g. ``{"logo": "data:image/png;base64,...", ...}``)
+    - NetLicensing *property* arrays  (``[{"name": "logo", "value": "..."}, ...]``)
+
+    Any field whose name appears in *fields* is dropped at every nesting level.
+    """
+    if isinstance(data, list):
+        return [strip_output_fields(item, fields) for item in data]
+    if isinstance(data, dict):
+        # 1. Drop matching top-level keys and recurse into kept values.
+        result = {k: strip_output_fields(v, fields) for k, v in data.items() if k not in fields}
+        # 2. If this dict contains a NetLicensing "property" list, also filter
+        #    entries whose "name" field matches.
+        if "property" in result and isinstance(result["property"], list):
+            result["property"] = [
+                p
+                for p in result["property"]
+                if not (isinstance(p, dict) and p.get("name") in fields)
+            ]
+        return result
+    return data

src/netlicensing_mcp/tools/products.py

@@ -3,6 +3,7 @@
 from __future__ import annotations
 
 from netlicensing_mcp.client import nl_delete, nl_get, nl_post
+from netlicensing_mcp.tools.helpers import strip_output_fields
 
 
 async def list_products(filter_str: str | None = None) -> dict:
@@ -13,12 +14,12 @@ async def list_products(filter_str: str | None = None) -> dict:
     params: dict[str, str] = {}
     if filter_str:
         params["filter"] = filter_str
-    return await nl_get("/product", params or None)
+    return strip_output_fields(await nl_get("/product", params or None))
 
 
 async def get_product(product_number: str) -> dict:
     """Get a single product by its number."""
-    return await nl_get(f"/product/{product_number}")
+    return strip_output_fields(await nl_get(f"/product/{product_number}"))
 
 
 async def create_product(
@@ -38,6 +39,9 @@ async def create_product(
     vat_mode: GROSS | NET
     licensee_auto_create: if true, non-existing licensees are created on first validation.
     licensee_secret_mode: DISABLED | PREDEFINED | CLIENT
+
+    Note: 'logo' is intentionally not passed — omitting it avoids accidentally
+    clearing an existing product logo on the NetLicensing server.
     """
     data: dict[str, str] = {
         "number": number,
@@ -55,7 +59,7 @@ async def create_product(
         data["vatMode"] = vat_mode
     if licensee_secret_mode:
         data["licenseeSecretMode"] = licensee_secret_mode
-    return await nl_post("/product", data)
+    return strip_output_fields(await nl_post("/product", data))
 
 
 async def update_product(
@@ -72,6 +76,9 @@ async def update_product(
     """Update fields of an existing product.
 
     licensee_secret_mode: DISABLED | PREDEFINED | CLIENT
+
+    Note: 'logo' is intentionally not passed — omitting it preserves the
+    existing product logo on the NetLicensing server.
     """
     data: dict[str, str] = {}
     if name is not None:
@@ -90,7 +97,7 @@ async def update_product(
         data["vatMode"] = vat_mode
     if licensee_secret_mode is not None:
         data["licenseeSecretMode"] = licensee_secret_mode
-    return await nl_post(f"/product/{product_number}", data)
+    return strip_output_fields(await nl_post(f"/product/{product_number}", data))
 
 
 async def delete_product(product_number: str, force_cascade: bool = False) -> str: