review-pr

A checklist for reviewing a Nylas CLI pull request, which is a proposed set of code changes, against its architecture, command-line conventions, security, tests, and documentation.

In plain words
What is it for?
Use it to inspect a branch or pull request, verify hexagonal architecture, check CLI flags and help text, confirm tests pass, and find missing documentation.
Why use it?
It gives reviewers one consistent process for checking code structure, user-facing commands, secrets, API error handling, test coverage, and documentation updates.

Command 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 commands/nylas/cli/review-pr
Clone the repo
git clone --depth 1 https://github.com/nylas/cli

Made for: Claude Code.

Per session 21 Only the description is in the session, so the agent can decide to use it. The body loads when it is invoked.
When invoked 629 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.00629
Opus 5 $0.00010 $0.00315
Sonnet 5 $0.00004 $0.00126
Haiku 4.5 $0.00002 $0.00063

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

Security

Grade A, and why

review-pr 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/commands/review-pr.md · 98 lines

How it starts

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

Review Pull Request

Review code changes following nylas CLI standards and best practices.

Instructions

  1. First, get the diff to review:
git diff main...HEAD

Or for a specific PR:

gh pr diff <pr-number>
  1. Review checklist:

Architecture

  • Changes follow hexagonal architecture (domain → ports → adapters → CLI)
  • No direct dependencies on concrete implementations (use interfaces)
  • New code is in the correct layer/package

Code Quality

  • Functions are appropriately sized (<50 lines ideal)
  • Error messages are user-friendly with suggestions
  • No hardcoded credentials or secrets
  • Context is passed to all API calls

CLI Standards

  • Commands follow naming conventions (newXxxCmd)
  • Flags have descriptions and appropriate defaults
  • Help text includes examples
  • Output supports --format flag where appropriate

Testing

  • Unit tests added/updated for new functionality
  • Mock implementations updated if interface changed
  • Integration tests added for user-facing features
  • Tests pass: go test ./...

Documentation

  • docs/COMMANDS.md or docs/commands/tui.md updated if user-facing changes
  • Code comments for non-obvious logic
  • Examples in command help text

Security

  • No hardcoded API keys, tokens, or passwords
  • No secrets in logs or error messages
  • Input validation where user data is used
  • No command injection vulnerabilities
  1. Run verification:
# Build
go build ./...

# Lint
golangci-lint run

# Tests
go test ./... -short

# Security scan
make security

# Integration tests (if credentials available)
go test -tags=integration ./internal/cli/integration/...
  1. Provide feedback in this format:

Review Output

Summary

Brief overview of the changes.

Issues Found

Severity File:Line Issue Suggestion
🔴 Critical path/file.go:42 Description How to fix
🟡 Warning path/file.go:100 Description How to fix
🔵 Info path/file.go:15 Description How to fix

Read the full file on GitHub · 98 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 · 98 lines · 21 tokens per session scan A ed344ab8639c

Subscribe to this mod's changes

review-pr is a command published in the GitHub repository nylas/cli (68 stars, last pushed 13d ago), licensed MIT. It adds 21 tokens to every session and 629 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.