When to invoke this skill

Preamble (run first)

_UPD=$(~/.claude/skills/gstack/bin/gstack-update-check 2>/dev/null || .claude/skills/gstack/bin/gstack-update-check 2>/dev/null || true)
[ -n "$_UPD" ] && echo "$_UPD" || true
mkdir -p ~/.gstack/sessions
touch ~/.gstack/sessions/"$PPID"
_SESSIONS=$(find ~/.gstack/sessions -mmin -120 -type f 2>/dev/null | wc -l | tr -d ' ')
find ~/.gstack/sessions -mmin +120 -type f -exec rm {} + 2>/dev/null || true
_PROACTIVE=$(~/.claude/skills/gstack/bin/gstack-config get proactive 2>/dev/null || echo "true")
_PROACTIVE_PROMPTED=$([ -f ~/.gstack/.proactive-prompted ] && echo "yes" || echo "no")
_BRANCH=$(git branch --show-current 2>/dev/null || echo "unknown")
echo "BRANCH: $_BRANCH"
_SKILL_PREFIX=$(~/.claude/skills/gstack/bin/gstack-config get skill_prefix 2>/dev/null || echo "false")
echo "PROACTIVE: $_PROACTIVE"
echo "PROACTIVE_PROMPTED: $_PROACTIVE_PROMPTED"
echo "SKILL_PREFIX: $_SKILL_PREFIX"
source <(~/.claude/skills/gstack/bin/gstack-repo-mode 2>/dev/null) || true
REPO_MODE=${REPO_MODE:-unknown}
echo "REPO_MODE: $REPO_MODE"
_SESSION_KIND=$(~/.claude/skills/gstack/bin/gstack-session-kind 2>/dev/null || echo "interactive")
case "$_SESSION_KIND" in spawned|headless|interactive) ;; *) _SESSION_KIND="interactive" ;; esac
echo "SESSION_KIND: $_SESSION_KIND"
# Conductor host: AskUserQuestion is unreliable here (native disabled, MCP
# variant flaky), so skills render decisions as prose instead of calling the
# tool. Gated on !headless so an eval/CI run INSIDE Conductor (GSTACK_HEADLESS)
# still BLOCKs rather than rendering prose to nobody.
if [ "$_SESSION_KIND" != "headless" ] && { [ -n "${CONDUCTOR_WORKSPACE_PATH:-}" ] || [ -n "${CONDUCTOR_PORT:-}" ]; }; then
  echo "CONDUCTOR_SESSION: true"
fi
_ACTIVATED=$([ -f ~/.gstack/.activated ] && echo "yes" || echo "no")
_FIRST_LOOP_SHOWN=$([ -f ~/.gstack/.first-loop-tip-shown ] && echo "yes" || echo "no")
echo "ACTIVATED: $_ACTIVATED"
echo "FIRST_LOOP_SHOWN: $_FIRST_LOOP_SHOWN"
# First-run project detection: run the detector ONLY on the first-ever skill run
# (ACTIVATED=no, interactive) so it stays off the hot path for every run after.
_FIRST_TASK=""
if [ "$_ACTIVATED" = "no" ] && [ "$_SESSION_KIND" != "headless" ]; then
  _FIRST_TASK=$(~/.claude/skills/gstack/bin/gstack-first-task-detect 2>/dev/null || true)
fi
echo "FIRST_TASK: $_FIRST_TASK"
_LAKE_SEEN=$([ -f ~/.gstack/.completeness-intro-seen ] && echo "yes" || echo "no")
echo "LAKE_INTRO: $_LAKE_SEEN"
_TEL=$(~/.claude/skills/gstack/bin/gstack-config get telemetry 2>/dev/null || true)
_TEL_PROMPTED=$([ -f ~/.gstack/.telemetry-prompted ] && echo "yes" || echo "no")
_TEL_START=$(date +%s)
_SESSION_ID="$$-$(date +%s)"
echo "TELEMETRY: ${_TEL:-off}"
echo "TEL_PROMPTED: $_TEL_PROMPTED"
_EXPLAIN_LEVEL=$(~/.claude/skills/gstack/bin/gstack-config get explain_level 2>/dev/null || echo "default")
if [ "$_EXPLAIN_LEVEL" != "default" ] && [ "$_EXPLAIN_LEVEL" != "terse" ]; then _EXPLAIN_LEVEL="default"; fi
echo "EXPLAIN_LEVEL: $_EXPLAIN_LEVEL"
_QUESTION_TUNING=$(~/.claude/skills/gstack/bin/gstack-config get question_tuning 2>/dev/null || echo "false")
echo "QUESTION_TUNING: $_QUESTION_TUNING"
mkdir -p ~/.gstack/analytics
if [ "$_TEL" != "off" ]; then
echo '{"skill":"design-review","ts":"'$(date -u +%Y-%m-%dT%H:%M:%SZ)'","repo":"'$(_repo=$(basename "$(git rev-parse --show-toplevel 2>/dev/null)" 2>/dev/null | tr -cd 'a-zA-Z0-9._-'); echo "${_repo:-unknown}")'"}'  >> ~/.gstack/analytics/skill-usage.jsonl 2>/dev/null || true
fi
for _PF in $(find ~/.gstack/analytics -maxdepth 1 -name '.pending-*' 2>/dev/null); do
  if [ -f "$_PF" ]; then
    if [ "$_TEL" != "off" ] && [ -x "~/.claude/skills/gstack/bin/gstack-telemetry-log" ]; then
      ~/.claude/skills/gstack/bin/gstack-telemetry-log --event-type skill_run --skill _pending_finalize --outcome unknown --session-id "$_SESSION_ID" 2>/dev/null || true
    fi
    rm -f "$_PF" 2>/dev/null || true
  fi
  break
done
eval "$(~/.claude/skills/gstack/bin/gstack-slug 2>/dev/null)" 2>/dev/null || true
_LEARN_FILE="${GSTACK_HOME:-$HOME/.gstack}/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
    ~/.claude/skills/gstack/bin/gstack-learnings-search --limit 3 2>/dev/null || true
  fi
else
  echo "LEARNINGS: 0"
fi
~/.claude/skills/gstack/bin/gstack-timeline-log '{"skill":"design-review","event":"started","branch":"'"$_BRANCH"'","session":"'"$_SESSION_ID"'"}' 2>/dev/null &
_HAS_ROUTING="no"
if [ -f CLAUDE.md ] && grep -q "## Skill routing" CLAUDE.md 2>/dev/null; then
  _HAS_ROUTING="yes"
fi
_ROUTING_DECLINED=$(~/.claude/skills/gstack/bin/gstack-config get routing_declined 2>/dev/null || echo "false")
echo "HAS_ROUTING: $_HAS_ROUTING"
echo "ROUTING_DECLINED: $_ROUTING_DECLINED"
_VENDORED="no"
if [ -d ".claude/skills/gstack" ] && [ ! -L ".claude/skills/gstack" ]; then
  if [ -f ".claude/skills/gstack/VERSION" ] || [ -d ".claude/skills/gstack/.git" ]; then
    _VENDORED="yes"
  fi
fi
echo "VENDORED_GSTACK: $_VENDORED"
echo "MODEL_OVERLAY: claude"
_CHECKPOINT_MODE=$(~/.claude/skills/gstack/bin/gstack-config get checkpoint_mode 2>/dev/null || echo "explicit")
_CHECKPOINT_PUSH=$(~/.claude/skills/gstack/bin/gstack-config get checkpoint_push 2>/dev/null || echo "false")
echo "CHECKPOINT_MODE: $_CHECKPOINT_MODE"
echo "CHECKPOINT_PUSH: $_CHECKPOINT_PUSH"
# Plan-mode hint for skills like /spec that branch behavior on plan-mode state.
# Claude Code exposes plan mode via system reminders; we detect best-effort
# from CLAUDE_PLAN_FILE (set by the harness when plan mode is active) and
# fall back to "inactive". Codex hosts and Claude execution mode both end up
# inactive, which is the safe default (defaults to file+execute pipeline).
if [ -n "${CLAUDE_PLAN_FILE:-}${GSTACK_PLAN_MODE_FORCE:-}" ]; then
  export GSTACK_PLAN_MODE="active"
elif [ "${GSTACK_PLAN_MODE:-}" = "active" ]; then
  export GSTACK_PLAN_MODE="active"
else
  export GSTACK_PLAN_MODE="inactive"
fi
echo "GSTACK_PLAN_MODE: $GSTACK_PLAN_MODE"
[ -n "$OPENCLAW_SESSION" ] && echo "SPAWNED_SESSION: true" || true

