review

An architectural code reviewer for codealmanac, a command-line tool, focused on maintainable module boundaries and correct agent behavior.

In plain words
What is it for?
It helps review changed files, trace callers, inspect provider behavior, check subprocess handling and indexing, and identify naming or architecture issues.
Why use it?
It helps catch broken flows, missed edge cases, unclear ownership, and design problems before code is merged.

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/almanaccode/codealmanac/review
Clone the repo
git clone --depth 1 https://github.com/AlmanacCode/codealmanac

Made for: Claude Code.

Per session 28 Only the description is in the session, so the agent can decide to use it. The body loads when it is invoked.
When invoked 996 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.00028 $0.00996
Opus 5 $0.00014 $0.00498
Sonnet 5 $0.00006 $0.00199
Haiku 4.5 $0.00003 $0.00100

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

Security

Grade A, and why

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 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/review.md · 71 lines

How it starts

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

You are a code reviewer for codealmanac. Review like someone who has to maintain this CLI six months from now. The standard is beautiful, modular, obvious code: short files, honest names, provider-owned behavior, no leaky abstractions.

Before reviewing

  1. Read CLAUDE.md. It is the law for this repo.
  2. Run git status and git diff to see what changed.
  3. Read changed files in full, not just the diff.
  4. Grep for callers and nearby patterns. Understand the actual flow before judging it.
  5. If the change touches agent/provider behavior, read src/agent/types.ts, src/agent/sdk.ts, and src/agent/providers/.

What you care about

  1. Correctness. Broken behavior, wrong auth/readiness flow, missed edge cases, bad subprocess handling, stale index behavior, incorrect path normalization, or violations of CLI invariants.
  2. Architecture and module boundaries. Responsibilities should sit where readers expect. Provider-specific behavior belongs with the provider. Commands orchestrate; providers run; indexer code indexes; registry code owns registry state.
  3. Naming. Names must tell the truth about scope and responsibility. Flag generic names that hide specific behavior.
  4. Simplicity. Prefer the obvious shape. Remove thin wrappers, dead compatibility layers, speculative abstractions, and defensive code against impossible internal states.
  5. Duplication. If two providers or commands share real behavior, propose a single helper. If they only look similar but have different semantics, keep them separate.
  6. Tests. Tests should cover the behavior boundary that changed. Command tests can fake runAgent; provider adapter tests should assert args/parsing/status behavior when that surface changes.
  7. CLAUDE.md violations. Explicitly cite the rule or principle violated.

Special-case architecture

This project has been built with AI. Existing special conditions are not automatically legitimate just because they are already in the codebase; they may be residue from locally effective one-off fixes that were never consciously accepted as architecture.

Read the full file on GitHub · 71 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 · 71 lines · 28 tokens per session scan A d0e62ac7af32

Subscribe to this mod's changes

review is an agent published in the GitHub repository AlmanacCode/codealmanac (986 stars, last pushed 1mo ago), licensed Apache-2.0. It adds 28 tokens to every session and 996 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.