@mmerterden/multi-agent-pipeline 11.1.0 → 11.3.0

package/pipeline/scripts/smoke-existing-discovery-gate.sh

@@ -1,71 +0,0 @@
-#!/usr/bin/env bash
-#
-# smoke-existing-discovery-gate.sh
-#
-# Asserts the figma-to-component orchestrator has the Phase 1.5 existing-
-# component discovery gate wired in:
-#   - phase-1.5-existing-discovery.md exists with the canonical anchors
-#   - phase-2-orchestrator ENTRY GATE consults discovery.status
-#   - phase-6-code-connect ENTRY GATE skips publish when already mapped
-#
-# This is Defect 10 from the GH1241 stability plan (docs/STABILITY-FIX-PLAN.md).
-
-set -euo pipefail
-
-SELF_DIR=$(cd "$(dirname "$0")" && pwd)
-PIPELINE_ROOT=$(cd "$SELF_DIR/.." && pwd)
-
-cd "$PIPELINE_ROOT"
-
-FAIL=0
-
-GATE_DOC="skills/figma-ios/figma-to-component/phases/phase-1.5-existing-discovery.md"
-PHASE2="skills/figma-ios/figma-to-component/phases/phase-2-orchestrator.md"
-PHASE6="skills/figma-ios/figma-to-component/phases/phase-6-code-connect.md"
-
-# 1. Phase 1.5 doc exists.
-if [ ! -f "$GATE_DOC" ]; then
-  echo "FAIL [smoke-existing-discovery-gate]: $GATE_DOC missing"
-  exit 1
-fi
-
-# 2. Phase 1.5 contains the four canonical statuses.
-declare -a REQUIRED_STATUSES=(
-  "GREENFIELD_OK"
-  "ALREADY_MAPPED"
-  "EXISTING_SOURCE_NO_CC"
-  "GAP_ANALYSIS"
-)
-for st in "${REQUIRED_STATUSES[@]}"; do
-  if ! grep -qF "$st" "$GATE_DOC"; then
-    echo "FAIL [smoke-existing-discovery-gate]: $GATE_DOC must define status $st"
-    FAIL=$((FAIL + 1))
-  fi
-done
-
-# 3. Phase 1.5 writes to agent-state.discovery.{status,sourcePath,gaps}.
-if ! grep -qE 'discovery.*status|"discovery"' "$GATE_DOC"; then
-  echo "FAIL [smoke-existing-discovery-gate]: $GATE_DOC must write to agent-state.discovery"
-  FAIL=$((FAIL + 1))
-fi
-
-# 4. Phase 2 ENTRY GATE consults discovery.status.
-if ! grep -qE "discovery\.status|agent-state\.discovery" "$PHASE2"; then
-  echo "FAIL [smoke-existing-discovery-gate]: $PHASE2 ENTRY GATE must consult discovery.status"
-  FAIL=$((FAIL + 1))
-fi
-
-# 5. Phase 6 ENTRY GATE skips when ALREADY_MAPPED or .figma.swift exists.
-if ! grep -qE "ALREADY_MAPPED|already published|GAP_ANALYSIS" "$PHASE6"; then
-  echo "FAIL [smoke-existing-discovery-gate]: $PHASE6 ENTRY GATE must skip when ALREADY_MAPPED / GAP_ANALYSIS"
-  FAIL=$((FAIL + 1))
-fi
-
-if [ "$FAIL" -gt 0 ]; then
-  echo ""
-  echo "smoke-existing-discovery-gate: $FAIL violation(s)."
-  echo "Defect 10 contract  -  see docs/STABILITY-FIX-PLAN.md."
-  exit 1
-fi
-
-echo "smoke-existing-discovery-gate: ok"

package/pipeline/scripts/smoke-figma-android-parity.sh

