ADR-015 ActifActive contract

Hook automatique de rappel ADR sur modifications critiques

DateDate
2026-05-28
DécideursDecision makers
Design System Lead

English summary: Adds a PostToolUse hook that reminds the agent to ask "do you want an ADR for this?" whenever tokens/, guidelines/, or components/ are touched, since documented rules were being forgotten in practice. A 2026-05-31 amendment restricts the reminder to Write (new files) only, since it fired too often on routine Edit bug fixes and became noise rather than a safeguard.

>

*The original French version follows below — preserved unaltered as the historical record.*


Contexte

Les fichiers tokens/, guidelines/, et components/ contiennent les décisions structurantes du système de design : valeurs de tokens, contrats de composants, documentation normative. Toute modification dans ces zones devrait, en principe, être accompagnée d'un ADR ou d'un TCR qui en explique le pourquoi.

En pratique, la documentation était oubliée. Un agent ou un humain modifiait tokens/semantic.json pour ajouter un token, commitait, et aucun ADR n'était créé. L'historique git conservait le _quoi_ mais perdait le _pourquoi_.

La question posée était :

Comment s'assurer qu'une modification dans une zone critique du système déclenche systématiquement une réflexion sur la nécessité d'un ADR, sans bloquer le flux de travail ni imposer un ADR pour chaque changement mineur ?


Décision

Ajout d'un hook PostToolUse dans .claude/settings.json qui s'active après chaque opération Write ou Edit sur un fichier dont le chemin contient /tokens/, /guidelines/, ou /components/.

Le hook injecte un rappel dans le contexte du modèle via additionalContext :

RAPPEL ADR : Le fichier modifié est dans une zone critique du système de design.
Demande immédiatement à l'utilisateur : Souhaitez-vous créer un ADR
pour documenter ce changement ? (Oui/Non)

Ce rappel oblige Claude à poser la question à chaque modification critique. L'utilisateur répond Oui ou Non — aucun ADR n'est créé sans décision humaine explicite.

Configuration (/.claude/settings.json)

{
  "hooks": {
    "PostToolUse": [
      {
        "matcher": "Write|Edit",
        "hooks": [
          {
            "type": "command",
            "command": "f=$(jq -r '.tool_input.file_path // \"\"'); if echo \"$f\" | grep -qE '/(tokens|guidelines|components)/'; then printf '{...additionalContext...}' \"$f\"; fi",
            "statusMessage": "Vérification ADR…"
          }
        ]
      }
    ]
  }
}

Alternatives rejetées

AlternativeRaison du rejet
Règle documentée dans CLAUDE.md uniquementUne règle textuelle dans un fichier de contexte dépend de la bonne lecture de ce fichier par l'agent. Un hook s'exécute systématiquement, indépendamment du contexte chargé.
Hook bloquant (exit non-zéro)Bloquer le Edit jusqu'à création d'un ADR serait trop contraignant : tous les changements ne justifient pas un ADR (correction de typo, reformatage). Le rappel non-bloquant laisse la décision à l'humain.
Hook sur git commit (pre-commit)Un hook git vérifie au moment du commit, trop tard dans le flux : la question pertinente est posée _pendant_ la modification, quand le contexte est encore actif.
Notification système (OS notification)Les notifications OS sont ignorées lors d'un travail dans le terminal. Le additionalContext est injecté directement dans la réponse de Claude — impossible à manquer.
ADR obligatoire pour chaque changementSurcharge documentaire. Un changement de valeur de token primitif de #3B82F6 à #3B81F5 ne justifie pas un ADR complet. Le rappel filtre par contexte humain.

Conséquences

Pour les agents IA :

  • Après chaque Edit ou Write dans une zone critique, Claude pose la question ADR

sans que l'utilisateur ait à y penser

  • Le hook s'exécute via le harness Claude Code — il ne dépend pas du prompt,

du contexte chargé, ni de la mémoire de la session

Pour les humains :

  • La friction documentaire est minimale : une question Oui/Non après chaque changement
  • Les ADR oubliés deviennent structurellement impossibles sur les zones couvertes
  • Le comportement est transparent et auditable dans .claude/settings.json

Zones couvertes :

  • tokens/ — primitifs, sémantiques, composants
  • guidelines/ — documentation normative des composants et fondations
  • components/ — implémentations Web Components

Zones non couvertes (volontairement) :

  • .claude/ — configuration interne, pas de décision de design
  • src/ — code applicatif, couvert par d'autres processus de revue
  • decisions/ — les ADR eux-mêmes ne déclenchent pas un rappel récursif

Coût accepté :

  • Le rappel s'affiche même pour des changements mineurs (correction de typo dans un token)
  • L'utilisateur doit répondre Non explicitement — légère friction pour les changements évidents

Amendement — 2026-05-31 : rappel ADR sur Write uniquement

Problème observé

Lors des sessions de développement des composants agtc-badge, agtc-card et des corrections WCAG qui ont suivi, le hook se déclenchait sur chaque Edit dans les zones critiques — y compris pour des corrections de bugs sans décision architecturale (changement d'un ratio de contraste insuffisant, correction d'une typo dans une story, ajustement d'une couleur qui échoue un test axe).

Résultat concret : pour corriger les 27 violations WCAG du badge (3 couleurs de texte à remplacer), le hook a interrompu le travail 6 fois pour poser une question dont la réponse était systématiquement non. La friction documentaire devenait du bruit plutôt qu'une garde-fou utile.

Distinction fondamentale

Type de changementBesoin d'ADRExemple
Nouveau fichier (Write)Oui — c'est une création, donc une décisionAjouter agtc-badge.js, créer un token
Modification (Edit)Rarement — souvent un bug fix ou un ajustementCorriger un contraste, renommer une variable

La question "faut-il un ADR ?" est pertinente quand on crée quelque chose (nouveau composant, nouveau token, nouvelle guideline). Elle est presque toujours non pertinente quand on modifie quelque chose qui existe déjà — l'ADR original couvre déjà la décision.

Changement appliqué

Le hook PostToolUse est maintenant séparé en deux entrées :

{
  "hooks": {
    "PostToolUse": [
      {
        "matcher": "Write",
        "hooks": [{ "type": "command", "command": "...rappel ADR..." }]
      },
      {
        "matcher": "Write|Edit",
        "hooks": [{ "type": "command", "command": "...log construction..." }]
      }
    ]
  }
}
  • Rappel ADR : Write uniquement — se déclenche à la création d'un fichier
  • Log de construction : Write|Edit — inchangé, capture toutes les modifications

Coût accepté de l'amendement

Un Edit qui représente réellement une nouvelle décision architecturale ne déclenchera plus automatiquement le rappel. La discipline reste nécessaire : si une modification de token ou de composant encode une décision non triviale, l'humain ou l'agent doit créer l'ADR manuellement. Le hook n'est plus le seul garde-fou pour les modifications.


Incidents ou déclencheurs (version originale)

Constat répété : lors de sessions de travail sur les tokens et les guidelines, des modifications structurantes étaient commitées sans ADR associé. L'historique git conservait le diff mais pas la justification. Ce hook est la réponse procédurale à ce manque de discipline documentaire.

← ADR-014 ADR-016 →