code-review

$npx mdskill add langchain-ai/deepagents/code-review

Validate code correctness, style, and safety before delivery.

  • Ensures changes solve issues, handle edges, and avoid side effects.
  • Executes pytest, ruff, and custom lint scripts for verification.
  • Compares output against a structured checklist of quality criteria.
  • Recommends fixes immediately when any checklist item fails validation.

SKILL.md

.github/skills/code-reviewView on GitHub ↗
---
name: code-review
description: Perform a structured code review of changes, checking for correctness, style, tests, and potential issues.
---

# Code Review Skill

Use this skill after implementing changes to validate your work before delivering.

## Review Checklist

### 1. Correctness
- [ ] Changes solve the original issue/task
- [ ] No unintended side effects on existing functionality
- [ ] Edge cases are handled
- [ ] Error handling is appropriate (not excessive)

### 2. Code Quality
- [ ] Code matches existing style and patterns
- [ ] No unnecessary complexity or abstraction
- [ ] Variable and function names are clear
- [ ] No dead code, commented-out code, or TODOs left behind

### 3. Tests
- [ ] New functionality has test coverage
- [ ] Existing tests still pass
- [ ] Tests cover both happy path and error cases
- [ ] Tests are not brittle (don't test implementation details)

### 4. Safety
- [ ] No hardcoded secrets or credentials
- [ ] User input is validated at boundaries
- [ ] No SQL injection, XSS, or command injection vectors
- [ ] File operations use safe paths

## Process

1. Read each modified file end-to-end (not just the diff)
2. Run the test suite: `execute("python -m pytest -v")`
3. Run linters if available: `execute("ruff check .")`
4. Run the bundled lint check: `execute("python /skills/code-review/lint_check.py .")`
5. Check against each item in the review checklist
6. If any issues found, fix them and re-review
7. When everything passes, the review is complete

## Helper Scripts

- **`/skills/code-review/lint_check.py`** — Scans Python files for missing
  docstrings, long functions (>50 lines), and bare `except:` clauses. Run it
  via `execute("python /skills/code-review/lint_check.py [path ...]")`.

More from langchain-ai/deepagents

SkillDescription
analyze-market>-
arxiv-searchSearches arXiv for preprints and academic papers, retrieves abstracts, and filters by topic. Use when the user asks to find research papers, search arXiv, look up preprints, find academic articles in physics, math, CS, biology, statistics, or related fields.
blog-postWrites and structures long-form blog posts, creates tutorial outlines, and optimizes content for SEO with cover image generation. Use when the user asks to write a blog post, article, how-to guide, tutorial, technical writeup, thought leadership piece, or long-form content.
coding-prefsRead the user's coding preferences from /memory/coding-prefs.md before making non-trivial style decisions, and append new preferences when the user gives durable feedback.
competitor-analysis>-
cudf-analyticsUse for GPU-accelerated data analysis on datasets, CSVs, or tabular data using NVIDIA cuDF. Triggers when tasks involve groupby aggregations, statistical summaries, anomaly detection, or large-scale data profiling.
cuml-machine-learningUse for GPU-accelerated machine learning on tabular data using NVIDIA cuML. Triggers when tasks involve classification, regression, clustering, dimensionality reduction, or model training on datasets.
data-visualizationUse for creating publication-quality charts and multi-panel analysis summaries. Triggers when tasks involve visualizing data, plotting results, creating charts, or producing visual reports from analysis output.
gpu-document-processingUse when processing large PDFs, document collections, or bulk text extraction tasks that benefit from GPU-accelerated processing. Triggers when the user provides large documents or needs bulk document analysis.
langgraph-docsFetches and references LangGraph Python documentation to build stateful agents, create multi-agent workflows, and implement human-in-the-loop patterns. Use when the user asks about LangGraph, graph agents, state machines, agent orchestration, LangGraph API, or needs LangGraph implementation guidance.