| name | review-workflow |
| description | Internal Dynamic Workflow asset for /lets:review --workflow. Not auto-triggered - it is a workflow script invoked via scriptPath, not a conversational skill. Holds the fan-out -> dedupe -> adversarial-verify -> aggregate chain that runs off-context. |
| user-invocable | false |
review-workflow (Dynamic Workflow asset)
This folder is the reference example of the Dynamic Workflow asset standard (see CLAUDE.md -> "Dynamic Workflow assets"). It is NOT a conversational skill: review.workflow.js is executed by the Workflow tool, invoked from /lets:review's Workflow Mode via:
Workflow({ scriptPath: "${CLAUDE_PLUGIN_ROOT}/skills/review-workflow/review.workflow.js", args })
${CLAUDE_PLUGIN_ROOT} is substituted at command-load time, so review.md carries the literal absolute path. Treat review.workflow.js as a template the command points at, not a script to reproduce inline.
What it does (off-context)
A multi-stage chain so per-agent reports never enter the conversation - only the final aggregate returns:
- Review - fan out the selected
lets:<name> review agents in parallel (structured FINDING_SCHEMA output).
- Reduce - split systemic (any tier) into its own section, NIT-filter the rest (unless small diff), dedupe (keep highest tier), sort by tier.
- Verify - for each BLOCKER/SUGGESTION, fan out
lets:skeptic agents (2, or 3 for BLOCKER) to refute it. Apply the asymmetric drop rule: SUGGESTION drops on a simple majority real=false; BLOCKER drops only on near-unanimous high-confidence refute, otherwise downgrades. Count refuted.
- Aggregate - verdict over the verified set, per-agent summary, counts.
args contract (built by the command, passed as a JSON value)
| key | type | meaning |
|---|
agents | [{name}] | selected review agents (short names, no lets: prefix; exclude actor) |
mode | string | PR-{n} | local-review | branch-review | file |
projectRoot | string | absolute project root (agents must not read outside it) |
claudeMd | string | CLAUDE.md context |
changedFiles | string | changed-file list with stats (or single path for --file) |
code | string | the diff (or full file content for --file) |
smallDiff | bool | true keeps NIT findings (diff < 50 lines) |
systemicCheck | bool | false for --file (no diff baseline) |
Returns
{ verdict, findings[], systemic[], summary{}, counts{blockers,suggestions,total,refuted,verify_failed} } - the same shape review.md Step 8/8.5 persists. verify_failed > 0 means that many findings could not be verified (skeptics errored) and were kept unverified - surface it, never treat as "verified clean". findings + verdict are the /lets:github-pr --json contract; do not rename them.
Constraints (Dynamic Workflow runtime)
- No filesystem - the script returns data;
/lets:review persists files.
- No sibling
import - all logic stays inline in review.workflow.js.
- No
Date.now() / Math.random() / new Date().
- Top-level
await/return are used (the runtime wraps the body), so the file is NOT Node-importable - it has no clean unit test; the verdict/dedupe/verify logic is kept in sync with review.md prose by discipline and validated by the live smoke test.
