← ClaudeAtlas

review-core-boundarieslisted

Use when reviewing shared core/infrastructure module changes for dependency direction, feature-boundary leakage, abstraction stability, composition, and blast-radius risks. Dispatched by review-orchestrator for --core-boundaries, --project-conventions, --all, or src/core/** changes.
MrCipherSmith/helyx · ★ 1 · AI & Automation · score 68
Install: claude install-skill MrCipherSmith/helyx
# Review — Core Boundaries Reviewer for shared infrastructure modules. A core module should provide stable foundations used by feature modules; it should not accumulate feature-specific behaviour. --- ## Scope Applicable to folders such as `src/core/**`, `core/**`, `shared/**`, `foundation/**`, or whatever the repository documents as its shared infrastructure layer. If a more specific module reviewer also applies, run both. --- ## Checklist - Shared/core modules contain reusable utilities, base components, base stores, primitives, and infrastructure. - Feature-specific code does not move into core just to avoid imports. - Core does not import feature/domain modules. - Dependencies stay minimal and point inward: feature modules depend on core, not the reverse. - Prefer composition through interfaces, base classes, adapters, or callbacks over hard-coded feature knowledge. - Public core APIs remain stable and domain-neutral. - New exports are added only when there is a real shared consumer need. - Changes are conservative because core has broad blast radius. - Generic helpers remain generic; names, types, and state do not leak a single feature's language. - Resource-owning utilities document and test cleanup semantics. --- ## Orchestrated Review Contract When dispatched by `review-orchestrator`, follow the provided `reviewer-input.schema.json` payload. Return a `REVIEW_RESULT` object compatible with `skills/review-orchestrator/reviewer-finding.schema.json`, then a