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.

72

Quality

91%

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

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 instruction-only skill body: lean, codebase-specific directives with a clear scope-then-review-then-report workflow, explicit verification steps, a re-review feedback loop, and well-signaled one-level-deep references to shared repo docs. The single gap is the absence of concrete commands or examples for the diff/metadata mechanics the workflow relies on.

DimensionReasoningScore

Conciseness

The body is dense with domain-specific directives and zero padding — e.g. 'Check feature ownership, dependency direction, renderer/main isolation, and that cross-process calls use the typed preload and ORPC/IPC surface' — with no explanations of concepts Claude already knows. Every token earns its place, matching anchor 5 rather than anchor 4, which reserves room for 'minor instances of over-explanation' that are absent here.

5 / 5

Actionability

Guidance is concrete and grounded in named surfaces — 'verify the actual base and head from its metadata before calculating the changed paths and diff. Include any relevant uncommitted work separately; never guess either ref', 'For SQLite, Drizzle, credentials and backups, inspect privacy, atomicity, recovery and migration behavior' — fitting anchor 4's 'mostly executable guidance with minor gaps'. It is not anchor 5 because no commands or worked examples are given for the mechanics (e.g. how to fetch PR metadata or compute the diff), and a few directives remain abstract ('trace safe continuation and final reporting to their owners').

4 / 5

Workflow Clarity

The scope section is a numbered sequence with explicit verification checkpoints ('verify the actual base and head from its metadata', 'never guess either ref') and a genuine feedback loop ('If the PR base, head or merge base changed after earlier review, recalculate the diff and review the newly affected paths'), followed by a structured review checklist and a defined reporting protocol — matching anchor 5's 'clear sequence with explicit validation steps; feedback loops; checklists'. This is a read-only review skill, so the destructive-operation cap does not apply.

5 / 5

Progressive Disclosure

The ~30-line body is a well-organized overview (scope → behavior checklist → reporting) with clearly signaled, one-level-deep references placed exactly where needed — '[failure ownership](../../../docs/error-handling.md)', '[test reliability](../../../docs/test-reliability.md)', '[Agent lessons](../../lessons.md)', '[upgrade instructions](../../../docs/upgrade-guides/README.md)' — and no bundle files exist that should have been split out, matching anchor 5's clear overview with well-signaled references. It is not anchor 4, since navigation and signaling leave no notable organization gaps.

5 / 5

Total

19

/

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 strong description: it states a concrete what (semantic defects, boundary violations, evidence gaps in an Antigravity Manager diff), an explicit 'Use for' when-clause covering PR reviews and post-validation review, and a clear negative scope. The only room for improvement is broader natural trigger vocabulary and slightly fuller enumeration of the review's coverage.

DimensionReasoningScore

Specificity

Names the domain ('a concrete Antigravity Manager diff') and several specific review targets — 'semantic defects, boundary violations and evidence gaps' — plus scope conditions, matching anchor 4's 'several specific actions; minor gaps'. It falls short of anchor 5 because the description does not enumerate more of the review's actual coverage (e.g. async lifecycle, UX or upgrade/migration checks that the body shows), yet it clearly exceeds anchor 3's '1-2 concrete actions'.

4 / 5

Completeness

The 'what' is explicit ('Review a concrete Antigravity Manager diff for semantic defects, boundary violations and evidence gaps') and the 'when' is an explicit trigger clause ('Use for pull request reviews or the final review after implementation validation') with a concrete negative boundary ('this skill does not select or rerun tests'), matching anchor 5's 'clearly and explicitly answers both what AND when with concrete trigger phrases'.

5 / 5

Trigger Term Quality

Natural phrases users would say are present — 'pull request reviews', 'diff', 'review', 'implementation validation' — fitting anchor 4's 'good keyword coverage; a few natural terms missing'. It does not reach anchor 5 because common synonyms and variations such as 'code review', 'PR', or 'review changes' are absent, and it exceeds anchor 3 because the core trigger vocabulary is well covered rather than partially missing.

4 / 5

Distinctiveness Conflict Risk

The 'Antigravity Manager' niche scopes the skill tightly, and it explicitly disambiguates from its closest sibling ('after implementation validation; this skill does not select or rerun tests'), giving clear distinct triggers with minimal conflict risk — anchor 5. It is not anchor 4, because the main overlap risk (the validation skill) is already addressed by an explicit exclusion rather than left as a minor gap.

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: 7 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.