Drop indent=2 from JSON responses to reduce token usage (#34)

Author: bigcat88

src/nc_mcp_server/tools/activity.py

@@ -124,4 +124,4 @@ async def get_activity(
                 "since": next_since,
             },
         }
-        return json.dumps(response, indent=2, default=str)
+        return json.dumps(response, default=str)

src/nc_mcp_server/tools/announcements.py

@@ -71,7 +71,7 @@ async def list_announcements(offset: int = 0) -> str:
                 "has_more": len(announcements) == page_limit,
             },
         }
-        return json.dumps(result, indent=2, default=str)
+        return json.dumps(result, default=str)
 
 
 def _register_write_tools(mcp: FastMCP) -> None:
@@ -125,7 +125,7 @@ async def create_announcement(
             "comments": 1 if comments else 0,
         }
         data = await client.ocs_post(ANNOUNCEMENTS_API, data=post_data)
-        return json.dumps(_format_announcement(data), indent=2, default=str)
+        return json.dumps(_format_announcement(data), default=str)
 
 
 def _register_destructive_tools(mcp: FastMCP) -> None:

src/nc_mcp_server/tools/app_management.py

@@ -45,7 +45,7 @@ async def list_apps(app_filter: str = "enabled") -> str:
         client = get_client()
         params = {} if app_filter == "all" else {"filter": app_filter}
         data = await client.ocs_get(APPS_API, params=params)
-        return json.dumps(data["apps"], indent=2)
+        return json.dumps(data["apps"])
 
     @mcp.tool(annotations=READONLY)
     @require_permission(PermissionLevel.READ)
