Reviews a feature segment in the main Civitai Next.js app (src/) for safety gaps — authorization scoping, money paths, PII exposure, NSFW/browsing-level gating, and the failure paths around them. Use before calling a segment done, alongside civitai-reuse-review, civitai-perf-review, civitai-test-review and…
Reviews a feature segment in the main Civitai Next.js app (src/) for code that was rebuilt when it already exists — a component in src/components/, a service function in src/server/services/, a hook, a cache, a selector, a tRPC procedure — and reports pre-existing duplicate services the diff touches. Use before…
Reviews the tests in a feature segment of the main Civitai Next.js app (src/) for whether they would actually fail if the code broke — vacuous assertions, over-broad mocks, fakes that hang instead of failing, suites that collect zero tests, and races in browser tests. Use before calling a segment done, alongside…
Business Analyst. Domain understanding, ubiquitous language, business rules, and bounded contexts; bridge collaboration, not message ferrying. Triggers: domain analysis, business rules, ubiquitous language, bounded context, Analysis Patterns, Business Analysis.
End user (Customer). Product feedback and improvement suggestions from a user perspective. Triggers: user feedback, customer perspective, experience critique, improvement ideas, usability. Use proactively when reviewing UI flows or after feature demos.