discovery-media-player 0.1.91 → 0.1.93

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "discovery-media-player",
-  "version": "0.1.91",
+  "version": "0.1.93",
   "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/schema.js

@@ -303,7 +303,43 @@ async function vraimentSonderTout() {
   for (const [cle] of quandNegatif) { connues.delete(cle); reponses.delete(cle); }
   quandNegatif.clear();
   for (const nom of Object.keys(ATTENDUES)) await attendue(nom);
-  return etatDuSchema();
+  const etat = etatDuSchema();
+  await ajouterSansRang(etat);
+  return etat;
+}
+
+/**
+ * ⚠️ `mod_seq` NUL EST AMBIGU, ET UN NOMBRE AMBIGU NE MESURE RIEN. Le rattrapage de 0016 laisse
+ * DÉLIBÉRÉMENT sans rang les messages des présentations scellées (le scellé refuse de les écrire). Mais
+ * rien ne distingue « nul par décision » de « nul parce que le rattrapage n'a jamais tourné » : les
+ * deux se lisent « ligne sans rang ». On rend donc les DEUX nombres — total, et la part qui est
+ * légitimement scellée. Égaux ⇒ tout nul est un scellé attendu ; divergents ⇒ le rattrapage a laissé
+ * des lignes NON scellées derrière lui, anomalie à corriger. C'est la classe « une garde qu'on n'a pas
+ * vue refuser ne garde rien » appliquée à un compteur : le compteur devient un verdict. (relevé 2e hôte)
+ *
+ * Seulement si la colonne est là (sinon la question n'a pas de sens), et JAMAIS bloquant : un
+ * diagnostic ne doit pas casser la carte qu'il enrichit — mieux vaut pas de compteur qu'une carte muette.
+ */
+async function ajouterSansRang(etat) {
+  const r = reponses.get("doc_presentation_messages.mod_seq");
+  if (!r || !r.present) return;
+  try {
+    // ⚠️ `selectAll`, JAMAIS un `limit` — ET C'ÉTAIT LE DÉFAUT DE LA PREMIÈRE VERSION (relevé 2e hôte).
+    // Un compteur borné qui ne dit pas qu'il est borné redevient le nombre ambigu qu'il remplaçait, et
+    // une liste tronquée a EXACTEMENT la forme d'une liste complète. Les deux bornes mentaient en sens
+    // OPPOSÉS : une liste de nuls coupée (l'ordre physique groupe par présentation) pouvait ne voir que
+    // des scellées → total = dontScellees → faux « tout va bien » sur une base où des non-scellées ont
+    // été oubliées ; et la liste de scellées, sans `limit`, tombait sous le plafond IMPLICITE de
+    // PostgREST (db-max-rows) → setScellees incomplet → dontScellees sous-compté → fausse ALERTE
+    // permanente sur un hôte à >1000 archives. `selectAll` pagine jusqu'au bout : le transport reste
+    // borné à la taille RÉELLE du problème (zéro sur un hôte sain), et le compteur ne peut plus mentir
+    // par troncature. `is.null`/`eq` restent de la syntaxe portable.
+    const scellees = await PLAYER.db.selectAll("doc_presentations?active=eq.false&control_hash=is.null&select=slug");
+    const setScellees = new Set((Array.isArray(scellees) ? scellees : []).map((p) => p.slug));
+    const nuls = await PLAYER.db.selectAll("doc_presentation_messages?mod_seq=is.null&select=slug");
+    const liste = Array.isArray(nuls) ? nuls : [];
+    etat.sansRang = { total: liste.length, dontScellees: liste.filter((m) => setScellees.has(m.slug)).length };
+  } catch { /* best-effort : un compteur absent vaut mieux qu'une carte en échec */ }
 }
 
 /** La réponse, retenue pour qui la demandera — sans repasser par la base. */