# Doncheli Pr Review

> 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".

- **Type:** Skill
- **Install:** `agentstack add skill-doncheli-don-cheli-sdd-doncheli-pr-review`
- **Verified:** Yes — security-reviewed for prompt injection and unsafe behavior
- **Seller:** [doncheli](https://agentstack.voostack.com/s/doncheli)
- **Installs:** 0
- **Category:** [AI & ML](https://agentstack.voostack.com/c/ai-and-ml)
- **Latest version:** 0.1.0
- **License:** Apache-2.0
- **Upstream author:** [doncheli](https://github.com/doncheli)
- **Source:** https://github.com/doncheli/don-cheli-sdd/tree/main/.opencode/skills/doncheli-pr-review
- **Website:** https://doncheli.tv

## Install

```sh
agentstack add skill-doncheli-don-cheli-sdd-doncheli-pr-review
```

Requires the [AgentStack CLI](https://agentstack.voostack.com/docs/cli). Works with Claude Code, Cursor, and any MCP-compatible agent.

## About

# 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
4. Categorize each finding: [blocker | warning | suggestion | nitpick]
5. Blockers must be resolved before merge — list them prominently
6. Never approve a PR with unresolved blockers
7. Generate a structured review comment ready to post with `gh pr review`
8. 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.
```

## Source & license

This open-source skill is cataloged on AgentStack and links to its original source — we do not rehost the code.

- **Author:** [doncheli](https://github.com/doncheli)
- **Source:** [doncheli/don-cheli-sdd](https://github.com/doncheli/don-cheli-sdd)
- **License:** Apache-2.0
- **Homepage:** https://doncheli.tv

Install and usage instructions live in the source repository linked above.

## Pricing

- **Free** — Free

## Security capabilities

Automated source analysis of v0.1.0 — what this tool can access:

- **Network access:** no
- **Filesystem access:** no
- **Shell / process execution:** no
- **Environment & secrets:** no
- **Dynamic code execution:** no

*"Yes" means the capability is present in the source — more access means more to trust, not that it is unsafe.*


## Versions

- **0.1.0** — security scan: passed — Imported from the upstream source.

## Links

- Listing page: https://agentstack.voostack.com/l/skill-doncheli-don-cheli-sdd-doncheli-pr-review
- Seller: https://agentstack.voostack.com/s/doncheli
- Browse the marketplace: https://agentstack.voostack.com/browse

---
Listed on AgentStack — the marketplace for AI agent skills and MCP servers. Every listing is security-reviewed. Creators keep 70%.