CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

agm-code-review

Review a concrete Antigravity Manager diff for semantic defects, boundary violations and evidence gaps. Use for pull request reviews or the final review after implementation validation; this skill does not select or rerun tests.

Quality

88%

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

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

This is a disciplined, expert-oriented instruction skill: a clearly sequenced multi-round review workflow with explicit verification checkpoints and escalation rules, concrete finding-tracking states, and a record template. Detail is appropriately pushed to conditionally-loaded external references, and the only weaknesses are minor redundancy between the description and body and the absence of a worked finding example.

DimensionReasoningScore

Conciseness

The body is dense, assumes competence, and contains no tutorial filler — every section instructs rather than explains — but a few defensive clarifications repeat guidance already stated elsewhere ('review itself does not execute these checks', 'A review does not replace executed validation', 'the limit is not approval') and could be trimmed, matching the 4 anchor rather than the lean-everything-earns-its-place 5.

4 / 5

Actionability

For an instruction-only skill the guidance is highly actionable: concrete steps ('verify the actual base and head from its metadata before calculating the changed paths... never guess either ref'), a copy-paste record template, explicit finding states (open/fixed/not a defect/accepted limitation), and named checks per boundary type. It stays at 4 rather than 5 because key criteria (failure ownership, behavioral evidence contract, test reliability) are delegated to external documents rather than stated inline, and there is no worked example of a well-formed finding.

4 / 5

Workflow Clarity

The process is clearly sequenced (establish scope → review rounds → review behavior → report findings) with explicit checkpoints and feedback loops: recalculate the diff when base/head change, verify open findings in follow-up rounds, reopen IDs only with new evidence, and a defined stop-and-escalate condition after the third substantive round. This matches the 5 anchor's explicit validation steps and error-recovery loops, and it is not the 4 anchor since no meaningful checkpoint is missing.

5 / 5

Progressive Disclosure

The SKILL.md is a well-organized overview whose deep domain criteria are delegated via clearly signaled, one-level-deep references with explicit load conditions ('Read the architecture, security or testing reference when the changed boundary needs it', 'Use the bounded historical review evaluation... when investigating a specific reviewer miss'). No bundle files exist, no references are nested, and navigation is easy — matching the 5 anchor; it avoids the 4 anchor's 'minor organization gaps' since every reference states when and why to read it.

5 / 5

Total

18

/

20

Passed

Description

87%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 tightly written, third-person description that states concrete capabilities, an explicit 'Use for...' trigger clause, and a clear boundary against the sibling validation skill. The only room for growth is broader natural-language trigger synonyms (e.g. 'code review', 'review my changes').

DimensionReasoningScore

Specificity

The description names concrete review targets — 'semantic defects, boundary violations and evidence gaps' — plus an explicit exclusion ('does not select or rerun tests'), which is more specific than naming a domain with generic actions, but it centers on a single action (reviewing a diff) rather than the multiple distinct concrete actions of the 5 anchor.

4 / 5

Completeness

It explicitly answers both parts: what ('Review a concrete Antigravity Manager diff for semantic defects, boundary violations and evidence gaps') and when ('Use for pull request reviews or the final review after implementation validation') with concrete trigger phrases, matching the 5 anchor; it is not the 4 anchor because the 'when' clause is explicit and specific rather than vague.

5 / 5

Trigger Term Quality

It includes natural phrases a user would say such as 'pull request reviews', 'review' and 'diff', giving good keyword coverage; common variations like 'code review', 'review my changes' or 'review the PR' are absent, keeping it below the comprehensive 5 anchor.

4 / 5

Distinctiveness Conflict Risk

The project-specific 'Antigravity Manager' scope, the pull-request/final-review triggers, and the explicit boundary ('this skill does not select or rerun tests') carve out a clear niche with minimal overlap, even against the closely related validation skill referenced in the body.

5 / 5

Total

18

/

20

Passed

Validation

93%

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

Validation — 15 / 16 Passed

Validation for skill structure

CriteriaDescriptionResult

relative_links

Relative link issues: 10 suspicious

Warning

Total

15

/

16

Passed

Repository

Draculabo/AntigravityManager

Path

.agents/skills/agm-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.