CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

code-review-helper

帮助用户进行代码审查,识别常见代码问题并给出修改建议。

49

Quality

62%

Does it follow best practices?

Run evals on this skill

Adds up to 20 points to the overall score

View guide
SecuritybySnyk

Passed

No findings from the security scan

Fix and improve this skill with Tessl

tessl review fix ./.agents/skills/skill-upper/evals/fixtures/skills/sample-no-evals/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

57%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 lean and well-organized with a sensible section structure, and the trigger scenarios in the body partially compensate for the description's missing 'when'. However, it reads as a feature brochure rather than an operating manual: it never tells the agent how to actually perform a review or what the severity levels mean.

Suggestions

Add a concrete review procedure (e.g. scan diff hunk by hunk → check against the listed issue categories → assign severity → verify each finding against the code before reporting it) so the agent knows how to execute rather than just what to output.

Define the severity levels used in the output format (e.g. 严重/警告/建议 with one-line criteria each) and include a short example report so the output spec is actionable.

Merge or trim the redundant intro line and 使用方式 section, which restate the description without adding operational value.

DimensionReasoningScore

Conciseness

The body is short with no concept re-education, but the opening line restates the description and the 使用方式 section ("Skill 会自动分析并输出审查报告") adds little value, matching the anchor of efficient content with minor instances that could be trimmed.

4 / 5

Actionability

Issue categories ("空指针风险、未处理异常、资源泄漏") and an output structure ("严重程度 + 位置 + 描述") are named, but there is no review procedure, no severity scale definition, and no example report — high-level hints missing the specific steps to execute, which matches anchor 2 rather than 3.

2 / 5

Workflow Clarity

The single action is stated ("直接将代码片段或 diff 发给 Agent"), but the analysis process — how to scan the code, how to classify severity, how to decide what counts as a problem — is entirely implicit, fitting the anchor of a sequence present with checkpoints missing or implicit.

3 / 5

Progressive Disclosure

The skill is under 50 lines, needs no external references, and none are bundled; the four well-organized section headers (功能, 触发场景, 使用方式, 输出格式) satisfy the simple-skill exception for a top score.

5 / 5

Total

14

/

20

Passed

Description

50%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 is serviceable: it states a clear domain and two actions in concise third-person Chinese. Its main weaknesses are the complete absence of a 'when to use' trigger clause and the generic phrase "常见代码问题" instead of concrete capabilities.

Suggestions

Add an explicit trigger clause, e.g. "当用户要求 review 代码、提交 diff 想要反馈、或想检查代码质量时使用。"

Replace the generic "识别常见代码问题" with the concrete issue categories already listed in the body (空指针风险、未处理异常、资源泄漏、命名规范) so the description covers what the skill actually does.

Include natural user phrasings and synonyms ("帮我 review", "代码 diff", "PR 反馈") to improve trigger term coverage and distinguish the skill from generic lint/quality tools.

DimensionReasoningScore

Specificity

The description names the domain (代码审查) and two actions ("识别常见代码问题", "给出修改建议"), but "常见代码问题" is generic and the action list is not comprehensive, matching the anchor for 1-2 concrete actions rather than several specific ones.

3 / 5

Completeness

It clearly answers "what" (helps review code, identify issues, give suggestions) but contains no "when" / "Use when..." trigger guidance at all, which caps completeness at 3 per the judging guidelines.

3 / 5

Trigger Term Quality

"代码审查" is a natural phrase users would say, but the description misses common variations such as "review 这段代码", "diff", "PR", or "代码质量", fitting the anchor of some relevant keywords without synonyms or variations.

3 / 5

Distinctiveness Conflict Risk

"代码审查" denotes a real niche, but the phrasing is broad and would overlap with other review, quality, or lint-type skills, matching the anchor of somewhat specific with overlap risk.

3 / 5

Total

12

/

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

LuminairPrime/git-worktrees-prime

Path

.agents/skills/skill-upper/evals/fixtures/skills/sample-no-evals/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.