tina4-python 3.13.14__tar.gz → 3.13.15__tar.gz

{tina4_python-3.13.14 → tina4_python-3.13.15}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: tina4-python
-Version: 3.13.14
+Version: 3.13.15
 Summary: Tina4 Python v3 — Zero-dependency, lightweight web framework
 Author-email: Andre van Zuydam <andrevanzuydam@gmail.com>
 License: MIT

{tina4_python-3.13.14 → tina4_python-3.13.15}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "tina4-python"
-version = "3.13.14"
+version = "3.13.15"
 description = "Tina4 Python v3 — Zero-dependency, lightweight web framework"
 authors = [
     {name = "Andre van Zuydam", email = "andrevanzuydam@gmail.com"}

{tina4_python-3.13.14 → tina4_python-3.13.15}/tina4_python/__init__.py

@@ -8,7 +8,7 @@ Tina4 Python v3.0 — Zero-dependency, lightweight web framework.
 
 One import, everything works.
 """
-__version__ = "3.13.14"
+__version__ = "3.13.15"
 
 # ── Route decorators ──
 from tina4_python.core.router import (  # noqa: E402, F401

{tina4_python-3.13.14 → tina4_python-3.13.15}/tina4_python/database/postgres.py

@@ -207,6 +207,35 @@ class PostgreSQLAdapter(DatabaseAdapter):
         except Exception:
             pass
 
+    def _end_read_txn(self):
+        """v3.13.15 (issue #51): close the implicit transaction a read
+        opens, so the connection doesn't leak as ``idle in transaction``.
+
+        psycopg2 runs with ``autocommit = False`` (the framework's safe
+        default — writes must be committed explicitly). A bare ``SELECT``
+        therefore opens a transaction too, and with nothing to commit it
+        sits ``idle in transaction`` for the life of the connection,
+        holding a pool slot and any locks it touched. Short-lived boot
+        connections (the migration runner's ``MAX(batch)`` lookup) leak
+        one each, eventually exhausting ``max_connections`` — at which
+        point module autodiscovery fails mid-boot and every route 404s
+        while ``/health-check`` still passes ("ready but broken").
+
+        A read has nothing to persist, so a ROLLBACK is the safe close.
+        Inside an explicit ``start_transaction()`` the caller owns the
+        transaction, so we defer (mirrors the heal / error-path guards).
+        Writes go through :meth:`execute`, which is deliberately left
+        untouched — with autocommit off they must be committed explicitly,
+        so auto-closing them here would silently drop data.
+        """
+        if self._in_transaction or self._conn is None:
+            return
+        try:
+            self._conn.rollback()
+        except Exception:
+            # Never let bookkeeping mask the rows we already fetched.
+            pass
+
     def connect(self, connection_string: str, username: str = "", password: str = "", **kwargs):
         """Connect to PostgreSQL.
 
@@ -357,6 +386,10 @@ class PostgreSQLAdapter(DatabaseAdapter):
         self._exec_with_handling(cursor, paginated_sql, paginated_params)
         rows = [self._decode_blobs(dict(row)) for row in cursor.fetchall()]
 
+        # v3.13.15 (#51): rows are materialised above — close the implicit
+        # read transaction so the connection doesn't sit idle-in-transaction.
+        self._end_read_txn()
+
         return DatabaseResult(records=rows, count=total, limit=limit, offset=offset, sql=sql, adapter=self)
 
     def fetch_one(self, sql: str, params: list = None) -> dict | None:
@@ -368,7 +401,13 @@ class PostgreSQLAdapter(DatabaseAdapter):
         cursor = self._conn.cursor(cursor_factory=psycopg2.extras.RealDictCursor)
         self._exec_with_handling(cursor, sql, params)
         row = cursor.fetchone()
-        return self._decode_blobs(dict(row)) if row else None
+        result = self._decode_blobs(dict(row)) if row else None
+        # v3.13.15 (#51): close the implicit read transaction so the
+        # connection doesn't sit 'idle in transaction' (psycopg2 autocommit
+        # is off). This is the path the migration runner's MAX(batch)
+        # lookup leaked through.
+        self._end_read_txn()
+        return result
 
     @staticmethod
     def _decode_blobs(row: dict) -> dict: