fix(server): move bcrypt to spawn_blocking, async file I/O, validate JWT_SECRET

Three independent hardening changes: 1. bcrypt on a blocking thread: hash() and verify() are CPU-bound (~300 ms at cost 12). Running them directly on an async task starved the Tokio runtime under concurrent load. Wrapped in spawn_blocking. 2. Async avatar file I/O: std::fs::write/rename/remove_file in an async handler blocks the executor. Replaced with tokio::fs equivalents. 3. JWT_SECRET minimum length: a secret shorter than 32 bytes is fatally weak. validate_jwt_secret() now rejects it at startup with a clear message rather than silently accepting it. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
data(seeds): regenerate difficulty seeds (2026-06-04)
2026-06-08 11:05:45 -07:00 · 2026-06-08 11:05:37 -07:00 · 2026-06-08 11:05:31 -07:00 · 2026-06-08 11:05:23 -07:00
10 changed files with 976 additions and 225 deletions
@@ -0,0 +1,228 @@
+# Android testing
+
+This directory contains lightweight Android test helpers for Ferrous Solitaire.
+They are intended to run against either a physical Android device or an emulator
+connected through `adb`. When no device is connected the smoke script can
+automatically launch an AVD for you.
+
+## Prerequisites
+
+- Android SDK and NDK installed.
+- `adb` available on `PATH`.
+- One device/emulator visible in `adb devices`, **or** at least one AVD created
+  (the script will launch one automatically if `LAUNCH_AVD=1`, which is the default).
+- If multiple devices are connected, set `ADB_SERIAL` to the target device serial.
+- Environment variables required by `scripts/build_android_apk.sh` when building:
+
+```sh
+export ANDROID_HOME=/path/to/android-sdk
+export ANDROID_NDK_HOME=/path/to/android-ndk
+export BUILD_TOOLS_VERSION=34.0.0
+export PLATFORM=android-34
+```
+
+## Smoke test
+
+From the workspace root (`Rusty_Solitaire/`):
+
+```sh
+scripts/android_smoke.sh
+```
+
+The smoke test first checks whether `adb` can see a ready device. If no device
+is connected and `LAUNCH_AVD=1` (default), it:
+
+1. locates the `emulator` binary under `ANDROID_HOME` or `PATH`,
+2. picks the first available AVD (or uses `AVD_NAME`),
+3. launches the emulator in the foreground (or headless with `AVD_HEADLESS=1`),
+4. waits for `sys.boot_completed=1` before proceeding,
+5. dismisses the lock screen so the screenshot shows the app.
+
+Once a device is ready (auto-launched or pre-existing) the script:
+
+1. builds the APK using `scripts/build_android_apk.sh`,
+2. installs it with `adb install -r -d` so debug smoke builds can replace newer local builds,
+3. force-stops the package by default for a clean launch,
+4. clears `logcat`,
+5. launches `com.ferrousapp.solitaire/android.app.NativeActivity`,
+6. waits for the app to settle,
+7. verifies the process is still running,
+8. captures a screenshot and `logcat`, and
+9. fails on fatal log patterns such as native crashes, JNI fatal errors, real ANRs,
+   and Rust panics.
+
+On exit the script kills any emulator it launched (`SHUTDOWN_AVD_ON_EXIT=1` by
+default). Set `SHUTDOWN_AVD_ON_EXIT=0` to keep the emulator open for inspection.
+
+Artifacts are written to `target/android-smoke/<timestamp>/` by default. A successful run includes:
+
+- `device.txt` — selected device and display metadata,
+- `df-data-before.txt` / `df-data-after.txt` — emulator/device storage snapshots,
+- `emulator.log` — stdout/stderr from the emulator process (AVD runs only),
+- `emulator.pid` — PID of the emulator process (AVD runs only),
+- `launch.png` — screenshot after the wait period,
+- `logcat.txt` — full captured log,
+- `log-summary.txt` — grep summary for warnings, errors, JNI, safe-area, and crash terms, and
+- `pid.txt` — running app process id.
+
+## Creating an AVD
+
+If no AVDs exist, create one before running the smoke test:
+
+```sh
+# Install a system image
+"$ANDROID_HOME/cmdline-tools/latest/bin/sdkmanager" \
+  'system-images;android-34;google_apis;x86_64'
+
+# Create the AVD
+"$ANDROID_HOME/cmdline-tools/latest/bin/avdmanager" create avd \
+  -n Pixel_7_API_34 \
+  -k 'system-images;android-34;google_apis;x86_64' \
+  --device 'pixel_7'
+```
+
+Then run the smoke test — it will pick `Pixel_7_API_34` automatically:
+
+```sh
+scripts/android_smoke.sh
+```
+
+## Faster iteration
+
+If you already built the APK and only want to reinstall/relaunch:
+
+```sh
+BUILD_APK=0 scripts/android_smoke.sh
+```
+
+If the APK is already installed and you only want to relaunch/capture logs:
+
+```sh
+BUILD_APK=0 INSTALL_APK=0 scripts/android_smoke.sh
+```
+
+By default the script force-stops the package before launch so logcat and screenshots represent a clean app start. To test warm-launch behavior instead:
+
+```sh
+BUILD_APK=0 INSTALL_APK=0 FORCE_STOP=0 scripts/android_smoke.sh
+```
+
+This is also useful when an already-installed build is good enough for launch/log checks. On install failure, the script writes `adb-install.txt`, storage snapshots, and installed-package diagnostics to the output directory.
+
+If install fails with `INSTALL_FAILED_UPDATE_INCOMPATIBLE`, the smoke script uninstalls the package and retries once by default (`RESET_ON_SIGNATURE_MISMATCH=1`). This resets app data on the device/emulator. Disable it with:
+
+```sh
+RESET_ON_SIGNATURE_MISMATCH=0 scripts/android_smoke.sh
+```
+
+To write artifacts to a stable path:
+
+```sh
+OUT_DIR=target/android-smoke/latest scripts/android_smoke.sh
+```
+
+When reusing an output directory, previous files are removed by default so stale artifacts do not contaminate the latest result. To keep existing files:
+
+```sh
+CLEAN_OUT_DIR=0 OUT_DIR=target/android-smoke/latest scripts/android_smoke.sh
+```
+
+To target a specific device when more than one is attached:
+
+```sh
+ADB_SERIAL=emulator-5554 scripts/android_smoke.sh
+```
+
+To wait longer for safe-area inset polling or slow devices:
+
+```sh
+WAIT_SECS=8 scripts/android_smoke.sh
+```
+
+## AVD options
+
+To pick a specific AVD by name instead of auto-selecting the first one:
+
+```sh
+AVD_NAME=Pixel_7_API_34 scripts/android_smoke.sh
+```
+
+To run headless (no emulator window) — useful in CI or on a display-less machine:
+
+```sh
+AVD_HEADLESS=1 scripts/android_smoke.sh
+```
+
+To give a slow machine more time to boot the emulator (default is 120 s):
+
+```sh
+AVD_BOOT_TIMEOUT=180 scripts/android_smoke.sh
+```
+
+To keep the emulator running after the test (useful for manual inspection):
+
+```sh
+SHUTDOWN_AVD_ON_EXIT=0 scripts/android_smoke.sh
+```
+
+To pass extra flags to the emulator (e.g. disable snapshot for a completely
+cold boot, or change GPU mode):
+
+```sh
+AVD_EXTRA_ARGS="-gpu swiftshader_indirect" scripts/android_smoke.sh
+```
+
+To disable AVD auto-launch entirely and fail immediately if no device is
+connected:
+
+```sh
+LAUNCH_AVD=0 scripts/android_smoke.sh
+```
+
+For build-only validation without requiring a connected device, use the lower-level APK builder directly:
+
+```sh
+scripts/build_android_apk.sh
+```
+
+For smoke testing, `scripts/android_smoke.sh` defaults to the connected device's primary ABI when `BUILD_APK=1`, which keeps emulator APKs much smaller than the full multi-ABI default. You can still override it explicitly:
+
+```sh
+ABIS=x86_64 scripts/android_smoke.sh
+```
+
+For build-only validation, `scripts/build_android_apk.sh` still defaults to all configured ABIs unless you set `ABIS` yourself:
+
+```sh
+ABIS=x86_64 scripts/build_android_apk.sh
+```
+
+The APK builder signs debug builds with a persistent keystore at `target/android/debug.keystore` by default. This avoids signature churn across smoke-test runs.
+
+The APK builder also strips native debug symbols by default before packaging (`STRIP_NATIVE_LIBS=1`). This keeps debug APKs installable on emulators with limited `/data` storage. To preserve native debug symbols for low-level debugging:
+
+```sh
+STRIP_NATIVE_LIBS=0 ABIS=x86_64 scripts/build_android_apk.sh
+```
+
+## Device checklist
+
+The script is only a smoke test. Before shipping Android builds, also verify:
+
+- safe-area insets arrive and shift the HUD after a few seconds,
+- HUD does not overlap the top status bar,
+- modal Done buttons are above the gesture/navigation bar,
+- stock tap works,
+- drag-and-drop works on tableau, waste, and foundation piles,
+- Settings/Help/Profile modals open and close,
+- login tokens persist after app restart, and
+- `target/android-smoke/.../logcat.txt` contains no fatal JNI/native crash output.
+
+## Notes
+
+- `adb shell input tap X Y` uses physical pixels, not Bevy logical pixels.
+- The project’s common test device mapping is physical `1080×2400`, Bevy logical
+  `900×2000`, scale factor `1.20`; multiply logical coordinates by `1.20` for
+  scripted `adb shell input` commands on that device.
+- Keep generated screenshots/logs under `target/android-smoke/` so they stay out
+  of source control.
@@ -0,0 +1,362 @@
+#!/usr/bin/env bash
+# Android smoke test for Ferrous Solitaire.
+#
+# Builds (optional), installs, launches, captures logcat + screenshot, and
+# fails on fatal Android log patterns. Designed as a lightweight device/emulator
+# sanity check rather than a full UI automation suite.
+#
+# Required:
+#   adb on PATH
+#   Android SDK/NDK env required by scripts/build_android_apk.sh when BUILD_APK=1
+#
+# Optional environment:
+#   BUILD_APK=1|0          Build APK before install (default: 1)
+#   INSTALL_APK=1|0        Install APK before launch (default: 1)
+#   RESET_ON_SIGNATURE_MISMATCH=1|0
+#                           Uninstall/retry if debug signatures differ (default: 1)
+#   LAUNCH_APP=1|0         Launch app before checks (default: 1)
+#   FORCE_STOP=1|0         Force-stop package before launch for clean logs (default: 1)
+#   CAPTURE_SCREENSHOT=1|0 Capture screenshot (default: 1)
+#   ADB_SERIAL=...         Device serial to use when multiple devices are connected
+#   APK_PATH=...           APK to install (default: target/debug/apk/ferrous-solitaire.apk)
+#   PACKAGE=...            Android package (default: com.ferrousapp.solitaire)
+#   ACTIVITY=...           Activity class (default: android.app.NativeActivity)
+#   OUT_DIR=...            Artifact directory (default: target/android-smoke/<timestamp>)
+#   CLEAN_OUT_DIR=1|0      Remove prior artifacts from OUT_DIR first (default: 1)
+#   WAIT_SECS=...          Seconds to wait after launch (default: 5)
+#   ABIS=...               Passed to build script. If unset and BUILD_APK=1,
+#                           defaults to the connected device's primary ABI.
+#
+# AVD auto-launch (used when no device/emulator is already connected):
+#   LAUNCH_AVD=1|0         Auto-launch an AVD when no device is ready (default: 1)
+#   AVD_NAME=...           AVD name to launch (default: first from `emulator -list-avds`)
+#   AVD_BOOT_TIMEOUT=...   Seconds to wait for the emulator to finish booting (default: 120)
+#   AVD_HEADLESS=1|0       Run with -no-window -no-audio for CI/no-display environments (default: 0)
+#   AVD_EXTRA_ARGS=...     Extra arguments appended verbatim to the emulator command line
+#   SHUTDOWN_AVD_ON_EXIT=1|0
+#                           Kill the AVD this script launched on exit (default: 1).
+#                           Set to 0 to leave the emulator running after the test.
+#
+# Examples:
+#   scripts/android_smoke.sh
+#   BUILD_APK=0 scripts/android_smoke.sh
+#   LAUNCH_AVD=0 scripts/android_smoke.sh            # error out if no device, never auto-launch
+#   AVD_NAME=Pixel_7_API_34 scripts/android_smoke.sh
+#   AVD_HEADLESS=1 scripts/android_smoke.sh          # CI / no-display
+#   SHUTDOWN_AVD_ON_EXIT=0 scripts/android_smoke.sh  # keep emulator open after test
+#   OUT_DIR=target/android-smoke/latest WAIT_SECS=8 scripts/android_smoke.sh
+set -euo pipefail
+
+REPO_ROOT="$(cd "$(dirname "${BASH_SOURCE[0]}")/.." && pwd)"
+cd "$REPO_ROOT"
+
+BUILD_APK="${BUILD_APK:-1}"
+INSTALL_APK="${INSTALL_APK:-1}"
+RESET_ON_SIGNATURE_MISMATCH="${RESET_ON_SIGNATURE_MISMATCH:-1}"
+LAUNCH_APP="${LAUNCH_APP:-1}"
+FORCE_STOP="${FORCE_STOP:-1}"
+CAPTURE_SCREENSHOT="${CAPTURE_SCREENSHOT:-1}"
+APK_PATH="${APK_PATH:-target/debug/apk/ferrous-solitaire.apk}"
+PACKAGE="${PACKAGE:-com.ferrousapp.solitaire}"
+ACTIVITY="${ACTIVITY:-android.app.NativeActivity}"
+WAIT_SECS="${WAIT_SECS:-5}"
+OUT_DIR="${OUT_DIR:-target/android-smoke/$(date +%Y%m%d-%H%M%S)}"
+CLEAN_OUT_DIR="${CLEAN_OUT_DIR:-1}"
+REMOTE_SCREENSHOT="/sdcard/ferrous-solitaire-smoke.png"
+
+LAUNCH_AVD="${LAUNCH_AVD:-1}"
+AVD_NAME="${AVD_NAME:-}"
+AVD_BOOT_TIMEOUT="${AVD_BOOT_TIMEOUT:-120}"
+AVD_HEADLESS="${AVD_HEADLESS:-0}"
+AVD_EXTRA_ARGS="${AVD_EXTRA_ARGS:-}"
+SHUTDOWN_AVD_ON_EXIT="${SHUTDOWN_AVD_ON_EXIT:-1}"
+
+ADB=(adb)
+if [ -n "${ADB_SERIAL:-}" ]; then
+  ADB+=( -s "$ADB_SERIAL" )
+fi
+
+# PID of any emulator we start so the EXIT trap can clean it up.
+_LAUNCHED_EMULATOR_PID=""
+
+_cleanup_emulator() {
+  if [ -n "$_LAUNCHED_EMULATOR_PID" ] && [ "$SHUTDOWN_AVD_ON_EXIT" = "1" ]; then
+    echo ">>> shutdown emulator (PID $_LAUNCHED_EMULATOR_PID)"
+    kill "$_LAUNCHED_EMULATOR_PID" 2>/dev/null || true
+  fi
+}
+trap _cleanup_emulator EXIT
+
+require_cmd() {
+  command -v "$1" >/dev/null 2>&1 || {
+    echo "missing required command: $1" >&2
+    exit 1
+  }
+}
+
+mkdir -p "$OUT_DIR"
+if [ "$CLEAN_OUT_DIR" = "1" ]; then
+  find "$OUT_DIR" -mindepth 1 -maxdepth 1 -exec rm -rf {} +
+fi
+require_cmd adb
+
+# ---------------------------------------------------------------------------
+# Device / emulator availability
+# ---------------------------------------------------------------------------
+DEVICE_STATE="$("${ADB[@]}" get-state 2>/dev/null || true)"
+if [ "$DEVICE_STATE" != "device" ]; then
+  if [ "$LAUNCH_AVD" != "1" ]; then
+    adb devices > "$OUT_DIR/adb-devices.txt" 2>&1 || true
+    if [ -n "${ADB_SERIAL:-}" ]; then
+      echo "Android device '$ADB_SERIAL' is not connected/ready (state: ${DEVICE_STATE:-unknown})." >&2
+    else
+      echo "No Android device/emulator is connected and ready." >&2
+    fi
+    echo "Run 'adb devices' or start an emulator, then retry." >&2
+    echo "Device list saved to $OUT_DIR/adb-devices.txt" >&2
+    exit 1
+  fi
+
+  # --- locate emulator binary -----------------------------------------------
+  # Priority: ANDROID_HOME env → PATH → common SDK install locations.
+  _find_sdk_root() {
+    for candidate in \
+        "$HOME/Android/Sdk" \
+        "$HOME/Library/Android/sdk" \
+        "/opt/android-sdk" \
+        "/usr/lib/android-sdk"; do
+      [ -d "$candidate" ] && echo "$candidate" && return
+    done
+  }
+
+  EMULATOR_BIN=""
+  if [ -n "${ANDROID_HOME:-}" ] && [ -x "$ANDROID_HOME/emulator/emulator" ]; then
+    EMULATOR_BIN="$ANDROID_HOME/emulator/emulator"
+  elif command -v emulator >/dev/null 2>&1; then
+    EMULATOR_BIN="$(command -v emulator)"
+  else
+    _SDK_ROOT="$(_find_sdk_root)"
+    if [ -n "$_SDK_ROOT" ] && [ -x "$_SDK_ROOT/emulator/emulator" ]; then
+      EMULATOR_BIN="$_SDK_ROOT/emulator/emulator"
+    fi
+  fi
+
+  if [ -z "$EMULATOR_BIN" ]; then
+    echo "No Android device found and 'emulator' binary is not available." >&2
+    echo "  • Install the Android SDK emulator component, or" >&2
+    echo "  • Set ANDROID_HOME to your SDK root, or" >&2
+    echo "  • Start a device/emulator manually then retry with LAUNCH_AVD=0." >&2
+    exit 1
+  fi
+  echo ">>> emulator binary: $EMULATOR_BIN"
+
+  # --- select AVD -----------------------------------------------------------
+  if [ -z "$AVD_NAME" ]; then
+    AVD_NAME="$("$EMULATOR_BIN" -list-avds 2>/dev/null | head -n 1 | tr -d '\r')"
+    if [ -z "$AVD_NAME" ]; then
+      echo "No AVDs found. Create one first, for example:" >&2
+      echo "  sdkmanager 'system-images;android-34;google_apis;x86_64'" >&2
+      echo "  avdmanager create avd -n Pixel_7_API_34 \\" >&2
+      echo "    -k 'system-images;android-34;google_apis;x86_64' --device 'pixel_7'" >&2
+      exit 1
+    fi
+    echo ">>> auto-selected AVD: $AVD_NAME"
+  fi
+
+  # --- launch emulator -------------------------------------------------------
+  EMULATOR_ARGS=( -avd "$AVD_NAME" -no-snapshot-load )
+  [ "$AVD_HEADLESS" = "1" ] && EMULATOR_ARGS+=( -no-window -no-audio )
+  # Split AVD_EXTRA_ARGS on whitespace only (disable glob expansion).
+  set -f
+  # shellcheck disable=SC2206
+  [ -n "$AVD_EXTRA_ARGS" ] && EMULATOR_ARGS+=( $AVD_EXTRA_ARGS )
+  set +f
+
+  echo ">>> launch emulator: $AVD_NAME"
+  "$EMULATOR_BIN" "${EMULATOR_ARGS[@]}" > "$OUT_DIR/emulator.log" 2>&1 &
+  _LAUNCHED_EMULATOR_PID=$!
+  echo "$_LAUNCHED_EMULATOR_PID" > "$OUT_DIR/emulator.pid"
+  echo "    emulator PID: $_LAUNCHED_EMULATOR_PID"
+  echo "    emulator log: $OUT_DIR/emulator.log"
+
+  # --- wait for adb transport -----------------------------------------------
+  # Poll adb get-state (≠ wait-for-device which blocks indefinitely) so we can
+  # honour AVD_BOOT_TIMEOUT for the whole boot sequence.
+  echo ">>> waiting for device to appear in adb (timeout: ${AVD_BOOT_TIMEOUT}s)"
+  _ELAPSED=0
+  while true; do
+    _STATE="$("${ADB[@]}" get-state 2>/dev/null || true)"
+    if [ "$_STATE" = "device" ] || [ "$_STATE" = "offline" ]; then
+      break
+    fi
+    if [ "$_ELAPSED" -ge "$AVD_BOOT_TIMEOUT" ]; then
+      echo "Device did not appear in adb within ${AVD_BOOT_TIMEOUT}s" >&2
+      echo "emulator log:" >&2
+      tail -20 "$OUT_DIR/emulator.log" >&2 || true
+      exit 1
+    fi
+    sleep 3
+    _ELAPSED=$(( _ELAPSED + 3 ))
+    echo "    ... ${_ELAPSED}s / ${AVD_BOOT_TIMEOUT}s"
+  done
+
+  # Capture emulator serial (emulator-5554 etc.) so all subsequent adb calls
+  # target the right device when ADB_SERIAL was not set by the caller.
+  if [ -z "${ADB_SERIAL:-}" ]; then
+    _EMU_SERIAL="$(adb devices 2>/dev/null | awk '/^emulator-/{print $1; exit}' | tr -d '\r')"
+    if [ -n "$_EMU_SERIAL" ]; then
+      ADB_SERIAL="$_EMU_SERIAL"
+      ADB=(adb -s "$ADB_SERIAL")
+      echo ">>> detected emulator serial: $ADB_SERIAL"
+    fi
+  fi
+
+  # --- wait for full Android boot -------------------------------------------
+  # adb get-state returning "device" means the transport is up, but the
+  # Android framework may still be initialising. Poll sys.boot_completed.
+  echo ">>> waiting for boot_completed (timeout: ${AVD_BOOT_TIMEOUT}s)"
+  _BOOT_ELAPSED=0
+  _BOOT_INTERVAL=5
+  while true; do
+    _BOOT="$("${ADB[@]}" shell getprop sys.boot_completed 2>/dev/null | tr -d '\r')"
+    if [ "$_BOOT" = "1" ]; then
+      echo ">>> emulator boot complete"
+      break
+    fi
+    if [ "$_BOOT_ELAPSED" -ge "$AVD_BOOT_TIMEOUT" ]; then
+      echo "Emulator did not finish booting within ${AVD_BOOT_TIMEOUT}s" >&2
+      echo "emulator log:" >&2
+      tail -20 "$OUT_DIR/emulator.log" >&2 || true
+      exit 1
+    fi
+    sleep "$_BOOT_INTERVAL"
+    _BOOT_ELAPSED=$(( _BOOT_ELAPSED + _BOOT_INTERVAL ))
+    echo "    ... ${_BOOT_ELAPSED}s / ${AVD_BOOT_TIMEOUT}s (boot_completed='${_BOOT}')"
+  done
+
+  # Dismiss the lock screen so later screencap shows the app, not the keyguard.
+  "${ADB[@]}" shell input keyevent 82 2>/dev/null || true
+
+  # Final sanity check — device must be fully ready before we proceed.
+  DEVICE_STATE="$("${ADB[@]}" get-state 2>/dev/null || true)"
+  if [ "$DEVICE_STATE" != "device" ]; then
+    echo "Emulator is running but adb state is '${DEVICE_STATE:-unknown}'." >&2
+    exit 1
+  fi
+fi
+
+# ---------------------------------------------------------------------------
+# Device metadata
+# ---------------------------------------------------------------------------
+{
+  echo "adb_serial=${ADB_SERIAL:-default}"
+  echo "package=$PACKAGE"
+  echo "activity=$ACTIVITY"
+  echo "device_state=$DEVICE_STATE"
+  "${ADB[@]}" shell getprop ro.product.model 2>/dev/null | tr -d '\r' | sed 's/^/product_model=/'
+  "${ADB[@]}" shell getprop ro.build.version.release 2>/dev/null | tr -d '\r' | sed 's/^/android_release=/'
+  "${ADB[@]}" shell getprop ro.build.version.sdk 2>/dev/null | tr -d '\r' | sed 's/^/android_sdk=/'
+  "${ADB[@]}" shell wm size 2>/dev/null | tr -d '\r' | sed 's/^/wm_size=/'
+  "${ADB[@]}" shell wm density 2>/dev/null | tr -d '\r' | sed 's/^/wm_density=/'
+} > "$OUT_DIR/device.txt"
+"${ADB[@]}" shell df -h /data > "$OUT_DIR/df-data-before.txt" 2>&1 || true
+
+if [ "$BUILD_APK" = "1" ]; then
+  if [ -z "${ABIS:-}" ]; then
+    DEVICE_ABI="$("${ADB[@]}" shell getprop ro.product.cpu.abi 2>/dev/null | tr -d '\r')"
+    case "$DEVICE_ABI" in
+      x86_64|arm64-v8a|armeabi-v7a)
+        export ABIS="$DEVICE_ABI"
+        ;;
+      armeabi*)
+        export ABIS="armeabi-v7a"
+        ;;
+      *)
+        echo "Could not map device ABI '$DEVICE_ABI'; using build script default ABIS." >&2
+        ;;
+    esac
+  fi
+  echo ">>> build Android APK${ABIS:+ (ABIS=$ABIS)}"
+  scripts/build_android_apk.sh
+fi
+
+if [ "$INSTALL_APK" = "1" ]; then
+  [ -f "$APK_PATH" ] || {
+    echo "APK not found: $APK_PATH" >&2
+    echo "Set APK_PATH or run with BUILD_APK=1." >&2
+    exit 1
+  }
+  ls -lh "$APK_PATH" > "$OUT_DIR/apk.txt"
+  echo ">>> install $APK_PATH"
+  if ! "${ADB[@]}" install -r -d "$APK_PATH" > "$OUT_DIR/adb-install.txt" 2>&1; then
+    if [ "$RESET_ON_SIGNATURE_MISMATCH" = "1" ] && grep -q "INSTALL_FAILED_UPDATE_INCOMPATIBLE" "$OUT_DIR/adb-install.txt"; then
+      echo ">>> signature mismatch; uninstalling $PACKAGE and retrying install"
+      "${ADB[@]}" uninstall "$PACKAGE" > "$OUT_DIR/adb-uninstall-before-retry.txt" 2>&1 || true
+      if "${ADB[@]}" install -r -d "$APK_PATH" > "$OUT_DIR/adb-install-retry.txt" 2>&1; then
+        cat "$OUT_DIR/adb-install-retry.txt" >> "$OUT_DIR/adb-install.txt"
+      else
+        cat "$OUT_DIR/adb-install.txt" >&2
+        cat "$OUT_DIR/adb-install-retry.txt" >&2
+        "${ADB[@]}" shell df -h /data > "$OUT_DIR/df-data-after-install-failure.txt" 2>&1 || true
+        "${ADB[@]}" shell pm list packages | grep -F "$PACKAGE" > "$OUT_DIR/installed-package.txt" 2>&1 || true
+        echo "APK install retry failed. Diagnostics saved in $OUT_DIR" >&2
+        exit 1
+      fi
+    else
+      cat "$OUT_DIR/adb-install.txt" >&2
+      "${ADB[@]}" shell df -h /data > "$OUT_DIR/df-data-after-install-failure.txt" 2>&1 || true
+      "${ADB[@]}" shell pm list packages | grep -F "$PACKAGE" > "$OUT_DIR/installed-package.txt" 2>&1 || true
+      echo "APK install failed. Diagnostics saved in $OUT_DIR" >&2
+      echo "If the package is already installed and you only need launch/log checks, retry with INSTALL_APK=0." >&2
+      exit 1
+    fi
+  fi
+fi
+
+if [ "$FORCE_STOP" = "1" ]; then
+  echo ">>> force-stop $PACKAGE"
+  "${ADB[@]}" shell am force-stop "$PACKAGE" || true
+fi
+
+echo ">>> clear logcat"
+"${ADB[@]}" logcat -c
+
+if [ "$LAUNCH_APP" = "1" ]; then
+  echo ">>> launch $PACKAGE/$ACTIVITY"
+  "${ADB[@]}" shell am start -n "$PACKAGE/$ACTIVITY" > "$OUT_DIR/am-start.txt"
+fi
+
+echo ">>> wait ${WAIT_SECS}s"
+sleep "$WAIT_SECS"
+
+PID="$("${ADB[@]}" shell pidof "$PACKAGE" | tr -d '\r' || true)"
+if [ -z "$PID" ]; then
+  "${ADB[@]}" logcat -d > "$OUT_DIR/logcat.txt" || true
+  echo "app process is not running after launch: $PACKAGE" >&2
+  echo "logcat saved to $OUT_DIR/logcat.txt" >&2
+  exit 1
+fi
+echo "$PID" > "$OUT_DIR/pid.txt"
+
+if [ "$CAPTURE_SCREENSHOT" = "1" ]; then
+  echo ">>> capture screenshot"
+  "${ADB[@]}" shell screencap -p "$REMOTE_SCREENSHOT"
+  "${ADB[@]}" pull "$REMOTE_SCREENSHOT" "$OUT_DIR/launch.png" >/dev/null
+  "${ADB[@]}" shell rm -f "$REMOTE_SCREENSHOT" >/dev/null 2>&1 || true
+fi
+
+echo ">>> capture logcat"
+"${ADB[@]}" logcat -d > "$OUT_DIR/logcat.txt"
+grep -iE "panic|fatal|jni|native crash|\bANR\b|exception|error|warn|keystore|safe_area" "$OUT_DIR/logcat.txt" > "$OUT_DIR/log-summary.txt" || true
+"${ADB[@]}" shell df -h /data > "$OUT_DIR/df-data-after.txt" 2>&1 || true
+
+# Fatal patterns only. Avoid matching generic "error" because Android logs are
+# noisy and many non-fatal framework lines contain that word.
+if grep -iE "fatal exception|jni detected error|native crash|signal [0-9]+|ANR in|Application Not Responding|Input dispatching timed out|thread exiting with uncaught exception|panicked at" "$OUT_DIR/logcat.txt"; then
+  echo "Android smoke test found fatal log output" >&2
+  echo "Artifacts saved in $OUT_DIR" >&2
+  exit 1
+fi
+
+echo ">>> Android smoke test passed"
+echo "Artifacts saved in $OUT_DIR"
@@ -6,11 +6,15 @@
 # ndk-build crate that we couldn't isolate; running each Android toolchain
 # step explicitly gives us a debuggable pipeline.
 #
