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 high-caliber skill body: a seven-step read-only workflow with executable commands, concrete error triage, cross-validation between sampled data and code inventory, and an unusually honest 'Limits of the sample' disclosure section that separates 'correct' from 'fits the sample'. Progressive disclosure is textbook — catalogues live in verified one-level-deep references. The only deduction is minor verbosity from deliberate repetition (the legacy-flag ban is stated three times) and a few wordy asides.
Suggestions
State the legacy-layout-flag prohibition once (e.g., in the Output format section, linking to mapping-keys.md) instead of three times across the Analysis checklist, Output format, and reference pointer — the repetition costs tokens without adding enforcement.
Consolidate the 'one mapping serves both the live syncer and the bulk exporter' point, which currently appears in the intro, the description, and mapping-keys.md, into a single statement.
Tighten the `<skill-dir>` resolution and `!`-prefix explanations by a sentence each — the rule (resolve to an absolute path; the value must bypass the transcript) can be stated in one line apiece.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is efficient and assumes competence — it never explains generic concepts (JWT, REST, JSON) and spends its tokens on domain facts Claude cannot know (endpoint semantics, `slices` crowding, key-direction quirks like name/image reading 'v3-key-to-v2-source, backwards from every other string key'). A few spots could be trimmed: the legacy-layout-flag prohibition is stated three times (Analysis checklist, Output format, and again via mapping-keys.md), 'one mapping serves both paths' recurs in the intro and elsewhere, and the `<skill-dir>`/`!`-prefix digressions run a sentence longer than needed. This sits above anchor 3 ('mostly efficient but some unnecessary explanation') but short of anchor 5 ('every token earns its place') because of that deliberate repetition and a few wordy asides. | 4 / 5 |
Actionability | Guidance is fully executable: copy-paste commands (`python3 <skill-dir>/scripts/fetch_sample.py --out v3sync-sample.json`, the `! export ...` secret handshake, `test -n "$STREAM_API_SECRET" && echo SECRET_SET || echo SECRET_MISSING`, `scripts/scan_v2_calls.py --root <repo> --out v2-code-scan.json`, `git log -S'<field>'`), concrete error triage (401/403 = bad secret or wrong token type; 404 = endpoints not deployed; empty activities vs. reactions proving data exists), a ready-to-paste output JSON, and placeholder resolution instructions for `<skill-dir>` including the two possible install paths. This matches anchor 5 ('Fully executable; copy-paste ready code or commands; specific examples cover the common cases'). | 5 / 5 |
Workflow Clarity | A numbered seven-step workflow with an explicit analysis checklist, explicit validation checkpoints (cross-check the code inventory against the sample in step 4; 'Before finalizing, check each key you emitted against the worked examples'), and feedback loops for error recovery (re-run the sample to firm up single-row decisions, `--feed` re-reads for under-covered feed groups, per-feed errors recorded without stopping the run). The operation is read-only, so the destructive-batch cap does not apply. This matches anchor 5 ('Clear sequence with explicit validation steps; feedback loops for error recovery; checklists for complex processes'). | 5 / 5 |
Progressive Disclosure | Structure is exemplary: the body holds the workflow and decision checklist while heavy catalogues are offloaded one level deep to real, clearly signaled files — references/mapping-keys.md (verified to exist, holds the full key catalogue so the body can say 'do not guess key names from memory'), references/code-inventory.md, references/porting-plan.md, plus scripts/fetch_sample.py and scripts/scan_v2_calls.py, both present. References are linked at their point of use, and pack-level files (../stream/RULES.md, peers.yaml) are flagged up front. This matches anchor 5 ('Clear overview with well-signaled one-level-deep references; content appropriately split; easy navigation'). | 5 / 5 |
Total | 19 / 20 Passed |