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.
npx agentmods add skills/jamestorrevillas/dev-skills/code-reviewnpx skills add jamestorrevillas/dev-skills --skill code-reviewgit clone --depth 1 https://github.com/jamestorrevillas/dev-skillsWrote 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.
[](https://agentmods.dev/skills/jamestorrevillas/dev-skills/code-review)<a href="https://agentmods.dev/skills/jamestorrevillas/dev-skills/code-review"><img src="https://agentmods.dev/badge/skills/jamestorrevillas/dev-skills/code-review.svg" alt="Measured on agentmods" height="20"></a>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.
| Model | Per session | Once invoked |
|---|---|---|
| Fable 5 | $0.00062 | $0.01148 |
| Opus 5 | $0.00031 | $0.00574 |
| Sonnet 5 | $0.00012 | $0.00230 |
| Haiku 4.5 | $0.00006 | $0.00115 |
Grade A, and why
code-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 4d 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.
How it starts
The opening of the file, as written. The whole thing — 144 lines — stays where its author put it; the contents beside it link to each section on GitHub.
Code Review
Review Philosophy
Review the code, not the person. The goal is to ship better software, not to win arguments. Every comment should make the codebase better or help the author learn — preferably both.
Feedback Categories
Always tag review comments to set expectations:
| Tag | Meaning | Must Fix? |
|---|---|---|
BLOCKER |
Breaks functionality, security risk, violates standards | Yes |
WARNING |
Technical debt, potential issue, should be addressed | Recommended |
SUGGESTION |
Optional improvement, alternative approach | No |
QUESTION |
Asking for clarification or understanding | No |
NITPICK |
Minor style or preference | Author decides |
PRAISE |
Acknowledge good work | N/A |
Code Quality Checklist
Correctness
- Does the code do what the ticket/story requires?
- Are edge cases handled? (null, empty, boundary values)
- Is error handling present and meaningful?
- Are async operations awaited properly?
- Are there any obvious race conditions?
Readability
- Are variable and function names descriptive?
- Is the code self-documenting (minimal need for comments)?
- Are complex sections commented with the "why"?
- Is there dead code or commented-out code that should be removed?
- Is function length reasonable (< 40 lines as a guideline)?
Architecture
- Does this follow the existing patterns in the codebase?
- Is there unnecessary coupling between modules?
- Is the change in the right layer (controller vs service vs repository)?
- Would this be hard to test? (if yes, it needs refactoring)
- Is this solving the right problem at the right level?
Performance
- Are there N+1 query problems (loops that trigger individual DB queries)?
- Are expensive operations cached where appropriate?
- Is pagination implemented for list endpoints?
- Are there unnecessary re-renders (frontend)?
Tests
- Are tests included for new behavior?
- Do tests cover the happy path AND error paths?
- Are tests testing behavior, not implementation details?
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.
- 4d ago First seen · 144 lines · 0 tokens per session scan A f177c39fa1f8
code-review is a skill published in the GitHub repository jamestorrevillas/dev-skills (3 stars, last pushed 5mo ago), licensed MIT. It adds 62 tokens to every session and 1,148 once invoked, about $0.0003 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.
Other skills, from other repositories
jira-cli
Interact with Jira from the command line to create, list, view, edit, and transition issues, manage sprints and epics, and perform common Jira workflows. Use when the user asks about Jira tasks, tickets, issues, sprints, or needs to manage project work items.
azure-prices
Look up and compare Azure service pricing using the Azure Retail Prices API. Use this skill whenever the user asks about Azure costs, pricing, rates, or wants to compare prices across regions or services — even if they don't say "pricing" explicitly. Trigger for questions like "how much does a D2 v2 VM cost?"…
calculator
Performs arbitrary-precision arithmetic calculations including addition, subtraction, multiplication, division, and exponents. Use when the user asks to calculate, compute, or evaluate math expressions, or when precise decimal arithmetic is needed to avoid floating-point errors.
az-cli
Use the Azure CLI (az) to manage Azure resources from the command line. Trigger this skill whenever the user asks to create, configure, manage, deploy, or interact with any Azure resource — even if they don't explicitly mention "az cli". Also trigger when the user asks about Azure CLI commands, syntax, or wants to…
la-le-me
以中文、English 或中英双语记录、解释和复盘成人日常排便与肠道健康信息，包括 Bristol 便形文字分类、频次和时段趋势、食物线索、可持续饮食调整、季节菜谱、每日排便提醒，以及危险信号分流和经同意的附近就医机构查找。Use for Chinese, English, or bilingual requests about poop, stool, bowel movements, constipation or diarrhea tendencies, foods associated with a bowel change, gut-friendly meals, recurring bowel-health…
code-review
Use when user wants code reviewed for correctness, style, bugs, and maintainability. Triggers on: review code, code review, check PR, 審查程式碼, 檢查程式碼. Produces severity-classified findings with a verdict. Do NOT use for security-focused audit (prefer security-audit) or SQL-focused review (prefer sql-review).