Content
65%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 actionable CLI reference with excellent copy-paste commands and honest documentation of known failure modes. It is held back by an inline config block duplicating the reference file, a broken reference to a nonexistent message-composition.md, and missing validation steps around destructive message operations.
Suggestions
Remove the ~50-line inline TOML example (it duplicates references/configuration.md) and replace it with a one-line pointer plus the wizard command.
Fix or remove the reference to references/message-composition.md in the Tips section, since that file does not exist in the bundle.
Add verification guidance around destructive operations, e.g. confirm the envelope ID by re-listing before "message delete"/"message move", and note how to undo or check the destination folder afterward.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly efficient command-focused content, but the ~50-line inline TOML config block substantially duplicates references/configuration.md and 163-mail-specific notes are scattered across three separate sections, so it could be noticeably tightened. | 3 / 5 |
Actionability | Every operation ships copy-paste-ready commands, including the non-interactive pipeline "export EDITOR=cat; himalaya template write -H ... | himalaya template send", a fully executable Python smtplib attachment example, and warnings quoting exact error messages ("cannot parse MML message: empty body"). | 5 / 5 |
Workflow Clarity | Prerequisites and a setup sequence are present with useful error-recovery notes, but destructive operations ("himalaya message delete 42", "himalaya message move 42") lack any validation or verification steps, capping workflow clarity at 3. | 3 / 5 |
Progressive Disclosure | Sections are well organized and references/configuration.md is a real one-level-deep reference, but the body cites references/message-composition.md which does not exist in the bundle, and configuration content that belongs in the reference file is duplicated inline in SKILL.md. | 3 / 5 |
Total | 14 / 20 Passed |