review-flow: Skill for Claude Code

.claude/skills/review-back/SKILL.md

review-back is a skill for Claude Code from DGouron/review-flow. It costs 76 tokens per session (4,891 once invoked), scanned A, original, MIT.

A structured review process for backend merge requests or pull requests, which are proposed code changes awaiting review. It examines architecture, TypeScript practices, design principles, tests, code quality, security, and performance, then produces a report and can post it on the change.

In plain words
What is it for?
Use it to review backend changes for design, maintainability, testing, security, and speed issues before they are merged.
Why use it?
It brings several review areas into one sequential check and explains why each reported problem matters and how to address it.

Skill for Claude Code

Written for Claude Code: installed under .claude/. Also seen: mentions CLAUDE.md; mentions Claude Code.

This is DGouron/review-flow's own configuration. It tells Claude Code how to work on review-flow itself, so it is not a mod to install elsewhere. Copy it as a starting point and replace the rules that are about this project. Everything review-flow configures →

Reuse

Borrowing it

Nothing to install: this file belongs to DGouron/review-flow. Take a copy, put it at the same path in your own repository, and replace the rules that are about this project with yours.

Copy the file
curl -O https://raw.githubusercontent.com/DGouron/review-flow/master/.claude/skills/review-back/SKILL.md
Clone the repo
git clone --depth 1 https://github.com/DGouron/review-flow

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-back

