Bill/grist-mcp-server
1
0
Fork 0
Code Issues Pull Requests 6 Actions Packages Projects Releases Wiki Activity

fix: use correct Grist API endpoint for modify_column

Browse Source 
The Grist API uses PATCH /tables/{table}/columns with a columns array
in the body, not PATCH /tables/{table}/columns/{column_id}. Updated
the endpoint to match the API spec.
This commit is contained in:
Bill Ballou
2026-01-01 10:10:49 -05:00
parent 9e96d18315
commit 49c5043661
2 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
src/grist_mcp/grist_client.py
View File

@@ -160,7 +160,8 @@ class GristClient:
if formula is not None: if formula is not None:
fields["formula"] = formula fields["formula"] = formula
await self._request("PATCH", f"/tables/{table}/columns/{column_id}", json={"fields": fields}) payload = {"columns": [{"id": column_id, "fields": fields}]}
await self._request("PATCH", f"/tables/{table}/columns", json=payload)
async def delete_column(self, table: str, column_id: str) -> None: async def delete_column(self, table: str, column_id: str) -> None:
"""Delete a column from a table.""" """Delete a column from a table."""

2
tests/unit/test_grist_client.py
View File

@@ -160,7 +160,7 @@ async def test_add_column(client, httpx_mock: HTTPXMock):
@pytest.mark.asyncio @pytest.mark.asyncio
async def test_modify_column(client, httpx_mock: HTTPXMock): async def test_modify_column(client, httpx_mock: HTTPXMock):
httpx_mock.add_response( httpx_mock.add_response(
url="https://grist.example.com/api/docs/abc123/tables/Table1/columns/Amount", url="https://grist.example.com/api/docs/abc123/tables/Table1/columns",
method="PATCH", method="PATCH",
json={}, json={},
) )