Content
78%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 lean, well-sequenced review procedure with concrete file-level guidance and clear conditionals. The main weakness is progressive disclosure: the body points to numerous external files (plays/, data/masvs/, data/mastg/, templates/) that are not shipped in the bundle, so the promised navigation breaks.
Suggestions
Ship the referenced files (plays/mobile-code-review.md, the data/masvs/ and data/mastg/ datasets, templates/finding.md) inside the skill bundle, or move them under references/ so the body's navigation resolves to real files.
Include one concrete inline example of a finding (e.g. a sample templates/finding.md entry) so the output format is unambiguous without relying on an external template.
Add an explicit validation/verification checkpoint in the workflow — e.g. 'confirm every MASTG test ID cited in a finding exists in data/mastg/ before finalizing' — to give the procedure a feedback loop.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and operational with no padding or explanation of concepts Claude already knows (no 'what is MASVS' preamble); every line carries procedure, file paths, or output spec, matching the 'lean and efficient; every token earns its place' anchor. | 5 / 5 |
Actionability | Gives concrete, executable guidance — specific fingerprint targets (AndroidManifest.xml, build.gradle, Info.plist, *.xcodeproj), exact file paths to load (data/masvs/MASVS-<GROUP>-<N>.md), and a finding template; falls short of 5 because no example finding is shown inline and the MASTG cross-ref grouping rule is specified only abstractly. | 4 / 5 |
Workflow Clarity | A clear 5-step sequence (Scope → Platform Detection → Systematic Review → Diff Analysis → Produce Findings) with explicit conditional checkpoints ('declare partial coverage', 'verify pinning, permissions ... are not weakened', dedup rule); not 5 because there is no explicit validate→fix→retry feedback loop, though this read-only review skill is not destructive. | 4 / 5 |
Progressive Disclosure | The body is well-structured as an overview and signals one-level-deep references with backticks (plays/mobile-code-review.md, data/masvs/, data/mastg/, templates/finding.md), but none of these referenced files are present in the skill's bundle (no references/scripts/assets dirs, and plays/data/templates do not exist), so the navigation it promises does not resolve. | 3 / 5 |
Total | 16 / 20 Passed |