CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

skillshare-code-review

Review a skillshare change before it becomes a pull request, or review an open pull request. Use this skill whenever the user asks to: review a diff, branch, commit range or PR, check a change before opening a PR, run a pre-Codex review, or find what a change missed. It checks correctness, the repository's documented rules, the originating issue or spec, every consumer of a changed concept, and cleanup (reuse, simplification, efficiency, depth of fix), then verifies each finding against the code before reporting it. Read-only unless the user asks for fixes.

Quality

96%

Does it follow best practices?

Run evals on this skill

Adds up to 20 points to the overall score

View guide
SecuritybySnyk

Low

Low-risk findings worth noting

SKILL.md
Quality
Evals
Security

Quality

Content

100%Weight 40%Scale 1-5

Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.

The body is a dense, fully executable review procedure: exact git/gh/python commands, a concrete argument-to-diff table, explicit per-pass briefs, a mandatory verification step with decision rules, and a fixed report format. Structure and validation are exemplary; there is nothing material to improve.

DimensionReasoningScore

Conciseness

The body is lean procedural instruction with no conceptual padding: 'Run `git fetch origin` first so the base is current. Stop here if the ref does not resolve or the diff is empty.' Every sentence instructs and assumes Claude's competence, matching the anchor where every token earns its place; there is no over-explanation to trim for a 4.

5 / 5

Actionability

Fully executable guidance throughout: exact commands ('git diff origin/main...HEAD', 'git worktree add', 'gh pr checkout <n>', 'python3 scripts/ai-context.py pull-requests', 'gh issue view <n>'), an argument-to-diff mapping table, and a copy-paste report template. Specific instructions cover the common cases (empty args, ref, PR number, path), matching the top anchor.

5 / 5

Workflow Clarity

Five clearly sequenced steps with explicit validation: a dedicated verification step with decision rules (CONFIRMED / PLAUSIBLE / Dropped), an early stop condition for unresolvable refs or empty diffs, and a fix feedback loop ('add a failing test first for each correctness fix, rerun the narrowest check that proves it'). This matches the anchor with explicit validation steps, feedback loops, and a checklist-style report format.

5 / 5

Progressive Disclosure

A single self-contained file with clear section headers, no nested references, and nothing that belongs in a separate file — the five pass briefs are one-line descriptions that correctly stay inline, and no bundle files exist to point to. Content is appropriately placed in one level with easy navigation, matching the top anchor.

5 / 5

Total

20

/

20

Passed

Description

96%Weight 40%Scale 1-5

Based on the skill's description, can an agent find and select it at the right time? Clear, specific descriptions lead to better discovery.

An excellent description: it states concrete capabilities comprehensively, gives an explicit and natural trigger clause, and defines scope (read-only by default). The only weakness is that its generic review triggers could collide with a general-purpose code-review skill outside the skillshare context.

DimensionReasoningScore

Specificity

Enumerates multiple concrete actions with comprehensive coverage: 'checks correctness, the repository's documented rules, the originating issue or spec, every consumer of a changed concept, and cleanup (reuse, simplification, efficiency, depth of fix), then verifies each finding against the code before reporting it'. This matches the anchor for multiple specific concrete actions with comprehensive coverage; there are no missing capability areas to place it at 4.

5 / 5

Completeness

Explicitly answers both questions: what ('It checks correctness, ... every consumer of a changed concept, and cleanup ..., then verifies each finding') and when ('Use this skill whenever the user asks to: review a diff, branch, commit range or PR, ...'). Clear 'what' plus explicit concrete trigger phrases is the top anchor; not 4 because the 'when' clause is already fully explicit and specific.

5 / 5

Trigger Term Quality

The trigger list covers the natural phrases a user would say: 'review a diff, branch, commit range or PR, check a change before opening a PR, run a pre-Codex review, or find what a change missed'. Synonym and variation coverage is comprehensive for this domain, matching the top anchor.

5 / 5

Distinctiveness Conflict Risk

The opening scopes the skill to a clear niche ('Review a skillshare change before it becomes a pull request'), but the trigger phrases 'review a diff, branch, commit range or PR' are generic code-review triggers that overlap with any general code-review skill. Mostly distinct with minor overlap risk against a closely related skill — the anchor for 4, not 5, since conflict risk with generic review requests is not minimal.

4 / 5

Total

19

/

20

Passed

Validation

75%

Checks the skill against the spec for correct structure and formatting. All validation checks must pass before discovery and implementation can be scored.

Validation — 12 / 16 Passed

Validation for skill structure

CriteriaDescriptionResult

metadata_version

'metadata.version' is missing

Warning

metadata_field

'metadata' should map string keys to string values

Warning

frontmatter_unknown_keys

Unknown frontmatter key(s) found; consider removing or moving to metadata

Warning

referenced_paths_exist

Referenced path issues: 2 missing

Warning

Total

12

/

16

Passed

Repository

runkids/skillshare

Path

.skillshare/skills/skillshare-code-review/SKILL.md

Reviewed

Table of Contents

Is this your skill?

If you maintain this skill, you can claim it as your own. Once claimed, you can manage eval scenarios, bundle related skills, attach documentation or rules, and ensure cross-agent compatibility.