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 highly effective skill body: executable code throughout, a gated workflow with error recovery, non-obvious gotchas (unit mismatches, join defaults, credential handling), and a well-signaled one-level-deep reference for mobile recordings. The only weakness is minor duplication and a few trimmable sentences that cost tokens without adding guidance.
Suggestions
State the (×N) first-timestamp-only caveat once (in "Read a web timeline") and drop the duplicate bullet from "Gotchas", keeping only the pointer to actions_df/rage_clicks()/rage_taps().
Merge "A denial of some other command does not mean Bash is blocked. Still try the bare `mp --version`." into the preceding fallback step, since it restates guidance the numbered list already implies.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and nearly every sentence carries non-obvious, library-specific knowledge (unit mismatches, fetch limits, credential masking, join defaults). It falls short of 5 due to minor duplication and padding: the (×N) first-timestamp-only caveat is stated twice verbatim ("Gotchas" and "Read a web timeline"), and a few sentences (e.g. "A denial of some other command does not mean Bash is blocked") restate what a prior sentence implies. Not 3: the over-explanation is minor and localized, not a pattern. | 4 / 5 |
Actionability | Fully executable, copy-paste-ready code with real method calls, parameter names, and expected outputs (e.g. `ws.replays_for_user("user-42", from_date=..., to_date=...)`, `print(bundle.top_clicks(10))`), plus exact CLI commands and self-documentation lookups (`mp help Workspace.replays_for_user`). The examples cover the common cases: fetch by user, by replay ID, correlate events, aggregate. Not 4: no gaps in concrete guidance. | 5 / 5 |
Workflow Clarity | The numbered Workflow gives a clear sequence with explicit gates (check `replay.capture` before reading any timeline; read aggregates before timelines) and fallback branching (no distinct_id → find candidate users via mixpanelyst first), and the "Run code" section contains a full error-recovery loop (interpreter path fails → ask user to run setup → bare `mp --version` probe → version check before trusting lookups). Not 4: checkpoints and recovery paths are explicit, not implicit. | 5 / 5 |
Progressive Disclosure | The SKILL.md is a working overview of the common web path, with exactly one clearly signaled, condition-gated, one-level-deep reference ([references/mobile.md](references/mobile.md), a real 128-line file) that is pointed to in two places (workflow step 2 and the closing section), plus an external tutorial via a single WebFetch URL. Mobile-specific detail is correctly split out. Not 4: the split and navigation are clean, with no inlined content that belongs in the reference. | 5 / 5 |
Total | 19 / 20 Passed |