lfx-self-serve: Skill for Claude Code

LFX Self-Serve PR Readiness

You are checking whether local commits are shaped correctly to open as a PR — branch name, ticket references (JIRA or GitHub Issue) in commit messages, conventional-commit format, rebase status, DCO + GPG signing on every commit, total diff size.

This skill does NOT audit code. CLAUDE.md's Pre-PR review section is the single owner of that protocol, and this check is a step inside its Mode 2. Before running it, confirm the work that precedes it in Mode 2 is complete: the whole-branch review returned a valid batch, its required findings are fixed, the documentation-currency updates are done, and all of that is in a signed/DCO commit against a clean tree. If any of that is outstanding, stop and finish it first.

The PR-shape checklist lives in references/pr-shape.md and is walked directly in this body.

Output: structured shape report with verdict NOT READY | READY WITH CHANGES | READY. No git mutations, no PR side-effects.

---

⚠ Mandatory: read the PR-shape checklist before any audit work

• .claude/skills/lfx-self-serve-pr-readiness/references/pr-shape.md — PR-shape sanity checklist with severity per item. Every finding must trace back to an item in this file. If you cannot quote the source, drop the finding.

---

Phase 1 — Parse arguments

Args format: [base-branch] [extra instructions].

• First token, if it looks like a ref (contains /, or is a plain branch name like main), is the base branch.
• Default base: origin/main.
• Everything else is extra focus (rarely used for shape; mostly for explanatory context).

Phase 2 — Gather PR-shape inputs

Normalize <base> first: if it contains no / (e.g., bare main), prefix with origin/ so the comparison runs against the freshly-fetched remote ref rather than a possibly-stale local branch.

Run in parallel:

git fetch origin
git rev-parse --abbrev-ref HEAD                                 # current branch
git diff --shortstat <base>...HEAD                              # additions/deletions (three-dot = merge-base..HEAD)
git diff --name-only <base>...HEAD                              # changed files (drives protected-files check)
git log --format='%H %s' <base>..HEAD                           # commit subjects on the branch
git log --format='%G? %h %s' <base>..HEAD                       # signature status per commit
git log --format=%B <base>..HEAD                                # commit bodies (Signed-off-by trailers)
git merge-base --is-ancestor <base> HEAD; echo $?               # rebase status (0 = ancestor)