-# Required environment:
-#   ANDROID_HOME            Path to Android SDK root
-#   ANDROID_NDK_HOME        Path to the specific NDK version
-#   BUILD_TOOLS_VERSION     e.g. "34.0.0"
-#   PLATFORM                e.g. "android-34"
+# Environment:
+#   ANDROID_HOME            Path to Android SDK root. If unset, common SDK
+#                           locations such as ~/Android/Sdk are tried.
+#   ANDROID_NDK_HOME        Path to the specific NDK version. If unset, the
+#                           newest $ANDROID_HOME/ndk/* directory is used.
+#   BUILD_TOOLS_VERSION     e.g. "34.0.0". If unset, newest installed build-tools
+#                           version is used.
+#   PLATFORM                e.g. "android-34". If unset, newest installed
+#                           $ANDROID_HOME/platforms/android-* platform is used.
 #
 # Optional environment:
 #   PROFILE                 "debug" (default) | "release"
@@ -19,7 +23,8 @@
 #                           fit the runner's disk budget — a full three-ABI
 #                           debug build can exceed 25 GB of target/ output.
 #   APK_OUT                 Output APK path (default: target/$PROFILE/apk/ferrous-solitaire.apk)
-#   KEYSTORE                Path to keystore for signing (default: generates a debug keystore)
+#   STRIP_NATIVE_LIBS       1 to strip .so files before packaging (default: 1)
+#   KEYSTORE                Path to keystore for signing (default: target/android/debug.keystore)
 #   KEYSTORE_PASS           Keystore password (default: "android" for the generated debug keystore)
 #   KEY_ALIAS               Key alias (default: "androiddebugkey")
 #   KEY_PASS                Key password (default: same as KEYSTORE_PASS)
