Content
73%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 strong, expert-level skill body: an explicitly sequenced workflow with real validation checkpoints, concrete executable commands, specific issue taxonomies with guideline citations, and a copy-paste report template. Its weaknesses are moderate redundancy (a duplicate no-issues report template and five near-identical invocation examples), inlined time-sensitive details that will age, and one cross-skill reference that is not resolvable within this bundle.
Suggestions
Replace the full "When No Issues Found" template with a short instruction (e.g. "If no issues: use the same report with zero issue counts and the 'Verified' checklist") to remove ~20 redundant lines.
Collapse the five invocation examples to two or three distinct phrasings, and trim the duplicate Device Coverage section/table in the report template to one representation.
Move the accepted-sizes table to references/screenshot-sizes.md (read on demand) and either bundle the device-control status-bar guidance or drop the dangling axiom-tools file pointer; keep the April 2027 / iOS 27.1 SDK requirement note with an explicit 'current as of' qualifier.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The core content is dense, high-value domain knowledge (exact pixel sizes, false-positive list, guideline citations), but there is noticeable tightening opportunity: the full second template under "When No Issues Found" duplicates the main report structure, the main template contains both a "Device Coverage" section and a "Device Coverage Summary" table, there are five near-identical invocation examples, and time-sensitive specifics ("required from April 2027 for apps using the iOS 27.1 SDK or later", model-year lists) are inlined without a deprecation note. This exceeds anchor 4's 'minor instances of over-explanation' but is far from anchor 2's padded conceptual explanation. | 3 / 5 |
Actionability | The guidance is highly concrete — an executable sips dimension-check command, specific placeholder strings ("Lorem ipsum", "STAGING", "example@email.com"), a complete markdown report template, and per-issue What/Why/Fix structure. It misses anchor 5 because the bash loop uses placeholder filenames ("<file1>", "<file2>") requiring substitution rather than being copy-paste ready, and sips is macOS-only with no stated fallback — minor gaps characteristic of anchor 4. | 4 / 5 |
Workflow Clarity | Steps 1–5 are explicitly sequenced with genuine validation checkpoints throughout: "Do not proceed without a folder path", "If 0 images found, report and stop", a 20-image cap with user notification, uncertainty handling ("flag it as MEDIUM (not CRITICAL) with a note"), and a feedback loop ("Re-run validation" after fixes). Checklists exist in the form of the CRITICAL/HIGH/MEDIUM taxonomy and the false-positive list. The batch-operation cap does not apply because verification is embedded at every stage, matching anchor 5 rather than anchor 4. | 5 / 5 |
Progressive Disclosure | The body is well-organized with clear section headers, and its one external reference is clearly signaled ("see axiom-tools (`skills/device-control-ref.md`, Status bar for screenshots)"). It falls short of anchor 5 because that referenced file is not part of this bundle (no references/ directory exists, so the pointer cannot be resolved locally), and the ~15-line accepted-sizes table is borderline content that could live in a reference file. It is above anchor 3 since the structure is good and the reference is not buried. | 4 / 5 |
Total | 16 / 20 Passed |