agentskills.codes
CO

common-code-review

by li-lanceSource

Conduct high-quality, persona-driven code reviews. Use when reviewing PRs, critiquing code quality, or analyzing changes for team feedback. (triggers: review, pr, critique, analyze code)

Install

mkdir -p .claude/skills/common-code-review && curl -L -o skill.zip "https://agentskills.codes/api/skills/download/15943" && unzip -o skill.zip -d .claude/skills/common-code-review && rm skill.zip

Installs to .claude/skills/common-code-review

Activation

This is the description your AI agent reads to decide when to run this skill — the better it matches your request, the more reliably it fires.

Conduct high-quality, persona-driven code reviews. Use when reviewing PRs, critiquing code quality, or analyzing changes for team feedback. (triggers: review, pr, critique, analyze code)
186 chars✓ has a “when” trigger

About this skill

Code Review Expert

Priority: P1 (OPERATIONAL)

You are a Principal Engineer. Focus on logic, security, and architecture. Be constructive.

Review Principles

  • Substance > Style: Ignore formatting. Find bugs & design flaws.
  • Questions > Commands: "Does this handle null?" vs "Fix this."
  • Readability: Group by [BLOCKER], [MAJOR], [NIT].
  • Cross-Check: Enforce P0 rules from active framework skills.

Review Checklist (Mandatory)

  • Shields Up: Injection? Auth? Secrets?
  • Performance: Big O? N+1 queries? Memory leaks?
  • Correctness: Requirements met? Edge cases?
  • Clean Code: DRY? SOLID? Intent-revealing names?

See references/checklist.md for detailed inspection points.

Output Format (Strict)

Use the following format for every issue found:

[SEVERITY] [File] Issue Description
Why: Explanation of risk or impact.
Fix: 1-2 line code suggestion or specific action.

Anti-Patterns

  • No Nitpicking: Don't flood with minor style comments.
  • No Vague Demands: "Fix this" -> Explain why and how.
  • No Ghosting: Always review tests and edge cases.

References

More by li-lance

View all by li-lance →

Android Dependency Injection (Hilt)

li-lance

Standards for Hilt Setup, Scoping, and Modules

00

Android Navigation

li-lance

Standards for Jetpack Navigation Compose (Type-safe)

00

angular-ssr

li-lance

Implement Angular SSR with hydration, TransferState caching, and per-route render modes. Use when configuring Angular Universal SSR, client hydration, static prerendering, or preventing double-fetching. (triggers: **/*.server.ts, server.ts, hydration, transferState, afterNextRender, isPlatformServer

00

php-concurrency

li-lance

Implement concurrency and non-blocking I/O in modern PHP. Use when implementing concurrent requests, async processing, or non-blocking I/O in PHP. (triggers: **/*.php, Fiber, suspend, resume, non-blocking, async)

00

react-performance

li-lance

Optimize React rendering, bundle size, and data fetching performance. Use when optimizing React rendering performance, reducing re-renders, or improving bundle size. (triggers: **/*.tsx, **/*.jsx, waterfall, bundle, lazy, suspense, dynamic)

00

common-workflow-writing

li-lance

Rules for writing concise, token-efficient workflow and skill files. Prevents over-building that requires costly optimization passes. Use when creating or editing workflow files, SKILL.md files, or new skill definitions. (triggers: .agent/workflows/*.md, SKILL.md, create workflow, write workflow, ne

00

Search skills

Search the agent skills registry