@mostajs/recette 0.2.0 → 0.3.0

package/CHANGELOG.md

@@ -2,6 +2,49 @@
 
 **Auteur** : Dr Hamid MADANI <drmdh@msn.com>
 
+## 0.3.0 — 2026-10-23
+
+### Corrigé
+
+- **Une ligne, une issue : le DERNIER relevé fait foi.** Le formulaire enregistre à chaque
+  changement, comme un questionnaire ; un testeur qui hésitait entre `ko` et `ok`, ou qui écrivait
+  sa remarque en plusieurs fois, laissait plusieurs relevés pour la même ligne, et tous étaient
+  comptés. Une hésitation devenait un `ko` de plus au dossier, une remarque tapée en trois fois
+  trois `ok` — et le verdict proposé au certificateur s'en trouvait faussé. Les compteurs du
+  dossier et de `etat()` ne retiennent plus que le dernier relevé de chaque (feuillet, ligne).
+  **T-REC-15** le verrouille.
+
+### Ajouté
+
+- `releves.derniersDuFeuillet(feuilletId)` — le relevé qui fait foi pour chaque ligne, issue et
+  remarque comprises : ce qu'un formulaire réaffiche.
+- Le dossier figé porte `relevesRemplaces` : combien de relevés ont été remplacés. L'historique
+  reste en base, rien n'est effacé.
+
+> Le **contenu** d'un dossier figé change de forme (`releves` ne garde que les relevés qui font
+> foi) : son empreinte n'est pas comparable à celle d'un dossier figé en 0.2.x. Les dossiers déjà
+> figés ne sont pas touchés.
+
+## 0.2.1 — 2026-09-10
+
+Correction née d'une panne **en production**, invisible depuis les essais.
+
+### Corrigé
+
+- **Le contenu du dossier revenait en CHAÎNE** de plusieurs dialectes SQL (sqljs le premier) là
+  où un dépôt en mémoire rend l'objet. L'enveloppe exportée transportait donc une chaîne, le
+  destinataire recalculait l'empreinte sur autre chose, et **refusait tout dossier** — avec un
+  message parlant de falsification. Le moteur normalise désormais à la lecture : `dossiers.of()`
+  et l'export rendent toujours un objet.
+- **`dossiers.exporter()` vérifie l'empreinte avant de signer.** Un document dont l'empreinte ne
+  recouvre plus le contenu relu est refusé ici, en nommant la cause, plutôt que d'aller se faire
+  rejeter chez le destinataire qui, lui, n'aura que « signature ou empreinte invalide » à offrir.
+
+### Ajouté aux essais
+
+- **T-REC-14** monte un dépôt qui imite sqljs — écriture d'objet, lecture de texte. C'est le cas
+  qui manquait : quatorze essais verts et rien ne fonctionnait en ligne.
+
 ## 0.2.0 — 2026-09-10
 
 Le sceau savait fermer un pli ; il ne savait pas l'ouvrir. C'est ce qui manquait pour que QaTrax

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mostajs/recette",
-  "version": "0.2.0",
+  "version": "0.3.0",
   "description": "Validation & recette applicative : trame `mostajs-devtest/1` reçue signée du certificateur, points enrichis POUR LE CLIENT (l'import n'écrase jamais l'enrichissement), feuillets composés à l'image du manifeste PDF (ok/ko/er), anomalies déclarées puis qualifiées, dossier figé et EXPORTÉ SIGNÉ — aucun appel réseau —, certificat reçu et vérifié, feu d'exploitation DÉRIVÉ du certificat. Powers ValiTrax (valitrax.mostajs.dev).",
   "author": "Dr Hamid MADANI <drmdh@msn.com>",
   "license": "AGPL-3.0-or-later",

package/src/recette.js

@@ -31,6 +31,17 @@ const rang = (e) => PARCOURS.indexOf(e);
 const need = (v, msg) => { if (v === undefined || v === null || v === '') throw new Error(msg); return v; };
 const asDate = (d) => (d ? new Date(d) : null);
 
