matrix_sdk 2.1.0 → 2.3.0

data/lib/matrix_sdk/errors.rb

@@ -41,9 +41,13 @@ module MatrixSdk
   end
 
   class MatrixNotAuthorizedError < MatrixRequestError; end
+
   class MatrixForbiddenError < MatrixRequestError; end
+
   class MatrixNotFoundError < MatrixRequestError; end
+
   class MatrixConflictError < MatrixRequestError; end
+
   class MatrixTooManyRequestsError < MatrixRequestError; end
 
   # An error raised when errors occur in the connection layer

data/lib/matrix_sdk/mxid.rb

@@ -32,13 +32,13 @@ module MatrixSdk
     #
     # @return [String]
     def homeserver
-      port_s = port ? ':' + port.to_s : ''
+      port_s = port ? ":#{port}" : ''
       domain ? domain + port_s : ''
     end
 
     # Gets the homserver part of the ID as a suffix (':homeserver')
     def homeserver_suffix
-      ':' + homeserver if domain
+      ":#{homeserver}" if domain
     end
 
     def to_s
@@ -49,18 +49,13 @@ module MatrixSdk
     #
     # @return [Symbol] The MXID type, one of (:user_id, :room_id, :event_id, :group_id, or :room_alias)
     def type
-      case sigil
-      when '@'
-        :user_id
-      when '!'
-        :room_id
-      when '$'
-        :event_id
-      when '+'
-        :group_id
-      when '#'
-        :room_alias
-      end
+      {
+        '@' => :user_id,
+        '!' => :room_id,
+        '$' => :event_id,
+        '+' => :group_id,
+        '#' => :room_alias
+      }[sigil]
     end
 
     # Checks if the ID is valid

data/lib/matrix_sdk/protocols/cs.rb

@@ -584,7 +584,7 @@ module MatrixSdk::Protocols::CS
     }
     content.merge!(params.fetch(:extra_content)) if params.key? :extra_content
 
-    send_message_event(room_id, 'm.room.message', content, params)
+    send_message_event(room_id, 'm.room.message', content, **params)
   end
 
   # Send a geographic location to a room
@@ -610,7 +610,7 @@ module MatrixSdk::Protocols::CS
     content[:info][:thumbnail_url] = params.delete(:thumbnail_url) if params.key? :thumbnail_url
     content[:info][:thumbnail_info] = params.delete(:thumbnail_info) if params.key? :thumbnail_info
 
-    send_message_event(room_id, 'm.room.message', content, params)
+    send_message_event(room_id, 'm.room.message', content, **params)
   end
 
   # Send a plaintext message to a room
@@ -628,7 +628,7 @@ module MatrixSdk::Protocols::CS
       msgtype: params.delete(:msg_type) { 'm.text' },
       body: message
     }
-    send_message_event(room_id, 'm.room.message', content, params)
+    send_message_event(room_id, 'm.room.message', content, **params)
   end
 
   # Send a plaintext emote to a room
@@ -646,7 +646,7 @@ module MatrixSdk::Protocols::CS
       msgtype: params.delete(:msg_type) { 'm.emote' },
       body: emote
     }
-    send_message_event(room_id, 'm.room.message', content, params)
+    send_message_event(room_id, 'm.room.message', content, **params)
   end
 
   # Send a plaintext notice to a room
@@ -664,7 +664,7 @@ module MatrixSdk::Protocols::CS
       msgtype: params.delete(:msg_type) { 'm.notice' },
       body: notice
     }
-    send_message_event(room_id, 'm.room.message', content, params)
+    send_message_event(room_id, 'm.room.message', content, **params)
   end
 
   # Report an event in a room
@@ -702,7 +702,7 @@ module MatrixSdk::Protocols::CS
   # @return [Response] A response hash with the message information containing :start, :end, and :chunk fields
   # @see https://matrix.org/docs/spec/client_server/latest.html#get-matrix-client-r0-rooms-roomid-messages
   #      The Matrix Spec, for more information about the call and response
