Content
88%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 dense, command-first skill body with genuinely executable guidance, strong preflight/fallback validation loops, and correct delegation of report generation to bundled scripts. The main improvements are trimming long inline shell scaffolding and splitting per-tool interpretation detail into reference files.
Suggestions
Move the multi-line shell scaffolding (the trace-file stable-size polling loop, the ARTIFACT_DIR setup) into the bundled scripts so SKILL.md keeps one-line invocations, reducing token cost without losing actionability.
Extract the 'Reading Simpleperf' interpretation guide and the Perfetto inspection checklist into references/ files (e.g., references/interpreting-simpleperf.md, referenced one level deep), keeping SKILL.md as the trace-type decision and capture overview.
Consolidate the repeated 'wait until the output file exists and its size is stable' warnings into one stated invariant applied to all trace pulls.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Almost every section is commands plus terse, domain-specific caveats (sampled-vs-wall-clock limits, heapprofd buffer health) that Claude would not reliably know, so it is not verbose in the rubric's sense. Not 5 because some blocks could be tightened — e.g., the ~20-line inline file-size polling loop and repeated warnings about pulling stable trace files. | 4 / 5 |
Actionability | Fully executable copy-paste bash for every capture path: simpleperf record/stop/pull, the bundled report scripts with arguments, Perfetto background + flush-wait + pull, gfxinfo reset/framestats, am dumpheap + shark-cli, and a complete heapprofd config. Placeholders (SERIAL, PACKAGE, ARTIFACT_DIR, SKILL_DIR) are all defined, and the '...when supported by the installed Simpleperf' hedges are explicitly justified tool-availability notes, not pseudocode. | 5 / 5 |
Workflow Clarity | The 5-step Core Workflow is sequenced per tool with explicit validation checkpoints and recovery loops: preflight 'dumpsys package ... grep DEBUGGABLE|profileable', the 'Operation not permitted' → Ctrl-C fallback, 'discard that capture' rule for gfxinfo on ANR screens, wait-for-stable-size before pulling traces, and the final Report checklist with caveats and next-step guidance. Not 4: checkpoints are explicit, not implicit. | 5 / 5 |
Progressive Disclosure | Scored against the actual bundle (scripts/simpleperf_hotspots.sh and scripts/heapprofd_reports.sh — both exist and are referenced with full invocation lines and expected outputs), and the cross-skill pointer to ../android-emulator-qa/SKILL.md is clearly signaled. Not 5: the SKILL.md carries ~280 lines with all per-tool interpretation detail inline; the 'Reading Simpleperf' guidance and Perfetto inspection notes are candidates for references/ files, and there is no references/ tier at all. | 4 / 5 |
Total | 18 / 20 Passed |