skills.rest
tusharg1993
tusharg1993COMMUNITY
Explore all skills by this author →
GitHub

review

Identify correctness, safety, performance, and compliance issues in ClickHouse PR patches.

Updated Mar 10, 2026
Search Tags:#performance#safety#code-review#cpp#pull-request#clickhouse#maintainer
One-click install
npx skills add https://github.com/tusharg1993/ClickHouse --skill review-tusharg1993
Or copy as Structured Prompt for Agent▼
Please help me install this Agent Skill.
Skill: review
Source: https://github.com/tusharg1993/ClickHouse/tree/main/.claude/skills/review
Command: npx skills add https://github.com/tusharg1993/ClickHouse --skill review-tusharg1993

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

Reviewers and maintainers need a rigorous, structured way to evaluate ClickHouse PRs for correctness, safety, performance, and compliance, reducing risk and review time.

Core Features & Use Cases

  • Structured PR evaluation: checks for correctness, safety, performance, and compliance in diffs.
  • Risk summarization: highlights potential issues and suggested remediations with tests linkage.
  • Guided review process: provides actionable steps and checkpoints for maintainers.

Quick Start

Provide a PR number or branch name to begin a thorough code-review pass.

Frequently Asked Questions about review

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

FAQPage Schema
How do I review a ClickHouse pull request for safety and performance issues?▼

To review a ClickHouse pull request for safety and performance, apply a structured evaluation process to the diff to identify correctness, concurrency, and memory risks, then summarize them with concrete remediation steps.

What is the best way to evaluate correctness and compliance in C++ PR patches?▼

Evaluating correctness and compliance in C++ PR patches involves analyzing the code changes for serialization and Keeper-related risks, providing guided checkpoints to ensure safe and incremental merges.

How does a structured code review identify concurrency and memory risks in ClickHouse core code?▼

A structured code review identifies concurrency and memory risks by analyzing the PR patch across core code areas, highlighting potential safety issues, and linking them to suggested tests and follow-up actions.

Can I get remediation steps and test guidance for a ClickHouse diff?▼

Yes, you can get remediation steps and test guidance for a ClickHouse diff by running a thorough code-review pass that summarizes risks and provides actionable steps for maintainers to enable safe integration.

What specific problems does a rigorous PR review solve for ClickHouse maintainers?▼

A rigorous PR review solves the problem of evaluating ClickHouse PRs for correctness, safety, performance, and compliance, reducing both merge risk and overall review time for maintainers.

When do I need to apply a structured evaluation to a ClickHouse pull request?▼

You need to apply a structured evaluation to a ClickHouse pull request when changes impact core code, including memory, concurrency, serialization, and Keeper-related modifications, ensuring safe incremental merges.

Related Skills

Explore More Software Engineering Skills→
SriRamkunamsettySriRamkunamsetty

github-code-review

Automate structured code review and inline commenting for local git changes or GitHub PRs.

Community
Advanced
89jobrien89jobrien

diagnose-ci-failures

Diagnose CI failures for a PR and generate a remediation plan.

Community
Advanced
Oscar-Ivan-SalasOscar-Ivan-Salas

lint-and-validate

Execute linting, type checks, and security audits on Node.js and Python codebases.

Community
Intermediate
macaughmacaugh
5

Static Vulnerability Analysis

Detects injection, memory safety and authorization flaws in multiple languages.

Community
Advanced