Submit

Code Review Checklist

code-review-checklist

Code review guidelines covering code quality, security, and best practices.

darthlinuxer/Agentic-Skills·0installs1starsSecurity

SKILL.md

Full skill instructions

Code Review Checklist

Quick Review Checklist

Correctness

Code does what it's supposed to do
Edge cases handled
Error handling in place
No obvious bugs

Security

Input validated and sanitized
No SQL/NoSQL injection vulnerabilities
No XSS or CSRF vulnerabilities
No hardcoded secrets or sensitive credentials
AI-Specific: Protection against Prompt Injection (if applicable)
AI-Specific: Outputs are sanitized before being used in critical sinks

Performance

No N+1 queries
No unnecessary loops
Appropriate caching
Bundle size impact considered

Code Quality

Clear naming
DRY - no duplicate code
SOLID principles followed
Appropriate abstraction level

Testing

Unit tests for new code
Edge cases tested
Tests readable and maintainable

Documentation

Complex logic commented
Public APIs documented
README updated if needed

AI & LLM Review Patterns (2025)

Logic & Hallucinations

Chain of Thought: Does the logic follow a verifiable path?
Edge Cases: Did the AI account for empty states, timeouts, and partial failures?
External State: Is the code making safe assumptions about file systems or networks?

Prompt Engineering Review

// ❌ Vague prompt in code
const response = await ai.generate(userInput);

// ✅ Structured & Safe prompt
const response = await ai.generate({
  system: "You are a specialized parser...",
  input: sanitize(userInput),
  schema: ResponseSchema
});

Anti-Patterns to Flag

// ❌ Magic numbers
if (status === 3) { ... }

// ✅ Named constants
if (status === Status.ACTIVE) { ... }

// ❌ Deep nesting
if (a) { if (b) { if (c) { ... } } }

// ✅ Early returns
if (!a) return;
if (!b) return;
if (!c) return;
// do work

// ❌ Long functions (100+ lines)
// ✅ Small, focused functions

// ❌ any type
const data: any = ...

// ✅ Proper types
const data: UserData = ...

Review Comments Guide

// Blocking issues use 🔴
🔴 BLOCKING: SQL injection vulnerability here

// Important suggestions use 🟡
🟡 SUGGESTION: Consider using useMemo for performance

// Minor nits use 🟢
🟢 NIT: Prefer const over let for immutable variable

// Questions use ❓
❓ QUESTION: What happens if user is null here?

Install skill

Copy and run in your terminal

npx -y skills add https://github.com/darthlinuxer/Agentic-Skills --skill code-review-checklist

View on GitHub

.agent/skills/code-review-checklist/SKILL.md

More Security skills

View all

Security

supabase

Handles the full Supabase workflow from schema changes to deployment, with built-in security guardrails that catch common traps like RLS...

2.7K 308.9K

View

Security

neon-postgres

Comprehensive guides and best practices for Neon Serverless Postgres, covering setup, connection methods, authentication, and platform APIs.

98 216.2K

View

Security

firebase-auth-basics

Guide for setting up and using Firebase Authentication. Use this skill when the user's app requires user sign-in, user management, or secure data access using auth rules.

462 163.8K

View

Security

firebase-security-rules-auditor

A skill to evaluate how secure Firestore security rules are. Use this when Firestore security rules are updated to ensure that the generated rules are extremely secure and robust.

462 127.7K

View

Security

firebase-ai-logic-basics

Official skill for integrating Firebase AI Logic (Gemini API) into web applications. Covers setup, multimodal inference, structured output, and security.

462 125.1K

View

Security

better-auth-best-practices

Complete Better Auth server and client setup with database adapters, session management, plugins, and security configuration.

222 118.2K

View

Security

Deploy and manage projects on Vercel using token-based authentication. Use when working with Vercel CLI using access tokens rather than interactive login — e.g. "deploy to vercel", "set up vercel", "add environment variables to vercel".

31.9K 116.1K

View

Security

cloudflare

Complete Cloudflare platform integration with decision trees for compute, storage, AI, networking, security, and infrastructure-as-code.

3K 110.7K

View

Security

azure-compliance

Run Azure compliance and security audits with azqr plus Key Vault expiration checks. Covers best-practice assessment, resource review, policy/compliance validation, and security posture checks. WHEN: compliance scan, security audit, BEFORE running azqr (compliance cli tool), Azure best practices, Key Vault expiration check, expired certificates, expiring secrets, orphaned resources, compliance assessment.

253 103K

View