Plan Mode Safe Operations

Skill Invocation During Plan Mode

• Missing ~/.claude/skills/gstack/.feature-prompted-continuous-checkpoint: AskUserQuestion for Continuous checkpoint auto-commits. If accepted, run ~/.claude/skills/gstack/bin/gstack-config set checkpoint_mode continuous. Always touch marker.
• Missing ~/.claude/skills/gstack/.feature-prompted-model-overlay: inform "Model overlays are active. MODEL_OVERLAY shows the patch." Always touch marker.

v1 prompts are simpler: first-use jargon glosses, outcome-framed questions, shorter prose. Keep default or restore terse?

• A) Keep the new default (recommended — good writing helps everyone)
• B) Restore V0 prose — set explain_level: terse

rm -f ~/.gstack/.writing-style-prompt-pending
touch ~/.gstack/.writing-style-prompted

open https://garryslist.org/posts/boil-the-ocean
touch ~/.gstack/.completeness-intro-seen

Help gstack get better. Share usage data only: skill, duration, crashes, stable device ID. No code or file paths. Your repo name is recorded locally only and stripped before any upload.

• A) Help gstack get better! (recommended)
• B) No thanks

Anonymous mode sends only aggregate usage, no unique ID.

• A) Sure, anonymous is fine
• B) No thanks, fully off

touch ~/.gstack/.telemetry-prompted

Let gstack proactively suggest skills, like /qa for "does this work?" or /investigate for bugs?

• A) Keep it on (recommended)
• B) Turn it off — I'll type /commands myself

touch ~/.gstack/.proactive-prompted

First-run guidance (one-time)

~/.claude/skills/gstack/bin/gstack-telemetry-log --event-type first_task_scaffold_shown --skill "TASK_TOKEN" --outcome shown 2>/dev/null || true
touch ~/.gstack/.activated 2>/dev/null || true

Tip: gstack pays off when you complete one loop — plan → review → ship. A common first loop: /office-hours or /spec to shape it, /plan-eng-review to lock it, then /ship.

gstack works best when your project's CLAUDE.md includes skill routing rules.

• A) Add routing rules to CLAUDE.md (recommended)
• B) No thanks, I'll invoke skills manually

## Skill routing

When the user's request matches an available skill, invoke it via the Skill tool. When in doubt, invoke the skill.

Key routing rules:
- Product ideas/brainstorming → invoke /office-hours
- Strategy/scope → invoke /plan-ceo-review
- Architecture → invoke /plan-eng-review
- Design system/plan review → invoke /design-consultation or /plan-design-review
- Full review pipeline → invoke /autoplan
- Bugs/errors → invoke /investigate
- QA/testing site behavior → invoke /qa or /qa-only
- Code review/diff check → invoke /review
- Visual polish → invoke /design-review
- Ship/deploy/PR → invoke /ship or /land-and-deploy
- Save progress → invoke /context-save
- Resume context → invoke /context-restore
- Author a backlog-ready spec/issue → invoke /spec

This project has gstack vendored in .claude/skills/gstack/. Vendoring is deprecated. Migrate to team mode?

• A) Yes, migrate to team mode now
• B) No, I'll handle it myself

1. Run git rm -r .claude/skills/gstack/
2. Run echo '.claude/skills/gstack/' >> .gitignore
3. Run ~/.claude/skills/gstack/bin/gstack-team-init required (or optional)
4. Run git add .claude/ .gitignore CLAUDE.md && git commit -m "chore: migrate gstack from vendored to team mode"
5. Tell the user: "Done. Each developer now runs: cd ~/.claude/skills/gstack && ./setup --team"

eval "$(~/.claude/skills/gstack/bin/gstack-slug 2>/dev/null)" 2>/dev/null || true
touch ~/.gstack/.vendoring-warned-${SLUG:-unknown}

• Do NOT use AskUserQuestion for interactive prompts. Auto-choose the recommended option.
• Do NOT run upgrade checks, telemetry prompts, routing injection, or lake intro.
• Focus on completing the task and reporting results via prose output.
• End with a completion report: what shipped, decisions made, anything uncertain.

AskUserQuestion Format

Tool resolution (read first)

When AskUserQuestion is unavailable or a call fails

1. Auto-decide denial (NOT a failure). The result contains [plan-tune auto-decide] <id> → <option> — the preference hook working as designed. Proceed with that option. Do NOT retry, do NOT fall back to prose.
2. Genuine failure — no variant in your tool list, OR the variant is present but the call returns an error / missing result (MCP transport error, empty result, host bug — e.g. Conductor's MCP AskUserQuestion is flaky and returns [Tool result missing due to internal error]).
   • If it was present and errored (not absent), retry the SAME call once — but only if no answer could have surfaced (a missing-result error can arrive after the user already saw the question; retrying would double-prompt, so if it may have reached them, treat as pending, don't retry).
   • Then branch on SESSION_KIND (echoed by the preamble; empty/absent ⇒ interactive):
     • spawned → defer to the Spawned session block: auto-choose the recommended option. Never prose, never BLOCKED.
     • headless → BLOCKED — AskUserQuestion unavailable; stop and wait (no human can answer).
     • interactive → prose fallback (below).

1. A clear ELI10 of the issue itself — plain English on what's being decided and why it matters (the question, not per-choice), naming the stakes. Lead with it.
2. Completeness scores per choice — explicit Completeness: X/10 on EACH choice (10 complete, 7 happy-path, 3 shortcut); use the kind-note when options differ in kind not coverage, but never silently drop the score.
3. The recommendation and why — a Recommendation: <choice> because <reason> line plus the (recommended) marker on that choice.

Format

D<N> — <one-line question title>
Project/branch/task: <1 short grounding sentence using _BRANCH>
ELI10: <plain English a 16-year-old could follow, 2-4 sentences, name the stakes>
Stakes if we pick wrong: <one sentence on what breaks, what user sees, what's lost>
Recommendation: <choice> because <one-line reason>
Completeness: A=X/10, B=Y/10   (or: Note: options differ in kind, not coverage — no completeness score)
Pros / cons:
A) <option label> (recommended)
  ✅ <pro — concrete, observable, ≥40 chars>
  ❌ <con — honest, ≥40 chars>
B) <option label>
  ✅ <pro>
  ❌ <con>
Net: <one-line synthesis of what you're actually trading off>

Handling 5+ options — split, never drop

• Batch into ≤4-groups — for coherent alternatives (e.g. version bumps, layout variants). One call, 5th surfaced only if first 4 don't fit.
• Split per-option — for independent scope items (e.g. "ship E1..E6?"). Fire N sequential calls, one per option. Default to this when unsure.

Self-check before emitting

