pyarchinit-mini 2.4.1__tar.gz → 2.4.2__tar.gz

{pyarchinit_mini-2.4.1 → pyarchinit_mini-2.4.2}/CHANGELOG.md

@@ -1,3 +1,20 @@
+## [2.4.2] - 2026-05-18
+
+### Fixed (IT)
+- `_get_session()` (Spec 3-bis endpoints) ora acquisisce la sessione DB in modo
+  lazy via `get_db_session()` se nessun before_request hook ha pre-impostato
+  `g.db_session`. Risolve l'errore HTTP 500 "g.db_session not set" che
+  bloccava load / save / + Row / Export yEd su Adarte in 2.4.0-2.4.1.
+- Aggiunto `teardown_request` sul blueprint che rilascia la sessione lazy
+  alla fine della request.
+
+### Fixed (EN)
+- `_get_session()` now lazily opens a DB session via `get_db_session()` when no
+  before_request hook has bound one to `g.db_session`. Fixes the HTTP 500
+  "g.db_session not set" that broke load / save / + Row / Export on production
+  in 2.4.0-2.4.1.
+- Added `teardown_request` hook releasing the lazy session at request end.
+
 ## [2.4.1] - 2026-05-18
 
 ### Fixed (IT)

{pyarchinit_mini-2.4.1/pyarchinit_mini.egg-info → pyarchinit_mini-2.4.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pyarchinit-mini
-Version: 2.4.1
+Version: 2.4.2
 Summary: Lightweight archaeological data management system with REST API, Web UI, Desktop GUI, CLI, multi-user authentication, real-time collaboration, analytics dashboard, PyArchInit import/export, Heriverse/ATON export integration, and 3D CRUD viewer
 Home-page: https://github.com/enzococca/pyarchinit-mini
 Author: PyArchInit Team

{pyarchinit_mini-2.4.1 → pyarchinit_mini-2.4.2}/pyarchinit_mini/__init__.py

@@ -23,7 +23,7 @@ Features:
 - Scalable and modular architecture
 """
 
-__version__ = "2.4.1"
+__version__ = "2.4.2"
 __author__ = "PyArchInit Team"
 __email__ = "enzo.ccc@gmail.com"
 

{pyarchinit_mini-2.4.1 → pyarchinit_mini-2.4.2}/pyarchinit_mini/web_interface/harris_creator_routes.py

@@ -638,22 +638,42 @@ from pyarchinit_mini.harris_swimlane.exceptions import PeriodSyncError
 
 
 def _get_session():
-    """Get the request-bound SQLAlchemy session.
-
-    Requires the Flask app to set g.db_session in a before_request hook
-    (the production app does this; tests do too). Raises if not set —
-    fails loud rather than silently returning a context-manager generator
-    that downstream callers will treat as a Session.
+    """Get a request-scoped SQLAlchemy session for Spec 3-bis endpoints.
+
+    Reuses ``g.db_session`` if a before_request hook has set it. Otherwise
+    lazily opens one through the same path as ``get_db_session()`` and
+    binds it to ``g`` (the teardown_request hook below releases it). This
+    keeps the API contract simple — endpoints just call ``_get_session()``
+    and get a real ``Session`` back, whether or not the host app wires a
+    before_request hook (the production app in 2.4.x does not).
     """
     db = getattr(g, "db_session", None)
-    if db is None:
-        raise RuntimeError(
-            "g.db_session not set. The Flask app must set it in a "
-            "before_request hook before invoking Spec 3-bis endpoints."
-        )
+    if db is not None:
+        return db
+    cm = get_db_session()
+    db = cm.__enter__()
+    g.db_session = db
+    g._db_session_cm = cm
     return db
 
 
+@harris_creator_bp.teardown_request
+def _release_lazy_db_session(exc):  # noqa: ARG001 — exc is part of the signature
+    """Close any session that ``_get_session()`` opened lazily for this request."""
+    cm = getattr(g, "_db_session_cm", None)
+    if cm is None:
+        return
+    try:
+        cm.__exit__(type(exc) if exc else None, exc, getattr(exc, "__traceback__", None))
+    except Exception:
+        pass
+    finally:
+        try:
+            delattr(g, "_db_session_cm")
+        except AttributeError:
+            pass
+
+
 @harris_creator_bp.get("/api/swimlanes/<site>")
 def api_get_swimlanes(site: str):
     """List swimlane rows for the site."""

{pyarchinit_mini-2.4.1 → pyarchinit_mini-2.4.2/pyarchinit_mini.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pyarchinit-mini
-Version: 2.4.1
+Version: 2.4.2
 Summary: Lightweight archaeological data management system with REST API, Web UI, Desktop GUI, CLI, multi-user authentication, real-time collaboration, analytics dashboard, PyArchInit import/export, Heriverse/ATON export integration, and 3D CRUD viewer
 Home-page: https://github.com/enzococca/pyarchinit-mini
 Author: PyArchInit Team

{pyarchinit_mini-2.4.1 → pyarchinit_mini-2.4.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "pyarchinit-mini"
-version = "2.4.1"
+version = "2.4.2"
 description = "Lightweight archaeological data management system with REST API, Web UI, Desktop GUI, CLI, multi-user authentication, real-time collaboration, analytics dashboard, PyArchInit import/export, Heriverse/ATON export integration, and 3D CRUD viewer"
 readme = "README.md"
 license = {text = "GPL-2.0"}

pyarchinit_mini-2.4.2/tests/integration/test_harris_swimlane_routes_session_missing.py

@@ -0,0 +1,37 @@
+"""Verify _get_session() lazily acquires a session when g.db_session is not bound.
+
+Prior to 2.4.2, _get_session() raised RuntimeError when no before_request hook
+had set g.db_session — but no such hook exists in the production app, so every
+swimlane request 500'd. 2.4.2 lazy-acquires a session through the same path as
+get_db_session() and binds it to g for the duration of the request.
+"""
+import pytest
+from flask import Flask
+from sqlalchemy import create_engine
+
+from pyarchinit_mini.models.base import Base
+from pyarchinit_mini.web_interface.harris_creator_routes import harris_creator_bp
+
+
+@pytest.fixture
+def client_no_session(tmp_path, monkeypatch):
+    db_path = tmp_path / "swimlane_lazy.db"
+    db_url = f"sqlite:///{db_path}"
+    # Pre-create the schema so lazy session has tables to query.
+    Base.metadata.create_all(create_engine(db_url))
+    monkeypatch.setenv("DATABASE_URL", db_url)
+    app = Flask(__name__)
+    app.config["TESTING"] = True
+    app.register_blueprint(harris_creator_bp)
+    # Deliberately NO before_request hook setting g.db_session — we rely on
+    # _get_session() lazy acquisition.
+    yield app.test_client()
+
+
+def test_swimlanes_endpoint_lazy_acquires_session(client_no_session):
+    """Without a pre-bound g.db_session, the endpoint should still answer 200
+    by lazily opening a fresh session via get_db_session()."""
+    r = client_no_session.get("/harris-creator/api/swimlanes/EmptySite")
+    assert r.status_code == 200, r.get_data(as_text=True)
+    body = r.get_json()
+    assert isinstance(body, list)

pyarchinit_mini-2.4.1/tests/integration/test_harris_swimlane_routes_session_missing.py

@@ -1,23 +0,0 @@
-"""Verify _get_session() fails loud when g.db_session is not bound."""
-import pytest
-from flask import Flask
-
-from pyarchinit_mini.web_interface.harris_creator_routes import harris_creator_bp
-
-
-@pytest.fixture
-def client_no_session():
-    app = Flask(__name__)
-    app.config["TESTING"] = True
-    app.register_blueprint(harris_creator_bp)
-    # Deliberately NO before_request hook setting g.db_session
-    yield app.test_client()
-
-
-def test_swimlanes_endpoint_fails_loud_without_session(client_no_session):
-    r = client_no_session.get("/harris-creator/api/swimlanes/Volterra")
-    # Should return 500 with explicit error, NOT silent AttributeError
-    assert r.status_code == 500
-    body = r.get_json() or {}
-    # Either "internal" generic error catch or our explicit message
-    assert "error" in body