CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

review-correctness-and-data-integrity

Review a code change for realistic functional defects, data loss, duplicate writes, ordering mistakes, and broken integration contracts. Use as one lens in a code review run when correctness or data integrity may be affected.

1.08x
Quality

89%

Does it follow best practices?

Impact

98%

1.08x

Average score across 18 eval scenarios

SecuritybySnyk

Passed

No findings from the security scan

The canonical home for this skill is tessl/code-review

SKILL.md
Quality
Evals
Security

Quality

Content

92%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 lean, instruction-only review-lens skill: concrete check categories, a clear reporting structure, and an explicit threshold gate, with no wasted tokens. Its only notable gap is the lack of a worked example showing what a well-formed finding looks like.

Suggestions

Add one brief example finding that instantiates the Reporting section's structure (boundary named, breaking change, consumer at risk, missing migration) to lift actionability to fully executable guidance.

DimensionReasoningScore

Conciseness

Every sentence is a directive with no padding and no explanation of concepts Claude already knows; the body is ~35 lines of dense, purposeful instruction matching the "lean and efficient; every token earns its place" anchor.

5 / 5

Actionability

Concrete, executable directives throughout: specific edge-case categories ("absent or empty values, boundary values, inverted conditions"), a tracing procedure ("trace a record from input to persistence and check the state left by an interrupted run"), a concrete file-reading step (the AGENTS.md/CLAUDE.md chain), and a structured reporting spec. Absence of code is not penalized for an instruction-only skill, but no worked example of an actual finding is given, leaving the minor gap that characterizes a 4.

4 / 5

Workflow Clarity

This is a single-purpose skill under 50 lines, and the Scope → Method → Threshold → Reporting flow is clear and well-organized, so the simple-skill exception applies. The Threshold section acts as an explicit decision checkpoint ("Do not report guards for states the types or the callers already rule out"), and the skill is analysis-only, so the destructive/batch validation cap does not.

5 / 5

Progressive Disclosure

The skill is under 50 lines with no need for external bundle references (none exist and none are needed), and it is organized into clean labeled sections (Scope, Method, Threshold, Reporting), which per the rubric guidance earns a 5. The AGENTS.md/CLAUDE.md mention is operational reading material, not a bundle-reference problem.

5 / 5

Total

19

/

20

Passed

Description

80%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 strong third-person description that names concrete defect classes and includes an explicit 'Use when'-style trigger clause. The trigger is phrased as an abstract condition rather than concrete user-facing phrases, and a few natural synonyms are missing.

Suggestions

Make the trigger clause more concrete and user-facing, e.g. "Use when reviewing a diff or PR where bugs, data loss, or broken contracts between services could occur", so it matches what a user would naturally say.

Add one or two common synonyms ("bugs", "errors", "data corruption") to the defect list to broaden natural trigger-term coverage.

DimensionReasoningScore

Specificity

"Review a code change for realistic functional defects, data loss, duplicate writes, ordering mistakes, and broken integration contracts" enumerates five concrete defect classes, matching the anchor-5 pattern of multiple specific, comprehensive items rather than the minor-gaps coverage of a 4.

5 / 5

Completeness

It clearly answers both what (first sentence) and when ("Use as one lens in a code review run when correctness or data integrity may be affected"), so the missing-trigger cap does not apply. However, the 'when' clause is a conditional abstraction rather than concrete user trigger phrases, matching the anchor-4 case where 'when' could be more explicit or specific.

4 / 5

Trigger Term Quality

Good natural keyword coverage ("code review", "correctness", "data integrity", "data loss", "functional defects", "duplicate writes"), but common synonyms users would naturally say such as "bugs", "errors", or "data corruption" are missing, so it falls short of the comprehensive-synonyms anchor for 5.

4 / 5

Distinctiveness Conflict Risk

"One lens in a code review run" carves a clear correctness/data-integrity niche with distinct trigger terms, but the broad "code review" framing creates minor overlap risk with sibling review-lens skills, matching "mostly distinct; minor overlap risk" rather than the minimal-conflict anchor for 5.

4 / 5

Total

17

/

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

tesslio/product-plugins

Path

code-review/skills/review-correctness-and-data-integrity/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.