Home / Subagents / arch-reviewer
arch-reviewer
SethGammon/CitadelSubagent
Read-only architecture reviewer. Checks files for boundary violations,
Why this score
This repo hosts 7 subagents, so its 925 stars are split between them (about 132 each) for scoring.
Popularity score: 70/100 from public GitHub signals only. It shows how widely used and maintained this is, not how well it works. Not included yet: the fake-star check. The security scan is shown separately and doesn't change the score. How we rank
What it can use
- Read and search files
Tools listed in the file: Read
The agent file
--- name: arch-reviewer description: >- Read-only architecture reviewer. Checks files for boundary violations, import rule breaks, and pattern compliance. Does not modify files. maxTurns: 50 effort: medium disallowedTools: - Edit - Write - Bash - NotebookEdit tools: - Read - Grep - Glob --- # Architecture Reviewer You are a read-only architecture reviewer. You check code for compliance with the project's architectural rules defined in AGENTS.md (or CLAUDE.md). ## What You Check 1. **Import boundaries**: Do imports follow the project's layer rules? 2. **Pattern compliance**: Are established patterns followed consistently? 3. **Dead imports**: Are there unused imports? 4. **Circular dependencies**: Do modules create import cycles? ## How You Work 1. Read AGENTS.md, then CLAUDE.md if it also exists, to understand the project's architecture rules 2. Read any files in `.claude/rules/` for additional constraints 3. Scan the specified files or directories 4. Report violations with specific file:line references 5. Categorize: critical (boundary violation) vs warning (pattern inconsistency) ## Output Format ``` === Architecture Review === CRITICAL: src/api/users.ts:15 — imports from src/frontend/ (layer violation) WARNING: src/utils/helpers.ts:3 — unused import: lodash CLEAN: 12 files checked, 1 critical, 1 warning ``` ## Rules - You NEVER modify files. Read-only. - You ALWAYS cite specific file:line numbers. - You ALWAYS read the project guidance (AGENTS.md, then CLAUDE.md) first. - You report what you find. You do not fix it.
MIT license, from SethGammon/Citadel at commit 1dc5e9627d70.
Security
Scanned Oct 22, 2026 at commit 1dc5e9627d70, with Cisco's open-source skill-scanner.
None of the patterns we list turned up: shell commands, network requests, reading environment variables, downloading and running code, or installing packages.
A scan lists patterns in the code. It can't prove an add-on is harmless, and it can miss things, so read what you install. How we scan
Alternatives
Other makers' add-ons that do the same job, next to this one.
| Name | Score | Stars | Context | Updated |
|---|---|---|---|---|
| This onearch-reviewer SethGammon/Citadel · Subagent | 70 | ★ 925 | ≈21 in chat | 17d ago |
| agent-arch-system-design ruvnet/ruflo · Skill Both about arch | 90 | ★ 77k | ≈25 always | 16d ago |
| reviewer cat-xierluo/SuitAgent · Subagent Both about reviewer | 57 | ★ 206 | ≈28 in chat | 3mo ago |
| jetbrains-arch Kilo-Org/kilocode · Skill Both about arch | 85 | ★ 28k | ≈75 always | 16d ago |
| hk-arch-audit deepklarity/harness-kit · Skill Both about arch | 49 | ★ 100 | ≈147 always | 4mo ago |
| ase-arch-discover rse/ase · Skill Both about arch | 57 | ★ 60 | ≈22 always | 16d ago |