Content
96%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.
This is an exemplary operational skill body: dense with hard-won failure knowledge, fully copy-paste-ready, and sequenced with explicit validation checkpoints and recovery guidance. The only structural improvement available is splitting the licensing and provenance detail into a reference file to slim the main document.
Suggestions
Split the licensing section (order id, 1Password vault location, About-panel guidance) and the provenance caveats for the pre-made dyld exports into a reference file (e.g. references/licensing.md), keeping a one-line pointer in SKILL.md.
Trim personal-context specifics (e.g. the full order id) out of the main body and into that reference to keep the core query workflow front and center.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Nearly every line encodes non-obvious operational knowledge Claude could not infer ('mcporter accepts an unknown --params flag silently', 'preferences domain com.cryptic-apps.hopper-web-4', 'perform action "AXPress" ... never a synthetic click', 'document names carry no .hop extension'). There is no padding and no explanation of concepts Claude already knows; every token earns its place. | 5 / 5 |
Actionability | The body is fully executable throughout: the mcporter config-add command with exact binary path, --args call examples with literal JSON payloads, a raw stdio JSON-RPC heredoc for isolating transport faults, an import poll loop, an osascript AXPress snippet, and a daemon restart. Copy-paste-ready commands cover the common cases plus failure modes. | 5 / 5 |
Workflow Clarity | The 'Query workflow' is explicitly numbered with a validation checkpoint ('Validate the hypothesis with LLDB/logging/repro before editing app code'), and the body adds verification after setup, poll-instead-of-sleep on long imports, a control-run on a second OS before concluding 'regression', and a failure-handling section with recovery loops (retry after checking windows, restart daemon over process kills). No destructive or batch operations apply a cap. | 5 / 5 |
Progressive Disclosure | No bundle files exist (no references/, scripts/, or assets/), and the body is well-sectioned with clearly signaled external resources ('$one-password', '$remote-mac', '~/Library/CloudStorage/Dropbox/Hopper/'). It is not 5 because at ~115 lines it exceeds the simple-skill (<50 line) exception and content that could live in a separate file (licensing specifics, provenance caveats for the pre-made exports) is inlined; it is not 3 because structure is good and nothing is buried. | 4 / 5 |
Total | 19 / 20 Passed |