engineering-arch-principle-engineer

Reviews existing codebases and architectures to produce prioritized, principle-grounded engineering critique reports.

Updated Jan 28, 2026
One-click install
npx skills add https://github.com/scanady/nexus-skills --skill engineering-arch-principle-engineer-scanady
Or copy as Structured Prompt for Agent▼
Please help me install this Agent Skill.
Skill: engineering-arch-principle-engineer
Source: https://github.com/scanady/nexus-skills/tree/main/skills/engineering-arch-principle-engineer
Command: npx skills add https://github.com/scanady/nexus-skills --skill engineering-arch-principle-engineer-scanady

SYSTEM DOCUMENTATION & REQUIREMENTS

💡 This Skill includes references (resource) components.

What problem does it solve? Engineering teams often receive vague or noisy code reviews that mix stylistic preferences with real architectural issues, or get pushed toward costly rewrites when a small patch would suffice. This Skill delivers a disciplined, senior-level review of an existing codebase that separates genuine defects from taste, ranks findings by leverage, and recommends the smallest viable change. ## Core Features & Use Cases - Principle-grounded critique: Every recommendation cites a named engineering principle (smallest viable change, distillation, surgical change, short feedback loops, subtraction over addition) drawn from Karpathy, Cherny, and Dohmke. - Structured six-step workflow: Frame, survey, diagnose, prioritize, recommend, and explicitly state what NOT to do, preventing premature-abstraction and rewrite-happy advice. - Standardized report output: Produces a fixed-format report with system read, strengths to preserve, top 3 highest-leverage changes, detailed findings with file:line references, and open questions. - Use Case: Ask it to review a service that has grown organically for two years. It reads the entry points, API surface, tests, and build scripts, then returns a report identifying three high-leverage simplifications (e.g., a single-implementer interface to delete, a slow test loop to fix) plus a list of tempting refactors to avoid. ## Quick Start Ask the agent to review this codebase for over-engineering and tell me the top three highest-leverage changes to make.

Frequently Asked Questions about engineering-arch-principle-engineer

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

FAQPage Schema
How do I get a senior-level code review of my codebase?▼

Invoke this Skill and point it at your repository. It reads the README, entry points, public API surface, tests, and build scripts, then produces a structured report ranking findings by impact and ease, with a concrete surgical fix for each.

How to tell if my code is over-engineered?▼

The review checks for premature abstraction signals: interfaces with one implementer, factories building one type, plugin systems with one plugin, and indirection without payoff. Each finding cites the violated principle and the smallest change to fix it.

Can this review a greenfield system design before I build it?▼

No. This Skill is explicitly scoped to critiquing existing codebases and architectures, not greenfield design. It also does not perform security audits; use a dedicated system-design or security-review skill for those tasks.

What does the engineering review report contain?▼

The report has six sections: a system read, strengths to preserve, the top three highest-leverage changes, detailed findings with file references and before/after fixes, a what-NOT-to-do list, and open questions for the team.

Will it recommend rewriting my project?▼

Only with evidence of genuine architectural failure. The default bias is the smallest viable change: a 10-line patch over a rewrite, subtraction over addition, and matching the codebase's existing conventions rather than imposing new ones.