@@ -63,7 +63,7 @@ async def get_app_info(app_id: str) -> str:
         """
         client = get_client()
         data = await client.ocs_get(f"{APPS_API}/{app_id}")
-        return json.dumps(_format_app(data), indent=2, default=str)
+        return json.dumps(_format_app(data), default=str)
 
 
 def _register_write_tools(mcp: FastMCP) -> None:

src/nc_mcp_server/tools/calendar.py

@@ -358,7 +358,7 @@ async def list_calendars() -> str:
             context="List calendars",
         )
         calendars = _parse_calendars_xml(response.text or "", user)
-        return json.dumps(calendars, indent=2)
+        return json.dumps(calendars)
 
     @mcp.tool(annotations=READONLY)
     @require_permission(PermissionLevel.READ)
@@ -410,7 +410,7 @@ async def get_events(
             event = _format_event(ical_data)
             event["etag"] = etag
             events.append(event)
-        return json.dumps(events, indent=2)
+        return json.dumps(events)
 
     @mcp.tool(annotations=READONLY)
     @require_permission(PermissionLevel.READ)
@@ -428,7 +428,7 @@ async def get_event(calendar_id: str, event_uid: str) -> str:
         _href, etag, ical_data = await _find_event(calendar_id, event_uid)
         event = _format_event(ical_data)
         event["etag"] = etag
-        return json.dumps(event, indent=2)
+        return json.dumps(event)
 
 
 def _register_create_event(mcp: FastMCP) -> None:
@@ -490,7 +490,7 @@ async def create_event(
             headers={"Content-Type": "text/calendar; charset=utf-8"},
             context=f"Create event in '{calendar_id}'",
         )
-        return json.dumps({"uid": uid, "summary": summary}, indent=2)
+        return json.dumps({"uid": uid, "summary": summary})
 
 
 def _register_update_event(mcp: FastMCP) -> None:

src/nc_mcp_server/tools/collectives.py

@@ -54,7 +54,7 @@ async def list_collectives() -> str:
         client = get_client()
         data = await client.ocs_get(f"{API}/collectives")
         collectives = [_format_collective(c) for c in data["collectives"]]
-        return json.dumps(collectives, indent=2, default=str)
+        return json.dumps(collectives, default=str)
 
     @mcp.tool(annotations=READONLY)
     @require_permission(PermissionLevel.READ)
@@ -73,7 +73,7 @@ async def get_collective_pages(collective_id: int) -> str:
         client = get_client()
         data = await client.ocs_get(f"{API}/collectives/{collective_id}/pages")
         pages = [_format_page(p) for p in data["pages"]]
-        return json.dumps(pages, indent=2, default=str)
+        return json.dumps(pages, default=str)
 
     @mcp.tool(annotations=READONLY)
     @require_permission(PermissionLevel.READ)
@@ -92,7 +92,7 @@ async def get_collective_page(collective_id: int, page_id: int) -> str:
         client = get_client()
         data = await client.ocs_get(f"{API}/collectives/{collective_id}/pages/{page_id}")
         page = data["page"]
-        return json.dumps(_format_page(page), indent=2, default=str)
+        return json.dumps(_format_page(page), default=str)
 
 
 def _register_write_tools(mcp: FastMCP) -> None:
@@ -119,7 +119,7 @@ async def create_collective(name: str, emoji: str | None = None) -> str:
             post_data["emoji"] = emoji
         data = await client.ocs_post_json(f"{API}/collectives", json_data=post_data)
         collective = data["collective"]
-        return json.dumps(_format_collective(collective), indent=2, default=str)
+        return json.dumps(_format_collective(collective), default=str)
 
     @mcp.tool(annotations=ADDITIVE)
     @require_permission(PermissionLevel.WRITE)
@@ -147,7 +147,7 @@ async def create_collective_page(collective_id: int, parent_id: int, title: str)
             json_data={"title": title},
         )
         page = data["page"]
-        return json.dumps(_format_page(page), indent=2, default=str)
+        return json.dumps(_format_page(page), default=str)
 
 
 def _register_destructive_tools(mcp: FastMCP) -> None:
@@ -184,7 +184,7 @@ async def restore_collective(collective_id: int) -> str:
         client = get_client()
         data = await client.ocs_patch(f"{API}/collectives/trash/{collective_id}")
         collective = data["collective"]
-        return json.dumps(_format_collective(collective), indent=2, default=str)
+        return json.dumps(_format_collective(collective), default=str)
 
     @mcp.tool(annotations=DESTRUCTIVE)
     @require_permission(PermissionLevel.DESTRUCTIVE)
@@ -239,7 +239,7 @@ async def restore_collective_page(collective_id: int, page_id: int) -> str:
         client = get_client()
         data = await client.ocs_patch(f"{API}/collectives/{collective_id}/pages/trash/{page_id}")
         page = data["page"]
-        return json.dumps(_format_page(page), indent=2, default=str)
+        return json.dumps(_format_page(page), default=str)
 
     @mcp.tool(annotations=DESTRUCTIVE)
     @require_permission(PermissionLevel.DESTRUCTIVE)

src/nc_mcp_server/tools/comments.py

@@ -134,7 +134,7 @@ async def list_comments(file_id: int, limit: int = 20, offset: int = 0) -> str:
                 "has_more": len(comments) == limit,
             },
         }
-        return json.dumps(response_data, indent=2, default=str)
+        return json.dumps(response_data, default=str)
 
 
 def _register_write_tools(mcp: FastMCP) -> None:
@@ -168,7 +168,7 @@ async def add_comment(file_id: int, message: str) -> str:
         )
         location = str(response.headers.get("Content-Location", ""))
         comment_id = location.rstrip("/").split("/")[-1] if location else "unknown"
-        return json.dumps({"id": comment_id, "message": message}, indent=2)
+        return json.dumps({"id": comment_id, "message": message})
 
     @mcp.tool(annotations=ADDITIVE_IDEMPOTENT)
     @require_permission(PermissionLevel.WRITE)
@@ -204,7 +204,7 @@ async def edit_comment(file_id: int, comment_id: int, message: str) -> str:
             headers={"Content-Type": "application/xml; charset=utf-8"},
             context=f"Edit comment {comment_id} on file {file_id}",
         )
-        return json.dumps({"id": comment_id, "message": message}, indent=2)
+        return json.dumps({"id": comment_id, "message": message})
 
 
 def _register_destructive_tools(mcp: FastMCP) -> None:

src/nc_mcp_server/tools/files.py

@@ -76,7 +76,7 @@ async def list_directory(path: str = "/") -> str:
         # First entry is the directory itself — skip it
         if entries and entries[0]["path"].rstrip("/") == path.strip("/"):
             entries = entries[1:]
-        return json.dumps(entries, indent=2, default=str)
+        return json.dumps(entries, default=str)
 
     @mcp.tool(annotations=READONLY)
     @require_permission(PermissionLevel.READ)
@@ -158,7 +158,7 @@ async def search_files(
                 "has_more": len(results) == limit,
             },
         }
-        return json.dumps(response_data, indent=2, default=str)
+        return json.dumps(response_data, default=str)
 
 
 def _register_write_tools(mcp: FastMCP) -> None:

src/nc_mcp_server/tools/mail.py

@@ -100,7 +100,7 @@ async def list_mail_accounts() -> str:
         client = get_client()
         data = await client.ocs_get(f"{MAIL_OCS}/account/list")
         accounts = [_format_account(a) for a in data]
-        return json.dumps(accounts, indent=2)
+        return json.dumps(accounts)
 
     @mcp.tool(annotations=READONLY)
     @require_permission(PermissionLevel.READ)
@@ -119,7 +119,7 @@ async def list_mailboxes(account_id: int) -> str:
         client = get_client()
         data = await client.ocs_get(f"{MAIL_OCS}/ocs/mailboxes", params={"accountId": str(account_id)})
         mailboxes = [_format_mailbox(mb) for mb in data]
-        return json.dumps(mailboxes, indent=2)
+        return json.dumps(mailboxes)
 
     @mcp.tool(annotations=READONLY)
     @require_permission(PermissionLevel.READ)
@@ -153,7 +153,7 @@ async def list_mail_messages(mailbox_id: int, limit: int = 20, cursor: int | Non
         if messages:
             pagination["next_cursor"] = min(m["id"] for m in messages)
         result: dict[str, Any] = {"data": messages, "pagination": pagination}
-        return json.dumps(result, indent=2)
+        return json.dumps(result)
 
     @mcp.tool(annotations=READONLY)
     @require_permission(PermissionLevel.READ)
@@ -171,7 +171,7 @@ async def get_mail_message(message_id: int) -> str:
         """
         client = get_client()
         data = await client.ocs_get(f"{MAIL_OCS}/message/{message_id}")
