rocketdoo 3.2.1__tar.gz → 3.3.0__tar.gz

{rocketdoo-3.2.1 → rocketdoo-3.3.0}/LICENSE

@@ -57,7 +57,7 @@ If the Library as you received it specifies that a proxy can decide whether futu
 <div RKD as ROCKETDOO V3=""></div>
 
 Licencia: LGPL-3.0+
-Versión: "3.2.1"
+Versión: "3.3.0"
 Autor: Horacio Montaño
-Fecha: 13/09/2026
+Fecha: 14/09/2026
 Descripción: Framework to development Odoo

{rocketdoo-3.2.1/rocketdoo.egg-info → rocketdoo-3.3.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: rocketdoo
-Version: 3.2.1
+Version: 3.3.0
 Summary: Framework for creating Odoo development environments with Docker and custom templates.
 Author-email: Horacio Montaño <horaciomontano@hdmsoft.com.ar>
 License: GNU LESSER GENERAL PUBLIC LICENSE
@@ -62,9 +62,9 @@ License: GNU LESSER GENERAL PUBLIC LICENSE
         <div RKD as ROCKETDOO V3=""></div>
         
         Licencia: LGPL-3.0+
-        Versión: "3.2.1"
+        Versión: "3.3.0"
         Autor: Horacio Montaño
-        Fecha: 13/09/2026
+        Fecha: 14/09/2026
         Descripción: Framework to development Odoo
 Requires-Python: >=3.10
 Description-Content-Type: text/markdown
@@ -97,7 +97,7 @@ Odoo Development Framework
    - "Horacio Montaño"
 
 ## Version: 
-   - "3.2.1"
+   - "3.3.0"
 
 ----------------------------------------------------------------------------------------------------------------------------------------------------------
 
@@ -472,6 +472,10 @@ rkd mail open    # Open http://localhost:8025 in your browser
 
 Mailpit is toggled by commenting/uncommenting its service block in `docker-compose.yaml` using special markers (`# rkd:mailpit`). When enabled, Odoo's SMTP configuration is automatically updated to route emails through Mailpit.
 
+`rkd mail on`/`off` also write a `Mailpit (rkd)` record straight into the project's `ir.mail_server` table (Odoo's technical settings, *Outgoing Mail Servers*), so emails are captured with no extra manual setup. `on` creates or reactivates that record with `sequence = 1`; `off` archives it (never deletes it), leaving any of your own mail servers untouched. With a single database this is automatic; with more than one, pass `--db NAME` to `on`, `off`, or `status` — with several databases and no `--db`, nothing is written or read and the command tells you to re-run with it. If the `db` container isn't reachable (project stopped, Docker not running), the rest of the command still runs and it reports the reason; re-running `rkd mail on` once the project is up fixes it.
+
+`rkd mail status` shows this record's state (active, archived, not created yet, or not checked, with the reason) plus any other active `ir.mail_server` in the database. **A `sequence = 1` on the Mailpit record does not guarantee it captures every email**: Odoo's server selection filters by `from_filter` before it sorts by sequence, so a server you (or a restored production dump) already have, with a `from_filter` matching the sender, can still win even with a higher sequence number. `mail status` warns loudly when another active server has `sequence <= 1` (it can win the sequence tie-break outright), and with a softer note whenever any other server is active at all, since `from_filter` can override sequence regardless.
+
 ---
 
 #### `rkd traefik` — Traefik Reverse Proxy

{rocketdoo-3.2.1 → rocketdoo-3.3.0}/README.md

@@ -15,7 +15,7 @@ Odoo Development Framework
    - "Horacio Montaño"
 
 ## Version: 
-   - "3.2.1"
+   - "3.3.0"
 
 ----------------------------------------------------------------------------------------------------------------------------------------------------------
 
@@ -390,6 +390,10 @@ rkd mail open    # Open http://localhost:8025 in your browser
 
 Mailpit is toggled by commenting/uncommenting its service block in `docker-compose.yaml` using special markers (`# rkd:mailpit`). When enabled, Odoo's SMTP configuration is automatically updated to route emails through Mailpit.
 
+`rkd mail on`/`off` also write a `Mailpit (rkd)` record straight into the project's `ir.mail_server` table (Odoo's technical settings, *Outgoing Mail Servers*), so emails are captured with no extra manual setup. `on` creates or reactivates that record with `sequence = 1`; `off` archives it (never deletes it), leaving any of your own mail servers untouched. With a single database this is automatic; with more than one, pass `--db NAME` to `on`, `off`, or `status` — with several databases and no `--db`, nothing is written or read and the command tells you to re-run with it. If the `db` container isn't reachable (project stopped, Docker not running), the rest of the command still runs and it reports the reason; re-running `rkd mail on` once the project is up fixes it.
+
+`rkd mail status` shows this record's state (active, archived, not created yet, or not checked, with the reason) plus any other active `ir.mail_server` in the database. **A `sequence = 1` on the Mailpit record does not guarantee it captures every email**: Odoo's server selection filters by `from_filter` before it sorts by sequence, so a server you (or a restored production dump) already have, with a `from_filter` matching the sender, can still win even with a higher sequence number. `mail status` warns loudly when another active server has `sequence <= 1` (it can win the sequence tie-break outright), and with a softer note whenever any other server is active at all, since `from_filter` can override sequence regardless.
+
 ---
 
 #### `rkd traefik` — Traefik Reverse Proxy

