mcp-gee-sweet 0.9.2.dev241__tar.gz → 0.9.2.dev243__tar.gz

{mcp_gee_sweet-0.9.2.dev241 → mcp_gee_sweet-0.9.2.dev243}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: mcp-gee-sweet
-Version: 0.9.2.dev241
+Version: 0.9.2.dev243
 Summary: MCP server for Google Workspace — Sheets, Drive, Docs (HTML or Markdown), and Calendar.
 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.9.2.dev241 → mcp_gee_sweet-0.9.2.dev243}/docs/roadmap.md

@@ -273,7 +273,7 @@ Same shape as v0.8.1 — stabilize on defects that surfaced since v0.9.0 shipped
 **Calendar hardening** _(triaged out of `backlog` 2026-09-11)_
 
 **Testing** _(triaged out of `backlog` 2026-09-11)_
-- [ ] **Bundle:** test-suite tightening, one PR closing [#392](https://github.com/khuisman/mcp-gee-sweet/issues/392), [#526](https://github.com/khuisman/mcp-gee-sweet/issues/526), [#551](https://github.com/khuisman/mcp-gee-sweet/issues/551), [#561](https://github.com/khuisman/mcp-gee-sweet/issues/561), [#574](https://github.com/khuisman/mcp-gee-sweet/issues/574) — lane B ([#891](https://github.com/khuisman/mcp-gee-sweet/issues/891))
+- [x] **Bundle:** test-suite tightening, one PR closing [#392](https://github.com/khuisman/mcp-gee-sweet/issues/392), [#526](https://github.com/khuisman/mcp-gee-sweet/issues/526), [#551](https://github.com/khuisman/mcp-gee-sweet/issues/551), [#561](https://github.com/khuisman/mcp-gee-sweet/issues/561), [#574](https://github.com/khuisman/mcp-gee-sweet/issues/574) — lane B (PR #924) ([#891](https://github.com/khuisman/mcp-gee-sweet/issues/891))
 - [ ] `qa`: `search_files` has no unit test coverage for the error-dict path ([#573](https://github.com/khuisman/mcp-gee-sweet/issues/573))
 
 **Infrastructure**

{mcp_gee_sweet-0.9.2.dev241 → mcp_gee_sweet-0.9.2.dev243}/src/mcp_gee_sweet/tools/sheets/helpers.py

@@ -65,9 +65,9 @@ def _parse_a1_notation(range_str: str) -> dict[str, int]:
     if not range_str:
         raise ValueError("Invalid A1 notation: empty string")
 
-    match = re.match(r"^([A-Z]+)?(\d+)?(?::([A-Z]+)?(\d+)?)?$", range_str.upper())
+    match = re.fullmatch(r"([A-Z]+)?(\d+)?(?::([A-Z]+)?(\d+)?)?", range_str.upper())
 
-    if not match:
+    if not match or not any(match.groups()):
         raise ValueError(f"Invalid A1 notation: {range_str}")
 
     start_col, start_row, end_col, end_row = match.groups()

{mcp_gee_sweet-0.9.2.dev241 → mcp_gee_sweet-0.9.2.dev243}/tests/sheets/test_helpers.py

@@ -48,6 +48,11 @@ class TestLetterToColumnIndex:
 
 
 class TestParseA1Notation:
+    @pytest.mark.parametrize("range_str", [":", "A1\n", "1:2\n", "B:D\n"])
+    def test_incomplete_or_trailing_newline_range_raises(self, range_str):
+        with pytest.raises(ValueError, match="Invalid A1 notation"):
+            _parse_a1_notation(range_str)
+
     def test_single_cell(self):
         result = _parse_a1_notation("A1")
         assert result["startColumnIndex"] == 0

{mcp_gee_sweet-0.9.2.dev241 → mcp_gee_sweet-0.9.2.dev243}/tests/sheets/test_structure.py

@@ -3,6 +3,8 @@
 from itertools import groupby
 from unittest.mock import AsyncMock, MagicMock
 
+import pytest
+
 from mcp_gee_sweet.cache import SheetInfo
 from mcp_gee_sweet.tools.sheets import structure as sheets_structure_module
 
@@ -1260,6 +1262,19 @@ class TestResizeColumns:
 
 
 class TestFormatCells:
+    @pytest.mark.parametrize("range_str", [":", "A1\n"])
+    async def test_malformed_range_does_not_format_any_cells(self, range_str):
+        svc = self._sheets_service()
+        result = await _structure_tools["format_cells"](
+            spreadsheet_id="ss1",
+            sheet="Sheet1",
+            range=range_str,
+            bold=True,
+            ctx=_make_ctx(sheets_service=svc, cache=None),
+        )
+        assert "error" in result
+        svc.spreadsheets.return_value.batchUpdate.assert_not_called()
+
     def _sheets_service(self, sheet_id=0):
         mock = MagicMock()
         mock.spreadsheets.return_value.get.return_value.execute.return_value = {