OpenSkillIndexโ† back to search
claude-code

Goblin-mode review-this-branch

{{ ๐›€๐›€๐›€ }} Assess branch readiness for PR submission

install
source ยท Clone the upstream repo
git clone https://github.com/JasonWarrenUK/goblin-mode
Claude Code ยท Install into ~/.claude/skills/
T=$(mktemp -d) && git clone --depth=1 https://github.com/JasonWarrenUK/goblin-mode "$T" && mkdir -p ~/.claude/skills && cp -r "$T/skills/review-this-branch" ~/.claude/skills/jasonwarrenuk-goblin-mode-review-this-branch && rm -rf "$T"
manifest: skills/review-this-branch/SKILL.md
source content

Assess whether this branch is ready to be submitted as a reviewable pull request.

Analysis Steps

  1. Identify the base branch (usually 
    main
    ) and list all commits since diverging
  2. 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
  3. 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
  4. 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>
  5. Scan for anti-patterns:
    • Debugging artefacts (
      console.log
      , commented-out code)
    • Unresolved conflict markers (
      <<<<<<<
      , 
      =======
      , 
      >>>>>>>
      )
    • TODO
      / 
      FIXME
      comments introduced in this branch
    • Diff size >500 lines (flag for possible split)
  6. 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!:
or 
BREAKING 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.