Architecture Critic

Senior-architect review persona: 9-point structural checklist, YAGNI/astronaut-architecture guardrails.

Door cofin · cofin/flow

Werkt met setup ★ 8.4/10

Architecture Critic — Senior-architect review persona: 9-point structural checklist, YAGNI/astronaut-architecture guardrails.

Wat het doet

Applies a senior-architect persona with a 6-12 month horizon to review component boundaries, coupling, cohesion, abstraction level, data flow, scaling, and testability, reporting each finding as structural problem plus long-term consequence plus recommendation. Triggers on requests to review architecture, evaluate component boundaries, question whether an abstraction or new layer is justified, or assess maintainability risk of a design/refactor.

Testrapport

Real gap: the 'References Index' points to `../perspectives/references/stances.md`, a sibling skill file that lives outside this folder in the cofin/flow plugin. Copy just architecture-critic/ as the repo's own layout implies and that link 404s -- confirmed by copying the folder into ~/.claude/skills and checking the path resolves to nothing. The core 9-point checklist and persona are self-contained and work fine standalone; only the 'perspectives'/'consensus' cross-skill hooks are broken.

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

Installatie

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

Commando's en voorbeeldprompts

  • /architecture-criticSenior-architect review persona: 9-point structural checklist, YAGNI/astronaut-architecture guardrails.

Skills reageren op gewone verzoeken — geen commando's om te onthouden. Na installatie activeren prompts zoals deze de skill (in het Engels):

  • 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