Content
88%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 dense, highly actionable operational skill with concrete tool calls, CLI fallbacks, explicit validation gates, and error-recovery feedback loops appropriate to a pre-edit safety gate. The two weaknesses are structural redundancy (the workflow is restated across four sections) and a single monolithic file with no reference split for a ~165-line body.
Suggestions
Consolidate the repeated CLI-fallback phrasing: state the 'node .gitnexus/run.cjs → npx gitnexus' substitution once and reference it, instead of restating the full fallback on every command in the Workflow, Tools, and Example sections.
Consider moving the detailed Risk Assessment tables and the full impact/detect_changes parameter reference into a reference file (e.g. references/risk.md), keeping SKILL.md as the overview + workflow, to improve progressive disclosure at this length.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is information-dense and assumes Claude's competence — it never explains what git, impact analysis, or a worktree is — and all caveats are tool-specific (MCP default-repo policy, worktree auto-detection, partial/truncated flags) so they earn their tokens. Not a 5 because the same material is restated across the Workflow, Checklist, Tools, and Example sections (the CLI fallback 'node .gitnexus/run.cjs ...' is repeated verbatim four-plus times, and repo/worktree identity echoing recurs), which could be tightened; not a 3 because none of it is explanation of concepts Claude already knows. | 4 / 5 |
Actionability | It gives copy-paste-ready, fully parameterized tool calls (e.g. impact({target: "validateUser", repo: "my-app", direction: "upstream", minConfidence: 0.8, maxDepth: 3})) plus executable CLI fallbacks (node .gitnexus/run.cjs impact "X" --direction upstream --repo .) and a worked example showing expected output. Not a 4 because the common cases are covered with concrete, executable commands rather than guidance with minor gaps. | 5 / 5 |
Workflow Clarity | A clear numbered workflow (0-4) with a parallel checklist and explicit validation gates — bind repo identity, 'warn on HIGH/CRITICAL and stop on UNKNOWN', 'Confirm the checkout you edited is the checkout that was diffed' — plus feedback loops for recovery (stale index → run analyze; partial/truncated → re-run rather than tick the check). Not a 4 because validation checkpoints and error-recovery loops are explicit, not merely mostly present. The destructive/batch cap does not apply since validation is thorough. | 5 / 5 |
Progressive Disclosure | Well-organized into clear sections (When to Use, Bind the repository first, Workflow, Checklist, Understanding Output, Risk Assessment, Tools, Example) and self-contained with no nested or buried references; the only inline references are runtime URIs (gitnexus://repo/{name}/processes) and CLI commands, not bundle files. Not a 5 because there are no one-level-deep reference files and the single file inlines material (detailed tool params, risk tables) that, at ~165 lines, could arguably be split out; not a 3 because structure is clear and content is appropriately placed, not a wall of text with buried references. | 4 / 5 |
Total | 18 / 20 Passed |