agentleFS
excattexcatt/superclaude-plusplus/skills/critique/SKILL.md

critique

Skill14 starsChanged 37 days agoMITView on GitHub

Evaluate design from a UX perspective, assessing visual hierarchy, information architecture, emotional resonance, cognitive load, and overall quality with quantitative scoring, persona-based testing, automated anti-pattern detection, and actionable feedback. Use when the user asks to review, critique, evaluate, or give feedback on a design or component.

80 other projects use a version of this, 38 of them edited.

Adopted by

Most recently taken up by

Used with

Projects with it also use teach-impeccable, arrange and 2 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.
  • HK-hubSkill

    teach-impeccable

    One-time setup that gathers design context for your project and saves it to your AI config file. Run once to establish persistent design guidelines.

    Also in 19 projects that use this

  • HK-hubSkill

    arrange

    Improve layout, spacing, and visual rhythm. Fixes monotonous grids, inconsistent spacing, and weak visual hierarchy. Use when the user mentions layout feeling off, spacing issues, visual hierarchy, crowded UI, alignment problems, or wanting better composition.

    Also in 14 projects that use this

  • zwy123zwySkill

    normalize

    将用户随意输入的提示词规范化为结构清晰、便于大模型执行的标准化提示词。适用于需要整理、统一提示词格式时调用。

    Also in 19 projects that use this

  • +1 more from HK-hub

The file

Show its rules and text