@@ -28,18 +33,63 @@
 #   $APK_OUT                Signed, zipaligned APK
 set -euo pipefail

-: "${ANDROID_HOME:?ANDROID_HOME must be set}"
-: "${ANDROID_NDK_HOME:?ANDROID_NDK_HOME must be set}"
-: "${BUILD_TOOLS_VERSION:?BUILD_TOOLS_VERSION must be set}"
-: "${PLATFORM:?PLATFORM must be set (e.g. android-34)}"
+infer_latest_dir_name() {
+  local pattern="$1"
+  local latest=""
+  shopt -s nullglob
+  local dirs=( $pattern )
+  shopt -u nullglob
+  if [ ${#dirs[@]} -gt 0 ]; then
+    latest="$(printf '%s\n' "${dirs[@]}" | sort -V | tail -n 1)"
+    basename "$latest"
+  fi
+}
+
+if [ -z "${ANDROID_HOME:-}" ]; then
+  for candidate in "$HOME/Android/Sdk" "$HOME/Library/Android/sdk" "/opt/android-sdk" "/usr/lib/android-sdk"; do
+    if [ -d "$candidate" ]; then
+      ANDROID_HOME="$candidate"
+      export ANDROID_HOME
+      break
+    fi
+  done
+fi
+: "${ANDROID_HOME:?ANDROID_HOME must be set or discoverable under a common SDK path}"
+
+if [ -z "${ANDROID_NDK_HOME:-}" ]; then
+  NDK_VERSION="$(infer_latest_dir_name "$ANDROID_HOME/ndk/*")"
+  if [ -n "$NDK_VERSION" ]; then
+    ANDROID_NDK_HOME="$ANDROID_HOME/ndk/$NDK_VERSION"
+    export ANDROID_NDK_HOME
+  fi
+fi
+: "${ANDROID_NDK_HOME:?ANDROID_NDK_HOME must be set or discoverable under ANDROID_HOME/ndk}"
+
+if [ -z "${BUILD_TOOLS_VERSION:-}" ]; then
+  BUILD_TOOLS_VERSION="$(infer_latest_dir_name "$ANDROID_HOME/build-tools/*")"
+  export BUILD_TOOLS_VERSION
+fi
+: "${BUILD_TOOLS_VERSION:?BUILD_TOOLS_VERSION must be set or discoverable under ANDROID_HOME/build-tools}"
+
+if [ -z "${PLATFORM:-}" ]; then
+  PLATFORM="$(infer_latest_dir_name "$ANDROID_HOME/platforms/android-*")"
+  export PLATFORM
+fi
+: "${PLATFORM:?PLATFORM must be set or discoverable under ANDROID_HOME/platforms}"

 PROFILE="${PROFILE:-debug}"
 ABIS="${ABIS:-arm64-v8a armeabi-v7a x86_64}"
 APK_OUT="${APK_OUT:-target/${PROFILE}/apk/ferrous-solitaire.apk}"
+STRIP_NATIVE_LIBS="${STRIP_NATIVE_LIBS:-1}"

 REPO_ROOT="$(cd "$(dirname "${BASH_SOURCE[0]}")/.." && pwd)"
 cd "$REPO_ROOT"

+echo ">>> Android SDK: $ANDROID_HOME"
+echo ">>> Android NDK: $ANDROID_NDK_HOME"
+echo ">>> Build tools: $BUILD_TOOLS_VERSION"
+echo ">>> Platform: $PLATFORM"
+
 BT="$ANDROID_HOME/build-tools/$BUILD_TOOLS_VERSION"
 PLATFORM_JAR="$ANDROID_HOME/platforms/$PLATFORM/android.jar"
 MANIFEST="solitaire_app/android/AndroidManifest.xml"
@@ -69,6 +119,24 @@ fi
 echo ">>> cargo ndk ${CARGO_NDK_ARGS[*]}"
 cargo ndk "${CARGO_NDK_ARGS[@]}"

+if [ "$STRIP_NATIVE_LIBS" = "1" ]; then
+  LLVM_STRIP=""
+  shopt -s nullglob
+  STRIP_CANDIDATES=( "$ANDROID_NDK_HOME"/toolchains/llvm/prebuilt/*/bin/llvm-strip )
+  shopt -u nullglob
+  if [ ${#STRIP_CANDIDATES[@]} -gt 0 ]; then
+    LLVM_STRIP="${STRIP_CANDIDATES[0]}"
+  fi
+  if [ -z "$LLVM_STRIP" ]; then
+    echo "llvm-strip not found under ANDROID_NDK_HOME; native libraries will remain unstripped" >&2
+  else
+    echo ">>> strip native libraries with $LLVM_STRIP"
+    find "$STAGING/lib" -name '*.so' -print0 | while IFS= read -r -d '' so; do
+      "$LLVM_STRIP" --strip-debug "$so"
+    done
+  fi
+fi
+
 # --- 2. compile + link resources and manifest ------------------------------
 if [ -d "$RES_DIR" ]; then
  echo ">>> aapt2 compile resources"
@@ -120,11 +188,15 @@ rm -f "$STAGING/app-unsigned.apk"

 # --- 5. sign ---------------------------------------------------------------
 if [ -z "${KEYSTORE:-}" ]; then
-  # Generate a deterministic debug keystore on the fly.
-  KEYSTORE="$STAGING/debug.keystore"
-  KEYSTORE_PASS="${KEYSTORE_PASS:-android}"
-  KEY_ALIAS="${KEY_ALIAS:-androiddebugkey}"
-  KEY_PASS="${KEY_PASS:-$KEYSTORE_PASS}"
+  KEYSTORE="target/android/debug.keystore"
+fi
+
+KEYSTORE_PASS="${KEYSTORE_PASS:-android}"
+KEY_ALIAS="${KEY_ALIAS:-androiddebugkey}"
+KEY_PASS="${KEY_PASS:-$KEYSTORE_PASS}"
+
+if [ ! -f "$KEYSTORE" ]; then
+  mkdir -p "$(dirname "$KEYSTORE")"
  echo ">>> generating debug keystore at $KEYSTORE"
  keytool -genkeypair -v \
    -keystore "$KEYSTORE" \
@@ -65,10 +65,9 @@ pub fn run() {
    // operations will fail gracefully with TokenError::KeychainUnavailable.
    //
    // Android: `keyring` isn't compiled in (its `rpassword` transitive
-    // pulls a libc symbol Android's bionic doesn't expose). `auth_tokens`
-    // ships an Android stub that returns KeychainUnavailable for every
-    // call — the runtime behaviour is "session login required each launch"
-    // until we wire Android Keystore via JNI in the Phase-Android round.
+    // pulls a libc symbol Android's bionic doesn't expose). The Android
+    // auth-token path uses Android Keystore via JNI; `android_main` passes
+    // the process JavaVM pointer into `solitaire_data` before `run()`.
    #[cfg(not(target_os = "android"))]
    if let Err(e) = keyring::use_native_store(true) {
        eprintln!(
@@ -366,6 +365,10 @@ fn set_window_icon(
 #[cfg(target_os = "android")]
 #[unsafe(no_mangle)]
 fn android_main(android_app: bevy::android::android_activity::AndroidApp) {
+    let vm_ptr = android_app.vm_as_ptr().cast();
+    if let Err(e) = solitaire_data::init_android_jvm(vm_ptr) {
+        eprintln!("warn: could not initialise Android Keystore JNI ({e})");
+    }
    let _ = bevy::android::ANDROID_APP.set(android_app);
    run();
 }
@@ -19,11 +19,14 @@ use jni::{
 };
 use serde::{Deserialize, Serialize};
 use std::collections::HashMap;
+use std::ffi::c_void;
 use std::path::PathBuf;
+use std::sync::OnceLock;

 use crate::auth_tokens::TokenError;

 const KEY_ALIAS: &str = "ferrous_solitaire_token_key";
+static ANDROID_JVM: OnceLock<JavaVM> = OnceLock::new();

 #[derive(Serialize, Deserialize)]
 struct TokenBlob {
@@ -36,17 +39,37 @@ struct TokenBlob {
 // JVM helper
 // ---------------------------------------------------------------------------

+/// Initialise Android Keystore access with the process-wide `JavaVM*`.
+///
+/// This is called by `solitaire_app` from Android startup code. Keeping the
+/// raw JVM pointer here avoids making `solitaire_data` depend on the app or
+/// engine layer just to reach platform startup state.
+pub fn init_android_jvm(vm_ptr: *mut c_void) -> Result<(), TokenError> {
+    if vm_ptr.is_null() {
+        return Err(TokenError::KeychainUnavailable(
+            "JavaVM pointer is null".into(),
+        ));
+    }
+    if ANDROID_JVM.get().is_some() {
+        return Ok(());
+    }
+
+    // SAFETY: `vm_ptr` is supplied by Android startup code and must be the
+    // process-wide JavaVM* for this app. `OnceLock` keeps the wrapper alive for
+    // the process lifetime.
+    let vm = unsafe { JavaVM::from_raw(vm_ptr.cast()) }
+        .map_err(|e| TokenError::Keyring(format!("JavaVM: {e}")))?;
+    let _ = ANDROID_JVM.set(vm);
+    Ok(())
+}
+
 fn with_jvm<F, R>(f: F) -> Result<R, TokenError>
 where
    F: for<'env> FnOnce(&mut JNIEnv<'env>) -> Result<R, jni::errors::Error>,
 {
-    let app = bevy::android::ANDROID_APP
+    let vm = ANDROID_JVM
        .get()
-        .ok_or_else(|| TokenError::KeychainUnavailable("ANDROID_APP not initialised".into()))?;
-
-    // SAFETY: vm_as_ptr() is the process-wide JavaVM* set by the Android runtime.
-    let vm = unsafe { JavaVM::from_raw(app.vm_as_ptr().cast()) }
-        .map_err(|e| TokenError::Keyring(format!("JavaVM: {e}")))?;
+        .ok_or_else(|| TokenError::KeychainUnavailable("Android JavaVM not initialised".into()))?;

    let mut env = vm
        .attach_current_thread_permanently()
@@ -14,15 +14,13 @@
 //! the Bevy `App`). If no default store is set, all operations in this module
 //! will return [`TokenError::KeychainUnavailable`].
 //!
-//! # Android stub
+//! # Android
 //!
 //! `keyring-core` cannot compile for the android target (its `rpassword`
 //! transitive dep uses `libc::__errno_location`, which Android's bionic
-//! doesn't expose). On Android every function in this module returns
-//! [`TokenError::KeychainUnavailable`] so callers can detect the fallback
-//! the same way they handle a Linux box without Secret Service. The
-//! real Android backend will arrive in the Phase-Android round when we
-//! wire Android Keystore via JNI.
+//! doesn't expose). On Android this module delegates to an Android Keystore
+//! JNI backend. `solitaire_app` must call `solitaire_data::init_android_jvm`
+//! from Android startup before token operations can succeed.
 //!
 //! # Note: no unit tests — requires live OS keychain.

@@ -26,227 +26,227 @@ use solitaire_core::game_state::DifficultyLevel;

 /// 40 seeds proven winnable within the Easy budget (≤ 1 000 states).
 pub const EASY_SEEDS: &[u64] = &[
-    // Generated by solitaire_assetgen::gen_difficulty_seeds (tier=Easy, date=2026-05-09)
-    0xD1FF_0000_0000_0001,
-    0xD1FF_0000_0000_0002,
-    0xD1FF_0000_0000_0007,
-    0xD1FF_0000_0000_0008,
+    // Generated by solitaire_assetgen::gen_difficulty_seeds (tier=Easy, date=2026-06-04)
    0xD1FF_0000_0000_0009,
-    0xD1FF_0000_0000_000E,
-    0xD1FF_0000_0000_0013,
-    0xD1FF_0000_0000_0015,
-    0xD1FF_0000_0000_0018,
-    0xD1FF_0000_0000_001D,
-    0xD1FF_0000_0000_0021,
-    0xD1FF_0000_0000_0022,
-    0xD1FF_0000_0000_0026,
-    0xD1FF_0000_0000_002C,
-    0xD1FF_0000_0000_002E,
-    0xD1FF_0000_0000_002F,
-    0xD1FF_0000_0000_0035,
-    0xD1FF_0000_0000_0036,
-    0xD1FF_0000_0000_003C,
-    0xD1FF_0000_0000_0045,
-    0xD1FF_0000_0000_0046,
-    0xD1FF_0000_0000_0048,
-    0xD1FF_0000_0000_0049,
-    0xD1FF_0000_0000_004D,
-    0xD1FF_0000_0000_004F,
-    0xD1FF_0000_0000_0050,
-    0xD1FF_0000_0000_0051,
-    0xD1FF_0000_0000_0053,
-    0xD1FF_0000_0000_0054,
-    0xD1FF_0000_0000_0057,
-    0xD1FF_0000_0000_0058,
-    0xD1FF_0000_0000_005A,
-    0xD1FF_0000_0000_005B,
-    0xD1FF_0000_0000_005C,
-    0xD1FF_0000_0000_005D,
-    0xD1FF_0000_0000_005F,
-    0xD1FF_0000_0000_0061,
-    0xD1FF_0000_0000_0062,
-    0xD1FF_0000_0000_0063,
-    0xD1FF_0000_0000_0069,
+    0xD1FF_0000_0000_0087,
+    0xD1FF_0000_0000_00EB,
+    0xD1FF_0000_0000_017F,
+    0xD1FF_0000_0000_01CE,
+    0xD1FF_0000_0000_020F,
+    0xD1FF_0000_0000_0251,
+    0xD1FF_0000_0000_0275,
+    0xD1FF_0000_0000_029C,
+    0xD1FF_0000_0000_02BD,
+    0xD1FF_0000_0000_02ED,
+    0xD1FF_0000_0000_038F,
+    0xD1FF_0000_0000_03C9,
+    0xD1FF_0000_0000_0415,
+    0xD1FF_0000_0000_045F,
+    0xD1FF_0000_0000_04C4,
+    0xD1FF_0000_0000_04CC,
+    0xD1FF_0000_0000_04EE,
+    0xD1FF_0000_0000_0631,
+    0xD1FF_0000_0000_0651,
+    0xD1FF_0000_0000_0689,
+    0xD1FF_0000_0000_0735,
+    0xD1FF_0000_0000_0748,
+    0xD1FF_0000_0000_0801,
+    0xD1FF_0000_0000_0820,
+    0xD1FF_0000_0000_08F9,
+    0xD1FF_0000_0000_091C,
+    0xD1FF_0000_0000_0937,
+    0xD1FF_0000_0000_09A6,
+    0xD1FF_0000_0000_09C3,
+    0xD1FF_0000_0000_09DD,
+    0xD1FF_0000_0000_0BD9,
+    0xD1FF_0000_0000_0BEC,
+    0xD1FF_0000_0000_0BF2,
+    0xD1FF_0000_0000_0C1B,
+    0xD1FF_0000_0000_0C26,
+    0xD1FF_0000_0000_0C36,
+    0xD1FF_0000_0000_0C4B,
+    0xD1FF_0000_0000_0C78,
+    0xD1FF_0000_0000_0CBC,
 ];

 /// 40 seeds proven winnable within the Medium budget (≤ 5 000 states).
 pub const MEDIUM_SEEDS: &[u64] = &[
-    // Generated by solitaire_assetgen::gen_difficulty_seeds (tier=Medium, date=2026-05-09)
-    0xD1FF_0000_0000_0000,
+    // Generated by solitaire_assetgen::gen_difficulty_seeds (tier=Medium, date=2026-06-04)
    0xD1FF_0000_0000_0012,
-    0xD1FF_0000_0000_0016,
-    0xD1FF_0000_0000_001B,
-    0xD1FF_0000_0000_001C,
-    0xD1FF_0000_0000_0020,
-    0xD1FF_0000_0000_002A,
-    0xD1FF_0000_0000_0034,
-    0xD1FF_0000_0000_003A,
-    0xD1FF_0000_0000_0041,
-    0xD1FF_0000_0000_0043,
-    0xD1FF_0000_0000_0060,
-    0xD1FF_0000_0000_006A,
-    0xD1FF_0000_0000_006C,
-    0xD1FF_0000_0000_006E,
-    0xD1FF_0000_0000_006F,
-    0xD1FF_0000_0000_0071,
-    0xD1FF_0000_0000_0072,
-    0xD1FF_0000_0000_0075,
-    0xD1FF_0000_0000_0076,
-    0xD1FF_0000_0000_007B,
-    0xD1FF_0000_0000_007E,
-    0xD1FF_0000_0000_0081,
-    0xD1FF_0000_0000_0083,
-    0xD1FF_0000_0000_0084,
-    0xD1FF_0000_0000_0087,
-    0xD1FF_0000_0000_0090,
-    0xD1FF_0000_0000_0092,
-    0xD1FF_0000_0000_0093,
-    0xD1FF_0000_0000_0098,
+    0xD1FF_0000_0000_002C,
+    0xD1FF_0000_0000_004B,
+    0xD1FF_0000_0000_0052,
+    0xD1FF_0000_0000_0058,
+    0xD1FF_0000_0000_005E,
+    0xD1FF_0000_0000_0063,
    0xD1FF_0000_0000_0099,
-    0xD1FF_0000_0000_009A,
-    0xD1FF_0000_0000_009E,
-    0xD1FF_0000_0000_00A5,
-    0xD1FF_0000_0000_00A8,
-    0xD1FF_0000_0000_00AA,
-    0xD1FF_0000_0000_00AB,
-    0xD1FF_0000_0000_00AE,
+    0xD1FF_0000_0000_00A9,
    0xD1FF_0000_0000_00AF,
-    0xD1FF_0000_0000_00B0,
+    0xD1FF_0000_0000_00BB,
+    0xD1FF_0000_0000_00D1,
+    0xD1FF_0000_0000_00E3,
+    0xD1FF_0000_0000_0108,
+    0xD1FF_0000_0000_010D,
+    0xD1FF_0000_0000_0110,
+    0xD1FF_0000_0000_012F,
+    0xD1FF_0000_0000_0139,
+    0xD1FF_0000_0000_013C,
+    0xD1FF_0000_0000_0148,
+    0xD1FF_0000_0000_015E,
+    0xD1FF_0000_0000_016A,
+    0xD1FF_0000_0000_016F,
+    0xD1FF_0000_0000_0179,
+    0xD1FF_0000_0000_019E,
+    0xD1FF_0000_0000_01A8,
+    0xD1FF_0000_0000_01AB,
+    0xD1FF_0000_0000_01B5,
+    0xD1FF_0000_0000_01B8,
+    0xD1FF_0000_0000_01D3,
+    0xD1FF_0000_0000_01EE,
+    0xD1FF_0000_0000_01F3,
+    0xD1FF_0000_0000_0202,
+    0xD1FF_0000_0000_0203,
+    0xD1FF_0000_0000_021E,
+    0xD1FF_0000_0000_022C,
+    0xD1FF_0000_0000_022D,
+    0xD1FF_0000_0000_0233,
+    0xD1FF_0000_0000_0245,
+    0xD1FF_0000_0000_024E,
 ];

 /// 40 seeds proven winnable within the Hard budget (≤ 25 000 states).
 pub const HARD_SEEDS: &[u64] = &[
-    // Generated by solitaire_assetgen::gen_difficulty_seeds (tier=Hard, date=2026-05-09)
-    0xD1FF_0000_0000_001F,
-    0xD1FF_0000_0000_0024,
-    0xD1FF_0000_0000_0025,
-    0xD1FF_0000_0000_0031,
-    0xD1FF_0000_0000_0032,
-    0xD1FF_0000_0000_003E,
-    0xD1FF_0000_0000_004A,
-    0xD1FF_0000_0000_006D,
+    // Generated by solitaire_assetgen::gen_difficulty_seeds (tier=Hard, date=2026-06-04)
+    0xD1FF_0000_0000_0006,
+    0xD1FF_0000_0000_0008,
+    0xD1FF_0000_0000_000F,
+    0xD1FF_0000_0000_0011,
+    0xD1FF_0000_0000_0022,
+    0xD1FF_0000_0000_0023,
+    0xD1FF_0000_0000_002A,
+    0xD1FF_0000_0000_002D,
+    0xD1FF_0000_0000_0040,
+    0xD1FF_0000_0000_0042,
+    0xD1FF_0000_0000_0050,
+    0xD1FF_0000_0000_005B,
+    0xD1FF_0000_0000_005D,
+    0xD1FF_0000_0000_0067,
+    0xD1FF_0000_0000_0069,
+    0xD1FF_0000_0000_006E,
+    0xD1FF_0000_0000_0072,
    0xD1FF_0000_0000_0079,
    0xD1FF_0000_0000_007C,
    0xD1FF_0000_0000_0080,
-    0xD1FF_0000_0000_008A,
-    0xD1FF_0000_0000_0097,
+    0xD1FF_0000_0000_0081,
+    0xD1FF_0000_0000_0083,
+    0xD1FF_0000_0000_0091,
+    0xD1FF_0000_0000_009B,
+    0xD1FF_0000_0000_00A1,
    0xD1FF_0000_0000_00B1,
-    0xD1FF_0000_0000_00B2,
-    0xD1FF_0000_0000_00B3,
-    0xD1FF_0000_0000_00B5,
-    0xD1FF_0000_0000_00B7,
-    0xD1FF_0000_0000_00B8,
-    0xD1FF_0000_0000_00B9,
-    0xD1FF_0000_0000_00BA,
-    0xD1FF_0000_0000_00BB,
-    0xD1FF_0000_0000_00BC,
-    0xD1FF_0000_0000_00BD,
-    0xD1FF_0000_0000_00C2,
    0xD1FF_0000_0000_00C3,
-    0xD1FF_0000_0000_00C5,
-    0xD1FF_0000_0000_00CC,
-    0xD1FF_0000_0000_00CE,
-    0xD1FF_0000_0000_00D1,
-    0xD1FF_0000_0000_00D2,
    0xD1FF_0000_0000_00D6,
-    0xD1FF_0000_0000_00D7,
-    0xD1FF_0000_0000_00DC,
-    0xD1FF_0000_0000_00DF,
-    0xD1FF_0000_0000_00E0,
-    0xD1FF_0000_0000_00E1,
-    0xD1FF_0000_0000_00E4,
-    0xD1FF_0000_0000_00E6,
-    0xD1FF_0000_0000_00E7,
+    0xD1FF_0000_0000_00DD,
+    0xD1FF_0000_0000_00E8,
+    0xD1FF_0000_0000_00F2,
+    0xD1FF_0000_0000_0101,
+    0xD1FF_0000_0000_010F,
+    0xD1FF_0000_0000_0113,
+    0xD1FF_0000_0000_0118,
+    0xD1FF_0000_0000_0119,
+    0xD1FF_0000_0000_012D,
+    0xD1FF_0000_0000_0133,
+    0xD1FF_0000_0000_0144,
+    0xD1FF_0000_0000_0147,
 ];

 /// 40 seeds proven winnable within the Expert budget (≤ 100 000 states).
 pub const EXPERT_SEEDS: &[u64] = &[
-    // Generated by solitaire_assetgen::gen_difficulty_seeds (tier=Expert, date=2026-05-09)
-    0xD1FF_0000_0000_0006,
-    0xD1FF_0000_0000_000B,
-    0xD1FF_0000_0000_0019,
+    // Generated by solitaire_assetgen::gen_difficulty_seeds (tier=Expert, date=2026-06-04)
+    0xD1FF_0000_0000_0000,
+    0xD1FF_0000_0000_0002,
+    0xD1FF_0000_0000_000A,
+    0xD1FF_0000_0000_0013,
+    0xD1FF_0000_0000_0017,
+    0xD1FF_0000_0000_001C,
+    0xD1FF_0000_0000_001F,
+    0xD1FF_0000_0000_0021,
+    0xD1FF_0000_0000_0024,
+    0xD1FF_0000_0000_0029,
+    0xD1FF_0000_0000_002E,
+    0xD1FF_0000_0000_0035,
+    0xD1FF_0000_0000_0045,
+    0xD1FF_0000_0000_0048,
+    0xD1FF_0000_0000_0049,
+    0xD1FF_0000_0000_004F,
+    0xD1FF_0000_0000_0062,
+    0xD1FF_0000_0000_006D,
+    0xD1FF_0000_0000_0074,
+    0xD1FF_0000_0000_0076,
    0xD1FF_0000_0000_0082,
-    0xD1FF_0000_0000_00CB,
-    0xD1FF_0000_0000_00D5,
-    0xD1FF_0000_0000_00D8,
-    0xD1FF_0000_0000_00E8,
-    0xD1FF_0000_0000_00EA,
-    0xD1FF_0000_0000_00EB,
-    0xD1FF_0000_0000_00EC,
+    0xD1FF_0000_0000_008F,
+    0xD1FF_0000_0000_0090,
+    0xD1FF_0000_0000_0097,
+    0xD1FF_0000_0000_009A,
+    0xD1FF_0000_0000_009F,
+    0xD1FF_0000_0000_00A5,
+    0xD1FF_0000_0000_00A8,
+    0xD1FF_0000_0000_00AD,
+    0xD1FF_0000_0000_00AE,
+    0xD1FF_0000_0000_00B8,
+    0xD1FF_0000_0000_00B9,
+    0xD1FF_0000_0000_00BC,
+    0xD1FF_0000_0000_00C5,
+    0xD1FF_0000_0000_00CA,
+    0xD1FF_0000_0000_00CE,
+    0xD1FF_0000_0000_00DE,
    0xD1FF_0000_0000_00ED,
-    0xD1FF_0000_0000_00F2,
-    0xD1FF_0000_0000_00F3,
-    0xD1FF_0000_0000_00F4,
-    0xD1FF_0000_0000_00FE,
-    0xD1FF_0000_0000_00FF,
-    0xD1FF_0000_0000_0102,
-    0xD1FF_0000_0000_0103,
-    0xD1FF_0000_0000_0104,
-    0xD1FF_0000_0000_0105,
-    0xD1FF_0000_0000_0106,
-    0xD1FF_0000_0000_0109,
-    0xD1FF_0000_0000_010B,
-    0xD1FF_0000_0000_010C,
-    0xD1FF_0000_0000_0110,
-    0xD1FF_0000_0000_0113,
-    0xD1FF_0000_0000_0114,
-    0xD1FF_0000_0000_011B,
-    0xD1FF_0000_0000_011C,
-    0xD1FF_0000_0000_011E,
-    0xD1FF_0000_0000_0120,
-    0xD1FF_0000_0000_0121,
-    0xD1FF_0000_0000_0122,
-    0xD1FF_0000_0000_0123,
-    0xD1FF_0000_0000_0124,
-    0xD1FF_0000_0000_0126,
-    0xD1FF_0000_0000_012B,
-    0xD1FF_0000_0000_012C,
-    0xD1FF_0000_0000_012E,
+    0xD1FF_0000_0000_00EE,
+    0xD1FF_0000_0000_00EF,
 ];

 /// 40 seeds proven winnable only within the Grandmaster budget (≤ 200 000 states).
 pub const GRANDMASTER_SEEDS: &[u64] = &[
-    // Generated by solitaire_assetgen::gen_difficulty_seeds (tier=Grandmaster, date=2026-05-09)
-    0xD1FF_0000_0000_0027,
-    0xD1FF_0000_0000_00A0,
-    0xD1FF_0000_0000_00C4,
-    0xD1FF_0000_0000_00D4,
-    0xD1FF_0000_0000_00DE,
-    0xD1FF_0000_0000_00F9,
-    0xD1FF_0000_0000_0107,
-    0xD1FF_0000_0000_0108,
-    0xD1FF_0000_0000_0130,
-    0xD1FF_0000_0000_0132,
-    0xD1FF_0000_0000_0133,
-    0xD1FF_0000_0000_0134,
+    // Generated by solitaire_assetgen::gen_difficulty_seeds (tier=Grandmaster, date=2026-06-04)
+    0xD1FF_0000_0000_003C,
+    0xD1FF_0000_0000_0047,
+    0xD1FF_0000_0000_005A,
+    0xD1FF_0000_0000_009C,
+    0xD1FF_0000_0000_00D2,
+    0xD1FF_0000_0000_00F4,
+    0xD1FF_0000_0000_00F6,
+    0xD1FF_0000_0000_0104,
+    0xD1FF_0000_0000_0106,
+    0xD1FF_0000_0000_0111,
+    0xD1FF_0000_0000_0112,
+    0xD1FF_0000_0000_0116,
+    0xD1FF_0000_0000_0117,
+    0xD1FF_0000_0000_011A,
+    0xD1FF_0000_0000_0123,
+    0xD1FF_0000_0000_012B,
+    0xD1FF_0000_0000_012E,
    0xD1FF_0000_0000_0135,
-    0xD1FF_0000_0000_0137,
-    0xD1FF_0000_0000_0139,
    0xD1FF_0000_0000_013A,
-    0xD1FF_0000_0000_013D,
-    0xD1FF_0000_0000_013F,
-    0xD1FF_0000_0000_0140,
+    0xD1FF_0000_0000_013B,
    0xD1FF_0000_0000_0141,
-    0xD1FF_0000_0000_0142,
-    0xD1FF_0000_0000_0143,
-    0xD1FF_0000_0000_0145,
-    0xD1FF_0000_0000_0146,
    0xD1FF_0000_0000_014A,
    0xD1FF_0000_0000_014B,
-    0xD1FF_0000_0000_014C,
-    0xD1FF_0000_0000_014D,
-    0xD1FF_0000_0000_014F,
+    0xD1FF_0000_0000_014E,
    0xD1FF_0000_0000_0150,
-    0xD1FF_0000_0000_0151,
-    0xD1FF_0000_0000_0152,
-    0xD1FF_0000_0000_0153,
+    0xD1FF_0000_0000_0155,
    0xD1FF_0000_0000_0157,
    0xD1FF_0000_0000_0158,
-    0xD1FF_0000_0000_015B,
+    0xD1FF_0000_0000_0159,
+    0xD1FF_0000_0000_015A,
    0xD1FF_0000_0000_015C,
-    0xD1FF_0000_0000_015E,
-    0xD1FF_0000_0000_0162,
-    0xD1FF_0000_0000_0164,
+    0xD1FF_0000_0000_015D,
+    0xD1FF_0000_0000_015F,
+    0xD1FF_0000_0000_0166,
+    0xD1FF_0000_0000_0173,
+    0xD1FF_0000_0000_0174,
+    0xD1FF_0000_0000_0178,
+    0xD1FF_0000_0000_017D,
+    0xD1FF_0000_0000_0182,
+    0xD1FF_0000_0000_0187,
 ];

 // ---------------------------------------------------------------------------
@@ -145,6 +145,8 @@ pub use settings::{

 #[cfg(target_os = "android")]
 mod android_keystore;
+#[cfg(target_os = "android")]
+pub use android_keystore::init_android_jvm;

 #[cfg(not(target_arch = "wasm32"))]
 pub mod auth_tokens;
@@ -74,6 +74,20 @@ struct UserRow {
 /// bcrypt work factor. Cost 12 ≈ 300 ms on modern hardware — balances security against registration latency.
 pub const BCRYPT_COST: u32 = 12;

+async fn hash_password(password: String) -> Result<String, AppError> {
+    tokio::task::spawn_blocking(move || hash(password, BCRYPT_COST))
+        .await
+        .map_err(|e| AppError::Internal(format!("password hash task failed: {e}")))?
+        .map_err(AppError::from)
+}
+
+async fn verify_password(password: String, password_hash: String) -> Result<bool, AppError> {
+    tokio::task::spawn_blocking(move || verify(password, &password_hash))
+        .await
+        .map_err(|e| AppError::Internal(format!("password verify task failed: {e}")))?
+        .map_err(AppError::from)
+}
+
 // ---------------------------------------------------------------------------
 // Token generation helpers
 // ---------------------------------------------------------------------------
@@ -191,7 +205,7 @@ pub async fn register(
    }

    let user_id = Uuid::new_v4().to_string();
-    let password_hash = hash(&body.password, BCRYPT_COST)?;
+    let password_hash = hash_password(body.password).await?;
    let now = Utc::now().to_rfc3339();

    sqlx::query!(
@@ -236,7 +250,7 @@ pub async fn login(
        .password_hash
        .ok_or_else(|| AppError::Internal("password hash missing".into()))?;

-    let valid = verify(&body.password, &row_hash)?;
+    let valid = verify_password(body.password, row_hash).await?;
    if !valid {
        tracing::warn!(username = %username, "login: invalid password");
        return Err(AppError::InvalidCredentials);
@@ -378,23 +392,28 @@ pub async fn upload_avatar(
    }

    // Write to avatars/ directory, replacing any previous file for this user.
-    std::fs::create_dir_all("avatars").map_err(|e| AppError::Internal(e.to_string()))?;
+    tokio::fs::create_dir_all("avatars")
+        .await
+        .map_err(|e| AppError::Internal(e.to_string()))?;
    let filename = format!("{}.{}", user.user_id, ext);
    let path = std::path::Path::new("avatars").join(&filename);
    let tmp_path = std::path::Path::new("avatars").join(format!("{}.{}.tmp", user.user_id, ext));
    // Write to a temp file then atomically rename so concurrent readers never
    // see a partially-written avatar.
-    std::fs::write(&tmp_path, &body).map_err(|e| AppError::Internal(e.to_string()))?;
-    if let Err(e) = std::fs::rename(&tmp_path, &path) {
-        let _ = std::fs::remove_file(&tmp_path);
+    tokio::fs::write(&tmp_path, &body)
+        .await
+        .map_err(|e| AppError::Internal(e.to_string()))?;
+    if let Err(e) = tokio::fs::rename(&tmp_path, &path).await {
+        let _ = tokio::fs::remove_file(&tmp_path).await;
        return Err(AppError::Internal(e.to_string()));
    }
    // Remove stale files with other extensions after the atomic rename.
    for old_ext in &["jpg", "png", "webp", "gif"] {
        if *old_ext != ext {
-            let _ = std::fs::remove_file(
+            let _ = tokio::fs::remove_file(
                std::path::Path::new("avatars").join(format!("{}.{}", user.user_id, old_ext)),
-            );
+            )
+            .await;
        }
    }

@@ -454,7 +473,7 @@ pub async fn reset_password(
    let user_id =
        user_id.ok_or_else(|| AppError::NotFound(format!("user '{username}' not found")))?;

-    let new_hash = hash(new_password, BCRYPT_COST)?;
+    let new_hash = hash_password(new_password.to_string()).await?;

    sqlx::query!(
        "UPDATE users SET password_hash = ? WHERE id = ?",
@@ -39,6 +39,18 @@ use std::{
    str::FromStr,
 };

+const JWT_SECRET_MIN_BYTES: usize = 32;
+
+fn validate_jwt_secret(secret: &str) -> Result<(), String> {
+    if secret.len() < JWT_SECRET_MIN_BYTES {
+        Err(format!(
+            "JWT_SECRET must be at least {JWT_SECRET_MIN_BYTES} bytes; generate a high-entropy production secret"
+        ))
+    } else {
+        Ok(())
+    }
+}
+
 #[tokio::main]
 async fn main() {
    // Load .env file if present (silently ignored when absent).
@@ -103,8 +115,13 @@ async fn run_reset_password(username: &str) {
 /// optionally `SERVER_PORT`) in the environment.
 async fn run_server() {
    let db_url = std::env::var("DATABASE_URL").expect("DATABASE_URL must be set");
-    // Load JWT_SECRET once at startup — a missing secret is a fatal configuration error.
+    // Load JWT_SECRET once at startup — a missing or weak secret is a fatal
+    // configuration error rather than a per-request failure.
    let jwt_secret = std::env::var("JWT_SECRET").expect("JWT_SECRET must be set");
+    if let Err(msg) = validate_jwt_secret(&jwt_secret) {
+        eprintln!("{msg}");
+        std::process::exit(1);
+    }
    let port: u16 = std::env::var("SERVER_PORT")
        .unwrap_or_else(|_| "8080".into())
        .parse()
@@ -137,3 +154,30 @@ async fn run_server() {

    axum::serve(listener, app).await.expect("server error");
 }
+
+#[cfg(test)]
+mod tests {
+    use super::*;
+
+    #[test]
+    fn jwt_secret_rejects_short_secret() {
+        let secret = "x".repeat(JWT_SECRET_MIN_BYTES - 1);
+        let err = validate_jwt_secret(&secret).expect_err("short secret must be rejected");
+        assert!(
+            err.contains("JWT_SECRET must be at least"),
+            "error should explain the minimum length, got: {err}"
+        );
+    }
+
+    #[test]
+    fn jwt_secret_accepts_exact_minimum_length() {
+        let secret = "x".repeat(JWT_SECRET_MIN_BYTES);
+        assert!(validate_jwt_secret(&secret).is_ok());
+    }
+
+    #[test]
+    fn jwt_secret_accepts_longer_secret() {
+        let secret = "x".repeat(JWT_SECRET_MIN_BYTES + 16);
+        assert!(validate_jwt_secret(&secret).is_ok());
+    }
+}
Author	SHA1	Message	Date
funman300	7dbf34c163	fix(server): move bcrypt to spawn_blocking, async file I/O, validate JWT_SECRET Build and Deploy / build-and-push (push) Successful in 5m20s Details Web E2E / web-e2e (push) Failing after 3m23s Details Three independent hardening changes: 1. bcrypt on a blocking thread: hash() and verify() are CPU-bound (~300 ms at cost 12). Running them directly on an async task starved the Tokio runtime under concurrent load. Wrapped in spawn_blocking. 2. Async avatar file I/O: std::fs::write/rename/remove_file in an async handler blocks the executor. Replaced with tokio::fs equivalents. 3. JWT_SECRET minimum length: a secret shorter than 32 bytes is fatally weak. validate_jwt_secret() now rejects it at startup with a clear message rather than silently accepting it. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-06-08 11:05:45 -07:00
funman300	7fa91b6fb4	data(seeds): regenerate difficulty seeds (2026-06-04) Replace the 2026-05-09 seed lists with seeds regenerated on 2026-06-04. All seeds remain verified winnable within their respective solver budgets. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-06-08 11:05:37 -07:00
funman300	becfda0f6c	fix(android): auto-discover SDK/NDK in build script, strip native libs build_android_apk.sh no longer requires all four env vars to be set manually. It probes common SDK paths and uses the newest installed build-tools/NDK/platform when vars are absent. Also adds llvm-strip pass to strip debug symbols from .so files before packaging (controlled by STRIP_NATIVE_LIBS, default 1), moves the debug keystore to a stable target/android/debug.keystore path, and prints resolved paths at start. Also adds scripts/ANDROID_TESTING.md and scripts/android_smoke.sh for on-device smoke testing. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-06-08 11:05:31 -07:00
funman300	fa786bafcf	feat(android): wire Android Keystore JNI via OnceLock Remove the dependency on bevy::android::ANDROID_APP inside android_keystore.rs. Instead, solitaire_data owns a process-wide OnceLock<JavaVM> initialised by a new pub fn init_android_jvm(). solitaire_app calls it from android_main before run() so JNI is ready before any auth-token operation can execute. - android_keystore: drop ANDROID_APP import; add ANDROID_JVM OnceLock and init_android_jvm(vm_ptr: *mut c_void) - solitaire_data/lib.rs: re-export init_android_jvm for android target - auth_tokens.rs: update doc comment (Android backend is now complete) - solitaire_app/lib.rs: call init_android_jvm from android_main Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-06-08 11:05:23 -07:00