Content
81%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 highly actionable — exact Desktop Commander call patterns, per-OS command tables, and a safety section with genuine validation checkpoints for destructive operations. Its weaknesses are efficiency (re-teaching Claude cross-shell equivalents and chaining semantics it already knows) and structure (a single 185-line file where a reference bundle would allow a leaner overview).
Suggestions
Delete or drastically shrink §2's equivalents table and the 'Notes that bite people' paragraph — Claude already knows PowerShell vs bash syntax; keep only the non-obvious rule (prefer cross-platform tools like python/node/git over shell-specific syntax) and the pointer to detect the shell via get_config.
Replace §7's chaining explanation ('&& runs the next only if the previous succeeded; ; runs regardless; || runs only on failure') with the one non-obvious fact (PowerShell hosts before v7 lack &&/||) and the destructive-command chaining rule.
Move the §2 equivalents table and §4 per-CLI details into references/ files (e.g. references/shell-commands.md, references/clis.md), leaving SKILL.md a lean overview with one-level-deep, clearly signaled references.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The Desktop Commander specifics (get_config fields, start_process/interact_with_process patterns, blockedCommands) earn their tokens, but several sections re-explain what Claude already knows: the §2 bash/PowerShell/cmd equivalents table, 'Notes that bite people' about Windows backslashes, and §7's '&& runs the next only if the previous succeeded; ; runs regardless' chaining semantics. | 3 / 5 |
Actionability | Fully executable throughout: exact tool invocations ('call get_config and read systemInfo', 'start_process("python3 -i")', 'interact_with_process(pid, "...")') and copy-paste-ready per-OS commands ('lsof -i :3000', 'Get-NetTCPConnection -LocalPort 3000', 'netstat -ano | findstr :3000'), covering the common cases including the full REPL workhorse pattern. | 5 / 5 |
Workflow Clarity | Clear sequencing (detect environment → pick shell-correct command → run via the right process pattern → diagnose from real output), with validation checkpoints around destructive operations ('show the process line before terminating it', 'ls the glob before rm it', prefer dry-run first) and an explicit validate→name-cause→fix→retry feedback loop in §6, satisfying the destructive-ops validation requirement. | 5 / 5 |
Progressive Disclosure | No bundle files exist, and the body is well-sectioned with clear §-cross-references; but at ~185 lines it exceeds the simple-skill case, and reference-style material (the §2 equivalents table, §4 CLI catalog, §7 destructive list) is inlined where a references/ file would keep SKILL.md a leaner overview — good structure with minor organization gaps. | 4 / 5 |
Total | 17 / 20 Passed |