sg-review

name	sg-review
description	Use this when implementation is complete and a code review is needed — derives the git range automatically and invokes superpowers:requesting-code-review.
argument-hint	[description of what was implemented]

Detect the user's input language and respond in that language throughout this skill's output. - Korean input → respond in Korean - English input → respond in English - Mixed input → match the dominant language Invoke the superpowers:requesting-code-review Skill with structured context: DESCRIPTION (from $ARGUMENTS or recent commit subject), BASE_SHA (merge-base with main), and HEAD_SHA (current HEAD). Ensures the Skill has enough context to dispatch a useful review subagent.

<execution_context> Self-contained. Reads git history to derive BASE_SHA and HEAD_SHA, then delegates to superpowers:requesting-code-review Skill. </execution_context>

1. **Derive git range.** ```bash HEAD_SHA=$(git rev-parse HEAD) BASE_SHA=$(git merge-base HEAD main 2>/dev/null \ || git merge-base HEAD master 2>/dev/null \ || git rev-parse HEAD~1 2>/dev/null \ || git rev-parse HEAD) echo "Initial range: $BASE_SHA..$HEAD_SHA" ```

1.5. Auto-commit gate when BASE==HEAD + dirty working tree (Phase 42/43 retro P1 #2 closure).

When BASE_SHA == HEAD_SHA and the working tree has uncommitted phase implementation, offer to auto-commit before exiting with the original error. This closes the recurring manual-commit step that surfaced in Phase 42 and Phase 43 after sg-parallel-execute (which intentionally does not commit per D-10/D-16 staging).

if [ "$BASE_SHA" = "$HEAD_SHA" ]; then
  WORKING_PORCELAIN=$(git status --porcelain 2>/dev/null)

  if [ -z "$WORKING_PORCELAIN" ]; then
    echo "Error: BASE_SHA == HEAD_SHA and working tree is clean — no commits to review."
    echo "Run from a feature branch that has diverged from main, or make new commits."
    exit 1
  fi

  # Identify current phase slug from HANDOFF.md last data row (Phase column)
  CURRENT_PHASE_SLUG=$(grep -E '^\| [0-9]{4}-' .planning/HANDOFF.md 2>/dev/null \
    | tail -1 | awk -F'|' '{gsub(/^[ \t]+|[ \t]+$/,"",$3); print $3}')
  PHASE_NUM_AUTO=$(printf '%s' "$CURRENT_PHASE_SLUG" | sed -E 's/^0*([0-9]+)-.*/\1/')
  PHASE_PAD_AUTO=$(printf "%02d" "$PHASE_NUM_AUTO" 2>/dev/null || echo "$PHASE_NUM_AUTO")

  # Candidate files: skills/, .agents/skills/, current phase SUMMARY + parallel_groups
  CANDIDATES=$(printf '%s\n' "$WORKING_PORCELAIN" \
    | awk '{sub(/^[^ ]+ +/, ""); print}' \
    | grep -E "^(skills/|\.agents/skills/|\.planning/phases/${PHASE_PAD_AUTO}-[^/]*/(.*SUMMARY|parallel_groups))" \
    || true)

  if [ -z "$CANDIDATES" ]; then
    echo "Error: BASE_SHA == HEAD_SHA — working tree has changes but none look like phase implementation."
    echo ""
    echo "Working tree state:"
    printf '%s\n' "$WORKING_PORCELAIN" | sed 's/^/  /'
    echo ""
    echo "Commit your changes manually before sg-review, or pass an explicit base SHA."
    exit 1
  fi

  echo ""
  echo "BASE_SHA == HEAD_SHA but working tree contains likely Phase ${PHASE_NUM_AUTO} implementation:"
  printf '%s\n' "$CANDIDATES" | sed 's/^/  /'
  echo ""
fi

If CANDIDATES is non-empty, invoke AskUserQuestion to confirm auto-commit:

AskUserQuestion(
  questions: [{
    question: "Auto-commit these files with derived message before review? (Phase 42/43 retro P1 #2 closure)",
    header: "auto-commit",
    multiSelect: false,
    options: [
      { label: "Auto-commit and proceed (Recommended)", description: "Stage candidates and commit with derived feat(NN) message, then recompute BASE_SHA=HEAD~1 and proceed with review." },
      { label: "Cancel", description: "Exit with original BASE==HEAD error. Commit manually then re-run sg-review." }
    ]
  }]
)

If "Auto-commit and proceed" selected:

# Stage candidates (one per line, preserve spaces in path)
while IFS= read -r f; do
  [ -z "$f" ] && continue
  git add "$f"
done <<EOF
$CANDIDATES
EOF

# Derive commit subject from phase slug: "43-pick-display-polish" → "43" + "pick display polish"
PHASE_SUBJECT_SLUG=$(printf '%s' "$CURRENT_PHASE_SLUG" | sed -E 's/^[0-9]+-//' | tr '-' ' ')
COMMIT_MSG=$(printf 'feat(%s): %s\n\nAuto-committed by sg-review (Phase 42/43 retro P1 #2 closure).\nWorking tree contained uncommitted Phase %s implementation; sg-parallel-execute completed without commit per D-10/D-16 staging.\n\nCo-Authored-By: Claude Opus 4.7 (1M context) <[e-mail]>\n' "$PHASE_PAD_AUTO" "$PHASE_SUBJECT_SLUG" "$PHASE_NUM_AUTO")
git commit -m "$COMMIT_MSG"

# Recompute SHAs after auto-commit
HEAD_SHA=$(git rev-parse HEAD)
BASE_SHA=$(git rev-parse HEAD~1)
echo "Auto-committed. New range: $BASE_SHA..$HEAD_SHA"

If "Cancel" selected:

echo "Cancelled. Commit your changes manually then re-run /super-gsd:sg-review."
exit 1

After this gate, control flow resumes with BASE_SHA != HEAD_SHA guaranteed.

echo "Reviewing: $BASE_SHA..$HEAD_SHA"

2. Determine description. Use $ARGUMENTS as the description of what was implemented. If $ARGUMENTS is empty, fall back to the most recent commit subject (no SHA prefix):

   if [ -n "$ARGUMENTS" ]; then
     DESCRIPTION="$ARGUMENTS"
   else
     DESCRIPTION=$(git log --format=%s -1)
     DESCRIPTION="${DESCRIPTION:-(no commit message found)}"
   fi
   

3. Read plan/requirements (best-effort). If a PLAN.md for the current phase exists, read its <objective> section as requirements context. Otherwise use a default string:

   Read .planning/STATE.md, then extract the Phase: value from the YAML frontmatter. Set PHASE_NUM.
   if [ -n "$PHASE_NUM" ]; then
     PHASE_PAD=$(printf "%02d" "$PHASE_NUM")
   else
     PHASE_PAD=""
   fi
   if [ -n "$PHASE_PAD" ]; then
     PLAN_FILE=$(ls .planning/phases/${PHASE_PAD}-*/*-PLAN.md 2>/dev/null | tail -1)
     PLAN_FILE=${PLAN_FILE:-$(ls .planning/milestones/v1.0-phases/${PHASE_PAD}-*/*-PLAN.md 2>/dev/null | tail -1)}
   else
     PLAN_FILE=""
   fi
   if [ -n "$PLAN_FILE" ]; then
     Read the PLAN_FILE path, then extract the text content between <objective> and </objective> tags. Set PLAN_REQUIREMENTS.
   else
     PLAN_REQUIREMENTS="(no plan file found — review current HEAD changes)"
   fi
   

