Set-OPS-Public/scripts/couverture_gui.py
Daniel Allaire 03c628d5e1 GUI : le formulaire des bases est GENERE depuis le schema
Etape 3, sur un seul registre — `bases_donnees`, le plus simple et le seul ou
observe et editable coincidaient deja. Les cinq autres gardent leurs formulaires
ecrits a la main : on ne bascule pas six vues d un coup.

CE QUI DISPARAIT DU JAVASCRIPT

Huit `<label>` en dur, trois constructions de `<option>`, et la regle qui
choisissait la source du consommateur selon la portee. Cette derniere ne vivait
que dans le JS ; elle est desormais DECLAREE au schema (`x-source-selon`), donc
lisible et gardee.

Le formulaire rend exactement les memes huit champs qu avant — verifie en
EXECUTANT le moteur sous node avec le schema et des donnees reelles, pas
seulement en passant `node --check`.

LA BOUCLE EST FERMEE DES DEUX COTES

Le chemin de SAUVEGARDE enumerait lui aussi les sept champs en dur. Un champ
ajoute au registre serait apparu au formulaire genere et aurait disparu
SILENCIEUSEMENT a l enregistrement — le pire des deux mondes. Il derive
maintenant du schema, valeurs par defaut comprises (`default`).

LA SEPARATION FORME / COHERENCE, MONTREE

  portee=groupe + consommateur APPLICATION   -> REFUSE par valider_bases
  portee=application + consommateur app      -> ACCEPTE
  secret absent                              -> REFUSE

Le schema a rempli la FORME (les defauts `groupe` et `principale` se sont
poses), le validateur a attrape l INCOHERENCE. Aucune de ces trois regles ne
s exprime en JSON Schema, et vouloir l y mettre creerait la seconde source de
verite que ce depot refuse.

CHAMPS_ECRITS_PAR_GUI COMMENCE A DISPARAITRE

Renomme CHAMPS_ECRITS_A_LA_MAIN, et `bases_donnees` en est SORTIE : sa
couverture se derive du schema. P19 lit desormais `champs_ecrits_par_gui()`, qui
reunit les deux. Le jour ou la table sera vide, elle gardera un mecanisme au
lieu d une liste.

UNE GARDE A CORRIGER AU PASSAGE

`declaration_derive()` verifiait que chaque champ declare apparait dans le
SOURCE du GUI. Pour un registre genere il n y apparait plus — c est le but.
Elle aurait crie sur precisement le progres qu elle devait constater. Les
registres generes en sont exemptes : c est P61 qui tient la promesse pour eux.

make prouver : CONFORME, 60 OK, 0 echec, 1 saute.
P07 (node --check), P19 (couverture), P61 (schema) : verts.

Reste : les cinq autres vues, et le trou de la nomenclature — que le passage au
generateur fermera par construction, puisque le schema decrit deja `categorie`
et `service`.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Crgis8CxCWkAGFA1ecBz3q
2026-09-08 15:30:08 -04:00

161 lines
6.5 KiB
Python

