From 58ea21804410e8dc0d8188761dea7e98aa980c4b Mon Sep 17 00:00:00 2001 From: Viktor Barzin Date: Sat, 14 Mar 2026 11:14:05 +0000 Subject: [PATCH] fix: add preview to delete response for MCP server compatibility --- src/claude_memory/api/app.py | 4 ++-- tests/test_api.py | 6 ++++-- 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/src/claude_memory/api/app.py b/src/claude_memory/api/app.py index 20c0152..f0c91d6 100644 --- a/src/claude_memory/api/app.py +++ b/src/claude_memory/api/app.py @@ -200,7 +200,7 @@ async def delete_memory(memory_id: int, user: AuthUser = Depends(get_current_use async with pool.acquire() as conn: row = await conn.fetchrow( - "SELECT id, vault_path FROM memories WHERE id = $1 AND user_id = $2", + "SELECT id, vault_path, substr(content, 1, 50) AS preview FROM memories WHERE id = $1 AND user_id = $2", memory_id, user.user_id, ) @@ -216,7 +216,7 @@ async def delete_memory(memory_id: int, user: AuthUser = Depends(get_current_use user.user_id, ) - return {"deleted": memory_id} + return {"deleted": memory_id, "preview": row["preview"]} @app.post("/api/memories/{memory_id}/secret", response_model=SecretResponse) diff --git a/tests/test_api.py b/tests/test_api.py index 9b80572..e5059d0 100644 --- a/tests/test_api.py +++ b/tests/test_api.py @@ -194,7 +194,7 @@ async def test_list_returns_only_user_memories(client): @pytest.mark.asyncio async def test_delete_only_user_memories(client): ac, conn, app_mod = client - conn.fetchrow.return_value = _make_memory_row(id=10, vault_path=None) + conn.fetchrow.return_value = _make_memory_row(id=10, vault_path=None, preview="test content") conn.execute.return_value = None async with ac: @@ -204,7 +204,9 @@ async def test_delete_only_user_memories(client): ) assert resp.status_code == 200 - assert resp.json() == {"deleted": 10} + data = resp.json() + assert data["deleted"] == 10 + assert "preview" in data # Verify both SELECT and DELETE include user_id fetchrow_args = conn.fetchrow.call_args