From a39edc9bd3cf3c951ab53ecb79a77df4394e9ee4 Mon Sep 17 00:00:00 2001
From: mcalinghee <mcalinghee.dev@gmail.com>
Date: Sat, 9 May 2026 14:33:58 +0200
Subject: [PATCH] add displayname and avatar url when adding user with admin
 api

---
 crates/handlers/src/admin/v1/users/add.rs | 27 ++++++++++++++++++++++-
 docs/api/spec.json                        | 14 ++++++++++++
 2 files changed, 40 insertions(+), 1 deletion(-)

diff --git a/crates/handlers/src/admin/v1/users/add.rs b/crates/handlers/src/admin/v1/users/add.rs
index 9b3307e8d..976ae818f 100644
--- a/crates/handlers/src/admin/v1/users/add.rs
+++ b/crates/handlers/src/admin/v1/users/add.rs
@@ -103,6 +103,12 @@ pub struct Request {
     /// tokens (like with admin access) for them
     #[serde(default)]
     skip_homeserver_check: bool,
+
+    /// The displayname of the user to add.
+    displayname: Option<String>,
+
+    /// The avatar URL of the user to add.
+    avatar_url: Option<String>,
 }
 
 pub fn doc(operation: TransformOperation) -> TransformOperation {
@@ -165,8 +171,16 @@ pub async fn handler(
 
     let user = repo.user().add(&mut rng, &clock, params.username).await?;
 
+    let mut provision_request = ProvisionRequest::new(&user.username, &user.sub, false);
+    if let Some(displayname) = params.displayname {
+        provision_request = provision_request.set_displayname(displayname);
+    }
+    if let Some(avatar_url) = params.avatar_url {
+        provision_request = provision_request.set_avatar_url(avatar_url);
+    }
+
     homeserver
-        .provision_user(&ProvisionRequest::new(&user.username, &user.sub, false))
+        .provision_user(&provision_request)
         .await
         .map_err(RouteError::Homeserver)?;
 
@@ -197,6 +211,8 @@ mod tests {
             .bearer(&token)
             .json(serde_json::json!({
                 "username": "alice",
+                "displayname": "Alice Test",
+                "avatar_url": "mxc://homeserver/4880dc98b127f4a5f4c3c9f588e1f37af70047da1810312767102517248",
             }));
 
         let response = state.request(request).await;
@@ -221,6 +237,15 @@ mod tests {
         // Check that the user was created on the homeserver
         let result = state.homeserver_connection.query_user("alice").await;
         assert!(result.is_ok());
+        let user = result.unwrap();
+        assert_eq!(user.displayname, Some("Alice Test".to_owned()));
+        assert_eq!(
+            user.avatar_url,
+            Some(
+                "mxc://homeserver/4880dc98b127f4a5f4c3c9f588e1f37af70047da1810312767102517248"
+                    .to_owned()
+            )
+        );
     }
 
     #[sqlx::test(migrator = "mas_storage_pg::MIGRATOR")]
diff --git a/docs/api/spec.json b/docs/api/spec.json
index ac56910b8..bf6a62f3e 100644
--- a/docs/api/spec.json
+++ b/docs/api/spec.json
@@ -6301,6 +6301,20 @@
             "description": "Skip checking with the homeserver whether the username is available.\n\n Use this with caution! The main reason to use this, is when a user used\n by an application service needs to exist in MAS to craft special\n tokens (like with admin access) for them",
             "type": "boolean",
             "default": false
+          },
+          "displayname": {
+            "description": "The displayname of the user to add.",
+            "type": [
+              "string",
+              "null"
+            ]
+          },
+          "avatar_url": {
+            "description": "The avatar URL of the user to add.",
+            "type": [
+              "string",
+              "null"
+            ]
           }
         },
         "required": [