Content
80%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.
The body is a well-structured, command-dense reference with excellent progressive disclosure into two real, accurately described reference files. Its weaknesses are the missing validation/verification steps around destructive and batch mail operations and the irrelevant PilotDeck migration metadata that pads the file.
Suggestions
Add verification checkpoints after destructive or batch operations, e.g. after "himalaya message move <id> <folder>" re-run "himalaya envelope list" to confirm the message landed correctly.
Remove or relocate the "PilotDeck Migration Note" section; it is review-process metadata, not skill content, and wastes context tokens.
Clarify the search syntax (e.g. that "from alice@example.com subject invoice" are filter arguments to "envelope list") so the one search example is unambiguous.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and command-focused with minimal prose ("Use `himalaya` for IMAP/SMTP email from shell"), but the "PilotDeck Migration Note" section is review/migration metadata that does not help execute the skill and could be trimmed, fitting the efficient-with-minor-excess (4) anchor rather than every-token-earns-its-place (5). | 4 / 5 |
Actionability | Every section provides copy-paste-ready commands (e.g. "himalaya envelope list", "himalaya message reply <id>", "himalaya flag add <id> --flag seen") covering the common mail cases (list, read, search, compose, reply, forward, copy, move, delete, flag), matching the fully-executable anchor. | 5 / 5 |
Workflow Clarity | Sections are sensibly sequenced (Setup → Read/search → Write → Organize) and a Safety section says "Confirm before sending, deleting, or moving many messages", but destructive and batch operations (delete, move, copy) lack validation/verification checkpoints (e.g. re-listing the folder to confirm the move), which caps workflow clarity at 3 per the rubric guidelines. | 3 / 5 |
Progressive Disclosure | The body is a concise overview with two clearly signaled, one-level-deep references ("references/configuration.md": account config, auth, backend setup; "references/message-composition.md": MML compose syntax), both of which exist on disk and match their descriptions; detailed MML content is correctly deferred ("read `references/message-composition.md` first"). | 5 / 5 |
Total | 17 / 20 Passed |