• D header present
• ELI10 paragraph present (stakes line too)
• Recommendation line present with concrete reason
• Completeness scored (coverage) OR kind-note present (kind)
• Every option has ≥2 ✅ and ≥1 ❌, each ≥40 chars (or hard-stop escape)
• (recommended) label on one option (even for neutral-posture)
• Dual-scale effort labels on effort-bearing options (human / CC)
• Net line closes the decision
• You are calling the tool, not writing prose — unless CONDUCTOR_SESSION: true (then prose is the DEFAULT, not the tool) OR the documented failure fallback applies (then: prose with the mandatory triad — issue ELI10, per-choice Completeness, Recommendation + (recommended) — and a "reply with a letter" instruction, then STOP)
• Non-ASCII characters (CJK / accents) written directly, NOT \u-escaped
• If you had 5+ options, you split (or batched into ≤4-groups) — did NOT drop any
• If you split, you checked dependencies between options before firing the chain
• If a per-option Hold fires, you stopped the chain immediately (didn't queue)

Artifacts Sync (skill start)

_GSTACK_HOME="${GSTACK_HOME:-$HOME/.gstack}"
# Prefer the v1.27.0.0 artifacts file; fall back to brain file for users
# upgrading mid-stream before the migration script runs.
if [ -f "$HOME/.gstack-artifacts-remote.txt" ]; then
  _BRAIN_REMOTE_FILE="$HOME/.gstack-artifacts-remote.txt"
else
  _BRAIN_REMOTE_FILE="$HOME/.gstack-brain-remote.txt"
fi
_BRAIN_SYNC_BIN="~/.claude/skills/gstack/bin/gstack-brain-sync"
_BRAIN_CONFIG_BIN="~/.claude/skills/gstack/bin/gstack-config"

# /sync-gbrain context-load: teach the agent to use gbrain when it's available.
# Per-worktree pin: post-spike redesign uses kubectl-style `.gbrain-source` in the
# git toplevel to scope queries. Look for the pin in the worktree (not a global
# state file) so that opening worktree B without a pin doesn't claim "indexed"
# just because worktree A was synced. Empty string when gbrain is not
# configured (zero context cost for non-gbrain users).
_GBRAIN_CONFIG="$HOME/.gbrain/config.json"
if [ -f "$_GBRAIN_CONFIG" ] && command -v gbrain >/dev/null 2>&1; then
  _GBRAIN_VERSION_OK=$(gbrain --version 2>/dev/null | grep -c '^gbrain ' || echo 0)
  if [ "$_GBRAIN_VERSION_OK" -gt 0 ] 2>/dev/null; then
    _GBRAIN_PIN_PATH=""
    _REPO_TOP=$(git rev-parse --show-toplevel 2>/dev/null || echo "")
    if [ -n "$_REPO_TOP" ] && [ -f "$_REPO_TOP/.gbrain-source" ]; then
      _GBRAIN_PIN_PATH="$_REPO_TOP/.gbrain-source"
    fi
    if [ -n "$_GBRAIN_PIN_PATH" ]; then
      echo "GBrain configured. Prefer \`gbrain search\`/\`gbrain query\` over Grep for"
      echo "semantic questions; use \`gbrain code-def\`/\`code-refs\`/\`code-callers\` for"
      echo "symbol-aware code lookup. See \"## GBrain Search Guidance\" in CLAUDE.md."
      echo "Run /sync-gbrain to refresh."
    else
      echo "GBrain configured but this worktree isn't pinned yet. Run \`/sync-gbrain --full\`"
      echo "before relying on \`gbrain search\` for code questions in this worktree."
      echo "Falls back to Grep until pinned."
    fi
  fi
fi

_BRAIN_SYNC_MODE=$("$_BRAIN_CONFIG_BIN" get artifacts_sync_mode 2>/dev/null || echo off)

# Detect remote-MCP mode (Path 4 of /setup-gbrain). Local artifacts sync is
# a no-op in remote mode; the brain server pulls from GitHub/GitLab on its
# own cadence. Read claude.json directly to keep this preamble fast (no
# subprocess to claude CLI on every skill start).
_GBRAIN_MCP_MODE="none"
if command -v jq >/dev/null 2>&1 && [ -f "$HOME/.claude.json" ]; then
  _GBRAIN_MCP_TYPE=$(jq -r '.mcpServers.gbrain.type // .mcpServers.gbrain.transport // empty' "$HOME/.claude.json" 2>/dev/null)
  case "$_GBRAIN_MCP_TYPE" in
    url|http|sse) _GBRAIN_MCP_MODE="remote-http" ;;
    stdio) _GBRAIN_MCP_MODE="local-stdio" ;;
  esac
fi

if [ -f "$_BRAIN_REMOTE_FILE" ] && [ ! -d "$_GSTACK_HOME/.git" ] && [ "$_BRAIN_SYNC_MODE" = "off" ]; then
  _BRAIN_NEW_URL=$(head -1 "$_BRAIN_REMOTE_FILE" 2>/dev/null | tr -d '[:space:]')
  if [ -n "$_BRAIN_NEW_URL" ]; then
    echo "ARTIFACTS_SYNC: artifacts repo detected: $_BRAIN_NEW_URL"
    echo "ARTIFACTS_SYNC: run 'gstack-brain-restore' to pull your cross-machine artifacts (or 'gstack-config set artifacts_sync_mode off' to dismiss forever)"
  fi
fi

if [ -d "$_GSTACK_HOME/.git" ] && [ "$_BRAIN_SYNC_MODE" != "off" ]; then
  _BRAIN_LAST_PULL_FILE="$_GSTACK_HOME/.brain-last-pull"
  _BRAIN_NOW=$(date +%s)
  _BRAIN_DO_PULL=1
  if [ -f "$_BRAIN_LAST_PULL_FILE" ]; then
    _BRAIN_LAST=$(cat "$_BRAIN_LAST_PULL_FILE" 2>/dev/null || echo 0)
    _BRAIN_AGE=$(( _BRAIN_NOW - _BRAIN_LAST ))
    [ "$_BRAIN_AGE" -lt 86400 ] && _BRAIN_DO_PULL=0
  fi
  if [ "$_BRAIN_DO_PULL" = "1" ]; then
    ( cd "$_GSTACK_HOME" && git fetch origin >/dev/null 2>&1 && git merge --ff-only "origin/$(git rev-parse --abbrev-ref HEAD)" >/dev/null 2>&1 ) || true
    echo "$_BRAIN_NOW" > "$_BRAIN_LAST_PULL_FILE"
  fi
  "$_BRAIN_SYNC_BIN" --once 2>/dev/null || true
fi

if [ "$_GBRAIN_MCP_MODE" = "remote-http" ]; then
  # Remote-MCP mode: local artifacts sync is a no-op (brain admin's server
  # pulls from GitHub/GitLab). Show the user this is by design, not broken.
  _GBRAIN_HOST=$(jq -r '.mcpServers.gbrain.url // empty' "$HOME/.claude.json" 2>/dev/null | sed -E 's|^https?://([^/:]+).*|\1|')
  echo "ARTIFACTS_SYNC: remote-mode (managed by brain server ${_GBRAIN_HOST:-remote})"
elif [ -d "$_GSTACK_HOME/.git" ] && [ "$_BRAIN_SYNC_MODE" != "off" ]; then
  _BRAIN_QUEUE_DEPTH=0
  [ -f "$_GSTACK_HOME/.brain-queue.jsonl" ] && _BRAIN_QUEUE_DEPTH=$(wc -l < "$_GSTACK_HOME/.brain-queue.jsonl" | tr -d ' ')
  _BRAIN_LAST_PUSH="never"
  [ -f "$_GSTACK_HOME/.brain-last-push" ] && _BRAIN_LAST_PUSH=$(cat "$_GSTACK_HOME/.brain-last-push" 2>/dev/null || echo never)
  echo "ARTIFACTS_SYNC: mode=$_BRAIN_SYNC_MODE | last_push=$_BRAIN_LAST_PUSH | queue=$_BRAIN_QUEUE_DEPTH"
else
  echo "ARTIFACTS_SYNC: off"
fi

gstack can publish your artifacts (CEO plans, designs, reports) to a private GitHub repo that GBrain indexes across machines. How much should sync?

• A) Everything allowlisted (recommended)
• B) Only artifacts
• C) Decline, keep everything local

# Chosen mode: full | artifacts-only | off
"$_BRAIN_CONFIG_BIN" set artifacts_sync_mode <choice>
"$_BRAIN_CONFIG_BIN" set artifacts_sync_mode_prompted true

"~/.claude/skills/gstack/bin/gstack-brain-sync" --discover-new 2>/dev/null || true
"~/.claude/skills/gstack/bin/gstack-brain-sync" --once 2>/dev/null || true

Model-Specific Behavioral Patch (claude)

Voice

• Lead with the point. Say what it does, why it matters, and what changes for the builder.
• Be concrete. Name files, functions, line numbers, commands, outputs, evals, and real numbers.
• Tie technical choices to user outcomes: what the real user sees, loses, waits for, or can now do.
• Be direct about quality. Bugs matter. Edge cases matter. Fix the whole thing, not the demo path.
• Sound like a builder talking to a builder, not a consultant presenting to a client.
• Never corporate, academic, PR, or hype. Avoid filler, throat-clearing, generic optimism, and founder cosplay.
• No em dashes. No AI vocabulary: delve, crucial, robust, comprehensive, nuanced, multifaceted, furthermore, moreover, additionally, pivotal, landscape, tapestry, underscore, foster, showcase, intricate, vibrant, fundamental, significant.
• The user has context you do not: domain knowledge, timing, relationships, taste. Cross-model agreement is a recommendation, not a decision. The user decides.

Context Recovery

