review

Review Ethereum execution client diffs for consensus, security, and performance issues.

1.6k|720|Updated Aug 23, 2017
One-click install
npx skills add https://github.com/NethermindEth/nethermind --skill review-nethermindeth
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: review
Source: https://github.com/NethermindEth/nethermind/tree/main/.agents/skills/review
Command: npx skills add https://github.com/NethermindEth/nethermind --skill review-nethermindeth

SYSTEM DOCUMENTATION & REQUIREMENTS

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

What problem does it solve?

This Skill automates in-depth code reviews for Ethereum execution clients, ensuring consensus correctness, security, and performance.

Core Features & Use Cases

  • Comprehensive Analysis: Checks for bugs, security vulnerabilities, breaking changes, and adherence to best practices.
  • Targeted Review: Focuses on critical areas like EVM correctness, Engine API compliance, and performance hot paths.
  • Use Case: When submitting a Pull Request for a new feature or bug fix in an Ethereum client, use this Skill to perform a thorough audit before merging.

Quick Start

Use the review skill to check the current pull request for potential issues.

Frequently Asked Questions about review

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

FAQPage Schema
How do I perform a security audit on an Ethereum execution client pull request?

Deep code review for Ethereum clients analyzes pull request diffs against codebase rules, checking for consensus correctness, security vulnerabilities, and performance hot paths before merging.

What does consensus correctness analysis check in Ethereum client code reviews?

Consensus correctness analysis checks Ethereum client diffs for bugs, security vulnerabilities, and breaking changes, prioritizing critical paths like EVM correctness and Engine API compliance to ensure robust network agreement.

Can I use automated code review to detect breaking changes in EVM implementations?

Automated code review detects breaking changes in EVM implementations by analyzing diffs against domain-specific checks and flagging high-confidence findings related to consensus correctness and Engine API compliance.

What's the best way to check a pull request for Engine API compliance?

The best way to check a pull request for Engine API compliance is performing a targeted deep code review that analyzes diffs against domain-specific checks, prioritizing critical paths and flagging high-confidence findings.

Does this code review skill flag performance hot paths in Ethereum client updates?

The code review skill flags performance hot paths in Ethereum client updates by analyzing pull request diffs against codebase rules, focusing on performance analysis alongside consensus correctness and robustness.

When do I need deep code review for Ethereum client changes?

You need deep code review for Ethereum client changes when submitting a pull request for a new feature or bug fix, ensuring thorough audits of consensus correctness, security, and performance before merging.