plan-review

Review feature development architecture, code quality, tests, and performance.

1|Updated Oct 9, 2018

Search Tags:#testing #architecture #quality assurance #code review #performance #feature development

Ready to run

npx skills add https://github.com/Apostolos-Delis/Dotfiles --skill plan-review-apostolos-delis

💬 Prompt for ChatGPT / Claude Chat▼

Please help me install this Agent Skill.
Skill: plan-review
Source: https://github.com/Apostolos-Delis/Dotfiles/tree/main/.agents/skills/plan-review
Command: npx skills add https://github.com/Apostolos-Delis/Dotfiles --skill plan-review-apostolos-delis

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

This Skill provides a structured, interactive review process for planned or in-progress feature development, ensuring architectural soundness and code quality before significant integration.

Core Features & Use Cases

Architecture Review: Evaluates component boundaries, data flow, API design, and pattern consistency.
Code Quality Review: Assesses readability, DRY principles, complexity, elegance, and error handling.
Test Review: Checks for coverage gaps, edge cases, test quality, and integration coverage.
Performance Review: Identifies potential issues with query efficiency, memory usage, concurrency, and scaling.
Use Case: Before merging a new feature branch, use this Skill to get a comprehensive review of its design, code, tests, and performance implications.

Quick Start

Run an architecture-first review for the current feature branch.

Frequently Asked Questions about plan-review

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

FAQPage Schema

How do I conduct an architecture review for a new feature branch?▼

A code quality review for feature development assesses readability, DRY principles, complexity, elegance, and error handling. This Skill analyzes your changed files and surrounding context to provide actionable feedback on these specific code quality metrics.

What is the best way to check test coverage gaps and edge cases before merging?▼

The best way to check test coverage gaps and edge cases before merging is using this Skill's interactive test review. It evaluates changed files to identify missing integration coverage and assess overall test quality for your in-progress feature development.

Does this feature review process evaluate performance implications like query efficiency and concurrency?▼

Yes, the feature review process evaluates performance implications including query efficiency, memory usage, concurrency, and scaling. This Skill identifies these potential performance issues by interactively analyzing your planned feature development changes.

Can I use an interactive code review tool for in-progress feature development?▼

Yes, you can use this interactive code review tool for in-progress feature development. It conducts a multi-stage review of architecture, code quality, test coverage, and performance, providing user-driven decision points for each review section.

When do I need a structured architectural review for my feature work?▼

You need a structured architectural review for feature work before significant integration or merging a new feature branch. This ensures architectural soundness, verifies API design, and validates test coverage and performance implications early in the development cycle.