Content
82%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 content is a strong, highly actionable command reference with genuine safety and error-recovery guidance for an outward-facing messaging skill. Its main weaknesses are mild redundancy, time-sensitive references placed inline, and the Flagg/Executor operational detail that could be split into a reference file.
Suggestions
State the '"sent" is optimistic, not delivery-confirmed' caveat once (keep it in Delivery Notes and drop the duplicate bullet in Setup) to tighten conciseness.
Move the Flagg/Executor operational details (LaunchAgent, bearer token, health/restart commands, receipts path) into a references/flagg.md file, keeping a one-line pointer in SKILL.md to improve progressive disclosure.
Add an explicit post-send verification step for terminal-mode sends (e.g., check Messages.app for the delivery indicator before reporting success), mirroring the retry guidance already present for Flagg.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is largely efficient with every command example earning its place, but the 'sent optimistically' caveat is stated twice (Setup and Delivery Notes) and dated/time-sensitive material (ADR-0067, the 2026-09 receipts path) sits inline rather than in a deprecation-style section. | 4 / 5 |
Actionability | Every operation has a copy-paste-ready command — 'imsg chats --limit 10 --json', 'imsg history --chat-id <id> --limit 20 --attachments --json', three send variants, the brew install line, and concrete health-check commands — covering the common cases fully. | 5 / 5 |
Workflow Clarity | A safe sequence is present (read-only first, 'Always confirm recipient and message text before imsg send', 'Call imsg_status first', 'check Messages.app before retrying'), but the terminal-mode send path lacks an explicit post-send delivery-verification loop, leaving a minor validation gap. | 4 / 5 |
Progressive Disclosure | Sections are clear and well-organized with no bundle files to reference, but the dense Flagg/Executor operational block (LaunchAgent details, bearer token, health/restart commands) is inlined where it would fit naturally in a separate reference file. | 4 / 5 |
Total | 17 / 20 Passed |