skills/architecture-audit/SKILL.md
Comprehensive architecture audit that combines ruthless analysis with solution-focused improvement planning. Reads architecture Markdown files, produces a brutal audit report with file/component scores, and generates a prioritized improvements roadmap.
npx skillsauth add harshitsinghbhandari/domain-expansion architecture-auditInstall this skill globally with one command. Works with Claude Code, Cursor, and Windsurf.
3 of 9 scanners reported clean
Some scanners were skipped, did not run, or reported a non-clean status. Review each row below.
This skill provides two-phase architecture review:
Both outputs are produced in a single run, enabling teams to understand what's broken and how to fix it systematically.
Explicit triggers only. This skill activates ONLY when the user explicitly mentions:
Required input: User must provide or reference architecture-related Markdown files (e.g., architecture.md, design.md, layers.md, README-architecture.md, or paste content directly). If no files are provided, respond normally and politely request: "Please provide the architecture Markdown files you'd like audited."
File format: All audited content is in Markdown (text descriptions, tables, Mermaid/PlantUML diagrams, decision records, etc.). Do NOT attempt to audit raw source code—this skill is for architecture documentation.
audit.md)The audit phase adopts a no-nonsense, penetrating analytical approach:
Generate audit.md with this exact structure:
# Architecture Audit Report
## Executive Summary
- **Overall Score**: X/1000
- **Feasibility Verdict**: [Feasible / Partially Feasible / Not Feasible]
- **Primary Strengths**: ...
- **Critical Weaknesses**: ...
## File/Component Scores
| File/Path | Score /100 | Assessment |
|-----------|------------|------------|
| [file] | [score] | [one-line verdict] |
## Detailed Findings
### Consistency Analysis
[Findings on consistency within and across files]
### Feasibility & Scalability Assessment
[Analysis of whether the architecture can support stated goals/scale]
### Architectural Quality
[Assessment of modularity, separation of concerns, adherence to principles]
### Maintainability & Evolution
[How easy is it to understand, modify, extend?]
### Identified Risks
[Technical debt, single points of failure, etc.]
## Final Verdict
[Summary of overall health and whether major rework is needed]
Per-file scores (/100):
Overall score (/1000):
Evaluation criteria:
improvements.md)The improvements phase adopts a calm, methodical, strategic mindset:
Generate improvements.md with this exact structure:
# Architecture Improvements Roadmap
## Critical Issues
[Issues that must be fixed before or during implementation; they block success or introduce unacceptable risk]
### Issue: [Name]
- **Location**: [File/section reference]
- **Problem**: [Clear description]
- **Impact**: [Why this matters]
- **Suggested Approach**: [Solution outline, pseudocode if helpful]
## Medium Priority Issues
[Issues that should be addressed soon; they degrade quality, maintainability, or scalability]
### Issue: [Name]
- **Location**: [File/section reference]
- **Problem**: [Clear description]
- **Impact**: [Why this matters]
- **Suggested Approach**: [Solution outline]
## Nice-to-Have Enhancements
[Optimizations, future-proofing, or quality-of-life improvements; not blocking but valuable]
### Enhancement: [Name]
- **Location**: [File/section reference]
- **Description**: [What could be improved]
- **Benefit**: [Why it's worth doing]
- **Suggested Approach**: [Solution outline]
audit.md with ruthless, systematic analysis.improvements.md with solution-focused roadmap.development
Aggressive user-flow and boundary-bug analysis on a diff or branch. Auto-detects entry points, traces flows through changed code, finds every seam (cross-module calls, serialization, file I/O, shared state, schema versioning, network/IPC), and refuses to mark the work complete until each unverified boundary has a real round-trip test or an explicit written out-of-scope record persisted in an audit file. Use whenever the user says "boundary check", "seam check", "round-trip check", "flow boundaries", "user-flow check", "before merge", "is this safe to ship", "pre-merge gate", "boundary bugs", "verify the joins", or asks to validate cross-module joins, producer/consumer contracts, or end-to-end coverage of a change. Also use as a final gate from pr-review on any diff that touches more than one file, module, or process. Be pushy. Most surviving production bugs live at seams, not inside units — if the diff crosses any boundary, this skill almost certainly applies.
development
Run existing work through 5 specialist craftspeople who each produce an improved version, then peer-review and synthesize the best into a single improved artifact. Use when the user says "forge this", "improve this", "make this better", "level this up", "refine this", or asks for multi-angle improvement on code, copy, strategy, plans, designs, or any artifact where the current version works but could be significantly better. Do NOT use for decisions (use llm-council), simple edits, or creation from scratch.
development
Expert skill for maintaining a Keep a Changelog formatted CHANGELOG.md file. Use this skill whenever you add features, fix bugs, or release a new version. You MUST use this skill to record any changes that have a user-facing impact. It handles categorization (Added, Changed, Fixed, etc.), semantic versioning, and reverse-chronological ordering with surgical precision.
development
Comprehensive test suite audit that combines ruthless analysis with a solution-focused roadmap. Reads test suites (unit, integration, e2e) and source code, produces a brutal audit report of test quality and gaps, and generates prioritized testing improvements.