-  def get_room_messages(room_id, token, direction, limit: 10, **params)
+  def get_room_messages(room_id, token, direction:, limit: 10, **params)
     query = {
       from: token,
       dir: direction,
@@ -767,6 +767,30 @@ module MatrixSdk::Protocols::CS
     request(:get, :client_r0, "/rooms/#{room_id}/state", query: query)
   end
 
+  # Retrieves number of events that happened just before and after the specified event
+  #
+  # @param room_id [MXID,String] The room to get events from.
+  # @param event_id [MXID,String] The event to get context around.
+  # @option params [Integer] :limit (10) The limit of messages to retrieve
+  # @option params [String] :filter A filter to limit the retrieval to
+  # @return [Response] A response hash with contextual event information
+  # @see https://matrix.org/docs/spec/client_server/r0.6.1#get-matrix-client-r0-rooms-roomid-context-eventid
+  #      The Matrix Spec, for more information about the call and response
+  # @example Find event context with filter and limit specified
+  #   api.get_room_event_context('#room:example.com', '$event_id:example.com', filter: { types: ['m.room.message'] }.to_json, limit: 20)
+  def get_room_event_context(room_id, event_id, **params)
+    query = {}
+    query[:user_id] = params.delete(:user_id) if protocol?(:AS) && params.key?(:user_id)
+
+    query[:limit] = params.fetch(:limit) if params.key? :limit
+    query[:filter] = params.fetch(:filter) if params.key? :filter
+
+    room_id = ERB::Util.url_encode room_id.to_s
+    event_id = ERB::Util.url_encode event_id.to_s
+
+    request(:get, :client_r0, "/rooms/#{room_id}/context/#{event_id}", query: query)
+  end
+
   ## Specialized getters for specced state
   #
 
@@ -780,7 +804,7 @@ module MatrixSdk::Protocols::CS
   # @see https://matrix.org/docs/spec/client_server/latest.html#m-room-name
   #      The Matrix Spec, for more information about the event and data
   def get_room_name(room_id, **params)
-    get_room_state(room_id, 'm.room.name', params)
+    get_room_state(room_id, 'm.room.name', **params)
   end
 
   # Sets the display name of a room
@@ -795,7 +819,7 @@ module MatrixSdk::Protocols::CS
     content = {
       name: name
     }
-    send_state_event(room_id, 'm.room.name', content, params)
+    send_state_event(room_id, 'm.room.name', content, **params)
   end
 
   # Gets the current topic of a room
@@ -808,7 +832,7 @@ module MatrixSdk::Protocols::CS
   # @see https://matrix.org/docs/spec/client_server/latest.html#m-room-topic
   #      The Matrix Spec, for more information about the event and data
   def get_room_topic(room_id, **params)
-    get_room_state(room_id, 'm.room.topic', params)
+    get_room_state(room_id, 'm.room.topic', **params)
   end
 
   # Sets the topic of a room
@@ -823,7 +847,7 @@ module MatrixSdk::Protocols::CS
     content = {
       topic: topic
     }
-    send_state_event(room_id, 'm.room.topic', content, params)
+    send_state_event(room_id, 'm.room.topic', content, **params)
   end
 
   # Gets the current avatar URL of a room
@@ -836,7 +860,7 @@ module MatrixSdk::Protocols::CS
   # @see https://matrix.org/docs/spec/client_server/latest.html#m-room-avatar
   #      The Matrix Spec, for more information about the event and data
   def get_room_avatar(room_id, **params)
-    get_room_state(room_id, 'm.room.avatar', params)
+    get_room_state(room_id, 'm.room.avatar', **params)
   end
 
   # Sets the avatar URL for a room
@@ -851,7 +875,7 @@ module MatrixSdk::Protocols::CS
     content = {
       url: url
     }
-    send_state_event(room_id, 'm.room.avatar', content, params)
+    send_state_event(room_id, 'm.room.avatar', content, **params)
   end
 
   # Gets a list of current aliases of a room
@@ -878,7 +902,7 @@ module MatrixSdk::Protocols::CS
   #      .compact
   #   # => ["#synapse:im.kabi.tk", "#synapse:matrix.org", "#synapse-community:matrix.org", "#synapse-ops:matrix.org", "#synops:matrix.org", ...
   def get_room_aliases(room_id, **params)
-    get_room_state(room_id, 'm.room.aliases', params)
+    get_room_state(room_id, 'm.room.aliases', **params)
   end
 
   # Gets a list of pinned events in a room
@@ -891,7 +915,7 @@ module MatrixSdk::Protocols::CS
   # @see https://matrix.org/docs/spec/client_server/latest.html#m-room-pinned-events
   #      The Matrix Spec, for more information about the event and data
   def get_room_pinned_events(room_id, **params)
-    get_room_state(room_id, 'm.room.pinned_events', params)
+    get_room_state(room_id, 'm.room.pinned_events', **params)
   end
 
   # Sets the list of pinned events in a room
@@ -906,7 +930,7 @@ module MatrixSdk::Protocols::CS
     content = {
       pinned: events
     }
-    send_state_event(room_id, 'm.room.pinned_events', content, params)
+    send_state_event(room_id, 'm.room.pinned_events', content, **params)
   end
 
   # Gets the configured power levels for a room
@@ -918,7 +942,7 @@ module MatrixSdk::Protocols::CS
   # @see https://matrix.org/docs/spec/client_server/latest.html#m-room-power-levels
   #      The Matrix Spec, for more information about the event and data
   def get_room_power_levels(room_id, **params)
-    get_room_state(room_id, 'm.room.power_levels', params)
+    get_room_state(room_id, 'm.room.power_levels', **params)
   end
   alias get_power_levels get_room_power_levels
 
@@ -932,7 +956,7 @@ module MatrixSdk::Protocols::CS
   #      The Matrix Spec, for more information about the event and data
   def set_room_power_levels(room_id, content, **params)
     content[:events] = {} unless content.key? :events
-    send_state_event(room_id, 'm.room.power_levels', content, params)
+    send_state_event(room_id, 'm.room.power_levels', content, **params)
   end
   alias set_power_levels set_room_power_levels
 
@@ -945,7 +969,7 @@ module MatrixSdk::Protocols::CS
   # @see https://matrix.org/docs/spec/client_server/latest.html#m-room-join-rules
   #      The Matrix Spec, for more information about the event and data
   def get_room_join_rules(room_id, **params)
-    get_room_state(room_id, 'm.room.join_rules', params)
+    get_room_state(room_id, 'm.room.join_rules', **params)
   end
 
   # Sets the join rules for a room
@@ -961,7 +985,7 @@ module MatrixSdk::Protocols::CS
       join_rule: join_rule
     }
 
-    send_state_event(room_id, 'm.room.join_rules', content, params)
+    send_state_event(room_id, 'm.room.join_rules', content, **params)
   end
 
   # Gets the guest access settings for a room
