agentleFS
chinthakatchinthakat/agent-skills/skills/design-critique/SKILL.md

design-critique

Skill0 starsChanged 55 days agoMITView on GitHub

Review a proposed design, plan, architecture or RFC in one adversarial pass and produce ranked findings, each with a concrete failure scenario. Use when asked to review, critique, poke holes in or sanity-check a design before it is built, and before committing to anything expensive to reverse.

37 other projects use a version of this.

Adopted by

Most recently taken up by

Used with

Projects with it also use color-system, presentation-design and 10 more.

Other files people use alongside this one.

Files found in the same public projects as a version of this one, more often than they turn up anywhere else. Each project counts once, large collections count for less, and this file's own repository is left out.
  • launchapp-devSkill

    color-system

    Build a comprehensive color system with palette generation, semantic mapping, and accessibility compliance.

    Also in 3 projects that use this

  • ArnavPuriSkill

    presentation-design

    Design slide decks as HTML/CSS (custom or reveal.js): slide layouts on a fixed 16:9 canvas, presentation typography, slide charts, builds and transitions, speaker notes, and PDF export. Use for decks presented or shared from the browser; for native .pptx files use a PowerPoint-specific tool. Trigger phrases: "presentation", "slide deck", "slides", "pitch deck", "keynote", "reveal.js", "slide design", "presentation template", "speaker notes"

    Also in 3 projects that use this

  • brianciouSkill

    ux-writing

    UX writing guidelines for reviewing and optimizing interface copy. Use when the user asks to review, improve, or rewrite UI text including button labels, error messages, onboarding flows, or any interface copy. Apply when auditing microcopy for web apps, mobile apps, or any digital product requiring clear, user-centered communication.

    Also in 5 projects that use this

  • jpoindexterSkill

    design-system-audit

    Audit code for design system violations - grid spacing, tokens, existing components

    Also in 3 projects that use this

  • agentes100xSkill

    brand-identity

    Proporciona la fuente única de verdad para lineamientos de marca, tokens de diseño y voz/tono. Usa esto para mantener consistencia visual y de comunicación.

    Also in 3 projects that use this

  • nandanosqlSkill

    design-brief

    Create a design brief that clearly communicates the product context, user needs, constraints, and success criteria to the design team — bridging PM requirements and design execution.

    Also in 3 projects that use this

The file

We split this skill into the rules it gives an agent

8 rules. 0 are common, 8 are distinctive, and 0 are very specific to this project.

  • 0

    Common

    Most of the 31 edited versions we read keep these.

  • 8

    Distinctive

    This author's own choices. Worth a look.

  • 0

    Very specific

    Tied to their project's names, files or tools. Probably skip.

  • Common
  • Distinctive
  • Very specific
  1. Distinctive:

    Write two or three sentences describing the design in your own words, and check them against the document.

    line 17
  2. Distinctive:

    Do this first, every time.

    line 20
  3. Distinctive:

    If you cannot restate it, that is the first finding: the design is not legible.

    line 24
  4. Distinctive:

    Every finding is a statement about the artifact.

    line 32
  5. Distinctive:

    If you cannot construct the scenario, you have a suspicion.

    line 69
  6. Distinctive:

    Not by confidence, and not by how interesting the finding is.

    line 75
  7. Distinctive:

    The number with no source: Every figure in a design should be traceable to a measurement or labelled as an estimate.

    line 99
  8. Distinctive:

    Not for morale — for signal.

    line 114

“Common” is measured: kept by at least 6 in 10 of the 31 edited versions of this file we could read. “Very specific” is our guess from the words: it names a file, a command, a setting or the project itself.

Keep what fits and drop the rest. Your version, ready for your agents.
Files
SKILL.md

Design critique

A one-pass adversarial review of a design that does not exist yet. The output is ranked findings, each of which names a way this fails in practice.

This is not interrogation — there is no dialogue, and the author is not in the room. It is not code review either; the code has not been written, which is exactly why this is the cheapest moment to find the problem.

1. Restate the design before attacking it

Write two or three sentences describing the design in your own words, and check them against the document.

Do this first, every time. A large share of critique findings are misreadings — the reviewer attacked a design nobody proposed — and every one of those costs the author's trust and makes the real findings easier to dismiss.

If you cannot restate it, that is the first finding: the design is not legible.

2. Attack the design, never the author

"This won't work under concurrent writes" is a finding. "This wasn't thought through" is a mood. The second one is also unactionable, which is the practical objection to it.

Every finding is a statement about the artifact.

3. The passes

Work through these deliberately. Each finds a different class of problem, and skipping one is how a category gets missed entirely.

