nsheaps/review
plugins/sdlc-utils/skills/review/SKILL.md
Code review practices and quality checks. Use when the user asks to "review code", "review a PR", "code review", "check code quality", "review changes", "score this code", or when evaluating code for merge readiness. Covers review checklists, scoring criteria, feedback conventions, and iterative improvement until quality thresholds are met.
$ install --global
skillsauthnpx skillsauth add nsheaps/ai-mktpl reviewInstall 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 22, 2026, 4:17 PM57.7s1 file scanned
SKILL.md
- name:
- review
- description:
- >
Code Review
Structured code review for evaluating and improving code quality before merge.
Review Categories
Evaluate code across these dimensions, scoring each 0-100:
| Category | What to check | | ----------------- | ------------------------------------------------- | | Simplicity | Is the code as simple as it can be? | | Correctness | Does it do what the spec says? | | Security | Are there vulnerabilities or unsafe patterns? | | Performance | Are there obvious performance issues? | | Maintainability | Can someone else understand and modify this? | | Pattern adherence | Does it follow existing codebase conventions? | | Test coverage | Are changes covered by tests? | | Documentation | Are public APIs and non-obvious logic documented? |
Score Thresholds
| Score | Status | Meaning | | ------ | ------ | --------------------------- | | >= 85% | Pass | Ready to merge | | 70-84% | Warn | Should address before merge | | < 70% | Block | Must address before merge |
Review Workflow
Step 1: Understand Context
- Read the PR description and linked spec/issue
- Understand what the code is supposed to do
- Check the full diff against the base branch (not just latest commit)
Step 2: Review
For each category:
- Examine the relevant code
- Note specific findings with file and line references
- Assign a score
- Provide actionable feedback for anything below 85%
Step 3: Provide Feedback
- Be specific: reference exact lines and files
- Be actionable: say what to change, not just what is wrong
- Assign priority levels to all findings: P0 (critical), P1 (important), P2 (nice-to-have)
- All findings must be listed — none should be silently dismissed
Step 4: Iterate
If scores are below threshold, the author addresses feedback and requests re-review. Repeat until all categories pass.
Verdicts
| Verdict | When | | --------------- | ------------------------------------------------- | | Approve | All categories >= 85%, no P0 or P1 issues | | Comment | Only P2 follow-ups remain | | Request Changes | Any category < 70% or security/correctness issues |
Anti-Patterns
| Anti-Pattern | Instead | | ----------------------- | ------------------------------------------ | | Rubber-stamping | Actually read and evaluate the code | | Nitpicking style only | Focus on substance (correctness, security) | | Vague feedback | Give specific, actionable comments | | Reviewing only the diff | Understand the full context |
External References
- Google Engineering Practices: Code Review
- Conventional Comments
Related Skills
nsheaps/github-app-session-env
tools
VerifiedTrustedCommunity
Manually reproduce what the github-app plugin's SessionStart hook does to make a GitHub App installation token usable in the current session — materialize the PEM, generate the token, isolate GH_CONFIG_DIR, write the runtime env file, and wire CLAUDE_ENV_FILE so every Bash call sees GH_TOKEN/GITHUB_TOKEN. Use when the hook did not run, the token is missing from the environment, or a shell/teammate needs the token wired up by hand. <example>GH_TOKEN isn't set even though github-app is configured</example> <example>the github-app SessionStart hook didn't run, set up the token manually</example> <example>wire the github app token into CLAUDE_ENV_FILE</example> <example>gh keeps falling back to the wrong account, isolate GH_CONFIG_DIR</example>
3SKILL.mdUpdated Jun 9, 2026
nsheaps/github-app-git-identity
tools
VerifiedTrustedCommunity
Manually configure the GitHub App bot git identity the way the github-app plugin's SessionStart hook does — resolve the app slug and bot user ID, build the <slug>[bot] name and noreply email, set GIT_AUTHOR_*/GIT_COMMITTER_* env vars, and write an isolated GIT_CONFIG_GLOBAL with the gh auth git-credential helper. Use when commits are attributed to the wrong account, "Author identity unknown" appears, or git identity must be set up by hand. <example>my commits are showing up as the handler, not the bot</example> <example>git says Author identity unknown after the github-app hook ran</example> <example>configure the github app bot git identity manually</example> <example>set up the gh credential helper for git push</example>
3SKILL.mdUpdated Jun 9, 2026
nsheaps/spec-management
tools
VerifiedTrustedCommunity
Manages spec files for requirements capture and validation
3SKILL.mdUpdated Jun 7, 2026
nsheaps/plugins/bash-command-rejection/skills/bash-chaining-alternatives
tools
VerifiedTrustedCommunity
# Bash Chaining Alternatives This skill teaches you how to work around the bash command chaining restriction enforced by this plugin. ## Why Chaining is Blocked The `bash-command-rejection` plugin blocks these operators: | Operator | Name | Why Blocked | | -------- | ---------- | ----------------------------------------------------------------------------------- | | `&&` | AND chain | Runs cmd2 only if cmd1 su
3SKILL.mdUpdated Jun 7, 2026