CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

code-review

Procedure to review a pull request in mixpanel-headless and check each finding before it is posted. Use for every pull request review in this repository.

Quality

90%

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.

An exemplary instruction-only skill body: a tight, four-phase procedure with concrete uv commands, exact file paths, explicit validation gates, and clean delegation of standards to REVIEW.md and area guides. No dimension shows meaningful weakness.

DimensionReasoningScore

Conciseness

The body is lean and imperative throughout ('Do not infer behavior from a name, a docstring, or a comment', 'Do not run `just check` or the full test suite. CI runs them.'), and every line adds non-obvious project knowledge (e.g. 'The unit tests run offline with mocks', why `tests/live/` is off-limits). Nothing is padded or explains concepts Claude already knows, matching the anchor where every token earns its place.

5 / 5

Actionability

Concrete executable commands with exact invocation patterns are given (`uv run python -c "..."`, `uv run pytest <file>::<test> -q`, `uv run mp help <Name>`) alongside exact file paths for each check (`src/mixpanel_headless/__init__.py`, a `CHANGELOG.md` entry under `## Unreleased`, `docs/guide/`). This is copy-paste-ready guidance fully covering the skill's common cases, and per the rubric notes the absence of code in an instruction-only skill is not penalized.

5 / 5

Workflow Clarity

Four clearly sequenced phases (collect context → check each finding → check public API changes → post) with an explicit validation gate and feedback loop: 'If you cannot support the finding with a line or a check, do not post it.' Posting rules ('For a bug, state the input that fails and the wrong result', 'If the PR is clean, say so') act as a checklist, matching the top anchor; checkpoints are explicit, not implicit as at level 4.

5 / 5

Progressive Disclosure

The body is under 50 lines with well-organized sections, and it appropriately delegates detail via clearly signaled, one-level-deep references — priorities, evidence standards, and severity scale live in `REVIEW.md`, and per-area guidance in `.github/instructions/` files — rather than inlining them. No bundle files exist to verify, and navigation is easy, matching the top anchor.

5 / 5

Total

20

/

20

Passed

Description

75%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.

A concise, repo-scoped description with an explicit and specific 'Use when' clause and minimal conflict risk. Its main weaknesses are limited action enumeration (only two capabilities mentioned) and missing natural trigger variations such as 'PR' or 'code review'.

Suggestions

Add one or two more concrete actions to the what-clause, e.g. 'assign severities and verify public API changes against the built-in reference and CHANGELOG', to lift specificity toward comprehensive coverage.

Include the natural trigger variations users actually say, e.g. 'Use for every pull request (PR) review or code review in this repository', to broaden trigger-term coverage.

Optionally name the key artifact the procedure consumes (REVIEW.md priorities) so the what-clause signals the skill's evidence standards.

DimensionReasoningScore

Specificity

The description names the domain ("review a pull request in mixpanel-headless") and two concrete actions ("review a pull request", "check each finding before it is posted"), matching the anchor for 1-2 concrete actions but not comprehensive coverage. It does not enumerate the fuller procedure (severity assignment, public API checks), so it falls short of the level-4 anchor listing several specific actions.

3 / 5

Completeness

It explicitly answers both what ("Procedure to review a pull request in mixpanel-headless and check each finding before it is posted") and when ("Use for every pull request review in this repository") with a concrete trigger clause. The when-clause is fully explicit, so the level-4 anchor (when could be more explicit) does not apply.

5 / 5

Trigger Term Quality

The natural phrase "pull request review" is present, but common variations a user would say — "PR", "code review" — are missing, matching the anchor for relevant keywords without common variations. Keyword coverage beyond the single core phrase is too thin for the good-coverage level-4 anchor.

3 / 5

Distinctiveness Conflict Risk

The description is scoped to a single named repository ("mixpanel-headless", "in this repository") and a single task, giving it a clear niche with distinct triggers and minimal conflict risk with other skills.

5 / 5

Total

16

/

20

Passed

Validation

100%

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

Validation — 16 / 16 Passed

Validation for skill structure

No warnings or errors.

Repository

mixpanel/mixpanel-headless

Path

.github/skills/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.