$npx -y skills add PolicyEngine/policyengine-claude --skill review-programALWAYS LOAD THIS SKILL for PolicyEngine PR reviews, including when the user invokes $review-program or Codex /review on a PolicyEngine PR. Performs read-only code validation, source-reference checks, regulatory review, optional PDF audit, summary reporting, and optional GitHub co
| 1 | # review-program |
| 2 | |
| 3 | Thin launcher. The canonical workflow — phases, roles, gates, severity rules, artifact |
| 4 | contracts — lives in [references/workflow.md](references/workflow.md). Read it completely |
| 5 | before acting and follow it exactly; this file only adapts it to this surface. Do not |
| 6 | redefine flags, phases, severity rules, file ownership, or completion gates here. |
| 7 | |
| 8 | Use this skill for PolicyEngine-specific PR reviews even when the user starts from a |
| 9 | generic built-in review command (such as Codex `/review`): the canonical workflow adds |
| 10 | PolicyEngine source, reference, regulatory, PDF, and test-review rules a generic review |
| 11 | lacks. |
| 12 | |
| 13 | Treat the text after `$review-program` or `/review` as the raw workflow arguments: |
| 14 | |
| 15 | ```text |
| 16 | $review-program [PR_NUMBER_OR_SEARCH] [PDF_URL] [--local] [--local-diff] [--full] |
| 17 | [--skip-pdf] [--600dpi] [--resume] [--incremental REPORT] [--prefix NAME] |
| 18 | ``` |
| 19 | |
| 20 | Mandatory completion gate: |
| 21 | |
| 22 | - Do not stop after individual validators or PDF audits. |
| 23 | - The review is incomplete until `{RUN_ROOT}/{PREFIX}-review-full-report.md` and |
| 24 | `{RUN_ROOT}/{PREFIX}-review-summary.md` both exist. |
| 25 | - Phase 6 consolidation is required before displaying or posting findings. |
| 26 | |
| 27 | Surface adapters: |
| 28 | |
| 29 | - **Claude Code**: also read |
| 30 | [references/claude-launcher.md](references/claude-launcher.md) — it maps canonical |
| 31 | roles to this plugin's agent types and the workflow's abstract operations to Claude |
| 32 | Code mechanics. |
| 33 | - **Codex**: use the delegation mapping below. |
| 34 | |
| 35 | Delegation mapping (when subagent use is available and authorized): |
| 36 | |
| 37 | - Map every role in the canonical Roles table to a `worker` when it writes report or |
| 38 | artifact files, or an `explorer` for read-only codebase questions. |
| 39 | - Pass concrete `RUN_ROOT`, `WORKTREE_ID`, and `PREFIX` values and the role's task spec |
| 40 | from the canonical workflow to every subagent; name the role's skills explicitly in the |
| 41 | subagent prompt — do not assume a worker infers them from parent context. |
| 42 | - Every subagent finishes by writing its assigned file and returning the one-line DONE |
| 43 | message from the canonical completion contract. |
| 44 | - If subagents are unavailable or not authorized, execute the roles directly in phase |
| 45 | order, preserving the same handoff files and read-only contract. |