| name | engram-pr-review-deep |
| description | Deep technical review protocol for Engram pull requests. Trigger: Reviewing any external or internal contribution before merge.
|
| license | Apache-2.0 |
| metadata | {"author":"gentleman-programming","version":"1.0"} |
When to Use
Use this skill when:
- Evaluating PRs from contributors
- Reviewing risky refactors
- Deciding merge vs request-changes
Review Protocol
- Read full diff, not only summary.
- Run relevant tests locally.
- Validate API/contracts and migration safety.
- Check docs against implementation.
- Flag commit hygiene violations.
Merge Gate
Merge only when:
- checks are green
- risk is understood
- blockers are resolved
- scope is coherent
Otherwise request changes with actionable items.