+/**
+ * Le CONTENU d'un dossier, quel que soit ce que le stockage a bien voulu en faire.
+ *
+ * Un champ `json` revient en OBJET d'un dépôt en mémoire, et en CHAÎNE de plusieurs dialectes
+ * SQL — sqljs le premier. Sans cette normalisation, l'enveloppe exportée transportait une
+ * chaîne là où le destinataire attend un objet, et l'empreinte recalculée à l'arrivée ne
+ * pouvait plus correspondre : tout dossier était refusé, pour une raison invisible depuis les
+ * essais, qui n'utilisent que des dépôts en mémoire.
+ */
+const contenuDe = (d) => (typeof d?.contenu === 'string' ? JSON.parse(d.contenu) : d?.contenu);
+
 export function createRecette({ repositories, ports = {}, clock = () => new Date(), numeroteur } = {}) {
   const repo = need(repositories, 'repositories requis');
   for (const c of ['campagnes', 'points', 'feuillets', 'releves', 'anomalies', 'dossiers', 'certificats']) {
@@ -250,6 +261,29 @@ export function createRecette({ repositories, ports = {}, clock = () => new Date
     },
   };
 
+  /**
+   * UNE LIGNE, UNE ISSUE : le DERNIER relevé de chaque (feuillet, ligne) fait foi.
+   *
+   * Un relevé ne s'efface pas, il se remplace : le formulaire enregistre à chaque changement, comme
+   * un questionnaire, et le testeur qui hésite entre `ko` et `ok`, ou qui écrit sa remarque en
+   * plusieurs fois, laisse plusieurs relevés pour la même ligne. Les compter tous faisait d'une
+   * hésitation un `ko` de plus au dossier, et d'une remarque tapée en trois fois trois `ok` :
+   * le verdict proposé au certificateur s'en trouvait faussé. L'historique reste en base ; le
+   * dossier dit combien de relevés ont été remplacés.
+   */
+  const instant = (r) => +new Date(r.createdAt || r.at || 0);
+  function derniers(rs) {
+    const parLigne = new Map();
+    rs.forEach((r, i) => {
+      const cle = `${r.feuilletId}\u0000${r.ref}`;
+      const avant = parLigne.get(cle);
+      // À instant égal, l'ordre de lecture départage : le plus tard lu est le plus récent.
+      if (!avant || instant(r) >= instant(avant.r)) parLigne.set(cle, { r, i });
+    });
+    return [...parLigne.values()].sort((a, b) => a.i - b.i).map((x) => x.r);
+  }
+  const compter = (rs) => ({ ok: rs.filter((r) => r.issue === 'ok').length, ko: rs.filter((r) => r.issue === 'ko').length, er: rs.filter((r) => r.issue === 'er').length });
+
   const releves = {
     /**
      * Consigner une observation.
@@ -274,6 +308,8 @@ export function createRecette({ repositories, ports = {}, clock = () => new Date
     },
     of: (campagneId) => repo.releves.find((r) => r.campagneId === campagneId),
     duFeuillet: (feuilletId) => repo.releves.find((r) => r.feuilletId === feuilletId),
+    /** Le relevé QUI FAIT FOI pour chaque ligne d'un feuillet : le dernier consigné (voir `derniers`). */
+    derniersDuFeuillet: async (feuilletId) => derniers(await repo.releves.find((r) => r.feuilletId === feuilletId)),
   };
 
   const anomalies = {
@@ -357,14 +393,16 @@ export function createRecette({ repositories, ports = {}, clock = () => new Date
       if (obstacles.length && !force) throw new Error(`figement refusé — ${obstacles.join(' · ')}`);
 
       const fs = await feuillets.of(campagneId);
-      const rs = await releves.of(campagneId);
+      const tous = await releves.of(campagneId);
+      const rs = derniers(tous);
       const as = await anomalies.of(campagneId);
       const contenu = {
         campagne: { no: c.no, releaseRef: c.releaseRef, projet: c.projet, perimetre: c.perimetre, instanceUrl: c.instanceUrl },
         feuillets: fs.map((f) => ({ role: f.role, langue: f.langue, state: f.state, planVersion: f.planVersion })),
         releves: rs.map((r) => ({ ref: r.ref, issue: r.issue, at: r.at, source: r.source, confiance: r.confiance, texte: r.texte })),
         anomalies: as.map((a) => ({ titre: a.titre, graviteProposee: a.graviteProposee, graviteQualifiee: a.graviteQualifiee, state: a.state, bugRef: a.bugRef, motifEcart: a.motifEcart })),
-        compteurs: { ok: rs.filter((r) => r.issue === 'ok').length, ko: rs.filter((r) => r.issue === 'ko').length, er: rs.filter((r) => r.issue === 'er').length },
+        compteurs: compter(rs),
+        relevesRemplaces: tous.length - rs.length,
         obstaclesAuFigement: obstacles,   // consignés même en `force` : un dossier dit ce qu'il vaut
       };
       const empreinte = createHash('sha256').update(JSON.stringify(contenu)).digest('hex');
@@ -392,12 +430,21 @@ export function createRecette({ repositories, ports = {}, clock = () => new Date
       const sceau = ports.sceau;
       if (!sceau?.signer) throw new Error("aucun sceau configuré : un export non signé ne prouve rien, il est refusé");
 
+      // ON VÉRIFIE AVANT DE SIGNER. Exporter un document dont l'empreinte ne recouvre plus le
+      // contenu, c'est fabriquer un refus chez le destinataire — et un refus qu'il mettra sur le
+      // compte d'une falsification. Mieux vaut refuser ici, en nommant la cause.
+      const contenu = contenuDe(d);
+      const verif = createHash('sha256').update(JSON.stringify(contenu)).digest('hex');
+      if (verif !== d.empreinte) {
+        throw new Error(`empreinte du dossier ${d.no} incohérente avec son contenu relu (${verif.slice(0, 12)}… ≠ ${String(d.empreinte).slice(0, 12)}…) : le stockage n'a pas rendu ce qui lui a été confié`);
+      }
+
       const enveloppe = {
         type: 'mostajs-recette-dossier/1',
         emisLe: clock().toISOString(),
         campagne: { no: c.no, releaseRef: c.releaseRef },
         dossier: { no: d.no, empreinte: d.empreinte, figeLe: d.figeLe },
-        contenu: d.contenu,
+        contenu,
       };
       const charge = JSON.stringify(enveloppe);
       const signature = await sceau.signer(charge);
@@ -416,7 +463,12 @@ export function createRecette({ repositories, ports = {}, clock = () => new Date
       const d = need(await dossiers.of(campagneId), 'aucun dossier pour cette campagne');
       return repo.dossiers.update(d.id, { depotRef: need(depotRef, 'depotRef requis') });
     },
-    of: async (campagneId) => (await repo.dossiers.find((x) => x.campagneId === campagneId))[0] || null,
+    of: async (campagneId) => {
+      const d = (await repo.dossiers.find((x) => x.campagneId === campagneId))[0] || null;
+      // On rend TOUJOURS un objet : l'appelant n'a pas à connaître le dialecte SQL sous-jacent
+      // pour relire le contenu du dossier qu'il vient de figer.
+      return d ? { ...d, contenu: contenuDe(d) } : null;
+    },
   };
 
   const certificats = {
@@ -474,12 +526,12 @@ export function createRecette({ repositories, ports = {}, clock = () => new Date
   async function etat(campagneId) {
     const c = await getCampagne(campagneId);
     const fs = await feuillets.of(campagneId);
-    const rs = await releves.of(campagneId);
+    const rs = derniers(await releves.of(campagneId));
     const as = await anomalies.of(campagneId);
     return {
       campagne: { no: c.no, state: c.state, releaseRef: c.releaseRef },
       feuillets: { remis: fs.length, rendus: fs.filter((f) => f.state !== 'remis').length },
-      releves: { ok: rs.filter((r) => r.issue === 'ok').length, ko: rs.filter((r) => r.issue === 'ko').length, er: rs.filter((r) => r.issue === 'er').length },
+      releves: compter(rs),
       anomalies: { declarees: as.filter((a) => a.state === 'declaree').length, bloquantes: as.filter((a) => a.graviteQualifiee === 'bloquante').length, total: as.length },
       obstacles: await obstaclesAuDepot(campagneId),
     };