Code Quality Review

A code-review guide that checks code quality, architecture, project standards, types, documentation, tests, security, and Git practices.

In plain words
What is it for?
Use it to review backend, frontend, test, authentication, and ingress changes against the listed project guidelines.
Why use it?
It turns a broad review into a defined checklist and applies the project's own rule files to the relevant parts of the codebase.

Skill for Claude CodeCodexCursor

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 skills/asap-protocol/asap-protocol/code-quality-review
Any agent
npx skills add asap-protocol/asap-protocol --skill code-quality-review
Clone the repo
git clone --depth 1 https://github.com/asap-protocol/asap-protocol

Made for: Claude Code, Codex, Cursor.

Per session 21 Skills are progressive disclosure: only the name and description are preloaded; the body loads when the skill is used.
When invoked 439 The whole file, excluding the scripts and references it only reads on demand.
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.00021 $0.00439
Opus 5 $0.00010 $0.00219
Sonnet 5 $0.00004 $0.00088
Haiku 4.5 $0.00002 $0.00044

Measured 3d ago against content hash 7c4a658a0dd1, method: parsed. Prices are Anthropic first-party input rates as of 2026-08-30, from the pricing page.

Security

Grade A, and why

Code Quality Review 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 3d 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.

.cursor/skills/code-quality-review/SKILL.md · 51 lines

What it actually says

Code Quality Review

You are an expert code quality reviewer.

Pre-requisites

Read before reviewing:

  1. Clean code (required): .cursor/rules/agent-clean-code.mdc
  2. Product layout: .cursor/rules/architecture-principles.mdc
  3. Backend: .cursor/rules/python-best-practices.mdc (when reviewing src/asap/ or tests/)
  4. Frontend: .cursor/rules/frontend-best-practices.mdc (when reviewing apps/web/ or packages/ui/)
  5. Tests: .cursor/rules/testing-standards.mdc (when the diff adds or changes tests)
  6. Security (ingress/auth changes): .cursor/rules/security-standards.mdc
  7. Git hygiene: .cursor/rules/git-commits.mdc
  8. Agent index: .cursor/README.md

Review process

1. Structure and architecture

  • Check separation of concerns; code in the right layer per architecture-principles.mdc.
  • Identify logic that belongs in services, handlers, or components.
  • Flag functions longer than 40 lines (split per agent-clean-code.mdc).

2. Standards compliance

  • Types: Are all public functions and methods typed?
  • Docs: Do public APIs have docstrings (intent + example)?
  • Tests: Per agent-clean-code.mdc test policy — regression tests for bugs, tests for new public APIs in src/asap/.

3. Error handling

  • Are exceptions caught specifically (not bare except)?
  • Do error messages include the offending value and expected shape?
  • Is there structured logging for failures?

Output format

Provide analysis with:

  • Severity: Critical / Major / Minor
  • Location: File and line
  • Recommendation: Code snippet of the fix
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. 3d ago First seen · 51 lines · 21 tokens per session scan A 7c4a658a0dd1

Subscribe to this mod's changes

Code Quality Review is a skill published in the GitHub repository asap-protocol/asap-protocol (14 stars, last pushed 3d ago), licensed Apache-2.0. It adds 21 tokens to every session and 439 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-30.

Related

Other skills, from other repositories

inkbox-python

Use when writing Python code that imports from inkbox, uses pip install inkbox, or when adding email, mailbox imports, phone, text/SMS, iMessage, A2A task/message history, contacts, notes, contact rules, vault, tunnels, mailbox storage, mail clients (IMAP/SMTP), or agent identity features using the Inkbox Python SDK.

inkbox-ai/inkbox · 80 tokens

inkbox-ts

Use when writing TypeScript or JavaScript code that imports from @inkbox/sdk, uses npm install @inkbox/sdk, or when adding email, mailbox imports, phone, text/SMS, iMessage, A2A task/message history, contacts, notes, contact rules, vault, tunnels, mailbox storage, mail clients (IMAP/SMTP), or agent identity features…

inkbox-ai/inkbox · 89 tokens

inkbox-cli

Use when running or writing shell commands with the Inkbox CLI (inkbox / @inkbox/cli) for identities, email, mailbox imports, phone, text/SMS, iMessage, A2A task/message history, contacts, notes, contact rules, vault, mailbox storage, mail clients (IMAP/SMTP), phone number, webhook, or signup workflows.

inkbox-ai/inkbox · 81 tokens

inkbox-tunnels

Use when bringing a local server online behind a public Inkbox URL — covers Python, TypeScript, and Rust tunnel runtimes, edge vs passthrough TLS, forwarding, handlers, and local liveness.

inkbox-ai/inkbox · 47 tokens

inkbox-agent-self-signup

Use when guiding or implementing the Inkbox agent self-signup flow, including verification, resend-verification, signup restrictions, and optional signup fields like agent handles or mailbox local parts.

inkbox-ai/inkbox · 43 tokens

inkbox-onboarding

Use when setting up an existing Inkbox identity for email, SMS, iMessage, calls, inbound handling, and recurring communications triage with the API, CLI, Python SDK, or TypeScript SDK.

inkbox-ai/inkbox · 46 tokens