review-runtime-reliability-performance

What problem does it solve?

Merging code changes with unaddressed failure semantics, concurrency lifecycle, or performance scalability gaps leads to production outages, data integrity issues, and system collapse under load. This Skill eliminates the risk of missing these critical issues during pull request review by providing a structured, static-analysis-first evaluation flow.

Core Features & Use Cases

• Structured Failure Semantics Review: Evaluates error contracts, retry/timeout/cancellation behavior, and silent failure paths for API handlers, async workers, and external dependency calls.
• Concurrency Lifecycle Safety Checks: Assesses synchronization discipline, cancellation safety, bounded parallelism, and graceful shutdown logic to prevent data races, deadlocks, and resource leaks.
• Performance and Scaling Risk Assessment: Identifies algorithmic anti-patterns, N+1 access issues, tail latency risks, and overload collapse vulnerabilities before they reach production.
• Use Case: When reviewing a PR that adds a new payment processing async worker, this Skill flags missing idempotency guarantees for retries, unbounded parallelism that could exhaust worker pools under high load, and lack of cancellation handling that leaves in-flight tasks running during deployment.

Quick Start

Use the review-runtime-reliability-performance skill to review the pull request diff at /your/repo/path for failure semantics, concurrency lifecycle, and performance risks, and output the findings to review-findings.json.