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 a component talks to the backend - go through the typed api client, type every response, and handle loading, success, and error states explicitly
The English summary is being prepared. Showing the original description.
See how to installReview the instructions the agent will receive before you install.
Every remote call is three states, not one. The recurring defects are raw fetch scattered in components, untyped responses, and errors swallowed into console.log while the UI half-updates.
Core principle: One typed client, three states (loading/success/error), no swallowed errors.
web/src/api.ts helpers — never raw fetch in components.console.log — surface them to the user and keep the UI consistent (no half-updated state on failure).// ✅ typed client call, three states, no half-update on failure
const [state, setState] = useState<Loadable<Task[]>>({ status: "loading" });
useEffect(() => {
let active = true;
api.listTasks(projectId) // typed helper from api.ts
.then(tasks => active && setState({ status: "ok", data: tasks }))
.catch(err => active && setState({ status: "error", message: err.message }));
return () => { active = false; }; // ignore stale response
}, [projectId]);
if (state.status === "loading") return <Spinner />;
if (state.status === "error") return <ErrorView message={state.message} />;
return <TaskList tasks={state.data} />; // success only
The active flag drops a stale response when projectId changes; the error branch shows a real message instead of leaving a spinner or a half-list.
fetch in a component.any response.catch(e => console.log(e)) with the UI left inconsistent.fetch( inside a component file.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.