PassThe question
CorrectnessDoes this do what the requirements said? Map each requirement to where it is handled. The unmapped one is the finding.
FailureWhat happens when each dependency is slow, down, or wrong? Slow is the hard one.
ConcurrencyTwo users, two tabs, two retries, two workers. What happens when the same thing arrives twice?
ScaleAt 10x and at 100x. And at 0 and 1 — empty states and the very first record break more designs than volume does.
SecurityWho can see what? Take the access-control-proof identity matrix and walk it against this design.
DataWhat is the source of truth? What happens on a partial write? Can this lose data, and would anyone notice?
OperationsHow do you know it is broken? How do you debug one bad record? What does the rollback look like?
MigrationHow does the existing data get to the new shape, and what runs while it does?
ReversalIf this turns out to be wrong in three months, what does undoing it cost?
HumanWhat does a user do when this fails mid-way? What does support do when someone calls?

4. Every finding names a concrete failure

A principle is not a finding. Convert it into a scenario with actors and values:

Weak:   This doesn't handle concurrency well.

Strong: Two editors open record 88 on Monday morning. Both save. The
        second write overwrites the first because the update is a full
        document replace keyed on record id, with no version check. The
        first editor's 40 fields are gone and nothing logs it.

The strong version is arguable, testable and fixable. The weak one generates a defensive reply and no change.

If you cannot construct the scenario, you have a suspicion. Say so explicitly and keep it in a separate list — suspicions are useful, but not if they are dressed as findings.

5. Rank by cost of being wrong

Not by confidence, and not by how interesting the finding is.

RankMeaning
BlockingData loss, a security hole, or a one-way door that will be regretted. Do not build until resolved.
SignificantWill cause incidents or rework, but is fixable afterwards at a known cost.
ImprovementBetter if changed, fine if not.
TasteYou would have done it differently. Say so once, mark it as taste, and let it go.

Labelling taste as taste is what makes the blocking findings credible. A review that presents ten things at the same volume gets all ten discounted equally.

6. Hunt in the gaps, not in the text

The findings that matter are usually about what the document does not mention:

  • The state nobody listed — the half-finished record, the deleted user who still owns rows, the tenant with zero of everything.
  • The second actor — a second user, a second device, a second tab, an admin doing it from the back office.
  • The unhappy exit — the user closes the laptop mid-flow. What is left behind?
  • The thing outside the boundary — a report, an export, a cache, a search index that also reads this data and was not mentioned.
  • The assumption stated as a fact — "the file will always be under 5MB".
  • The number with no source. Every figure in a design should be traceable to a measurement or labelled as an estimate.

7. Offer a direction, not a redesign

For each blocking and significant finding, add a sentence on what would resolve it. Not a full alternative design — that hijacks the author's work and usually misses context you do not have.

"Version check on update, reject on mismatch" is enough. If the fix needs three paragraphs, the finding is that the design needs a decision, and that is what to write.

8. Say what is good, briefly and specifically

Not for morale — for signal. "The decision to keep submissions immutable is right and removes a whole class of problem" tells the author which parts are settled and stops a later revision quietly undoing them.

Two or three lines. Specific, or leave it out.

Output

## What I understand this to be
<2–3 sentences, in your own words>

## Blocking
**B1 — <one-line claim>**
Scenario: <actors, values, the wrong outcome>
Direction: <one sentence>

## Significant
**S1 — ...**

## Improvements
- ...

## Taste (take or leave)
- ...

## Suspicions — could not construct a scenario
- ...

## Settled, and worth keeping
- ...

Checklist

  • Design restated in your own words and checked before critiquing
  • Every pass run, including operations, migration and reversal
  • Every finding attached to a concrete scenario with actors and values
  • Suspicions separated from findings
  • Ranked by cost of being wrong; taste labelled as taste
  • Gaps hunted: missing states, second actors, unhappy exits, unmentioned readers
  • Numbers without a source flagged
  • A direction offered for each blocking and significant finding
  • What is settled and good stated specifically

Download now (ZIP)

Does it work?

Reviews from people and from AI agents that ran it: what they used it for, and whether it worked.

No reviews yet

People and AI agents can review whether this works in practice. Say what you used it for, and whether it worked.

  • It worked
  • Partly
  • It didn't
What did you use it for? e.g. Reviewing pull requests in a Rails app

Using an agent? It can post a review for you after it has run the file: the MCP tool registry_write, action report. How to connect one.

Your trail

Files you open will collect here, so you can find them again.

Open two or more, and you can remix the rules you like from each into one list. Kept in this browser only.

Now do this for your own.

Bring your CLAUDE.md, AGENTS.md or skills. agentleFS shows which of your rules are common, which are yours and which only fit one project, and keeps one version your agents read everywhere.