Content
75%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.
This is a strong, highly actionable instruction skill: exact commands with flags, a fixed output template, and an unusually good failure-handling section. Its main weaknesses are redundancy (citation, read-only, and meeting-loading rules each stated multiple times) and a structural overlap between the Phases and GBrain-Native Context Loading sections that muddies execution order.
Suggestions
Deduplicate the citation, read-only, and gap-flagging rules: state each once in the Contract and reference it from Anti-Patterns instead of restating it.
Merge Phases step 1 with the 'Before a meeting' subsection so there is a single participant-resolution procedure, and remove or relocate the out-of-scope 'Before an email reply' section.
Clarify execution order by placing 'GBrain-Native Context Loading' before the Phases section (or explicitly numbering it within the workflow) since it claims to run 'before generating any briefing'.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is skill-specific with no padding about concepts Claude already knows, but it repeats itself: the citation rule appears in the Contract ('Every fact... includes an inline [Source: slug, updated DATE] citation'), again in 'Citation in Briefings', and again in Anti-Patterns; the read-only rule is stated in the Contract, Anti-Patterns, and Back-Linking sections; and meeting-context loading is covered twice (Phases step 1 vs. 'GBrain-Native Context Loading'). The 'Before an email reply' subsection is out of scope for a briefing skill. This fits 'mostly efficient but includes some unnecessary explanation or could be tightened' rather than the 4 anchor, where only minor trimming would be needed. | 3 / 5 |
Actionability | The guidance is fully executable with copy-paste-ready commands and exact flags ('gbrain salience --days 7', 'gbrain recall --since-last-run --supersessions --pending --rollup --json', 'gbrain whois <email> --json', 'gbrain waiting --json') plus concrete query strings and a complete output format template. Specific examples cover the common cases (meeting prep, deals, open loops, people in play), matching the 'fully executable, copy-paste ready, covers common cases' anchor. | 5 / 5 |
Workflow Clarity | The sequence is clear (pre-briefing pulls 0a-0e, then six phases, then a fixed output format) and the 'When it fails' section provides genuine error-recovery feedback loops (retry after named sync, degrade notices, ship partial briefing and name the missing section), with the exact-match identity rule acting as a validation checkpoint. It falls short of 5 because ordering is ambiguous: 'GBrain-Native Context Loading' says 'Before generating any briefing' yet appears after the Phases section, which already gives its own, weaker participant-search steps ('Search gbrain for each participant by name'), leaving it unclear which procedure governs. | 4 / 5 |
Progressive Disclosure | Good single-file structure with clear sections and well-signaled, one-level-deep references to external rule files ('skills/_brain-filing-rules.md', '../conventions/brain-first.md#identity-exact-match', '../conventions/agent-operator-protocol.md', 'skills/google-loops/SKILL.md'); no bundle directories (references/, scripts/, assets/) exist, so everything lives in SKILL.md. It is not a 5 because at ~200 lines some content (the detailed gbrain command reference and failure-mode handling) could be split out, and the referenced external files cannot be verified within this bundle. | 4 / 5 |
Total | 16 / 20 Passed |