From 2e078498b19fff6a1080f9792f7846f7627e7531 Mon Sep 17 00:00:00 2001
From: Neovasky <neovasky@gmail.com>
Date: Sun, 27 Jul 2025 00:46:57 -0400
Subject: [PATCH] refactor(calendar): optimize logging for production readiness

- Change routine operation logs from info to debug level
- Simplify success messages for better readability
- Remove redundant calendar/path information from log messages
- Align logging style with repository standards

Following patterns established by repository maintainer in WebDAV client cleanup.
---
 nextcloud_mcp_server/client/calendar.py | 26 +++++++++----------------
 1 file changed, 9 insertions(+), 17 deletions(-)

diff --git a/nextcloud_mcp_server/client/calendar.py b/nextcloud_mcp_server/client/calendar.py
index eefe2cc..7df6da3 100644
--- a/nextcloud_mcp_server/client/calendar.py
+++ b/nextcloud_mcp_server/client/calendar.py
@@ -120,7 +120,7 @@ class CalendarClient(BaseNextcloudClient):
                     }
                 )
 
-            logger.info(f"Found {len(calendars)} calendars for user {self.username}")
+            logger.debug(f"Found {len(calendars)} calendars")
             return calendars
 
         except HTTPStatusError as e:
@@ -210,7 +210,7 @@ class CalendarClient(BaseNextcloudClient):
                 if len(events) >= limit:
                     break
 
-            logger.info(f"Found {len(events)} events in calendar {calendar_name}")
+            logger.debug(f"Found {len(events)} events")
             return events
 
         except HTTPStatusError as e:
@@ -242,9 +242,7 @@ class CalendarClient(BaseNextcloudClient):
             )
             response.raise_for_status()
 
-            logger.info(
-                f"Successfully created event {event_uid} in calendar {calendar_name}"
-            )
+            logger.debug(f"Created event {event_uid}")
             return {
                 "uid": event_uid,
                 "href": event_path,
@@ -300,9 +298,7 @@ class CalendarClient(BaseNextcloudClient):
             )
             response.raise_for_status()
 
-            logger.info(
-                f"Successfully updated event {event_uid} in calendar {calendar_name}"
-            )
+            logger.debug(f"Updated event {event_uid}")
             return {
                 "uid": event_uid,
                 "href": event_path,
@@ -326,14 +322,12 @@ class CalendarClient(BaseNextcloudClient):
             response = await self._client.delete(event_path)
             response.raise_for_status()
 
-            logger.info(
-                f"Successfully deleted event {event_uid} from calendar {calendar_name}"
-            )
+            logger.debug(f"Deleted event {event_uid}")
             return {"status_code": response.status_code}
 
         except HTTPStatusError as e:
             if e.response.status_code == 404:
-                logger.info(f"Event {event_uid} not found in calendar {calendar_name}")
+                logger.debug(f"Event {event_uid} not found")
                 return {"status_code": 404}
             logger.error(f"HTTP error deleting event: {e}")
             raise e
@@ -363,9 +357,7 @@ class CalendarClient(BaseNextcloudClient):
             event_data["href"] = event_path
             event_data["etag"] = etag
 
-            logger.info(
-                f"Successfully retrieved event {event_uid} from calendar {calendar_name}"
-            )
+            logger.debug(f"Retrieved event {event_uid}")
             return event_data, etag
 
         except HTTPStatusError as e:
@@ -948,7 +940,7 @@ class CalendarClient(BaseNextcloudClient):
             )
             response.raise_for_status()
 
-            logger.info(f"Successfully created calendar: {calendar_name}")
+            logger.debug(f"Created calendar: {calendar_name}")
             return {
                 "name": calendar_name,
                 "display_name": display_name or calendar_name,
@@ -969,7 +961,7 @@ class CalendarClient(BaseNextcloudClient):
             response = await self._client.delete(calendar_path)
             response.raise_for_status()
 
-            logger.info(f"Successfully deleted calendar: {calendar_name}")
+            logger.debug(f"Deleted calendar: {calendar_name}")
             return {"status_code": response.status_code}
 
         except Exception as e: