discovery-media-player 0.1.108 → 0.1.110

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "discovery-media-player",
-  "version": "0.1.108",
+  "version": "0.1.110",
   "description": "Self-hosted document viewer: per-recipient tracked links, reading analytics, live presentation. The core knows nothing about the application hosting it — everything it borrows arrives through an injected context.",
   "keywords": [
     "pdf-viewer",

package/server/presentations.js

@@ -1134,4 +1134,4 @@ async function listPresentationsForDoc(docId, email, isAdmin, autoriseLarge) {
 module.exports = {
   reacteurDepuisJeton,
   purgerPerimees,
-  messagePublic, CHAMPS_PUBLICS, cheminPieceJointe, init, createPresentation, getPresentation, setPage, endPresentation, addMessage, listMessages, toggleReaction, editMessage, deleteMessage, setChatLock, createUploadUrl, reclaimPresentation, touchPresentation, listActivePresentations, handoverPresentation, endPresentationByOwner, recordAttendance, presentationStats, listPresentationsForDoc, switchPresentationDoc, setPresentationContent };
+  messagePublic, CHAMPS_PUBLICS, cheminPieceJointe, init, createPresentation, getPresentation, setPage, endPresentation, addMessage, listMessages, toggleReaction, editMessage, deleteMessage, setChatLock, createUploadUrl, reclaimPresentation, touchPresentation, listActivePresentations, handoverPresentation, endPresentationByOwner, recordAttendance, presentationStats, listPresentationsForDoc, switchPresentationDoc, setPresentationContent , STALE_MS};

package/server/schema.js

@@ -403,7 +403,20 @@ async function ajouterPresence(etat) {
     // lecteur, dans six mois, c'est nous. On adjoint donc le nombre qui désambiguïse, comme
     // `dontScellees` l'a fait pour `sansRang` : 0 actives + 0 avecJeton = repos ; N actives + 0
     // avecJeton = ANOMALIE. (Précision du second hôte.)
-    const actives = await PLAYER.db.request(`doc_presentations?active=eq.true&select=slug${bornee}`);
+    // ⚠️ « ACTIVE » N'EST PAS « VIVANTE », ET LE DÉNOMINATEUR A BESOIN DE LA SECONDE. Une présentation
+    // dont le présentateur a fermé l'onglet sans clôturer reste `active = true` : la compter ferait
+    // annoncer « N actives et 0 avecJeton = ANOMALIE » alors que personne n'est parti nulle part. La
+    // fiabilité de ce nombre dépendrait alors de la DISCIPLINE DE CLÔTURE — et le second hôte, dont la
+    // clôture a été impossible trois jours durant (0008), est bien placé pour le dire : ses sept
+    // présentations sont inactives mais gardent toutes leur control_hash, donc aucune n'a été close
+    // proprement. Un dénominateur qui dépend d'une discipline humaine produit des anomalies fantômes.
+    //
+    // On réutilise donc LE seuil qui définit déjà « vivante » ailleurs dans le code — `STALE_MS`, le
+    // même que celui du balayage des présentations orphelines — plutôt que d'inventer un second
+    // nombre qui divergerait. Le présentateur bat toutes les 30 s (`present-touch`) : une présentation
+    // sans battement depuis trois minutes est abandonnée, pas silencieuse. (Relevé du second hôte.)
+    const vivantDepuis = new Date(Date.now() - require("./presentations").STALE_MS).toISOString();
+    const actives = await PLAYER.db.request(`doc_presentations?active=eq.true&last_seen=gt.${encodeURIComponent(vivantDepuis)}&select=slug${bornee}`);
     const nActives = Array.isArray(actives) ? actives.length : 0;
     // ⚠️ `couvre` VOYAGE AVEC LES NOMBRES, ET C'EST LE POINT. Le commentaire ci-dessus protège celui
     // qui lit la SOURCE ; celui qui court un risque, lui, lit le JSON — dans six mois, dans un cockpit,
@@ -429,10 +442,20 @@ async function ajouterPresence(etat) {
       // `avecJeton` qui prend le relais comme SIGNE DE VIE — nul pendant une présentation en cours
       // veut alors dire que quelque chose est cassé.
       couvre: (PLAYER.config && PLAYER.config.presenceStrict)
+        // ⚠️ LA RÈGLE PORTE SUR LE CAS PRÉSENT, PAS SUR UN GABARIT. Une première version injectait
+        // `nActives` dans une phrase écrite pour N > 0 et rendait, à N = 0 : « 0 active et 0 avecJeton
+        // = ANOMALIE ; 0 active et 0 avecJeton = repos normal » — une contradiction dans la même ligne.
+        // Formellement construite, fausse dans le cas du moment : exactement le défaut que ce champ
+        // existe pour éviter. On BRANCHE donc sur l'état réel plutôt que d'interpoler un nombre dans un
+        // texte générique — et au repos on annonce ce que le nombre voudra dire quand ça tournera.
         ? "jauge de transition PÉRIMÉE — porte fermée : sansJeton est structurellement nul et ne mesure "
-          + "plus rien. Le signe de vie est avecJeton, À CROISER AVEC presentationsActives : "
-          + nActives + " active(s) et 0 avecJeton = ANOMALIE ; 0 active et 0 avecJeton = repos normal. "
-          + "avecJeton inclut les bootstraps auto-déclarés — pas une preuve d'identité."
+          + "plus rien. Le signe de vie est avecJeton, à croiser avec presentationsActives. "
+          + (nActives > 0
+            ? nActives + " présentation(s) en cours : avecJeton DOIT être non nul — " + nAvec + " observé"
+              + (nAvec > 0 ? ", donc les présences s'enregistrent." : " : ANOMALIE, les présences ne s'enregistrent plus.")
+            : "aucune présentation en cours : avecJeton nul est le repos NORMAL. Le jour où une "
+              + "présentation tournera, un avecJeton nul voudra dire que les présences ne s'enregistrent plus.")
+          + " avecJeton inclut les bootstraps auto-déclarés — pas une preuve d'identité."
         : "jauge-de-migration — sansJeton dit combien de clients ANCIENS battent encore (0 = on peut "
           + "fermer). Instrument de TRANSITION : après fermeture il vaudra 0 quoi qu'il arrive, et le "
           + "relais sera avecJeton croisé avec presentationsActives. "