Content
78%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 highly actionable, domain-dense operating manual with real commands, output shapes, and validated batch-archive patterns. Its main structural weakness is that it is monolithic — the Front search API reference and sender heuristics are strong candidates for one-level-deep reference files — and it carries minor duplication and time-sensitive field notes.
Suggestions
Move the 'Front Search API Reference' section (~45 lines of filter/status/date syntax) into a references/front-search.md file and keep only the essential gotchas (no is:unread, unix timestamps, no negation, 15-filter max) inline with a clear pointer.
Deduplicate the [aih] VIP rule, which appears nearly verbatim in both 'Key Context' and 'Signals for "Reply Needed"', and consolidate it into one canonical location.
Add an explicit verification checkpoint before the fast `archive-ids` path (e.g., echo the ID list with subjects and confirm before executing), mirroring the dry-run discipline already present for archive-bulk.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Nearly every token carries non-obvious domain detail (Front search quirks, CLI flags, cache TTL, real IDs) with no explanations of things Claude already knows, but the [aih] VIP rule is duplicated nearly verbatim in two sections and the "migration to `body` underway" field note is time-sensitive churn that could be trimmed. | 4 / 5 |
Actionability | Fully executable, copy-paste-ready commands with flags, pagination, and realistic examples (e.g., `joelclaw email inbox -q "is:open is:unreplied" -n 50`, jq extraction patterns with a backwards-compatible variant), plus the exact read output shape. | 5 / 5 |
Workflow Clarity | The four-step triage workflow is clearly sequenced and batch/destructive operations have validation (archive-bulk is dry-run by default, "Always dry-run first to verify the query matches what you expect"), but the fast `archive-ids` path has no explicit verify-before-execute checkpoint and 429/rate-limit handling is advisory rather than a recovery loop. | 4 / 5 |
Progressive Disclosure | Sections are flat and well-organized with no nested references, but the ~45-line Front Search API Reference is inlined in SKILL.md instead of a separate references/ file — no bundle reference files exist at all, so content that should be separate is inline. | 3 / 5 |
Total | 16 / 20 Passed |