From a0ddfc0fd77c44d992afbfe01525aa9f68ec9ca7 Mon Sep 17 00:00:00 2001
From: bones_was_here <bones_was_here@xonotic.au>
Date: Wed, 8 Jul 2026 01:04:14 +1000
Subject: [PATCH] Fix overly strict client capabilities check (#1158)

These are optional per the spec and `m.room_versions` seems to be the
only one Draupnir needs. Removed `m.set_avatar_url` and
`m.set_displayname` because they're deprecated as well as unused.

Fixes https://github.com/the-draupnir-project/Draupnir/issues/1102

Signed-off-by: bones_was_here <bones_was_here@xonotic.au>
---
 .../src/Client/ClientCapabilityNegotiation.ts | 74 ++++++++-----------
 1 file changed, 30 insertions(+), 44 deletions(-)

diff --git a/packages/matrix-protection-suite/src/Client/ClientCapabilityNegotiation.ts b/packages/matrix-protection-suite/src/Client/ClientCapabilityNegotiation.ts
index 3564adb2..ecd70105 100644
--- a/packages/matrix-protection-suite/src/Client/ClientCapabilityNegotiation.ts
+++ b/packages/matrix-protection-suite/src/Client/ClientCapabilityNegotiation.ts
@@ -28,56 +28,42 @@ const RoomVersionsCapability = Type.Object(
 export type ClientCapabilities = Static<typeof ClientCapabilities>;
 export const ClientCapabilities = Type.Object(
   {
-    "m.change_password": Type.Object(
-      {
-        enabled: Type.Boolean(),
-      },
-      {
-        description:
-          "Capability to indicate if the user can change their password.",
-      }
+    "m.change_password": Type.Optional(
+      Type.Object(
+        {
+          enabled: Type.Boolean(),
+        },
+        {
+          description:
+            "Capability to indicate if the user can change their password.",
+        }
+      )
     ),
 
     "m.room_versions": RoomVersionsCapability,
 
-    "m.set_displayname": Type.Object(
-      {
-        enabled: Type.Boolean(),
-      },
-      {
-        description:
-          "Capability to indicate if the user can change their display name.",
-      }
+    "m.3pid_changes": Type.Optional(
+      Type.Object(
+        {
+          enabled: Type.Boolean(),
+        },
+        {
+          description:
+            "Capability to indicate if the user can change 3PID associations on their account.",
+        }
+      )
     ),
 
-    "m.set_avatar_url": Type.Object(
-      {
-        enabled: Type.Boolean(),
-      },
-      {
-        description:
-          "Capability to indicate if the user can change their avatar.",
-      }
-    ),
-
-    "m.3pid_changes": Type.Object(
-      {
-        enabled: Type.Boolean(),
-      },
-      {
-        description:
-          "Capability to indicate if the user can change 3PID associations on their account.",
-      }
-    ),
-
-    "m.get_login_token": Type.Object(
-      {
-        enabled: Type.Boolean(),
-      },
-      {
-        description:
-          "Capability to indicate if the user can generate tokens to log further clients into their account.",
-      }
+    "m.get_login_token": Type.Optional(
+      Type.Object(
+        {
+          enabled: Type.Boolean(),
+        },
+        {
+          description:
+            "Capability to indicate if the user can generate tokens to log further clients into their account.",
+        }
+      )
     ),
   },
   {