jasonwarrenuk/review-this-branch
skills/review-this-branch/SKILL.md
{{ 𝛀𝛀𝛀 }} Assess branch readiness for PR submission
$ install --global
skillsauthnpx skillsauth add jasonwarrenuk/goblin-mode review-this-branchInstall 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 6, 2026, 2:17 AM4.4s1 file scanned
SKILL.md
- name:
- review-this-branch
- description:
- {{ 𝛀𝛀𝛀 }} Assess branch readiness for PR submission
- model:
- opus
- disable-model-invocation:
- true
- allowed-tools:
- ["Read", "Glob", "Grep", "Bash(git:*)"]
Assess whether this branch is ready to be submitted as a reviewable pull request.
Analysis Steps
- Identify the base branch (usually
main) and list all commits since diverging - Review the full diff for:
- Incomplete work (half-implemented features, placeholder logic)
- Obvious bugs or unhandled edge cases
- Code quality issues that would block a reviewer
- Check commit history:
- Conventional commit compliance (
type(scope): description) - Logical atomicity — each commit should represent one coherent change
- No WIP, fixup, or junk commits that should have been cleaned up
- Conventional commit compliance (
- Check branch name against naming conventions:
- Format:
<prefix>/<short-description>(lowercase, hyphens, imperative) - Valid prefixes:
feat,fix,enhance,refactor,types,perf,styles,layout,docs,test,deps,config,build,agents,chore,ci,deploy,spike,experiment,wip,hotfix - Breaking change branches should use
<prefix>/breaking-<description>
- Format:
- Scan for anti-patterns:
- Debugging artefacts (
console.log, commented-out code) - Unresolved conflict markers (
<<<<<<<,=======,>>>>>>>) TODO/FIXMEcomments introduced in this branch- Diff size >500 lines (flag for possible split)
- Debugging artefacts (
- Detect breaking changes:
- Removed or renamed exports, functions, types, or components
- Changed function signatures (required parameters added/reordered)
- Modified return types in a widening direction
- Database schema changes (columns removed/renamed, constraints changed)
- HTTP API changes (routes, methods, request/response shapes)
- New required environment variables
- Changed component props (removed, renamed, type-changed, newly required)
Output Format
Verdict
One of: Ready | Needs Work | Blocked
Branch Health
- Branch name compliance
- Commit message quality
- Commit atomicity
Code Quality Flags
Specific issues found, with file and line references where possible. Omit this section if none found.
Breaking Changes
Any detected breaking changes with context. Flag format:
⚠️ Breaking change — consider
feat!:orBREAKING CHANGE:footer
Omit this section if none found.
Recommended Next Steps
Ordered list — blockers first, then improvements, then nice-to-haves.
If the verdict is Ready, offer to run /git:pull-request immediately.
Related Skills
jasonwarrenuk/writing-style
development
VerifiedTrustedCommunity
Writing style guide for Jason Warren. Use this skill whenever writing prose, reports, documentation, or any substantive text for Jason — including drafting sections, editing existing content, or rewriting passages. Also use when Jason asks you to review or improve writing. Trigger on any request involving writing, drafting, editing, or composing text that isn't purely code. This includes github Pull Requests & Linear tasks
4SKILL.mdUpdated Apr 6, 2026
jasonwarrenuk/wip-version
testing
VerifiedTrustedCommunity
{{ 𝚫𝚫𝚫 }} Check all version number props and update them
4SKILL.mdUpdated Apr 6, 2026
jasonwarrenuk/wip-roadmap
tools
VerifiedTrustedCommunity
{{ 𝛀𝛀𝛀 }} Map out project status, direction, and next steps
4SKILL.mdUpdated Apr 6, 2026
jasonwarrenuk/wip-pr-review
development
VerifiedTrustedCommunity
{{ 𝛀𝛀𝛀 }} Review code changes on the current branch against its open PR
4SKILL.mdUpdated Apr 6, 2026