Content
82%Weight 40%Scale 1-5Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
The body is a dense, executable six-phase workflow with per-phase tool calls, confirmation gates, and templates properly externalized to reference files. Its main weaknesses are the contradictory deprecation banner versus the session-folder output rules, two referenced templates living outside the bundle, and some repetition of the session-folder rule.
Suggestions
Resolve the deprecation conflict: the top banner redirects outputs to `docs/projects/active/PI-N/sprints/SN/stories/{id}/mockups/` + `02-design-ui.md` while Phase 6 mandates `docs/ux-sessions/{SESSION_SLUG}/` — commit to one authoritative output path, or strip the legacy workflow entirely if the skill is truly absorbed by `/ux-ui` (workflow_clarity).
Vendor the two templates borrowed from ux-disruptivo (`ascii-mockup-guide.md`, `ui-spec-template.md`) into this skill's `references/`, or inline the minimal sections needed, so every referenced path resolves within the bundle (progressive_disclosure).
State the session-folder rule once: "Never write to `/tmp`, `docs/flow-specs/`, or `docs/ui-specs/`" appears twice and each Output A–D repeats a location the folder tree already declared (conciseness).
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and assumes Claude's competence — no concept explanations, just tool lists, exact Globs, and tables — with templates correctly externalized. Not 5: the session-folder rule is stated three times ("Never write to `/tmp`, `docs/flow-specs/`, or `docs/ui-specs/`" appears in the Session Folder section and again in Phase 6, and each Output A–D repeats the location the folder tree already declared), so not every token earns its place. Not 3: there is no padded explanation of things Claude already knows, only minor repetition that could be trimmed. | 4 / 5 |
Actionability | Fully copy-paste ready: exact Glob patterns ("frontend/src/app/(main)/[tenantId]/(dashboard)/**/page.tsx"), exact Read/Grep targets ("Read: frontend/src/components/shared/layout/app-sidebar.tsx"), executable bash ("mkdir -p \"docs/ux-sessions/$SESSION/prototype\"", "python3 -m http.server 8888 -d \"docs/ux-sessions/$SESSION/prototype/\""), and a concrete per-phase output format including an example ASCII journey diagram and sidebar before/after. Specific examples cover the common cases. | 5 / 5 |
Workflow Clarity | A clear 6-phase sequence with tools per phase and explicit user-confirmation gates after each phase ("**Gate:** Present the map and ask...", "**Gate:** User confirms prototype reflects what they want"), plus Edge Cases and Common Mistakes tables. Not 5: the top deprecation banner ("Outputs nuevos van a `docs/projects/active/PI-N/sprints/SN/stories/{id}/mockups/`") directly contradicts the body's session-folder rule ("All outputs land in the session folder... `docs/ux-sessions/{SESSION_SLUG}/FLOW-SPEC.md`"), leaving the authoritative output destination ambiguous mid-workflow. Not 3: sequence and validation checkpoints are otherwise complete and explicit. | 4 / 5 |
Progressive Disclosure | References are one level deep, clearly signaled, and loaded on demand at the right phase ("Load `references/journey-template.md`" in Phase 2, `navigation-patterns.md` in Phase 4, `html-prototype-system.md` in Phase 5, `flow-spec-template.md`/`ui-spec-template.md` in Phase 6), and all four in-bundle files exist with matching purpose. Not 5: two of the six referenced paths do not resolve inside this bundle ("`.claude/skills/ux-disruptivo/references/ascii-mockup-guide.md`" and `ui-spec-template.md` from ux-disruptivo), so they cannot be verified or loaded within the skill itself. Not 3: bulk material is properly split out and navigation to it is easy. | 4 / 5 |
Total | 17 / 20 Passed |