@@ -973,7 +997,7 @@ module MatrixSdk::Protocols::CS
   # @see https://matrix.org/docs/spec/client_server/latest.html#m-room-guest-access
   #      The Matrix Spec, for more information about the event and data
   def get_room_guest_access(room_id, **params)
-    get_room_state(room_id, 'm.room.guest_access', params)
+    get_room_state(room_id, 'm.room.guest_access', **params)
   end
 
   # Sets the guest access settings for a room
@@ -989,7 +1013,7 @@ module MatrixSdk::Protocols::CS
       guest_access: guest_access
     }
 
-    send_state_event(room_id, 'm.room.guest_access', content, params)
+    send_state_event(room_id, 'm.room.guest_access', content, **params)
   end
 
   # Gets the creation configuration object for a room
@@ -1001,7 +1025,7 @@ module MatrixSdk::Protocols::CS
   # @see https://matrix.org/docs/spec/client_server/latest.html#m-room-create
   #      The Matrix Spec, for more information about the event and data
   def get_room_creation_info(room_id, **params)
-    get_room_state(room_id, 'm.room.create', params)
+    get_room_state(room_id, 'm.room.create', **params)
   end
 
   # Gets the encryption configuration for a room
@@ -1013,7 +1037,7 @@ module MatrixSdk::Protocols::CS
   # @see https://matrix.org/docs/spec/client_server/latest.html#m-room-encryption
   #      The Matrix Spec, for more information about the event and data
   def get_room_encryption_settings(room_id, **params)
-    get_room_state(room_id, 'm.room.encryption', params)
+    get_room_state(room_id, 'm.room.encryption', **params)
   end
 
   # Sets the encryption configuration for a room
@@ -1032,7 +1056,7 @@ module MatrixSdk::Protocols::CS
       rotation_period_ms: rotation_period_ms,
       rotation_period_msgs: rotation_period_msgs
     }
-    send_state_event(room_id, 'm.room.encryption', content, params)
+    send_state_event(room_id, 'm.room.encryption', content, **params)
   end
 
   # Gets the history availabiilty for a room
@@ -1044,7 +1068,7 @@ module MatrixSdk::Protocols::CS
   # @see https://matrix.org/docs/spec/client_server/latest.html#m-room-history-visibility
   #      The Matrix Spec, for more information about the event and data
   def get_room_history_visibility(room_id, **params)
-    get_room_state(room_id, 'm.room.history_visibility', params)
+    get_room_state(room_id, 'm.room.history_visibility', **params)
   end
 
   # Sets the history availability for a room
@@ -1060,7 +1084,7 @@ module MatrixSdk::Protocols::CS
       history_visibility: visibility
     }
 
-    send_state_event(room_id, 'm.room.history_visibility', content, params)
+    send_state_event(room_id, 'm.room.history_visibility', content, **params)
   end
 
   # Gets the server ACLs for a room
@@ -1072,7 +1096,7 @@ module MatrixSdk::Protocols::CS
   # @see https://matrix.org/docs/spec/client_server/latest.html#m-room-server-acl
   #      The Matrix Spec, for more information about the event and data
   def get_room_server_acl(room_id, **params)
-    get_room_state(room_id, 'm.room.server_acl', params)
+    get_room_state(room_id, 'm.room.server_acl', **params)
   end
 
   # Sets the server ACL configuration for a room
@@ -1085,14 +1109,14 @@ module MatrixSdk::Protocols::CS
   # @return [Response] The resulting state event
   # @see https://matrix.org/docs/spec/client_server/latest.html#m-room-guest-server-acl
   #      The Matrix Spec, for more information about the event and data
-  def set_room_server_acl(room_id, allow_ip_literals: false, allow:, deny:, **params)
+  def set_room_server_acl(room_id, allow:, deny:, allow_ip_literals: false, **params)
     content = {
       allow_ip_literals: allow_ip_literals,
       allow: allow,
       deny: deny
     }
 
-    send_state_event(room_id, 'm.room.server_acl', content, params)
+    send_state_event(room_id, 'm.room.server_acl', content, **params)
   end
 
   def leave_room(room_id, **params)
@@ -1637,7 +1661,7 @@ module MatrixSdk::Protocols::CS
 
     room_id = ERB::Util.url_encode room_id.to_s
 
-    request(:get, :client_r0, "/rooms/#{room_id}/members", query: query)
+    request(:get, :client_r0, "/rooms/#{room_id}/members", query: query.merge(params))
   end
 
   # Gets a list of the joined members in a room
@@ -1712,7 +1736,7 @@ module MatrixSdk::Protocols::CS
   #   # => { :device_keys => { :'@alice:example.com' => { :ABCDEFGHIJ => { ...
   # @see https://matrix.org/docs/spec/client_server/latest#post-matrix-client-r0-keys-query
   #      The Matrix Spec, for more information about the parameters and data
