code-review-excellence

Master effective code review practices to provide constructive feedback, catch bugs early, and foster knowledge sharing while maintaining team morale. Use when reviewing pull requests, establishing review standards, or mentoring developers.

1.25x

Quality

47%

Does it follow best practices?

Impact

85%

1.25x

Average score across 6 eval scenarios

Securityby

Passed

No known issues

Optimize this skill with Tessl

npx tessl skill review --optimize ./plugins/developer-essentials/skills/code-review-excellence/SKILL.md

Evaluation results

80%

34%

Code Review: User Authentication API Endpoint

Review format and severity labeling

Criteria

Without context

With context

Template: Summary section

100%

Template: Strengths section

100%

Template: Required Changes section

100%

Template: Suggestions section

100%

Template: Verdict section

25%

100%

Blocking label used

100%

Nit/suggestion labels used

100%

Praise label used

50%

Question approach

Collaborative language

60%

Identifies SQL injection

100%

Identifies weak hashing

100%

97%

17%

Code Review: React User Dashboard Components

TypeScript-specific bug detection

Criteria

Without context

With context

Flags 'any' in Props

100%

Flags 'any' in api-client

100%

Flags unhandled async errors

100%

Clear verdict

50%

100%

Flags prop mutation

100%

Does NOT flag formatting/style

100%

Question approach

100%

Collaborative language

40%

70%

Flags test mutation issue

100%

Includes positive feedback

100%

83%

Code Review: Notification Service and Cart Logic

Python-specific bug detection

Criteria

Without context

With context

Flags mutable default: metadata

100%

Flags mutable default: channels

100%

Flags mutable default: preferences

100%

Flags mutable class attributes

100%

Flags broad exception catch

100%

Flags implementation-testing in tests

100%

70%

Does NOT flag formatting/style

100%

Question approach

16%

Collaborative language

30%

60%

Includes positive feedback

60%

100%

63%

Analytics Reporting Module Review

PR size management and performance review

Criteria

Without context

With context

Flags PR size

N+1 in engagement report

100%

N+1 in retention cohort

100%

Missing pagination flagged

100%

50%

Hardcoded magic number

Difficult feedback pattern

100%

Severity labels used

60%

100%

Clear verdict

100%

Question approach

Collaborative language

20%

60%

No formatting comments

100%

Test quality issue

25%

87%

100%

User Authentication Service Review

Security checklist application

Criteria

Without context

With context

Flags hardcoded secret

100%

SQL injection: login/register

100%

SQL injection: search or reset

100%

Flags weak password hashing

100%

Flags JWT expiry disabled

100%

Flags missing authorization on admin endpoint

100%

Flags eval() with user input

100%

Flags password exposed in response

100%

Flags missing input validation

100%

Flags missing rate limiting

100%

Uses blocking label for critical issues

100%

No formatting comments

100%

89%

32%

Repository Pattern Migration Review

Architectural review guidance

Criteria

Without context

With context

Recommends design doc first

66%

Suggests staged PRs

100%

Raises alternatives

75%

Design question framing

100%

Flags PR size concern

100%

No scope creep

100%

Clear verdict

100%

Collaborative language

28%

57%

Positive feedback included

100%

No formatting comments

100%

Repository: wshobson/agents
Commit: 70444e5

Evaluated: about 1 month ago
Agent: Claude Code
Model: Claude Sonnet 4.6

Table of Contents

Code Review: User Authentication API Endpoint Code Review: React User Dashboard Components Code Review: Notification Service and Cart Logic Analytics Reporting Module Review User Authentication Service Review Repository Pattern Migration Review

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.