Home / Subagents / arch-reviewer

arch-reviewer

SethGammon/CitadelSubagent

Read-only architecture reviewer. Checks files for boundary violations,

ScannedSubagentSource: GitHub
925 stars Updated 17d ago License MIT Context ≈21 tokens in your chat
70popular

Why this score

Stars15/35
Usage (forks)5/15
Upkeep25/25
License15/15
Owner history10/10

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.

NameScoreStarsContextUpdated
This onearch-reviewer SethGammon/Citadel · Subagent 70★ 925≈21 in chat17d ago
agent-arch-system-design ruvnet/ruflo · Skill Both about arch 90★ 77k≈25 always16d ago
reviewer cat-xierluo/SuitAgent · Subagent Both about reviewer 57★ 206≈28 in chat3mo ago
jetbrains-arch Kilo-Org/kilocode · Skill Both about arch 85★ 28k≈75 always16d ago
hk-arch-audit deepklarity/harness-kit · Skill Both about arch 49★ 100≈147 always4mo ago
ase-arch-discover rse/ase · Skill Both about arch 57★ 60≈22 always16d ago