discovery-media-player 0.1.168 → 0.1.169

package/package.json

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

@@ -91,6 +91,15 @@ function creerCache(options) {
   // sans eux, « le cache tient » est une phrase, pas une mesure. Compteurs de processus, comme tout
   // ici : jamais remis à zéro, à lire en deltas.
   let nServies = 0, nRegroupees = 0, nProduites = 0, picEnVol = 0;
+  // ⚠️ LES OBSERVATEURS DE FENÊTRE, ET POURQUOI ILS NE TOUCHENT PAS À L'ÉTAT DU CACHE. `picEnVol`
+  // est un maximum DEPUIS LE DÉMARRAGE : quand le rapport de charge le recopiait tel quel à côté de
+  // hits/misses/coalesced convertis en DELTAS, une position calme héritait du pic d'une position
+  // chargée et affirmait une concurrence qu'elle n'avait jamais vue. Remettre `picEnVol` à zéro
+  // entre deux positions serait pire : on modifierait le SYSTÈME MESURÉ pour arranger l'instrument,
+  // et le pic historique — qui a son utilité — disparaîtrait. Un observateur est donc un compteur
+  // SÉPARÉ, ouvert et refermé par qui mesure, sans que le cache change de comportement.
+  // Relevé par un audit externe (CODEX, 15/09).
+  const observateurs = new Set();
 
   const oublier = (k) => {
     const e = entrees.get(k);
@@ -154,6 +163,7 @@ function creerCache(options) {
       nEnVol += 1;
       nProduites += 1;
       if (nEnVol > picEnVol) picEnVol = nEnVol;
+      for (const o of observateurs) if (nEnVol > o.pic) o.pic = nEnVol;
       // ⚠️ L'ÉCHÉANCE PART DE LA RÉSOLUTION, PAS DE LA DEMANDE. Posée à la demande, elle expirait
       // AVANT que la production ne réponde dès que celle-ci dépassait le TTL — et le regroupement ne
       // servait alors plus à rien pour exactement les producteurs lents, les seuls qu'il valait la
@@ -210,6 +220,17 @@ function creerCache(options) {
      * plus grand nombre de productions simultanées. Les refus sont dans `satures()`.
      */
     compteurs: () => ({ hits: nServies, coalesced: nRegroupees, misses: nProduites, peakInFlight: picEnVol }),
+    /**
+     * Ouvre une fenêtre d'observation du pic en vol : `{ pic(), fermer() }`. Le cache n'est pas
+     * modifié — ni remis à zéro, ni ralenti ; seul un compteur parallèle suit les productions
+     * simultanées TANT QUE la fenêtre est ouverte. `fermer()` est impératif : un observateur oublié
+     * survit au relevé qui l'a demandé.
+     */
+    observerEnVol: () => {
+      const o = { pic: nEnVol };
+      observateurs.add(o);
+      return { pic: () => o.pic, fermer: () => observateurs.delete(o) };
+    },
     /**
      * Ce que le plafond a refusé depuis le démarrage de ce processus.
      *

package/server/handler.js

@@ -1249,6 +1249,19 @@ module.exports = { __contexte: () => PLAYER, handler, init, TIERS, POLITIQUE_PER
   // ⚠️ COUTURE DE BANC, PAS D'API : le cache de lecture est global au module, et un banc qui laisse des
   // lectures en vol contamine le suivant (128 promesses éternelles, 503 partout — trouvé par un audit
   // externe sous mélange, graine 20260913). Un banc doit pouvoir VÉRIFIER qu'il rend le cache vide.
-  __cacheLecture: cacheLecture };
+  __cacheLecture: cacheLecture,
+  // ⚠️ COUTURE DE MESURE : LIRE LES COMPTEURS SANS LES INCRÉMENTER. Le rapport de charge relevait
+  // ces grandeurs par un `GET ?contract=1` — c'est-à-dire par une requête qui TRAVERSE le handler et
+  // incrémente `mesures.statuts.ok` au passage. Le delta d'une fenêtre de 1 000 requêtes valait donc
+  // 1 001, systématiquement, et pour chaque position : l'observateur se comptait lui-même. Une
+  // soustraction cachée aurait corrigé le chiffre en aggravant le problème — un instrument qui se
+  // retranche discrètement est plus difficile à auditer qu'un instrument faux. La couture rend l'état
+  // tel qu'il est, sans le modifier ; ce que la fenêtre contient d'autre que la charge se DIT, dans
+  // `counters.observerOverheadRequests`. Défaut relevé par un audit externe (CODEX, 15/09).
+  __compteursSansObserver: () => ({
+    lectureSaturee: { total: cacheLecture.satures().total },
+    relaisRefuses: { total: relaisRefusesTotal },
+    mesures: mesures.relever(),
+  }) };
 
 // redeploy: forcer le build production (Vercel a sauté la prod du merge #463 — wording re-partage).