Repository navigation
docs(architecture): fix Mermaid label cropping + align Sirene RGPD on #93 - #95
Merged
Merged
Conversation
…93 Several node labels overflowed their boxes in LR flowcharts (text cropped at render): STG_FT, INT_ANA, FCT_OFF in the dbt lineage diagram, and GAP in the monitoring diagram. Split the long lines with extra <br/> — no text or meaning changed, only line-break points. Also correct STG_SI: "masquage RGPD" → "tags PII · non exposé", aligning the diagram with PR #93 (no masking is applied; Sirene is a terminal model, PII tagged for traceability but never exposed downstream — D14-bis).
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Contexte
Plusieurs labels de nœuds des diagrammes Mermaid de
docs/architecture.mddébordaient de leur boîte (texte tronqué au rendu, en
flowchart LR) :titres
stg_france_travail__offres,int_offres_analytiques,fct_offrescoupés, ainsi que le nœud
GAPdu diagramme monitoring. Cause : des lignes delabel trop longues pour la largeur allouée par le layout LR.
Au passage, le nœud
STG_SIportait l'assertion « masquage RGPD », que laPR #93 vient de corriger comme inexacte côté code dbt.
Changements
docs/architecture.md— 5 nœuds, 2 diagrammes :STG_FT,INT_ANA,FCT_OFF<br/>supplémentaires). Aucun texte modifié, seuls les points de coupure changent.GAP(monitoring)STG_SIDécision technique
STG_SI: aucun masquage n'est appliqué (cf. PR #93). Sirene est un modèleterminal — la jointure FT × Sirene a été abandonnée (D14-bis, SIRET absent
des offres), donc la PII est taguée pour traçabilité (
tag:pii) mais jamaisexposée en aval. Le label reflète désormais ce fait, plus l'intention erronée.
Validation
.md). Aucune logique touchée.Conventional Commitpassed ; hooks ruffSkipped— pas de fichier Python).Refs : PR #93 (correction RGPD Sirene), D14-bis (jointure FT × Sirene abandonnée), D109 (trou alerte scheduler).