Skillshub common-architecture-audit
Protocol for auditing structural debt, logic leakage, and fragmentation across Web, Mobile, and Backend. (triggers: package.json, pubspec.yaml, go.mod, pom.xml, nest-cli.json, architecture audit, code review, tech debt, logic leakage, refactor)
install
source · Clone the upstream repo
git clone https://github.com/ComeOnOliver/skillshub
Claude Code · Install into ~/.claude/skills/
T=$(mktemp -d) && git clone --depth=1 https://github.com/ComeOnOliver/skillshub "$T" && mkdir -p ~/.claude/skills && cp -r "$T/skills/HoangNguyen0403/agent-skills-standard/common-architecture-audit" ~/.claude/skills/comeonoliver-skillshub-common-architecture-audit && rm -rf "$T"
manifest:
skills/HoangNguyen0403/agent-skills-standard/common-architecture-audit/SKILL.mdsource content
Architecture Audit
Priority: P1 (STANDARD)
📋 Audit Protocol
1. Structural Duplication Discovery (Universal)
Identify split sources of truth by searching for redundant directory patterns or naming conventions.
- Services: Compare
vsService.ts
vsServiceNew.ts
.ServiceV2.ts - Versioning: Check for
,/v1
or "Refactor" folders./v2 - Action:
# Identify potential duplicates or legacy files find . -type f -name "*New.*" | sed 's/New//'
2. Logic Leakage Analysis (Ecosystem Specific)
Detect business logic trapped in the wrong layer (e.g., UI layer in apps, Controller layer in APIs).
🌐 Web (React/Next.js/Vue)
- Action:
grep -rE "useEffect|useState|useMemo" components --include="*.tsx" | wc -l - Threshold: If
hook count > 20xcomponents/
folder, architecture is Monolithic.hooks/
📱 Mobile (Flutter/React Native)
- Action:
grep -rE "http\.|dio\.|socket\." lib/widgets --include="*.dart" | wc -l - Threshold: I/O or state mutation > 5 lines in
is 🟠 High Debt.build()
⚙️ Backend (NestJS/Go/Spring)
- Action:
grep -rE "Repository\.|Query\.|db\." src/controllers --include="*.ts" | wc -l - Threshold: Controllers must only handle request parsing and response formatting.
3. Monolith Detection (Ecosystem Specific)
Identify massive files violating Single Responsibility Principle.
- Thresholds:
- UI: > 500 lines (🟡 Medium), > 1,000 lines (🔴 Critical).
- Backend Services: > 1,500 lines indicate "God Class".
- Action:
find . -type f \( -name "*.tsx" -o -name "*.dart" -o -name "*.go" -o -name "*.java" \) | xargs wc -l | awk '$1 > 1000'
4. Resource Performance Audit (Universal)
Check for large metadata or constants impacting IDE performance and binary size.
- Threshold: Resources > 1,000 lines require granulation.
- Action:
find . -type f \( -name "*constants*" -o -name "*.graphql" -o -name "*strings*" \) | xargs wc -l | awk '$1 > 1000'
⚖️ Scoring Impact
- Layer Violation: -15 per business logic instance in UI/Controller layer.
- Structural Fragmentation: -10 per duplicated legacy entity.
- Monoliths: -10 per unit > 1,000 lines.
📚 Reference Links
🚫 Anti-Patterns
- Do NOT use standard patterns if specific project rules exist.
- Do NOT ignore error handling or edge cases.