harness-pr-review

name	harness-pr-review
description	Review a PR for six-layer harness, provider, ADR, and enforcement consistency
metadata	{"type":"Playbook","origin":"harness-engineering/ch15","helpful":0,"harmful":0,"status":"active"}

Harness PR Review

Steps

1. List changed files and classify their layers.
2. Inspect import changes for reverse dependencies or layer skips.
3. Search for process.env, ORM clients, vendor SDKs, and raw fetch.
4. Confirm cross-cutting concerns use providers.
5. Decide whether ADR or skill updates are required.
6. Run the relevant static checks.

Checklist

• Changed files are mapped to layers
• Imports are forward-only
• process.env and ORM access stay inside allowed layers
• Cross-cutting concerns go through providers
• ADR / skill update need is explicitly decided