Content
96%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.
An excellent operational skill body: dense, executable, and validated, with destructive-operation warnings and symptom-indexed troubleshooting. The only structural critique is that everything lives in the one file at ~87 lines, slightly over what a simple no-reference skill would be.
Suggestions
Move the step-by-step 'Empty board (0 beads)' recovery walkthrough into a references/recovery.md and keep only the confirmation steps and the 'never force-push' warning in SKILL.md, keeping the main file a lean overview.
Relocate the version-bump procedure ('change compose.yaml..., then docker compose build --no-cache scotty') to a short reference so routine run/stop usage does not load it.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Nearly every line carries repo-specific facts Claude cannot know (e.g. 'In the image node is just a symlink to bun', 'The Dockerfile creates that directory with mode 0777', 'SCOTTY_USER="$(id -u):$(id -g)" does not fix rootless Podman'). No padding or explanation of concepts Claude already knows; the one inline version number ('default 1.4.4') sits in the Versions section where version facts belong. Matches 'lean and efficient; every token earns its place'. | 5 / 5 |
Actionability | Fully copy-paste-ready commands throughout: 'bun run scotty:up', 'SCOTTY_USER=0:0 bun run scotty:up', 'chmod -R a+rX .beads', 'mv .beads/embeddeddolt ".beads/embeddeddolt.empty-$(date +%Y%m%d-%H%M%S)"', 'bun x bd bootstrap -y', 'git ls-remote origin refs/dolt/data', plus a symptom-to-fix table. This matches the 'fully executable; specific examples cover the common cases' anchor. | 5 / 5 |
Workflow Clarity | The 'Empty board' recovery is a numbered sequence with explicit validation checkpoints ('bun x bd stats must show a non-zero total', 'a SHA means history exists'), a fallback path for divergent histories, and a destructive-operation guard ('Never bd dolt push --force'). Quick diagnosis gives symptom-to-fix feedback loops. This matches the top anchor: clear sequence, explicit validation, error-recovery guidance. | 5 / 5 |
Progressive Disclosure | The single file is well organized with clear section headers and tables, and there are no nested or orphaned references (no bundle files exist). However, at ~87 lines it exceeds the under-50-line simple-skill case, and some operational detail (e.g. the version-bump procedure and the full recovery walkthrough) is inlined that could live in a reference file, matching 'good structure; most content appropriately placed; minor organization gaps' rather than a clearly split overview. | 4 / 5 |
Total | 19 / 20 Passed |