Content
56%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 skill's body has a well-sequenced batch workflow with strong validation coverage and several concrete gbrain commands, but it suffers from redundant repetition of the same safety/sample rules, vague instructions for the core generic import path, and references to bundle files that don't exist. Splitting per-source guides into reference files and adding a concrete import command for the generic path would improve it most.
Suggestions
Provide a concrete import command or tool-call example for the generic path (e.g. the put_page invocation for 'Import the vault directory into gbrain' and the CSV row-to-page loop) (actionability).
Deduplicate the sample-test and safety rules by stating them once in Contract and referencing that from Phases/Verification/Anti-Patterns (conciseness).
Add an explicit recovery step to Verification (what to do when page count mismatches or health check fails) (workflow_clarity).
Move per-source migration details (Notion, CSV, Roam) into a references/ file and fix or create the referenced paths like skills/conventions/brain-routing.md (progressive_disclosure).
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly tight tables and numbered steps, but it repeats the same guidance multiple times — the 5-10 file sample test appears in 'Contract' ('Migration is tested on a sample (5-10 files)'), 'Phases' step 3, 'Verification' step 4, and 'Anti-Patterns' — and the company-repositories section restates its safety constraints several times ('Never fix the source files...', 'Do not activate a global schema...', 'do not change the approval...'). Not the padding of the 1-2 anchors, but clearly more than the 'minor instances' of the 4 anchor. | 3 / 5 |
Actionability | Concrete, executable commands exist in places ('gbrain extract links --source db --dry-run', 'gbrain sources inspect <path> --profile company-brain --json', 'gbrain sync --brain <id> --source <id> --no-embed --no-pull'), but the generic path is instruction-level only: 'Import the vault directory into gbrain', 'For each row in the CSV, create a page', and 'Store each page in gbrain' name no tool call, command, or code for the core import operation. This mixes executable commands with missing key details, matching the 3 anchor better than 4. | 3 / 5 |
Workflow Clarity | The sequence is clear with explicit validation checkpoints for a batch operation — sample test (step 3) before bulk import, then a dedicated Verification section (page count, health check, round-trip export, spot-check, search test), satisfying the batch-operation validation requirement. It falls short of 5 because no error-recovery loop is specified (what to do when verification fails or page counts mismatch) and the generic-phases list is awkwardly prefaced by the company-repository redirect, creating some dual-path ambiguity. | 4 / 5 |
Progressive Disclosure | The file is well-sectioned with clear headers, but it references files that are not part of the bundle and do not exist ('skills/conventions/brain-routing.md', 'skills/conventions/untrusted-content.md', 'docs/guides/company-brain-ingestion.md') — no references/, scripts/, or assets/ directories are present — and per-source migration details that would fit separate reference files are all inlined in a 175-line monolith. This matches 'some structure but could be better organized' with unverifiable references, not the well-signaled, verified one-level references of a 4. | 3 / 5 |
Total | 13 / 20 Passed |