review

When to invoke

Use when asked to "review this PR", "code review", "pre-landing review", or "check my diff".

Proactively suggest when the user is about to merge or land code changes.

Preamble

eval "$(~/.vibestack/bin/vibe-slug 2>/dev/null)" 2>/dev/null || SLUG="unknown"
_LEARN_FILE="${VIBESTACK_HOME:-$HOME/.vibestack}/projects/${SLUG:-unknown}/learnings.jsonl"
if [ -f "$_LEARN_FILE" ]; then
  _LEARN_COUNT=$(wc -l < "$_LEARN_FILE" 2>/dev/null | tr -d ' ')
  echo "LEARNINGS: $_LEARN_COUNT entries loaded"
  if [ "$_LEARN_COUNT" -gt 5 ] 2>/dev/null; then
    ~/.vibestack/bin/vibe-learnings-search --limit 5 2>/dev/null || true
  fi
else
  echo "LEARNINGS: none yet"
fi

{{include lib/snippets/session-host.md}}

{{include lib/snippets/decision-brief.md}}

{{include lib/snippets/working-protocols.md}}

{{include lib/snippets/state-protocols.md}}

Claimed Limitations Need Evidence

"Codex isn't authenticated", "the API can't do that", "that can't be checked from here" — each is a material claim, and this skill hands such claims to the user as review output. State one only with the verbatim error, the documented statement, or a live probe in hand. Pattern-matching a failure to a familiar story is a guess wearing a conclusion's clothes, and when the probe is one command away, run the command instead of asking the user to confirm your hunch.

Plan Mode Safe Operations

If the host is in plan mode when /review is invoked, these are allowed because they produce the review rather than change the user's work: reading any file, git fetch / git diff / git log, the specialist and adversarial subagents, the read-only Codex passes, and writes under ~/.vibestack/ (learnings, decisions, review log).

Step 5's Fix-First is not: no Edit or Write to repository files, no auto-fix. Report each finding with its recommended fix and let the user apply them once the plan is accepted.

Skill Invocation During Plan Mode

If the user invokes this skill during plan mode, the skill takes precedence — treat its steps as instructions to execute now, not as a plan to propose. Its own AskUserQuestion calls are part of the review, not a plan-mode violation: answering them is how the review reaches a verdict.