@@ -1,148 +0,0 @@
-#!/usr/bin/env bash
-# smoke-figma-android-parity.sh  -  WS-7 contract.
-#
-# Verifies the Android platform skill tree under pipeline/skills/figma-android/
-# mirrors the iOS peer for every platform-specific skill required by the
-# figma-to-component orchestrator. Also checks Android-specific details
-# (Kotlin file names, Compose tooling, Paparazzi/Roborazzi references).
-
-set -euo pipefail
-
-HERE="$(cd "$(dirname "$0")" && pwd)"
-ROOT="$(cd "$HERE/../.." && pwd)"
-IOS_DIR="$ROOT/pipeline/skills/figma-ios"
-ANDROID_DIR="$ROOT/pipeline/skills/figma-android"
-
-PASS=0
-FAIL=0
-pass() { PASS=$((PASS+1)); echo "  ✓ $1"; }
-fail() { FAIL=$((FAIL+1)); echo "  ✗ $1"; }
-
-[ -d "$IOS_DIR" ] || { echo "error: iOS tree missing at $IOS_DIR" >&2; exit 1; }
-[ -d "$ANDROID_DIR" ] || { echo "error: Android tree missing at $ANDROID_DIR" >&2; exit 1; }
-
-# Platform-specific skills that MUST exist on both sides.
-REQUIRED=(
-  figma-to-component
-  figma-component-implement
-  figma-component-test
-  figma-component-code-connect
-  figma-component-wiki
-)
-
-echo "→ 1. Android tree has required platform-specific skills"
-for skill in "${REQUIRED[@]}"; do
-  if [ -d "$ANDROID_DIR/$skill" ] && [ -f "$ANDROID_DIR/$skill/SKILL.md" ]; then
-    pass "$skill present"
-  else
-    fail "$skill missing"
-  fi
-done
-
-echo ""
-echo "→ 2. iOS tree has the same required skills (parity source)"
-for skill in "${REQUIRED[@]}"; do
-  if [ -d "$IOS_DIR/$skill" ] && [ -f "$IOS_DIR/$skill/SKILL.md" ]; then
-    pass "iOS $skill present"
-  else
-    fail "iOS $skill missing (parity source broken)"
-  fi
-done
-
-echo ""
-echo "→ 3. Every Android SKILL.md has frontmatter with platform: android"
-for skill in "${REQUIRED[@]}"; do
-  f="$ANDROID_DIR/$skill/SKILL.md"
-  [ -f "$f" ] || continue
-  if ! head -n 1 "$f" | grep -Fxq -- "---"; then
-    fail "$skill/SKILL.md missing frontmatter"
-    continue
-  fi
-  if awk '/^---$/{c++; next} c==1' "$f" | grep -Fq "platform: android"; then
-    pass "$skill stamped platform: android"
-  else
-    fail "$skill/SKILL.md missing platform: android stamp"
-  fi
-done
-
-echo ""
-echo "→ 4. Every Android SKILL.md stamped status: wip-v5 + sourced-from"
-for skill in "${REQUIRED[@]}"; do
-  f="$ANDROID_DIR/$skill/SKILL.md"
-  [ -f "$f" ] || continue
-  grep -q "^status: wip-v5" "$f" && pass "$skill status: wip-v5" || fail "$skill missing status stamp"
-  grep -q "^sourced-from: " "$f" && pass "$skill has sourced-from" || fail "$skill missing sourced-from"
-done
-
-echo ""
-echo "→ 5. Android implement skill targets Compose + Kotlin"
-IMPL="$ANDROID_DIR/figma-component-implement/SKILL.md"
-if [ -f "$IMPL" ]; then
-  if grep -Fq "@Composable" "$IMPL" && grep -Fq "Configuration.kt" "$IMPL" && grep -Fq "@Immutable" "$IMPL"; then
-    pass "implement references @Composable + Configuration.kt + @Immutable"
-  else
-    fail "implement missing Compose/Kotlin markers"
-  fi
-fi
-
-echo ""
-echo "→ 6. Android test skill targets Compose Testing + Paparazzi"
-TEST="$ANDROID_DIR/figma-component-test/SKILL.md"
-if [ -f "$TEST" ]; then
-  if grep -Fq "createComposeRule" "$TEST" && grep -Fq "Paparazzi" "$TEST"; then
-    pass "test references createComposeRule + Paparazzi"
-  else
-    fail "test missing Compose Testing / Paparazzi markers"
-  fi
-fi
-
-echo ""
-echo "→ 7. Android orchestrator routes Android-specific build (gradleModule)"
-ORCH="$ANDROID_DIR/figma-to-component/SKILL.md"
-if [ -f "$ORCH" ]; then
-  if grep -Fq "gradleModule" "$ORCH" && grep -Fq "./gradlew" "$ORCH"; then
-    pass "orchestrator references gradleModule + gradlew"
-  else
-    fail "orchestrator missing gradleModule / gradlew references"
-  fi
-fi
-
-echo ""
-echo "→ 8. figma-project-config schema declares Android build fields"
-SCHEMA="$ROOT/pipeline/schemas/figma-project-config.schema.json"
-if jq -e '.properties.build.properties.gradleModule' "$SCHEMA" >/dev/null 2>&1 \
- && jq -e '.properties.build.properties.variant' "$SCHEMA" >/dev/null 2>&1; then
-  pass "schema has build.gradleModule + build.variant"
-else
-  fail "schema missing Android build fields"
-fi
-
-echo ""
-echo "→ 9. Platform enum covers Android"
-if jq -e '.properties.project.properties.platform.enum | index("android")' "$SCHEMA" >/dev/null 2>&1; then
-  pass "project.platform enum includes android"
-else
-  fail "project.platform enum missing android"
-fi
-
-echo ""
-echo "→ 10. README.md present explaining the Android tree"
-README="$ANDROID_DIR/README.md"
-if [ -f "$README" ] && grep -Fq "Android Platform Skills" "$README"; then
-  pass "README.md present and descriptive"
-else
-  fail "README.md missing or not descriptive"
-fi
-
-echo ""
-echo "→ 11. component-dispatch.md ref acknowledges Android routing"
-DISPATCH="$ROOT/pipeline/commands/multi-agent/refs/component-dispatch.md"
-if grep -Fq "android" "$DISPATCH" && grep -Fq "figma-android" "$DISPATCH"; then
-  pass "component-dispatch.md routes iOS/Android"
-else
-  fail "component-dispatch.md missing Android routing"
-fi
-
-echo ""
-echo "══ figma-android-parity smoke: $PASS passed, $FAIL failed ══"
-[ "$FAIL" -eq 0 ]

