Repository navigation
OctoGuard 2, livraison 3a : la colonne, la vue d'ensemble, mes règles, le bac à sable - #942
Conversation
…e, mes règles, le bac à sable L'interface d'OctoGuard 2, première partie, telle que validée sur la maquette. - La colonne de l'admin bascule sur le menu d'OctoGuard au lieu de s'y ajouter (même largeur ; « Administration » en tête pour revenir sans changer de page). - Vue d'ensemble : état en une phrase, chiffres de la semaine, « À regarder » (dont « passer au cran » d'un clic pour une règle observée), dernières actions racontées en phrases, raccourcis illustrés, santé du moteur. - Mes règles : catégories en tête (Mots et insultes, Liens et spam, Nouveaux comptes, Contenu illisible, Sur mesure), un bloc par catégorie, une carte par règle avec interrupteur, cran, phrase, chiffres de la semaine. Tableau « Comprendre les crans » (message, auteur, équipe). Panneau de réglages : seuils, endroits, rôles exemptés, derniers déclenchements, suppression. - Mots interdits en trois cases (mot entier, début de mot, n'importe où) + « Toujours autorisés », alerte quand un mot ratisse trop large, lots prêts à l'emploi, essai en direct. Copie fidèle du moteur (parité vérifiée sur 40 cas avec matchers2.ts), même syntaxe stockée : aucun changement du cœur. - Bac à sable branché sur le vrai moteur (POST /sandbox, sans effet de bord). Quatre crans affichés (Observer, Signaler, Avertir, Supprimer) : « Retenir » arrivera avec le lot du cœur qui le rend possible. Bienvenue, commandes, webhook, journal et sourdines gardent leur page actuelle jusqu'à la 3b. i18n : 289 clés og2.* en fr et en, dont 109 clés dynamiques vérifiées une à une.
- Réglages : n'envoyer que les champs réellement modifiés. Enregistrer sans rien changer envoyait tout le formulaire, et le serveur marquait alors une règle du profil de base « personnalisée » (elle ne recevait plus les mises à jour du profil). Sans changement, plus aucune requête. - Mots interdits : basculer l'interrupteur ou le cran de la carte effaçait la saisie en cours. L'éditeur ne se remet à zéro que si la version enregistrée change vraiment. - Essai en direct : vider le champ le remplissait aussitôt avec l'exemple. - Panneau : le focus clavier se pose dans le panneau à l'ouverture. Prouvé dans le navigateur, écritures interceptées (rien n'atteint le serveur) : 12/12 avec le correctif, et les mêmes tests échouent sur l'ancien code.
|
Passe de vérification avant fusion : quatre défauts trouvés à la relecture du code, corrigés dans 1037b4c.
Preuve : un test navigateur avec les écritures OctoGuard interceptées (rien n'atteint le serveur) passe 12/12 avec le correctif. Les mêmes tests échouent sur l'ancien code (7 échecs, aux endroits annoncés). Re-vérifié ensuite : |
Le serveur accepte les réglages d'une règle tels quels (RuleParams en passthrough). Le moteur se protège d'une valeur vide (retour au défaut), pas d'une valeur absurde : Inondation à 1 répétition, -1 lien, 1 % de capitales feraient prendre TOUS les messages. - Réglages : chaque seuil doit être un entier dans ses bornes, avec des planchers volontaires (Majuscules : 5 lettres, 50 %) ; sites bien formés, enregistrés en minuscules, au moins un. Sinon rien ne part et le panneau dit pourquoi. - Mots interdits : au-delà de 500 mots (ce que le moteur lit), Enregistrer est bloqué avec un message, au lieu d'ignorer la fin sans bruit. - Test des droits du menu (admin, gardien OctoGuard, modérateur, équipe sans droit de modération). Navigateur, écritures interceptées : 11/11 (seuils absurdes, site mal formé, échec serveur qui fait revenir cran et interrupteur, 501 mots). Sur l'ancien code, le panneau acceptait « 1 répétition » sans un mot.
|
Deuxième passe de vérification, corrigée dans f9b3e8d. Trouvé : le serveur accepte les réglages d'une règle tels quels ( Corrigé côté écran (le cœur reste intact) :
Vérifié aussi :
Batterie complète sur le code final :
Non couvert en navigateur : le bouton « Passer au cran » de la Vue d'ensemble, absent faute de règle observée ayant pris des messages cette semaine. Il envoie |
…/invite) La validation des sites ajoutée dans f9b3e8d n'acceptait que les noms de domaine nus. Or la règle « Invitations vers d'autres serveurs » du profil de base contient discord.com/invite et discordapp.com/invite : ses réglages ne pouvaient plus être enregistrés du tout (prouvé sur ce code-là). Même expression que le cœur (services/octoguard/params.ts, PR à part) : domaine, *.domaine, domaine/chemin. Un lien collé (https://discord.gg/) est ramené au site au lieu d'être refusé. Aide mise à jour (fr, en).
|
Troisième passe : le bouton « Passer au cran » est testé en vrai. Un relais local injecte des prises fictives dans /stats uniquement, et l'écriture est interceptée. Résultat : 5 suggestions affichées, un seul PUT au corps exactement {"dry_run":false}, et la suggestion disparaît. Défaut trouvé et corrigé (dd57ae8) : la validation des sites refusait discord.com/invite, donc la règle « Invitations » du profil de base ne pouvait plus être enregistrée, prouvé sur le code d'avant. Elle reprend maintenant l'expression du cœur (#943). Batterie : check 0 erreur, i18n 5/5, vitest 404/404, navigateur 31+12+11+2+3 sans échec, aucune erreur JS. |
Première partie de l'interface d'OctoGuard 2 (CDC
NODYX_OCTOGUARD_V2_CDC.md, E4c), d'après la maquette validée le 03/11. Frontend seul : aucun changement du cœur.Ce qui change
La colonne. En entrant dans OctoGuard, la colonne de l'admin affiche le menu d'OctoGuard à la place du sien (même largeur, aucune place perdue). « Administration » en tête la fait revenir sans quitter la page. Les anciens onglets disparaissent.
Vue d'ensemble (
/admin/octoguard) : l'état en une phrase, les chiffres de la semaine (pris, auraient été pris, annulés, signalements), « À regarder » (dont « Passer au cran » d'un clic pour une règle observée qui a pris des messages), les dernières actions racontées en phrases, des raccourcis illustrés, la santé du moteur.Mes règles (
/admin/octoguard/rules) :Mots interdits : trois cases (mot entier, début de mot, n'importe où) et « Toujours autorisés », avec un exemple concret par case, une alerte quand un mot ratisse trop large (« débil » prendrait « débilitant ») corrigeable d'un clic, des lots prêts à l'emploi et un essai en direct qui surligne le texte tel que tapé. La syntaxe stockée ne change pas (
mot,mot*,*mot*).Bac à sable (
/admin/octoguard/sandbox) : branché sur le vrai moteur (POST /admin/octoguard/sandbox, sans effet de bord), avec la raison quand une règle ne s'applique pas (endroit, rôle exempté, compte non visé).Volontairement pas dans cette PR
Vérifications
matchers2.ts. Parité vérifiée sur 40 cas en appelant les deux (déguisements, mots épelés avec points ou espaces, exceptions, HTML, traits d'union).keywords.test.ts,rules.test.ts) ; suite frontend 400/400.npm run check: 0 erreur, aucun avertissement dans les fichiers touchés.og2.*en fr et en ; les 109 clés construites dynamiquement vérifiées une à une.