review-checklist

Automate pre-merge code review for database, security, and string issues.

Updated Mar 18, 2026
One-click install
npx skills add https://github.com/wdapp/SClaw --skill review-checklist-wdapp
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: review-checklist
Source: https://github.com/wdapp/SClaw/tree/main/skills/review-checklist
Command: npx skills add https://github.com/wdapp/SClaw --skill review-checklist-wdapp

SYSTEM DOCUMENTATION & REQUIREMENTS

💡 This Skill includes scripts (resource) and references (resource) components.

What problem does it solve?

This Skill provides a structured pre-merge review checklist to ensure code quality and security, saving time and reducing the likelihood of post-merge issues.

Core Features & Use Cases

  • Comprehensive Checklist: A detailed list of items to check before merging code, based on common issues caught by automated reviewers.
  • Database Operations: Ensures multi-step database operations are wrapped in transactions and migrations are atomic.
  • Security & Data Safety: Validates that sensitive data is handled securely and securely, with no secrets in logs or error messages.
  • String Safety: Ensures string operations are safe and secure, with no byte-index slicing or case-sensitive path comparisons.
  • Trait Wrappers & Decorator Chain: Checks that new trait methods are properly delegated and tested.
  • Tests: Ensures tests are written correctly, with no mutation of global statics and no real network requests.
  • Comments & Documentation: Validates that code comments and documentation are up-to-date and accurate.

Quick Start

To use the review-checklist skill, activate it with the command "run review-checklist" and follow the checklist provided.

Frequently Asked Questions about review-checklist

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

FAQPage Schema
How do I use a pre-merge code review checklist to improve code quality?

Pre-merge code review checklists improve code quality by providing a structured list of items to validate before merging. Activate the checklist with a specific command and manually review items covering database operations, security, and string safety.

What is included in a comprehensive code review checklist for security and database operations?

A comprehensive code review checklist for security and database operations includes verifying that multi-step database operations are wrapped in transactions, migrations are atomic, and sensitive data is kept out of logs and error messages.

How do I check for string safety and secure data handling before merging code?

Check for string safety before merging code by ensuring there is no byte-index slicing or case-sensitive path comparisons. Secure data handling is verified by confirming sensitive information is protected and not exposed in logs.

Does a pre-merge code review checklist check for proper test writing and documentation?

Yes, a pre-merge code review checklist checks that tests are written correctly without mutating global statics or making real network requests, and validates that code comments and documentation are accurate.

Can I use a code review checklist to validate trait wrappers and decorator chains?

Yes, you can use a code review checklist to validate trait wrappers and decorator chains. It checks that new trait methods are properly delegated and tested before the code is merged.