Code Reviewer

What it does

When your agent uses it

• Tasks that involve Code review
• Tasks that involve Pull requests

Example prompts

• “Review my branch”
• “Check my changes before I open a PR”
• “Review this implementation for correctness”
• “/code-reviewer”

Workflow steps

8 steps, taken from the first numbered list in SKILL.md.

1. 1Determine scope: Run git diff main...HEAD to get all changes since branch divergence.
2. 2Apply coding standards: Check against frl-coding-standard and, for test code,
3. 3Check async correctness: Look for join()/get() in production code, blocking inside
4. 4Check exception structure: Static messages with structured context — see frl-coding-standard for the correct API per layer (addLogInfo()…
5. 5Check logging: KeyValueLogMessage.of() with static text — no string concatenation.
6. 6Verify test coverage: Are new code paths covered? Is the right test type used (yamsql
7. 7Check PR hygiene: Will the PR title make sense
8. 8DRY check: Is there repeated logic that should be extracted?

What it can do on your machine

Read from SKILL.md and the folder at commit f266052. It shows what the files ask for, not the result of running them.

• Tool permissions

  Pre-approves nothing: there is no allowed-tools line, so your agent's usual permission prompts apply.

  From allowed-tools in the SKILL.md frontmatter.

• Runs code

  Shell commands in SKILL.md call:

  • git

  From the folder's file list and the shell code blocks in SKILL.md.

• Network

  No URLs in SKILL.md. Its commands use git, which can reach the network depending on how they are called.

  From URLs in SKILL.md, links to its own repository left out.

• Credentials

  Names no API keys, tokens, secrets or passwords.

  From names ending in _API_KEY, _TOKEN, _SECRET, _KEY or _PASSWORD in SKILL.md.

Context cost

Code Reviewer loads about 585 tokens when it runs. Until then it costs about 59 tokens; SKILL.md has 221 words of instructions outside code blocks.

Always · name and description, kept in context so the agent knows when to use it

~59

When it runs · the whole SKILL.md, loaded when a task matches

~585

Estimates: characters ÷ 4, the usual rule of thumb; real counts depend on the model's tokenizer. Scripts and assets cost tokens only if the agent reads them.

Safety

Automated static check — not a guarantee. Review scripts before installing. It scans the text of SKILL.md for risky patterns (piping downloads into a shell, reading credential files, hidden Unicode, destructive commands); files beside SKILL.md are not scanned.

SKILL.md

The full file from FoundationDB/fdb-record-layer at commit f266052, republished under its Apache-2.0 licence (© FoundationDB). 221 words, ~585 tokens.

name

code-reviewer

description

Code review the current branch for adherence to project standards, correctness, and best practices. Some example usages: "Review my branch" "Check my changes before I open a PR" "Review this implementation for correctness"

### Code Review Summary
[Brief overall assessment]

### Critical Issues
[Violations of coding standards, async safety problems, or correctness bugs that must be fixed]

### Suggestions for Improvement
[Code quality, performance, maintainability — not blocking but worth addressing]

### Positive Observations
[What the code does well]

### Testing Recommendations
[Specific test suggestions if coverage is missing or could be improved]

© FoundationDB, Apache-2.0. Rendered from Markdown: HTML in the file is shown as text, images as links, and headings moved down two levels. Raw file

Files

Just SKILL.md in .claude/skills/code-reviewer of FoundationDB/fdb-record-layer.

Open the folder on GitHubat commit f266052