id-reviewlisted

# /foundry:id-review — the infra-delivery two-lens advisory review procedure (general always + security on high-risk, over diff + plan-as-DATA + policy-risk findings) The `infra-delivery` step sequence (a documented procedure this skill family forms — no workflow engine or state-machine file ships)'s **review** step (step 10) — the infra analog of `sd-review`. A separate-context, infra-specific **mistake-catcher** pass over an infra change BEFORE the merge pivot. Without a procedure the implementing agent reviews in its own already-biased context, runs a single ad-hoc pass, forgets the security lens on an IAM-policy / connection-string / provider-bump change, misses that a plan REPLACE/deletes a stateful resource, or (worst) treats its own advisory review as if it were an approval to merge. This skill is the disciplined procedure the agent runs instead: it **dispatches the two MERGED review lenses** — the general code lens **always**, the security lens **when the change is high-risk** — feeds them the **infra context** (the diff + the change's **policy-risk findings** + the plan's **`actions_detail`**, all as **plan-as-DATA**), collates their categorized findings for the trusted operator, and **surfaces the extra-approval expectation** for the operator/reviewer to weigh at the merge floor. It defines **no reviewer of its own**; the `pr-reviewer` and `security-reviewer` agents own the review craft. ## When to trigger - An infra atom's implementation is complete and the chan