matrx-orm 3.1.128__tar.gz → 3.1.129__tar.gz

{matrx_orm-3.1.128 → matrx_orm-3.1.129}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: matrx-orm
-Version: 3.1.128
+Version: 3.1.129
 Summary: Async-first PostgreSQL ORM with bidirectional migrations, schema introspection, many-to-many relationships, and built-in state caching
 Project-URL: Homepage, https://github.com/AI-Matrix-Engine/aidream-current
 Project-URL: Repository, https://github.com/AI-Matrix-Engine/aidream-current

{matrx_orm-3.1.128 → matrx_orm-3.1.129}/matrx_orm/core/FEATURE.md

@@ -36,6 +36,11 @@ Pinned by `tests/level1/test_registry.py`.
 
 ## Database execution errors
 
+A SELECT whose checkout disconnects retries once on a fresh checkout, including
+when asyncpg has detached the pool proxy and its `is_closed()` accessor itself
+raises. Unrelated interface errors and ambiguous writes are never replayed.
+Guard: `tests/level1/test_read_connection_retry.py`.
+
 **A read-only-transaction write refusal retries on a fresh checkout.** PostgreSQL
 rejects the statement before execution, so this narrow failure is replay-safe;
 ambiguous write failures remain single-attempt.

{matrx_orm-3.1.128 → matrx_orm-3.1.129}/matrx_orm/core/async_db_manager.py

@@ -1349,9 +1349,17 @@ class AsyncDatabaseManager:
                         # replaced by Transaction.__aexit__'s InterfaceError.
                         # Retry only when asyncpg confirms the checkout itself
                         # is closed; other InterfaceErrors remain diagnostic.
-                        connection_closed = bool(
-                            getattr(conn, "is_closed", lambda: False)()
-                        )
+                        try:
+                            connection_closed = bool(
+                                getattr(conn, "is_closed", lambda: False)()
+                            )
+                        except asyncpg.exceptions.InterfaceError as checkout_error:
+                            # asyncpg detaches a terminated pool proxy. Even its
+                            # is_closed() accessor then raises instead of returning
+                            # True; that secondary error must not hide the loss.
+                            if "released back to the pool" not in str(checkout_error):
+                                raise e
+                            connection_closed = True
                         if (
                             read_only
                             and connection_closed

{matrx_orm-3.1.128 → matrx_orm-3.1.129}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "matrx-orm"
-version = "3.1.128"
+version = "3.1.129"
 description = "Async-first PostgreSQL ORM with bidirectional migrations, schema introspection, many-to-many relationships, and built-in state caching"
 readme = "README.md"
 license = { text = "MIT" }

{matrx_orm-3.1.128 → matrx_orm-3.1.129}/tests/level1/test_read_connection_retry.py

@@ -42,7 +42,8 @@ async def test_read_retries_connection_loss_once_on_fresh_checkout(monkeypatch)
 
 
 @pytest.mark.asyncio
-async def test_read_retries_disconnect_masked_by_reproof_transaction(monkeypatch) -> None:
+@pytest.mark.parametrize("released", [False, True])
+async def test_read_retries_disconnect_masked_by_reproof_transaction(monkeypatch, released) -> None:
     checkout_count = 0
 
     class MaskingTransaction:
@@ -67,6 +68,10 @@ async def test_read_retries_disconnect_masked_by_reproof_transaction(monkeypatch
             return MaskingTransaction()
 
         def is_closed(self) -> bool:
+            if released:
+                raise asyncpg.exceptions.InterfaceError(
+                    "cannot call Connection.is_closed(): connection has been released back to the pool"
+                )
             return True
 
     class HealthyConnection:
@@ -92,3 +97,32 @@ async def test_read_retries_disconnect_masked_by_reproof_transaction(monkeypatch
 
     assert rows == [{"value": 1}]
     assert checkout_count == 2
+
+
+@pytest.mark.asyncio
+@pytest.mark.parametrize("query,released", [("UPDATE public.example SET value = 1", True), ("SELECT 1", False)])
+async def test_interface_failure_does_not_replay_writes_or_unrelated_errors(monkeypatch, query, released):
+    from matrx_orm.exceptions import ConnectionError as ORMConnectionError
+    checkout_count = 0
+
+    class Connection:
+        async def fetch(self, *_args, **_kwargs):
+            raise asyncpg.exceptions.InterfaceError("original operation failure")
+
+        def is_closed(self):
+            if released:
+                raise asyncpg.exceptions.InterfaceError("connection has been released back to the pool")
+            return False
+
+    @asynccontextmanager
+    async def checkout(_cls, _config_name, _timeout):
+        nonlocal checkout_count
+        checkout_count += 1
+        yield Connection()
+
+    monkeypatch.setattr(AsyncDatabaseManager, "get_connection", classmethod(checkout))
+    with pytest.raises(Exception) as caught:
+        await AsyncDatabaseManager.execute_query("test", query)
+    assert checkout_count == 1
+    assert isinstance(caught.value, ORMConnectionError) is released
+    assert "original operation failure" in str(caught.value)