kienbui1995/pr-workflow
skills/pr-workflow/SKILL.md
Use when creating pull requests - PR structure, description templates, review checklist, merge strategies, branch naming
testing
Updated Apr 24, 2026
$ install --global
skillsauthnpx skillsauth add kienbui1995/magic-powers pr-workflowInstall this skill globally with one command. Works with Claude Code, Cursor, and Windsurf.
Security Scan Results
3 of 9 scanners reported clean
Some scanners were skipped, did not run, or reported a non-clean status. Review each row below.
3
Scanners Passed
9
Scanners in report
Clean
TrivyContainer and dependency vulnerability scanner
95%
Clean
SemgrepStatic code analysis for vulnerabilities
95%
Clean
mcp-scan (Snyk)Model Context Protocol security validation
95%
Skipped
Snyk (dep)Open source security scanning
50%
Skipped
Socket.devSupply chain security analysis
50%
Skipped
VirusTotalMulti-engine malware detection
50%
Skipped
CrowdStrikeAdvanced threat intelligence
50%
Skipped
OSV-ScannerOpen Source Vulnerability database check
50%
Skipped
OWASP Dep-Check
50%
Last scanned: Apr 24, 2026, 4:04 AM275.4s1 file scanned
SKILL.md
- name:
- pr-workflow
- description:
- Use when creating pull requests - PR structure, description templates, review checklist, merge strategies, branch naming
PR Workflow
Overview
Good PRs are small, well-described, and easy to review. A PR is a communication tool, not just a code delivery mechanism.
When to Use
- Creating a pull request
- Reviewing PR size and structure
- Setting up PR templates and branch policies
- Choosing merge strategy
PR Size Rules
| Size | Lines Changed | Review Time | Quality | |------|--------------|-------------|---------| | ✅ Small | <200 | 15 min | Best reviews | | ⚠️ Medium | 200-400 | 30 min | Acceptable | | ❌ Large | >400 | 60+ min | Rubber-stamp risk |
If PR is too large: split into stacked PRs (infra → logic → UI).
PR Description Template
## What
Brief description of the change.
## Why
Link to issue/ticket. Context on why this approach.
## How
Key implementation decisions. What reviewers should focus on.
## Testing
- [ ] Unit tests added/updated
- [ ] Manual testing done
- [ ] Edge cases considered
## Screenshots
(if UI change)
Branch Naming
feature/TICKET-123-add-user-auth
fix/TICKET-456-null-pointer-login
chore/update-dependencies
docs/api-authentication-guide
Merge Strategies
| Strategy | History | Use When | |----------|---------|----------| | Squash merge | Clean, 1 commit per PR | Default for most teams | | Merge commit | Preserves all commits | Need full history | | Rebase | Linear history | Small teams, clean commits |
Review Checklist (for reviewers)
- [ ] Does the code do what the PR description says?
- [ ] Are there tests for the changes?
- [ ] No obvious security issues (SQL injection, XSS, auth bypass)
- [ ] No hardcoded secrets or config
- [ ] Error handling present
- [ ] Naming is clear and consistent
Anti-Patterns
| Pattern | Fix | |---------|-----| | "WIP" PRs open for days | Use draft PRs, keep them short-lived | | No description | Require PR template | | Mixing refactor + feature | Separate PRs | | Approving without reading | Use review checklist |
Integration
- magic-powers:requesting-code-review — how to ask for effective reviews
- magic-powers:receiving-code-review — how to respond to feedback
- magic-powers:finishing-a-development-branch — merge and cleanup workflow
- magic-powers:ci-cd-pipeline — CI must pass before merge
Related Skills
kienbui1995/xr-interface-design
content-media
VerifiedTrustedCommunity
Use when designing for XR (AR/VR/MR), choosing interaction modes, or adapting 2D UI patterns for spatial computing
SKILL.mdUpdated Apr 24, 2026
kienbui1995/writing-skills
testing
VerifiedTrustedCommunity
Use when creating new skills, editing existing skills, or verifying skills work before deployment
SKILL.mdUpdated Apr 24, 2026
kienbui1995/writing-plans
development
VerifiedTrustedCommunity
Use when you have a spec or requirements for a multi-step task, before touching code
SKILL.mdUpdated Apr 24, 2026
kienbui1995/workflow-templates
development
VerifiedTrustedCommunity
Use when executing a structured workflow — select and run a feature, bugfix, refactor, research, or incident template with correct agent and model assignments per phase.
SKILL.mdUpdated Apr 24, 2026