-  def keys_query(timeout: nil, device_keys:, token: nil, **params)
+  def keys_query(device_keys:, timeout: nil, token: nil, **params)
     body = {
       timeout: (timeout || 10) * 1000,
       device_keys: device_keys
@@ -1823,7 +1847,7 @@ module MatrixSdk::Protocols::CS
   # @see https://matrix.org/docs/spec/client_server/latest#get-matrix-client-r0-pushrules
   #      The Matrix Spec, for more information about the parameters and data
   def get_pushrules
-    request(:get, :client_r0, '/pushrules')
+    request(:get, :client_r0, '/pushrules/')
   end
 
   # Retrieves a single registered push rule for the current user
@@ -1834,7 +1858,7 @@ module MatrixSdk::Protocols::CS
   # @return [Response] A response hash containing the full data of the requested push rule
   # @see https://matrix.org/docs/spec/client_server/latest#get-matrix-client-r0-pushrules-scope-kind-ruleid
   #      The Matrix Spec, for more information about the parameters and data
-  def get_pushrule(scope: 'global', kind:, id:)
+  def get_pushrule(kind:, id:, scope: 'global')
     scope = ERB::Util.url_encode scope.to_s
     kind = ERB::Util.url_encode kind.to_s
     id = ERB::Util.url_encode id.to_s
@@ -1850,7 +1874,7 @@ module MatrixSdk::Protocols::CS
   # @return [Response] A response hash containing an :enabled key for if the rule is enabled or not
   # @see https://matrix.org/docs/spec/client_server/latest#get-matrix-client-r0-pushrules-scope-kind-ruleid-enabled
   #      The Matrix Spec, for more information about the parameters and data
-  def get_pushrule_enabled(scope: 'global', kind:, id:)
+  def get_pushrule_enabled(kind:, id:, scope: 'global')
     scope = ERB::Util.url_encode scope.to_s
     kind = ERB::Util.url_encode kind.to_s
     id = ERB::Util.url_encode id.to_s
@@ -1867,7 +1891,7 @@ module MatrixSdk::Protocols::CS
   # @return [Response] An empty response hash if the push rule was enabled/disabled successfully
   # @see https://matrix.org/docs/spec/client_server/latest#put-matrix-client-r0-pushrules-scope-kind-ruleid-enabled
   #      The Matrix Spec, for more information about the parameters and data
-  def set_pushrule_enabled(enabled, scope: 'global', kind:, id:)
+  def set_pushrule_enabled(enabled, kind:, id:, scope: 'global')
     scope = ERB::Util.url_encode scope.to_s
     kind = ERB::Util.url_encode kind.to_s
     id = ERB::Util.url_encode id.to_s
@@ -1887,7 +1911,7 @@ module MatrixSdk::Protocols::CS
   # @return [Response] A response hash containing an :enabled key for if the rule is enabled or not
   # @see https://matrix.org/docs/spec/client_server/latest#get-matrix-client-r0-pushrules-scope-kind-ruleid-actions
   #      The Matrix Spec, for more information about the parameters and data
-  def get_pushrule_actions(scope: 'global', kind:, id:)
+  def get_pushrule_actions(kind:, id:, scope: 'global')
     scope = ERB::Util.url_encode scope.to_s
     kind = ERB::Util.url_encode kind.to_s
     id = ERB::Util.url_encode id.to_s
@@ -1904,7 +1928,7 @@ module MatrixSdk::Protocols::CS
   # @return [Response] An empty response hash if the push rule actions were modified successfully
   # @see https://matrix.org/docs/spec/client_server/latest#put-matrix-client-r0-pushrules-scope-kind-ruleid-actions
   #      The Matrix Spec, for more information about the parameters and data
-  def set_pushrule_actions(actions, scope: 'global', kind:, id:)
+  def set_pushrule_actions(actions, kind:, id:, scope: 'global')
     scope = ERB::Util.url_encode scope.to_s
     kind = ERB::Util.url_encode kind.to_s
     id = ERB::Util.url_encode id.to_s

data/lib/matrix_sdk/protocols/msc.rb

@@ -47,7 +47,7 @@ module MatrixSdk::Protocols::MSC
     query[:user_id] = params.delete(:user_id) if protocol?(:AS) && params.key?(:user_id)
 
     req = Net::HTTP::Get.new(homeserver.dup.tap do |u|
-      u.path = api_to_path(:client_r0) + '/sync/sse'
+      u.path = "#{api_to_path :client_r0}/sync/sse"
       u.query = URI.encode_www_form(query)
     end)
     req['accept'] = 'text/event-stream'

data/lib/matrix_sdk/response.rb

@@ -46,7 +46,9 @@ module MatrixSdk
       attr_reader :api
 
       def respond_to_missing?(name, *_args)
-        key? name
+        return true if key? name
+
+        super
       end
 
       def method_missing(name, *args)

data/lib/matrix_sdk/room.rb

@@ -1,62 +1,45 @@
 # frozen_string_literal: true
 
 require 'matrix_sdk'
+require 'matrix_sdk/util/events'
+require 'matrix_sdk/util/tinycache'
 
 module MatrixSdk
   # A class for tracking the information about a room on Matrix
   class Room
     extend MatrixSdk::Extensions
+    extend MatrixSdk::Util::Tinycache
     include MatrixSdk::Logging
 
-    # @!attribute [rw] canonical_alias
-    #   @return [String, nil] the canonical alias of the room
     # @!attribute [rw] event_history_limit
     #   @return [Fixnum] the limit of events to keep in the event log
-    attr_accessor :canonical_alias, :event_history_limit
+    attr_accessor :event_history_limit
     # @!attribute [r] id
     #   @return [String] the internal ID of the room
     # @!attribute [r] client
     #   @return [Client] the client for the room
-    # @!attribute [rw] name
-    #   @return [String, nil] the user-provided name of the room
-    #   @see reload_name!
-    # @!attribute [rw] topic
-    #   @return [String, nil] the user-provided topic of the room
-    #   @see reload_topic!
-    # @!attribute [r] aliases
-    #   @return [Array(String)] a list of user-set aliases for the room
-    #   @see add_alias
-    #   @see reload_alias!
-    # @!attribute [rw] join_rule
-    #   @return [:invite, :public] the join rule for the room -
-    #                    either +:invite+ or +:public+
-    # @!attribute [rw] guest_access
-    #   @return [:can_join, :forbidden] the guest access for the room -
-    #                    either +:can_join+ or +:forbidden+
-    # @!attribute [r] members
-    #   @return [Array(User)] the members of the room
-    #   @see reload_members!
     # @!attribute [r] events
     #   @return [Array(Object)] the last +event_history_limit+ events to arrive in the room
     #   @see https://matrix.org/docs/spec/client_server/r0.3.0.html#get-matrix-client-r0-sync
     #        The timeline events are what will end up in here
-    attr_reader :id, :client, :topic, :aliases, :members, :events
+    attr_reader :id, :client, :events
 
-    # @!attribute [r] on_event
-    #   @return [EventHandlerArray] The list of event handlers for all events
-    # @!attribute [r] on_state_event
-    #   @return [EventHandlerArray] The list of event handlers for only state events
-    # @!attribute [r] on_ephemeral_event
-    #   @return [EventHandlerArray] The list of event handlers for only ephemeral events
-    events :event, :state_event, :ephemeral_event
     # @!method inspect
     #   An inspect method that skips a handful of instance variables to avoid
     #   flooding the terminal with debug data.
     #   @return [String] a regular inspect string without the data for some variables
-    ignore_inspect :client, :members, :events, :prev_batch, :logger,
-                   :on_event, :on_state_event, :on_ephemeral_event
+    ignore_inspect :client, :events, :prev_batch, :logger
+
+    # Requires heavy lookups, so they're cached for an hour
+    cached :joined_members, :aliases, cache_level: :all, expires_in: 60 * 60
+    # Only cache unfiltered requests for all members
+    cached :all_members, unless: proc { |args| args.any? }, cache_level: :all, expires_in: 3600
+
+    # Much simpler to look up, and lighter data-wise, so the cache is wider
+    cached :canonical_alias, :name, :avatar_url, :topic, :guest_access, :join_rule, :power_levels, cache_level: :some, expires_in: 15 * 60
 
     alias room_id id
+    alias members joined_members
 
     # Create a new room instance
     #
@@ -82,38 +65,52 @@ module MatrixSdk
     def initialize(client, room_id, data = {})
       raise ArgumentError, 'Must be given a Client instance' unless client.is_a? Client
 
+      @client = client
       room_id = MXID.new room_id unless room_id.is_a?(MXID)
       raise ArgumentError, 'room_id must be a valid Room ID' unless room_id.room_id?
 
-      event_initialize
-
-      @name = nil
-      @topic = nil
-      @canonical_alias = nil
-      @aliases = []
-      @join_rule = nil
-      @guest_access = nil
-      @world_readable = nil
-      @members = []
       @events = []
-      @members_loaded = false
       @event_history_limit = 10
-      @avatar_url = nil
 
       @prev_batch = nil
 
       data.each do |k, v|
-        instance_variable_set("@#{k}", v) if instance_variable_defined? "@#{k}"
+        next if %i[client].include? k
+
+        if respond_to?("#{k}_cached?".to_sym) && send("#{k}_cached?".to_sym)
+          tinycache_adapter.write(k, v)
+        elsif instance_variable_defined? "@#{k}"
+          instance_variable_set("@#{k}", v)
+        end
       end
 
-      @client = client
       @id = room_id.to_s
 
-      @name_checked = Time.new(0)
-
       logger.debug "Created room #{room_id}"
     end
 
+    #
+    # Event handlers
+    #
+
+    # @!attribute [r] on_event
+    #   @return [EventHandlerArray] The list of event handlers for all events
+    def on_event
+      ensure_room_handlers[:event]
+    end
+
+    # @!attribute [r] on_state_event
+    #   @return [EventHandlerArray] The list of event handlers for only state events
+    def on_state_event
+      ensure_room_handlers[:state_event]
+    end
+
+    # @!attribute [r] on_ephemeral_event
+    #   @return [EventHandlerArray] The list of event handlers for only ephemeral events
+    def on_ephemeral_event
+      ensure_room_handlers[:ephemeral_event]
+    end
+
     #
     # State readers
     #
@@ -142,19 +139,33 @@ module MatrixSdk
       'Empty Room'
     end
 
+    # @return [String, nil] the canonical alias of the room
+    def canonical_alias
+      client.api.get_room_state(id, 'm.room.canonical_alias')[:alias]
+    rescue MatrixSdk::MatrixNotFoundError
+      nil
+    end
+
     # Populates and returns the #members array
     #
     # @return [Array(User)] The list of members in the room
     def joined_members
-      return members if @members_loaded && !members.empty?
-
-      client.api.get_room_joined_members(id)[:joined].each do |mxid, data|
-        ensure_member(User.new(client, mxid.to_s,
-                               display_name: data.fetch(:display_name, nil),
-                               avatar_url: data.fetch(:avatar_url, nil)))
+      client.api.get_room_joined_members(id)[:joined].map do |mxid, data|
+        User.new(client, mxid.to_s,
+                 display_name: data.fetch(:display_name, nil),
+                 avatar_url: data.fetch(:avatar_url, nil))
       end
-      @members_loaded = true
-      members
+    end
+
+    # Get all members (member events) in the room
+    #
+    # @note This will also count members who've knocked, been invited, have left, or have been banned.
+    #
+    # @param params [Hash] Additional query parameters to pass to the room member listing - e.g. for filtering purposes.
+    #
+    # @return [Array(User)] The complete list of members in the room, regardless of membership state
+    def all_members(**params)
+      client.api.get_room_members(id, **params)[:chunk].map { |ch| client.get_user(ch[:state_key]) }
     end
 
     # Gets the current name of the room, querying the API if necessary
@@ -163,10 +174,7 @@ module MatrixSdk
     #
     # @return [String,nil] The room name - if any
     def name
-      return @name if Time.now - @name_checked < 900
-
-      @name_checked = Time.now
-      @name ||= client.api.get_room_name(id)
+      client.api.get_room_name(id)[:name]
     rescue MatrixNotFoundError
       # No room name has been specified
       nil
@@ -176,18 +184,34 @@ module MatrixSdk
     #
     # @return [String,nil] The avatar URL - if any
     def avatar_url
-      @avatar_url ||= client.api.get_room_avatar(id).url
+      client.api.get_room_avatar(id)[:url]
     rescue MatrixNotFoundError
       # No avatar has been set
       nil
     end
 
+    # Gets the room topic - if any
+    #
+    # @return [String,nil] The topic of the room
+    def topic
+      client.api.get_room_topic(id)[:topic]
+    rescue MatrixNotFoundError
+      # No room name has been specified
+      nil
+    end
+
+    # Gets the guest access rights for the room
+    #
+    # @return [:can_join,:forbidden] The current guest access right
     def guest_access
-      @guest_access ||= client.api.get_room_guest_access(id).guest_access.to_sym
+      client.api.get_room_guest_access(id)[:guest_access].to_sym
     end
 
+    # Gets the join rule for the room
+    #
+    # @return [:public,:knock,:invite,:private] The current join rule
     def join_rule
-      @join_rule ||= client.api.get_room_join_rules(id).join_rule.to_sym
+      client.api.get_room_join_rules(id)[:join_rule].to_sym
     end
 
     # Checks if +guest_access+ is set to +:can_join+
@@ -200,6 +224,34 @@ module MatrixSdk
       join_rule == :invite
     end
 
+    # Gets the history visibility of the room
+    #
+    # @return [:invited,:joined,:shared,:world_readable] The current history visibility for the room
+    def history_visibility
+      client.api.get_room_state(id, 'm.room.history_visibility')[:history_visibility].to_sym
+    end
+
+    # Checks if the room history is world readable
+    #
+    # @return [Boolean] If the history is world readable
+    def world_readable?
+      history_visibility == :world_readable
+    end
+    alias world_readable world_readable?
+
+    # Gets the room aliases
+    #
+    # @return [Array[String]] The assigned room aliases
+    def aliases
+      client.api.get_room_aliases(id).aliases
+    rescue MatrixNotFoundError
+      data = client.api.get_room_state_all(id)
+      data.select { |chunk| chunk[:type] == 'm.room.aliases' && chunk.key?(:content) && chunk[:content].key?(:aliases) }
+          .map { |chunk| chunk[:content][:aliases] }
+          .flatten
+          .compact
+    end
+
     #
     # Message handling
     #
@@ -340,7 +392,13 @@ module MatrixSdk
     # @param reverse [Boolean] whether to fill messages in reverse or not
     # @param limit [Integer] the maximum number of messages to backfill
     # @note This will trigger the `on_event` events as messages are added
-    def backfill_messages(reverse = false, limit = 10)
+    def backfill_messages(*args, reverse: false, limit: 10)
+      # To be backwards-compatible
+      if args.length == 2
+        reverse = args.first
+        limit = args.last
+      end
+
       data = client.api.get_room_messages(id, @prev_batch, direction: :b, limit: limit)
 
       events = data[:chunk]
@@ -458,7 +516,7 @@ module MatrixSdk
           @room
         end
         tag_obj.define_singleton_method(:add) do |tag, **data|
-          @room.add_tag(tag.to_s.to_sym, data)
+          @room.add_tag(tag.to_s.to_sym, **data)
           self[tag.to_s.to_sym] = data
           self
         end
@@ -503,22 +561,16 @@ module MatrixSdk
     #
     # @param name [String] The new name to set
     def name=(name)
+      tinycache_adapter.write(:name, name)
       client.api.set_room_name(id, name)
-      @name = name
+      name
     end
 
     # Reloads the name of the room
     #
     # @return [Boolean] if the name was changed or not
     def reload_name!
-      data = begin
-               client.api.get_room_name(id)
-             rescue MatrixNotFoundError
-               nil
-             end
-      changed = data[:name] != @name
-      @name = data[:name] if changed
-      changed
+      clear_name_cache
     end
     alias refresh_name! reload_name!
 
@@ -526,22 +578,16 @@ module MatrixSdk
     #
     # @param topic [String] The new topic to set
     def topic=(topic)
+      tinycache_adapter.write(:topic, topic)
       client.api.set_room_topic(id, topic)
-      @topic = topic
+      topic
     end
 
     # Reloads the topic of the room
     #
     # @return [Boolean] if the topic was changed or not
     def reload_topic!
-      data = begin
-               client.api.get_room_topic(id)
-             rescue MatrixNotFoundError
-               nil
-             end
-      changed = data[:topic] != @topic
-      @topic = data[:topic] if changed
-      changed
+      clear_topic_cache
     end
     alias refresh_topic! reload_topic!
 
@@ -550,7 +596,7 @@ module MatrixSdk
     # @return [Boolean] if the addition was successful or not
     def add_alias(room_alias)
       client.api.set_room_alias(id, room_alias)
-      @aliases << room_alias
+      tinycache_adapter.read(:aliases) << room_alias if tinycache_adapter.exist?(:aliases)
       true
     end
 
@@ -560,21 +606,7 @@ module MatrixSdk
     # @note The list of aliases is not sorted, ordering changes will result in
     #       alias list updates.
     def reload_aliases!
-      begin
-        new_aliases = client.api.get_room_aliases(id).aliases
-      rescue MatrixNotFoundError
-        data = client.api.get_room_state_all(id)
-        new_aliases = data.select { |chunk| chunk[:type] == 'm.room.aliases' && chunk.key?(:content) && chunk[:content].key?(:aliases) }
-                          .map { |chunk| chunk[:content][:aliases] }
-                          .flatten
-                          .compact
-      end
-
-      return false if new_aliases.nil?
-
-      changed = new_aliases != aliases
-      @aliases = new_aliases if changed
-      changed
+      clear_aliases_cache
     end
     alias refresh_aliases! reload_aliases!
 
@@ -583,7 +615,7 @@ module MatrixSdk
     # @param invite_only [Boolean] If it should be invite only or not
     def invite_only=(invite_only)
       self.join_rule = invite_only ? :invite : :public
-      @join_rule == :invite
+      invite_only
     end
 
     # Sets the join rule of the room
@@ -591,7 +623,8 @@ module MatrixSdk
     # @param join_rule [:invite,:public] The join rule of the room
     def join_rule=(join_rule)
       client.api.set_room_join_rules(id, join_rule)
-      @join_rule = join_rule
+      tinycache_adapter.write(:join_rule, join_rule)
+      join_rule
     end
 
     # Sets if guests are allowed in the room
@@ -599,7 +632,7 @@ module MatrixSdk
     # @param allow_guests [Boolean] If guests are allowed to join or not
     def allow_guests=(allow_guests)
       self.guest_access = (allow_guests ? :can_join : :forbidden)
-      @guest_access == :can_join
+      allow_guests
     end
 
     # Sets the guest access status for the room
@@ -607,7 +640,8 @@ module MatrixSdk
     # @param guest_access [:can_join,:forbidden] The new guest access status of the room
     def guest_access=(guest_access)
       client.api.set_room_guest_access(id, guest_access)
-      @guest_access = guest_access
+      tinycache_adapter.write(:guest_access, guest_access)
+      guest_access
     end
 
     # Sets a new avatar URL for the room
@@ -618,7 +652,89 @@ module MatrixSdk
       raise ArgumentError, 'Must be a valid MXC URL' unless avatar_url.is_a? URI::MATRIX
 
       client.api.set_room_avatar(id, avatar_url)
-      @avatar_url = avatar_url
+      tinycache_adapter.write(:avatar_url, avatar_url)
+      avatar_url
+    end
+
+    # Get the power levels of the room
+    #
+    # @note The returned power levels are cached for a minute
+    # @return [Hash] The current power levels as set for the room
+    # @see Protocols::CS#get_power_levels
+    def power_levels
+      client.api.get_power_levels(id)
+    end
+
+    # Gets the power level of a user in the room
+    #
+    # @param user [User,MXID,String] The user to check the power level for
+    # @param use_default [Boolean] Should the user default level be checked if no user-specific one exists
+    # @return [Integer,nil] The current power level for the requested user, nil if there's no user specific level
+    #   and use_default is false
+    def user_powerlevel(user, use_default: true)
+      user = user.id if user.is_a? User
+      user = MXID.new(user.to_s) unless user.is_a? MXID
+      raise ArgumentError, 'Must provide a valid user or MXID' unless user.user?
+
+      level = power_levels[:users][user.to_s.to_sym]
+      level = power_levels[:users_default] || 0 if level.nil? && use_default
+      level
+    end
+
+    # Check if a user is an admin in the room
+    #
+    # @param user [User,MXID,String] The user to check for admin privileges
+    # @param target_level [Integer] The power level that's to be considered as admin privileges
+    # @return [Boolean] If the requested user has a power level highe enough to be an admin
+    # @see #user_powerlevel
+    def admin?(user, target_level: 100)
+      level = user_powerlevel(user, use_default: false)
+      return false unless level
+
+      level >= target_level
+    end
+
+    # Make a user an admin in the room
+    #
+    # @param user [User,MXID,String] The user to give admin privileges
+    # @param level [Integer] The power level to set the user to
+    # @see #modify_user_power_levels
+    def admin!(user, level: 100)
+      return true if admin?(user, target_level: level)
+
+      user = user.id if user.is_a? User
+      user = MXID.new(user.to_s) unless user.is_a? MXID
+      raise ArgumentError, 'Must provide a valid user or MXID' unless user.user?
+
+      modify_user_power_levels({ user.to_s.to_sym => level })
+    end
+
+    # Check if a user is a moderator in the room
+    #
+    # @param user [User,MXID,String] The user to check for admin privileges
+    # @param target_level [Integer] The power level that's to be considered as admin privileges
+    # @return [Boolean] If the requested user has a power level highe enough to be an admin
+    # @see #user_powerlevel
+    def moderator?(user, target_level: 50)
+      level = user_powerlevel(user, use_default: false)
+      return false unless level
+
+      level >= target_level
+    end
+
+    # Make a user a moderator in the room
+    #
+    # @param user [User,MXID,String] The user to give moderator privileges
+    # @param level [Integer] The power level to set the user to
+    # @see #modify_user_power_levels
+    def moderator!(user, level: 50)
+      return true if moderator?(user, target_level: level)
+
+      user = user.id if user.is_a? User
+      user = MXID.new(user.to_s) unless user.is_a? MXID
+      raise ArgumentError, 'Must provide a valid user or MXID' unless user.user?
+
+      modify_user_power_levels({ user.to_s.to_sym => level })
     end
 
     # Modifies the power levels of the room
@@ -629,7 +745,8 @@ module MatrixSdk
     def modify_user_power_levels(users = nil, users_default = nil)
       return false if users.nil? && users_default.nil?
 
-      data = client.api.get_power_levels(id)
+      data = power_levels_without_cache
+      tinycache_adapter.write(:power_levels, data)
       data[:users_default] = users_default unless users_default.nil?
 
       if users
@@ -650,7 +767,8 @@ module MatrixSdk
     def modify_required_power_levels(events = nil, params = {})
       return false if events.nil? && (params.nil? || params.empty?)
 
-      data = client.api.get_power_levels(id)
+      data = power_levels_without_cache
+      tinycache_adapter.write(:power_levels, data)
       data.merge!(params)
       data.delete_if { |_k, v| v.nil? }
 
@@ -667,18 +785,103 @@ module MatrixSdk
     private
 
     def ensure_member(member)
+      tinycache_adapter.write(:joined_members, []) unless tinycache_adapter.exist? :joined_members
+
+      members = tinycache_adapter.read(:joined_members) || []
       members << member unless members.any? { |m| m.id == member.id }
     end
 
+    def handle_power_levels(event)
+      tinycache_adapter.write(:power_levels, event[:content])
+    end
+
+    def handle_room_name(event)
+      tinycache_adapter.write(:name, event[:content][:name])
+    end
+
+    def handle_room_topic(event)
+      tinycache_adapter.write(:topic, event[:content][:topic])
+    end
+
+    def handle_room_guest_access(event)
+      tinycache_adapter.write(:guest_access, event[:content][:guest_access].to_sym)
+    end
+
+    def handle_room_join_rules(event)
+      tinycache_adapter.write(:join_rule, event[:content][:join_rule].to_sym)
+    end
+
+    def handle_room_member(event)
+      return unless client.cache == :all
+
+      if event[:content][:membership] == 'join'
+        ensure_member(client.get_user(event[:state_key]).dup.tap do |u|
+          u.instance_variable_set :@display_name, event[:content][:displayname]
+        end)
+      elsif tinycache_adapter.exist? :joined_members
+        members = tinycache_adapter.read(:joined_members)
+        members.delete_if { |m| m.id == event[:state_key] }
+      end
+    end
+
+    def handle_room_canonical_alias(event)
+      canonical_alias = tinycache_adapter.write :canonical_alias, event[:content][:alias]
+
+      data = tinycache_adapter.read(:aliases) || []
+      data << canonical_alias
+      tinycache_adapter.write(:aliases, data)
+    end
+
+    def handle_room_aliases(event)
+      tinycache_adapter.write(:aliases, []) unless tinycache_adapter.exist? :aliases
+
+      aliases = tinycache_adapter.read(:aliases) || []
+      aliases.concat event[:content][:aliases]
+
+      tinycache_adapter.write(:aliases, aliases)
+    end
+
+    def room_handlers?
+      client.instance_variable_get(:@room_handlers).key? id
+    end
+
+    def ensure_room_handlers
+      client.instance_variable_get(:@room_handlers)[id] ||= {
+        event: MatrixSdk::EventHandlerArray.new,
+        state_event: MatrixSdk::EventHandlerArray.new,
+        ephemeral_event: MatrixSdk::EventHandlerArray.new
+      }
+    end
+
+    INTERNAL_HANDLERS = {
+      'm.room.aliases' => :handle_room_aliases,
+      'm.room.canonical_alias' => :handle_room_canonical_alias,
+      'm.room.guest_access' => :handle_room_guest_access,
+      'm.room.join_rules' => :handle_room_join_rules,
+      'm.room.member' => :handle_room_member,
+      'm.room.name' => :handle_room_name,
+      'm.room.power_levels' => :handle_power_levels,
+      'm.room.topic' => :handle_room_topic
+    }.freeze
     def put_event(event)
+      ensure_room_handlers[:event].fire(MatrixEvent.new(self, event), event[:type]) if room_handlers?
+
       @events.push event
       @events.shift if @events.length > @event_history_limit
-
-      fire_event MatrixEvent.new(self, event)
     end
 
     def put_ephemeral_event(event)
-      fire_ephemeral_event MatrixEvent.new(self, event)
+      return unless room_handlers?
+
+      ensure_room_handlers[:ephemeral_event].fire(MatrixEvent.new(self, event), event[:type])
+    end
+
+    def put_state_event(event)
+      send(INTERNAL_HANDLERS[event[:type]], event) if INTERNAL_HANDLERS.key? event[:type]
+
+      return unless room_handlers?
+
+      ensure_room_handlers[:state_event].fire(MatrixEvent.new(self, event), event[:type])
     end
   end
 end