A code-review workflow for Salesforce projects, where Salesforce is a cloud platform for business applications. It checks Apex, LWC, Flow, triggers, and database queries for common quality and security problems.

Use it to review Apex, LWC, Flow, triggers, SOQL queries, integrations, or architecture and organize findings by severity.

It helps find issues such as inefficient database use, unsafe access to data, poor handling of many records, and violations of Salesforce limits before they cause trouble.

sf-review — Skill for Claude Code by divingsbysangam

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.

Any agent

npx skills add divingsbysangam/salesforce-compound-engineering-plugin --skill sf-review

Clone the repo

git clone --depth 1 https://github.com/divingsbysangam/salesforce-compound-engineering-plugin

Made for: Claude Code.

Or install sf-compound-engineering, the plugin that ships this one along with the rest of its 68 skills, 1 hook, 2 MCP servers.

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.

README.md

[![agentmods](https://agentmods.dev/badge/skills/divingsbysangam/salesforce-compound-engineering-plugin/sf-review/github.svg)](https://agentmods.dev/skills/divingsbysangam/salesforce-compound-engineering-plugin/sf-review)

Your own site

<a href="https://agentmods.dev/skills/divingsbysangam/salesforce-compound-engineering-plugin/sf-review"><img src="https://agentmods.dev/badge/skills/divingsbysangam/salesforce-compound-engineering-plugin/sf-review/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.

Your own site · 80×15

<a href="https://agentmods.dev/skills/divingsbysangam/salesforce-compound-engineering-plugin/sf-review"><img src="https://agentmods.dev/badge/skills/divingsbysangam/salesforce-compound-engineering-plugin/sf-review.svg" alt="Reviewed on agentmods" width="80" height="20"></a>

Per session 98 Skills are progressive disclosure: only the name and description are preloaded; the body loads when the skill is used.

When invoked 1,259 The whole file, excluding the scripts and references it only reads on demand.

Security scan A 0 findings. 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: 7 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 Prompt Injection · line 47
Large whitespace padding was detected (a block of blank lines or a long run of spaces). This can push injected instructions below or to the right of the visible area so a human reviewer never sees them while the agent still reads them. Manual review of the hidden content is recommended.

Fix: Remove the large whitespace padding (blank-line blocks or long space runs) and review any content hidden below or to the right of it. Keep skill files compact and reviewable so no instructions can be
medium Prompt Injection · line 47
Large whitespace padding was detected (a block of blank lines or a long run of spaces). This can push injected instructions below or to the right of the visible area so a human reviewer never sees them while the agent still reads them. Manual review of the hidden content is recommended.

Fix: Remove the large whitespace padding (blank-line blocks or long space runs) and review any content hidden below or to the right of it. Keep skill files compact and reviewable so no instructions can be
medium Prompt Injection · line 51
Large whitespace padding was detected (a block of blank lines or a long run of spaces). This can push injected instructions below or to the right of the visible area so a human reviewer never sees them while the agent still reads them. Manual review of the hidden content is recommended.

Fix: Remove the large whitespace padding (blank-line blocks or long space runs) and review any content hidden below or to the right of it. Keep skill files compact and reviewable so no instructions can be
medium Prompt Injection · line 51
Large whitespace padding was detected (a block of blank lines or a long run of spaces). This can push injected instructions below or to the right of the visible area so a human reviewer never sees them while the agent still reads them. Manual review of the hidden content is recommended.

Fix: Remove the large whitespace padding (blank-line blocks or long space runs) and review any content hidden below or to the right of it. Keep skill files compact and reviewable so no instructions can be
medium Prompt Injection · line 52
Large whitespace padding was detected (a block of blank lines or a long run of spaces). This can push injected instructions below or to the right of the visible area so a human reviewer never sees them while the agent still reads them. Manual review of the hidden content is recommended.

Fix: Remove the large whitespace padding (blank-line blocks or long space runs) and review any content hidden below or to the right of it. Keep skill files compact and reviewable so no instructions can be
medium Prompt Injection · line 52
Large whitespace padding was detected (a block of blank lines or a long run of spaces). This can push injected instructions below or to the right of the visible area so a human reviewer never sees them while the agent still reads them. Manual review of the hidden content is recommended.

Fix: Remove the large whitespace padding (blank-line blocks or long space runs) and review any content hidden below or to the right of it. Keep skill files compact and reviewable so no instructions can be
medium Prompt Injection · line 53
Large whitespace padding was detected (a block of blank lines or a long run of spaces). This can push injected instructions below or to the right of the visible area so a human reviewer never sees them while the agent still reads them. Manual review of the hidden content is recommended.

Fix: Remove the large whitespace padding (blank-line blocks or long space runs) and review any content hidden below or to the right of it. Keep skill files compact and reviewable so no instructions can be

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.

Model	Per session	Once invoked
Fable 5.1	$0.00098	$0.01259
Opus 5	$0.00049	$0.00629
Sonnet 5	$0.00020	$0.00252
Haiku 4.5	$0.00010	$0.00126

Measured 9d ago against content hash b252ec6f637e, method: parsed. Prices are Anthropic first-party input rates as of 2026-09-09, from the pricing page.

Security

Grade A, and why

sf-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 9d 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.

skills/sf-review/SKILL.md · 73 lines

How it starts

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

/sf-review

Principles enforced: 1 (preserve the quality ceiling), 3 (jagged intelligence), 5 (taste and oversight). See PRINCIPLES.md.

Required reads

Procedure lives in sibling files, not only in this orchestrator:

Review Depth Levels — read references/review-depth-levels.md before acting on this section.
Step 1: Identify and Classify Files — read references/step-1-identify-and-classify-files.md before acting on this section.
Step 2: Dispatch Review Personas in Parallel — read references/step-2-dispatch-review-personas-in-parallel.md before acting on this section.
Step 3: Parallel Research (comprehensive depth only) — read references/step-3-parallel-research-comprehensive-depth-only.md before acting on this section.
Step 4: Consolidate Findings — read references/step-4-consolidate-findings.md before acting on this section.
Output Format — read references/output-format.md before acting on this section.

Copy-paste-to-agent

Review Salesforce code by dispatching parallel review agents based on file type
(Apex / LWC / Flow / Integration / Architecture). Output findings categorized as
Critical / High / Medium / Low. Critical and High findings are non-negotiable
abort triggers — see "Non-Negotiable Gates" below. If the target isn't specified,
review the current git diff.

You are reviewing Salesforce code using parallel agent dispatch for speed and thoroughness.

Goal

Review the code at: $ARGUMENTS.target

If no target specified, review uncommitted changes (git diff).

Non-Negotiable Gates (Principle 1)

The following findings are abort triggers, not warnings. They block the review from passing regardless of how minor the surrounding diff is. They exist because vibe coding does not exempt the diff from production-grade Salesforce constraints.

Gate	What it catches	Owning agent
Security regression	CRUD/FLS bypass, SOQL injection, sharing-model violation, hardcoded credential, unsafe `without sharing`	`apex-security-sentinel`, `lwc-security-reviewer`, `integration-security-sentinel`
Governor regression	SOQL/DML inside loops, missing bulkification, non-selective query on >100k-row object	`apex-governor-guardian`, `apex-bulkification-reviewer`, `flow-governor-monitor`
Test coverage regression	Production class without test class, test class with no `assertEquals`/`assertTrue`, bulk path untested at 200+ records	`apex-test-coverage-analyst`
Trigger context regression	Recursion guard missing, mixed-DML violation, handler bypassing the project's trigger framework	`apex-trigger-architect`
Sharing regression	`without sharing` introduced without justification, sharing-recalculation skipped on owner change	`sharing-security-analyst` (when present)

Read the full file on GitHub · 73 lines

Files

What ships with it

48 files beside SKILL.md in the same directory: the scripts, references and assets a skill reads on demand. Not counted in the per-session cost; read them before you install if any of them is executable.

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.

9d ago First seen · 73 lines · 98 tokens per session scan A b252ec6f637e

Subscribe to this mod's changes

sf-review is a skill published in the GitHub repository divingsbysangam/salesforce-compound-engineering-plugin (10 stars, last pushed 5d ago), licensed MIT. It adds 98 tokens to every session and 1,259 once invoked, about $0.0005 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.

Code review Security

autoreview

Pre-commit/ship code review: Codex default; optional Claude or Pi.

openclaw/clawhub · 20 tokens

rework-rate

Measure and interpret PR rework rate — the emerging 5th DORA metric.

bradygaster/squad · 20 tokens

omh-code-review

This is a Hermes-native code-review workflow skill.

rlaope/oh-my-hermes · 46 tokens

revdiff-plan

Review the last Codex assistant message (plan, analysis, or proposal) with inline annotations in a TUI overlay. Extracts the most recent response from Codex rollout files and opens it in revdiff for review and annotation. Activates on "revdiff-plan", "review plan with revdiff", "annotate plan", "review last response"…

umputun/revdiff · 84 tokens

code-reviewer

Code review specialist focused on patterns, bugs, security, and performance.

RightNow-AI/openfang · 17 tokens

agent-teams-simplify-and-harden

Implementation + audit loop using parallel agent teams with structured simplify, harden, and document passes. Spawns implementation agents to do the work, then audit agents to find complexity, security gaps, and spec deviations, then loops until code compiles cleanly, all tests pass, and auditors find zero issues or…

LeoYeAI/openclaw-master-skills · 115 tokens