# Review Grant

> Review an explicitly identified marin-iac grant PR that edits IAM data or a deploy-target module, confirm its decrypted principals and roles, then apply only the confirmed grant.

- Skill: `marin-community/review-grant` (Agent Skill)
- Install (CLI): `npx skillmds@latest add marin-community/review-grant`
- Raw SKILL.md: https://api.skillmd.com/api/skills/marin-community/review-grant/raw
- Safety review: pending
- Works with: Claude Code, Claude.ai, OpenAI Codex
- Category: DevOps & Infra
- Author: marin-community (https://skillmd.com/u/marin-community)
- Updated: 2026-09-17
- Page: https://skillmd.com/skills/marin-community/review-grant

---


# Skill: Review a user grant

A grant PR (usually from the `add-grant` skill) uses opaque `human-NNN`
references whose emails are KMS ciphertext in `iam_data.yaml`. This skill
reveals the real grant, gets an explicit human confirmation, then lands it and
applies it.

Read first:

- `infra/pulumi/README.md` — the marin-iac stacks and the `pulumi up`
  prerequisites (you need `roles/cloudkms.cryptoKeyEncrypterDecrypter` on the key
  and permission to update the `marin` stack).

**Never approve or merge before the user confirms the decrypted grant.** The
whole point is that a second person sees the actual identity and access before it
is applied.

## 1. Fetch the PR

```bash
gh pr view <n> --repo marin-community/marin --json title,body,headRefName,files,url
gh pr checkout <n>          # pull the branch into the worktree
git fetch origin main
```

Confirm the diff only touches grant surfaces under `infra/pulumi/src/iac/gcp/`:
`iam_data.yaml` and/or a deploy-target IAM module. If it
changes anything else (code, other Pulumi resources), stop and review it as an
ordinary PR, not a grant.

## 2. Decrypt the changed principals

Turn the changed opaque principal references into real emails:

```bash
git diff origin/main...HEAD -- infra/pulumi/src/iac/gcp \
  | uv run --package marin-iac --extra deploy \
      python infra/pulumi/iam_principal.py decrypt --diff
```

Each output line is `+ user:<email>` (added) or `- user:<email>` (removed). Map
each back to the role and resource it sits under in the diff. The decryptor
shows the principal; read the surrounding `role` and container
(`project_grants`, a specific bucket/secret/repository/service account, or a
deploy-target module) from the diff hunk.

## 3. Present the grant and get confirmation

Print a plain-language summary, one line per grant, and ask the user to confirm.
For example:

```
PR #1234 grants:
  + alice@openathena.ai → roles/storage.objectViewer on project hai-gcp-models
  + alice@openathena.ai → IAP viewer on evaldash.oa.dev
  - bob@openathena.ai → roles/bigquery.dataViewer (revoked)
Apply this? (yes/no)
```

Call out anything that looks off: a broader role than the resource needs, a
principal you do not recognize, a domain wildcard, or a revocation that might
cut off active access. If the user does not clearly approve, stop and report
back — do not merge.

## 4. Approve, merge, land

After the user confirms:

```bash
gh pr review <n> --repo marin-community/marin --approve
gh pr merge <n> --repo marin-community/marin --squash
git checkout main && git pull origin main   # land the merged change locally
```

Wait for the merge to land on `main` and pull it before applying, so `pulumi up`
runs against the committed state.

## 5. Apply with pulumi up

The change is not live until `pulumi up` runs — CI never applies. Identify the
affected stack(s) from the diff:

- Any grant surface under `infra/pulumi/src/iac/gcp/` → the **`marin`** stack in
  `infra/pulumi`.

Prompt the user: they can run it themselves, or ask you to. If you run it, per
stack:

```bash
cd infra/<dir>
pulumi stack select <stack>
pulumi preview      # confirm ONLY the intended grant is added/removed, no other drift
pulumi up
```

Read the preview before applying. For the `marin` stack, expect only the
`IAMMember` create/delete for this grant — **any NodePool or other unexpected
`replace`/`delete` means stop** and reconcile, exactly as the pulumi README
warns. Once `up` is clean, tell the user the grant is live.

## 6. Confirm on the PR

Comment on the merged PR that `pulumi up` ran and the grant is live, so the
requester and any watcher see the change reached production, not just `main`:

```bash
gh pr comment <n> --repo marin-community/marin \
  --body "🤖 \`pulumi up\` on the \`<stack>\` stack succeeded — the grant is live."
```

An agent comment must begin with `🤖` (see AGENTS.md). If `pulumi up` did not run
(the user is applying it themselves), skip this and let them confirm instead.