Assorted minor fixes

matrix_appservice_kakaotalk/commands/auth.py

@@ -66,7 +66,7 @@ async def login(evt: CommandEvent) -> None:
             "action": "Login",
             "room_id": evt.room_id,
             "next": enter_password,
-            "email": evt.args[0],
+            "email": email,
         }
 
     """ TODO Implement web login

matrix_appservice_kakaotalk/commands/conn.py

@@ -96,7 +96,7 @@ async def ping(evt: CommandEvent) -> None:
     management_only=True,
     help_section=SECTION_CONNECTION,
     help_text="(Re)connect to KakaoTalk chats & sync any missed chat updates",
-    help_args="[number_of_channels_to_sync]",
+    help_args="[_number of channels to sync_]",
 )
 async def sync(evt: CommandEvent) -> None:
     try:
@@ -104,7 +104,7 @@ async def sync(evt: CommandEvent) -> None:
     except IndexError:
         sync_count = None
     except ValueError:
-        await evt.reply("**Usage:** `sync [number_of_channels_to_sync]`")
+        await evt.reply("The number of channels to sync must either be an integer, or be left unspecified.")
         return
 
     await evt.mark_read()

matrix_appservice_kakaotalk/db/__init__.py

@@ -32,7 +32,6 @@ __all__ = [
     "init",
     "Message",
     "Portal",
-    "ThreadType",
     "Puppet",
     "User",
 ]

matrix_appservice_kakaotalk/example-config.yaml

@@ -93,9 +93,7 @@ bridge:
     # {userid} is replaced with the user ID of the KakaoTalk user.
     username_template: "kakaotalk_{userid}"
     # Displayname template for KakaoTalk users.
-    # {displayname} is replaced with the display name of the KakaoTalk user
+    # {displayname} is replaced with the display name of the KakaoTalk user.
-    #               as defined below in displayname_preference.
-    # Keys available for displayname_preference are also available here.
     displayname_template: "{displayname} (KT)"
 
     # The prefix for commands. Only required in non-management rooms.

matrix_appservice_kakaotalk/user.py

@@ -135,7 +135,7 @@ class User(DBUser, BaseUser):
         self._is_connected = None
         self._connection_time = time.monotonic()
         self._sync_lock = SimpleLock(
-            "Waiting for thread sync to finish before handling %s", log=self.log
+            "Waiting for channel sync to finish before handling %s", log=self.log
         )
         self._is_rpc_reconnecting = False
         self._logged_in_info = None
@@ -348,7 +348,7 @@ class User(DBUser, BaseUser):
                     state_event=BridgeStateEvent.TRANSIENT_DISCONNECT,
                 )
                 await asyncio.sleep(60)
-                await self.reload_session(event_id, retries - 1, is_startup)
+                await self.reload_session(event_id, retries - 1)
             else:
                 await self.send_bridge_notice(
                     notice,
@@ -685,8 +685,9 @@ class User(DBUser, BaseUser):
             if self.temp_disconnect_notices:
                 await self.send_bridge_notice(
                     "Disconnected from KakaoTalk: backend helper module exited. "
-                    "Will reconnect once module resumes."
+                    "Will reconnect once the module resumes."
                 )
+            await self.push_bridge_state(BridgeStateEvent.TRANSIENT_DISCONNECT)
             self._is_rpc_reconnecting = True
             asyncio.create_task(self.reload_session())