eval "$(~/.claude/skills/gstack/bin/gstack-slug 2>/dev/null)"
_PROJ="${GSTACK_HOME:-$HOME/.gstack}/projects/${SLUG:-unknown}"
if [ -d "$_PROJ" ]; then
  echo "--- RECENT ARTIFACTS ---"
  find "$_PROJ/ceo-plans" "$_PROJ/checkpoints" -type f -name "*.md" 2>/dev/null | xargs ls -t 2>/dev/null | head -3
  [ -f "$_PROJ/${_BRANCH}-reviews.jsonl" ] && echo "REVIEWS: $(wc -l < "$_PROJ/${_BRANCH}-reviews.jsonl" | tr -d ' ') entries"
  [ -f "$_PROJ/timeline.jsonl" ] && tail -5 "$_PROJ/timeline.jsonl"
  if [ -f "$_PROJ/timeline.jsonl" ]; then
    _LAST=$(grep "\"branch\":\"${_BRANCH}\"" "$_PROJ/timeline.jsonl" 2>/dev/null | grep '"event":"completed"' | tail -1)
    [ -n "$_LAST" ] && echo "LAST_SESSION: $_LAST"
    _RECENT_SKILLS=$(grep "\"branch\":\"${_BRANCH}\"" "$_PROJ/timeline.jsonl" 2>/dev/null | grep '"event":"completed"' | tail -3 | grep -o '"skill":"[^"]*"' | sed 's/"skill":"//;s/"//' | tr '\n' ',')
    [ -n "$_RECENT_SKILLS" ] && echo "RECENT_PATTERN: $_RECENT_SKILLS"
  fi
  _LATEST_CP=$(find "$_PROJ/checkpoints" -name "*.md" -type f 2>/dev/null | xargs ls -t 2>/dev/null | head -1)
  [ -n "$_LATEST_CP" ] && echo "LATEST_CHECKPOINT: $_LATEST_CP"
  if [ -f "$_PROJ/decisions.active.json" ]; then
    echo "--- ACTIVE DECISIONS (recent, scope-relevant) ---"
    ~/.claude/skills/gstack/bin/gstack-decision-search --recent 5 2>/dev/null
    echo "--- END DECISIONS ---"
  fi
  echo "--- END ARTIFACTS ---"
fi

Writing Style (skip entirely if EXPLAIN_LEVEL: terse appears in the preamble echo OR the user's current message explicitly requests terse / no-explanations output)

• Gloss curated jargon on first use per skill invocation, even if the user pasted the term.
• Frame questions in outcome terms: what pain is avoided, what capability unlocks, what user experience changes.
• Use short sentences, concrete nouns, active voice.
• Close decisions with user impact: what the user sees, waits for, loses, or gains.
• User-turn override wins: if the current message asks for terse / no explanations / just the answer, skip this section.
• Terse mode (EXPLAIN_LEVEL: terse): no glosses, no outcome-framing layer, shorter responses.

Completeness Principle — Boil the Ocean

Confusion Protocol

Continuous Checkpoint Mode

WIP: <concise description of what changed>

[gstack-context]
Decisions: <key choices made this step>
Remaining: <what's left in the logical unit>
Tried: <failed approaches worth recording> (omit if none)
Skill: </skill-name-if-running>
[/gstack-context]

Context Health (soft directive)

Question Tuning (skip entirely if QUESTION_TUNING: false)

~/.claude/skills/gstack/bin/gstack-question-log '{"skill":"design-review","question_id":"<id>","question_summary":"<short>","category":"<approval|clarification|routing|cherry-pick|feedback-loop>","door_type":"<one-way|two-way>","options_count":N,"user_choice":"<key>","recommended":"<key>","session_id":"'"$_SESSION_ID"'"}' 2>/dev/null || true

~/.claude/skills/gstack/bin/gstack-question-preference --write '{"question_id":"<id>","preference":"<pref>","source":"inline-user","free_text":"<optional original words>"}'

Repo Ownership — See Something, Say Something

• solo — You own everything. Investigate and offer to fix proactively.
• collaborative / unknown — Flag via AskUserQuestion, don't fix (may be someone else's).

Search Before Building

• Layer 1 (tried and true) — don't reinvent. Layer 2 (new and popular) — scrutinize. Layer 3 (first principles) — prize above all.

jq -n --arg ts "$(date -u +%Y-%m-%dT%H:%M:%SZ)" --arg skill "SKILL_NAME" --arg branch "$(git branch --show-current 2>/dev/null)" --arg insight "ONE_LINE_SUMMARY" '{ts:$ts,skill:$skill,branch:$branch,insight:$insight}' >> ~/.gstack/analytics/eureka.jsonl 2>/dev/null || true

Completion Status Protocol

• DONE — completed with evidence.
• DONE_WITH_CONCERNS — completed, but list concerns.
• BLOCKED — cannot proceed; state blocker and what was tried.
• NEEDS_CONTEXT — missing info; state exactly what is needed.

Operational Self-Improvement

~/.claude/skills/gstack/bin/gstack-learnings-log '{"skill":"SKILL_NAME","type":"operational","key":"SHORT_KEY","insight":"DESCRIPTION","confidence":N,"source":"observed"}'

Telemetry (run last)

_TEL_END=$(date +%s)
_TEL_DUR=$(( _TEL_END - _TEL_START ))
rm -f ~/.gstack/analytics/.pending-"$_SESSION_ID" 2>/dev/null || true
# Session timeline: record skill completion (local-only, never sent anywhere)
~/.claude/skills/gstack/bin/gstack-timeline-log '{"skill":"SKILL_NAME","event":"completed","branch":"'$(git branch --show-current 2>/dev/null || echo unknown)'","outcome":"OUTCOME","duration_s":"'"$_TEL_DUR"'","session":"'"$_SESSION_ID"'"}' 2>/dev/null || true
# Local analytics (gated on telemetry setting)
if [ "$_TEL" != "off" ]; then
echo '{"skill":"SKILL_NAME","duration_s":"'"$_TEL_DUR"'","outcome":"OUTCOME","browse":"USED_BROWSE","session":"'"$_SESSION_ID"'","ts":"'$(date -u +%Y-%m-%dT%H:%M:%SZ)'"}' >> ~/.gstack/analytics/skill-usage.jsonl 2>/dev/null || true
fi
# Remote telemetry (opt-in, requires binary)
if [ "$_TEL" != "off" ] && [ -x ~/.claude/skills/gstack/bin/gstack-telemetry-log ]; then
  ~/.claude/skills/gstack/bin/gstack-telemetry-log \
    --skill "SKILL_NAME" --duration "$_TEL_DUR" --outcome "OUTCOME" \
    --used-browse "USED_BROWSE" --session-id "$_SESSION_ID" 2>/dev/null &
fi

Plan Status Footer

/design-review: Design Audit → Fix → Verify

Setup

$B status 2>/dev/null | grep -q "Mode: cdp" && echo "CDP_MODE=true" || echo "CDP_MODE=false"

git status --porcelain

• A) Commit my changes — commit all current changes with a descriptive message, then start design review
• B) Stash my changes — stash, run design review, pop the stash after
• C) Abort — I'll clean up manually

SETUP (run this check BEFORE any browse command)

_ROOT=$(git rev-parse --show-toplevel 2>/dev/null)
B=""
[ -n "$_ROOT" ] && [ -x "$_ROOT/.claude/skills/gstack/browse/dist/browse" ] && B="$_ROOT/.claude/skills/gstack/browse/dist/browse"
[ -z "$B" ] && B="$HOME/.claude/skills/gstack/browse/dist/browse"
if [ -x "$B" ]; then
  echo "READY: $B"
else
  echo "NEEDS_SETUP"
fi

1. Tell the user: "gstack browse needs a one-time build (~10 seconds). OK to proceed?" Then STOP and wait.
2. Run: cd <SKILL_DIR> && ./setup
3. If bun is not installed:
   bashCopy

   if ! command -v bun >/dev/null 2>&1; then
     BUN_VERSION="1.3.10"
     BUN_INSTALL_SHA="bab8acfb046aac8c72407bdcce903957665d655d7acaa3e11c7c4616beae68dd"
     tmpfile=$(mktemp)
     curl -fsSL "https://bun.sh/install" -o "$tmpfile"
     actual_sha=$(shasum -a 256 "$tmpfile" | awk '{print \$1}')
     if [ "$actual_sha" != "$BUN_INSTALL_SHA" ]; then
       echo "ERROR: bun install script checksum mismatch" >&2
       echo "  expected: $BUN_INSTALL_SHA" >&2
       echo "  got:      $actual_sha" >&2
       rm "$tmpfile"; exit 1
     fi
     BUN_VERSION="$BUN_VERSION" bash "$tmpfile"
     rm "$tmpfile"
   fi

Test Framework Bootstrap

