plan-eng-review

Review engineering plans for architecture, data flow, tests, and performance.

Updated Mar 25, 2026
One-click install
npx skills add https://github.com/fghaffar/faystack --skill plan-eng-review-fghaffar
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: plan-eng-review
Source: https://github.com/fghaffar/faystack/tree/main/plan-eng-review
Command: npx skills add https://github.com/fghaffar/faystack --skill plan-eng-review-fghaffar

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

This Skill provides an engineering-manager level review to lock architecture, data flow, diagrams, edge cases, and test coverage before implementation. It prevents costly rework by surfacing architectural mistakes, missing tests, performance pitfalls, and scope creep early in the cycle.

Core Features & Use Cases

  • Interactive plan walkthroughs: Step-by-step review of design docs, branch context, and proposed implementation with concrete recommendations.
  • Scope and complexity checks: Step 0 evaluates minimal changes, files touched, and suggests scope reduction when designs are overbuilt.
  • Diagrams, tests, and rollout guidance: Generates ASCII diagrams, test coverage suggestions, performance checks, and deployment/rollback recommendations.
  • Repo ownership and process checks: Honors repo mode, searches for existing design docs, and writes a Plan Status Footer to the plan file.
  • Use Case: Before shipping a payments feature, run this Skill to validate API boundaries, database migrations, failure modes, test plans, and rollout strategy so implementation is safe to start.

Quick Start

Run /plan-eng-review on your current branch and provide a link or summary of the design doc to get an architecture audit, diagrams, and a prioritized test checklist.

Frequently Asked Questions about plan-eng-review

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

FAQPage Schema
How do I review an engineering plan before implementation to validate architecture and test coverage?

Engineering plan reviews validate architecture, data flow, edge cases, and test coverage before implementation. By evaluating design docs and proposed changes, this process surfaces architectural mistakes and missing tests early to prevent costly rework.

What is the best way to reduce scope and complexity in a software design doc?

Reducing scope in a software design doc involves evaluating minimal changes and identifying touched files to suggest scope reductions when designs are overbuilt. This complexity check ensures the proposed implementation remains lean and manageable before locking the plan.

How do I generate architecture diagrams and test requirements for a feature plan?

Architecture diagrams and test requirements for a feature plan are generated during an interactive walkthrough. The review produces ASCII diagrams, targeted test coverage suggestions, and performance checks to guide safe implementation and rollout.

Can I run an engineering review on a pre-implementation branch without a full design document?

Yes, you can run an engineering review on a pre-implementation branch by providing a summary of the proposed change. The review searches for existing design docs, analyzes the branch context, and writes a completion status footer directly to the plan file.

Does an engineering plan review provide deployment and rollback guidance for database migrations?

Engineering plan reviews provide deployment and rollback guidance for database migrations and API boundaries. The review assesses failure modes and rollout strategy to ensure the implementation plan is safe to start.

What limitations should I consider when locking architecture plans for repository changes?

When locking architecture plans, limitations include respecting repository ownership modes and telemetry settings. The review process checks repo mode and honors existing repository configurations to ensure proposed changes align with established boundaries.