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.
The body is a tight, well-structured, actionable read-only skill with concrete commands, a pre-flight guard, a correctness caution, and graceful degradation. Its only real gap is that the URL parsing is described in prose rather than supplied as executable code.
Suggestions
Provide executable parsing for owner/repo (e.g., a shell or sed snippet) instead of describing extraction in prose, to make the skill fully copy-paste ready.
Add an explicit validation step that checks the parsed URL against the documented HTTPS/SSH patterns before setting 'Is GitHub', turning the CAUTION into a concrete checkpoint.
Optionally show the expected structured output shape (e.g., a small JSON/object example of owner/name/isGitHub) so downstream workflows know the contract.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and assumes Claude's competence (no explanation of what git, a remote, or GitHub is) and every section earns its place, matching the score-4 anchor 'Efficient; minor instances of over-explanation that could be trimmed'. It is not 5 because the example URL strings and the prerequisite prose could be marginally tightened, and not 3 because there is no padding or unnecessary concept explanation. | 4 / 5 |
Actionability | It gives copy-paste-ready commands ('git rev-parse --is-inside-work-tree 2>/dev/null', 'git config --get remote.origin.url') and concrete URL-parsing examples for HTTPS and SSH, matching the score-4 anchor 'Mostly executable guidance; concrete code or commands with minor gaps'. It is not 5 because the owner/repo extraction is described in prose ('Extract from the URL') rather than given as executable parsing code, leaving a minor gap. | 4 / 5 |
Workflow Clarity | The flow Prerequisites -> Execution -> Output -> Graceful Degradation is clearly sequenced with a pre-flight check (git availability) and an error-recovery path (return empty, do not error), matching the score-4 anchor 'Clear sequence with most checkpoints present; minor validation gaps'. It is not 5 because, while the CAUTION acts as a validation rule, there is no explicit validate-then-retry feedback loop or checklist, and not 3 because checkpoints are explicit and the degradation behavior is spelled out. | 4 / 5 |
Progressive Disclosure | The skill is a single under-50-line SKILL.md with no external references needed and is organized into well-signaled sections (Prerequisites, Execution, Output, Graceful Degradation), satisfying the simple-skill scoring note that progressive disclosure can score 5 with just well-organized sections. No bundle files exist, so there is no nested-reference problem to penalize. | 5 / 5 |
Total | 17 / 20 Passed |