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
- nisarth/claude-skills-storetheir own version, first seen Nov 8, 2026
- Tcuzzo/backs-aios-skillstheir own version, first seen Nov 1, 2026
- davisbuilds/dojotheir own version, first seen Nov 1, 2026
- vijaysuperpositionist/skillstheir own version, first seen Nov 1, 2026
- ArnavPuri/designskillstheir own version, first seen Nov 1, 2026
- microsoft/amplifier-app-papercliptheir own version, first seen Oct 25, 2026
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-devSkillcolor-system
Build a comprehensive color system with palette generation, semantic mapping, and accessibility compliance.
Also in 3 projects that use this
ArnavPuriSkillpresentation-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
brianciouSkillux-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
jpoindexterSkilldesign-system-audit
Audit code for design system violations - grid spacing, tokens, existing components
Also in 3 projects that use this
agentes100xSkillbrand-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
nandanosqlSkilldesign-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
Other versions37 projects use a version of this.
Word for word · 0
None found.
Edited · 0
None found.
Wrote their own version · 37
- anthropics/knowledge-work-plugins★ 35ksame name
- cuellarfr/design-skills★ 94same name
- adityareddy-dev/deskstudio★ 43same name
- tdimino/claude-code-minoan★ 41same name
- launchapp-dev/animus-cli★ 39same name
- ArnavPuri/designskills★ 17same name
- henrique-simoes/Istara★ 15same name
- aiatelie/ai-atelie★ 9same name
- mae616/ai-template★ 9same name
- shayan-m81/layerwright★ 8same name
27 more not listed
How matches are found. Word for word: the exact same text. Edited: shares passages with this file (“99% same” means 99% of its five-word phrases also appear here; 10% or more counts). Their own version: similar meaning, or, weakest, the same name. We can’t tell which project had it first. Projects are counted among the 200 most-starred candidates, and mentions are links to the repository we found, up to 50 per site, so both counts are a floor.
In this repository17 other files it gives its agents.
- access-control-proofSkill
- agent-instructionsSkill
- contract-changeSkill
- data-change-scriptSkill
- deploy-safetySkill
- evidence-layersSkill
- interface-craftSkill
- issue-ledgerSkill
- mobile-field-readinessSkill
- offline-first-syncSkill
- parallel-worktreesSkill
- requirements-elicitationSkill
- root-cause-investigationSkill
- solution-architectureSkill
- ux-state-coverageSkill
- versioned-publishingSkill
- skill-nameSkill
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
- Distinctive: line 17
Write two or three sentences describing the design in your own words, and check them against the document.
- Distinctive: line 20
Do this first, every time.
- Distinctive: line 24
If you cannot restate it, that is the first finding: the design is not legible.
- Distinctive: line 32
Every finding is a statement about the artifact.
- Distinctive: line 69
If you cannot construct the scenario, you have a suspicion.
- Distinctive: line 75
Not by confidence, and not by how interesting the finding is.
- Distinctive: line 99
The number with no source: Every figure in a design should be traceable to a measurement or labelled as an estimate.
- Distinctive: line 114
Not for morale — for signal.
“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.
Files
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.
| Pass | The question |
| Correctness | Does this do what the requirements said? Map each requirement to where it is handled. The unmapped one is the finding. |
| Failure | What happens when each dependency is slow, down, or wrong? Slow is the hard one. |
| Concurrency | Two users, two tabs, two retries, two workers. What happens when the same thing arrives twice? |
| Scale | At 10x and at 100x. And at 0 and 1 — empty states and the very first record break more designs than volume does. |
| Security | Who can see what? Take the access-control-proof identity matrix and walk it against this design. |
| Data | What is the source of truth? What happens on a partial write? Can this lose data, and would anyone notice? |
| Operations | How do you know it is broken? How do you debug one bad record? What does the rollback look like? |
| Migration | How does the existing data get to the new shape, and what runs while it does? |
| Reversal | If this turns out to be wrong in three months, what does undoing it cost? |
| Human | What 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.
| Rank | Meaning |
| Blocking | Data loss, a security hole, or a one-way door that will be regretted. Do not build until resolved. |
| Significant | Will cause incidents or rework, but is fixable afterwards at a known cost. |
| Improvement | Better if changed, fine if not. |
| Taste | You 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
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
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.

