Mmcp.market

doncheli-pr-review skill

by doncheli·doncheli/don-cheli-sdd·57 stars·Apache-2.0

Perform a structured code review of a pull request aligned with SDD principles. Activate when user mentions "PR review", "pull request", "review PR", "code review", "review this diff".

A100/100content scan

Is the doncheli-pr-review skill safe?

Clean: nothing in its files matched our rules. We read 1 file in the folder on 2026-09-28.

No findings.

Install the doncheli-pr-review skill

A skill is a folder. Copy it into your agent's skills folder and the agent loads it when the task matches its description. 

git clone --depth 1 https://github.com/doncheli/don-cheli-sdd.git /tmp/don-cheli-sdd
mkdir -p ~/.claude/skills
cp -r /tmp/don-cheli-sdd/.opencode/skills/doncheli-pr-review ~/.claude/skills/doncheli-pr-review
available in every project

In the Claude apps, zip the folder and upload it from the Skills settings. The folder on GitHub

The instructions your agent would load

SKILL.md as published, without the frontmatter. Read it on GitHub

Don Cheli: PR Review

Instructions

  1. Accept a PR number, URL, or diff as input
  2. Fetch the diff using gh pr diff or read the provided diff
  3. Review in this sequence:
  • Spec alignment: does the code implement what the spec says? Flag any drift
  • Tests: are new behaviors covered? Is coverage maintained above 85%?
  • Code quality: complexity, duplication, naming, error handling
  • Security: OWASP Top 10 surface, secrets in code, input validation
  • Breaking changes: API contract changes, schema migrations, deprecations
  1. Categorize each finding: [blocker | warning | suggestion | nitpick]
  2. Blockers must be resolved before merge — list them prominently
  3. Never approve a PR with unresolved blockers
  4. Generate a structured review comment ready to post with gh pr review
  5. If spec docs are missing for a non-trivial feature, flag as a warning

Output Format

## PR Review — #142 "feat: add voice mode integration"

### Spec Alignment ✅
All scenarios in specs/voice.feature are covered by the implementation.

### Blockers 🔴
1. src/voice/transcriber.ts:34 — API key hardcoded. Must use env var VOICE_API_KEY.

### Warnings 🟡
1. src/voice/transcriber.ts — coverage 72%, below 85% threshold.
2. No migration doc for the new voice_sessions table.

### Suggestions 🟢
1. Consider extracting the cleanup logic at line 88 into a private method.

### Nitpicks
1. Line 12: typo "recieve" → "receive"

### Verdict: CHANGES REQUESTED
Resolve 1 blocker before merging.

More skills from doncheli/don-cheli-sdd

  • Adoncheli-api-contractDesign complete API contracts covering endpoints, auth, rate limiting, error handling, retries, circuit breaker and idempotency. Activate when user mentions "api contract", "api design", "endpoint", "webhook", "REST", "GraphQL", "OpenAPI", "design the API".
  • Adoncheli-api-contractDesign complete API contracts covering endpoints, auth, rate limiting, error handling, retries, circuit breaker and idempotency. Activate when user mentions "api contract", "api design", "endpoint", "webhook", "REST", "GraphQL", "OpenAPI", "design the API".
  • Adoncheli-audit-trailRecord and query the decision log for a project. Activate when user mentions "audit", "trail", "log decisions", "decision history", "why was this decided", "ADR", "architecture decision".
  • Adoncheli-audit-trailRecord and query the decision log for a project. Activate when user mentions "audit", "trail", "log decisions", "decision history", "why was this decided", "ADR", "architecture decision".
  • Adoncheli-changelogAuto-generate CHANGELOG.md entries from git commit history. Activate when user mentions "changelog", "release notes", "what changed", "generate changelog", "CHANGELOG", "release history".
  • Adoncheli-changelogAuto-generate CHANGELOG.md entries from git commit history. Activate when user mentions "changelog", "release notes", "what changed", "generate changelog", "CHANGELOG", "release history".
  • Adoncheli-context-healthReport the current state of the context window and recommend compression or cleanup actions. Activate when user mentions "context health", "context window", "how much context", "context full", "running out of context", "compress context".
  • Adoncheli-context-healthReport the current state of the context window and recommend compression or cleanup actions. Activate when user mentions "context health", "context window", "how much context", "context full", "running out of context", "compress context".
  • Adoncheli-data-policyAudit and document what personal or sensitive data the project collects, processes, and stores. Activate when user mentions "privacy", "data policy", "what data", "GDPR", "personal data", "data retention", "PII".
  • Adoncheli-data-policyAudit and document what personal or sensitive data the project collects, processes, and stores. Activate when user mentions "privacy", "data policy", "what data", "GDPR", "personal data", "data retention", "PII".
  • Adoncheli-debateRun an adversarial multi-role debate to surface trade-offs and reach a reasoned decision. Activate when user mentions "debate", "discuss", "trade-off", "decision", "compare options", "pros and cons", "choose between".
  • Adoncheli-debateRun an adversarial multi-role debate to surface trade-offs and reach a reasoned decision. Activate when user mentions "debate", "discuss", "trade-off", "decision", "compare options", "pros and cons", "choose between".

All agent skills → · MCP servers