mcp-gee-sweet 0.7.0.dev67__tar.gz → 0.7.0.dev68__tar.gz

{mcp_gee_sweet-0.7.0.dev67 → mcp_gee_sweet-0.7.0.dev68}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: mcp-gee-sweet
-Version: 0.7.0.dev67
+Version: 0.7.0.dev68
 Summary: MCP server for Google Workspace — Sheets, Drive, Docs, and Calendar. 60+ tools for AI clients.
 Project-URL: Homepage, https://github.com/khuisman/mcp-gee-sweet
 Project-URL: Bug Tracker, https://github.com/khuisman/mcp-gee-sweet/issues

{mcp_gee_sweet-0.7.0.dev67 → mcp_gee_sweet-0.7.0.dev68}/docs/configuration.md

@@ -69,7 +69,7 @@ Access logs are emitted at `INFO` level — they appear when `DEBUG_LEVEL` is `D
 
 <!-- Auto-generated by `scripts/gen_tool_docs.py` — do not edit by hand, your changes will be overwritten on the next commit. -->
 
-By default all 84 tools are registered. Use `ENABLED_TOOLS` (or `--include-tools` on the CLI) to restrict the server to exactly the tools you need. This reduces the AI's context window cost — each registered tool is a name the model must reason about on every call.
+By default all 87 tools are registered. Use `ENABLED_TOOLS` (or `--include-tools` on the CLI) to restrict the server to exactly the tools you need. This reduces the AI's context window cost — each registered tool is a name the model must reason about on every call.
 
 ```bash
 # Environment variable
@@ -100,7 +100,7 @@ create_doc,get_doc_content,get_doc_structure,write_doc_content,insert_doc_text,i
 
 **Calendar only:**
 ```
-list_calendars,get_calendar,list_events,get_event,create_event,update_event,delete_event,find_free_slots
+list_calendars,get_calendar,create_calendar,update_calendar,delete_calendar,list_events,get_event,create_event,update_event,delete_event,find_free_slots
 ```
 
 See [Tools](tools.md) for the full list of tool names.

{mcp_gee_sweet-0.7.0.dev67 → mcp_gee_sweet-0.7.0.dev68}/docs/qa/tests/calendar.md

@@ -105,6 +105,144 @@ Fixtures: see [`docs/qa/setup.md`](../setup.md). Substitute your `{CALENDAR_ID}`
 
 ---
 
