agent-review

A review checklist for plans and code changes in a toolkit repository, based on the rules in its AGENTS.md file.

In plain words
What is it for?
Use it to review implementation plans or diffs for requirements coverage, lockfiles, documentation, shell scripts, symlinks, and commit conventions.
Why use it?
It helps catch missing files, broken links, outdated lists, unsafe Git actions, and missing verification before changes are accepted.

Skill for Claude CodeCodex

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 skills/sflandergan/agentic-coding/agent-review
Any agent
npx skills add sflandergan/agentic-coding --skill agent-review
Clone the repo
git clone --depth 1 https://github.com/sflandergan/agentic-coding

Made for: Claude Code, Codex.

Per session 22 Skills are progressive disclosure: only the name and description are preloaded; the body loads when the skill is used.
When invoked 842 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.00022 $0.00842
Opus 5 $0.00011 $0.00421
Sonnet 5 $0.00004 $0.00168
Haiku 4.5 $0.00002 $0.00084

Measured 2d ago against content hash 5d5add2286fc, method: parsed. Prices are Anthropic first-party input rates as of 2026-08-30, from the pricing page.

Security

Grade A, and why

agent-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 2d 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.

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.

.agents/skills/agent-review/SKILL.md · 74 lines

How it starts

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

Agent Review — Self-Maintenance

Review plans and diffs against the conventions in AGENTS.md.

Review Checklist

For plans

  • If spec is present, check coverage: every requirement has a task
  • File mapping includes core/stack ripple check
  • Sync invariants addressed (README, lockfiles, dot-mapping)
  • No placeholders (TBD, TODO, "similar to Task N")
  • Verification commands included per task
  • Task boundaries produce one commit each

For diffs

  • Changes match the approved plan
  • SKILL.md frontmatter has name and description
  • Agent .md has description in frontmatter
  • Symlinks in .claude/skills/ resolve correctly
  • skills-lock.json reflects this repo's self-maintenance skills
  • core/skills-lock.json reflects target-project template skills
  • README agent/skill lists match actual files
  • Shell scripts pass shellcheck and bash -n
  • No force-push, no branch deletion, no worktree removal
  • Commits are concise, imperative, scoped to one logical change
  • No Co-Authored-By lines

Review Priorities

  1. Bugs, broken scripts, incorrect symlinks
  2. Missing sync: README vs files, lockfiles vs installed skills
  3. Convention violations from AGENTS.md
  4. Unnecessary scope expansion

PR Comment Review

Use the github-pr-comments skill and run:

bash .agents/skills/github-pr-comments/scripts/fetch-pr-comments.sh [<pr>]

Inline comments must be checked against the current diff before accepting or dismissing them. Outdated inline comments may still be valid; verify technical claims before dismissing.

Required Workflow

Use this standard self-maintenance review workflow unless the user explicitly requests a different scope:

  1. Read open PR comments first by using the github-pr-comments skill. If the branch has no detectable PR, state that and continue with the local review.
  2. Identify whether the review target is a spec/plan or a diff. If multiple candidate plans exist and the user did not state which one to use, ask before continuing the plan-conformance part of the review.
  3. Review the target yourself against AGENTS.md, repository conventions, sync invariants, and any approved spec or plan.
  4. Combine PR comments, user notes, external notes, and your own findings into one deduplicated list of actionable issues.
  5. Present suggested fixes as blocking issues and advisory suggestions first. Do not edit files yet.
  6. For approved spec/plan review fixes, update the reviewed spec or plan directly under plans/**.
  7. For approved diff review fixes that need planning or exceed trivial review-scoped changes, write a review-finding implementation plan next to the original plan, for example plans/<feature-dir>/review-findings.md. Do not adapt an unrelated implementation plan for new review findings, and do not create a new date-prefixed folder for review findings when an original plan exists.
  8. After plan updates or review-finding plan creation, self-review every tracked remark and finding. Map each item to the changed section, review-finding plan section, or intentional unresolved status.
  9. Draft exact GitHub replies for resolved PR comments and ask for explicit approval before posting. Approval to edit files or write a plan does not authorize posting GitHub comments.

Read the full file on GitHub · 74 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. 2d ago First seen · 74 lines · 22 tokens per session scan A 5d5add2286fc

Subscribe to this mod's changes

agent-review is a skill published in the GitHub repository sflandergan/agentic-coding (2 stars, last pushed 2mo ago), licensed MIT. It adds 22 tokens to every session and 842 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-31.

Related

Other skills, from other repositories

systematic-debugging

Use when encountering any bug, test failure, or unexpected behavior, before proposing fixes.

obra/superpowers · 21 tokens

brainstorming

You MUST use this before any creative work - creating features, building components, adding functionality, or modifying behavior. Explores user intent, requirements and design before implementation.

obra/superpowers · 37 tokens

auto-perf-optimize

Run agent-driven VS Code performance or memory investigations. Use when asked to launch Code OSS, automate a VS Code scenario, run the Chat memory smoke runner, capture renderer heap snapshots, take workflow screenshots, compare run summaries, or drive a repeatable scenario before heap-snapshot analysis.

microsoft/vscode · 62 tokens

chat-perf

Run chat perf benchmarks and memory leak checks against the local dev build or any published VS Code version. Use when investigating chat rendering regressions, validating perf-sensitive changes to chat UI, or checking for memory leaks in the chat response pipeline.

microsoft/vscode · 51 tokens

chat-pet-sprite-creation

Use when creating or changing VS Code chat pet sprite art, sprite sheets, state animations, eye treatments, Stable/Insiders variants, or pet transitions under src/vs/workbench/contrib/chat/browser/widget/media/chatPet.

microsoft/vscode · 53 tokens

cpu-profile-analysis

Analyze V8/Chrome CPU profiles (.cpuprofile) and DevTools trace files (Trace-.json). Use when: profiling performance, investigating slow functions, comparing code paths, finding bottlenecks, analyzing timeToRequest, understanding call trees from sampling profiler data, analyzing layout/paint/rendering, investigating…

microsoft/vscode · 71 tokens