-        return json.dumps(_format_message_full(data), indent=2)
+        return json.dumps(_format_message_full(data))
 
 
 def _register_write_tools(mcp: FastMCP) -> None:

src/nc_mcp_server/tools/notifications.py

@@ -28,7 +28,7 @@ async def list_notifications() -> str:
         data = await client.ocs_get(
             "apps/notifications/api/v2/notifications",
         )
-        return json.dumps(data, indent=2, default=str)
+        return json.dumps(data, default=str)
 
     @mcp.tool(annotations=DESTRUCTIVE)
     @require_permission(PermissionLevel.DESTRUCTIVE)

src/nc_mcp_server/tools/shares.py

@@ -70,7 +70,7 @@ async def list_shares(
             params["subfiles"] = "true"
         data = await client.ocs_get(SHARES_API, params=params)
         shares = [_format_share(s) for s in data]
-        return json.dumps(shares, indent=2, default=str)
+        return json.dumps(shares, default=str)
 
     @mcp.tool(annotations=READONLY)
     @require_permission(PermissionLevel.READ)
@@ -87,7 +87,7 @@ async def get_share(share_id: int) -> str:
         client = get_client()
         data = await client.ocs_get(f"{SHARES_API}/{share_id}")
         share = _format_share(data[0])
-        return json.dumps(share, indent=2, default=str)
+        return json.dumps(share, default=str)
 
 
 _SUPPORTED_SHARE_TYPES = {0, 1, 3, 4, 6, 10}
@@ -163,7 +163,7 @@ async def create_share(
         if public_upload:
             data["publicUpload"] = "true"
         result = await client.ocs_post(SHARES_API, data=data)
-        return json.dumps(_format_share(result), indent=2, default=str)
+        return json.dumps(_format_share(result), default=str)
 
 
 def _register_update_share(mcp: FastMCP) -> None:
@@ -213,7 +213,7 @@ async def update_share(
         if hide_download is not None:
             data["hideDownload"] = "true" if hide_download else "false"
         result = await client.ocs_put(f"{SHARES_API}/{share_id}", data=data)
-        return json.dumps(_format_share(result), indent=2, default=str)
+        return json.dumps(_format_share(result), default=str)
 
 
 def _register_destructive_tools(mcp: FastMCP) -> None: