Architecture Critic

Senior-arkitekt review persona: 9-punkts strukturel checkliste, YAGNI/astronaut-arkitektur guardrails.

Af cofin · cofin/flow

Virker med opsætning ★ 8.4/10

Architecture Critic — Senior-arkitekt review persona: 9-punkts strukturel checkliste, YAGNI/astronaut-arkitektur guardrails.

Hvad det gør

Anvender en senior-arkitekt persona med en 6-12 måneders horisont til at gennemgå komponentgrænser, kobling, kohæsion, abstraktionsniveau, dataflow, skalering og testbarhed, og rapporterer hvert fund som et strukturelt problem plus langsigtet konsekvens plus anbefaling. Triggere på anmodninger om at gennemgå arkitektur, evaluere komponentgrænser, spørge, om en abstraktion eller et nyt lag er berettiget, eller vurdere vedligeholdelsesrisikoen af et design/refactor.

Testrapport

Reelt hul: 'References Index' peger på `../perspectives/references/stances.md`, en søsterskill-fil, der ligger uden for denne mappe i cofin/flow-plugin'et. Kopier kun architecture-critic/ som repoens egen layout antyder, og den reference fejler 404 — bekræftet ved at kopiere mappen ind i ~/.claude/skills og kontrollere, at stien ikke løser sig. Kernen 9-punkts checkliste og persona er selvstændige og fungerer fint alene; kun 'perspectives'/'consensus' cross-skill hooks er ødelagte.

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

Installation

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

Kommandoer og eksempelprompter

  • /architecture-criticSenior-arkitekt review persona: 9-punkts strukturel checkliste, YAGNI/astronaut-arkitektur guardrails.

Skills udløses af almindelige forespørgsler — ingen kommandoer at huske. Efter installationen aktiverer prompter som disse skillen (på engelsk):

  • 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