#!/usr/bin/env python3
"""Le GUI sait-il editer tout ce que les plans reels contiennent ?
Principe 10 : « un operateur humain exploite Set-OPS sans IA — la doc, `make` et
le GUI suffisent ». Ce principe se dement en silence des qu'un registre accepte un
champ que l'interface ne sait pas ecrire : l'operateur doit editer le YAML a la
main, et personne ne s'en apercoit avant de buter dessus.
Ce script rend le trou VISIBLE et mesurable. Il croise :
- les champs REELLEMENT presents dans les plans (instance courante + tous les
modeles decouverts par scripts/modeles.py) ;
- `champs_ecrits_par_gui()` de scripts/inventory_gui.py, qui declare ce que les
fonctions de sauvegarde ecrivent.
Deux sens de verification :
1. champ dans un plan, absent du GUI -> TROU d'interface (signale) ;
2. champ declare au GUI, introuvable dans son source -> DERIVE de la
declaration (signale) — le tableau ment sur ce que le GUI fait.
Usage :
python3 scripts/couverture_gui.py lister
python3 scripts/couverture_gui.py verifier # rc=2 s'il reste un trou
python3 scripts/couverture_gui.py verifier --tolerer nomenclature
"""
from __future__ import annotations
import argparse
import os
import sys
from pathlib import Path
import yaml
from inventory_gui import REGISTRES_GENERES, champs_ecrits_par_gui
from modeles import decouvrir as decouvrir_modeles
from inventory_rules import instance_courante, instance_courante # noqa: E402
RACINE = Path(__file__).resolve().parents[1]
INSTANCE = instance_courante()
SOURCE_GUI = RACINE / "scripts" / "inventory_gui.py"
# (fichier du plan, cle racine, nom du registre) — les registres tables-de-tables.
REGISTRES = [
("serveurs.yml", "serveurs", "serveurs"),
("applications.yml", "applications", "applications"),
("bases-donnees.yml", "bases_donnees", "bases_donnees"),
("bases-donnees.yml", "serveurs_bd", "serveurs_bd"),
("domaines.yml", "domaines_publics", "domaines_publics"),
("nomenclature.yml", "fonctions", "nomenclature"),
]
def _plans() -> list[Path]:
"""Dossiers plan/ a inspecter : l'instance courante + tous les modeles."""
trouves = []
if (INSTANCE / "plan").is_dir():
trouves.append(INSTANCE / "plan")
trouves += [m / "plan" for m in decouvrir_modeles() if (m / "plan").is_dir()]
return trouves
def champs_utilises() -> dict[str, dict[str, set[str]]]:
"""{registre: {champ: {origines}}} — champs vus dans les plans reels."""
vus: dict[str, dict[str, set[str]]] = {r: {} for _, _, r in REGISTRES}
for plan in _plans():
etiquette = plan.parent.name
for fichier, racine_cle, registre in REGISTRES:
p = plan / fichier
if not p.is_file():
continue
try:
data = yaml.safe_load(p.read_text(encoding="utf-8")) or {}
except yaml.YAMLError:
continue
entrees = (data or {}).get(racine_cle) or {}
if not isinstance(entrees, dict):
continue
for entree in entrees.values():
if isinstance(entree, dict):
for champ in entree:
vus[registre].setdefault(str(champ), set()).add(etiquette)
return vus
def declaration_derive() -> list[str]:
"""Champs declares au GUI mais introuvables dans son source (declaration qui ment)."""
src = SOURCE_GUI.read_text(encoding="utf-8")
# On coupe le tableau lui-meme : sa presence ne prouve rien.
debut = src.find("champs_ecrits_par_gui()")
fin = src.find("\n}\n", debut)
ailleurs = src[:debut] + src[fin:] if debut != -1 and fin != -1 else src
absents = []
for registre, champs in champs_ecrits_par_gui().items():
# UN REGISTRE GENERE N'A PLUS SES CHAMPS DANS LE SOURCE, et c'est le but : ils
# vivent dans le schema, et le formulaire en derive. Chercher `secret` dans le JS
# ferait crier cette garde sur precisement le progres qu'elle devrait constater.
# Pour ces registres-la, c'est P61 qui tient la promesse — elle verifie que le
# schema decrit tout ce que les plans contiennent.
if registre in REGISTRES_GENERES:
continue
for champ in sorted(champs):
if champ not in ailleurs:
absents.append(f"{registre}.{champ}")
return absents
def main() -> int:
parser = argparse.ArgumentParser(description="Couverture du schema du plan par le GUI.")
sub = parser.add_subparsers(dest="commande", required=True)
sub.add_parser("lister", help="Detaille champ par champ.")
pv = sub.add_parser("verifier", help="Echoue s'il reste un champ non editable.")
pv.add_argument("--tolerer", action="append", default=[], metavar="REGISTRE",
help="Registre dont les trous sont connus et assumes (repetable).")
args = parser.parse_args()
utilises = champs_utilises()
if args.commande == "lister":
for registre in sorted(utilises):
couverts = champs_ecrits_par_gui().get(registre, set())
print(f"\n== {registre} ==")
for champ in sorted(utilises[registre]):
etat = "GUI" if champ in couverts else "-- hors GUI --"
print(f" {champ:<18} {etat:<16} vu dans : "
f"{', '.join(sorted(utilises[registre][champ]))}")
return 0
tolere = set(args.tolerer)
trous: list[str] = []
for registre, champs in utilises.items():
if registre in tolere:
continue
couverts = champs_ecrits_par_gui().get(registre, set())
for champ in sorted(set(champs) - couverts):
trous.append(f"{registre}.{champ} (vu dans : "
f"{', '.join(sorted(champs[champ]))})")
derive = declaration_derive()
if trous:
print(f"erreur: {len(trous)} champ(s) present(s) dans un plan reel que le GUI "
f"ne sait pas ecrire :", file=sys.stderr)
for t in trous:
print(f" - {t}", file=sys.stderr)
if derive:
print(f"erreur: {len(derive)} champ(s) declare(s) dans champs_ecrits_par_gui() mais "
f"introuvable(s) dans le source du GUI : {', '.join(derive)}", file=sys.stderr)
if trous or derive:
return 2
total = sum(len(c) for c in utilises.values())
print(f"GUI : les {total} champ(s) des plans reels sont editables "
f"({len(_plans())} plan(s) inspecte(s))"
+ (f", registres toleres : {', '.join(sorted(tolere))}." if tolere else "."))
return 0
if __name__ == "__main__":
raise SystemExit(main())