Content
86%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 efficient, well-structured body for a simple tool-listing skill: lean prose, concrete tool inventory, executable config example, and a useful error-recovery tip. The only meaningful gap is the absence of a worked tool-call example and an explicit prepare_rename-then-rename safety sequence, which keep actionability and workflow clarity just below top marks.
Suggestions
Add one concrete example tool call showing argument shape, e.g., a sample lsp.diagnostics or lsp.goto_definition invocation with file/line/character values.
Make the rename safety sequence explicit: 'Before lsp.rename, call lsp.prepare_rename at the position; only apply the rename if it reports valid.'
Note how to configure a new language server (edit .codex/lsp-client.json, then re-run lsp.status) so the status-tip recovery loop is complete.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~31-line body is lean with zero padding: a one-line tool-invocation caveat ('lsp.*/mcp__lsp__* are tool-call names, not shell commands'), seven one-line tool descriptions, a minimal copy-paste config snippet, and a single closing tip. It assumes Claude's competence and never explains concepts Claude already knows — a clean match for anchor 5. | 5 / 5 |
Actionability | Concrete tool names with parameter hints ('severity: "error"', 'from file, line, and character') and a complete, executable config JSON make the guidance mostly executable. However, there is no worked example of an actual tool call (argument shapes, a sample lsp.diagnostics or lsp.rename invocation), so it stops short of anchor 5's copy-paste-ready coverage of common cases. | 4 / 5 |
Workflow Clarity | For a simple single-purpose skill the action is unambiguous, and an error-recovery pointer exists ('Use lsp.status first when diagnostics report a missing language server'). But the one genuinely multi-step flow — checking rename validity with lsp.prepare_rename before applying lsp.rename — is only implied by list order, not sequenced; this minor checkpoint gap places it at anchor 4 rather than 5. | 4 / 5 |
Progressive Disclosure | The skill is under 50 lines with no need for external references (no references/, scripts/, or assets/ exist), and the body is organized into clear 'Tools' and 'Config' sections with nothing that belongs in a separate file. Per the simple-skill guideline, well-organized sections alone merit anchor 5. | 5 / 5 |
Total | 18 / 20 Passed |