7.77.0 — corbeille réparée (FK favorites/recents) + MCP workspaces
- detach_page_references : nettoyage des références NO ACTION avant toute suppression définitive (unitaire, Empty Trash, purge 30 j) - MCP : list_workspaces / get_workspace ; descriptions collections clarifiées (bases de données, pas des workspaces) - 96 tests verts sur le périmètre (MCP, trash, cohérence)
This commit is contained in:
@@ -51,6 +51,26 @@ EFFECTIVE_WORKSPACE_SQL = (
|
||||
)
|
||||
|
||||
|
||||
def detach_page_references(conn, page_ids) -> None:
|
||||
"""Supprime les lignes qui référencent des pages en ``NO ACTION``.
|
||||
|
||||
``favorites.page_id`` et ``recents.page_id`` référencent
|
||||
``pages(id)`` sans CASCADE et la connexion applique
|
||||
``PRAGMA foreign_keys=ON`` : supprimer définitivement une page
|
||||
réelle — ouverte un jour (recents) ou mise en favori — levait un
|
||||
``IntegrityError`` (500 côté API, échec silencieux de la purge
|
||||
automatique des 30 jours). À appeler avant tout
|
||||
``DELETE FROM pages`` définitif. (Régression du 2026-10-22 :
|
||||
les pages synthétiques des tests n'avaient ni favori ni récent.)
|
||||
"""
|
||||
ids = [int(i) for i in page_ids]
|
||||
if not ids:
|
||||
return
|
||||
ph = ",".join("?" * len(ids))
|
||||
conn.execute(f"DELETE FROM favorites WHERE page_id IN ({ph})", ids)
|
||||
conn.execute(f"DELETE FROM recents WHERE page_id IN ({ph})", ids)
|
||||
|
||||
|
||||
def effective_workspace_sql(alias: str = "c") -> str:
|
||||
"""Workspace effectif d'une collection (colonne ou ``NULL`` si orpheline)."""
|
||||
return EFFECTIVE_WORKSPACE_SQL.format(a=alias)
|
||||
|
||||
@@ -16,6 +16,7 @@ from app.db import get_conn
|
||||
from app.services.collection_lifecycle import (
|
||||
collections_hosted_by_page,
|
||||
delete_collections,
|
||||
detach_page_references,
|
||||
)
|
||||
|
||||
logger = logging.getLogger(__name__)
|
||||
@@ -71,6 +72,7 @@ def purge_expired(days: int = 30) -> dict:
|
||||
"WHERE parent_id=?",
|
||||
(page_id, page_id),
|
||||
)
|
||||
detach_page_references(conn, [page_id])
|
||||
conn.execute("DELETE FROM pages WHERE id=?", (page_id,))
|
||||
delete_collections(conn, collections)
|
||||
purged.append(page_id)
|
||||
|
||||
Reference in New Issue
Block a user