package/pipeline/scripts/smoke-figma-credential-store.sh

@@ -1,105 +0,0 @@
-#!/usr/bin/env bash
-# smoke-figma-credential-store.sh  -  drift gate for figma skill Keychain access.
-#
-# In v8.3.x the figma skill family was migrated off raw `security` calls and
-# routed through the cross-platform credential helper:
-#   ~/.claude/lib/credential-store.sh  (Claude Code installs)
-#   ~/.copilot/lib/credential-store.sh (Copilot CLI installs)
-# Each bash block uses a `CRED_STORE` resolver that picks whichever helper is
-# present so the same source ships to both CLIs.
-#
-# This smoke locks the migration in place. It re-fails the build if anyone
-# reverts a figma skill back to a direct `security find-generic-password` /
-# `add-generic-password` / `delete-generic-password` call against one of OUR
-# managed token names.
-#
-# Two intentional exceptions:
-#   - Subprocess calls that read the `Claude Code-credentials` keychain entry
-#     (Claude Code's own OAuth-token store, not ours).
-#   - Pedagogical references inside CHANGELOG / MIGRATION docs.
-#
-# Exit 0 = no drift, 1 = at least one figma file regressed.
-
-set -uo pipefail
-
-ROOT="$(cd "$(dirname "$0")/../.." && pwd)"
-
-pass=0
-fail=0
-failures=()
-record_pass() { pass=$((pass + 1)); printf '  \033[0;32mPASS\033[0m %s\n' "$1"; }
-record_fail() { fail=$((fail + 1)); failures+=("$1"); printf '  \033[0;31mFAIL\033[0m %s\n' "$1"; }
-
-printf '→ smoke-figma-credential-store: figma skills route Keychain via the helper, not raw `security`\n'
-
-# 1. The 8 figma surfaces that were migrated must keep using "$CRED_STORE" or
-#    `~/.claude/lib/credential-store.sh` / `~/.copilot/lib/credential-store.sh`.
-MIGRATED_FILES=(
-  "pipeline/skills/figma-common/figma-issue/SKILL.md"
-  "pipeline/skills/figma-common/figma-component-confluence-sync/SKILL.md"
-  "pipeline/skills/figma-common/figma-setup/SKILL.md"
-  "pipeline/skills/figma-common/figma-validate/SKILL.md"
-  "pipeline/skills/figma-ios/figma-to-component/phases/phase-6-code-connect.md"
-  "pipeline/skills/figma-ios/figma-to-component/reference/code-connect.md"
-  "pipeline/skills/figma-ios/figma-to-component/reference/rest-api-script.md"
-  "pipeline/skills/figma-ios/figma-to-component/reference/tools.md"
-  "pipeline/skills/figma-ios/figma-to-component/scripts/phase1-gather.py"
-)
-
-for rel in "${MIGRATED_FILES[@]}"; do
-  abs="$ROOT/$rel"
-  base="$(basename "$rel")"
-  if [ ! -f "$abs" ]; then
-    record_fail "$base missing  -  figma skill set drifted out of inventory"
-    continue
-  fi
-
-  # Each of these must reference the helper at least once (resolver, get/set/delete call,
-  # or python subprocess wrapper). If the helper reference is gone, the file regressed.
-  if grep -qE 'credential-store\.sh|CRED_STORE|keychain\.py' "$abs"; then
-    record_pass "$base references the credential helper"
-  else
-    record_fail "$base lost its credential-store.sh reference"
-  fi
-
-  # Now look for raw `security {find,add,delete}-generic-password` calls
-  # against OUR managed tokens. Anything matching one of these service names
-  # is a regression. (Claude Code's own `Claude Code-credentials` entry is
-  # NOT in this list  -  the figma-setup wizard is allowed to read it.)
-  managed_tokens='FIGMA_ACCESS_TOKEN|FIGMA_MCP_TOKEN|FIGMA_API_USER|JIRA_TOKEN|CONFLUENCE_TOKEN'
-  bad=$(grep -nE "security (find|add|delete)-generic-password.*-s [\"']?($managed_tokens)[\"']?" "$abs" 2>/dev/null || true)
-  if [ -z "$bad" ]; then
-    record_pass "$base contains no raw security calls against managed tokens"
-  else
-    record_fail "$base regressed to raw security on a managed token:
-$bad"
-  fi
-done
-
-# 2. The cross-CLI contract doc must point callers at credential-store.sh as
-#    the canonical path. If it ever flips back to raw `security` in the
-#    "canonical" column the principle is broken.
-CONTRACT="$ROOT/pipeline/commands/multi-agent/refs/cross-cli-contract.md"
-if [ -f "$CONTRACT" ] && grep -qE 'canonical path is .*credential-store\.sh' "$CONTRACT"; then
-  record_pass "cross-cli-contract.md declares credential-store.sh as canonical"
-else
-  record_fail "cross-cli-contract.md no longer declares credential-store.sh as canonical"
-fi
-
-# 3. The Copilot install template allowlist must list the helper, not just raw `security`.
-TEMPLATE="$ROOT/install/templates/copilot-instructions.md"
-if [ -f "$TEMPLATE" ] && grep -q 'credential-store\.sh' "$TEMPLATE"; then
-  record_pass "install template allowlist references credential-store.sh"
-else
-  record_fail "install template allowlist no longer references credential-store.sh"
-fi
-
-# Summary
-total=$((pass + fail))
-printf '\n→ smoke-figma-credential-store: %d/%d passed\n' "$pass" "$total"
-if [ "$fail" -ne 0 ]; then
-  printf '\nFailures:\n'
-  for f in "${failures[@]}"; do printf '  - %s\n' "$f"; done
-  exit 1
-fi
-exit 0

