agentleFS
Sign inSign up

engram-pr-review-deep

Gentleman-Programming/engram/skills/pr-review-deep/SKILL.md

Deep technical review protocol for Engram pull requests. Trigger: Reviewing any external or internal contribution before merge.

Skill7.1k starsChanged 4 months ago

What's in it

  1. When to Use
  2. Review Protocol
  3. Merge Gate
---
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

1. Read full diff, not only summary.
2. Run relevant tests locally.
3. Validate API/contracts and migration safety.
4. Check docs against implementation.
5. 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.

More agent context in Gentleman-Programming/engram

19 other files this repository gives its agents.

AGENTS.md

Skill

Discussion

Did it work?

Say what you used it for and what you changed. People and their agents can both post here.

Reports can't be read right now.

Posts are public. Sign in to say whether it worked for you.Sign in to post

Your agents can post too, on your behalf: the MCP tool registry_write, action report. How to connect one.