Content
63%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 well-structured, largely actionable instruction skill with clear phases, concrete gbrain commands, exact formats, and thoughtful failure handling for a consent-sensitive ambient process. Its main weaknesses are redundant restatement of the authorization rules across five sections, undefined notability criteria, and dangling external references that break navigation.
Suggestions
Consolidate the consent/opt-in rules into a single section — they are currently repeated in 'Contract', 'Enablement before capture', 'Per-User Storage Policy', 'When it fails', and 'Anti-Patterns' — and replace the slogan lines ('An unlinked mention is a broken brain') with the operative rule only.
Fix or inline the dangling references: neither 'skills/conventions/quality.md' nor '../conventions/agent-operator-protocol.md' exists in this bundle; either ship them or inline the relevant error-handling steps.
Define the notability criteria for entity creation (Phase 2 currently says only 'If notable, save the supplied information') and specify the page structure to write, so the capture steps are fully executable.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient with concrete commands, but the consent/authorization rules are restated across 'Contract', 'Enablement before capture', 'Per-User Storage Policy', 'When it fails', and 'Anti-Patterns', and slogan lines ('Ideas are the intellectual capital. Entities are bookkeeping.', 'An unlinked mention is a broken brain.') pad the token budget. Matches 'mostly efficient but includes some unnecessary explanation or could be tightened' — the padding is repetition rather than explanation of known concepts, so above 2 but below 4. | 3 / 5 |
Actionability | Concrete executable guidance throughout: 'gbrain search "name"', 'gbrain timeline-add <slug> <date> "<summary>"', exact back-link format '- **YYYY-MM-DD** | Referenced in [page title](path) — brief context', page paths like 'originals/{slug}', and signal-log line formats. Minor gaps keep it from 5: notability criteria are undefined ('If notable, save...') and the structure of pages to write is unspecified. | 4 / 5 |
Workflow Clarity | Phases are clearly sequenced (idea detection → entity handling with search/thin/rich branches → signal logging) with skip conditions, explicit failure handling ('write_pending ... exit 10 ... poll the receipt or re-read and merge on the next turn'), and a readback verification checkpoint ('verify captured content with an actual readback'). Not 5: validation is mentioned but not woven into an explicit fix-and-retry loop; not 3: checkpoints and error recovery are explicit. | 4 / 5 |
Progressive Disclosure | Well-sectioned single file, but both referenced paths ('skills/conventions/quality.md', '../conventions/agent-operator-protocol.md') are dangling — neither exists in this bundle — and the Iron Law back-link procedure and the long 'memory.auto_writeback'/Stop-hook paragraph are inlined detail that belongs in reference files. Some structure and organization, but broken references and inline bulk prevent a 4. | 3 / 5 |
Total | 14 / 20 Passed |