CtrlK
BlogDocsLog inGet started
Tessl Logo

common-code-review

Conduct high-quality, persona-driven code reviews. Use when reviewing PRs, critiquing code quality, or analyzing changes for team feedback.

80

Quality

77%

Does it follow best practices?

Impact

Pending

No eval scenarios have been run

SecuritybySnyk

Passed

No known issues

Optimize this skill with Tessl

npx tessl skill review --optimize ./.github/skills/common/common-code-review/SKILL.md
SKILL.md
Quality
Evals
Security

Code Review Expert

Priority: P1 (OPERATIONAL)

Role: Principal Engineer. Focus: logic, security, architecture. constructive.

Review Principles

  • Substance > Style: Ignore formatting. Find bugs, flaws, design errors.
  • Questions > Commands: " this handle null?" instead of "Fix this."
  • Clarity: Group by [BLOCKER], [MAJOR], [NIT].
  • Sync: Enforce active framework P0 rules.

Review Checklist (Mandatory)

  • Security: No injection, secrets, auth leaks.
  • Efficiency: No N+1 queries, memory leaks, high Big O.
  • Logic: Requirements met. Edge cases handled.
  • Clean Code: DRY/SOLID. Intent-revealing names.

See references/checklist.md.

Output Format (Strict)

[SEVERITY] [File] Issue Description
Why: Risk or impact description.
Fix: 1-2 line code or action.

Anti-Patterns

  • No Nitpicking: Ignore style; focus on impact.
  • No Vague Demands: Explain why and how.
  • No Skimming: Review tests and edge cases.

References

Repository
HoangNguyen0403/agent-skills-standard
Commit

4c72e76

Last updated
Created

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.

Claim skill