skills/cross-harness-review/SKILL.md
Run a read-only code review through the opposite AI coding harness and return uinaf `review`-style findings, evidence, unverified gaps, and a ship-it / needs-review / blocked verdict. Use for `/review-with-claudex`, cross-harness review, opposite-model review, Claude reviewing Codex work, or Codex reviewing Claude work. Not an implementation or fix loop.
npx skillsauth add uinaf/skills cross-harness-reviewInstall this skill globally with one command. Works with Claude Code, Cursor, and Windsurf.
3 of 9 scanners reported clean
Some scanners were skipped, did not run, or reported a non-clean status. Review each row below.
Use the other coding harness as the independent reviewer, then report in the compact review verdict shape.
For CLI flag details, fallback commands, and hosted-review cautions, read references/headless-modes.md.
Check only the opposite harness:
# Codex active -> Claude reviewer
command -v claude
claude --version
claude auth status
# Claude active -> Codex reviewer
command -v codex
codex --version
codex login status
Report only installed/authenticated status. Keep account details, tokens, and config contents out of the report.
Use /review or $review if available. Otherwise follow the uinaf review contract:
findings first, then verdict exactly one of ship it / needs review / blocked,
then evidence, unverified, next, and optional notes.
Target: <uncommitted diff | branch vs base | commit | PR | paths>
Review read-only. Leave files, staging, commits, and fixes untouched.
Prioritize bugs, regressions, missing tests, silent failures, and contract drift. Use concise evidence with file/line references and targeted command results.
Use Claude Code print mode:
claude -p \
--permission-mode dontAsk \
--allowedTools "Read,Glob,Grep,LS,Bash(git *),Bash(rg *),Bash(jq *),Bash(npm *),Bash(pnpm *)" \
"<review prompt>"
Use hosted claude ultrareview only when the user explicitly asks and accepts the cost.
Use Codex exec review:
codex exec review --uncommitted --ephemeral - <<'PROMPT'
<shared prompt with Target set to current uncommitted diff>
PROMPT
Variants: branch review uses --base <base-ref>, commit review uses --commit <sha>, and custom scope belongs in the prompt.
Match review:
- findings: none
- verdict: ship it
- evidence: Claude review covered the uncommitted diff and found no material issues
- unverified: runtime smoke not rerun
- next: verify
If the opposite harness cannot run:
- findings: none
- verdict: blocked
- evidence: Claude Code is not installed or authenticated, so no independent cross-harness review ran
- unverified: review target not inspected by the opposite harness
- next: review
development
Ban direct `useEffect` in React code. Use when writing, refactoring, reviewing, or migrating React components or hooks that import, call, add, or replace direct `useEffect`; when an agent reaches for effects for derived state, fetching, event reactions, resets, or external sync; or when adding lint/agent rules for a no-direct-useEffect policy. Do not use for ordinary React work with no effect smell, non-React code, or legitimate effect architecture outside React.
development
Independently audit existing code, diffs, branches, or pull requests by spawning mandatory concern-specific reviewer subagents, then synthesizing their evidence into a ship decision. Use when triaging PR risk, deciding whether someone else's change is safe to ship, or following up after runtime proof. Invocation is explicit authorization to use reviewer subagents. Produces a `ship it` / `needs review` / `blocked` verdict. Do not use to self-check a change you just authored.
testing
Set up or align a repository's GitHub collaboration and delivery surface: repo settings, branch/ruleset policy, PR and security templates, Actions hardening, GitHub Environments, release workflows, and deploy workflows. Use when standardizing GitHub setup for repos, CI/CD, publishing versioned packages, or deploying running apps; route app deploy details to deploy references and package publish details to release references.
development
Run structured Codex/Claude autoreview closeout for local changes, pull requests, branch diffs, or commits: choose the target, validate findings, rerun focused tests, and repeat review until clean. Use when asked for autoreview, second-model review, pre-merge review, or readiness-to-ship review.