package/pipeline/scripts/smoke-figma-cross-cli-inventory.sh

@@ -1,177 +0,0 @@
-#!/usr/bin/env bash
-# smoke-figma-cross-cli-inventory.sh  -  enforces the cross-CLI parity
-# inventory for the v5.0.0 figma-to-component skill set.
-#
-# Source-tree assertions (does not require install.js to have deployed
-# the skills). Complements smoke-cross-cli-behavior.sh (installed-tree)
-# and smoke-commands-skills-parity.sh (multi-agent commands).
-#
-# Contract source of truth: refs/cross-cli-contract.md section1.1.
-#
-# Verifies:
-#   1. Every platform-specific figma-android skill exists in figma-ios too.
-#   2. Every required figma-* skill from the contract table exists in figma-ios.
-#   3. Every SKILL.md that claims "platform: android" lives under figma-android/.
-#   4. Every SKILL.md that claims "platform: ios" lives under figma-ios/.
-#   5. No orphaned figma skill directories (not in the contract).
-#   6. Contract doc references both figma-ios and figma-android paths.
-
-set -euo pipefail
-
-HERE="$(cd "$(dirname "$0")" && pwd)"
-ROOT="$(cd "$HERE/../.." && pwd)"
-IOS_DIR="$ROOT/pipeline/skills/figma-ios"
-ANDROID_DIR="$ROOT/pipeline/skills/figma-android"
-COMMON_DIR="$ROOT/pipeline/skills/figma-common"
-CONTRACT="$ROOT/pipeline/commands/multi-agent/refs/cross-cli-contract.md"
-
-PASS=0
-FAIL=0
-pass() { PASS=$((PASS+1)); echo "  ✓ $1"; }
-fail() { FAIL=$((FAIL+1)); echo "  ✗ $1"; }
-
-for d in "$IOS_DIR" "$ANDROID_DIR" "$COMMON_DIR"; do
-  [ -d "$d" ] || { echo "error: $d missing" >&2; exit 1; }
-done
-[ -f "$CONTRACT" ] || { echo "error: $CONTRACT missing" >&2; exit 1; }
-
-# Platform-specific skills (required on BOTH sides per contract)
-PLATFORM_SPECIFIC=(
-  figma-to-component
-  figma-component-implement
-  figma-component-test
-  figma-component-code-connect
-  figma-component-wiki
-)
-
-# Platform-agnostic (required on at least iOS side today  -  WS-7b moves to figma-common/)
-COMMON_REQUIRED=(
-  figma-component-start
-  figma-component-confluence-sync
-  figma-component-status-update
-  figma-commit
-  figma-iteration-commit
-  figma-issue
-  figma-iterate
-  figma-cli-iterate
-  figma-cli-lean-iterate
-  figma-cli-iterate-mend
-  figma-cli-skip
-  figma-fix
-  figma-mend
-  figma-review
-  figma-validate
-  figma-setup
-  figma-remote-mcp-auth
-  figma-ui-patterns
-  figma-utility
-  figma-skip
-  figma-form-integration
-  figma-price-integration
-)
-
-echo "→ 1. Platform-specific skills exist on BOTH figma-ios and figma-android"
-for skill in "${PLATFORM_SPECIFIC[@]}"; do
-  ios_ok="false"
-  and_ok="false"
-  [ -f "$IOS_DIR/$skill/SKILL.md" ] && ios_ok="true"
-  [ -f "$ANDROID_DIR/$skill/SKILL.md" ] && and_ok="true"
-  if [ "$ios_ok" = "true" ] && [ "$and_ok" = "true" ]; then
-    pass "$skill present on both platforms"
-  else
-    fail "$skill parity broken (ios=$ios_ok android=$and_ok)"
-  fi
-done
-
-echo ""
-echo "→ 2. Platform-agnostic skills live in figma-common/ (WS-7b ✓)"
-for skill in "${COMMON_REQUIRED[@]}"; do
-  if [ -f "$COMMON_DIR/$skill/SKILL.md" ]; then
-    pass "$skill present in figma-common"
-  else
-    fail "$skill missing in figma-common"
-  fi
-done
-
-echo ""
-echo "→ 2b. Platform-agnostic skills are NOT duplicated under figma-ios"
-DUPES=0
-for skill in "${COMMON_REQUIRED[@]}"; do
-  if [ -d "$IOS_DIR/$skill" ]; then
-    fail "duplication: $skill exists in BOTH figma-ios/ and figma-common/"
-    DUPES=$((DUPES+1))
-  fi
-done
-[ "$DUPES" -eq 0 ] && pass "no common-skill duplication in figma-ios"
-
-echo ""
-echo "→ 3. figma-android SKILL.md files stamp platform: android"
-while IFS= read -r -d '' f; do
-  base=$(dirname "$f" | xargs basename)
-  if head -n 1 "$f" | grep -Fxq -- "---"; then
-    if awk '/^---$/{c++; next} c==1' "$f" | grep -Fq "platform: android"; then
-      pass "$base stamps platform: android"
-    else
-      fail "$base missing 'platform: android' stamp"
-    fi
-  fi
-done < <(find "$ANDROID_DIR" -maxdepth 2 -name SKILL.md -print0)
-
-echo ""
-echo "→ 4. figma-ios SKILL.md files do NOT claim platform: android (only ios or none)"
-while IFS= read -r -d '' f; do
-  if awk '/^---$/{c++; next} c==1' "$f" | grep -Fq "platform: android"; then
-    fail "$(dirname "$f" | xargs basename) claims platform: android but lives under figma-ios/"
-  fi
-done < <(find "$IOS_DIR" -maxdepth 2 -name SKILL.md -print0)
-pass "no cross-platform stamp leakage in figma-ios/"
-
-echo ""
-echo "→ 5. No orphaned figma-android directories (contract-table inventory)"
-EXPECTED_ANDROID=("${PLATFORM_SPECIFIC[@]}")
-ORPHANS=0
-for d in "$ANDROID_DIR"/*/; do
-  [ -d "$d" ] || continue
-  name=$(basename "$d")
-  case " ${EXPECTED_ANDROID[*]} " in
-    *" $name "*) ;;
-    *) fail "unexpected dir: figma-android/$name (not in contract)"; ORPHANS=$((ORPHANS+1)) ;;
-  esac
-done
-[ "$ORPHANS" -eq 0 ] && pass "no orphaned figma-android directories"
-
-echo ""
-echo "→ 6. cross-cli-contract.md references both platforms"
-if grep -Fq "pipeline/skills/figma-ios" "$CONTRACT" && grep -Fq "pipeline/skills/figma-android" "$CONTRACT"; then
-  pass "contract references figma-ios + figma-android paths"
-else
-  fail "contract missing one or both figma platform paths"
-fi
-
-echo ""
-echo "→ 7. Contract section 1.1 present (figma subphase skills section)"
-if grep -Fq "1.1 Figma component subphase skills" "$CONTRACT"; then
-  pass "section 1.1 heading present"
-else
-  fail "section 1.1 heading missing"
-fi
-
-echo ""
-echo "→ 8. Contract section 1.2 routing rules present"
-if grep -Fq "1.2 Figma-skill routing from multi-agent" "$CONTRACT"; then
-  pass "section 1.2 heading present"
-else
-  fail "section 1.2 heading missing"
-fi
-
-echo ""
-echo "→ 9. Contract section 1.3 frontmatter requirements present"
-if grep -Fq "1.3 Figma skill frontmatter" "$CONTRACT"; then
-  pass "section 1.3 heading present"
-else
-  fail "section 1.3 heading missing"
-fi
-
-echo ""
-echo "══ figma-cross-cli-inventory smoke: $PASS passed, $FAIL failed ══"
-[ "$FAIL" -eq 0 ]

package/pipeline/scripts/smoke-figma-sync.sh

@@ -1,149 +0,0 @@
-#!/usr/bin/env bash
-# smoke-figma-sync.sh  -  WS-9 contract for sync-figma-source.sh.
-#
-# Verifies:
-#   1. sync script exists + executable.
-#   2. prefs.schema.json declares global.figmaSource with correct shape.
-#   3. sync script handles missing figmaSource gracefully (exit 0 w/ message).
-#   4. sync script handles missing path file.
-#   5. dry-run mode writes nothing.
-#   6. CHANGELOG note format is correct (includes `### Sync YYYY-MM-DD`).
-#   7. Watermark update logic (lastCommit + lastSyncedAt) runs atomically.
-
-set -euo pipefail
-
-HERE="$(cd "$(dirname "$0")" && pwd)"
-ROOT="$(cd "$HERE/../.." && pwd)"
-SYNC="$ROOT/pipeline/scripts/sync-figma-source.sh"
-SCHEMA="$ROOT/pipeline/schemas/prefs.schema.json"
-TMP="$(mktemp -d -t figma-sync-smoke-XXXX)"
-trap 'rm -rf "$TMP"' EXIT
-
-PASS=0
-FAIL=0
-pass() { PASS=$((PASS+1)); echo "  ✓ $1"; }
-fail() { FAIL=$((FAIL+1)); echo "  ✗ $1"; }
-
-need() { command -v "$1" >/dev/null 2>&1 || { echo "error: $1 required" >&2; exit 127; }; }
-need jq
-need git
-
-[ -x "$SYNC" ] && pass "sync script exists and is executable" || fail "sync script not executable at $SYNC"
-
-echo ""
-echo "→ 2. prefs schema declares global.figmaSource with correct shape"
-for field in path branch lastSyncedAt lastCommit; do
-  if jq -e ".properties.global.properties.figmaSource.properties.$field" "$SCHEMA" >/dev/null 2>&1; then
-    pass "figmaSource.$field declared"
-  else
-    fail "figmaSource.$field missing from schema"
-  fi
-done
-
-echo ""
-echo "→ 3. missing figmaSource → exit 0 with skip message"
-cat > "$TMP/prefs-no-source.json" <<'EOF'
-{ "schemaVersion": "2.1.0", "global": {}, "projects": {} }
-EOF
-OUT=$(MULTI_AGENT_PREFS="$TMP/prefs-no-source.json" bash "$SYNC" 2>&1)
-RC=$?
-if [ "$RC" -eq 0 ] && echo "$OUT" | grep -Fq "figmaSource not configured"; then
-  pass "gracefully handles missing figmaSource"
-else
-  fail "did not skip cleanly (rc=$RC, out: $OUT)"
-fi
-
-echo ""
-echo "→ 4. figmaSource.path missing on disk → exit 1 with error"
-cat > "$TMP/prefs-bad-path.json" <<EOF
-{ "schemaVersion": "2.1.0", "global": { "figmaSource": { "path": "$TMP/does-not-exist", "branch": "develop" } }, "projects": {} }
-EOF
-set +e
-OUT=$(MULTI_AGENT_PREFS="$TMP/prefs-bad-path.json" bash "$SYNC" 2>&1)
-RC=$?
-set -e
-if [ "$RC" -ne 0 ] && echo "$OUT" | grep -Fq "does not exist"; then
-  pass "errors cleanly on missing path"
-else
-  fail "did not error on missing path (rc=$RC)"
-fi
-
-echo ""
-echo "→ 5. dry-run mode does not mutate prefs"
-# Build a tiny fake source with .instructions/figma/sample/SKILL.md
-FAKE_SRC="$TMP/fake-src"
-mkdir -p "$FAKE_SRC/.instructions/figma/sample" "$FAKE_SRC/.claude/skills/sample"
-cat > "$FAKE_SRC/.instructions/figma/sample/SKILL.md" <<'EOF'
----
-name: sample
-description: smoke fixture
----
-# sample
-EOF
-cat > "$FAKE_SRC/.claude/skills/sample/SKILL.md" <<'EOF'
----
-name: sample
-description: smoke stub
----
-# sample stub
-EOF
-(cd "$FAKE_SRC" && git init -q && git -c user.email=t@t -c user.name=t add . && git -c user.email=t@t -c user.name=t commit -q -m init)
-PREFS_BEFORE="$TMP/prefs-dry.json"
-cat > "$PREFS_BEFORE" <<EOF
-{ "schemaVersion": "2.1.0", "global": { "figmaSource": { "path": "$FAKE_SRC", "branch": "main" } }, "projects": {} }
-EOF
-CHECKSUM_BEFORE=$(jq -c '.' "$PREFS_BEFORE" | shasum | awk '{print $1}')
-MULTI_AGENT_PREFS="$PREFS_BEFORE" bash "$SYNC" --dry-run >/dev/null 2>&1 || true
-CHECKSUM_AFTER=$(jq -c '.' "$PREFS_BEFORE" | shasum | awk '{print $1}')
-[ "$CHECKSUM_BEFORE" = "$CHECKSUM_AFTER" ] && pass "dry-run leaves prefs untouched" || fail "dry-run mutated prefs"
-
-echo ""
-echo "→ 6. CHANGELOG heading pattern used by sync (grep shape)"
-if grep -Fq '### Sync $(date' "$SYNC" || grep -Eq '### Sync \$\(date' "$SYNC"; then
-  pass "sync emits '### Sync YYYY-MM-DD' heading"
-else
-  fail "sync script does not emit the canonical CHANGELOG heading"
-fi
-
-echo ""
-echo "→ 7. Watermark update uses atomic rename"
-if grep -Fq "mv \"\$TMP\" \"\$PREFS_FILE\"" "$SYNC"; then
-  pass "atomic rename on prefs update"
-else
-  fail "no atomic rename in sync script"
-fi
-
-echo ""
-echo "→ 8. --force-full flag is recognised"
-if grep -Fq -- "--force-full" "$SYNC"; then
-  pass "force-full flag supported"
-else
-  fail "force-full flag missing"
-fi
-
-echo ""
-echo "→ 9. --plan flag writes a structured JSON plan artifact"
-if grep -Fq -- "--plan" "$SYNC"; then
-  pass "plan flag recognised"
-else
-  fail "plan flag missing"
-fi
-if grep -Fq ".last-figma-sync-plan.json" "$SYNC"; then
-  pass "plan writes to .last-figma-sync-plan.json"
-else
-  fail "plan output path missing"
-fi
-if grep -Fq "overlaySkip" "$SYNC"; then
-  pass "plan records overlay-skip per skill"
-else
-  fail "plan does not record overlaySkip"
-fi
-if grep -Fq "tree" "$SYNC" && grep -Fq "figma-ios" "$SYNC" && grep -Fq "figma-common" "$SYNC"; then
-  pass "plan records destination tree (figma-ios vs figma-common)"
-else
-  fail "plan missing tree routing info"
-fi
-
-echo ""
-echo "══ figma-sync smoke: $PASS passed, $FAIL failed ══"
-[ "$FAIL" -eq 0 ]