Content
92%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 skill body: executable code with exact calls, parameters, and output columns; a checkpointed workflow with explicit error-recovery loops; and a clean one-level-deep split of mobile content into a well-signaled reference file. The only flaw is minor redundancy — two critical gotchas are each stated twice, which slightly inflates token cost without adding information.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and assumes competence — no filler like explaining what pandas or Unix timestamps are — and every section carries operational facts (unit mismatches, click-counting rules, credential handling). It is not a 5 because a few facts are stated twice: the "(×N) hides burst duration" rule appears nearly verbatim in both Gotchas ("A timeline line that ends in (×N) shows the first timestamp only") and "Read a web timeline" ("The line shows the first timestamp only, so it hides how long the run lasted"), and the include_mixpanel_events default is repeated in Gotchas and again in "Correlate a tracked event with the screen". It is above the midpoint because these are duplications of genuinely load-bearing gotchas, not padded explanations. | 4 / 5 |
Actionability | Guidance is copy-paste ready throughout: runnable Python ("ws.replays_for_user('user-42', from_date='2025-01-01', to_date='2025-01-31')"), concrete CLI commands ("mp replays for-user <distinct_id> --from ... --to ..."), self-documentation look-ups ("mp help Workspace --domain 'session replay'"), and exact expected outputs (columns of rage_taps, sample timeline lines). Not a 4 because the examples cover the common cases — discovery by user, fetch by ID, filtering, aggregation, correlation — with executable code for each. | 5 / 5 |
Workflow Clarity | The five-step Workflow is clearly sequenced with built-in checkpoints (check "replay.capture" before reading any timeline; read aggregates before choosing which timelines to read), and error-recovery loops are explicit: interpreter-path failure routes to the setup skill with a bare "mp --version" fallback and a version gate ("0.4.0 or later"), and SessionReplayAccessError routes to asking the project owner for the named permission. Not a 4 because validation checkpoints and recovery loops are present, not merely implied; the destructive/batch cap does not apply since this is read-only analysis. | 5 / 5 |
Progressive Disclosure | The overview is appropriately split: web-timeline rules live in SKILL.md, and mobile/screenshot detail lives in the real one-level-deep reference, signaled twice with explicit read conditions ("read references/mobile.md now, before you read any timeline" and "Read references/mobile.md when replay.capture == \"screenshot\""). Not a 4 because the reference path is verified to exist, contains no nested references, and is triggered by a concrete programmatic condition rather than a vague 'see also'. | 5 / 5 |
Total | 19 / 20 Passed |