Prompt file imported from reuben2crown/disclo (
.cursor/commands/code-review.md). Copyright stays with the author.
Review the changes on @branch:
- Think through how data flows in the app. Explain new patterns if they exist and why.
- Were there any changes that could affect infrastructure?
- Consider empty, loading, error, and offline states.
- Review frontend changes for a11y (keyboard navigation, focus management, ARIA roles, color contrast).
- If public APIs have changed, ensure backwards compat (or increment API version).
- Did we add any unnecessary dependencies? If there's a heavy dependency, could we inline a more minimal version?
- Did we add quality tests? Prefer fewer, high quality tests. Prefer integration tests for user flows.
- Were there schema changes which could require a database migration?
- Changes to auth flows or permissions? Run /security-review.
- If feature flags are set up, does this change require adding a new one?
- Are there places we should use caching?
- Are we missing critical o11y or logging on backend changes?
Disclo-specific checks
- Is
packages/corefree of imports frompackages/adapters? - Do all new adapter files implement only the
PlatformAdapterinterface methods? - Is plan enforcement middleware applied before business logic on all new premium routes?
- Are OAuth tokens and API keys encrypted via
packages/core/src/crypto.ts? - Are all inbound webhook signatures verified before processing?
- Does
process.envappear only inpackages/core/src/config.ts? - Are all external API calls (GPTZero, Hive, Anthropic, Stripe) wrapped in typed error classes?
- Are tests written before implementation (TDD)? Check commit history.
- Are there any
test.skiporit.skipin the new test files? - Does the audit log record all new compliance actions?
- For Green Claims changes: is there a consumer-visible badge, or is it correctly merchant-facing only?