We split this skill into the rules it gives an agent

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

  • 0

    Common

    Most of the 73 edited versions we read keep these.

  • 20

    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:

    If sub-agents are not available in the current environment, complete each assessment sequentially, writing findings to internal notes …

    line 25
  2. Distinctive:

    Tab isolation: When browser automation is available, each assessment MUST create its own new tab.

    line 27
  3. Distinctive:

    Read the relevant source files (HTML, CSS, JS/TS) and, if browser automation is available, visually inspect the live page.

    line 31
  4. Distinctive:

    Run the 8-item cognitive load checklist.

    line 42
  5. Distinctive:

    Check for progressive disclosure: is complexity revealed only when needed?

    line 44
  6. Distinctive:

    Emotional valleys: Check for anxiety spikes at high-stakes moments (payment, delete, commit).

    line 49
  7. Distinctive:

    Run the bundled deterministic detector, which flags 25 specific patterns (AI slop tells + general design quality).

    line 58
  8. Distinctive:

    Wait 2-3 seconds for the detector to render overlays

    line 90
  9. Distinctive:

    Cleanup: Stop the live server when done:

    line 92
  10. Distinctive:

    Alex (Power User): No keyboard shortcuts detected.

    line 160
  11. Distinctive:

    Don't soften criticism. Developers need honest feedback to ship great design.

    line 181
  12. Distinctive:

    Ask questions along these lines (adapt to the specific findings.

    line 187
  13. Distinctive:

    Scope: Ask how much the user wants to take on.

    line 193
  14. Distinctive:

    Every question must reference specific findings from the report.

    line 198
  15. Distinctive:

    Keep it to 2-4 questions maximum.

    line 199
  16. Distinctive:

    If findings are straightforward (e.g., only 1-2 clear issues), skip questions and go directly to Step 5.

    line 201
  17. Distinctive:

    Only recommend commands from: /animate, /quieter, /shape, /optimize, /adapt, /clarify, /layout, /distill, /delight, /design-audit, /harden, …

    line 216
  18. Distinctive:

    If the user chose a limited scope, only include items within that scope

    line 221
  19. Distinctive:

    If the user marked areas as off-limits, exclude commands that would touch those areas

    line 222
  20. Distinctive:

    After presenting the summary, tell the user:

    line 225

“Common” is measured: kept by at least 6 in 10 of the 73 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

STEPS

Step 1: Preparation

Invoke /impeccable, which contains design principles, anti-patterns, and the Context Gathering Protocol. Follow the protocol before proceeding. If no design context exists yet, you MUST run /impeccable teach first. Additionally gather: what the interface is trying to accomplish.

Step 2: Gather Assessments

Launch two independent assessments. Neither must see the other's output to avoid bias.

You SHOULD delegate each assessment to a separate sub-agent for independence. Use your environment's agent spawning mechanism (e.g., Claude Code's Agent tool, or Codex's subagent spawning). Sub-agents should return their findings as structured text. Do NOT output findings to the user yet.

If sub-agents are not available in the current environment, complete each assessment sequentially, writing findings to internal notes before proceeding.

Tab isolation: When browser automation is available, each assessment MUST create its own new tab. Never reuse an existing tab, even if one is already open at the correct URL. This prevents the two assessments from interfering with each other's page state.

Assessment A: LLM Design Review

Read the relevant source files (HTML, CSS, JS/TS) and, if browser automation is available, visually inspect the live page. Create a new tab for this; do not reuse existing tabs. After navigation, label the tab by setting the document title:

document.title = '[LLM] ' + document.title;

Think like a design director. Evaluate:

AI Slop Detection (CRITICAL): Does this look like every other AI-generated interface? Review against ALL DON'T guidelines in the impeccable skill. Check for AI color palette, gradient text, dark glows, glassmorphism, hero metric layouts, identical card grids, generic fonts, and all other tells. The test: If someone said "AI made this," would you believe them immediately?

Holistic Design Review: visual hierarchy (eye flow, primary action clarity), information architecture (structure, grouping, cognitive load), emotional resonance (does it match brand and audience?), discoverability (are interactive elements obvious?), composition (balance, whitespace, rhythm), typography (hierarchy, readability, font choices), color (purposeful use, cohesion, accessibility), states & edge cases (empty, loading, error, success), microcopy (clarity, tone, helpfulness).

Cognitive Load (consult cognitive-load):

  • Run the 8-item cognitive load checklist. Report failure count: 0-1 = low (good), 2-3 = moderate, 4+ = critical.
  • Count visible options at each decision point. If >4, flag it.
  • Check for progressive disclosure: is complexity revealed only when needed?

Emotional Journey:

  • What emotion does this interface evoke? Is that intentional?
  • Peak-end rule: Is the most intense moment positive? Does the experience end well?
  • Emotional valleys: Check for anxiety spikes at high-stakes moments (payment, delete, commit). Are there design interventions (progress indicators, reassurance copy, undo options)?

Nielsen's Heuristics (consult heuristics-scoring): Score each of the 10 heuristics 0-4. This scoring will be presented in the report.

Return structured findings covering: AI slop verdict, heuristic scores, cognitive load assessment, what's working (2-3 items), priority issues (3-5 with what/why/fix), minor observations, and provocative questions.

Assessment B: Automated Detection

Run the bundled deterministic detector, which flags 25 specific patterns (AI slop tells + general design quality).

CLI scan:

npx impeccable --json [--fast] [target]
  • Pass HTML/JSX/TSX/Vue/Svelte files or directories as [target] (anything with markup). Do not pass CSS-only files.
  • For URLs, skip the CLI scan (it requires Puppeteer). Use browser visualization instead.
  • For large directories (200+ scannable files), use --fast (regex-only, skips jsdom)
  • For 500+ files, narrow scope or ask the user
  • Exit code 0 = clean, 2 = findings

Browser visualization (when browser automation tools are available AND the target is a viewable page):

The overlay is a visual aid for the user. It highlights issues directly in their browser. Do NOT scroll through the page to screenshot overlays. Instead, read the console output to get the results programmatically.

  1. Start the live detection server:
    npx impeccable live &
    Note the port printed to stdout (auto-assigned). Use --port=PORT to fix it.
  2. Create a new tab and navigate to the page (use dev server URL for local files, or direct URL). Do not reuse existing tabs.
  3. Label the tab via javascript_tool so the user can distinguish it:
    document.title = '[Human] ' + document.title;
  4. Scroll to top to ensure the page is scrolled to the very top before injection
  5. Inject via javascript_tool (replace PORT with the port from step 1):
    const s = document.createElement('script'); s.src = 'http://localhost:PORT/detect.js'; document.head.appendChild(s);
  6. Wait 2-3 seconds for the detector to render overlays
  7. Read results from console using read_console_messages with pattern impeccable. The detector logs all findings with the [impeccable] prefix. Do NOT scroll through the page to take screenshots of the overlays.
  8. Cleanup: Stop the live server when done:
    npx impeccable live stop

For multi-view targets, inject on 3-5 representative pages. If injection fails, continue with CLI results only.

Return: CLI findings (JSON), browser console findings (if applicable), and any false positives noted.

Step 3: Generate Combined Critique Report

Synthesize both assessments into a single report. Do NOT simply concatenate. Weave the findings together, noting where the LLM review and detector agree, where the detector caught issues the LLM missed, and where detector findings are false positives.

Structure your feedback as a design director would:

Design Health Score

Consult heuristics-scoring

Present the Nielsen's 10 heuristics scores as a table:

#HeuristicScoreKey Issue
1Visibility of System Status?[specific finding or "n/a" if solid]
2Match System / Real World?
3User Control and Freedom?
4Consistency and Standards?
5Error Prevention?
6Recognition Rather Than Recall?
7Flexibility and Efficiency?
8Aesthetic and Minimalist Design?
9Error Recovery?
10Help and Documentation?
Total??/40[Rating band]

Be honest with scores. A 4 means genuinely excellent. Most real interfaces score 20-32.

Anti-Patterns Verdict

Start here. Does this look AI-generated?

LLM assessment: Your own evaluation of AI slop tells. Cover overall aesthetic feel, layout sameness, generic composition, missed opportunities for personality.

Deterministic scan: Summarize what the automated detector found, with counts and file locations. Note any additional issues the detector caught that you missed, and flag any false positives.

Visual overlays (if browser was used): Tell the user that overlays are now visible in the [Human] tab in their browser, highlighting the detected issues. Summarize what the console output reported.

Overall Impression

A brief gut reaction: what works, what doesn't, and the single biggest opportunity.

What's Working

Highlight 2-3 things done well. Be specific about why they work.

Priority Issues

The 3-5 most impactful design problems, ordered by importance.

For each issue, tag with P0-P3 severity (consult heuristics-scoring for severity definitions):

  • [P?] What: Name the problem clearly
  • Why it matters: How this hurts users or undermines goals
  • Fix: What to do about it (be concrete)
  • Suggested command: Which command could address this (from: /animate, /quieter, /shape, /optimize, /adapt, /clarify, /layout, /distill, /delight, /design-audit, /harden, /polish, /bolder, /typeset, /critique, /colorize, /overdrive)

Persona Red Flags

Consult personas

Auto-select 2-3 personas most relevant to this interface type (use the selection table in the reference). If CLAUDE.md contains a ## Design Context section from impeccable teach, also generate 1-2 project-specific personas from the audience/brand info.

For each selected persona, walk through the primary user action and list specific red flags found:

Alex (Power User): No keyboard shortcuts detected. Form requires 8 clicks for primary action. Forced modal onboarding. High abandonment risk.

Jordan (First-Timer): Icon-only nav in sidebar. Technical jargon in error messages ("404 Not Found"). No visible help. Will abandon at step 2.

Be specific. Name the exact elements and interactions that fail each persona. Don't write generic persona descriptions; write what broke for them.

Minor Observations

Quick notes on smaller issues worth addressing.

Questions to Consider

Provocative questions that might unlock better solutions:

  • "What if the primary action were more prominent?"
  • "Does this need to feel this complex?"
  • "What would a confident version of this look like?"

Remember:

  • Be direct. Vague feedback wastes everyone's time.
  • Be specific. "The submit button," not "some elements."
  • Say what's wrong AND why it matters to users.
  • Give concrete suggestions, not just "consider exploring..."
  • Prioritize ruthlessly. If everything is important, nothing is.
  • Don't soften criticism. Developers need honest feedback to ship great design.

Step 4: Ask the User

After presenting findings, use targeted questions based on what was actually found. STOP and call the AskUserQuestion tool to clarify. These answers will shape the action plan.

Ask questions along these lines (adapt to the specific findings; do NOT ask generic questions):

  1. Priority direction: Based on the issues found, ask which category matters most to the user right now. For example: "I found problems with visual hierarchy, color usage, and information overload. Which area should we tackle first?" Offer the top 2-3 issue categories as options.
  2. Design intent: If the critique found a tonal mismatch, ask whether it was intentional. For example: "The interface feels clinical and corporate. Is that the intended tone, or should it feel warmer/bolder/more playful?" Offer 2-3 tonal directions as options based on what would fix the issues found.
  3. Scope: Ask how much the user wants to take on. For example: "I found N issues. Want to address everything, or focus on the top 3?" Offer scope options like "Top 3 only", "All issues", "Critical issues only".
  4. Constraints (optional; only ask if relevant): If the findings touch many areas, ask if anything is off-limits. For example: "Should any sections stay as-is?" This prevents the plan from touching things the user considers done.

Rules for questions:

  • Every question must reference specific findings from the report. Never ask generic "who is your audience?" questions.
  • Keep it to 2-4 questions maximum. Respect the user's time.
  • Offer concrete options, not open-ended prompts.
  • If findings are straightforward (e.g., only 1-2 clear issues), skip questions and go directly to Step 5.

Step 5: Recommended Actions

After receiving the user's answers, present a prioritized action summary reflecting the user's priorities and scope from Step 4.

Action Summary

List recommended commands in priority order, based on the user's answers:

  1. /command-name: Brief description of what to fix (specific context from critique findings)
  2. /command-name: Brief description (specific context) ...

Rules for recommendations:

  • Only recommend commands from: /animate, /quieter, /shape, /optimize, /adapt, /clarify, /layout, /distill, /delight, /design-audit, /harden, /polish, /bolder, /typeset, /critique, /colorize, /overdrive
  • Order by the user's stated priorities first, then by impact
  • Each item's description should carry enough context that the command knows what to focus on
  • Map each Priority Issue to the appropriate command
  • Skip commands that would address zero issues
  • If the user chose a limited scope, only include items within that scope
  • If the user marked areas as off-limits, exclude commands that would touch those areas
  • End with /polish as the final step if any fixes were recommended

After presenting the summary, tell the user:

You can ask me to run these one at a time, all at once, or in any order you prefer.

Re-run /critique after fixes to see your score improve.

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.