Update Black to 2024 style and Python 3.10 target

Author: tulir

.github/workflows/python-package.yml

@@ -56,7 +56,7 @@ jobs:
     - uses: psf/black@stable
       with:
         src: "./mautrix"
-        version: "23.11.0"
+        version: "24.1.1"
     - name: pre-commit
       run: |
         pip install pre-commit

.pre-commit-config.yaml

@@ -8,13 +8,13 @@ repos:
       - id: check-yaml
       - id: check-added-large-files
   - repo: https://github.com/psf/black
-    rev: 23.11.0
+    rev: 24.1.1
     hooks:
       - id: black
         language_version: python3
         files: ^mautrix/.*\.pyi?$
   - repo: https://github.com/PyCQA/isort
-    rev: 5.12.0
+    rev: 5.13.2
     hooks:
       - id: isort
         files: ^mautrix/.*\.pyi?$

dev-requirements.txt

@@ -1,3 +1,3 @@
 pre-commit>=2.10.1,<3
 isort>=5.10.1,<6
-black>=23,<24
+black>=24,<25

mautrix/bridge/config.py

@@ -200,14 +200,18 @@ def namespaces(self) -> dict[str, list[dict[str, Any]]]:
                     "regex": re.escape(f"@{username_format}:{homeserver}").replace(regex_ph, ".*"),
                 }
             ],
-            "aliases": [
-                {
-                    "exclusive": True,
-                    "regex": re.escape(f"#{alias_format}:{homeserver}").replace(regex_ph, ".*"),
-                }
-            ]
-            if alias_format
-            else [],
+            "aliases": (
+                [
+                    {
+                        "exclusive": True,
+                        "regex": re.escape(f"#{alias_format}:{homeserver}").replace(
+                            regex_ph, ".*"
+                        ),
+                    }
+                ]
+                if alias_format
+                else []
+            ),
         }
 
     def generate_registration(self) -> None:

mautrix/client/api/filtering.py

@@ -50,9 +50,11 @@ async def create_filter(self, filter_params: Filter) -> FilterID:
         resp = await self.api.request(
             Method.POST,
             Path.v3.user[self.mxid].filter,
-            filter_params.serialize()
-            if isinstance(filter_params, Serializable)
-            else filter_params,
+            (
+                filter_params.serialize()
+                if isinstance(filter_params, Serializable)
+                else filter_params
+            ),
         )
         try:
             return resp["filter_id"]

mautrix/client/api/rooms.py

@@ -354,9 +354,12 @@ async def join_room(
         except KeyError:
             raise MatrixResponseError("`room_id` not in response.")
 
-    fill_member_event_callback: Callable[
-        [RoomID, UserID, MemberStateEventContent], Awaitable[MemberStateEventContent | None]
-    ] | None
+    fill_member_event_callback: (
+        Callable[
+            [RoomID, UserID, MemberStateEventContent], Awaitable[MemberStateEventContent | None]
+        ]
+        | None
+    )
 
     async def fill_member_event(
         self, room_id: RoomID, user_id: UserID, content: MemberStateEventContent

mautrix/crypto/encrypt_megolm.py

@@ -95,9 +95,9 @@ async def _encrypt_megolm_event(
                 {
                     "room_id": room_id,
                     "type": event_type.serialize(),
-                    "content": content.serialize()
-                    if isinstance(content, Serializable)
-                    else content,
+                    "content": (
+                        content.serialize() if isinstance(content, Serializable) else content
+                    ),
                 }
             )
         )

mautrix/types/event/type.pyi

@@ -18,6 +18,7 @@ class EventType(Serializable):
         ACCOUNT_DATA = "account_data"
         EPHEMERAL = "ephemeral"
         TO_DEVICE = "to_device"
+
     _by_event_type: ClassVar[dict[str, EventType]]
 
     ROOM_CANONICAL_ALIAS: "EventType"

mautrix/util/bridge_state.py

@@ -115,9 +115,10 @@ async def send(self, url: str, token: str, log: logging.Logger, log_sent: bool =
         self.send_attempts_ += 1
         headers = {"Authorization": f"Bearer {token}", "User-Agent": HTTPAPI.default_ua}
         try:
-            async with aiohttp.ClientSession() as sess, sess.post(
-                url, json=self.serialize(), headers=headers
-            ) as resp:
+            async with (
+                aiohttp.ClientSession() as sess,
+                sess.post(url, json=self.serialize(), headers=headers) as resp,
+            ):
                 if not 200 <= resp.status < 300:
                     text = await resp.text()
                     text = text.replace("\n", "\\n")

mautrix/util/message_send_checkpoint.py

@@ -57,12 +57,15 @@ async def send(self, endpoint: str, as_token: str, log: logging.Logger) -> None:
             return
         try:
             headers = {"Authorization": f"Bearer {as_token}", "User-Agent": HTTPAPI.default_ua}
-            async with aiohttp.ClientSession() as sess, sess.post(
-                endpoint,
-                json={"checkpoints": [self.serialize()]},
-                headers=headers,
-                timeout=ClientTimeout(30),
-            ) as resp:
+            async with (
+                aiohttp.ClientSession() as sess,
+                sess.post(
+                    endpoint,
+                    json={"checkpoints": [self.serialize()]},
+                    headers=headers,
+                    timeout=ClientTimeout(30),
+                ) as resp,
+            ):
                 if not 200 <= resp.status < 300:
                     text = await resp.text()
                     text = text.replace("\n", "\\n")