README.md
[![agentmods](https://agentmods.dev/badge/skills/dgouron/review-flow/review-back/github.svg)](https://agentmods.dev/skills/dgouron/review-flow/review-back)
Your own site
<a href="https://agentmods.dev/skills/dgouron/review-flow/review-back"><img src="https://agentmods.dev/badge/skills/dgouron/review-flow/review-back/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-back

Your own site · 80×15
<a href="https://agentmods.dev/skills/dgouron/review-flow/review-back"><img src="https://agentmods.dev/badge/skills/dgouron/review-flow/review-back.svg" alt="Reviewed on agentmods" width="80" height="20"></a>
Per session 76 Skills are progressive disclosure: only the name and description are preloaded; the body loads when the skill is used.
When invoked 4,891 The whole file, excluding the scripts and references it only reads on demand.
Security scan A 1 finding. A grade says what 26 rules found in the file — not that it is safe. Third-party audits
  • NVIDIA SkillSpector warn 7 Sept 2026
SkillSpector: 10 findings, up to medium

These are SkillSpector’s own severities. On a checked sample its high-severity flags on skills were ~96% false positives — a documented command, a public API, a “never do X” rule — so we show them as a caution to read, not a verdict. Why →

  • medium Agent Snooping · line 197
    Skill enumerates or reads other installed skills. Access to other skills' SKILL.md files or the skills directory reveals prompt instructions, capabilities, and secrets that should be invisible to peer skills.
    Fix: Remove all code or instructions that list or read other skills' files or directories. Skills should operate independently; cross-skill access is a privilege escalation.
  • medium Agent Snooping · line 214
    Skill enumerates or reads other installed skills. Access to other skills' SKILL.md files or the skills directory reveals prompt instructions, capabilities, and secrets that should be invisible to peer skills.
    Fix: Remove all code or instructions that list or read other skills' files or directories. Skills should operate independently; cross-skill access is a privilege escalation.
  • medium Agent Snooping · line 198
    Skill enumerates or reads other installed skills. Access to other skills' SKILL.md files or the skills directory reveals prompt instructions, capabilities, and secrets that should be invisible to peer skills.
    Fix: Remove all code or instructions that list or read other skills' files or directories. Skills should operate independently; cross-skill access is a privilege escalation.
  • medium Agent Snooping · line 233
    Skill enumerates or reads other installed skills. Access to other skills' SKILL.md files or the skills directory reveals prompt instructions, capabilities, and secrets that should be invisible to peer skills.
    Fix: Remove all code or instructions that list or read other skills' files or directories. Skills should operate independently; cross-skill access is a privilege escalation.
  • medium Agent Snooping · line 200
    Skill enumerates or reads other installed skills. Access to other skills' SKILL.md files or the skills directory reveals prompt instructions, capabilities, and secrets that should be invisible to peer skills.
    Fix: Remove all code or instructions that list or read other skills' files or directories. Skills should operate independently; cross-skill access is a privilege escalation.
  • medium Agent Snooping · line 273
    Skill enumerates or reads other installed skills. Access to other skills' SKILL.md files or the skills directory reveals prompt instructions, capabilities, and secrets that should be invisible to peer skills.
    Fix: Remove all code or instructions that list or read other skills' files or directories. Skills should operate independently; cross-skill access is a privilege escalation.
  • medium Agent Snooping · line 201
    Skill enumerates or reads other installed skills. Access to other skills' SKILL.md files or the skills directory reveals prompt instructions, capabilities, and secrets that should be invisible to peer skills.
    Fix: Remove all code or instructions that list or read other skills' files or directories. Skills should operate independently; cross-skill access is a privilege escalation.
  • medium Agent Snooping · line 293
    Skill enumerates or reads other installed skills. Access to other skills' SKILL.md files or the skills directory reveals prompt instructions, capabilities, and secrets that should be invisible to peer skills.
    Fix: Remove all code or instructions that list or read other skills' files or directories. Skills should operate independently; cross-skill access is a privilege escalation.
  • medium Agent Snooping · line 203
    Skill enumerates or reads other installed skills. Access to other skills' SKILL.md files or the skills directory reveals prompt instructions, capabilities, and secrets that should be invisible to peer skills.
    Fix: Remove all code or instructions that list or read other skills' files or directories. Skills should operate independently; cross-skill access is a privilege escalation.
  • medium Agent Snooping · line 335
    Skill enumerates or reads other installed skills. Access to other skills' SKILL.md files or the skills directory reveals prompt instructions, capabilities, and secrets that should be invisible to peer skills.
    Fix: Remove all code or instructions that list or read other skills' files or directories. Skills should operate independently; cross-skill access is a privilege escalation.
How audits are shown
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.00076 $0.04891
Opus 5.5 $0.00030 $0.01956
Sonnet 5.5 $0.00015 $0.00978
Haiku 4.5 $0.00008 $0.00489

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

Security

Grade A, and why

review-back scanned grade A with 1 finding 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 2mo 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.

Runs shell commandslowCapability

Expected in a hook, worth knowing in a rule or an instructions file.

4. **Synchronous I/O in request paths**: `readFileSync`/`execSync` inside HTTP handlers or hot loops is BLOCKING. Switch to async APIs.
.claude/skills/review-back/SKILL.md · 590 lines

How it starts

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

Code Review — Backend

Persona

Read .claude/roles/code-reviewer.md — adopt this profile and follow all its rules.

Context

You are: A demanding reviewer, expert in Clean Architecture, DDD, TypeScript, security, and performance for backend systems. You point out problems bluntly.

Your approach:

  • Direct and factual: no flattery, no "excellent work", no unearned compliments
  • Each point raised = 1 pedagogical lesson with a source
  • You explain the "why" before the "how"
  • You do not spare feelings — being too nice is counterproductive
  • KISS & YAGNI: You NEVER recommend unjustified refactoring

IMPORTANT — Direct tone, not condescending:

  • "Excellent work!", "Well done!", "This is great!" -> State the facts: "The Gateway pattern is correctly applied"
  • Point out problems: "Secret leak: API key hard-coded in source"
  • Explain why it is a problem and how to fix it

KISS/YAGNI guardrails:

"Simplicity—the art of maximizing the amount of work not done—is essential." — Agile Manifesto

"You Ain't Gonna Need It. Always implement things when you actually need them, never when you just foresee that you need them." — Ron Jeffries

Strict rules:

  • Do NOT recommend abstractions for 1-2 usages (premature DRY)
  • Do NOT recommend creating interfaces "just in case"
  • Do NOT recommend splitting into files if < 100 lines
  • Do NOT recommend Value Objects without clear business invariants
  • Recommend only if duplication > 70% across 2+ files
  • Recommend only if the violation impacts immediate maintainability
  • Prioritize quick-wins (imports, cleanup) before refactorings

BLOCKING rule — Missing tests:

"Never write production code without a failing test first." — CLAUDE.md, Absolute Rule

Any business logic added without a unit test is a BLOCKING correction.

This includes:

  • Pure functions (filters, transformations, validations)
  • Handlers with business logic
  • Use cases
  • Services
  • Presenters
  • Gateways

Justification: The project follows the TDD Detroit School. An MR without tests for new code cannot be merged.

Read the full file on GitHub · 590 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. 2mo ago First seen · 590 lines · 76 tokens per session scan A 1b149494f54d

Subscribe to this mod's changes

review-back is a skill published in the GitHub repository DGouron/review-flow (44 stars, last pushed today), licensed MIT. It adds 76 tokens to every session and 4,891 once invoked, about $0.0003 per session on Opus 5.5. A static security scan graded it A with 1 finding (runs shell commands). No closer match exists in the catalogue, so it is treated as the original; first seen 2026-08-30.

Related

Other skills, from other repositories

review-subdomain

Reviews a NestJS bounded context for Hexagonal Architecture + DDD + CQRS compliance in three steps — static rulebook via nestjs-hexagonal-check, semantic rulebook via Jev (one batch per file, skipped without a key) and a residual review by the architecture-reviewer agent for what the rulebook cannot decide. Produces a…

Softtor/nestjs-hexagonal · 93 tokens

onboard-project

Onboards a NestJS project to the nestjs-hexagonal rulebook, CLI and hooks — creates .claude/rulebook.yaml extending the plugin rulebooks with sha256 stamps, sets the optional TypeSafe key, pins the CLI as a dev dependency, runs the first baseline check and explains what the hooks will do from then on. Use when a…

Softtor/nestjs-hexagonal · 113 tokens

flow-next-impl-review

Carmack-level implementation review of changes via the configured backend. Use when asked to review code or a diff in a flow-next repo.

gmickel/flow-next · 34 tokens

flow-next-resolve-pr

Resolve PR review feedback. Fetches unresolved threads, triages, fixes, replies and resolves via GraphQL. Use when asked to address review comments.

gmickel/flow-next · 36 tokens

new-skill

Scaffold a new brooks-lint analysis skill so it passes npm run validate and npm run evals on the first try — generates skills/{name}/SKILL.md (with the mandatory "Do NOT trigger for:" clause and a Process section citing guide step ranges) plus skills/{name}/{name}-guide.md (sequentially numbered steps), then appends…

hyhmrright/brooks-lint · 145 tokens

api-contract-review

Use when auditing CloudBase cloud API wrappers, MCP tools, generated action metadata, or related docs for outdated or incorrect action names, parameters, casing, request shapes, or missing contract tests, especially during periodic quality review or before preparing corrective PRs.

TencentCloudBase/CloudBase-AI-Toolkit · 54 tokens