Update member count on room summary update

src/hooks/room/useRoomCallStatus.ts

@@ -67,7 +67,7 @@ export const useRoomCallStatus = (
 
     const hasGroupCall = useCall(room.roomId) !== null;
 
-    const memberCount = useRoomMemberCount(room, { includeFunctional: false });
+    const memberCount = useRoomMemberCount(room);
 
     const [mayEditWidgets, mayCreateElementCalls] = useTypedEventEmitterState(
         room,

src/hooks/useRoomMembers.ts

@@ -14,12 +14,11 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 
-import { useState } from "react";
+import { useCallback, useState } from "react";
 import { Room, RoomEvent, RoomMember, RoomStateEvent } from "matrix-js-sdk/src/matrix";
 import { throttle } from "lodash";
 
 import { useTypedEventEmitter } from "./useEventEmitter";
-import { getJoinedNonFunctionalMembers } from "../utils/room/getJoinedNonFunctionalMembers";
 
 // Hook to simplify watching Matrix Room joined members
 export const useRoomMembers = (room: Room, throttleWait = 250): RoomMember[] => {
@@ -43,42 +42,28 @@ type RoomMemberCountOpts = {
      * Wait time between room member count update
      */
     throttleWait?: number;
-    /**
-     * Whether to include functional members (bots, etc...) in the room count
-     * @default true
-     */
-    includeFunctional: boolean;
 };
 
 // Hook to simplify watching Matrix Room joined member count
-export const useRoomMemberCount = (
-    room: Room,
-    opts: RoomMemberCountOpts = { throttleWait: 250, includeFunctional: true },
-): number => {
+export const useRoomMemberCount = (room: Room, opts: RoomMemberCountOpts = { throttleWait: 250 }): number => {
     const [count, setCount] = useState<number>(room.getJoinedMemberCount());
 
-    const { throttleWait, includeFunctional } = opts;
+    const { throttleWait } = opts;
+
+    const onMembershipUpdate = useCallback(() => {
+        setCount(room.getJoinedMemberCount());
+    }, [room]);
 
     useTypedEventEmitter(
         room.currentState,
         RoomStateEvent.Members,
-        throttle(
-            () => {
-                // At the time where `RoomStateEvent.Members` is emitted the
-                // summary API has not had a chance to update the `summaryJoinedMemberCount`
-                // value, therefore handling the logic locally here.
-                //
-                // Tracked as part of https://github.com/vector-im/element-web/issues/26033
-                const membersCount = includeFunctional
-                    ? room.getMembers().reduce((count, m) => {
-                          return m.membership === "join" ? count + 1 : count;
-                      }, 0)
-                    : getJoinedNonFunctionalMembers(room).length;
-                setCount(membersCount);
-            },
-            throttleWait,
-            { leading: true, trailing: true },
-        ),
+        throttle(onMembershipUpdate, throttleWait, { leading: true, trailing: true }),
+    );
+
+    useTypedEventEmitter(
+        room,
+        RoomEvent.Summary,
+        throttle(onMembershipUpdate, throttleWait, { leading: true, trailing: true }),
     );
     return count;
 };