setopt +o nomatch 2>/dev/null || true  # zsh compat
# Detect project runtime
[ -f Gemfile ] && echo "RUNTIME:ruby"
[ -f package.json ] && echo "RUNTIME:node"
[ -f requirements.txt ] || [ -f pyproject.toml ] && echo "RUNTIME:python"
[ -f go.mod ] && echo "RUNTIME:go"
[ -f Cargo.toml ] && echo "RUNTIME:rust"
[ -f composer.json ] && echo "RUNTIME:php"
[ -f mix.exs ] && echo "RUNTIME:elixir"
# Detect sub-frameworks
[ -f Gemfile ] && grep -q "rails" Gemfile 2>/dev/null && echo "FRAMEWORK:rails"
[ -f package.json ] && grep -q '"next"' package.json 2>/dev/null && echo "FRAMEWORK:nextjs"
# Check for existing test infrastructure
ls jest.config.* vitest.config.* playwright.config.* .rspec pytest.ini pyproject.toml phpunit.xml 2>/dev/null
ls -d test/ tests/ spec/ __tests__/ cypress/ e2e/ 2>/dev/null
# Check opt-out marker
[ -f .gstack/no-test-bootstrap ] && echo "BOOTSTRAP_DECLINED"

B2. Research best practices

• "[runtime] best test framework 2025 2026"
• "[framework A] vs [framework B] comparison"

B3. Framework selection

B4. Install and configure

1. Install the chosen packages (npm/bun/gem/pip/etc.)
2. Create minimal config file
3. Create directory structure (test/, spec/, etc.)
4. Create one example test matching the project's code to verify setup works

B4.5. First real tests

1. Find recently changed files: git log --since=30.days --name-only --format="" | sort | uniq -c | sort -rn | head -10
2. Prioritize by risk: Error handlers > business logic with conditionals > API endpoints > pure functions
3. For each file: Write one test that tests real behavior with meaningful assertions. Never expect(x).toBeDefined() — test what the code DOES.
4. Run each test. Passes → keep. Fails → fix once. Still fails → delete silently.
5. Generate at least 1 test, cap at 5.

B5. Verify

# Run the full test suite to confirm everything works
{detected test command}

B5.5. CI/CD pipeline

# Check CI provider
ls -d .github/ 2>/dev/null && echo "CI:github"
ls .gitlab-ci.yml .circleci/ bitrise.yml 2>/dev/null

• runs-on: ubuntu-latest
• Appropriate setup action for the runtime (setup-node, setup-ruby, setup-python, etc.)
• The same test command verified in B5
• Trigger: push + pull_request

B6. Create TESTING.md

• Philosophy: "100% test coverage is the key to great vibe coding. Tests let you move fast, trust your instincts, and ship with confidence — without them, vibe coding is just yolo coding. With tests, it's a superpower."
• Framework name and version
• How to run tests (the verified command from B5)
• Test layers: Unit tests (what, where, when), Integration tests, Smoke tests, E2E tests
• Conventions: file naming, assertion style, setup/teardown patterns

B7. Update CLAUDE.md

• Run command and test directory
• Reference to TESTING.md
• Test expectations:
  • 100% test coverage is the goal — tests make vibe coding safe
  • When writing new functions, write a corresponding test
  • When fixing a bug, write a regression test
  • When adding error handling, write a test that triggers the error
  • When adding a conditional (if/else, switch), write tests for BOTH paths
  • Never commit code that makes existing tests fail

B8. Commit

git status --porcelain

DESIGN SETUP (run this check BEFORE any design mockup command)

_ROOT=$(git rev-parse --show-toplevel 2>/dev/null)
D=""
[ -n "$_ROOT" ] && [ -x "$_ROOT/.claude/skills/gstack/design/dist/design" ] && D="$_ROOT/.claude/skills/gstack/design/dist/design"
[ -z "$D" ] && D="$HOME/.claude/skills/gstack/design/dist/design"
if [ -x "$D" ]; then
  echo "DESIGN_READY: $D"
else
  echo "DESIGN_NOT_AVAILABLE"
fi
B=""
[ -n "$_ROOT" ] && [ -x "$_ROOT/.claude/skills/gstack/browse/dist/browse" ] && B="$_ROOT/.claude/skills/gstack/browse/dist/browse"
[ -z "$B" ] && B="$HOME/.claude/skills/gstack/browse/dist/browse"
if [ -x "$B" ]; then
  echo "BROWSE_READY: $B"
else
  echo "BROWSE_NOT_AVAILABLE (will use 'open' to view comparison boards)"
fi

• $D generate --brief "..." --output /path.png — generate a single mockup
• $D variants --brief "..." --count 3 --output-dir /path/ — generate N style variants
• $D compare --images "a.png,b.png,c.png" --output /path/board.html --serve — comparison board + HTTP server
• $D serve --html /path/board.html — serve comparison board and collect feedback via HTTP
• $D check --image /path.png --brief "..." — vision quality gate
• $D iterate --session /path/session.json --feedback "..." --output /path.png — iterate

eval "$(~/.claude/skills/gstack/bin/gstack-slug 2>/dev/null)"
REPORT_DIR="$HOME/.gstack/projects/$SLUG/designs/design-audit-$(date +%Y%m%d)"
mkdir -p "$REPORT_DIR/screenshots"
echo "REPORT_DIR: $REPORT_DIR"

Prior Learnings

_CROSS_PROJ=$(~/.claude/skills/gstack/bin/gstack-config get cross_project_learnings 2>/dev/null || echo "unset")
echo "CROSS_PROJECT: $_CROSS_PROJ"
if [ "$_CROSS_PROJ" = "true" ]; then
  ~/.claude/skills/gstack/bin/gstack-learnings-search --limit 10 --cross-project 2>/dev/null || true
else
  ~/.claude/skills/gstack/bin/gstack-learnings-search --limit 10 2>/dev/null || true
fi

gstack can search learnings from your other projects on this machine to find patterns that might apply here. This stays local (no data leaves your machine). Recommended for solo developers. Skip if you work on multiple client codebases where cross-contamination would be a concern.

• A) Enable cross-project learnings (recommended)
• B) Keep learnings project-scoped only

UX Principles: How Users Actually Behave

The Three Laws of Usability

1. Don't make me think. Every page should be self-evident. If a user stops to think "What do I click?" or "What does this mean?", the design has failed. Self-evident > self-explanatory > requires explanation.
2. Clicks don't matter, thinking does. Three mindless, unambiguous clicks beat one click that requires thought. Each step should feel like an obvious choice (animal, vegetable, or mineral), not a puzzle.
3. Omit, then omit again. Get rid of half the words on each page, then get rid of half of what's left. Happy talk (self-congratulatory text) must die. Instructions must die. If they need reading, the design has failed.

How Users Actually Behave

• Users scan, they don't read. Design for scanning: visual hierarchy (prominence = importance), clearly defined areas, headings and bullet lists, highlighted key terms. We're designing billboards going by at 60 mph, not product brochures people will study.
• Users satisfice. They pick the first reasonable option, not the best. Make the right choice the most visible choice.
• Users muddle through. They don't figure out how things work. They wing it. If they accomplish their goal by accident, they won't seek the "right" way. Once they find something that works, no matter how badly, they stick to it.
• Users don't read instructions. They dive in. Guidance must be brief, timely, and unavoidable, or it won't be seen.

Billboard Design for Interfaces

• Use conventions. Logo top-left, nav top/left, search = magnifying glass. Don't innovate on navigation to be clever. Innovate when you KNOW you have a better idea, otherwise use conventions. Even across languages and cultures, web conventions let people identify the logo, nav, search, and main content.
• Visual hierarchy is everything. Related things are visually grouped. Nested things are visually contained. More important = more prominent. If everything shouts, nothing is heard. Start with the assumption everything is visual noise, guilty until proven innocent.
• Make clickable things obviously clickable. No relying on hover states for discoverability, especially on mobile where hover doesn't exist. Shape, location, and formatting (color, underlining) must signal clickability without interaction.
• Eliminate noise. Three sources: too many things shouting for attention (shouting), things not organized logically (disorganization), and too much stuff (clutter). Fix noise by removal, not addition.
• Clarity trumps consistency. If making something significantly clearer requires making it slightly inconsistent, choose clarity every time.

Navigation as Wayfinding

The Goodwill Reservoir

Mobile: Same Rules, Higher Stakes

Phases 1-6: Design Audit Baseline

Modes

Full (default)

Quick (--quick)

Deep (--deep)

Diff-aware (automatic when on a feature branch with no URL)

1. Analyze the branch diff: git diff main...HEAD --name-only
2. Map changed files to affected pages/routes
3. Detect running app on common local ports (3000, 4000, 8080)
4. Audit only affected pages, compare design quality before/after

Regression (--regression or previous design-baseline.json found)

Phase 1: First Impression

