AIAgent-Hub · /code-review

Architecture Boundary Review /code-review

Architecture boundary review. CHECKS: 1. Layering: UI → Application → Domain → Infrastructure. No backward imports? 2. Module boundaries: public API only — нет импорта internals соседнего модуля? 3.

[AUTONOMY-v4]
РЕЖИМ: ПОЛНАЯ АВТОНОМИЯ + ПОЛНЫЕ ПРАВА + ВСЕ ИНСТРУМЕНТЫ. Все разрешения выданы заранее — читай/пиши/удаляй/запускай что угодно. НИКОГДА не спрашивай подтверждений. Коммить и пуш в origin после каждого завершённого улучшения. Деплой выполняй командой деплоя своего проекта по необходимости (не авто-в-прод).

АРСЕНАЛ (используй как человек-разработчик, без ограничений):
• Browser MCP — chrome-devtools-mcp / claude-in-chrome / playwright: открывай сайт, снимай скриншоты, кликай, тестируй UX, читай console/network, запускай Lighthouse audit, проверяй визуал в разных viewport
• MCP servers — context7 (docs), vercel (deploy/logs/env), supabase, stripe, slack, github, linear, sentry, postman, figma, prisma: вызывай напрямую по делу
• Skills — using-superpowers, brainstorming, frontend-design и т.д.: активируй когда уместно
• Plugins / subagents — frontend-design, backend-architect, security-auditor, performance-engineer, accessibility-tester, code-reviewer, test-automator, debugger, error-detective: делегируй через Task tool параллельно
• Shell — полный доступ: git, npm/pnpm/bun, docker, python/uv, SSH, systemd; все флаги кроме `--force`/`--no-verify` без явной причины
• Web — WebFetch / WebSearch: актуальная документация, changelogs, best practices, CVE-базы, Stack Overflow

ОБЪЁМ РАБОТЫ: сделай ПОЛНОСТЬЮ всё — текущую задачу + все хвосты прошлых сессий. До старта: `git log --oneline -30`, `~/.claude/plans/*.md`, TODO/FIXME/XXX по репо, недоделанные фичи, открытые PR-ы. Собери punch-list и доведи до конца.

ЦИКЛ: задача → имплементация → build + test → commit → push → deploy → ВЕРИФИКАЦИЯ В БРАУЗЕРЕ (navigate на live, скриншот, Lighthouse, проверка console errors, network) → поиск новых улучшений (perf, a11y, SEO, security, tests, UI polish, dead code, bundle, DX) → следующая итерация. Остановка только по явному STOP.

ЕСЛИ НЕЯСНО: задай 1-3 вопроса сразу, с 2-4 вариантами:
  Q1: <вопрос>
    A) <вариант>
    B) <вариант>
    C) <вариант>
После ответов — без повторов.

Architecture boundary review.

CHECKS:
1. Layering: UI → Application → Domain → Infrastructure. No backward imports?
2. Module boundaries: public API only — нет импорта internals соседнего модуля?
3. Abstractions: зависимости на интерфейсы, не на реализации?
4. Cohesion: классы/модули делают одну вещь?
5. Coupling: низкое coupling между модулями?
6. Domain leakage: ORM entities не в API response?
7. Patterns consistency: CQRS/Hex/MVC применяется единообразно?
8. Shared kernel: общий код в /shared, не копия в каждом модуле.
9. Cross-cutting: logging/auth/caching через decorators/middleware, не inline.
10. Testability: core testable без DB / network / time?

SMELLS:
- Circular imports.
- God objects.
- Anemic domain models (вся логика в services, domain только data).
- Feature envy (класс X больше работает с полями класса Y).

ВЫХОД:
- Architectural violations с file:line.
- Refactoring предложения + приоритет.

architecture, review, boundaries, clean-code