matrix-python 1.4.3a0__tar.gz → 1.4.5a0__tar.gz

{matrix_python-1.4.3a0 → matrix_python-1.4.5a0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: matrix-python
-Version: 1.4.3a0
+Version: 1.4.5a0
 Summary: An easy-to-use Matrix bot framework designed for quick development and minimal setup
 Author: Simon Roy,  Chris Dedman Rollet 
 Maintainer-email: Code Society Lab <admin@codesociety.xyz>
@@ -684,18 +684,18 @@ Project-URL: Source, https://github.com/Code-Society-Lab/matrixpy
 Project-URL: Issues, https://github.com/Code-Society-Lab/matrixpy/issues
 Requires-Python: >=3.10
 Description-Content-Type: text/markdown
-Requires-Dist: matrix-nio
+Requires-Dist: matrix-nio==0.25.2
 Requires-Dist: logger
-Requires-Dist: PyYAML
-Requires-Dist: markdown
-Requires-Dist: APScheduler
+Requires-Dist: PyYAML==6.0.3
+Requires-Dist: markdown==3.10.2
+Requires-Dist: APScheduler==3.11.2
 Provides-Extra: dev
-Requires-Dist: pytest; extra == "dev"
-Requires-Dist: pytest-asyncio; extra == "dev"
-Requires-Dist: black; extra == "dev"
-Requires-Dist: mypy; extra == "dev"
-Requires-Dist: types-PyYAML; extra == "dev"
-Requires-Dist: types-Markdown; extra == "dev"
+Requires-Dist: pytest==9.0.3; extra == "dev"
+Requires-Dist: pytest-asyncio==1.3.0; extra == "dev"
+Requires-Dist: black==26.3.1; extra == "dev"
+Requires-Dist: mypy==1.20.0; extra == "dev"
+Requires-Dist: types-PyYAML==6.0.12.20260408; extra == "dev"
+Requires-Dist: types-Markdown==3.10.2.20260408; extra == "dev"
 
 <div align="center">
   <em>A simple, developer-friendly library to create powerful <a href="https://matrix.org">Matrix</a> bots.</em>

{matrix_python-1.4.3a0 → matrix_python-1.4.5a0}/matrix/_version.py

@@ -18,7 +18,7 @@ version_tuple: tuple[int | str, ...]
 commit_id: str | None
 __commit_id__: str | None
 
-__version__ = version = '1.4.3a0'
-__version_tuple__ = version_tuple = (1, 4, 3, 'a0')
+__version__ = version = '1.4.5a0'
+__version_tuple__ = version_tuple = (1, 4, 5, 'a0')
 
-__commit_id__ = commit_id = 'gba1ba5d3c'
+__commit_id__ = commit_id = 'g1174e983d'

{matrix_python-1.4.3a0 → matrix_python-1.4.5a0}/matrix/bot.py

@@ -3,7 +3,7 @@ import inspect
 import asyncio
 import logging
 
-from typing import Union, Optional, Any
+from typing import Optional, Any
 
 from nio import AsyncClient, Event, MatrixRoom
 
@@ -15,7 +15,12 @@ from .extension import Extension
 from .registry import Registry
 from .help import HelpCommand, DefaultHelpCommand
 from .scheduler import Scheduler
-from .errors import AlreadyRegisteredError, CommandNotFoundError, CheckError
+from .errors import (
+    AlreadyRegisteredError,
+    CommandNotFoundError,
+    CheckError,
+    RoomNotFoundError,
+)
 
 
 class Bot(Registry):
@@ -36,7 +41,9 @@ class Bot(Registry):
 
         self._config: Config | None = None
         self._client: AsyncClient | None = None
+        self._synced: asyncio.Event = asyncio.Event()
         self._help: HelpCommand | None = help_
+
         self.extensions: dict[str, Extension] = {}
         self.scheduler: Scheduler = Scheduler()
         self.log: logging.Logger = logging.getLogger(__name__)
@@ -75,10 +82,23 @@ class Bot(Registry):
             except ValueError:
                 continue
 
-    def get_room(self, room_id: str) -> Room:
-        """Retrieve a Room instance based on the room_id."""
-        matrix_room = self.client.rooms[room_id]
-        return Room(matrix_room=matrix_room, client=self.client)
+    def get_room(self, room_id: str) -> Room | None:
+        """Retrieve a `Room` instance by its Matrix room ID.
+
+        Returns the `Room` object corresponding to `room_id` if it exists in
+        the client's known rooms. Returns `None` if the room cannot be found.
+
+        ## Example
+
+        ```python
+        room = bot.get_room("!abc123:matrix.org")
+        if room:
+            print(room.name)
+        ```
+        """
+        if matrix_room := self.client.rooms.get(room_id):
+            return Room(matrix_room=matrix_room, client=self.client)
+        return None
 
     def load_extension(self, extension: Extension) -> None:
         self.log.debug(f"Loading extension: '{extension.name}'")
@@ -255,10 +275,16 @@ class Bot(Registry):
             login_resp = await self.client.login(self.config.password)
             self.log.info("logged in: %s", login_resp)
 
-        self.scheduler.start()
+        sync_task = asyncio.create_task(self.client.sync_forever(timeout=30_000))
 
+        await self._wait_until_synced()
         await self._on_ready()
-        await self.client.sync_forever(timeout=30_000)
+
+        self.scheduler.start()
+        await sync_task
+
+    async def _wait_until_synced(self) -> None:
+        await self._synced.wait()
 
     # MATRIX EVENTS
 
@@ -266,6 +292,9 @@ class Bot(Registry):
         await self._process_commands(room, event)
 
     async def _on_matrix_event(self, matrix_room: MatrixRoom, event: Event) -> None:
+        if not self._synced.is_set():
+            self._synced.set()
+
         # ignore bot events
         if event.sender == self.client.user:
             return
@@ -276,6 +305,10 @@ class Bot(Registry):
 
         try:
             room = self.get_room(matrix_room.room_id)
+
+            if not room:
+                raise RoomNotFoundError(f"Room '{matrix_room.room_id}' not found.")
+
             await self._dispatch_matrix_event(room, event)
         except Exception as error:
             await self._on_error(error)
@@ -306,6 +339,10 @@ class Bot(Registry):
 
     async def _build_context(self, matrix_room: Room, event: Event) -> Context:
         room = self.get_room(matrix_room.room_id)
+
+        if not room:
+            raise RoomNotFoundError(f"Room '{matrix_room.room_id}' not found.")
+
         ctx = Context(bot=self, room=room, event=event)
         prefix = self.prefix or self.config.prefix
 

{matrix_python-1.4.3a0 → matrix_python-1.4.5a0}/matrix/errors.py

@@ -13,6 +13,10 @@ class MatrixError(Exception):
     pass
 
 
+class RoomNotFoundError(MatrixError):
+    pass
+
+
 class RegistryError(MatrixError):
     pass
 

{matrix_python-1.4.3a0 → matrix_python-1.4.5a0}/matrix/extension.py

@@ -17,7 +17,20 @@ class Extension(Registry):
         self._on_load: Optional[Callable] = None
         self._on_unload: Optional[Callable] = None
 
-    def get_room(self, room_id: str) -> Room:
+    def get_room(self, room_id: str) -> Room | None:
+        """Retrieve a `Room` instance by its Matrix room ID.
+
+        Returns the `Room` object corresponding to `room_id` if it exists in
+        the client's known rooms. Returns `None` if the room cannot be found.
+
+        ## Example
+
+        ```python
+        room = extension.get_room("!abc123:matrix.org")
+        if room:
+            print(room.name)
+        ```
+        """
         if self.bot is None:
             raise RuntimeError("Extension is not loaded")
         return self.bot.get_room(room_id)

{matrix_python-1.4.3a0 → matrix_python-1.4.5a0}/matrix/protocols.py

@@ -6,4 +6,4 @@ from matrix.room import Room
 class BotLike(Protocol):
     prefix: str | None
 
-    def get_room(self, room_id: str) -> Room: ...
+    def get_room(self, room_id: str) -> Room | None: ...

{matrix_python-1.4.3a0 → matrix_python-1.4.5a0}/matrix_python.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: matrix-python
-Version: 1.4.3a0
+Version: 1.4.5a0
 Summary: An easy-to-use Matrix bot framework designed for quick development and minimal setup
 Author: Simon Roy,  Chris Dedman Rollet 
 Maintainer-email: Code Society Lab <admin@codesociety.xyz>
@@ -684,18 +684,18 @@ Project-URL: Source, https://github.com/Code-Society-Lab/matrixpy
 Project-URL: Issues, https://github.com/Code-Society-Lab/matrixpy/issues
 Requires-Python: >=3.10
 Description-Content-Type: text/markdown
-Requires-Dist: matrix-nio
+Requires-Dist: matrix-nio==0.25.2
 Requires-Dist: logger
-Requires-Dist: PyYAML
-Requires-Dist: markdown
-Requires-Dist: APScheduler
+Requires-Dist: PyYAML==6.0.3
+Requires-Dist: markdown==3.10.2
+Requires-Dist: APScheduler==3.11.2
 Provides-Extra: dev
-Requires-Dist: pytest; extra == "dev"
-Requires-Dist: pytest-asyncio; extra == "dev"
-Requires-Dist: black; extra == "dev"
-Requires-Dist: mypy; extra == "dev"
-Requires-Dist: types-PyYAML; extra == "dev"
-Requires-Dist: types-Markdown; extra == "dev"
+Requires-Dist: pytest==9.0.3; extra == "dev"
+Requires-Dist: pytest-asyncio==1.3.0; extra == "dev"
+Requires-Dist: black==26.3.1; extra == "dev"
+Requires-Dist: mypy==1.20.0; extra == "dev"
+Requires-Dist: types-PyYAML==6.0.12.20260408; extra == "dev"
+Requires-Dist: types-Markdown==3.10.2.20260408; extra == "dev"
 
 <div align="center">
   <em>A simple, developer-friendly library to create powerful <a href="https://matrix.org">Matrix</a> bots.</em>

matrix_python-1.4.5a0/matrix_python.egg-info/requires.txt

@@ -0,0 +1,13 @@
+matrix-nio==0.25.2
+logger
+PyYAML==6.0.3
+markdown==3.10.2
+APScheduler==3.11.2
+
+[dev]
+pytest==9.0.3
+pytest-asyncio==1.3.0
+black==26.3.1
+mypy==1.20.0
+types-PyYAML==6.0.12.20260408
+types-Markdown==3.10.2.20260408

{matrix_python-1.4.3a0 → matrix_python-1.4.5a0}/pyproject.toml

@@ -5,33 +5,33 @@ requires = ["setuptools>=64", "setuptools_scm>=8"]
 name = "matrix-python"
 dynamic = ["version"]
 authors = [
-    { name="Simon Roy" },
-    { name=" Chris Dedman Rollet " }
+    { name = "Simon Roy" },
+    { name = " Chris Dedman Rollet " }
 ]
 maintainers = [
-    { name="Code Society Lab", email="admin@codesociety.xyz" }
+    { name = "Code Society Lab", email = "admin@codesociety.xyz" }
 ]
 description = "An easy-to-use Matrix bot framework designed for quick development and minimal setup"
 readme = "README.md"
-license = { file="LICENSE" }
+license = { file = "LICENSE" }
 requires-python = ">=3.10"
 
 dependencies = [
-    "matrix-nio",
+    "matrix-nio==0.25.2",
     "logger",
-    'PyYAML',
-    'markdown',
-    'APScheduler',
+    "PyYAML==6.0.3",
+    "markdown==3.10.2",
+    "APScheduler==3.11.2",
 ]
 
 [project.optional-dependencies]
 dev = [
-    "pytest",
-    "pytest-asyncio",
-    "black",
-    "mypy",
-    "types-PyYAML",
-    'types-Markdown',
+    "pytest==9.0.3",
+    "pytest-asyncio==1.3.0",
+    "black==26.3.1",
+    "mypy==1.20.0",
+    "types-PyYAML==6.0.12.20260408",
+    "types-Markdown==3.10.2.20260408",
 ]
 
 [project.urls]

{matrix_python-1.4.3a0 → matrix_python-1.4.5a0}/tests/test_bot.py

@@ -313,6 +313,30 @@ async def test_command_executes(bot):
     assert called, "Expected command handler to be called"
 
 
+@pytest.mark.asyncio
+async def test_command_not_processed_without_prefix(bot, room):
+    called = False
+
+    @bot.command()
+    async def greet(ctx):
+        nonlocal called
+        called = True
+
+    event = RoomMessageText.from_dict(
+        {
+            "content": {"body": "greet", "msgtype": "m.text"},
+            "event_id": "$id",
+            "origin_server_ts": 123456,
+            "sender": "@user:matrix.org",
+            "type": "m.room.message",
+        }
+    )
+
+    await bot._process_commands(room, event)
+
+    assert not called
+
+
 @pytest.mark.asyncio
 async def test_error_decorator_requires_coroutine(bot):
     with pytest.raises(TypeError):
@@ -376,18 +400,37 @@ def test_command_duplicate_raises(bot):
             pass
 
 
+import asyncio
+
+
+async def start_and_stop(coro):
+    task = asyncio.create_task(coro)
+    await asyncio.sleep(0)  # allow startup
+    task.cancel()
+    await asyncio.gather(task, return_exceptions=True)
+
+
 @pytest.mark.asyncio
 async def test_run_uses_token():
     bot = Bot()
     bot._load_config("tests/config_fixture_token.yaml")
 
     bot._client.sync_forever = AsyncMock()
-    bot.on_ready = AsyncMock()
+    bot._on_ready = AsyncMock()
+
+    # unblock readiness
+    bot._synced.set()
+
+    task = asyncio.create_task(bot.run())
+
+    await asyncio.sleep(0)
+    await asyncio.sleep(0)
 
-    await bot.run()
+    task.cancel()
+    await asyncio.gather(task, return_exceptions=True)
 
     assert bot._client.access_token == "abc123"
-    bot.on_ready.assert_awaited_once()
+    bot._on_ready.assert_awaited_once()
     bot._client.sync_forever.assert_awaited_once()
 
 
@@ -397,7 +440,15 @@ async def test_run_with_username_and_password(bot):
     bot._client.sync_forever = AsyncMock()
     bot._on_ready = AsyncMock()
 
-    await bot.run()
+    bot._synced.set()
+
+    task = asyncio.create_task(bot.run())
+
+    await asyncio.sleep(0)
+    await asyncio.sleep(0)
+
+    task.cancel()
+    await asyncio.gather(task, return_exceptions=True)
 
     bot._client.login.assert_awaited_once_with("grace1234")
     bot._on_ready.assert_awaited_once()
@@ -418,6 +469,39 @@ def test_start_handles_keyboard_interrupt(caplog):
     bot._client.close.assert_awaited_once()
 
 
+@pytest.mark.asyncio
+async def test_on_ready_called_only_once(bot):
+    # Prepare
+    bot._synced.set()
+    bot._on_ready = AsyncMock()
+
+    # Simulate run
+    await bot._wait_until_synced()
+    await bot._on_ready()
+
+    bot._on_ready.assert_awaited_once()
+
+
+@pytest.mark.asyncio
+async def test_scheduler_starts_after_ready(bot):
+    bot._synced.set()
+
+    order = []
+
+    async def ready():
+        order.append("ready")
+
+    bot._on_ready = AsyncMock(side_effect=ready)
+    bot.scheduler.start = MagicMock(side_effect=lambda: order.append("scheduler"))
+
+    # Simulate run
+    await bot._wait_until_synced()
+    await bot._on_ready()
+    bot.scheduler.start()
+
+    assert order == ["ready", "scheduler"]
+
+
 @pytest.mark.asyncio
 async def test_scheduled_task_in_scheduler(bot):
     @bot.schedule("* * * * *")
@@ -743,3 +827,25 @@ def test_unload_extension_logs_unloading(bot: Bot, loaded_extension: Extension):
     bot.unload_extension(loaded_extension.name)
 
     bot.log.debug.assert_any_call("unloaded extension '%s'", loaded_extension.name)
+
+
+def test_unload_extension_removes_only_its_jobs(bot: Bot):
+    ext_a = Extension(name="a")
+    ext_b = Extension(name="b")
+
+    @ext_a.schedule("* * * * *")
+    async def task():
+        pass
+
+    @ext_b.schedule("* * * * *")
+    async def task():
+        pass
+
+    bot.load_extension(ext_a)
+    bot.load_extension(ext_b)
+
+    bot.unload_extension("a")
+
+    job_names = [j.name for j in bot.scheduler.jobs]
+
+    assert "task" in job_names

matrix_python-1.4.3a0/matrix_python.egg-info/requires.txt

@@ -1,13 +0,0 @@
-matrix-nio
-logger
-PyYAML
-markdown
-APScheduler
-
-[dev]
-pytest
-pytest-asyncio
-black
-mypy
-types-PyYAML
-types-Markdown