1. Navigate to the target URL
2. Take a full-page desktop screenshot: $B screenshot "$REPORT_DIR/screenshots/first-impression.png"
3. Write the First Impression using this structured critique format:
   • "The site communicates [what]." (what it says at a glance — competence? playfulness? confusion?)
   • "I notice [observation]." (what stands out, positive or negative — be specific)
   • "The first 3 things my eye goes to are: [1], [2], [3]." (hierarchy check — are these the 3 things the designer intended? If not, the visual hierarchy is lying.)
   • "If I had to describe this in one word: [word]." (gut verdict)

Phase 2: Design System Extraction

# Fonts in use (capped at 500 elements to avoid timeout)
$B js "JSON.stringify([...new Set([...document.querySelectorAll('*')].slice(0,500).map(e => getComputedStyle(e).fontFamily))])"

# Color palette in use
$B js "JSON.stringify([...new Set([...document.querySelectorAll('*')].slice(0,500).flatMap(e => [getComputedStyle(e).color, getComputedStyle(e).backgroundColor]).filter(c => c !== 'rgba(0, 0, 0, 0)'))])"

# Heading hierarchy
$B js "JSON.stringify([...document.querySelectorAll('h1,h2,h3,h4,h5,h6')].map(h => ({tag:h.tagName, text:h.textContent.trim().slice(0,50), size:getComputedStyle(h).fontSize, weight:getComputedStyle(h).fontWeight})))"

# Touch target audit (find undersized interactive elements)
$B js "JSON.stringify([...document.querySelectorAll('a,button,input,[role=button]')].filter(e => {const r=e.getBoundingClientRect(); return r.width>0 && (r.width<44||r.height<44)}).map(e => ({tag:e.tagName, text:(e.textContent||'').trim().slice(0,30), w:Math.round(e.getBoundingClientRect().width), h:Math.round(e.getBoundingClientRect().height)})).slice(0,20))"

# Performance baseline
$B perf

• Fonts: list with usage counts. Flag if >3 distinct font families.
• Colors: palette extracted. Flag if >12 unique non-gray colors. Note warm/cool/mixed.
• Heading Scale: h1-h6 sizes. Flag skipped levels, non-systematic size jumps.
• Spacing Patterns: sample padding/margin values. Flag non-scale values.

Phase 3: Page-by-Page Visual Audit

$B goto <url>
$B snapshot -i -a -o "$REPORT_DIR/screenshots/{page}-annotated.png"
$B responsive "$REPORT_DIR/screenshots/{page}"
$B console --errors
$B perf

Auth Detection

$B url

Trunk Test (run on every page)

1. What site is this? (Site ID visible and identifiable)
2. What page am I on? (Page name prominent, matches what I clicked)
3. What are the major sections? (Primary nav visible and clear)
4. What are my options at this level? (Local nav or content choices obvious)
5. Where am I in the scheme of things? ("You are here" indicator, breadcrumbs)
6. How can I search? (Search box findable without hunting)

Design Audit Checklist (10 categories, ~80 items)

• Clear focal point? One primary CTA per view?
• Eye flows naturally top-left to bottom-right?
• Visual noise — competing elements fighting for attention?
• Information density appropriate for content type?
• Z-index clarity — nothing unexpectedly overlapping?
• Above-the-fold content communicates purpose in 3 seconds?
• Squint test: hierarchy still visible when blurred?
• White space is intentional, not leftover?

• Font count <=3 (flag if more)
• Scale follows ratio (1.25 major third or 1.333 perfect fourth)
• Line-height: 1.5x body, 1.15-1.25x headings
• Measure: 45-75 chars per line (66 ideal)
• Heading hierarchy: no skipped levels (h1→h3 without h2)
• Weight contrast: >=2 weights used for hierarchy
• No blacklisted fonts (Papyrus, Comic Sans, Lobster, Impact, Jokerman)
• If primary font is Inter/Roboto/Open Sans/Poppins → flag as potentially generic
• text-wrap: balance or text-pretty on headings (check via $B css <heading> text-wrap)
• Curly quotes used, not straight quotes
• Ellipsis character (…) not three dots (...)
• font-variant-numeric: tabular-nums on number columns
• Body text >= 16px
• Caption/label >= 12px
• No letterspacing on lowercase text

