tomevault-io Review pull request $ARGUMENTS for the Sword Art Online project.
Get the PR diff: !gh pr diff $0 2>/dev/null || echo "No PR number provided, reviewing staged changes" && git diff --staged
Review Criteria
- Architecture compliance — follows Clean Architecture layers, no boundary violations
- Effect-TS patterns — correct use of Effect.gen, Context.Tag, Layer, error types
- Security — no client trust, server validates everything, no hardcoded secrets
- Database — Kysely fluent API used correctly, migrations have matching up/down
- Type safety — branded types used, no
anyorascasts without justification - Tests — use cases have tests, domain logic has tests
- Module boundaries — no cross-module imports, EventBus for communication
Format: list issues by severity (blocker > warning > nit), with file paths and line numbers.
Converted and distributed by TomeVault — claim your Tome and manage your conversions.