CtrlK
BlogDocsLog inGet started
Tessl Logo

tidb-change-instruction-critic

Use when implementing a user- or reviewer-prescribed code change (including review comments with suggested fixes or options), especially when the requested edit may be risky, incomplete, ambiguous, or misaligned with TiDB correctness and compatibility constraints.

69

Quality

85%

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 an exemplar instruction-only skill: lean, tightly structured, with a numbered workflow, an explicit clarification gate, reusable question templates, and a completion checklist, all without padding or unnecessary references.

DimensionReasoningScore

Conciseness

The body is lean and efficient with no padding: it avoids explaining what TiDB or code review is and every section (Principle, Workflow, Question Patterns, Output Checklist) earns its place, matching 'Lean and efficient; assumes Claude's competence; every token earns its place'.

5 / 5

Actionability

The workflow gives concrete, executable guidance — decompose into intent/constraints/proposed method, mark items mandatory or negotiable, evaluate the requested approach plus an alternative, and produce a pre-coding self-analysis summary — with specific question-pattern templates, fully actionable without code, matching the instruction-skill 'fully executable; specific examples cover the common cases' anchor.

5 / 5

Workflow Clarity

A clearly numbered 7-step sequence with an explicit validation checkpoint ('Challenge gate before implementation') that gates coding on clarification, plus an Output Checklist for completion feedback, matching 'Clear sequence with explicit validation steps; feedback loops; checklists'.

5 / 5

Progressive Disclosure

Well-organized into clearly labeled sections (Trigger, Principle, Workflow, Question Patterns, Output Checklist) with no nested references and no inlined bulk content; under 50 lines with no need for external files, qualifying for the simple-skill exception at score 5.

5 / 5

Total

20

/

20

Passed

Description

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

The description has an explicit 'Use when' trigger and a TiDB-specific niche that distinguishes it, but it is a single dense sentence that bundles intent, trigger, and caveats together, slightly weakening readability and trigger-term naturalness.

Suggestions

Split the description into a concise 'what' clause and a separate 'Use when...' trigger clause listing the concrete situations (e.g., review comments with suggested fixes, multi-option change requests, risky patches).

Add more concrete action verbs that name what the skill actually does (e.g., 'decomposes, validates, compares solution candidates, and recommends') instead of the abstract 'Use when implementing...'.

Shorten or move the 'especially when... risky, incomplete, ambiguous' qualifier so the primary trigger phrase stays crisp and scannable.

DimensionReasoningScore

Specificity

Names the domain (TiDB code-change criticism) and a few concrete actions ('validating', 'aligning', 'evaluating') but the actions are described abstractly; no concrete skill-level operations are listed, matching 'Names domain and 1-2 concrete actions, but not comprehensive'.

3 / 5

Completeness

Has both a 'what' ('validate and critique implementation instructions') and an explicit 'when' ('Use when implementing a user- or reviewer-prescribed code change...'), but the 'when' is a long qualifier-heavy clause rather than a crisp concrete trigger, so it sits at 'both what and when; when could be more explicit or specific' rather than 5.

4 / 5

Trigger Term Quality

Good natural-term coverage including 'review comments', 'suggested fixes', 'options', and 'implementation direction' that users would plausibly say, though it leans on phrasing like 'prescribed code change' rather than the most common user phrasings, leaving a few natural terms missing.

4 / 5

Distinctiveness Conflict Risk

The TiDB-specific correctness/compatibility framing gives it a clear niche distinct from generic code-review skills, with only minor overlap risk against general code-change skills; it does not reach 5 because the 'implementing a prescribed change' trigger could overlap with broader editing skills.

4 / 5

Total

15

/

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.

Validation16 / 16 Passed

Validation for skill structure

No warnings or errors.

Repository
pingcap/tidb
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.