@scrymore/scry-deployer 0.12.0-next.20261006094957 → 0.12.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@scrymore/scry-deployer",
-  "version": "0.12.0-next.20261006094957",
+  "version": "0.12.0",
   "description": "A CLI to automate the deployment of Storybook static builds.",
   "main": "index.js",
   "bin": {

package/scripts/check-skill-install.sh

@@ -0,0 +1,73 @@
+#!/usr/bin/env bash
+# AT-16: install skills/scry-native-capture-setup from this checkout with `npx skills add` into a
+# throwaway HOME and project, then check the skill landed and its SKILL.md frontmatter parses.
+# Usage: bash scripts/check-skill-install.sh [skill-name]   (default: scry-native-capture-setup)
+# Also fails if the skill names the old epinnock/scry-node repo. Needs network for `npx skills` (npm registry) the first time. Never touches the real HOME or repo.
+set -euo pipefail
+SKILL="${1:-scry-native-capture-setup}"
+REPO="$(cd "$(dirname "$0")/.." && pwd)"
+TMP="$(mktemp -d)"
+trap 'rm -rf "$TMP"' EXIT
+mkdir -p "$TMP/home" "$TMP/project"
+git -C "$TMP/project" init -q .
+
+# F14: the skill must name the canonical repo, never the old epinnock/ name.
+if grep -rn "epinnock/scry-node" "$REPO/skills/$SKILL" >/dev/null 2>&1; then
+  echo "FAIL: skills/$SKILL still mentions epinnock/scry-node (use scryorg/scry-node)" >&2; exit 1
+fi
+grep -q "npx skills add scryorg/scry-node --skill $SKILL" "$REPO/skills/$SKILL/SKILL.md" \
+  || { echo "FAIL: SKILL.md does not show 'npx skills add scryorg/scry-node --skill $SKILL'" >&2; exit 1; }
+echo "canonical install command: npx skills add scryorg/scry-node --skill $SKILL (this script installs from the local checkout)"
+
+echo "install: npx skills add $REPO --skill $SKILL  (HOME=$TMP/home)"
+(cd "$TMP/project" && HOME="$TMP/home" npx --yes skills add "$REPO" --skill "$SKILL" -y >"$TMP/install.log" 2>&1) \
+  || { tail -20 "$TMP/install.log" >&2; echo "FAIL: skills add exited non-zero" >&2; exit 1; }
+
+# The installer writes the canonical copy to .agents/skills/<name> and links Claude Code to it
+# (other agents such as Amp get a rewritten copy under agent/skills; not checked here).
+FOUND="$TMP/project/.agents/skills/$SKILL/SKILL.md"
+[ -f "$FOUND" ] || { tail -20 "$TMP/install.log" >&2; echo "FAIL: .agents/skills/$SKILL/SKILL.md not found after install" >&2; exit 1; }
+echo "landed:  ${FOUND#"$TMP"/}"
+[ -f "$TMP/project/.claude/skills/$SKILL/SKILL.md" ] || { echo "FAIL: Claude Code link .claude/skills/$SKILL missing" >&2; exit 1; }
+echo "landed:  project/.claude/skills/$SKILL/SKILL.md (Claude Code)"
+
+# Only the requested skill may have been installed (no scry-setup riding along).
+OTHERS="$(find "$TMP/project/.agents/skills" -mindepth 2 -maxdepth 2 -name SKILL.md ! -path "*/$SKILL/*" 2>/dev/null || true)"
+[ -z "$OTHERS" ] || { echo "FAIL: unexpected extra skills installed: $OTHERS" >&2; exit 1; }
+
+SKILL_MD="$FOUND" SKILL_NAME="$SKILL" node -e '
+const fs = require("fs"), path = require("path");
+const file = process.env.SKILL_MD, name = process.env.SKILL_NAME;
+const text = fs.readFileSync(file, "utf8");
+const m = /^---\r?\n([\s\S]*?)\r?\n---\r?\n/.exec(text);
+if (!m) { console.error("FAIL: no YAML frontmatter"); process.exit(1); }
+const fm = {};
+for (const line of m[1].split(/\r?\n/)) { const k = /^([A-Za-z_-]+):\s*(.*)$/.exec(line); if (k) fm[k[1]] = k[2].trim(); }
+if (fm.name !== name) { console.error(`FAIL: frontmatter name "${fm.name}" != "${name}"`); process.exit(1); }
+if (!fm.description || fm.description.length < 40 || fm.description.length > 1024) { console.error("FAIL: description missing or outside 40..1024 chars"); process.exit(1); }
+// every relative link in SKILL.md must resolve inside the installed skill
+const dir = path.dirname(file), bad = [];
+for (const l of text.matchAll(/\]\((?!https?:|#)([^)\s]+)\)/g)) if (!fs.existsSync(path.join(dir, l[1]))) bad.push(l[1]);
+if (bad.length) { console.error("FAIL: broken relative links: " + bad.join(", ")); process.exit(1); }
+const files = fs.readdirSync(dir, { recursive: true }).filter((f) => fs.statSync(path.join(dir, f)).isFile());
+console.log(`frontmatter ok: name=${fm.name}, description ${fm.description.length} chars, ${files.length} files installed`);
+'
+
+# scry-native-capture-setup covers Flutter (feature flutter-capture): the Flutter files must have landed, the
+# skill must not still refuse Flutter, and UIKit-only must still be refused with the pointer to bundle.md.
+if [ "$SKILL" = scry-native-capture-setup ]; then
+  D="$TMP/project/.agents/skills/$SKILL"
+  for f in references/flutter.md references/bundle.md assets/make-scf.mjs assets/flutter/capture.sh assets/flutter/screens.dart \
+           assets/flutter/screens.json assets/flutter/scry_capture_test.dart assets/flutter/headless_scry_capture_test.dart \
+           assets/flutter/test_driver_integration_test.dart assets/flutter/scry_fonts.dart assets/flutter/scry_registry_test.dart; do
+    [ -f "$D/$f" ] || { echo "FAIL: $f missing from the installed skill" >&2; exit 1; }
+  done
+  if grep -Eqi 'not for[^.]*flutter|UIKit-only, Flutter' "$D/SKILL.md"; then
+    echo "FAIL: SKILL.md still refuses Flutter" >&2; exit 1
+  fi
+  grep -qi 'UIKit-only' "$D/SKILL.md" && grep -q 'references/bundle.md' "$D/SKILL.md" \
+    || { echo "FAIL: SKILL.md no longer refuses UIKit-only with the pointer to references/bundle.md" >&2; exit 1; }
+  grep -q 'references/flutter.md' "$D/SKILL.md" || { echo "FAIL: SKILL.md does not link references/flutter.md" >&2; exit 1; }
+  echo "flutter: reference and assets installed, Flutter no longer refused, UIKit-only still refused"
+fi
+echo "PASS: $SKILL installs and parses"

package/scripts/check-skill-sync.sh

@@ -0,0 +1,53 @@
+#!/usr/bin/env bash
+# Drift check: the skill's assets/ are copies of the Kettle sample apps (the sample apps are canonical).
+# Usage: bash scripts/check-skill-sync.sh <scry-sample-ios dir> <scry-sample-android dir> <scry-sample-flutter dir>
+# Exit 0 = every copy matches its sample and the CI templates keep the G5 guards; 1 = drift (diff shown).
+# The Flutter sample is the third argument; for the Flutter copies of make-scf.mjs (the sample's, iOS's and
+# Android's) the rule is byte-identical to the skill asset (feature flutter-capture, guarantee 6); see the end.
+set -euo pipefail
+[ $# -eq 3 ] || { echo "usage: $0 <ios-sample-dir> <android-sample-dir> <flutter-sample-dir>" >&2; exit 2; }
+IOS="$1"; AND="$2"; FLU="$3"
+ASSETS="$(cd "$(dirname "$0")/../skills/scry-native-capture-setup/assets" && pwd)"
+AK="$AND/app/src"; fail=0
+
+check() { # <asset> <sample file>
+  if [ ! -f "$2" ]; then echo "MISSING sample file: $2"; fail=1
+  elif diff -u "$2" "$ASSETS/$1" >"${TMPDIR:-/tmp}/skill-sync.diff"; then echo "ok     $1 == $2"
+  else echo "DRIFT  $1 differs from $2"; cat "${TMPDIR:-/tmp}/skill-sync.diff"; fail=1; fi
+}
+check ScryLaunch.swift        "$IOS/Kettle/ScryLaunch.swift"
+check ScryScreens.swift       "$IOS/Kettle/ScryScreens.swift"
+check capture-ios.sh          "$IOS/scripts/capture.sh"
+check make-scf.mjs            "$IOS/scripts/make-scf.mjs"
+check scry-capture-ios.yml    "$IOS/.github/workflows/scry-capture.yml"
+check ScryLaunch.kt           "$(ls "$AK"/debug/java/*/*/*/ScryLaunch.kt 2>/dev/null | head -1)"
+check ScryScreens.kt          "$(ls "$AK"/debug/java/*/*/*/ScryScreens.kt 2>/dev/null | head -1)"
+check ScryLaunch-release.kt   "$(ls "$AK"/release/java/*/*/*/ScryLaunch.kt 2>/dev/null | head -1)"
+check capture-android.sh      "$AND/scripts/capture.sh"
+check scry-capture-android.yml "$AND/.github/workflows/scry-capture.yml"
+
+# Flutter (the sample is canonical for logic; screens.dart and screens.json are templates the user fills in, so
+# they only have to exist in both places).
+F="$ASSETS/flutter"
+check flutter/scry_capture_test.dart          "$FLU/integration_test/scry_capture_test.dart"
+check flutter/test_driver_integration_test.dart "$FLU/test_driver/integration_test.dart"
+check flutter/headless_scry_capture_test.dart "$FLU/test/scry_capture_test.dart"
+check flutter/capture.sh                      "$FLU/scripts/capture.sh"
+check flutter/scry_fonts.dart                 "$FLU/test/scry_fonts.dart"
+check flutter/scry_registry_test.dart         "$FLU/test/scry_registry_test.dart"
+for t in screens.dart screens.json; do [ -f "$F/$t" ] && echo "ok     flutter/$t template present" || { echo "MISSING flutter/$t template"; fail=1; }; done
+[ -f "$FLU/integration_test/scry/screens.dart" ] && [ -f "$FLU/scripts/screens.json" ] \
+  && echo "ok     Flutter sample has its registry and screens.json" || { echo "MISSING Flutter sample registry or screens.json"; fail=1; }
+check make-scf.mjs                            "$FLU/scripts/make-scf.mjs"
+# make-scf.mjs is shared and must be byte-identical in the skill and all three samples (the iOS copy is checked above).
+check make-scf.mjs                            "$AND/scripts/make-scf.mjs"
+
+# G5 on the CI templates: default branch only, no pull_request(_target), no self-hosted, key only on upload.
+for y in scry-capture-ios.yml scry-capture-android.yml; do
+  f="$ASSETS/$y"
+  grep -Eq '^\s*(pull_request|pull_request_target)\s*:' "$f" && { echo "G5 FAIL $y: pull_request trigger"; fail=1; }
+  grep -qi 'self-hosted' "$f" && { echo "G5 FAIL $y: self-hosted"; fail=1; }
+  grep -q 'branches: \[main\]' "$f" || { echo "G5 FAIL $y: not limited to a branch"; fail=1; }
+  n="$(grep -c 'secrets\.' "$f")"; [ "$n" -eq 1 ] || { echo "G5 FAIL $y: secrets referenced $n times (want 1, the upload step)"; fail=1; }
+done
+[ "$fail" -eq 0 ] && echo "PASS: skill assets match the samples, CI guards intact" || { echo "FAIL: drift found" >&2; exit 1; }