Architecture Critic

Persona di revisione senior-architetto: checklist strutturale in 9 punti, guardrail YAGNI/architettura astronautica.

di cofin · cofin/flow

Richiede configurazione ★ 8.4/10

Architecture Critic — Persona di revisione senior-architetto: checklist strutturale in 9 punti, guardrail YAGNI/architettura astronautica.

Cosa fa

Applica una persona da senior-architetto con un orizzonte di 6-12 mesi per revisionare i confini dei componenti, l'accoppiamento, la coesione, il livello di astrazione, il flusso di dati, la scalabilità e la testabilità, riportando ogni riscontro come problema strutturale più conseguenza a lungo termine più raccomandazione. Si attiva su richieste di revisione dell'architettura, valutazione dei confini dei componenti, messa in discussione della giustificazione di un'astrazione o di un nuovo livello, o valutazione del rischio di manutenibilità di un design/refactor.

Rapporto di test

Gap reale: l' 'References Index' punta a `../perspectives/references/stances.md`, un file di skill fratello che si trova fuori da questa cartella nel plugin cofin/flow. Copiando solo architecture-critic/ come implica il layout del repository, quel link restituisce 404 — confermato copiando la cartella in ~/.claude/skills e verificando che il percorso non risolva nulla. La checklist principale in 9 punti e la persona sono autonome e funzionano bene da sole; solo gli hook cross-skill 'perspectives'/'consensus' sono rotti.

Testato il: 2026-07-15 · Claude Code 2.x (agent harness)

Installazione

git clone https://github.com/cofin/flow
mkdir -p ~/.claude/skills
cp -r flow/skills/architecture-critic ~/.claude/skills/architecture-critic

Comandi e prompt di esempio

  • /architecture-criticPersona di revisione senior-architetto: checklist strutturale in 9 punti, guardrail YAGNI/architettura astronautica.

Gli skill si attivano con richieste in linguaggio naturale, senza comandi da ricordare. Dopo l'installazione, prompt come questi lo attivano (in inglese):

  • review these component boundaries like a skeptical senior architect
  • is adding this new abstraction layer actually justified right now
  • critique the coupling and testability of this service design