Content
67%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, rule-driven overview with concrete tools, commands, and a clear workflow including validation and recovery, scoring well on actionability and workflow clarity. Its weak spots are conciseness (redundant restatements and emphatic padding) and progressive disclosure (heavily referenced resource files are absent from the bundle).
Suggestions
De-duplicate the 'components/ui/* is read-only' rule — state it once authoritatively in Guardrails and reference it elsewhere rather than repeating it three times.
Trim the dramatic emphasis in the proxy.ts and prefetch guardrails to a single crisp directive plus the rationale; drop stacked NEVER/restatement padding.
Either ship the referenced resources/*.md, ../_shared/core/*, and ../../rules/frontend.md files in the bundle, or mark which references are resolved by the parent project at runtime so progressive disclosure is verifiable.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly efficient and assumes Claude already knows React/Next.js/Tailwind, but it carries real redundancy — 'treat components/ui/* as read-only' is stated in Guardrails #2, the UI Implementation section, and the closing blockquote — and dramatic padding in the proxy.ts guardrail ('touch it and you die', multiple stacked NEVER clauses) that could be tightened. | 3 / 5 |
Actionability | It gives concrete executable guidance: real `rg` commands, named shadcn registry tools (`shadcn_search_items_in_registries`, `shadcn_get_add_command_for_items`), specific file paths (`packages/design-tokens`, `components/ui/*`), pinned library versions, and explicit directives like `prefetch={false}`; the gap is that actual component/TSX code is deferred to resources rather than shown inline. | 4 / 5 |
Workflow Clarity | A clear Entry → Scenes (PREPARE/ACQUIRE/ACT/VERIFY/FINALIZE) → Exit sequence is present with an explicit VERIFY checkpoint and feedback loops in 'Failure and recovery' plus a referenced checklist, so the destructive-codebase edits do not hit the missing-validation cap; it stops short of anchor 5 because concrete validation commands are referenced rather than stated inline. | 4 / 5 |
Progressive Disclosure | Content is well structured with a Stack Reference table and a dedicated References section giving one-level-deep, clearly signaled pointers to tech-stack/tailwind-rules/snippets/angular-rules/execution-protocol/checklist; it does not reach anchor 5 because none of the referenced resource files actually ship in this skill bundle, so the navigation cannot be verified against real bundle content. | 4 / 5 |
Total | 15 / 20 Passed |