coding-guidelines-reviewer

A read-only reviewer that checks finished changes against a repository's coding rules, testing requirements, and completion checklist.

In plain words
What is it for?
Use it after a change to select the appropriate rigor level and check requirements such as testing, coverage, and the project's definition of done.
Why use it?
It identifies process and quality gaps before work is marked complete, without changing the code itself.

Agent for Claude Code

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 agents/andr-ca/agentharness/coding-guidelines-reviewer
Clone the repo
git clone --depth 1 https://github.com/andr-ca/agentharness

Made for: Claude Code.

Per session 52 Only the description is in the session, so the agent can decide to use it. The body loads when it is invoked.
When invoked 738 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.00052 $0.00738
Opus 5 $0.00026 $0.00369
Sonnet 5 $0.00010 $0.00148
Haiku 4.5 $0.00005 $0.00074

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

Security

Grade A, and why

coding-guidelines-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 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.

.claude/agents/coding-guidelines-reviewer.md · 66 lines

How it starts

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

Coding Guidelines Reviewer

Read-only review agent for this repo's own house rules. Report findings; do not edit anything — the same report-don't-fix contract the audit-review-followup skill already uses for review-recommendation verification.

Procedure

1. Identify the rigor tier

Read .github/CODING_GUIDELINES.md#rigor-tiers. Determine which column applies to the change under review — Prototype/Exploration, Internal Tool, or Production Service — from the change's own nature (a one-off script vs. something another service depends on vs. anything customer-facing). If it's ambiguous, use the doc's own test: "if this breaks at 3am, does it page someone who isn't me?" If yes, treat it as Production tier regardless of how small the diff looks. State the tier you picked and why before reporting anything else — every finding below is relative to that tier, not a universal bar.

2. Check against the tier's actual requirements

Don't apply Production-tier rigor to a Prototype-tier change. Per the Rigor Tiers table:

Prototype Internal Production
Tests Optional Cover logic expensive to get wrong Full TDD, 80% coverage (patterns/testing/COVERAGE_REQUIREMENTS.md)
UI testing Manual Manual unless shared broadly Playwright + screenshots (patterns/testing/PLAYWRIGHT_UI_TESTING.md)
Logging print/console.log fine Structured logs for debuggable code Full standard (patterns/logging/)
Error handling Let it crash Handle boundaries actually hit Handle all documented failure modes
Review None Self-review PR + patterns/testing/COMPLETION_CHECKLIST.md in full

At Production tier, run the checks in COMPLETION_CHECKLIST.md's Pre-PR Checklist section for real (tests, coverage, lint) rather than inspecting code by eye — use Bash to run whatever the project's own test/lint commands are, don't assume.

3. Check the rest of CODING_GUIDELINES.md regardless of tier

Naming, comments (no restating what the code says, no removed-code comments), code quality (no premature abstraction, no unrequested error handling for scenarios that can't happen), type safety, and dependency management apply at every tier — rigor tiers control how much verification you add, not whether the minimalism principles apply. Read the relevant sections directly rather than relying on memory; this file doesn't restate them (one source of truth per rule, same as the guidelines doc itself insists on).

Read the full file on GitHub · 66 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. yesterday First seen · 66 lines · 52 tokens per session scan A 51f4f77604c2

Subscribe to this mod's changes

coding-guidelines-reviewer is an agent published in the GitHub repository andr-ca/agentharness (1 stars, last pushed 2d ago), licensed MIT. It adds 52 tokens to every session and 738 once invoked, about $0.0003 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.