3.9. Record review row in HANDOFF.md. superpowers:requesting-code-review is a terminal Skill so the moment immediately before invocation is the last possible point to record:

HANDOFF_FILE=".planning/HANDOFF.md"
if [ ! -f "$HANDOFF_FILE" ] || ! grep -q "Timestamp.*Phase.*From.*To.*Plan Hash" "$HANDOFF_FILE" 2>/dev/null; then
  mkdir -p "$(dirname "$HANDOFF_FILE")"
  printf '| Timestamp | Phase | From | To | Plan Hash | User |\n| --- | --- | --- | --- | --- | --- |\n' > "$HANDOFF_FILE"
fi
TS=$(date -u +%Y-%m-%dT%H:%M:%SZ)
Read .planning/STATE.md, then extract the Phase: value from the YAML frontmatter. Set PHASE_NUM_R.
PHASE_PAD_R=$(printf "%02d" "${PHASE_NUM_R:-0}" 2>/dev/null || echo "${PHASE_NUM_R:-0}")
PHASE_SLUG_R=$(ls -d .planning/phases/${PHASE_PAD_R}-* 2>/dev/null | head -1 | xargs basename 2>/dev/null)
[ -z "$PHASE_SLUG_R" ] && PHASE_SLUG_R="${PHASE_NUM_R:-unknown}"
Read .planning/HANDOFF.md, then extract the To column (5th pipe-delimited field) from the last row starting with "| " followed by a 4-digit year. Set FROM_STAGE_R (default "init" if empty).
GIT_USER=$(git config user.name 2>/dev/null || echo "-")
[ -z "$GIT_USER" ] && GIT_USER="-"
echo "| $TS | $PHASE_SLUG_R | $FROM_STAGE_R | review | - | $GIT_USER |" >> "$HANDOFF_FILE"

4. Invoke Skill with the structured context. Before calling Skill, substitute the actual resolved values for $DESCRIPTION, $PLAN_REQUIREMENTS, $BASE_SHA, and $HEAD_SHA captured in steps 1–3. Session control transfers to the skill; no steps execute after this point:

   Skill(skill="superpowers:requesting-code-review", args="## What Was Implemented
   

$DESCRIPTION

Requirements / Plan

$PLAN_REQUIREMENTS

Git Range

Base: $BASE_SHA Head: $HEAD_SHA")

</process>

<success_criteria>
1. superpowers:requesting-code-review Skill is invoked exactly once with a non-empty DESCRIPTION, a non-empty BASE_SHA, and a non-empty HEAD_SHA.
2. The git range reflects the current branch's changes since diverging from main.
3. When $ARGUMENTS is empty, DESCRIPTION falls back to the most recent git commit subject (no SHA prefix) rather than an empty string.
</success_criteria>