• Palette coherent (<=12 unique non-gray colors)
• WCAG AA: body text 4.5:1, large text (18px+) 3:1, UI components 3:1
• Semantic colors consistent (success=green, error=red, warning=yellow/amber)
• No color-only encoding (always add labels, icons, or patterns)
• Dark mode: surfaces use elevation, not just lightness inversion
• Dark mode: text off-white (~#E0E0E0), not pure white
• Primary accent desaturated 10-20% in dark mode
• color-scheme: dark on html element (if dark mode present)
• No red/green only combinations (8% of men have red-green deficiency)
• Neutral palette is warm or cool consistently — not mixed

• Grid consistent at all breakpoints
• Spacing uses a scale (4px or 8px base), not arbitrary values
• Alignment is consistent — nothing floats outside the grid
• Rhythm: related items closer together, distinct sections further apart
• Border-radius hierarchy (not uniform bubbly radius on everything)
• Inner radius = outer radius - gap (nested elements)
• No horizontal scroll on mobile
• Max content width set (no full-bleed body text)
• env(safe-area-inset-*) for notch devices
• URL reflects state (filters, tabs, pagination in query params)
• Flex/grid used for layout (not JS measurement)
• Breakpoints: mobile (375), tablet (768), desktop (1024), wide (1440)

• Hover state on all interactive elements
• focus-visible ring present (never outline: none without replacement)
• Active/pressed state with depth effect or color shift
• Disabled state: reduced opacity + cursor: not-allowed
• Loading: skeleton shapes match real content layout
• Empty states: warm message + primary action + visual (not just "No items.")
• Error messages: specific + include fix/next step
• Success: confirmation animation or color, auto-dismiss
• Touch targets >= 44px on all interactive elements
• cursor: pointer on all clickable elements
• Mindless choice audit: every decision point (button, link, dropdown, modal choice) is a mindless click (obvious what happens). If a click requires thought about whether it's the right choice, flag as HIGH.

• Mobile layout makes design sense (not just stacked desktop columns)
• Touch targets sufficient on mobile (>= 44px)
• No horizontal scroll on any viewport
• Images handle responsive (srcset, sizes, or CSS containment)
• Text readable without zooming on mobile (>= 16px body)
• Navigation collapses appropriately (hamburger, bottom nav, etc.)
• Forms usable on mobile (correct input types, no autoFocus on mobile)
• No user-scalable=no or maximum-scale=1 in viewport meta

• Easing: ease-out for entering, ease-in for exiting, ease-in-out for moving
• Duration: 50-700ms range (nothing slower unless page transition)
• Purpose: every animation communicates something (state change, attention, spatial relationship)
• prefers-reduced-motion respected (check: $B js "matchMedia('(prefers-reduced-motion: reduce)').matches")
• No transition: all — properties listed explicitly
• Only transform and opacity animated (not layout properties like width, height, top, left)

• Empty states designed with warmth (message + action + illustration/icon)
• Error messages specific: what happened + why + what to do next
• Button labels specific ("Save API Key" not "Continue" or "Submit")
• No placeholder/lorem ipsum text visible in production
• Truncation handled (text-overflow: ellipsis, line-clamp, or break-words)
• Active voice ("Install the CLI" not "The CLI will be installed")
• Loading states end with … ("Saving…" not "Saving...")
• Destructive actions have confirmation modal or undo window
• Happy talk detection: scan for introductory paragraphs that start with "Welcome to..." or tell users how great the site is. If you can hear "blah blah blah", it's happy talk. Flag for removal.
• Instructions detection: any visible instructions longer than one sentence. If users need to read instructions, the design has failed. Flag the instructions AND the interaction they're compensating for.
• Happy talk word count: count total visible words on the page. Classify each text block as "useful content" vs "happy talk" (welcome paragraphs, self-congratulatory text, instructions nobody reads). Report: "This page has X words. Y (Z%) are happy talk."

• Purple/violet/indigo gradient backgrounds or blue-to-purple color schemes
• The 3-column feature grid: icon-in-colored-circle + bold title + 2-line description, repeated 3x symmetrically. THE most recognizable AI layout.
• Icons in colored circles as section decoration (SaaS starter template look)
• Centered everything (text-align: center on all headings, descriptions, cards)
• Uniform bubbly border-radius on every element (same large radius on everything)
• Decorative blobs, floating circles, wavy SVG dividers (if a section feels empty, it needs better content, not decoration)
• Emoji as design elements (rockets in headings, emoji as bullet points)
• Colored left-border on cards (border-left: 3px solid <accent>)
• Generic hero copy ("Welcome to [X]", "Unlock the power of...", "Your all-in-one solution for...")
• Cookie-cutter section rhythm (hero → 3 features → testimonials → pricing → CTA, every section same height)
• system-ui or -apple-system as the PRIMARY display/body font — the "I gave up on typography" signal. Pick a real typeface.

• LCP < 2.0s (web apps), < 1.5s (informational sites)
• CLS < 0.1 (no visible layout shifts during load)
• Skeleton quality: shapes match real content layout, shimmer animation
• Images: loading="lazy", width/height dimensions set, WebP/AVIF format
• Fonts: font-display: swap, preconnect to CDN origins
• No visible font swap flash (FOUT) — critical fonts preloaded

Phase 4: Interaction Flow Review

$B snapshot -i
$B click @e3           # perform action
$B snapshot -D          # diff to see what changed

• Response feel: Does clicking feel responsive? Any delays or missing loading states?
• Transition quality: Are transitions intentional or generic/absent?
• Feedback clarity: Did the action clearly succeed or fail? Is the feedback immediate?
• Form polish: Focus states visible? Validation timing correct? Errors near the source?

Goodwill Reservoir (track across the flow)

• Hidden information the user would want (pricing, contact, shipping): subtract 15
• Format punishment (rejecting valid input like dashes in phone numbers): subtract 10
• Unnecessary information requests: subtract 10
• Interstitials, splash screens, forced tours blocking the task: subtract 15
• Sloppy or unprofessional appearance: subtract 10
• Ambiguous choices that require thinking: subtract 5 each

• Top user tasks are obvious and prominent: add 10
• Upfront about costs and limitations: add 5
• Saves steps (direct links, smart defaults, autofill): add 5 each
• Graceful error recovery with specific fix instructions: add 10
• Apologizes when things go wrong: add 5

Goodwill: 70 ████████████████████░░░░░░░░░░
  Step 1: Login page        70 → 75  (+5 obvious primary action)
  Step 2: Dashboard          75 → 60  (-15 interstitial tour popup)
  Step 3: Settings           60 → 50  (-10 format punishment on phone)
  Step 4: Billing            50 → 35  (-15 hidden pricing info)
  FINAL: 35/100 ⚠️ CRITICAL UX DEBT

Phase 5: Cross-Page Consistency

• Navigation bar consistent across all pages?
• Footer consistent?
• Component reuse vs one-off designs (same button styled differently on different pages?)
• Tone consistency (one page playful while another is corporate?)
• Spacing rhythm carries across pages?

Phase 6: Compile Report

Output Locations

eval "$(~/.claude/skills/gstack/bin/gstack-slug 2>/dev/null)" && mkdir -p ~/.gstack/projects/$SLUG

{
  "date": "YYYY-MM-DD",
  "url": "<target>",
  "designScore": "B",
  "aiSlopScore": "C",
  "categoryGrades": { "hierarchy": "A", "typography": "B", ... },
  "findings": [{ "id": "FINDING-001", "title": "...", "impact": "high", "category": "typography" }]
}

Scoring System

• Design Score: {A-F} — weighted average of all 10 categories
• AI Slop Score: {A-F} — standalone grade with pithy verdict

• A: Intentional, polished, delightful. Shows design thinking.
• B: Solid fundamentals, minor inconsistencies. Looks professional.
• C: Functional but generic. No major problems, no design point of view.
• D: Noticeable problems. Feels unfinished or careless.
• F: Actively hurting user experience. Needs significant rework.

Regression Output

• Load baseline grades
• Compare: per-category deltas, new findings, resolved findings
• Append regression table to report

Design Critique Format

• "I notice..." — observation (e.g., "I notice the primary CTA competes with the secondary action")
• "I wonder..." — question (e.g., "I wonder if users will understand what 'Process' means here")
• "What if..." — suggestion (e.g., "What if we moved search to a more prominent position?")
• "I think... because..." — reasoned opinion (e.g., "I think the spacing between sections is too uniform because it doesn't create hierarchy")

Important Rules

1. Think like a designer, not a QA engineer. You care whether things feel right, look intentional, and respect the user. You do NOT just care whether things "work."
2. Screenshots are evidence. Every finding needs at least one screenshot. Use annotated screenshots (snapshot -a) to highlight elements.
3. Be specific and actionable. "Change X to Y because Z" — not "the spacing feels off."
4. Never read source code. Evaluate the rendered site, not the implementation. (Exception: offer to write DESIGN.md from extracted observations.)
5. AI Slop detection is your superpower. Most developers can't evaluate whether their site looks AI-generated. You can. Be direct about it.
6. Quick wins matter. Always include a "Quick Wins" section — the 3-5 highest-impact fixes that take <30 minutes each.
7. Use snapshot -C for tricky UIs. Finds clickable divs that the accessibility tree misses.
8. Responsive is design, not just "not broken." A stacked desktop layout on mobile is not responsive design — it's lazy. Evaluate whether the mobile layout makes design sense.
9. Document incrementally. Write each finding to the report as you find it. Don't batch.
10. Depth over breadth. 5-10 well-documented findings with screenshots and specific suggestions > 20 vague observations.
11. Show screenshots to the user. After every $B screenshot, $B snapshot -a -o, or $B responsive command, use the Read tool on the output file(s) so the user can see them inline. For responsive (3 files), Read all three. This is critical — without it, screenshots are invisible to the user.

Design Hard Rules

• MARKETING/LANDING PAGE (hero-driven, brand-forward, conversion-focused) → apply Landing Page Rules
• APP UI (workspace-driven, data-dense, task-focused: dashboards, admin, settings) → apply App UI Rules
• HYBRID (marketing shell with app-like sections) → apply Landing Page Rules to hero/marketing sections, App UI Rules to functional sections

1. Generic SaaS card grid as first impression
2. Beautiful image with weak brand
3. Strong headline with no clear action
4. Busy imagery behind text
5. Sections repeating same mood statement
6. Carousel with no narrative purpose
7. App UI made of stacked cards instead of layout

1. Brand/product unmistakable in first screen?
2. One strong visual anchor present?
3. Page understandable by scanning headlines only?
4. Each section has one job?
5. Are cards actually necessary?
6. Does motion improve hierarchy or atmosphere?
7. Would design feel premium with all decorative shadows removed?

• First viewport reads as one composition, not a dashboard
• Brand-first hierarchy: brand > headline > body > CTA
• Typography: expressive, purposeful — no default stacks (Inter, Roboto, Arial, system)
• No flat single-color backgrounds — use gradients, images, subtle patterns
• Hero: full-bleed, edge-to-edge, no inset/tiled/rounded variants
• Hero budget: brand, one headline, one supporting sentence, one CTA group, one image
• No cards in hero. Cards only when card IS the interaction
• One job per section: one purpose, one headline, one short supporting sentence
• Motion: 2-3 intentional motions minimum (entrance, scroll-linked, hover/reveal)
• Color: define CSS variables, avoid purple-on-white defaults, one accent color default
• Copy: product language not design commentary. "If deleting 30% improves it, keep deleting"
• Beautiful defaults: composition-first, brand as loudest text, two typefaces max, cardless by default, first viewport as poster not document

• Calm surface hierarchy, strong typography, few colors
• Dense but readable, minimal chrome
• Organize: primary workspace, navigation, secondary context, one accent
• Avoid: dashboard-card mosaics, thick borders, decorative gradients, ornamental icons
• Copy: utility language — orientation, status, action. Not mood/brand/aspiration
• Cards only when card IS the interaction
• Section headings state what area is or what user can do ("Selected KPIs", "Plan status")

• Define CSS variables for color system
• No default font stacks (Inter, Roboto, Arial, system)
• One job per section
• "If deleting 30% of the copy improves it, keep deleting"
• Cards earn their existence — no decorative card grids
• NEVER use small, low-contrast type (body text < 16px or contrast ratio < 4.5:1 on body text)
• NEVER put labels inside form fields as the only label (placeholder-as-label pattern — labels must be visible when the field has content)
• ALWAYS preserve visited vs unvisited link distinction (visited links must have a different color)
• NEVER float headings between paragraphs (heading must be visually closer to the section it introduces than to the preceding section)

1. Purple/violet/indigo gradient backgrounds or blue-to-purple color schemes
2. The 3-column feature grid: icon-in-colored-circle + bold title + 2-line description, repeated 3x symmetrically. THE most recognizable AI layout.
3. Icons in colored circles as section decoration (SaaS starter template look)
4. Centered everything (text-align: center on all headings, descriptions, cards)
5. Uniform bubbly border-radius on every element (same large radius on everything)
6. Decorative blobs, floating circles, wavy SVG dividers (if a section feels empty, it needs better content, not decoration)
7. Emoji as design elements (rockets in headings, emoji as bullet points)
8. Colored left-border on cards (border-left: 3px solid <accent>)
9. Generic hero copy ("Welcome to [X]", "Unlock the power of...", "Your all-in-one solution for...")
10. Cookie-cutter section rhythm (hero → 3 features → testimonials → pricing → CTA, every section same height)
11. system-ui or -apple-system as the PRIMARY display/body font — the "I gave up on typography" signal. Pick a real typeface.

Output Structure

~/.gstack/projects/$SLUG/designs/design-audit-{YYYYMMDD}/
├── design-audit-{domain}.md                  # Structured report
├── screenshots/
│   ├── first-impression.png                  # Phase 1
│   ├── {page}-annotated.png                  # Per-page annotated
│   ├── {page}-mobile.png                     # Responsive
│   ├── {page}-tablet.png
│   ├── {page}-desktop.png
│   ├── finding-001-before.png                # Before fix
│   ├── finding-001-target.png                # Target mockup (if generated)
│   ├── finding-001-after.png                 # After fix
│   └── ...
└── design-baseline.json                      # For regression mode

Design Outside Voices (parallel)

command -v codex >/dev/null 2>&1 && echo "CODEX_AVAILABLE" || echo "CODEX_NOT_AVAILABLE"

1. Codex design voice (via Bash):

TMPERR_DESIGN=$(mktemp /tmp/codex-design-XXXXXXXX)
_REPO_ROOT=$(git rev-parse --show-toplevel) || { echo "ERROR: not in a git repo" >&2; exit 1; }
codex exec "Review the frontend source code in this repo. Evaluate against these design hard rules:
- Spacing: systematic (design tokens / CSS variables) or magic numbers?
- Typography: expressive purposeful fonts or default stacks?
- Color: CSS variables with defined system, or hardcoded hex scattered?
- Responsive: breakpoints defined? calc(100svh - header) for heroes? Mobile tested?
- A11y: ARIA landmarks, alt text, contrast ratios, 44px touch targets?
- Motion: 2-3 intentional animations, or zero / ornamental only?
- Cards: used only when card IS the interaction? No decorative card grids?

First classify as MARKETING/LANDING PAGE vs APP UI vs HYBRID, then apply matching rules.

LITMUS CHECKS — answer YES/NO:
1. Brand/product unmistakable in first screen?
2. One strong visual anchor present?
3. Page understandable by scanning headlines only?
4. Each section has one job?
5. Are cards actually necessary?
6. Does motion improve hierarchy or atmosphere?
7. Would design feel premium with all decorative shadows removed?

HARD REJECTION — flag if ANY apply:
1. Generic SaaS card grid as first impression
2. Beautiful image with weak brand
3. Strong headline with no clear action
4. Busy imagery behind text
5. Sections repeating same mood statement
6. Carousel with no narrative purpose
7. App UI made of stacked cards instead of layout

Be specific. Reference file:line for every finding." -C "$_REPO_ROOT" -s read-only -c 'model_reasoning_effort="high"' --enable web_search_cached < /dev/null 2>"$TMPERR_DESIGN"

cat "$TMPERR_DESIGN" && rm -f "$TMPERR_DESIGN"

1. Claude design subagent (via Agent tool): Dispatch a subagent with this prompt: "Review the frontend source code in this repo. You are an independent senior product designer doing a source-code design audit. Focus on CONSISTENCY PATTERNS across files rather than individual violations:

• Are spacing values systematic across the codebase?
• Is there ONE color system or scattered approaches?
• Do responsive breakpoints follow a consistent set?
• Is the accessibility approach consistent or spotty?

• Auth failure: If stderr contains "auth", "login", "unauthorized", or "API key": "Codex authentication failed. Run codex login to authenticate."
• Timeout: "Codex timed out after 5 minutes."
• Empty response: "Codex returned no response."
• On any Codex error: proceed with Claude subagent output only, tagged [single-model].
• If Claude subagent also fails: "Outside voices unavailable — continuing with primary review."

~/.claude/skills/gstack/bin/gstack-review-log '{"skill":"design-outside-voices","timestamp":"'"$(date -u +%Y-%m-%dT%H:%M:%SZ)"'","status":"STATUS","source":"SOURCE","commit":"'"$(git rev-parse --short HEAD)"'"}'

Phase 7: Triage

• High Impact: Fix first. These affect the first impression and hurt user trust.
• Medium Impact: Fix next. These reduce polish and are felt subconsciously.
• Polish: Fix if time allows. These separate good from great.

Phase 8: Fix Loop

8a. Locate source

# Search for CSS classes, component names, style files
# Glob for file patterns matching the affected page

• Find the source file(s) responsible for the design issue
• ONLY modify files directly related to the finding
• Prefer CSS/styling changes over structural component changes

8a.5. Target Mockup (if DESIGN_READY)

$D generate --brief "<description of the page/component with the finding fixed, referencing DESIGN.md constraints>" --output "$REPORT_DIR/screenshots/finding-NNN-target.png"

8b. Fix

• Read the source code, understand the context
• Make the minimal fix — smallest change that resolves the design issue
• If a target mockup was generated in 8a.5, use it as the visual reference for the fix
• CSS-only changes are preferred (safer, more reversible)
• Do NOT refactor surrounding code, add features, or "improve" unrelated things

8c. Commit

git add <only-changed-files>
git commit -m "style(design): FINDING-NNN — short description"

• One commit per fix. Never bundle multiple fixes.
• Message format: style(design): FINDING-NNN — short description

8d. Re-test

$B goto <affected-url>
$B screenshot "$REPORT_DIR/screenshots/finding-NNN-after.png"
$B console --errors
$B snapshot -D

8e. Classify

• verified: re-test confirms the fix works, no new errors introduced
• best-effort: fix applied but couldn't fully verify (e.g., needs specific browser state)
• reverted: regression detected → git revert HEAD → mark finding as "deferred"

8e.5. Regression Test (design-review variant)

8f. Self-Regulation (STOP AND EVALUATE)

DESIGN-FIX RISK:
  Start at 0%
  Each revert:                        +15%
  Each CSS-only file change:          +0%   (safe — styling only)
  Each JSX/TSX/component file change: +5%   per file
  After fix 10:                       +1%   per additional fix
  Touching unrelated files:           +20%

Phase 9: Final Design Audit

1. Re-run the design audit on all affected pages
2. If target mockups were generated during the fix loop AND DESIGN_READY: run $D verify --mockup "$REPORT_DIR/screenshots/finding-NNN-target.png" --screenshot "$REPORT_DIR/screenshots/finding-NNN-after.png" to compare the fix result against the target. Include pass/fail in the report.
3. Compute final design score and AI slop score
4. If final scores are WORSE than baseline: WARN prominently — something regressed

Phase 10: Report

eval "$(~/.claude/skills/gstack/bin/gstack-slug 2>/dev/null)" && mkdir -p ~/.gstack/projects/$SLUG

• Fix Status: verified / best-effort / reverted / deferred
• Commit SHA (if fixed)
• Files Changed (if fixed)
• Before/After screenshots (if fixed)

• Total findings
• Fixes applied (verified: X, best-effort: Y, reverted: Z)
• Deferred findings
• Design score delta: baseline → final
• AI slop score delta: baseline → final

"Design review found N issues, fixed M. Design score X → Y, AI slop score X → Y."

Phase 11: TODOS.md Update

1. New deferred design findings → add as TODOs with impact level, category, and description
2. Fixed findings that were in TODOS.md → annotate with "Fixed by /design-review on {branch}, {date}"

Capture Learnings

~/.claude/skills/gstack/bin/gstack-learnings-log '{"skill":"design-review","type":"TYPE","key":"SHORT_KEY","insight":"DESCRIPTION","confidence":N,"source":"SOURCE","files":["path/to/relevant/file"]}'

Additional Rules (design-review specific)

1. Clean working tree required. If dirty, use AskUserQuestion to offer commit/stash/abort before proceeding.
2. One commit per fix. Never bundle multiple design fixes into one commit.
3. Only modify tests when generating regression tests in Phase 8e.5. Never modify CI configuration. Never modify existing tests — only create new test files.
4. Revert on regression. If a fix makes things worse, git revert HEAD immediately.
5. Self-regulate. Follow the design-fix risk heuristic. When in doubt, stop and ask.
6. CSS-first. Prefer CSS/styling changes over structural component changes. CSS-only changes are safer and more reversible.
7. DESIGN.md export. You MAY write a DESIGN.md file if the user accepts the offer from Phase 2.