+## `create_calendar`
+
+### TC-CAL44: Create a secondary calendar ⚠️ destructive
+
+**Prompt**
+> "Create a new calendar called 'mcp-gee-sweet-qa-lifecycle-test' with description 'QA scratch calendar' and timezone 'America/Los_Angeles'"
+
+**Checks**
+- Returns `id`, `summary`, `description`, `time_zone`
+- `summary` is `mcp-gee-sweet-qa-lifecycle-test`
+- `time_zone` is `America/Los_Angeles`
+- The new calendar appears in a follow-up `list_calendars` call (cache invalidated)
+
+**Result (2026-07-05) ✅** — Created via OAuth; response had `summary`, `description`, and `time_zone: "America/Los_Angeles"` exactly as requested. Confirmed present in a follow-up `list_calendars` call.
+
+---
+
+### TC-CAL45: Create with only summary
+
+**Prompt**
+> "Create a new calendar called 'mcp-gee-sweet-qa-minimal'"
+
+**Checks**
+- Succeeds with only `summary` provided
+- `description` is `null`/absent
+- `time_zone` reflects the account default
+
+**Result (2026-07-05) ✅** — Created with only `summary` provided. `description: null`. `time_zone: "UTC"` (account default rather than the creator's local timezone — noted for documentation purposes). Confirmed present in a follow-up `list_calendars` call.
+
+---
+
+## `update_calendar`
+
+### TC-CAL46: Rename and change timezone ⚠️ destructive
+
+**Setup:** use the calendar created in TC-CAL44
+
+**Prompt**
+> "Update calendar 'mcp-gee-sweet-qa-lifecycle-test' — rename it to 'mcp-gee-sweet-qa-renamed' and set its timezone to 'America/New_York'"
+
+**Checks**
+- `summary` is `mcp-gee-sweet-qa-renamed`
+- `time_zone` is `America/New_York`
+- `description` unchanged from TC-CAL44
+- `get_calendar` on the same ID reflects the new values
+
+**Result (2026-07-05) ✅** — Patched with `summary: "mcp-gee-sweet-qa-renamed"` and `timezone: "America/New_York"`. Response reflected both changes; `description` unchanged from TC-CAL44 (`"QA scratch calendar"`).
+
+---
+
+### TC-CAL47: Change color only ⚠️ destructive
+
+**Setup:** use the calendar from TC-CAL44/46
+
+**Prompt**
+> "Change the color of calendar 'mcp-gee-sweet-qa-renamed' to color ID '5'"
+
+**Checks**
+- `color_id` in the response is `5`
+- `summary`/`description`/`time_zone` unchanged (verify with `get_calendar`)
+- Confirms the color patch goes through `calendarList().patch()`, not `calendars().patch()`
+
+**Result (2026-07-05) ✅** — Patched `color_id="5"` only (no summary/description/timezone). Response had `color_id: "5"` with `summary`, `description`, `time_zone` all unchanged from TC-CAL46 — confirms the tool fell back to `calendars().get()` for the base fields and routed color through `calendarList().patch()`.
+
+---
+
+### TC-CAL48: No fields provided
+
+**Prompt**
+> "Update calendar {CALENDAR_ID} with no changes — just fetch its current state through update_calendar"
+
+**Checks**
+- Returns current calendar metadata unchanged
+- `color_id` is `null` (no color patch attempted)
+- No API error
+
+**Result (2026-07-05) ✅** — Called with `calendar_id="primary"` and no other fields. Returned the account's real primary calendar summary/timezone unchanged, `color_id: null`. Confirms the no-op path uses `calendars().get()` without issuing any patch.
+
+---
+
+### TC-CAL49: Non-existent calendar ID
+
+**Prompt**
+> "Update calendar 'totally-invalid-cal-id@example.com' — rename it to 'Nope'"
+
+**Checks**
+- Returns `{"error": "..."}` — not a top-level exception
+
+**Result (2026-07-05) ✅** — Returned `{"error": "<HttpError 404 ... 'reason': 'notFound' ...>"}`. No exception raised.
+
+---
+
+## `delete_calendar`
+
+### TC-CAL50: Delete a calendar ⚠️ destructive
+
+**Setup:** use the calendar created in TC-CAL44 (after TC-CAL46/47 updates)
+
+**Prompt**
+> "Delete the calendar 'mcp-gee-sweet-qa-renamed'"
+
+**Checks**
+- Returns `{"calendar_id": "...", "action": "deleted"}`
+- Calendar no longer appears in a follow-up `list_calendars` call
+- `get_calendar` on the deleted ID returns an error
+
+**Result (2026-07-05) ✅** — Returned `{"calendar_id": "...", "action": "deleted"}`. Confirmed absent from a follow-up `list_calendars` call. `get_calendar` on the deleted ID returned a 404 `notFound` error.
+
+---
+
+### TC-CAL51: Delete the minimal test calendar ⚠️ destructive
+
+**Setup:** use the calendar created in TC-CAL45
+
+**Prompt**
+> "Delete the calendar 'mcp-gee-sweet-qa-minimal'"
+
+**Checks**
+- Returns `{"calendar_id": "...", "action": "deleted"}`
+- Cleans up the fixture so it doesn't linger in the account's calendar list
+
+**Result (2026-07-05) ✅** — Returned `{"calendar_id": "...", "action": "deleted"}`. Confirmed removed from the account's `list_calendars` output alongside TC-CAL50's cleanup.
+
+---
+
+### TC-CAL52: Non-existent calendar ID
+
+**Prompt**
+> "Delete the calendar 'totally-invalid-cal-id@example.com'"
+
+**Checks**
+- Returns `{"error": "..."}` — not a top-level exception
+- No side effects
+
+**Result (2026-07-05) ✅** — Returned `{"error": "<HttpError 404 ... 'reason': 'notFound' ...>"}`. No side effects.
+
+---
+
 ## `list_events`
 
 ### TC-CAL09: No time filters — upcoming events

{mcp_gee_sweet-0.7.0.dev67 → mcp_gee_sweet-0.7.0.dev68}/docs/tools.md

@@ -2,7 +2,7 @@
 
 <!-- Auto-generated by `scripts/gen_tool_docs.py` — do not edit by hand, your changes will be overwritten on the next commit. -->
 
-84 tools. All tool names are lowercase with underscores — use these exact strings with `ENABLED_TOOLS` or `--include-tools`.
+87 tools. All tool names are lowercase with underscores — use these exact strings with `ENABLED_TOOLS` or `--include-tools`.
 
 ## Sheets — data
 
@@ -141,6 +141,9 @@ Requires the `drive.activity.readonly` scope; uses the Drive Activity API v2.
 |---|---|---|
 | `list_calendars` | List all calendars accessible to the authenticated user. | — |
 | `get_calendar` | Fetch metadata for a single calendar. | `calendar_id` |
+| `create_calendar` | Create a new secondary calendar owned by the authenticated user. | `summary`, `description?`, `timezone?` |
+| `update_calendar` | Update fields on an existing calendar using patch semantics. Only the fields you provide are changed; omitted fields are left as-is. | `calendar_id`, `summary?`, `description?`, `timezone?`, `color_id?` |
+| `delete_calendar` | Permanently delete a calendar. For calendars you own, this deletes the calendar entirely for all users it is shared with — use with caution. To only stop seeing a calendar in your own list, use remove_calendar_from_list instead. | `calendar_id` |
 | `list_events` | List events in a calendar. | `calendar_id`, `time_min?`, `time_max?`, `query?`, `max_results?`, `expand_recurring?` |
 | `get_event` | Fetch a single event by calendar ID and event ID. | `calendar_id`, `event_id` |
 | `create_event` | Create a new event in a calendar. | `calendar_id`, `summary`, `start`, `end`, `description?`, `location?`, `attendees?`, `timezone?`, `recurrence?` |

{mcp_gee_sweet-0.7.0.dev67 → mcp_gee_sweet-0.7.0.dev68}/scripts/gen_tool_docs.py

@@ -215,6 +215,9 @@ SUBSETS = [
         [
             "list_calendars",
             "get_calendar",
+            "create_calendar",
+            "update_calendar",
+            "delete_calendar",
             "list_events",
             "get_event",
             "create_event",

{mcp_gee_sweet-0.7.0.dev67 → mcp_gee_sweet-0.7.0.dev68}/src/mcp_gee_sweet/tools/calendar.py

@@ -73,6 +73,140 @@ def register(tool):
         cache.store(calendar_id, result)
         return result
 
+    @tool(annotations=ToolAnnotations(title="Create Calendar", destructiveHint=True))
+    def create_calendar(
+        summary: str,
+        description: str | None = None,
+        timezone: str | None = None,
+        ctx: Context = None,
+    ) -> dict[str, Any]:
+        """
+        Create a new secondary calendar owned by the authenticated user.
+
+        Args:
+            summary: Calendar title.
+            description: Optional calendar description.
+            timezone: IANA timezone name (e.g. 'America/Los_Angeles'). Defaults to
+                      the account's timezone if omitted.
+
+        Returns:
+            Created calendar: id, summary, description, time_zone.
+        """
+        lc = ctx.request_context.lifespan_context
+
+        body: dict[str, Any] = {"summary": summary}
+        if description:
+            body["description"] = description
+        if timezone:
+            body["timeZone"] = timezone
+
+        try:
+            c = lc.calendar_service.calendars().insert(body=body).execute()
+        except Exception as e:
+            return {"error": str(e)}
+
+        lc.calendar_cache.mark_dirty(c["id"])
+        logger.debug("Created calendar %s", c["id"])
+        return {
+            "id": c["id"],
+            "summary": c.get("summary", ""),
+            "description": c.get("description"),
+            "time_zone": c.get("timeZone"),
+        }
+
+    @tool(annotations=ToolAnnotations(title="Update Calendar", destructiveHint=True))
+    def update_calendar(
+        calendar_id: str,
+        summary: str | None = None,
+        description: str | None = None,
+        timezone: str | None = None,
+        color_id: str | None = None,
+        ctx: Context = None,
+    ) -> dict[str, Any]:
+        """
+        Update fields on an existing calendar using patch semantics. Only the
+        fields you provide are changed; omitted fields are left as-is.
+
+        Note: color_id is set via the authenticated user's calendar list entry
+        (calendarList().patch()), not the calendar resource itself — it is a
+        per-user setting, not a property of the calendar shared with others.
+
+        Args:
+            calendar_id: The calendar ID, or 'primary'.
+            summary: New calendar title.
+            description: New calendar description.
+            timezone: New IANA timezone name (e.g. 'America/Los_Angeles').
+            color_id: New color ID from the palette returned by the Calendar API's
+                      colors().get() endpoint (predefined IDs, typically '1'-'24').
+
+        Returns:
+            Updated calendar: id, summary, description, time_zone, color_id.
+        """
+        lc = ctx.request_context.lifespan_context
+
+        patch: dict[str, Any] = {}
+        if summary is not None:
+            patch["summary"] = summary
+        if description is not None:
+            patch["description"] = description
+        if timezone is not None:
+            patch["timeZone"] = timezone
+
+        try:
+            if patch:
+                c = (
+                    lc.calendar_service.calendars()
+                    .patch(calendarId=calendar_id, body=patch)
+                    .execute()
+                )
+            else:
+                c = lc.calendar_service.calendars().get(calendarId=calendar_id).execute()
+
+            result_color_id = None
+            if color_id is not None:
+                list_entry = (
+                    lc.calendar_service.calendarList()
+                    .patch(calendarId=calendar_id, body={"colorId": color_id})
+                    .execute()
+                )
+                result_color_id = list_entry.get("colorId")
+        except Exception as e:
+            return {"error": str(e)}
+
+        lc.calendar_cache.mark_dirty(calendar_id)
+        logger.debug("Updated calendar %s", calendar_id)
+        return {
+            "id": c.get("id", calendar_id),
+            "summary": c.get("summary", ""),
+            "description": c.get("description"),
+            "time_zone": c.get("timeZone"),
+            "color_id": result_color_id,
+        }
+
+    @tool(annotations=ToolAnnotations(title="Delete Calendar", destructiveHint=True))
+    def delete_calendar(calendar_id: str, ctx: Context = None) -> dict[str, Any]:
+        """
+        Permanently delete a calendar. For calendars you own, this deletes the
+        calendar entirely for all users it is shared with — use with caution.
+        To only stop seeing a calendar in your own list, use
+        remove_calendar_from_list instead.
+
+        Args:
+            calendar_id: The calendar ID to delete.
+
+        Returns:
+            Confirmation with calendar_id and action 'deleted'.
+        """
+        lc = ctx.request_context.lifespan_context
+        try:
+            lc.calendar_service.calendars().delete(calendarId=calendar_id).execute()
+        except Exception as e:
+            return {"error": str(e)}
+
+        lc.calendar_cache.mark_dirty(calendar_id)
+        logger.debug("Deleted calendar %s", calendar_id)
+        return {"calendar_id": calendar_id, "action": "deleted"}
+
     @tool(annotations=ToolAnnotations(title="List Events", readOnlyHint=True))
     def list_events(
         calendar_id: str,

{mcp_gee_sweet-0.7.0.dev67 → mcp_gee_sweet-0.7.0.dev68}/tests/test_calendar.py

@@ -73,6 +73,192 @@ class TestListCalendars:
         cache.store_list.assert_called_once_with(result)
 
 
+class TestCreateCalendar:
+    """create_calendar inserts a new secondary calendar and invalidates the list cache."""
+
+    def test_builds_body_from_provided_fields_and_maps_response(self):
+        """summary/description/timezone go in the insert body; response is field-mapped."""
+        cal_svc = MagicMock()
+        cal_svc.calendars.return_value.insert.return_value.execute.return_value = {
+            "id": "new-cal-1",
+            "summary": "Team Events",
+            "description": "Shared team calendar",
+            "timeZone": "America/Los_Angeles",
+        }
+        cache = MagicMock()
+        ctx = _make_ctx(calendar_service=cal_svc, calendar_cache=cache)
+
+        result = _cal_tools["create_calendar"](
+            summary="Team Events",
+            description="Shared team calendar",
+            timezone="America/Los_Angeles",
+            ctx=ctx,
+        )
+
+        body = cal_svc.calendars.return_value.insert.call_args.kwargs["body"]
+        assert body == {
+            "summary": "Team Events",
+            "description": "Shared team calendar",
+            "timeZone": "America/Los_Angeles",
+        }
+        assert result["id"] == "new-cal-1"
+        assert result["time_zone"] == "America/Los_Angeles"
+
+    def test_omits_optional_fields_from_body_when_not_provided(self):
+        """When description/timezone are omitted, the insert body must only have summary."""
+        cal_svc = MagicMock()
+        cal_svc.calendars.return_value.insert.return_value.execute.return_value = {
+            "id": "new-cal-2",
+            "summary": "Minimal",
+        }
+        ctx = _make_ctx(calendar_service=cal_svc, calendar_cache=MagicMock())
+
+        _cal_tools["create_calendar"](summary="Minimal", ctx=ctx)
+
+        body = cal_svc.calendars.return_value.insert.call_args.kwargs["body"]
+        assert body == {"summary": "Minimal"}
+
+    def test_marks_cache_dirty_with_new_calendar_id(self):
+        """calendar_cache.mark_dirty must be called with the newly created calendar's id."""
+        cal_svc = MagicMock()
+        cal_svc.calendars.return_value.insert.return_value.execute.return_value = {
+            "id": "new-cal-3",
+            "summary": "Test",
+        }
+        cache = MagicMock()
+        ctx = _make_ctx(calendar_service=cal_svc, calendar_cache=cache)
+
+        _cal_tools["create_calendar"](summary="Test", ctx=ctx)
+
+        cache.mark_dirty.assert_called_once_with("new-cal-3")
+
+    def test_api_error_returns_error_dict_without_marking_cache_dirty(self):
+        """When calendars().insert() raises, result must be {"error": ...} and cache stays clean."""
+        cal_svc = MagicMock()
+        cal_svc.calendars.return_value.insert.return_value.execute.side_effect = Exception(
+            "quotaExceeded"
+        )
+        cache = MagicMock()
+        ctx = _make_ctx(calendar_service=cal_svc, calendar_cache=cache)
+
+        result = _cal_tools["create_calendar"](summary="Test", ctx=ctx)
+
+        assert "error" in result
+        cache.mark_dirty.assert_not_called()
+
+
+class TestUpdateCalendar:
+    """update_calendar uses patch semantics for core fields and calendarList for color."""
+
+    def test_only_summary_in_body_when_only_summary_provided(self):
+        """If only summary is given, the patch body must not include description or timeZone."""
+        cal_svc = MagicMock()
+        cal_svc.calendars.return_value.patch.return_value.execute.return_value = {
+            "id": "cal-1",
+            "summary": "Renamed",
+        }
+        ctx = _make_ctx(calendar_service=cal_svc, calendar_cache=MagicMock())
+
+        _cal_tools["update_calendar"](calendar_id="cal-1", summary="Renamed", ctx=ctx)
+
+        body = cal_svc.calendars.return_value.patch.call_args.kwargs["body"]
+        assert body == {"summary": "Renamed"}
+
+    def test_color_id_patches_calendar_list_not_calendars_resource(self):
+        """color_id must go through calendarList().patch(), not calendars().patch()."""
+        cal_svc = MagicMock()
+        cal_svc.calendars.return_value.get.return_value.execute.return_value = {
+            "id": "cal-1",
+            "summary": "Existing",
+        }
+        cal_svc.calendarList.return_value.patch.return_value.execute.return_value = {"colorId": "5"}
+        ctx = _make_ctx(calendar_service=cal_svc, calendar_cache=MagicMock())
+
+        result = _cal_tools["update_calendar"](calendar_id="cal-1", color_id="5", ctx=ctx)
+
+        cal_svc.calendars.return_value.patch.assert_not_called()
+        cal_svc.calendarList.return_value.patch.assert_called_once_with(
+            calendarId="cal-1", body={"colorId": "5"}
+        )
+        assert result["color_id"] == "5"
+
+    def test_no_fields_provided_falls_back_to_get_without_patching(self):
+        """With no fields set, the tool must fetch via calendars().get(), not patch()."""
+        cal_svc = MagicMock()
+        cal_svc.calendars.return_value.get.return_value.execute.return_value = {
+            "id": "cal-1",
+            "summary": "Unchanged",
+        }
+        ctx = _make_ctx(calendar_service=cal_svc, calendar_cache=MagicMock())
+
+        result = _cal_tools["update_calendar"](calendar_id="cal-1", ctx=ctx)
+
+        cal_svc.calendars.return_value.patch.assert_not_called()
+        cal_svc.calendars.return_value.get.assert_called_once_with(calendarId="cal-1")
+        assert result["summary"] == "Unchanged"
+        assert result["color_id"] is None
+
+    def test_marks_cache_dirty_with_correct_calendar_id(self):
+        """After a successful update, calendar_cache.mark_dirty is called with the calendar_id."""
+        cal_svc = MagicMock()
+        cal_svc.calendars.return_value.patch.return_value.execute.return_value = {
+            "id": "cal-2",
+            "summary": "Renamed",
+        }
+        cache = MagicMock()
+        ctx = _make_ctx(calendar_service=cal_svc, calendar_cache=cache)
+
+        _cal_tools["update_calendar"](calendar_id="cal-2", summary="Renamed", ctx=ctx)
+
+        cache.mark_dirty.assert_called_once_with("cal-2")
+
+    def test_api_error_returns_error_dict_without_marking_cache_dirty(self):
+        """When calendars().patch() raises, result must be {"error": ...} and cache stays clean."""
+        cal_svc = MagicMock()
+        cal_svc.calendars.return_value.patch.return_value.execute.side_effect = Exception(
+            "notFound"
+        )
+        cache = MagicMock()
+        ctx = _make_ctx(calendar_service=cal_svc, calendar_cache=cache)
+
+        result = _cal_tools["update_calendar"](
+            calendar_id="cal-missing", summary="Renamed", ctx=ctx
+        )
+
+        assert "error" in result
+        cache.mark_dirty.assert_not_called()
+
+
+class TestDeleteCalendar:
+    """delete_calendar returns a structured confirmation and invalidates the cache."""
+
+    def test_success_returns_confirmation_dict_and_marks_cache_dirty(self):
+        """Result must be {calendar_id, action='deleted'} and cache is dirtied."""
+        cal_svc = MagicMock()
+        cal_svc.calendars.return_value.delete.return_value.execute.return_value = None
+        cache = MagicMock()
+        ctx = _make_ctx(calendar_service=cal_svc, calendar_cache=cache)
+
+        result = _cal_tools["delete_calendar"](calendar_id="cal-1", ctx=ctx)
+
+        assert result == {"calendar_id": "cal-1", "action": "deleted"}
+        cache.mark_dirty.assert_called_once_with("cal-1")
+
+    def test_api_error_returns_error_dict_without_marking_cache_dirty(self):
+        """If delete raises, result has 'error' key and the cache must not be dirtied."""
+        cal_svc = MagicMock()
+        cal_svc.calendars.return_value.delete.return_value.execute.side_effect = Exception(
+            "Not Found"
+        )
+        cache = MagicMock()
+        ctx = _make_ctx(calendar_service=cal_svc, calendar_cache=cache)
+
+        result = _cal_tools["delete_calendar"](calendar_id="cal-missing", ctx=ctx)
+
+        assert "error" in result
+        cache.mark_dirty.assert_not_called()
+
+
 class TestCreateEvent:
     """create_event detects all-day events, passes attendees, and invalidates the cache."""