marcus/mcp-familywall
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: correct wallmood parameter name to wall_message_id (v0.4.3)

Browse Source 
API error message confirmed the expected parameter is 'wall_message_id',
not 'wallId'. SPEC.md updated to reflect verified finding.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
Marcus van Elst
2026-04-15 15:37:53 +02:00
parent c412c24c86
commit 9534658472
4 changed files with 15 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files
README.md
+1 -1
View File
@@ -2,7 +2,7 @@
MCP server for [Family Wall](https://www.familywall.com) -- read and manage your family's circles, lists, and tasks directly from Claude. MCP server for [Family Wall](https://www.familywall.com) -- read and manage your family's circles, lists, and tasks directly from Claude.
## Features (v0.4.2) ## Features (v0.4.3)
### Read ### Read
SPEC.md
+9 -5
View File
@@ -270,15 +270,18 @@ kritisch.
POST https://api.familywall.com/api/wallmood POST https://api.familywall.com/api/wallmood
Content-Type: application/x-www-form-urlencoded Content-Type: application/x-www-form-urlencoded
**Body-Parameter (zu verifizieren):** **Body-Parameter:**
| Parameter | Pflicht | Wert | | Parameter | Pflicht | Wert |
|---|---|---| |---|---|---|
| `wallId` | ja | Post-ID aus `get_activities` (Format `wall/23431854_31119189`) | | `wall_message_id` | ja (verifiziert) | Post-ID aus `get_activities` (Format `wall/23431854_31119189`) |
| `moodType` | vermutlich ja | `"LIKE"` zum Liken, `"NONE"` zum Unliken (zu verifizieren) | | `moodType` | vermutlich ja | `"LIKE"` zum Liken, `"NONE"` zum Unliken (zu verifizieren) |
**Verhalten:** Gleicher Endpoint für Like und Unlike. Mechanismus (Toggle vs. explizit) **Hinweis:** Parameter-Name `wall_message_id` wurde durch API-Fehlermeldung
und genaue Parameter-Namen sind noch zu verifizieren via `FW_DEBUG=1`. (`"missing value in: wall_message_id"`) verifiziert. Vorheriger Wert `wallId` war falsch.
**Verhalten:** Gleicher Endpoint für Like und Unlike. `moodType`-Werte und
Response-Struktur noch zu verifizieren via `FW_DEBUG=1`.
Ein Like erzeugt serverseitig einen neuen Wall-Eintrag (Nebeneffekt). Ein Like erzeugt serverseitig einen neuen Wall-Eintrag (Nebeneffekt).
@@ -298,4 +301,5 @@ Ein Like erzeugt serverseitig einen neuen Wall-Eintrag (Nebeneffekt).
- ~~`taskupdate2`: ob `taskListId` Pflichtfeld ist~~ → **nein**, nicht erforderlich (verifiziert) - ~~`taskupdate2`: ob `taskListId` Pflichtfeld ist~~ → **nein**, nicht erforderlich (verifiziert)
- ~~`taskmark`: korrekter Parameter-Name~~ → **`taskId`** (nicht `metaId`!) (verifiziert) - ~~`taskmark`: korrekter Parameter-Name~~ → **`taskId`** (nicht `metaId`!) (verifiziert)
- ~~`metadelete`: korrekter Parameter-Name + Response-Struktur~~ → **`id`**, Response `"true"` (verifiziert) - ~~`metadelete`: korrekter Parameter-Name + Response-Struktur~~ → **`id`**, Response `"true"` (verifiziert)
- `wallmood`: Parameter-Namen (`wallId`?, `moodType`?), Toggle vs. explizit, Response-Struktur - ~~`wallmood`: Parameter-Name `wallId`~~ → **`wall_message_id`** (verifiziert via API-Fehlermeldung)
- `wallmood`: `moodType`-Werte (`"LIKE"` / `"NONE"`?), Toggle vs. explizit, Response-Struktur
pyproject.toml
+1 -1
View File
@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
[project] [project]
name = "mcp-familywall" name = "mcp-familywall"
version = "0.4.2" version = "0.4.3"
description = "MCP server for Family Wall — read your family's lists and tasks via Claude" description = "MCP server for Family Wall — read your family's lists and tasks via Claude"
readme = "README.md" readme = "README.md"
requires-python = ">=3.12" requires-python = ">=3.12"
src/mcp_familywall/server.py
+4 -2
View File
@@ -490,9 +490,11 @@ def like_post(post_id: str, like: bool = True) -> str:
# wallmood is a toggle endpoint — same endpoint for like and unlike. # wallmood is a toggle endpoint — same endpoint for like and unlike.
# The 'like' parameter controls the intended state; since the endpoint # The 'like' parameter controls the intended state; since the endpoint
# toggles server-side state, it is sent as 'moodType' to allow the server # toggles server-side state, it is sent as 'moodType' to allow the server
# to differentiate. Exact parameter names to be verified via FW_DEBUG=1. # to differentiate.
# Verified: correct parameter name is 'wall_message_id' (API error message confirmed).
# 'moodType' values ("LIKE" / "NONE") still to be confirmed via FW_DEBUG=1.
params: dict[str, Any] = { params: dict[str, Any] = {
"wallId": post_id, "wall_message_id": post_id,
"moodType": "LIKE" if like else "NONE", "moodType": "LIKE" if like else "NONE",
} }