discovery-media-player 0.1.89 → 0.1.91

package/package.json

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

@@ -179,7 +179,7 @@ var Live=(function(){
   // battait deux fois. On les déclare au niveau du CYCLE DE VIE : ce qui doit être arrêté doit
   // être visible depuis l'endroit qui arrête.
   var _ordEtat=null,_ordChat=null,_filet=null;
-  var sb=null,ch=null,ME=null,SLUG=null,CONTROL=null,LOCKED=false,AUTHTOK=null,PRESENT=[],PRESNAME='',seen=Object.create(null),msgEls=Object.create(null),msgData=Object.create(null),replyCtx=null,typers=Object.create(null),pdfCache=Object.create(null),_tyT=0,_tyIv=0,_atIv=0,unread=0,autoOpened=false,_histDone=false,_phWired=false,_onMap=null,_onState=null,_peekT=0,MUTED=false;
+  var sb=null,ch=null,ME=null,SLUG=null,CONTROL=null,LOCKED=false,AUTHTOK=null,PRESENT=[],PRESNAME='',seen=Object.create(null),msgEls=Object.create(null),msgData=Object.create(null),replyCtx=null,typers=Object.create(null),pdfCache=Object.create(null),_tyT=0,_tyIv=0,_atIv=0,unread=0,autoOpened=false,_histDone=false,_phWired=false,_onMap=null,_onState=null,_peekT=0,MUTED=false,_maxSeq=0;
   try{ MUTED=localStorage.getItem('3dd-present-mute')==='1'; }catch(e){}
   // Couper/rétablir les notifications du chat (cloche) : coupé = plus de ticker ni de pulse (badge silencieux gardé).
   function applyMute(){ var b=document.getElementById('chatMute'); if(b){b.classList.toggle('muted',MUTED);b.title=MUTED?'Réactiver les notifications du chat':'Couper les notifications du chat';} setBadge(); }
@@ -317,7 +317,7 @@ var Live=(function(){
   // Crochet interne (même famille que __presRelireEtat) : le piège de focus ne se PROUVE que
   // piloté clavier dans un vrai navigateur, et la seule voie UI passe par un message à soi.
   try{window.__confirmDialog=confirmDialog;}catch(e){}
-  function history(){fetch('/api/doc?present='+encodeURIComponent(SLUG)+'&chat=1').then(function(r){return r.json();}).then(function(d){var box=document.getElementById('chatMsgs');if(d&&d.messages&&d.messages.length){d.messages.forEach(function(m){addMsg(m);});}else if(box&&!box.children.length){box.innerHTML='<div class=chat-empty>Aucun message. Lancez la discussion.</div>';}if(d&&typeof d.locked!=='undefined')applyLock(d.locked);_histDone=true;}).catch(function(){_histDone=true;});}
+  function history(){fetch('/api/doc?present='+encodeURIComponent(SLUG)+'&chat=1').then(function(r){return r.json();}).then(function(d){var box=document.getElementById('chatMsgs');if(d&&d.messages&&d.messages.length){d.messages.forEach(function(m){if(typeof m.mod_seq==='number'&&m.mod_seq>_maxSeq)_maxSeq=m.mod_seq;addMsg(m);});}else if(box&&!box.children.length){box.innerHTML='<div class=chat-empty>Aucun message. Lancez la discussion.</div>';}if(d&&typeof d.locked!=='undefined')applyLock(d.locked);_histDone=true;}).catch(function(){_histDone=true;});}
   function react(id,e){if(!ME||!id||!e)return;
     // ⚠️ ON ENVOIE L'ÉTAT VOULU, PAS « INVERSE ». Basculer n'a de sens qu'une fois : un renvoi
     // réseau, un double-clic, une reprise de requête, et la réaction que le participant vient
@@ -497,7 +497,7 @@ var Live=(function(){
       // arrive en retard, jamais à côté.
       function relireAvec(url,applique){
         var ord=Player.live.createScheduler(function(fini){
-          Player.live.fetchBorne('/api/doc?present='+encodeURIComponent(SLUG)+url)
+          Player.live.fetchBorne('/api/doc?present='+encodeURIComponent(SLUG)+((typeof url==='function')?url():url))
             .then(function(r){return r.json();})
             .then(function(d){if(d&&d.ok)applique(d);})
             .catch(function(){})
@@ -523,13 +523,13 @@ var Live=(function(){
       window.__presRelireEtat=function(){_ordEtat.signaler();};
       function relireEtat(){_ordEtat.signaler();}
       function relireChat(){_ordChat.signaler();}
-      _ordChat=relireAvec('&chat=1',function(d){
+      _ordChat=relireAvec(function(){return '&chat=1'+(_maxSeq>0?('&chatAfter='+_maxSeq):'');},function(d){
           // ⚠️ NOTIFIER CE QUI VIENT D'ARRIVER, ET SEULEMENT ÇA. 'addMsg' rend faux pour un
           // message déjà connu — la relecture ramène tout l'historique, donc sans cette condition
           // la pastille de non-lus compterait chaque message à chaque relecture. Et sans l'appel,
           // elle ne compte plus rien : c'est la régression que le test d'un hôte a attrapée en
           // lisant le source de ce paquet, à travers la frontière de deux dépôts.
-          if(d.messages)d.messages.forEach(function(m){if(addMsg(m))notifyMsg(m);else updateMsg(m);});
+          if(d.messages)d.messages.forEach(function(m){if(typeof m.mod_seq==='number'&&m.mod_seq>_maxSeq)_maxSeq=m.mod_seq;if(addMsg(m))notifyMsg(m);else updateMsg(m);});
           if(typeof d.locked!=='undefined')applyLock(d.locked);
       });
 

package/server/handler.js

@@ -693,6 +693,13 @@ async function handler(req, res) {
       // Le cache englobe la LECTURE ET la sérialisation : sous rafale, une seule des deux se paie.
       // `file=1` ne figure pas dans cette table, et c'est délibéré : il relaie des octets avec des
       // en-têtes de plage propres à l'appelant — ce n'est pas une charge partagée.
+      //
+      // ⚠️ CURSEUR DU CHAT DIFFÉRENTIEL. `chatAfter` = le plus grand `seq` que le client a déjà vu ;
+      // borné à un entier positif (sinon une valeur libre exploserait les clés de cache et l'URL de la
+      // requête base). Il entre dans la CLÉ DE CACHE : deux clients au même curseur — le cas normal,
+      // toute une salle à jour du même message — partagent la même lecture ; seuls ceux qui ont pris
+      // du retard divergent. Absent → relecture complète (client ancien, ou hôte non migré).
+      const chatAfter = (Number.isFinite(+q.chatAfter) && +q.chatAfter > 0) ? String(Math.trunc(+q.chatAfter)) : null;
       const LECTURES_PARTAGEES = [
         { param: "state", produire: async (p) => ({ ok: true, state: {
           active: p.active !== false,
@@ -723,12 +730,12 @@ async function handler(req, res) {
           // Signalé par la seconde passe d'audit (P0-4) ; la fuite n'y était pas.
           presenter_name: String(p.presenter_name || "") || null,
         } }) },
-        { param: "chat", produire: async (p, slug) => ({ ok: true, messages: await listMessages(slug), locked: !!p.chat_locked }) },
+        { param: "chat", suffixeCle: () => (chatAfter != null ? `:a${chatAfter}` : ""), produire: async (p, slug) => ({ ok: true, messages: await listMessages(slug, { after: chatAfter }), locked: !!p.chat_locked }) },
       ];
       for (const lecture of LECTURES_PARTAGEES) {
         if (String(q[lecture.param] || "") !== "1") continue;
         const slugLu = String(q.present);
-        const corps = await cacheLecture.lire(`${lecture.param}:${slugLu}`, async () => {
+        const corps = await cacheLecture.lire(`${lecture.param}:${slugLu}${lecture.suffixeCle ? lecture.suffixeCle() : ""}`, async () => {
           const p = await getPresentation(slugLu);
           return p ? JSON.stringify(await lecture.produire(p, slugLu)) : null;
         });

package/server/presentations.js

@@ -411,6 +411,9 @@ const CHAMPS_PUBLICS = [
   "id", "author_name", "author_avatar", "is_presenter", "is_member",
   "body", "attachment", "reactions", "reply_to", "reply_name", "reply_text",
   "deleted", "edited", "created_at",
+  // Rang de changement (migration 0016) : le client garde le plus grand vu et le renvoie pour relire
+  // en différentiel. Absent chez un hôte non migré → le client reste en relecture complète.
+  "mod_seq",
 ];
 
 /**
@@ -581,12 +584,32 @@ async function addMessage(slug, { name, email, avatar, isPresenter, isMember, bo
  * même chose — un `select` étroit et une projection — dont une seule était appliquée ici. Une
  * garde qui dépend de ce qu'on n'a pas demandé cède au premier champ qu'on demande.
  */
-async function listMessages(slug) {
-  // ⚠️ LES 300 PLUS RÉCENTS, PAS LES 300 PLUS ANCIENS (P1 performance). Avec `asc&limit=300`, au
-  // 301e message les nouveaux ne revenaient JAMAIS à ceux qui relisaient — seul l'auteur le voyait,
-  // par la réponse du POST. On trie `desc` (id départage created_at à la milliseconde) puis on
-  // REND en ordre chronologique, comme l'affichage l'attend.
-  const rows = await PLAYER.db.request(`doc_presentation_messages?slug=eq.${enc(String(slug || ""))}&select=id,author_name,author_avatar,author_hash,is_presenter,is_member,body,attachment,reactions,reply_to,reply_name,reply_text,deleted,edited,created_at&order=created_at.desc,id.desc&limit=300`);
+const CHAMPS_MSG = "id,author_name,author_avatar,author_hash,is_presenter,is_member,body,attachment,reactions,reply_to,reply_name,reply_text,deleted,edited,created_at";
+
+async function listMessages(slug, { after } = {}) {
+  const s = enc(String(slug || ""));
+  // Le chat différentiel repose sur la colonne `seq` (migration 0016), bumpée à CHAQUE écriture. On la
+  // sonde : présente → différentiel possible ; absente (hôte non migré) → on sert les 300 derniers et
+  // on n'ajoute pas `seq` au select (PostgREST rejetterait la requête ENTIÈRE sur une colonne inconnue).
+  const seqDispo = await require("./schema").attendue("chatDifferentiel");
+
+  // ⚠️ DIFFÉRENTIEL : le client donne le plus grand `mod_seq` déjà vu ; on ne rend QUE ce qui a changé
+  // depuis — nouveaux messages ET anciens mutés (réaction, édition, suppression), le trigger ayant
+  // avancé leur `mod_seq`. En ordre de `mod_seq` : le client fusionne par id (addMsg/updateMsg), l'ordre
+  // d'affichage tenant à created_at qu'il porte déjà. `limit=300` borne un rattrapage après longue
+  // coupure — le curseur avance, le signal suivant prend la suite.
+  if (after != null && seqDispo) {
+    const curseur = Math.max(0, Math.trunc(Number(after)) || 0);
+    const rows = await PLAYER.db.request(`doc_presentation_messages?slug=eq.${s}&mod_seq=gt.${curseur}&select=${CHAMPS_MSG},mod_seq&order=mod_seq.asc&limit=300`);
+    return Array.isArray(rows) ? rows.map(messagePublic) : [];
+  }
+
+  // ⚠️ CHARGE INITIALE (ou hôte non migré) : LES 300 PLUS RÉCENTS, PAS LES 300 PLUS ANCIENS (P1). Avec
+  // `asc&limit=300`, au 301e message les nouveaux ne revenaient JAMAIS à ceux qui relisaient — seul
+  // l'auteur, par la réponse du POST. On trie `desc` (id départage created_at à la ms) puis on REND en
+  // ordre chronologique. On renvoie `mod_seq` quand il est là : c'est ce qui amorce le curseur du client.
+  const select = seqDispo ? `${CHAMPS_MSG},mod_seq` : CHAMPS_MSG;
+  const rows = await PLAYER.db.request(`doc_presentation_messages?slug=eq.${s}&select=${select}&order=created_at.desc,id.desc&limit=300`);
   return Array.isArray(rows) ? rows.slice().reverse().map(messagePublic) : [];
 }
 

package/server/schema.js

@@ -69,6 +69,14 @@ const ATTENDUES = {
     migration: "supabase/migrations/0015-presence-atomique.sql",
     fonction: "écrire la présence atomiquement et plafonner la création de faux participants anonymes",
   },
+  // Chat différentiel : la colonne `mod_seq` (bumpée par trigger) est ce que le code sonde AVANT de
+  // servir en différentiel. Absente → on sert les 300 derniers comme avant (cf. listMessages). Ici la
+  // colonne EST le vrai signal de dégradation (le code la sonde), pas seulement un témoin — cas franc.
+  chatDifferentiel: {
+    table: "doc_presentation_messages", colonne: "mod_seq",
+    migration: "supabase/migrations/0016-chat-differentiel.sql",
+    fonction: "relire le chat en différentiel (mod_seq > curseur) au lieu des 300 derniers à chaque signal",
+  },
 };
 
 /**

package/supabase/init.sql

@@ -206,7 +206,10 @@ create table if not exists public.doc_presentation_messages (
   client_key    text,
   -- Rang de la dernière écriture de réactions acceptée : deux réactions simultanées ne
   -- s'écrasent plus — l'écriture conditionnée à un rang dépassé ne modifie rien, le serveur rejoue.
-  reactions_seq bigint not null default 0
+  reactions_seq bigint not null default 0,
+  -- Rang GLOBAL bumpé à chaque écriture (insert/update) par le trigger dpm_bump_modseq : le client
+  -- relit `mod_seq > curseur` au lieu des 300 derniers à chaque signal. Cf. migration 0016.
+  mod_seq       bigint
 );
 create index if not exists dpm_slug_idx on public.doc_presentation_messages (slug, created_at);
 -- ⚠️ Portée sur (slug, client_key), pas sur la clé seule : deux présentations n'ont aucune raison
@@ -218,6 +221,37 @@ create unique index if not exists dpm_client_key_uniq
   on public.doc_presentation_messages (slug, client_key)
   where client_key is not null;
 
+-- Chat différentiel (migration 0016) : rang global monotone bumpé à chaque écriture par un trigger.
+-- ⚠️ RÈGLE (sixième audit) : la colonne est assurée par un ALTER AVANT son index — sur une base d'hier,
+-- `create table if not exists` saute le corps et `mod_seq` n'existerait qu'ici.
+alter table public.doc_presentation_messages
+  add column if not exists mod_seq bigint;
+create sequence if not exists public.doc_presentation_messages_modseq;
+create or replace function public.player_bump_message_seq()
+returns trigger
+language plpgsql
+as $$
+begin
+  new.mod_seq := nextval('public.doc_presentation_messages_modseq');
+  return new;
+end;
+$$;
+drop trigger if exists dpm_bump_modseq on public.doc_presentation_messages;
+create trigger dpm_bump_modseq
+  before insert or update on public.doc_presentation_messages
+  for each row execute function public.player_bump_message_seq();
+-- ⚠️ Le rattrapage EXCLUT les présentations scellées (0007/0010) : le trigger dpm_archive_scellee lève
+-- sur toute écriture dans une présentation gelée, et une seule de ses lignes ferait échouer l'install.
+update public.doc_presentation_messages m
+  set mod_seq = nextval('public.doc_presentation_messages_modseq')
+  where m.mod_seq is null
+    and not exists (
+      select 1 from public.doc_presentations p
+      where p.slug = m.slug and p.active = false and p.control_hash is null
+    );
+create index if not exists idx_dpm_slug_modseq
+  on public.doc_presentation_messages (slug, mod_seq);
+
 create table if not exists public.doc_presentation_attendees (
   slug         text not null,
   attendee_key text not null,

package/supabase/migrations/0016-chat-differentiel.sql

@@ -0,0 +1,67 @@
+-- LE CHAT NE SE RELIT PLUS EN ENTIER À CHAQUE SIGNAL — SEULEMENT CE QUI A CHANGÉ DEPUIS.
+--
+-- ⚠️ P « architecture scalable » (audit CODEX 5.6). À chaque resynchronisation, l'audience relisait
+-- les 300 messages les plus récents — pour une salle de N spectateurs qui se resynchronisent, c'est N
+-- fois 300 lignes, alors que presque rien n'a bougé entre deux signaux. Le client sait déjà fusionner
+-- par id (addMsg/updateMsg) ; il lui manque de quoi demander « donne-moi ce qui a changé depuis X ».
+--
+-- Un chat est MUTABLE : un message ancien reçoit une réaction, est édité, est supprimé. Un curseur sur
+-- `id` (croissant à l'insertion) raterait ces mutations. D'où `mod_seq` — un rang bumpé à CHAQUE
+-- écriture, insertion ET mise à jour — par un TRIGGER, depuis une séquence (atomique, aucune course).
+-- Le nom suit la convention IMAP CONDSTORE (MODSEQ), qui résout exactement ce problème : un compteur
+-- monotone par collection, avancé à chaque changement, pour une synchro différentielle. Le client
+-- garde le plus grand `mod_seq` vu et demande `mod_seq > curseur` : il récupère les nouveaux messages
+-- ET les anciens qui ont changé, et rien d'autre.
+--
+-- ⚠️ POURQUOI UN TRIGGER, alors que reactions_seq est bumpé par le CODE : reactions_seq est LOCAL à un
+-- message (le rang de SA dernière écriture de réactions). `mod_seq` est GLOBAL et doit croître à chaque
+-- écriture de N'IMPORTE quel chemin (envoi, édition, suppression, réaction) sans les toucher tous —
+-- un before-insert-or-update le fait en un endroit, et `nextval` est atomique là où un
+-- lire-max-puis-écrire ne l'est pas. Même famille que le trigger de scellé d'archive (0007).
+--
+-- ⚠️ Sans lui : le player sonde la colonne `mod_seq` (server/schema.js) ; absente, il sert les 300 plus
+--    récents comme avant, et le client retombe sur la relecture complète. Rien ne casse, la charge
+--    n'est simplement pas encore allégée. Additive et rejouable ; applicable pendant que le code tourne.
+
+alter table public.doc_presentation_messages
+  add column if not exists mod_seq bigint;
+
+-- La séquence qui rend `mod_seq` globalement monotone, quel que soit le chemin d'écriture.
+create sequence if not exists public.doc_presentation_messages_modseq;
+
+create or replace function public.player_bump_message_seq()
+returns trigger
+language plpgsql
+as $$
+begin
+  new.mod_seq := nextval('public.doc_presentation_messages_modseq');
+  return new;
+end;
+$$;
+
+-- ⚠️ AVANT insert OU update : toute écriture acceptée avance le curseur. Un spectateur en retard
+-- récupère ainsi une réaction posée sur un vieux message, pas seulement les messages neufs.
+drop trigger if exists dpm_bump_modseq on public.doc_presentation_messages;
+create trigger dpm_bump_modseq
+  before insert or update on public.doc_presentation_messages
+  for each row execute function public.player_bump_message_seq();
+
+-- Rattrapage des lignes déjà présentes : sans `mod_seq`, elles seraient invisibles au différentiel.
+-- ⚠️ IL RESPECTE LE SCELLÉ D'ARCHIVE (0007/0010). Cet update écrit CHAQUE ligne — or le trigger
+-- dpm_archive_scellee LÈVE sur toute écriture dans une présentation scellée (active=false ET
+-- control_hash IS NULL). Une seule ligne d'une présentation scellée ferait échouer la migration
+-- ENTIÈRE. On exclut donc ces lignes — une présentation scellée est GELÉE, y compris son rang : ses
+-- messages restent sans mod_seq (historiques, jamais resynchronisés en direct, servis par la charge
+-- complète). Deux gardes correctes — le scellé (0007) et ce rattrapage — dont la COMPOSITION
+-- bloquerait sans cette exclusion, et qu'aucun des deux fichiers ne pouvait prévoir seul.
+update public.doc_presentation_messages m
+  set mod_seq = nextval('public.doc_presentation_messages_modseq')
+  where m.mod_seq is null
+    and not exists (
+      select 1 from public.doc_presentations p
+      where p.slug = m.slug and p.active = false and p.control_hash is null
+    );
+
+-- La lecture différentielle `slug=eq.X&mod_seq=gt.N&order=mod_seq.asc` doit rester rapide.
+create index if not exists idx_dpm_slug_modseq
+  on public.doc_presentation_messages (slug, mod_seq);