Code Review Skill
awesome-skills/code-review-skill
Provides comprehensive code review guidance for React 19, Vue 3, Angular 17+, Svelte 5, Rust, TypeScript, Java, Java 8, PHP, Ruby, Rails, Python, Django, FastAPI, Go, C/.NET, Kotlin, Swift, Dart…
Compares code paths that should behave the same, such as sync and async cache methods, and requires a concrete scenario where the two observably disagree.
$ npx skills add ben-manes/caffeine --skill audit-sibling-divergence -a claude-codeProject install by default; add -g for ~/.claude/skills/.
$ gh skill install ben-manes/caffeine audit-sibling-divergence --agent claude-codeProject scope by default; add --scope user for a personal install. Needs GitHub CLI 2.90.0 or later (public preview).
$ git clone --depth 1 https://github.com/ben-manes/caffeine.git skills-src && mkdir -p .claude/skills && cp -r skills-src/.claude/skills/audit-sibling-divergence .claude/skills/audit-sibling-divergence && rm -rf skills-srcUse ~/.claude/skills/ instead of .claude/skills for a personal install. The folder must contain SKILL.md.
Claude Code skills documentation · loads skills from .claude/skills/
Install the "audit-sibling-divergence" agent skill from https://github.com/ben-manes/caffeine/tree/master/.claude/skills/audit-sibling-divergence into .claude/skills/audit-sibling-divergence/ in this project. Copy the whole folder (SKILL.md and every file beside it), keep the folder name "audit-sibling-divergence", then confirm the skill loads.Claude Code copies the folder itself, the same result as the manual copy. Check what it changed before you commit it.
$skill-installer install https://github.com/ben-manes/caffeine/tree/master/.claude/skills/audit-sibling-divergenceType this inside Codex. $skill-installer <name> installs a curated skill from openai/skills. The installer writes to $CODEX_HOME/skills (default ~/.codex/skills). Restart Codex if the skill does not show up.
$ npx skills add ben-manes/caffeine --skill audit-sibling-divergence -a codexProject install goes to .agents/skills/; add -g for ~/.codex/skills/.
$ gh skill install ben-manes/caffeine audit-sibling-divergence --agent codexProject scope by default (.agents/skills/); add --scope user for a personal install.
$ git clone --depth 1 https://github.com/ben-manes/caffeine.git skills-src && mkdir -p .agents/skills && cp -r skills-src/.claude/skills/audit-sibling-divergence .agents/skills/audit-sibling-divergence && rm -rf skills-srcUse ~/.agents/skills/ instead of .agents/skills for a personal install.
Codex skills documentation · loads skills from .agents/skills/
Install the "audit-sibling-divergence" agent skill from https://github.com/ben-manes/caffeine/tree/master/.claude/skills/audit-sibling-divergence into .agents/skills/audit-sibling-divergence/ in this project. Copy the whole folder (SKILL.md and every file beside it), keep the folder name "audit-sibling-divergence", then confirm the skill loads.Codex copies the folder itself, the same result as the manual copy. Check what it changed before you commit it.
$ npx skills add ben-manes/caffeine --skill audit-sibling-divergence -a cursorProject install goes to .agents/skills/; add -g for ~/.cursor/skills/.
$ gh skill install ben-manes/caffeine audit-sibling-divergence --agent cursorProject scope by default (.agents/skills/); add --scope user for a personal install.
$ git clone --depth 1 https://github.com/ben-manes/caffeine.git skills-src && mkdir -p .cursor/skills && cp -r skills-src/.claude/skills/audit-sibling-divergence .cursor/skills/audit-sibling-divergence && rm -rf skills-srcUse ~/.cursor/skills/ instead of .cursor/skills for a personal install.
Cursor skills documentation · loads skills from .cursor/skills/, .agents/skills/, .claude/skills/, .codex/skills/
Install the "audit-sibling-divergence" agent skill from https://github.com/ben-manes/caffeine/tree/master/.claude/skills/audit-sibling-divergence into .cursor/skills/audit-sibling-divergence/ in this project. Copy the whole folder (SKILL.md and every file beside it), keep the folder name "audit-sibling-divergence", then confirm the skill loads.Cursor copies the folder itself, the same result as the manual copy. Check what it changed before you commit it.
$ gemini skills install https://github.com/ben-manes/caffeine.git --path .claude/skills/audit-sibling-divergence--scope user (default) or --scope workspace; --path is the subfolder of the repo that holds the skill; --consent skips the security confirmation prompt.
$ npx skills add ben-manes/caffeine --skill audit-sibling-divergence -a gemini-cliProject install goes to .agents/skills/; add -g for ~/.gemini/skills/.
$ gh skill install ben-manes/caffeine audit-sibling-divergence --agent gemini-cliProject scope by default (.agents/skills/); add --scope user for a personal install.
$ git clone --depth 1 https://github.com/ben-manes/caffeine.git skills-src && mkdir -p .gemini/skills && cp -r skills-src/.claude/skills/audit-sibling-divergence .gemini/skills/audit-sibling-divergence && rm -rf skills-srcUse ~/.gemini/skills/ instead of .gemini/skills for a personal install, then run /skills reload.
Gemini CLI skills documentation · loads skills from .gemini/skills/, .agents/skills/
Install the "audit-sibling-divergence" agent skill from https://github.com/ben-manes/caffeine/tree/master/.claude/skills/audit-sibling-divergence into .gemini/skills/audit-sibling-divergence/ in this project. Copy the whole folder (SKILL.md and every file beside it), keep the folder name "audit-sibling-divergence", then confirm the skill loads.Gemini CLI copies the folder itself, the same result as the manual copy. Check what it changed before you commit it.
$ gh skill install ben-manes/caffeine audit-sibling-divergenceInstalls for Copilot at project scope by default; add --scope user for a personal install. Preview a skill first with gh skill preview. Needs GitHub CLI 2.90.0 or later (public preview).
$ npx skills add ben-manes/caffeine --skill audit-sibling-divergence -a github-copilotProject install goes to .agents/skills/; add -g for ~/.copilot/skills/.
$ git clone --depth 1 https://github.com/ben-manes/caffeine.git skills-src && mkdir -p .github/skills && cp -r skills-src/.claude/skills/audit-sibling-divergence .github/skills/audit-sibling-divergence && rm -rf skills-srcUse ~/.copilot/skills/ instead of .github/skills for a personal install. Commit .github/skills so cloud agent and code review can use it.
GitHub Copilot skills documentation · loads skills from .github/skills/, .claude/skills/, .agents/skills/
Install the "audit-sibling-divergence" agent skill from https://github.com/ben-manes/caffeine/tree/master/.claude/skills/audit-sibling-divergence into .github/skills/audit-sibling-divergence/ in this project. Copy the whole folder (SKILL.md and every file beside it), keep the folder name "audit-sibling-divergence", then confirm the skill loads.GitHub Copilot copies the folder itself, the same result as the manual copy. Check what it changed before you commit it.
$ npx skills add ben-manes/caffeine --skill audit-sibling-divergence -a opencodeOpenCode documents no install command of its own. Project install goes to .agents/skills/; add -g for ~/.config/opencode/skills/.
$ gh skill install ben-manes/caffeine audit-sibling-divergence --agent opencodeProject scope by default (.agents/skills/); add --scope user for a personal install.
$ git clone --depth 1 https://github.com/ben-manes/caffeine.git skills-src && mkdir -p .opencode/skills && cp -r skills-src/.claude/skills/audit-sibling-divergence .opencode/skills/audit-sibling-divergence && rm -rf skills-srcUse ~/.config/opencode/skills/ instead of .opencode/skills for a personal install.
OpenCode skills documentation · loads skills from .opencode/skills/, .claude/skills/, .agents/skills/
Install the "audit-sibling-divergence" agent skill from https://github.com/ben-manes/caffeine/tree/master/.claude/skills/audit-sibling-divergence into .opencode/skills/audit-sibling-divergence/ in this project. Copy the whole folder (SKILL.md and every file beside it), keep the folder name "audit-sibling-divergence", then confirm the skill loads.OpenCode copies the folder itself, the same result as the manual copy. Check what it changed before you commit it.
audit-sibling-divergenceCompares code paths that should behave the same, such as sync and async cache methods, and requires a concrete scenario where the two observably disagree.
Instead of asking whether one code path is correct, this audit asks whether two paths that should give the same observable result actually agree. It was written for the Caffeine Java caching library, where mismatches between paired paths have caused real bugs, and it starts by inventorying matched pairs: sync and async caches, bounded and unbounded storage, generated node variants, view consistency, bulk versus single operations, fast versus slow read paths and adapter conformance.
One auditor is spawned per sibling pair, and a finding counts only when the auditor produces a witness scenario in which the paths diverge. It is meant for after major changes to the core cache classes or the jcache and guava adapters, before a major release and once per quarter as a baseline. The skill itself calls the run heavyweight, taking roughly four to six hours of agent time and many tokens, and points to a separate review command for routine pre-commit checks.
5 steps, taken from the step headings in SKILL.md.
Read from SKILL.md and the folder at commit e972fb0. It shows what the files ask for, not the result of running them.
Pre-approves these tools, so the agent can use them without asking each time:
ReadGrepGlobBashAgentWriteFrom allowed-tools in the SKILL.md frontmatter.
No scripts in the folder and no shell commands in SKILL.md.
From the folder's file list and the shell code blocks in SKILL.md.
Hosts in commands or code, which the agent is likely to contact:
raw.githubusercontent.comFrom URLs in SKILL.md, links to its own repository left out.
Names no API keys, tokens, secrets or passwords.
From names ending in _API_KEY, _TOKEN, _SECRET, _KEY or _PASSWORD in SKILL.md.
Audit Sibling Divergence loads about 4.3k tokens when it runs. Until then it costs about 88 tokens; SKILL.md has 1,289 words of instructions outside code blocks.
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.
The automated check noted patterns worth knowing about, such as sudo or a known installer.
allowed-tools: Read, Grep, Glob, Bash, Agent, WriteAutomated 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.
The full file from ben-manes/caffeine at commit e972fb0, republished under its Apache-2.0 licence (© ben-manes). 1,289 words, ~4,311 tokens.
.claude/skills/audit-sibling-divergence/SKILL.md (or your agent's skills folder).The existing snapshot audits look at one code path and ask "is it correct?" This audit looks at TWO paths that should produce the same observable result and asks "do they actually agree?" Divergence between matched paths is a confirmed historical bug pattern in Caffeine (refresh+expiration sync/async asymmetry, weak/strong publication differences, fast-path/slow-path disagreement under contention).
caffeine/src/javaPoet/java/.jcache/src/main, guava/src/main):
their write/expiry/exception-translation paths are sibling families with each
other AND with the external spec/reference they claim to match (Group G). This is
where the audit's highest-yield gap was — adapters are user-facing yet were never
in any audit's scope.Heavyweight (4-6h of agent time). Token-intensive. Do not run for routine
pre-commit review (use /review-change for that).
The priority matched pairs are listed below. Before launching, glance at the codebase to confirm each pair is still present and add any new pairs (e.g., a new view, a new feature with sync+async variants):
Group A — sync vs async cache (highest historical bug yield)
Cache.get(k, Function) vs AsyncCache.get(k, BiFunction) — load semantics, listener delivery, refresh hand-offLoadingCache.refresh(k) vs AsyncLoadingCache.refresh(k) — completion, exception, stale-detectionCache.asMap().compute* vs AsyncCache.asMap().compute* — atomicity, listener cause, weight deltaCache.invalidate* vs AsyncCache.synchronous().invalidate* — listener cause, in-flight value handlingGroup B — storage variants
PS, FS, PW, FW, PSAW, PSWMS, etc.)
— for each method declared in Node.java that has multiple subclass
implementations, verify all subclasses use consistent access modes,
lifecycle transitions, and weight disciplineGroup C — view consistency
keySet().contains(k) vs containsKey(k) vs asMap().get(k) != nullvalues().contains(v) vs containsValue(v)entrySet() iteration vs forEach() vs keySet() + get(k) per keysize() vs entrySet().size() vs counting via iteratorkeySet().remove(k) / entrySet().remove(entry) vs the corresponding map removal
— mapping outcome, blocking, and pending-load disposition, accounting for different return
contractsGroup D — bulk vs single-key
getAllPresent(keys) vs N×getIfPresent(k)getAll(keys, loader) vs N×get(k, loader)putAll(m) vs N×put(k, v)invalidateAll(keys) vs N×invalidate(k)invalidateAll() vs invalidateAll(allKeys())Group E — equivalent-by-construction
LoadingCache.get(k) vs Cache.get(k, cacheLoader::load)getOrDefault(k, d) vs (get(k) == null ? d : get(k)) (ignoring atomicity)putIfAbsent(k, v) vs compute(k, (key, val) -> val == null ? v : val)AsyncCache.synchronous() view vs an equivalent sync Cache built with
the same configurationGroup F — internal paths to the same outcome
getIfPresent optimistic) vs slow path (under
synchronized(node)) — same key, same logical time, must agree on
present-ness and value identityAddTask, UpdateTask, RemovalTask,
RemovedTask) — weight delta sign convention, telescoping sum
preservation across task orderingsGroup G — adapter conformance (user-facing: jcache/, guava/)
Expirable<V> and gates on
ExpiryPolicy must agree on the create/update guard, the put statistic, and the
CREATED/UPDATED/EXPIRED events: put/putAll (putNoCopyOrAwait), putIfAbsent
(putIfAbsentNoAwait), getAndPut, replace, and the EntryProcessor postProcess
CREATED/UPDATED/LOADED cases. (A zero-creation-expiry guard present in the put helpers
but missing from postProcess was a real bug — a phantom CREATED event + put stat.)CaffeinatedGuavaCache / facade-view
method vs the Caffeine method it delegates to: null-query tolerance, exception
translation (InvalidCacheLoadException / ExecutionException /
UncheckedExecutionException / ExecutionError by checked-ness), bulk partial results.CacheBuilder cache; adjudicate differences against Guava's contract and the accepted facade limits.If a new feature has a sync and async variant not listed above, add it as group H
before launching. When auditing the simulator, add reader-vs-sibling-reader (shared
binary/text formats, e.g. the libCacheSim family) and climber-vs-climber (shared
gradient/timestep convention) port pairs as a group — lower priority, since simulator
divergence yields misleading benchmark numbers rather than user-facing bugs. Sim-internal
pairs only: the production WindowClimber deliberately has NO faithful simulator reference
(product.Caffeine, the real cache, is the arbiter) — do not flag sim-reference-vs-production
divergence as a finding.
Launch one subagent per group (seven groups → seven parallel agents). Each agent gets the prompt below, adapted to its group. Run them in a single message so they execute in parallel.
Tell each subagent to write its report to a group-suffixed path
(.local/audits/<model>/audit-sibling-divergence-group<letter>.md), never to the
canonical audit-sibling-divergence.md — that path is reserved for the
orchestrator's consolidated report (Step 5), and parallel groups writing it
clobber each other. If a group returns its report inline instead, persist it
to the group-suffixed path before launching that group's evaluator.
You are auditing the Caffeine cache for sibling divergence: cases where two
code paths that should produce identical observable behavior do not.
YOUR GROUP: <group letter and pairs from the inventory>
The Caffeine source code is at:
- Core: caffeine/src/main/java/com/github/benmanes/caffeine/cache/
- Generated: caffeine/build/generated/sources/ (run `./gradlew :caffeine:generateNodes
:caffeine:generateLocalCaches` if empty)
- Generators: caffeine/src/javaPoet/java/com/github/benmanes/caffeine/cache/
- Adapters (Group G): guava/src/main/java/com/github/benmanes/caffeine/guava/ and
jcache/src/main/java/com/github/benmanes/caffeine/jcache/. For G2/G4 the reference
contract is external — the JSR-107 1.1.1 spec/TCK and a real Guava `CacheBuilder`
cache; construct the witness as a differential test (run both sides), not a read alone.
To read Guava's actual behavior (no clone needed), WebFetch a specific method from
`https://raw.githubusercontent.com/google/guava/master/guava/src/com/google/common/cache/LocalCache.java`
— the source-level complement to G4's executable oracle; the executable side must use
the pinned Guava version (`libs.versions.toml`), not master.
# Phase 0: Plan
For each pair in your group:
- State the contract the two paths jointly promise (what does an observer
see when they call A vs B?).
- Predict the 2-3 most likely categories of divergence (different access
mode, different listener cause, different exception handling, ordering of
notifications, in-flight value visibility, weight accounting, etc.).
- For blocking differences, identify which thread completes the awaited work. If executor
capacity affects progress, compare one explicitly sized executor with a spare-worker control;
distinguish application dependency cycles from a violated cache progress guarantee.
# Phase 1: Trace each side
For each pair:
1. Locate both implementations. Read each end to end (not just the diff).
2. Build a side-by-side table of the observable steps each path takes:
field reads/writes (with access mode), lock acquisitions, listener
invocations, exceptions thrown, return values.
3. Identify every step where the two paths differ. For each difference:
- Is the difference observable to a caller?
- Is it explained by an intentional design decision? (read
.claude/docs/design-decisions.md and .claude/rules/design-decisions.md
ONLY AFTER you have recorded the difference — design context causes
premature dismissal)
- If observable and unexplained, this is a candidate finding.
# Phase 2: Construct a witness
For each candidate finding, construct a CONCRETE WITNESS:
- Cache configuration (size, weigher, listener, expiry, ...)
- Exact sequence of method calls
- Expected observation if the paths agreed
- Actual observation given the divergence
- Strong enough that a developer could write a failing unit test from it
without further investigation.
A finding without a concrete witness is NOT acceptable — drop it.
# Phase 3: Self-challenge
For each finding, attempt to refute it by re-reading the source. If you
can construct a path through the code that resolves the divergence (e.g.,
the second path also fires the listener through a different route you
missed), drop the finding. Be ruthless — the user wants high-precision
findings, not volume.
For differing guards, compare reachable callers and preconditions on both paths. One path may
establish the condition elsewhere or never reach the guarded state; its safety does not make
the other path's guard redundant, and the difference need not imply a bug on either side.
# Phase 4: Output
For each surviving finding, output:
- PAIR: <pair label, e.g., A1>
- PATH-A: file:method
- PATH-B: file:method
- DIVERGENCE: <one-line description of what differs>
- WITNESS: <concrete scenario>
- OBSERVABLE: <what the caller sees that contradicts the joint contract>
- DESIGN-MATCH: <design-decisions.md item it partially matches, or "none">
- SEVERITY: critical | high | medium | low
- CONFIDENCE: high | medium
If a group has zero findings, output a coverage summary listing every pair
inspected, every method traced, and every difference dismissed (with the
reason for dismissal). Zero findings with thorough coverage is acceptable.
Zero findings with shallow coverage is not — keep looking.
DO NOT report:
- Performance differences (covered by /audit-performance)
- Style differences
- Comment/javadoc differences unless they constitute the contract drift
- Differences explicitly documented as intentional in design-decisions.md
(note them in the "explained" section instead)For each agent that returned findings OR a zero-findings coverage proof, spawn ONE evaluator subagent (general-purpose). The evaluator sees ONLY the reviewer's report — no source code.
You are challenging a differential audit report. Your job is to find what
the auditor MISSED.
For each finding:
1. Is the witness scenario actually reproducible? Identify any unstated
precondition (specific config, timing, prior state) that the witness
does not enumerate but requires.
2. Is the divergence actually observable to a caller? Or is it an internal
difference that produces the same external result?
3. Is the auditor's "joint contract" the actual contract, or did they
assume a stronger contract than the documentation promises?
For each zero-findings claim:
4. Given the auditor's stated coverage, what categories of divergence
might they have under-weighted? (E.g., focused on synchronous control
flow, missed exception paths; focused on happy path, missed in-flight
transitions.)
Output: prioritized list of challenges. Be specific about which finding
and which gap.Have the original reviewer address each challenge by re-reading source. Drop findings the reviewer cannot defend with concrete evidence. Add new findings the reviewer confirms.
Read .claude/docs/design-decisions.md and
.claude/rules/design-decisions.md. For each surviving finding, classify:
When several explained differences affect the same entry lifetime, check at most one concrete sequence combining them per group. Compare its observable outcome with the joint contract and the ruling's mechanism, consequence, trigger, and scope. Combining accepted behavior does not itself reopen a ruling. If reachability or the contract remains unresolved, retain the precise question under Residual risk rather than forcing a bug or documentation-gap label.
Triage confirmed findings by severity per .claude/docs/finding-taxonomy.md.
Tag each confirmed finding with the divergence axis:
Write the full report to .local/audits/<model>/audit-sibling-divergence.md
(see .claude/docs/audit-output.md).
Format:
# Sibling Divergence Audit
[N] differential auditors compared [M] sibling pairs across [G] groups.
[K] findings survived self-challenge and evaluator challenge.
## Confirmed divergences (likely bugs)
#1 [severity] [axis] PAIR — one-line summary
- PATH-A: file:method
- PATH-B: file:method
- DIVERGENCE: ...
- WITNESS: ...
- OBSERVABLE: ...
## Intentional divergences (documented)
- [pair] — link to design doc that explains the difference
## Documentation clarifications
- [pair] — the useful clarification and the existing guidance or prior wording decision checked
## Coverage summary
- Group A: [pairs inspected, methods traced, dismissals]
- Group B: ...
- ...
## Evaluator challenges
- [N] challenges received across [M] groups; [K] led to new findings;
[J] confirmed the original conclusion with additional evidence.
## Residual risk
What was not inspected and why.synchronous()
exposing in-flight CompletableFuture state in unexpected ways.© ben-manes, 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
Just SKILL.md in .claude/skills/audit-sibling-divergence of ben-manes/caffeine.
Open the folder on GitHubat commit e972fb0
Audit Sibling Divergence next to the 5 skills that share the most tags, products or categories with it. Stars are the repository's; “used in” counts other GitHub owners with a copy.
| Skill | Stars | Used in | Tokens | Auto-check | Licence | Repo updated |
|---|---|---|---|---|---|---|
| Audit Sibling Divergence this skillben-manes/caffeine | 18k | — | ~4.3k | Automated safety check: Notes | Apache-2.0 | |
| Code Review Skillawesome-skills/code-review-skill | 2.1k | — | ~2.8k | Automated safety check: Notes | MIT | |
| Cross-Language Coding Standardszereight/gitlab-mcp | 2k | 1 repos | ~1.4k | Automated safety check: Pass | MIT | |
| Code Qualitypiomin/claude-ai-spring-boot | 1.3k | — | ~2.2k | Automated safety check: Pass | Apache-2.0 | |
| Code Review Excellenceandrew-yangy/gru-ai | 155 | — | ~1.7k | Automated safety check: Notes | MIT | |
| Code Revieweralirezarezvani/claude-code-tresor | 777 | — | ~1.8k | Automated safety check: Pass | MIT |
awesome-skills/code-review-skill
Provides comprehensive code review guidance for React 19, Vue 3, Angular 17+, Svelte 5, Rust, TypeScript, Java, Java 8, PHP, Ruby, Rails, Python, Django, FastAPI, Go, C/.NET, Kotlin, Swift, Dart…
zereight/gitlab-mcp
Shared reference for naming, function size, complexity and error handling rules that reviewer agents apply across TypeScript, Python, Go, Rust, Java, C# and Swift.
piomin/claude-ai-spring-boot
Comprehensive code review for Java - clean code principles, API contracts, null safety, exception handling, and performance.
andrew-yangy/gru-ai
Provides comprehensive code review guidance for React 19, Vue 3, Rust, TypeScript, Java, Python, and C/C++.
alirezarezvani/claude-code-tresor
Automatic code quality and best practices analysis. An agent skill from alirezarezvani/claude-code-tresor.
dotnet/maui
Deep code review of PR or materialized candidate-patch changes for correctness, safety, and MAUI conventions.
ben-manes/caffeine
Runs controlled JMH experiments on the Caffeine cache to find shared contention and hot-path waste, then reviews correctness and returns a reviewable patch.
ben-manes/caffeine
Audits a module by walking its git history commit by commit, tracking unresolved issues forward, and reporting the ones that survive to HEAD as findings.
ben-manes/caffeine
Runs a hostile review of the Caffeine Java caching library with parallel subagents that get no design docs, then challenges and consolidates their findings.
ben-manes/caffeine
Audits the Caffeine cache source for hot-path costs such as allocations, contention and memory layout, reporting only findings tied to specific lines.
ben-manes/caffeine
Prices each step of the window climber algorithm by disabling it in turn, to find steps that no longer earn their keep and branches that no longer fire.
ben-manes/caffeine
Runs three parallel reviewers on a diff or branch, one blind, one design-aware and one matching past bug patterns, then triages their findings.
Works with
Categories
Compares code paths that should behave the same, such as sync and async cache methods, and requires a concrete scenario where the two observably disagree. Instead of asking whether one code path is correct, this audit asks whether two paths that should give the same observable result actually agree. It was written for the Caffeine Java caching library, where mismatches between paired paths have caused real bugs, and it starts by inventorying matched pairs: sync and async caches, bounded and unbounded storage, generated node variants, view consistency, bulk versus single operations, fast versus slow read paths and adapter conformance.
Audit Sibling Divergence fits situations like: after significant changes to a library with paired sync and async code paths; before a major release, as a check for silent drift between sibling implementations; auditing adapters against the specification they claim to match.
Run `npx skills add ben-manes/caffeine --skill audit-sibling-divergence -a claude-code`. Or copy the skill folder (.claude/skills/audit-sibling-divergence in ben-manes/caffeine) into .claude/skills/audit-sibling-divergence in your project. Claude Code loads it when a task matches its description.
Run `npx skills add ben-manes/caffeine --skill audit-sibling-divergence -a codex`. Or copy the skill folder (.claude/skills/audit-sibling-divergence in ben-manes/caffeine) into .agents/skills/audit-sibling-divergence in your project. Codex loads it when a task matches its description.
Cursor, Gemini CLI, GitHub Copilot and OpenCode also load SKILL.md folders. With the skills CLI, run `npx skills add ben-manes/caffeine --skill audit-sibling-divergence -a cursor` (or -a gemini-cli, github-copilot or opencode for the others). To copy it by hand, put the folder in .cursor/skills/audit-sibling-divergence, .gemini/skills/audit-sibling-divergence, .github/skills/audit-sibling-divergence and .opencode/skills/audit-sibling-divergence in your project.
SKILL.md names no scripts, command-line tools or credentials: Audit Sibling Divergence is instructions for the agent only. Its frontmatter pre-approves these tools: Read, Grep, Glob, Bash, Agent, Write.
SKILL.md names 1 domain. In commands or code: raw.githubusercontent.com; the agent is likely to contact it when it follows the instructions. This is read from the text; nothing was executed.
Our automated static check of SKILL.md found notes only (pre-approves every shell command (allowed-tools: bash)), nothing it rates as a warning. It is not a guarantee. Review the folder before installing.
Audit Sibling Divergence is published under the Apache-2.0 licence (the repository's licence). It allows redistribution, so the full SKILL.md is shown on this page.
About 4.3k tokens (SKILL.md is roughly 17k characters). Agents keep only the skill's name and description in context until a task matches; then they load SKILL.md in full.
Skills that share tags, products or a category with Audit Sibling Divergence: Code Review Skill (awesome-skills/code-review-skill, 2.1k stars), Cross-Language Coding Standards (zereight/gitlab-mcp, 2k stars), Code Quality (piomin/claude-ai-spring-boot, 1.3k stars) and Code Review Excellence (andrew-yangy/gru-ai, 155 stars). The comparison table on this page puts their stars, adoption, token cost, safety result and licence side by side.
ben-manes (a GitHub user) maintains it in ben-manes/caffeine, which has 17,882 GitHub stars. The repository holds 33 skills in this directory. The repository was last updated on October 9, 2026.
Source: ben-manes/caffeine on GitHub. Facts on this page come from the repository at the commit we read; the author's words are quoted as theirs.