Revue des ADR

VérifiéPrudence

Accepter ou rejeter une décision d'architecture (ADR) selon les instructions du propriétaire, avec mise à jour de l'index et du journal.

Spar Skills Guide Bot
DocumentationIntermédiaire
23/07/2026
Claude CodeCodexCursor
#adr#architecture-decisions#governance#changelog#okf

Recommandé pour

Synthèse automatique

Pilote l'acceptation ou le rejet d'une ADR proposée à la demande du propriétaire, en mettant à jour le statut, l'index, le journal et les marqueurs associés dans le dépôt.

Points forts

  • Décrit pas à pas les éditions de statut (frontmatter et section # Status) ainsi que la mise à jour de l'index, des jalons de GOAL.md et de CLAUDE.md
  • Traite explicitement le cas du rejet, avec retour arrière selon le déclencheur de rollback propre à l'ADR et exécution des tests
  • Rappelle que l'acceptation ne nécessite généralement aucun changement d'implémentation sous la politique propose-then-implement
  • Gère les ADR héritées sans frontmatter en repérant les conventions de statut dans le corps du document

Limites

  • Suppose l'existence des scripts okf (okf pending) et des fichiers docs/adr/index.md, docs/log.md, docs/GOAL.md
  • La décision reste entièrement au propriétaire : l'assistant ne fait qu'exécuter des éditions sur instruction
  • Ne couvre pas la rédaction initiale d'une ADR ni la conception architecturale
Quand l'utiliser

À utiliser lorsque le propriétaire examine les décisions en attente et demande d'accepter, de rejeter ou de réviser une ADR précise.

Quand l'éviter

À éviter si le dépôt n'utilise pas le format ADR/OKF décrit, ou s'il s'agit de créer une nouvelle ADR plutôt que de statuer sur une existante.

Analyse de sécurité

Prudence
Score qualité82/100

This is a legitimate ADR workflow and contains no exfiltration, obfuscation, curl-pipe-shell, or absolute destructive commands. However, it directs an agent to run repository scripts, edit/delete files, and potentially revert code, so it warrants caution rather than being treated as purely non-executing.

Points d'attention
  • •No allowed-tools are declared, but the instructions require shell execution (`bash scripts/okf pending`, repo test command) and file edits/deletions.
  • •Rejecting an ADR can revert implementation work and delete ADR files, which is potentially destructive without explicit human confirmation guardrails.
  • •The skill depends on repository-local scripts and test commands whose behavior is not specified here.
  • •It may modify multiple project files (`docs/adr/index.md`, `docs/GOAL.md`, `CLAUDE.md`, `docs/log.md`), so scope should be constrained.
Comment cette analyse est faite

Exemples

Accept a pending ADR
Accept ADR-0007 at my direction — flip the status, clean up the (proposed) markers in the index and GOAL.md, and add a dated log entry.
Reject and revert
Reject ADR-0012: follow its rollback trigger section to revert the work, run the tests, and log the reversal scope.
Check what's pending
What ADRs are awaiting my review right now? Run the pending scan and summarize them.

name: okf-adr-review description: Accept or reject a proposed ADR at the owner's direction — status flip, reversal per rollback trigger, index and log updates. Use when the owner reviews pending decisions, says to accept/reject an ADR, or asks what's awaiting review.

ADR review mechanics

The owner finds pending decisions with bash scripts/okf pending (the SessionStart hook also reports the count). The decision is the owner's; the edits can be yours at their direction.

Accepting an ADR:

  1. Flip status: proposed to status: accepted in the frontmatter, and update the body # Status section (date, "at the owner's direction").
  2. Remove any "(proposed)" markers the ADR carries in the ADR index entries (docs/adr/index.md by default; a layout: block in docs/okf-map.yml may relocate it), docs/GOAL.md milestones, or the current-state line of CLAUDE.md.
  3. No implementation change is usually needed: under the propose-then-implement policy the work already exists — acceptance makes it binding for future work.
  4. Add a dated docs/log.md entry recording the acceptance, and verify bash scripts/okf pending no longer lists it.

Rejecting an ADR:

  1. Revert the work built on it, per the ADR's own rollback / revisit trigger section — that section names what reverting takes.
  2. Either delete the ADR file and its index entry, or keep it with a status: rejected note recording why, per the owner's preference.
  3. Run the repo's test command after the reversal; a broken revert is not a completed rejection.
  4. Log the rejection and the reversal scope in docs/log.md.

Requested changes (neither accept nor reject): treat the owner's comments as a task — amend the ADR and the implementation together, keep status: proposed, and flag it again in the summary.

The pending scan reads frontmatter status: first, then the body conventions brownfield repos use (a - Status: X bullet or a ## Status section). ADRs with none of those are invisible to it — add frontmatter status to them when found, as a formatting-only fix; frontmatter stays the preferred form for new ADRs.

Du fichier au premier essai

Le téléchargement contient exactement le contenu affiché sur cette fiche. Cette copie peut différer de la dernière version du dépôt source.

Consulter la source et ses fichiers annexes

Ce téléchargement contient uniquement SKILL.md. Les scripts, références, modèles et dépendances éventuels ne sont pas inclus. Récupérez les fichiers nécessaires depuis la source en conservant leur arborescence avant de poursuivre.

1. Télécharger le fichier

Télécharger SKILL.md

Depuis la racine d’un projet de test, enregistrez le fichier sous .skill-downloads/adr-review-mrx3xwms/SKILL.md, ou utilisez la commande suivante. Ce dossier sert à examiner la skill avant son installation.

Commande pour macOS/Linux ou WSL : un shell POSIX et curl sont nécessaires. Sous Windows sans WSL, utilisez le bouton de téléchargement et créez le dossier manuellement.

mkdir -p '.skill-downloads/adr-review-mrx3xwms' && curl --fail --show-error --location 'https://skills-guides.com/api/skills/adr-review-mrx3xwms/download' --output '.skill-downloads/adr-review-mrx3xwms/SKILL.md.part' && mv '.skill-downloads/adr-review-mrx3xwms/SKILL.md.part' '.skill-downloads/adr-review-mrx3xwms/SKILL.md'

2. Vérifier les prérequis

Lisez SKILL.md avant de l’utiliser. Si le fichier cite des scripts ou services externes, examinez-les et préparez les accès nécessaires dans votre environnement de test.

Aucune section de prérequis reconnue dans ce fichier. Cela ne signifie pas qu’il est autonome : les bibliothèques, outils et accès peuvent être cités ailleurs dans le contenu ou dans le dépôt source.

Les plateformes de la fiche sont des indications du catalogue. Les chemins ci-dessous suivent la documentation des outils ; ils ne prouvent pas que cette skill y a été exécutée. Aucune version minimale de ces outils n’est attestée pour cette fiche.

3. Charger la skill

Après vérification, copiez le dossier complet dans le projet avec l’une des commandes ci-dessous, puis ouvrez une nouvelle session de l’outil. La commande s’arrête si un dossier du même nom existe déjà : comparez les fichiers avant de le remplacer.

Claude Code
test -f '.skill-downloads/adr-review-mrx3xwms/SKILL.md' && test -s '.skill-downloads/adr-review-mrx3xwms/SKILL.md' && mkdir -p '.claude/skills' && test ! -e '.claude/skills/okf-adr-review' && cp -R '.skill-downloads/adr-review-mrx3xwms' '.claude/skills/okf-adr-review'

Cursor
test -f '.skill-downloads/adr-review-mrx3xwms/SKILL.md' && test -s '.skill-downloads/adr-review-mrx3xwms/SKILL.md' && mkdir -p '.cursor/skills' && test ! -e '.cursor/skills/okf-adr-review' && cp -R '.skill-downloads/adr-review-mrx3xwms' '.cursor/skills/okf-adr-review'

Codex
test -f '.skill-downloads/adr-review-mrx3xwms/SKILL.md' && test -s '.skill-downloads/adr-review-mrx3xwms/SKILL.md' && mkdir -p '.agents/skills' && test ! -e '.agents/skills/okf-adr-review' && cp -R '.skill-downloads/adr-review-mrx3xwms' '.agents/skills/okf-adr-review'

OpenSkills

OpenSkills doit déjà être installé. La procédure locale install/read est contrôlée avec OpenSkills 1.5.0, qui exige Node.js ≥ 20.6.0. Aucun slug du catalogue n’est résolu automatiquement.

Installation officielle d’OpenSkills
openskills install './.skill-downloads/adr-review-mrx3xwms'

openskills read 'adr-review-mrx3xwms'

La lecture doit afficher le contenu du fichier installé et son dossier de base. Elle ne lance ni la skill ni ses dépendances.

Pour un agent qui lit AGENTS.md, synchronisez ensuite l’index. Cette commande modifie AGENTS.md : relisez le diff avant de l’utiliser.

openskills sync

Mode manuel ou autre outil : joignez SKILL.md à une conversation, ou ouvrez-le dans un agent capable de lire les fichiers. Ajoutez les ressources annexes nécessaires. Il n’existe pas de dossier universel reconnu par tous les outils.

4. Faire un premier essai

Utilisez une entrée courte et fictive : un endpoint d’API, un petit document ou un brief d’interface. Dans une nouvelle conversation, joignez le fichier ou donnez son chemin à l’agent, puis adaptez cette demande :

Lis .skill-downloads/adr-review-mrx3xwms/SKILL.md et applique ses instructions à cet exemple : [décrire une entrée de test]. Indique les fichiers, dépendances ou accès qui manquent. Produis le résultat demandé par la skill sans inventer de données ni exécuter de script non examiné.

Résultat attendu : une sortie correspondant aux instructions de la skill, ou une liste précise des prérequis manquants. Une simple confirmation de lecture ne valide pas le premier usage. Si une ressource externe est indisponible, ce parcours reste à tester.

Skills similaires