tina4-python 3.13.123__tar.gz → 3.13.125__tar.gz

{tina4_python-3.13.123 → tina4_python-3.13.125}/PKG-INFO

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

{tina4_python-3.13.123 → tina4_python-3.13.125}/pyproject.toml

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

{tina4_python-3.13.123 → tina4_python-3.13.125}/tina4_python/__init__.py

@@ -51,7 +51,7 @@ def _resolve_version() -> str:
     #
     # test_version_constant.py now asserts this literal equals the pyproject
     # version, so the release bump cannot leave it behind again.
-    return "3.13.123"
+    return "3.13.125"
 
 
 __version__ = _resolve_version()

{tina4_python-3.13.123 → tina4_python-3.13.125}/tina4_python/cli/__init__.py

@@ -31,8 +31,8 @@ from pathlib import Path
 
 # ── Field type mapping ────────────────────────────────────────────────
 FIELD_TYPE_MAP = {
-    "string":   {"orm": "StringField",   "sql": "TEXT",    "default": "''"},
-    "str":      {"orm": "StringField",   "sql": "TEXT",    "default": "''"},
+    "string":   {"orm": "StringField",   "sql": "VARCHAR(255)", "default": "''"},
+    "str":      {"orm": "StringField",   "sql": "VARCHAR(255)", "default": "''"},
     "int":      {"orm": "IntegerField",  "sql": "INTEGER", "default": "0"},
     "integer":  {"orm": "IntegerField",  "sql": "INTEGER", "default": "0"},
     "float":    {"orm": "NumericField",  "sql": "REAL",    "default": "0"},
@@ -41,7 +41,7 @@ FIELD_TYPE_MAP = {
     "bool":     {"orm": "BooleanField",  "sql": "INTEGER", "default": "0"},
     "boolean":  {"orm": "BooleanField",  "sql": "INTEGER", "default": "0"},
     "text":     {"orm": "TextField",     "sql": "TEXT",    "default": "''"},
-    "datetime": {"orm": "DateTimeField", "sql": "TEXT",    "default": "NULL"},
+    "datetime": {"orm": "DateTimeField", "sql": "TIMESTAMP", "default": "NULL"},
     "blob":     {"orm": "BlobField",     "sql": "BLOB",    "default": "NULL"},
 }
 
@@ -1941,7 +1941,7 @@ def _gen_migration(name: str, flags: dict = None, *,
             info = FIELD_TYPE_MAP.get(ftype, FIELD_TYPE_MAP["string"])
             default = f" DEFAULT {info['default']}" if info["default"] != "NULL" else ""
             col_lines.append(f"    {fname} {info['sql']}{default}")
-        col_lines.append("    created_at TEXT DEFAULT CURRENT_TIMESTAMP")
+        col_lines.append("    created_at TIMESTAMP DEFAULT CURRENT_TIMESTAMP")
 
         # ADR-0063: `tina4:edit` marker between the last column and the closing
         # paren, placed OUTSIDE the comma-joined column list so the SQL stays

{tina4_python-3.13.123 → tina4_python-3.13.125}/tina4_python/database/firebird.py

@@ -646,6 +646,7 @@ class FirebirdAdapter(SqlCrudMixin, DatabaseAdapter):
         sql = SQLTranslator.ilike_to_like(sql)
         sql = SQLTranslator.boolean_to_int(sql)
         sql = SQLTranslator.auto_increment_syntax(sql, "firebird")
+        sql = SQLTranslator.ddl_types(sql, "firebird")
         return sql
 
     def _supports_returning(self) -> bool:

{tina4_python-3.13.123 → tina4_python-3.13.125}/tina4_python/database/mssql.py

@@ -350,6 +350,7 @@ class MSSQLAdapter(SqlCrudMixin, DatabaseAdapter):
         sql = SQLTranslator.ilike_to_like(sql)
         sql = SQLTranslator.auto_increment_syntax(sql, "mssql")
         sql = SQLTranslator.boolean_to_int(sql)
+        sql = SQLTranslator.ddl_types(sql, "mssql")
         return sql
 
     def _supports_returning(self) -> bool:

{tina4_python-3.13.123 → tina4_python-3.13.125}/tina4_python/database/mysql.py

@@ -321,6 +321,7 @@ class MySQLAdapter(SqlCrudMixin, DatabaseAdapter):
         sql = SQLTranslator.concat_pipes_to_func(sql)
         sql = SQLTranslator.ilike_to_like(sql)
         sql = SQLTranslator.auto_increment_syntax(sql, "mysql")
+        sql = SQLTranslator.ddl_types(sql, "mysql")
         return sql
 
     def _supports_returning(self) -> bool:

{tina4_python-3.13.123 → tina4_python-3.13.125}/tina4_python/database/sql_translator.py

@@ -347,6 +347,49 @@ class SQLTranslator:
             return re.sub(r"\s*AUTOINCREMENT\b", "", sql, flags=re.IGNORECASE)
         return sql
 
+    @staticmethod
+    def ddl_types(sql: str, engine: str) -> str:
+        """Translate SQLite-canonical DDL column TYPES + CREATE-TABLE options to
+        the target engine.
+
+        ONLY acts on ``CREATE TABLE`` / ``ALTER TABLE`` statements, so a query or
+        INSERT that happens to contain the word ``TEXT`` (a column name, a string
+        literal) is never rewritten. Complements ``auto_increment_syntax`` (which
+        maps the id keyword) so ONE portable migration — and every
+        ``ORM.create_table()`` DDL, which is also SQLite-canonical — applies on
+        every engine instead of failing on Firebird/MSSQL.
+
+        * Firebird has no ``TEXT`` (-607), no ``REAL``, and no
+          ``CREATE TABLE IF NOT EXISTS``.
+        * MSSQL has no ``CREATE TABLE IF NOT EXISTS`` and its ``TIMESTAMP`` is a
+          rowversion, not a datetime — a ``created_at TIMESTAMP`` there is wrong.
+        * MySQL's ``TIMESTAMP`` carries auto-update / 2038 surprises, so a
+          datetime column maps to ``DATETIME`` (matching ORM.create_table()).
+        """
+        # Gate to DDL only, tolerating leading `-- ...` comment lines / blank
+        # lines that a migration file carries before its CREATE TABLE. A SELECT
+        # or INSERT that merely mentions a type keyword is never rewritten.
+        head = re.sub(r"^(?:\s*--[^\n]*\n)+", "", sql)
+        if not re.match(r"\s*(CREATE\s+TABLE|ALTER\s+TABLE)\b", head, re.IGNORECASE):
+            return sql
+        e = (engine or "").lower()
+        if e == "firebird":
+            sql = re.sub(r"\bIF\s+NOT\s+EXISTS\b", "", sql, flags=re.IGNORECASE)
+            # Map bare TEXT -> BLOB SUB_TYPE TEXT, but leave an existing
+            # "BLOB SUB_TYPE TEXT" intact (it already contains the word TEXT).
+            sql = re.sub(
+                r"\bBLOB\s+SUB_TYPE\s+TEXT\b", "\x00FBTEXT\x00", sql, flags=re.IGNORECASE
+            )
+            sql = re.sub(r"\bTEXT\b", "BLOB SUB_TYPE TEXT", sql, flags=re.IGNORECASE)
+            sql = sql.replace("\x00FBTEXT\x00", "BLOB SUB_TYPE TEXT")
+            sql = re.sub(r"\bREAL\b", "DOUBLE PRECISION", sql, flags=re.IGNORECASE)
+        elif e == "mssql":
+            sql = re.sub(r"\bIF\s+NOT\s+EXISTS\b", "", sql, flags=re.IGNORECASE)
+            sql = re.sub(r"\bTIMESTAMP\b", "DATETIME2", sql, flags=re.IGNORECASE)
+        elif e == "mysql":
+            sql = re.sub(r"\bTIMESTAMP\b", "DATETIME", sql, flags=re.IGNORECASE)
+        return sql
+
     @staticmethod
     def placeholder_style(sql: str, style: str = "?") -> str:
         """Convert ? placeholders to engine-specific style.

{tina4_python-3.13.123 → tina4_python-3.13.125}/tina4_python/session/__init__.py

@@ -174,18 +174,32 @@ class DatabaseSessionHandler(SessionHandler):
         # The table is now created on first use, inside that policy.
         self._table_ready = False
 
+    # The portable statement that shipped for every engine. VARCHAR(255) carries
+    # TEXT affinity and DOUBLE PRECISION an 8-byte float on SQLite; it is DOUBLE
+    # PRECISION (never REAL - REAL is single-precision float4 on PostgreSQL, which
+    # test_session_expiry_contract pins) on the networked engines. Correct on
+    # sqlite/postgres/mysql/mssql, which is why it was never per-engine before.
+    _CREATE_TABLE_DEFAULT = "CREATE TABLE tina4_session (session_id VARCHAR(255) PRIMARY KEY, data TEXT NOT NULL, expires_at DOUBLE PRECISION NOT NULL)"
+    # Firebird is the one engine that needs a different statement: it has NO TEXT
+    # type, so the payload must be a VARCHAR (ceiling 8191 characters on this
+    # engine alone), matching PHP/Ruby/Node. The prior single "data TEXT" CREATE
+    # failed on Firebird with -607 "source column TEXT does not exist", so the
+    # database session backend never worked there. table_exists() below covers
+    # Firebird's lack of IF NOT EXISTS.
+    _CREATE_TABLE_SQL = {
+        "firebird": "CREATE TABLE tina4_session (session_id VARCHAR(255) NOT NULL PRIMARY KEY, data VARCHAR(8191) NOT NULL, expires_at DOUBLE PRECISION NOT NULL)",
+    }
+
     def _ensure_table(self):
         if self._table_ready:
             return
         self._table_ready = True
         if not self._db.table_exists("tina4_session"):
-            self._db.execute("""
-                CREATE TABLE tina4_session (
-                    session_id VARCHAR(255) PRIMARY KEY,
-                    data TEXT NOT NULL,
-                    expires_at DOUBLE PRECISION NOT NULL
-                )
-            """)
+            try:
+                engine = (self._db.get_database_type() or "").lower()
+            except Exception:
+                engine = ""
+            self._db.execute(self._CREATE_TABLE_SQL.get(engine, self._CREATE_TABLE_DEFAULT))
             self._db.commit()
 
     def read(self, session_id: str) -> dict: