code-review

Review TypeScript pull requests for quality and security compliance.

80|49|Updated Mar 27, 2022
One-click install
npx skills add https://github.com/cartridge-gg/controller --skill code-review-cartridge-gg
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: code-review
Source: https://github.com/cartridge-gg/controller/tree/main/.agents/skills/code-review
Command: npx skills add https://github.com/cartridge-gg/controller --skill code-review-cartridge-gg

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

Reviewing code changes quickly and consistently to ensure quality, security, and maintainability across a codebase.

Core Features & Use Cases

  • Detailed review checkpoints for code quality, security, and performance.
  • Guidelines for TypeScript types, edge cases, and documentation alignment.
  • Use Case: when merging PRs in a multi-repo monorepo, this checklist helps maintain standards and catch issues before they reach production.

Quick Start

Run through the review checklist on the current PR to ensure quality and security before merging.

Frequently Asked Questions about code-review

High-intent search queries and answers about installing and using this skill.

FAQPage Schema
How do I review code changes in a pull request for security and quality?

To review code changes in a pull request, identify modified modules and apply a checklist assessing TypeScript correctness, error handling, tests, documentation, and security best practices before merging.

What is the best way to ensure TypeScript correctness and readability during code reviews?

The best way to ensure TypeScript correctness and readability during code reviews is to apply a standardized checklist that evaluates type usage, edge cases, and documentation alignment across affected modules.

How do I validate iframe or external interactions when reviewing a pull request?

To validate iframe or external interactions during a code review, assess the changes against project guidelines and security best practices to ensure all external interactions are properly validated.

Can I use a code review checklist for a multi-repo monorepo environment?

Yes, you can use a code review checklist in a multi-repo monorepo environment to maintain consistent quality and security standards across multiple repositories before changes reach production.

What does a thorough code review process cover for quality and maintainability?

A thorough code review process covers detailed checkpoints for code quality, security, and performance, ensuring changes are evaluated for correctness, readability, error handling, tests, and documentation alignment.

Why do I need to check tests and documentation when reviewing code changes?

You need to check tests and documentation when reviewing code changes to ensure documentation alignment, verify edge cases are handled, and maintain codebase maintainability and security before production merging.