Content
78%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.
A tight, well-structured reference skill: high signal density, concrete commands, situation-to-action tables, and explicit safety constraints. The main improvements are sequencing the home-deployment flow with validation checkpoints and trimming the few remaining asides.
Suggestions
Sequence the live-$HOME promotion flow explicitly (merge/check out in ~/dev/dotfiles -> just home-deploy <ref> -> just home-live) with a verification step such as confirming deployed symlinks.
Provide path-key derivation as a ready-to-run command (e.g. a git rev-parse + tr pipeline) instead of only a prose rule.
Trim the announce-at-start line and long parentheticals like the bin/vi neovim dispatch note to reclaim tokens.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and environment-specific (path keys, mise dot deployment, hook behavior) and assumes git competence without re-explanding basics, so nearly every token earns its place. It is not a 5 because of minor trims available: the announce-at-start line and several long parenthetical asides (e.g. the bin/vi dispatch explanation). | 4 / 5 |
Actionability | Guidance is concrete and copy-paste ready in the key places: "git rev-parse --path-format=absolute --git-common-dir", "just home-deploy <ref>", "just home-live", "wt shell <branch>", plus an explicit path-key rule and worked table examples. Not 5 because path-key computation is described but not given as a command, and the core create/switch/delete procedure is delegated to another skill. | 4 / 5 |
Workflow Clarity | The Quick Reference situation-to-action table and the Never list act as clear decision checkpoints, and the detection step ("Resolve via git rev-parse ... not --show-toplevel") is an explicit verification method with rationale. Not 5 because the "promote changes to live $HOME" flow (merge/checkout, home-deploy, home-live) is described but not sequenced with validation checkpoints. | 4 / 5 |
Progressive Disclosure | This is a single-purpose skill with no bundle files; the ~60-line body is well-organized into clearly headed sections (Core Convention, Dotfiles Repo, Quick Reference, Never, Integration) with no content that belongs in a separate file. The only pointers are clearly signaled cross-skill references (git-worktree, branch-finish) and repo-internal spec paths, all one level deep. | 5 / 5 |
Total | 17 / 20 Passed |