Content
76%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-organized, highly actionable command reference: nearly every section is executable copy-paste code with agent-specific guidance (PTY notes, JSON output, piping vs $EDITOR). Main gaps are the absent validation checkpoints around destructive operations (delete/move) and duplication between the inline configuration walkthrough and references/configuration.md.
Suggestions
Add validation checkpoints before destructive/batch operations — e.g. 'Read the message (himalaya message read <id>) to confirm the right ID before deleting' and verify folder names with 'himalaya folder list' before move/copy — which would lift workflow_clarity above the destructive-operation cap of 3.
Trim the inline config.toml to a minimal quick-start stub and delegate the full example, alias warning, and provider configs to references/configuration.md to remove the near-verbatim duplication.
Cut the opening paragraph that restates the frontmatter description, keeping only the Hermes-gateway disambiguation sentence.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dominated by lean, copy-paste command blocks, and prose sections carry non-obvious, high-value information (Hermes disambiguation, the v1.2.0 alias pitfall, argument ordering). However, the full config.toml block and the ~10-line alias warning are duplicated nearly verbatim in references/configuration.md, and the opening paragraph repeats the frontmatter — matching anchor 4 ('Efficient; minor instances of over-explanation that could be trimmed'). Not anchor 5 because that duplication could be trimmed; not anchor 3 because there is no padding or explanation of concepts Claude already knows. | 4 / 5 |
Actionability | Every section provides fully executable, copy-paste-ready commands covering the common cases: installation (three routes), configuration, list/search/read, reply/forward/compose (both piped and interactive), move/copy/delete, flags, attachments, multiple accounts, output formats, and debugging. This matches anchor 5 ('Fully executable; copy-paste ready code or commands; specific examples cover the common cases'). | 5 / 5 |
Workflow Clarity | The setup sequence is clear (verify install → configure account → credentials), but the skill's destructive operations ('himalaya message delete 42', 'himalaya message move') and send have no validation checkpoints — no confirm-the-ID-before-delete step and no post-operation verification. Per the judging guidelines, missing validation for destructive/batch operations caps workflow clarity at 3 ('Steps listed but validation gaps; sequence present but checkpoints missing or implicit'). | 3 / 5 |
Progressive Disclosure | A dedicated References section lists both bundle files with a parenthetical of what each covers, both referenced paths exist (references/configuration.md, references/message-composition.md), they are one level deep, and they are cross-linked inline (config comment and Tips). Minor organization gap: the Configuration Setup section duplicates a full config.toml and the alias warning already detailed in references/configuration.md — matching anchor 4 ('Good structure; most content is appropriately placed; references mostly clear; minor organization gaps'). Not anchor 5 because content is not cleanly split — the duplication blurs the overview/reference boundary. | 4 / 5 |
Total | 16 / 20 Passed |