From d2c6f1a967d55c22aab66502c511613aab2febdc Mon Sep 17 00:00:00 2001 From: damocles Date: Sun, 4 Oct 2026 20:16:23 +0200 Subject: [PATCH] hive-matrix-mcp: fix list_rooms/list_invites docs to say room_id, not id --- docs/tools/matrix.md | 4 ++-- hive-matrix-mcp/src/handlers.rs | 7 +++---- 2 files changed, 5 insertions(+), 6 deletions(-) diff --git a/docs/tools/matrix.md b/docs/tools/matrix.md index 21888b8c..43768a2c 100644 --- a/docs/tools/matrix.md +++ b/docs/tools/matrix.md @@ -48,7 +48,7 @@ room you haven't read yet. attachment from a message event to a local file and return its path; pair with `read_room` which surfaces attachments as the markers above - `list_rooms()` — enumerate joined rooms - (`{ id, canonical_alias, name, member_count }` per room) + (`{ room_id, canonical_alias, name, member_count }` per room) - `list_room_members(room)` — members of a room ### Room membership @@ -66,7 +66,7 @@ room you haven't read yet. a pending invite if one exists, but prefer `resolve_invite` for invites (it can reject too). - `list_invites()` — rooms this agent has been invited to but not - yet joined (`{ id, canonical_alias, name, inviter, topic }` per room — + yet joined (`{ room_id, canonical_alias, name, inviter, topic }` per room — `inviter` and `topic` are there so an agent has enough to judge an invite before `resolve_invite`ing it, not just a bare room id). diff --git a/hive-matrix-mcp/src/handlers.rs b/hive-matrix-mcp/src/handlers.rs index 5c211028..ca9277b3 100644 --- a/hive-matrix-mcp/src/handlers.rs +++ b/hive-matrix-mcp/src/handlers.rs @@ -33,10 +33,9 @@ pub struct InviteInfo { pub canonical_alias: Option, pub name: Option, /// Who sent the invite — the `sender` of the `m.room.member` invite - /// event for this account. `None` only on an SDK-side lookup failure - /// (`invite_details` errors if the room isn't genuinely in the - /// `Invited` state, which shouldn't happen here since we got the room - /// from `invited_rooms()` itself). + /// event for this account. `None` when the invite's member event isn't + /// in the local store yet (`invite_details` errors in that case), not + /// just on an outright lookup failure. pub inviter: Option, /// The room's topic, where set. Most invites to a freshly created room /// won't have one; `canonical_alias`/`name` are just as commonly unset,