implementation-review-checklist

A checklist for reviewing an implementation before giving the final response.

In plain words
What is it for?
Use it after a substantial implementation to inspect the diff, responsibility boundaries, API behavior, state consistency, duplication, regression paths, and tests.
Why use it?
It helps find problems in how the frontend, backend, storage, APIs, state, and tests fit together—not just whether the feature works once.

Cursor rule for Cursor

Install

Getting it into your agent

One page per mod, every tool's command on it. A separate URL per tool would split the same page into five that compete with each other.

agentmods
npx agentmods add rules/microsoft/data-formulator/implementation-review-checklist
Clone the repo
git clone --depth 1 https://github.com/microsoft/data-formulator

Made for: Cursor.

Per session 324 This file is loaded in full into every session.
When invoked 324 The same file — it is already loaded in full.
Security scan A 0 findings. Scan, not verified.
Origin original No closer match found in the catalogue.
Token cost

What it costs to keep this loaded

Counted locally with the o200k_base tokenizer, which is exact for GPT models; Claude uses its own tokenizer and its counts differ. Treat this as one consistent yardstick across the catalogue rather than a bill. Prices are per million input tokens.

ModelPer sessionOnce invoked
Fable 5 $0.00324 $0.00324
Opus 5 $0.00162 $0.00162
Sonnet 5 $0.00065 $0.00065
Haiku 4.5 $0.00032 $0.00032

Measured yesterday against content hash cd161143e4bb, method: parsed. Prices are Anthropic first-party input rates as of 2026-08-30, from the pricing page.

Security

Grade A, and why

implementation-review-checklist scanned grade A with 0 findings against 26 rules in 11 categories — prompt injection, anti-refusal, data exfiltration, privilege escalation, supply chain, agent snooping, system-prompt leakage, SSRF and excessive agency — measured yesterday.

A static scan of the body, not an audit. Every finding is printed with the line that produced it so you can judge whether it matters here. A mod is markdown that instructs an agent; that is exactly why what it instructs is worth reading.

Nothing flagged

None of the 26 patterns this scan looks for appear in this file: no shell pipes, no recursive deletes, no credential paths, no hidden text, no instruction-override or anti-refusal phrasing, no agent-config snooping. That is not a guarantee, it is the absence of the things that are checkable.

.cursor/rules/implementation-review-checklist.mdc · 29 lines

What it actually says

Implementation Review Checklist

After any substantive implementation, before the final response, review the uncommitted diff from a maintainer/code-review perspective. Do not only verify that the feature works.

Required Checks

  • Inspect git diff --stat and relevant diffs to understand the actual blast radius.
  • Check responsibility boundaries: frontend UI, API route, service/helper, storage/session, and tests should each own the right part of the behavior.
  • Check API semantics: endpoint names, HTTP methods, response shapes, and side effects must match the product action. Similar UI actions should call the same backend API.
  • Check frontend/backend state consistency: after mutations, make sure Redux/local state, backend session/token/vault state, and subsequent list/status APIs agree.
  • Check duplication: if two paths clear the same state or perform the same API flow, extract a small helper or route instead of copying logic.
  • Check regression surfaces: refresh/reload/list/status paths must respect new flags or state, not just the primary click path.
  • Check tests: add or update focused tests for the changed contract; include at least one test for the bug/regression path when practical.
  • Check docs/rules/dev-guides when the change introduces or changes a cross-cutting convention.

Reporting

If issues are found during this review, fix clear issues before finalizing. If a trade-off remains, call it out explicitly in the final response with the risk and why it was left as-is.

Changes

What this file has done since we first saw it

Hashed on every crawl. A supply-chain change to an agent config is a question of when, not whether, so the history is kept rather than the latest state alone.

  1. yesterday First seen · 29 lines · 324 tokens per session scan A cd161143e4bb

Subscribe to this mod's changes

implementation-review-checklist is a cursor rule published in the GitHub repository microsoft/data-formulator (17,048 stars, last pushed 3d ago), licensed MIT. It adds 324 tokens to every session, about $0.0016 per session on Opus 5. A static security scan graded it A with 0 findings. No closer match exists in the catalogue, so it is treated as the original; first seen 2026-08-30.