quality-reviewer

quality-reviewer is a cursor rule for Cursor from dasomel/oh-my-cursor. It costs 16 tokens per session (684 once invoked), scanned A, original, MIT.

A code-review guide focused on whether software is logically correct, maintainable, well-designed, and efficient. It checks for design problems such as overly large objects, repeated changes across many files, and excessive responsibilities.

In plain words
What is it for?
Use it after major code changes, before merging a feature branch, or when code works but its design, performance, or maintainability is in question.
Why use it?
It finds structural problems that may not cause an immediate error but can make future changes slower, riskier, or harder to understand.

Cursor rule for Cursor

Written for Cursor: a Cursor rule (.mdc).

Good fit Use it after major code changes, before merging a feature branch, or when code works but its design, performance, or maintainability is in question.

Compare 6 cursor rules from other repositories ↓
Install with agentmods
npx agentmods add rules/dasomel/oh-my-cursor/quality-reviewer
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.

Clone the repo
git clone --depth 1 https://github.com/dasomel/oh-my-cursor

Made for: Cursor.

Wrote this? Show the measurements

A badge with what this costs and how it scanned, read live from this page, so it follows the numbers instead of freezing them. Markdown for a README, HTML for a documentation site or a project page.

agentmods badge for quality-reviewer

README.md
[![agentmods](https://agentmods.dev/badge/rules/dasomel/oh-my-cursor/quality-reviewer.svg)](https://agentmods.dev/rules/dasomel/oh-my-cursor/quality-reviewer)
Your own site
<a href="https://agentmods.dev/rules/dasomel/oh-my-cursor/quality-reviewer"><img src="https://agentmods.dev/badge/rules/dasomel/oh-my-cursor/quality-reviewer.svg" alt="Measured on agentmods" height="20"></a>
Per session 16 Only the description is in the session, so the agent can decide to use it. The body loads when it is invoked.
When invoked 684 The whole file, excluding the scripts and references it only reads on demand.
Security scan A 0 findings. A grade says what 26 rules found in the file — not that it is safe.
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.1 $0.00016 $0.00684
Opus 5 $0.00008 $0.00342
Sonnet 5 $0.00003 $0.00137
Haiku 4.5 $0.00002 $0.00068

Measured 6d ago against content hash dda5bee30bbd, method: parsed. Prices are Anthropic first-party input rates as of 2026-09-07, from the pricing page.

Security

Grade A, and why

quality-reviewer 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 6d ago.

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.

rules/agents/quality-reviewer.mdc · 99 lines

How it starts

The opening of the file, as written. The whole thing — 99 lines — stays where its author put it; the contents beside it link to each section on GitHub.

Quality Reviewer Role

You are the Quality Reviewer — a specialist focused on code quality beyond surface-level review. You analyze logic correctness, design patterns, SOLID principles, and performance characteristics.

When to Adopt This Role

  • After significant code changes
  • Before merging feature branches
  • When code "works but doesn't feel right"
  • When the user asks for quality review or design review

How Quality Reviewer Differs from Reviewer

Aspect Reviewer Quality Reviewer
Focus Code correctness, style Design quality, patterns
Depth Line-by-line review Structural analysis
Output Comments with severity Pattern analysis report
Scope Changed files Changed files + their dependencies

Protocol

1. Pattern Analysis

Check for these anti-patterns:

- God objects / God functions (>200 lines or >5 responsibilities)
- Shotgun surgery (one change requires modifying many files)
- Feature envy (method uses another class's data more than its own)
- Primitive obsession (using primitives instead of domain types)
- Long parameter lists (>4 parameters)
- Boolean blindness (passing booleans instead of enums/types)
- Temporal coupling (methods must be called in specific order)
- Law of Demeter violations (a.b.c.d chaining)

2. SOLID Principles Check

- S: Does each module have a single reason to change?
- O: Can behavior be extended without modifying existing code?
- L: Can subtypes be substituted without breaking behavior?
- I: Are interfaces focused, or do they force unused implementations?
- D: Do high-level modules depend on abstractions?

3. Logic Correctness

- Off-by-one errors
- Null/undefined handling
- Race conditions in async code
- Error propagation completeness
- State machine validity
- Boundary conditions

4. Performance Review

- N+1 query patterns
- Unnecessary re-renders (React)
- Missing memoization for expensive computations
- Unbounded collections in memory
- Missing pagination
- Synchronous operations that should be async

Read the full file on GitHub · 99 lines

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. 6d ago First seen · 99 lines · 16 tokens per session scan A dda5bee30bbd

Subscribe to this mod's changes

quality-reviewer is a cursor rule published in the GitHub repository dasomel/oh-my-cursor (2 stars, last pushed 6mo ago), licensed MIT. It adds 16 tokens to every session and 684 once invoked, about $0.0001 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-31.