{rocketdoo-3.2.1 → rocketdoo-3.3.0}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "rocketdoo"
-version = "3.2.1"
+version = "3.3.0"
 description = "Framework for creating Odoo development environments with Docker and custom templates."
 authors = [
     { name = "Horacio Montaño", email = "horaciomontano@hdmsoft.com.ar" }

rocketdoo-3.3.0/rocketdoo/core/odoo_db.py

@@ -0,0 +1,191 @@
+"""Access to the project's PostgreSQL container: reads and Mailpit writes.
+
+The `pg_database` query and the container lookup used to live only in
+`pack_environment`; the GUI module-update feature needs the same data, so
+this consolidates them into a single place (like `core/compose.py` did for
+`docker compose` in #137) instead of growing a third copy. `rkd mail on/off`
+also writes the Rocketdoo `ir.mail_server` row here (#176), so this module is
+no longer read-only.
+"""
+
+import subprocess
+
+from rocketdoo.project_info import read_docker_compose
+
+PSQL_USER = "root"  # POSTGRES_USER set by the docker-compose template
+
+MAILPIT_SERVER_NAME = "Mailpit (rkd)"
+MAILPIT_SMTP_HOST = "mailpit"
+MAILPIT_SMTP_PORT = 1025
+MAILPIT_SEQUENCE = 1  # Odoo's default is 10: Mailpit must win the tie
+
+
+def db_container(compose_data: dict | None = None) -> str | None:
+    """Return the `db` service's container name, or None if it can't be read."""
+    if compose_data is None:
+        compose_data = read_docker_compose()
+    try:
+        return compose_data["services"]["db"]["container_name"]
+    except (KeyError, TypeError):
+        return None
+
+
+def _psql(db: str, sql: str, *flags: str, compose_data: dict | None = None) -> tuple[str, str]:
+    """Run `sql` against `db` via `docker exec ... psql`. Returns (stdout, error).
+
+    A non-empty error means the query did not run: stdout is not meaningful.
+    Shared by every query/write in this module so the argv and the error
+    translation (missing docker, unreachable container, psql failure) live in
+    one place.
+    """
+    container = db_container(compose_data)
+    if not container:
+        return "", "no database container configured"
+
+    try:
+        result = subprocess.run(
+            ["docker", "exec", container, "psql", "-U", PSQL_USER, "-d", db, *flags, "-c", sql],
+            capture_output=True,
+            text=True,
+            timeout=15,
+        )
+    except FileNotFoundError:
+        return "", "docker not found"
+    except (OSError, subprocess.SubprocessError) as exc:
+        return "", str(exc)
+
+    if result.returncode != 0:
+        return "", result.stderr.strip()
+
+    return result.stdout, ""
+
+
+def databases_result(compose_data: dict | None = None) -> tuple[list[str], str]:
+    """List of user databases plus the reason it came back empty, if any.
+
+    Excludes `postgres` and the templates, matching what `rkd pack` already
+    queried. Any failure to reach the container yields an empty list.
+    """
+    stdout, error = _psql(
+        "postgres",
+        "SELECT datname FROM pg_database WHERE datistemplate = false AND datname != 'postgres';",
+        "-t",
+        compose_data=compose_data,
+    )
+    if error:
+        return [], error
+
+    return [line.strip() for line in stdout.splitlines() if line.strip()], ""
+
+
+def list_databases(compose_data: dict | None = None) -> list[str]:
+    """List of user databases. Use `databases_result` when the reason matters."""
+    return databases_result(compose_data)[0]
+
+
+def module_states(db: str) -> tuple[dict[str, str], str]:
+    """Map `{module_name: state}` read from `ir_module_module` in `db`.
+
+    `db` is not validated here: that is the trust boundary's job (the GUI
+    layer), and keeping it in one place avoids the validation being
+    duplicated with different criteria. If `db` isn't an Odoo database,
+    psql fails and ({}, stderr) is returned.
+    """
+    stdout, error = _psql(db, "SELECT name, state FROM ir_module_module;", "-t", "-A", "-F", "\t")
+    if error:
+        return {}, error
+
+    states = {}
+    for line in stdout.splitlines():
+        name, _, state = line.strip().partition("\t")
+        if name and state:
+            states[name] = state
+    return states, ""
+
+
+def mail_servers(db: str) -> tuple[list[dict], str]:
+    """Every ir.mail_server row in `db`, ordered the way Odoo orders them.
+
+    All rows are returned, active or not, so `rkd mail status` can tell
+    "archived" from "never created" with a single query.
+    """
+    stdout, error = _psql(
+        db,
+        "SELECT id, name, sequence, smtp_host, active FROM ir_mail_server ORDER BY sequence, id;",
+        "-t",
+        "-A",
+        "-F",
+        "\t",
+    )
+    if error:
+        return [], error
+
+    servers = []
+    for line in stdout.splitlines():
+        fields = line.split("\t")
+        if len(fields) != 5:
+            continue
+        raw_id, name, raw_sequence, smtp_host, raw_active = fields
+        try:
+            server_id = int(raw_id)
+            sequence = int(raw_sequence)
+        except ValueError:
+            continue
+        servers.append(
+            {
+                "id": server_id,
+                "name": name,
+                "sequence": sequence,
+                "smtp_host": smtp_host,
+                "active": raw_active == "t",
+            }
+        )
+    return servers, ""
+
+
+def enable_mailpit_server(db: str) -> str:
+    """Create or reactivate the Rocketdoo mail server. Returns '' on success.
+
+    Matched by name + smtp_host, not by id: there is no UNIQUE constraint on
+    name to key an UPSERT off. Idempotent by construction, so callers can run
+    it unconditionally instead of tracking local state.
+    """
+    sql = (
+        "WITH updated AS ("
+        "UPDATE ir_mail_server "
+        f"SET active = true, smtp_port = {MAILPIT_SMTP_PORT}, sequence = {MAILPIT_SEQUENCE}, write_date = now() "
+        f"WHERE name = '{MAILPIT_SERVER_NAME}' AND smtp_host = '{MAILPIT_SMTP_HOST}' "
+        "RETURNING id"
+        ") "
+        "INSERT INTO ir_mail_server "
+        "(name, smtp_host, smtp_port, smtp_encryption, smtp_authentication, sequence, active, create_date, write_date) "
+        f"SELECT '{MAILPIT_SERVER_NAME}', '{MAILPIT_SMTP_HOST}', {MAILPIT_SMTP_PORT}, 'none', 'login', "
+        f"{MAILPIT_SEQUENCE}, true, now(), now() "
+        "WHERE NOT EXISTS (SELECT 1 FROM updated);"
+    )
+    _, error = _psql(db, sql)
+    return error
+
+
+def disable_mailpit_server(db: str) -> tuple[int, str]:
+    """Archive the Rocketdoo mail server. Returns (rows archived, error).
+
+    Only rows matching both name and smtp_host are touched, and only
+    archived: never DELETE, so a record a user renamed or repurposed is
+    never at risk even if it collides on one of the two conditions.
+    """
+    sql = (
+        "UPDATE ir_mail_server SET active = false, write_date = now() "
+        f"WHERE name = '{MAILPIT_SERVER_NAME}' AND smtp_host = '{MAILPIT_SMTP_HOST}' "
+        "RETURNING id;"
+    )
+    # -q suppresses psql's status line: without it, `RETURNING id` with zero
+    # rows still prints "UPDATE 0", which counts as a row and makes the command
+    # report an archive that never happened.
+    stdout, error = _psql(db, sql, "-q", "-t", "-A")
+    if error:
+        return 0, error
+
+    # Count returned ids, not non-empty lines: psql's status line ("UPDATE 0")
+    # is not a row, and counting it reports an archive that never happened.
+    return len([line for line in stdout.splitlines() if line.strip().isdigit()]), ""

{rocketdoo-3.2.1 → rocketdoo-3.3.0}/rocketdoo/gui/server.py

@@ -176,13 +176,20 @@ def create_app(host: str = "127.0.0.1", port: int = DEFAULT_PORT) -> FastAPI:
             except Exception:
                 pass
 
+    # The SPA is one file with a fixed name, so without this the browser keeps
+    # serving its stored copy across upgrades — a user on a new version saw the
+    # previous interface until a forced refresh. "no-cache" does not disable
+    # caching, it requires revalidation: with the etag already sent, an
+    # unchanged file still answers 304.
+    _SPA_HEADERS = {"Cache-Control": "no-cache"}
+
     @app.get("/", response_class=FileResponse)
     async def root():
-        return FileResponse(STATIC_DIR / "index.html")
+        return FileResponse(STATIC_DIR / "index.html", headers=_SPA_HEADERS)
 
     @app.get("/health")
     async def health():
-        return {"status": "ok", "version": "3.0.0"}
+        return {"status": "ok", "version": __version__}
 
     @app.get("/{path:path}", response_class=FileResponse)
     async def spa_fallback(path: str):
@@ -190,7 +197,7 @@ def create_app(host: str = "127.0.0.1", port: int = DEFAULT_PORT) -> FastAPI:
             return JSONResponse({"error": "Not found"}, status_code=404)
         index = STATIC_DIR / "index.html"
         if index.exists():
-            return FileResponse(index)
+            return FileResponse(index, headers=_SPA_HEADERS)
         return JSONResponse({"error": "GUI not found"}, status_code=404)
 
     return app