Merge pull request #416 from ugoano/feature/enhanced-sheets-formatting

feat(sheets): add text wrapping, alignment, and font formatting to format_sheet_range

Author: taylorwilsdon

README.md

@@ -936,6 +936,7 @@ attachments=[{
 | `create_spreadsheet` | **Core** | Create new spreadsheets |
 | `list_spreadsheets` | Extended | List accessible spreadsheets |
 | `get_spreadsheet_info` | Extended | Get spreadsheet metadata |
+| `format_sheet_range` | Extended | Apply colors, number formats, text wrapping, alignment, bold/italic, font size |
 | `create_sheet` | Complete | Add sheets to existing files |
 | `*_sheet_comment` | Complete | Read/create/reply/resolve comments |
 

README_NEW.md

@@ -121,8 +121,8 @@ export OAUTHLIB_INSECURE_TRANSPORT=1  # Development only
 | `create_spreadsheet` | Core | Create new spreadsheets with multiple sheets |
 | `list_spreadsheets` | Extended | List accessible spreadsheets |
 | `get_spreadsheet_info` | Extended | Get metadata, sheets, conditional formats |
+| `format_sheet_range` | Extended | Apply colors, number formats, text wrapping, alignment, bold/italic, font size |
 | `create_sheet` | Complete | Add sheets to existing spreadsheets |
-| `format_sheet_range` | Complete | Apply colors and number formats |
 | `add_conditional_formatting` | Complete | Add boolean or gradient rules |
 | `update_conditional_formatting` | Complete | Modify existing rules |
 | `delete_conditional_formatting` | Complete | Remove formatting rules |

core/tool_tiers.yaml

@@ -85,6 +85,7 @@ sheets:
   extended:
     - list_spreadsheets
     - get_spreadsheet_info
+    - format_sheet_range
   complete:
     - create_sheet
     - read_spreadsheet_comments

gcalendar/calendar_tools.py

@@ -376,9 +376,9 @@ async def get_events(
     if event_id:
         logger.info(f"[get_events] Retrieving single event with ID: {event_id}")
         event = await asyncio.to_thread(
-            lambda: service.events()
-            .get(calendarId=calendar_id, eventId=event_id)
-            .execute()
+            lambda: (
+                service.events().get(calendarId=calendar_id, eventId=event_id).execute()
+            )
         )
         items = [event]
     else:
@@ -701,13 +701,15 @@ async def create_event(
                     if drive_service:
                         try:
                             file_metadata = await asyncio.to_thread(
-                                lambda: drive_service.files()
-                                .get(
-                                    fileId=file_id,
-                                    fields="mimeType,name",
-                                    supportsAllDrives=True,
+                                lambda: (
+                                    drive_service.files()
+                                    .get(
+                                        fileId=file_id,
+                                        fields="mimeType,name",
+                                        supportsAllDrives=True,
+                                    )
+                                    .execute()
                                 )
-                                .execute()
                             )
                             mime_type = file_metadata.get("mimeType", mime_type)
                             filename = file_metadata.get("name")
@@ -735,24 +737,28 @@ async def create_event(
             if drive_service:
                 drive_service.close()
         created_event = await asyncio.to_thread(
-            lambda: service.events()
-            .insert(
-                calendarId=calendar_id,
-                body=event_body,
-                supportsAttachments=True,
-                conferenceDataVersion=1 if add_google_meet else 0,
+            lambda: (
+                service.events()
+                .insert(
+                    calendarId=calendar_id,
+                    body=event_body,
+                    supportsAttachments=True,
+                    conferenceDataVersion=1 if add_google_meet else 0,
+                )
+                .execute()
             )
-            .execute()
         )
     else:
         created_event = await asyncio.to_thread(
-            lambda: service.events()
-            .insert(
-                calendarId=calendar_id,
-                body=event_body,
-                conferenceDataVersion=1 if add_google_meet else 0,
+            lambda: (
+                service.events()
+                .insert(
+                    calendarId=calendar_id,
+                    body=event_body,
+                    conferenceDataVersion=1 if add_google_meet else 0,
+                )
+                .execute()
             )
-            .execute()
         )
     link = created_event.get("htmlLink", "No link available")
     confirmation_message = f"Successfully created event '{created_event.get('summary', summary)}' for {user_google_email}. Link: {link}"
@@ -976,9 +982,9 @@ async def modify_event(
     # Get the existing event to preserve fields that aren't being updated
     try:
         existing_event = await asyncio.to_thread(
-            lambda: service.events()
-            .get(calendarId=calendar_id, eventId=event_id)
-            .execute()
+            lambda: (
+                service.events().get(calendarId=calendar_id, eventId=event_id).execute()
+            )
         )
         logger.info(
             "[modify_event] Successfully retrieved existing event before update"
@@ -1035,14 +1041,16 @@ async def modify_event(
 
     # Proceed with the update
     updated_event = await asyncio.to_thread(
-        lambda: service.events()
-        .update(
-            calendarId=calendar_id,
-            eventId=event_id,
-            body=event_body,
-            conferenceDataVersion=1,
+        lambda: (
+            service.events()
+            .update(
+                calendarId=calendar_id,
+                eventId=event_id,
+                body=event_body,
+                conferenceDataVersion=1,
+            )
+            .execute()
         )
-        .execute()
     )
 
     link = updated_event.get("htmlLink", "No link available")
@@ -1096,9 +1104,9 @@ async def delete_event(
     # Try to get the event first to verify it exists
     try:
         await asyncio.to_thread(
-            lambda: service.events()
-            .get(calendarId=calendar_id, eventId=event_id)
-            .execute()
+            lambda: (
+                service.events().get(calendarId=calendar_id, eventId=event_id).execute()
+            )
         )
         logger.info("[delete_event] Successfully verified event exists before deletion")
     except HttpError as get_error:
@@ -1115,9 +1123,9 @@ async def delete_event(
 
     # Proceed with the deletion
     await asyncio.to_thread(
-        lambda: service.events()
-        .delete(calendarId=calendar_id, eventId=event_id)
-        .execute()
+        lambda: (
+            service.events().delete(calendarId=calendar_id, eventId=event_id).execute()
+        )
     )
 
     confirmation_message = f"Successfully deleted event (ID: {event_id}) from calendar '{calendar_id}' for {user_google_email}."