engram-pr-review-deep

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

Safety Notice

This listing is imported from skills.sh public index metadata. Review upstream SKILL.md and repository scripts before running.

Copy this and send it to your AI assistant to learn

Install skill "engram-pr-review-deep" with this command: npx skills add gentleman-programming/engram/gentleman-programming-engram-engram-pr-review-deep

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.

Source Transparency

This detail page is rendered from real SKILL.md content. Trust labels are metadata-based hints, not a safety guarantee.

Open in GitHubOpen in ClawHub

Related Skills

Related by shared tags or category signals.

Coding

engram-memory-protocol

No summary provided by upstream source.

Repository SourceNeeds Review
-27
gentleman-programming
Coding

gentleman-bubbletea

No summary provided by upstream source.

Repository SourceNeeds Review
-18
gentleman-programming
Coding

engram-testing-coverage

No summary provided by upstream source.

Repository SourceNeeds Review
-17
gentleman-programming