Use when writing acceptance criteria for a task - express each as an observable Given/When/Then that QA can execute, including negative cases
The English summary is being prepared. Showing the original description.
Use when writing React + TypeScript components - functional components, explicit typed props, derived state over effects, and no any/unsafe casts
The English summary is being prepared. Showing the original description.
See how to installReview the instructions the agent will receive before you install.
The compiler is your first test. The recurring defects are any/as casts that hide type errors, useEffect used where a derived value or event handler belongs, and state that should have been derived.
Core principle: Type at the source, derive over effect, functional components only.
any; avoid as casts — fix the type at the source.useState for genuinely local state; lift to a hook only when multiple components need it.useEffect has a correct dependency array and a cleanup when it subscribes/allocates.// ❌ effect + state to compute something derivable; `any` hides the shape
const [fullName, setFullName] = useState("");
useEffect(() => { setFullName(user.first + " " + user.last); }, [user]);
// ✅ derived value, typed
interface Props { user: User }
export function UserBadge({ user }: Props) {
const fullName = `${user.first} ${user.last}`; // derived, no state, no effect
return <span>{fullName}</span>;
}
The derived version has no effect to get wrong, no stale state, and the User type flows through.
any or as to silence the compiler instead of fixing the type.useEffect computing a value that could be derived inline.: any or as SomeType in the diff.useEffect whose only job is setState from props.No reviews yet. Tell others what you thought.
No login needed. Share how it worked for you, ask a question, or note where it did not run.
No comments yet. Be the first to write one.
Use when writing acceptance criteria for a task - express each as an observable Given/When/Then that QA can execute, including negative cases
The English summary is being prepared. Showing the original description.
Use when the diff adds or changes an endpoint, resolver, RPC, job or query that takes an object id, a role check, a request binding or a tenant filter - BOLA/IDOR, function-level authorization, mass assignment and tenant scoping
The English summary is being prepared. Showing the original description.
Use on every UI change - semantic HTML, labels for controls, keyboard-navigable dialogs/menus, visible focus, and never color as the only signal
The English summary is being prepared. Showing the original description.
Use when a task changes any screen, form, dialog, menu or control - Lighthouse/axe scan of the changed screens, a keyboard walk, and the thresholds that fail a task
The English summary is being prepared. Showing the original description.
How to work a task returned with review, QA or UAT findings. Use when a task is in need_revision or PR review comments are in your context.
The English summary is being prepared. Showing the original description.
Use when the repository keeps its alerting in code, or a person asks for alerts outside TaskTrooper - PrometheusRule YAML, a Cloud Monitoring alert policy in Terraform, a Datadog monitor, each routed to TaskTrooper's alert ingest URL so it opens an incident here
The English summary is being prepared. Showing the original description.