Content
67%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 well-structured and largely actionable, with a clearly sequenced workflow mapped to real scripts and meaningful validation for the batch-send operation. Its weaknesses are redundancy across three overlapping workflow descriptions, a broken script reference (send_brief.sh vs. send_slack.sh), and an undocumented bundle script (format_for_slack.sh).
Suggestions
Collapse 'What It Does', 'How It Works', and 'Research Process' into a single workflow section — they describe the same pipeline three times and account for most of the excess tokens.
Fix the Manual Usage example to reference the real script (scripts/send_slack.sh with its actual arguments '<person_name> <brief_text_file> <webhook_url>') and document or remove the unreferenced format_for_slack.sh.
Move the example brief and the research-output JSON schema into a references/ file (e.g. references/brief-format.md), keeping only a short inline sample in SKILL.md.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly information-dense (config keys, commands, JSON schemas), but the same workflow is described three times — 'What It Does' (steps 1-6), 'How It Works / Main Workflow' (steps 1-7), and 'Research Process' — which is unnecessary duplication, and the 30-line example brief plus the Privacy & Security section pad further. This matches anchor 3 ('mostly efficient but includes some unnecessary explanation or could be tightened'); not 2 because most sections carry concrete, non-redundant content. | 3 / 5 |
Actionability | Guidance is mostly executable: './scripts/run_daily.sh', 'node scripts/research_person.js "Jane Doe" "jane@example.com" "Example Corp"', 'gcalcli agenda today tomorrow', and a complete config.json block. Not 5 because the Manual Usage section invokes './scripts/send_brief.sh brief.html "Jane Doe"' — a script that does not exist in the bundle (the actual file is send_slack.sh with a different argument order) — and 'research_output.json meeting_context.json' are referenced without showing how to produce them. | 4 / 5 |
Workflow Clarity | The 7-step main workflow is clearly numbered with each step mapped to a specific script, and the batch-send risk (1 email per person) is covered by checkpoints: DRY_RUN preview, data/sent duplicate tracking, logs/run.log, and a Troubleshooting section keyed to failure modes. This matches anchor 4 ('clear sequence with most checkpoints present; minor validation gaps'); not 5 because validation (dry-run, sent-log check) lives in Tips/Troubleshooting rather than being explicit checkpoints inside the step sequence itself, and not 3 because feedback paths for the batch operation are genuinely present. | 4 / 5 |
Progressive Disclosure | Structure is good: well-labeled sections, all six bundle scripts live one level deep in scripts/ and most are clearly referenced from the workflow steps, matching the 'good structure; references mostly clear; minor organization gaps' anchor. Not 5 because of real bundle-structure mismatches: send_brief.sh is referenced but doesn't exist, format_for_slack.sh exists but is never mentioned, and the example brief and research-output JSON schema (~70 lines combined) are inlined where a references/ file could carry them. | 4 / 5 |
Total | 15 / 20 Passed |