review-readability

review-readability is an agent for Claude Code from Gentleman-Programming/gentle-pi. It costs 26 tokens per session (754 once invoked), scanned A, original, MIT.

A read-only code review agent focused on how easy software is to understand and maintain. It checks names, complexity, duplication, unused code, and whether a change is explained clearly.

In plain words
What is it for?
Use it to review pull requests or code changes for magic numbers, long parameter lists, duplicated logic, dead code, unclear naming, and vague context.
Why use it?
It finds clarity and maintenance problems without modifying the code, and requires concrete evidence for claims about complexity or unclear design.

Agent for Claude Code

Written for Claude Code: a Claude Code subagent (agents/*.md).

Good fit Use it to review pull requests or code changes for magic numbers, long parameter lists, duplicated logic, dead code, unclear naming, and vague context.

Compare 6 agents from other repositories ↓
Install with agentmods
npx agentmods add agents/gentleman-programming/gentle-pi/review-readability
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/Gentleman-Programming/gentle-pi

Made for: Claude Code.

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 review-readability

README.md
[![agentmods](https://agentmods.dev/badge/agents/gentleman-programming/gentle-pi/review-readability/github.svg)](https://agentmods.dev/agents/gentleman-programming/gentle-pi/review-readability)
Your own site
<a href="https://agentmods.dev/agents/gentleman-programming/gentle-pi/review-readability"><img src="https://agentmods.dev/badge/agents/gentleman-programming/gentle-pi/review-readability/github.svg" alt="Measured on agentmods" height="20"></a>

Or the 80×15 button, for a site that already has a row of RSS and ATOM ones. Only the verdict fits; the numbers stay here.

agentmods 80×15 button for review-readability

Your own site · 80×15
<a href="https://agentmods.dev/agents/gentleman-programming/gentle-pi/review-readability"><img src="https://agentmods.dev/badge/agents/gentleman-programming/gentle-pi/review-readability.svg" alt="Reviewed on agentmods" width="80" height="20"></a>
Per session 26 Only the description is in the session, so the agent can decide to use it. The body loads when it is invoked.
When invoked 754 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.00026 $0.00754
Opus 5.5 $0.00010 $0.00302
Sonnet 5.5 $0.00005 $0.00151
Haiku 4.5 $0.00003 $0.00075

Measured 1mo ago against content hash 9db1806ff9de, method: parsed. Prices are Anthropic first-party input rates as of 2026-10-07, from the pricing page.

Security

Grade A, and why

review-readability 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 1mo 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.

assets/agents/review-readability.md · 73 lines

How it starts

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

Manual/compat-lane only: the provider host-relay capture path never loads this agent definition; native lens capture materializes the Go-issued opaque prompt through the gentle-pi host relay.

You are R2 Readability, a read-only reviewer. Find clarity problems; do not fix them.

Review rules

  • Flag magic numbers that should be named constants or business-rule objects.
  • Flag long parameter lists that should be parameter objects.
  • Flag duplicated logic across components/hooks/modules.
  • Flag dead code: commented-out blocks, unused imports, unreachable branches, never-called functions.
  • Flag naming that hides intent or needs comment-heavy explanation.
  • Flag PR/context explanation that is too vague to review safely; require concrete intent and impact.
  • Require evidence for "too complex" claims: cite exact function, branch, or repeated pattern.
  • Do not flag a small helper or inline constant that is clear, local, and self-explanatory.

Output contract

Report findings only. Each finding must include severity: BLOCKER | CRITICAL | WARNING | SUGGESTION, affected files, evidence, and why it matters. If clean, return an empty findings ledger (a ledger record with zero rows) — never skip the ledger.

Review ledger contract

Run this selected lens exactly once against the supplied initial_review_tree.

Return candidate rows only; the controller freezes canonical rows and owns every authorization decision.

Do not persist state, mutate claims, launch actors, request fixes, validate fixes, or deliver anything.

Every candidate must include exact location, severity, claim, evidence_class (deterministic | inferential | insufficient), causal_disposition (introduced | behavior-activated | worsened | pre-existing | base-only | unknown), and proof_refs. Use only concrete changed-hunk:, candidate-created-path:, differential-test:, or before-after: proof. A stable ID is preferred; the controller assigns a missing ID. WARNING and SUGGESTION candidates are informational. If clean, return an empty candidate list.

Read the full file on GitHub · 73 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. 1mo ago First seen · 73 lines · 26 tokens per session scan A 9db1806ff9de

Subscribe to this mod's changes

review-readability is an agent published in the GitHub repository Gentleman-Programming/gentle-pi (804 stars, last pushed 1mo ago), licensed MIT. It adds 26 tokens to every session and 754 once invoked, about $0.0001 per session on Opus 5.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 agents, from other repositories

code-reviewer-design

name: code-reviewer-design description: Specialized code reviewer for spec compliance, architecture consistency, and pattern drift. Returns scored findings against Product-Spec and project conventions. skills: code-review model: inherit.

zxpmail/ReqForge · 0 tokens

code-reviewer-bug

name: code-reviewer-bug description: Specialized code reviewer for bug patterns — null safety, race conditions, resource leaks, logic and error-handling defects. Returns scored findings (severity × impact × confidence). skills: code-review model: inherit.

zxpmail/ReqForge · 0 tokens

code-reviewer-security

name: code-reviewer-security description: Specialized code reviewer for security — credential leaks, injection, XSS, path traversal, unsafe eval/deserialization, deprecated APIs. Returns scored findings. skills: code-review model: inherit.

zxpmail/ReqForge · 0 tokens

code-reviewer-types

name: code-reviewer-types description: Specialized code reviewer for type safety — language-aware nullability, unsafe casts, missing unions, broad params. Returns scored findings. skills: code-review model: inherit.

zxpmail/ReqForge · 0 tokens

code-reviewer

name: code-reviewer description: Dispatched by the main Agent when code review is needed. Coordinates parallel specialized review agents and aggregates their findings. skills: code-review model: opus color: red.

zxpmail/ReqForge · 0 tokens

architecture-scanner

Scan the codebase for deepening opportunities — shallow modules, pass-throughs, semantic duplicates. Read-only. Produces a visual HTML report with before/after diagrams. Routes: CODEBASE-HEALTH workflow.

romiluz13/cc10x · 47 tokens