Content
81%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.
A highly actionable, well-sequenced skill body with strong validation feedback loops and a genuinely useful progressive-disclosure structure. The main cost is token efficiency: core rules are duplicated across the Critical Rules, dedicated sections, and the pre-flight checklist, and the query-selector API spec is inlined where a reference file would keep the overview lean.
Suggestions
Consolidate the duplicated guidance: state each rule once (Critical Rules or the dedicated section), and reduce the Pre-Flight Checklist (Section 8) to a short pointer like "re-check Critical Rules 1-17" plus only the checklist items that add new detail (e.g. lineHeight/letterSpacing format, resize-before-sizing-modes).
Move the query() selector grammar, QueryResult method table, and editor-mode node-type lists (Sections 5 and 4) into a reference file (e.g. references/query-api.md), keeping only 2-3 high-value examples in SKILL.md.
Merge the overlapping error-recovery tables: the self-correction table in Section 7 substantially duplicates gotchas.md's WRONG/CORRECT coverage — keep the table in one place and cross-reference it from the other.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Content is mostly tool-specific knowledge Claude cannot know (atomicity, wrapper semantics, font-preload-before-appendChild), but the same rules are restated repeatedly: return-IDs appears in Critical Rule 15, Section 3, and workflow step 4; setCurrentPageAsync appears in Rule 9 and twice within Section 2; the Pre-Flight Checklist (Section 8) restates rules 1-17 nearly verbatim. Not 4: the repetition goes beyond "minor instances that could be trimmed" — consolidating the duplicated rule statements would cut a meaningful fraction of the file. | 3 / 5 |
Actionability | Fully executable throughout: copy-paste-ready JS for page iteration, query/set/auto-layout/screenshot APIs, inspection scripts, a selector-syntax spec, and an error-message→likely-cause→fix table. Not 4: examples cover the common cases with exact, runnable code and concrete fix steps, leaving no gaps. | 5 / 5 |
Workflow Clarity | Section 6 gives a clear sequenced workflow (inspect → skeleton → fill → validate → fix) with a per-step validation table, an explicit step order for complex tasks, and Section 7 adds a full feedback loop (stop → read error → fix → retry) plus atomicity guarantees. The pre-flight checklist covers batch/mutating operations. Not 4: validation checkpoints are explicit at every step, not just most. | 5 / 5 |
Progressive Disclosure | Good structure: all referenced files (gotchas.md, common-patterns.md, the pattern docs, plugin-api-standalone.index.md/.d.ts, working-with-design-systems/wwds.md) verified to exist, one level deep, with a when-to-load table (Section 10) and explicit "grep, don't load" guidance for the 455KB d.ts. Not 5: Section 5's selector grammar and QueryResult method table are reference-grade API surface inlined in SKILL.md rather than split into a reference file, making the overview heavier than ideal. | 4 / 5 |
Total | 17 / 20 Passed |