pubky-app: Skill for Claude Code

Local Code Review

Review a change against the project's rules with three focused read-only reviewers, a deterministic callouts pass, and the project's own checks when they are available. Print one status line once the scope is known, then the report. Do not narrate the steps in between.

Step 1 — Determine the scope

Work out what to review without asking when the answer is already known:

Ask only when the table gives no answer. Status line example: Reviewing branch diff against dev (12 files)…

Step 2 — Collect the diff

Write the diff to a temp file and keep the changed-file list. Reviewers read the file; the diff is never pasted into their prompts. Noise files are excluded from every scope. Set SCOPE to local, branch, pr or paths from Step 1 and run only that branch of the case.

git fetch -q origin dev
EXCLUDE=(':!package-lock.json' ':!**/__screenshots__/**' ':!public/sw.js' ':!src/libs/lucide/lucideIcons.aliases.ts' ':!src/libs/lucide/lucideIcons.nodes.ts' ':!src/libs/lucide/lucideIcons.tags.ts')
OUT="${TMPDIR:-/tmp}/pubky-review-$(date +%s).diff"

case "$SCOPE" in
  local)   # tracked changes (staged + unstaged) plus untracked files
    git diff HEAD -- . "${EXCLUDE[@]}" > "$OUT"
    git ls-files --others --exclude-standard -- . "${EXCLUDE[@]}" | while read -r f; do git diff --no-index -- /dev/null "$f" >> "$OUT" || true; done   # --no-index exits 1 on a diff
    git status --porcelain -- . "${EXCLUDE[@]}"                       # changed-file list (?? = untracked)
    ;;
  branch)
    RANGE="origin/dev...HEAD"
    git diff "$RANGE" -- . "${EXCLUDE[@]}" > "$OUT"
    git diff --name-status "$RANGE" -- . "${EXCLUDE[@]}"              # changed-file list
    ;;
  pr)      # N = the PR number; works for same-repo and fork PRs
    BASE=$(gh pr view "$N" --json baseRefName --jq .baseRefName 2>/dev/null || echo dev)   # PRs target dev; gh may be absent
    git fetch -q origin "$BASE" "+pull/$N/head:refs/remotes/origin/pr-$N"   # + so a force-pushed PR head still updates
    RANGE="origin/$BASE...origin/pr-$N"
    git diff "$RANGE" -- . "${EXCLUDE[@]}" > "$OUT"
    git diff --name-status "$RANGE" -- . "${EXCLUDE[@]}"              # changed-file list
    ;;
  paths)   # PATHS = the files or directories named by the user; tracked changes plus untracked files under them
    git diff HEAD -- "${PATHS[@]}" "${EXCLUDE[@]}" > "$OUT"
    git ls-files --others --exclude-standard -- "${PATHS[@]}" "${EXCLUDE[@]}" | while read -r f; do git diff --no-index -- /dev/null "$f" >> "$OUT" || true; done
    git status --porcelain -- "${PATHS[@]}" "${EXCLUDE[@]}"           # changed-file list
    ;;
esac