@wix/himalaya-cli 0.968.0 → 0.969.0

package/dist/cli.mjs

@@ -537,7 +537,7 @@ var init_store = __esm({
 
 // ../../core/serve-storage/src/file-store.ts
 import { mkdirSync as mkdirSync2, readFileSync as readFileSync2, rmSync, writeFileSync as writeFileSync2, readdirSync, existsSync } from "node:fs";
-import { join as join2, resolve } from "node:path";
+import { join as join2, resolve, sep } from "node:path";
 var FileReleaseStore;
 var init_file_store = __esm({
   "../../core/serve-storage/src/file-store.ts"() {
@@ -547,8 +547,37 @@ var init_file_store = __esm({
       constructor(root) {
         this.root = resolve(root);
       }
+      /** The app's directory — and the ONE place an app id becomes a path in this store.
+       *
+       * IT VALIDATES, because `reset()` below deletes this path recursively and `appDir` was a bare
+       * `join(this.root, app)`. `remove()` called `this.safe(app)` first and `reset()` did not, three
+       * lines apart, so `reset("../../user-code/himalaya")` escaped the root and recursively removed
+       * whatever the process could write. Every other method reached the same unguarded join.
+       *
+       * Guarding the CALL SITES is what produced that bug: two of eight remembered. So the guard moves
+       * into the derivation, where forgetting it is not possible — a method that wants the directory
+       * has to come through here.
+       *
+       * IT IS NOT THE PUBLIC APP-ID GRAMMAR, and that distinction cost a CI round. The first cut of
+       * this called `assertAppId` — the grammar the ROUTES enforce — and refused `_template`, the
+       * internal app `buildRelease` resets through this store (build.ts). Nothing is wrong with that
+       * caller: this store's key is a local build-output directory name, which is a wider namespace
+       * than a published app id, and a published id is validated where it is published.
+       *
+       * What remains is the invariant that actually carries the security property, and it is enough on
+       * its own: a path that RESOLVES outside the root is refused. `../../user-code/himalaya` and any
+       * absolute path resolve outside; `_template` resolves inside. The root ITSELF is refused too —
+       * `reset("")` would otherwise recursively delete the whole store. `safe()` runs first as the
+       * cheap, readable half (no separator, no `..`), the same rule this class already applies to every
+       * screen id and sha it turns into a filename; the resolved-path check is the belt behind it, and
+       * catches what a string test cannot, including a separator this one does not know about.
+       */
       appDir(app) {
-        return join2(this.root, app);
+        const dir = resolve(this.root, this.safe(app));
+        if (!dir.startsWith(this.root + sep)) {
+          throw new Error(`unsafe app id ${JSON.stringify(app)}: ${dir} is not inside the store root`);
+        }
+        return dir;
       }
       safe(name) {
         if (name.includes("/") || name.includes("..")) throw new Error(`unsafe name: ${name}`);
@@ -561,7 +590,7 @@ var init_file_store = __esm({
         mkdirSync2(join2(dir, "bundles"), { recursive: true });
       }
       async remove(app) {
-        rmSync(this.appDir(this.safe(app)), { recursive: true, force: true });
+        rmSync(this.appDir(app), { recursive: true, force: true });
       }
       async putManifest(app, manifest) {
         validateFunctionManifest(manifest);
@@ -771,7 +800,7 @@ var init_base64 = __esm({
 });
 
 // src/source-deny.ts
-import { sep } from "node:path";
+import { sep as sep2 } from "node:path";
 function isDeniedSourcePath(relPath) {
   const segments = relPath.split(/[\\/]/).filter(Boolean);
   if (segments.length === 0) return false;
@@ -781,7 +810,7 @@ function isDeniedSourcePath(relPath) {
   return DENY_FILES.some((re) => re.test(segments[segments.length - 1]));
 }
 function isDeniedSourceDir(name, parentRel) {
-  const base = name.split(sep).pop() ?? name;
+  const base = name.split(sep2).pop() ?? name;
   if (DENY_DIRS.has(base)) return true;
   return parentRel === "" && DENY_ROOT_DIRS.has(base);
 }
@@ -1289,7 +1318,7 @@ function sentinelizeIdentity(body, identity) {
   }
   if (identity.name) {
     const re = new RegExp(`\\b(${NAME_FIELDS.join("|")})(["']?\\s*[:=]\\s*)(["'])${escapeRe(identity.name)}\\3`, "g");
-    out = out.replace(re, (_m, key2, sep11, q) => `${key2}${sep11}${q}__HIMI_APP_NAME__${q}`);
+    out = out.replace(re, (_m, key2, sep12, q) => `${key2}${sep12}${q}__HIMI_APP_NAME__${q}`);
   }
   return out;
 }
@@ -2836,7 +2865,7 @@ __export(build_lib_exports, {
   umbrellaRewritePlugin: () => umbrellaRewritePlugin
 });
 import { fileURLToPath as fileURLToPath5, pathToFileURL as pathToFileURL2 } from "node:url";
-import { basename as basename4, dirname as dirname8, join as join12, resolve as resolve8, sep as sep2 } from "node:path";
+import { basename as basename4, dirname as dirname8, join as join12, resolve as resolve8, sep as sep3 } from "node:path";
 import { readdirSync as readdirSync6, existsSync as existsSync8, readFileSync as readFileSync10, realpathSync as realpathSync2, statSync as statSync2 } from "node:fs";
 async function loadCompiler() {
   try {
@@ -2884,7 +2913,7 @@ function realpathOrSelf(p) {
   }
 }
 function isUnder(p, root) {
-  return p === root || p.startsWith(root + sep2);
+  return p === root || p.startsWith(root + sep3);
 }
 function confinementRoots(contentRoot) {
   const roots = /* @__PURE__ */ new Set();
@@ -3174,7 +3203,7 @@ function tsExtResolvePlugin() {
 }
 function importMetaUrlPlugin({ scope } = {}) {
   const scopes = scope ? [.../* @__PURE__ */ new Set([realpathOrSelf(scope), resolve8(scope)])] : [];
-  const inScope = (p) => scopes.some((r) => p === r || p.startsWith(r + sep2));
+  const inScope = (p) => scopes.some((r) => p === r || p.startsWith(r + sep3));
   return {
     name: "himalaya-import-meta-url",
     setup(b) {
@@ -3830,7 +3859,7 @@ var init_tier5_dir = __esm({
 
 // src/tier5-typecheck.ts
 import { existsSync as existsSync9, readFileSync as readFileSync11 } from "node:fs";
-import { relative as relative2, resolve as resolve10, sep as sep3 } from "node:path";
+import { relative as relative2, resolve as resolve10, sep as sep4 } from "node:path";
 import { pathToFileURL as pathToFileURL3 } from "node:url";
 async function loadTypeScript() {
   const runtime = process.env.HIMI_AUTHORING_RUNTIME_DIR;
@@ -3869,7 +3898,7 @@ function pathsFor(options) {
   return declarationPaths(options.umbrellaDir);
 }
 function relativeFile(contentDir2, fileName) {
-  return relative2(contentDir2, fileName).split(sep3).join("/");
+  return relative2(contentDir2, fileName).split(sep4).join("/");
 }
 function belongsToContentWorkers(contentDir2, diagnostic) {
   if (!diagnostic.file) return true;
@@ -11268,7 +11297,7 @@ __export(icon_exports, {
 });
 import { createHash as createHash5 } from "node:crypto";
 import { existsSync as existsSync15, mkdirSync as mkdirSync9, readFileSync as readFileSync18, statSync as statSync4, writeFileSync as writeFileSync8 } from "node:fs";
-import { basename as basename5, dirname as dirname14, join as join19, resolve as resolve14, sep as sep4 } from "node:path";
+import { basename as basename5, dirname as dirname14, join as join19, resolve as resolve14, sep as sep5 } from "node:path";
 function readManifest(dir) {
   try {
     return JSON.parse(readFileSync18(join19(dir, MANIFEST), "utf8"));
@@ -11285,7 +11314,7 @@ function declaredIconName(dir) {
 function iconDest(dir, name) {
   const base = resolve14(dir);
   const abs = resolve14(base, name);
-  if (!abs.startsWith(base + sep4)) {
+  if (!abs.startsWith(base + sep5)) {
     throw new Error(
       `himi icon: himalaya.content.json declares "icon": "${name}", which resolves outside the package (${abs}). Declare a path inside it, e.g. "${DEFAULT_CONTENT_ICON}".`
     );
@@ -12091,6 +12120,12 @@ var init_wix_gateway_rules = __esm({
 // src/ux-evidence.ts
 import { existsSync as existsSync21, mkdirSync as mkdirSync10, readFileSync as readFileSync23, writeFileSync as writeFileSync9 } from "node:fs";
 import { dirname as dirname15, isAbsolute as isAbsolute2, join as join25, relative as relative3, resolve as resolve15 } from "node:path";
+function isNativePlatform(value) {
+  return typeof value === "string" && NATIVE_PLATFORMS.includes(value);
+}
+function isNativeAuditStatus(value) {
+  return typeof value === "string" && NATIVE_AUDIT_STATUSES.includes(value);
+}
 function readText(path) {
   try {
     return existsSync21(path) ? readFileSync23(path, "utf8") : null;
@@ -12162,7 +12197,7 @@ function recordRenderWidth(content, entry) {
     return null;
   }
 }
-function recordNativeAudit(content, platform, status, checks, reason) {
+function recordNativeAudit(content, platform, status, checks, reason, release) {
   const normalizedChecks = status === "unavailable" ? [] : [...new Set(checks.map((check) => check.trim()).filter(Boolean))];
   if (status !== "unavailable" && !normalizedChecks.length) {
     throw new Error("--checks needs at least one comma-separated check (for example: back,keyboard,voiceover)");
@@ -12173,13 +12208,89 @@ function recordNativeAudit(content, platform, status, checks, reason) {
   const parsed = parseEvidence(path);
   if (parsed.error) throw new Error(`${UX_EVIDENCE_PATH} ${parsed.error}`);
   const evidence = parsed.evidence ?? { version: 1, platforms: {} };
-  evidence.platforms[platform] = { status, checks: normalizedChecks, ...reason?.trim() ? { reason: reason.trim() } : {}, recordedAt: (/* @__PURE__ */ new Date()).toISOString() };
+  evidence.platforms[platform] = {
+    status,
+    checks: normalizedChecks,
+    ...reason?.trim() ? { reason: reason.trim() } : {},
+    ...release?.releaseId && release.releaseHash ? { releaseId: release.releaseId, releaseHash: release.releaseHash } : {},
+    recordedAt: (/* @__PURE__ */ new Date()).toISOString()
+  };
   writeEvidence(path, evidence);
   return evidence;
 }
+function uxEvidenceForRelease(content, release) {
+  const { releaseId, releaseHash } = release;
+  const path = uxEvidencePath(content);
+  const base = {
+    path: UX_EVIDENCE_PATH,
+    platforms: [],
+    failedPlatforms: [],
+    otherReleases: [],
+    unstamped: 0,
+    inheritedScreens: [],
+    malformed: []
+  };
+  if (!existsSync21(path)) return { ...base, state: "absent" };
+  const parsed = parseEvidence(path);
+  if (parsed.error) return { ...base, state: "unreadable", error: parsed.error };
+  if (!parsed.evidence) return { ...base, state: "absent" };
+  const recorded = [];
+  const malformed = [];
+  for (const [key2, value] of Object.entries(parsed.evidence.platforms)) {
+    if (!value || typeof value !== "object") continue;
+    const entry = value;
+    if (!isNativePlatform(key2)) {
+      malformed.push(`${key2} (not a platform this tool drives)`);
+      continue;
+    }
+    if (!isNativeAuditStatus(entry.status)) {
+      malformed.push(`${key2} (status ${JSON.stringify(entry.status ?? null)} is not one \`himi ux record\` writes)`);
+      continue;
+    }
+    if (!Array.isArray(entry.checks) || entry.checks.some((check) => typeof check !== "string")) {
+      malformed.push(`${key2} (no \`checks\` array \u2014 not a record \`himi ux record\` wrote)`);
+      continue;
+    }
+    if (typeof entry.recordedAt !== "string" || !entry.recordedAt) {
+      malformed.push(`${key2} (no \`recordedAt\` \u2014 not a record \`himi ux record\` wrote)`);
+      continue;
+    }
+    if (entry.reason !== void 0 && typeof entry.reason !== "string") {
+      malformed.push(`${key2} (\`reason\` is not text)`);
+      continue;
+    }
+    recorded.push([key2, entry]);
+  }
+  if (recorded.length === 0) return { ...base, state: "unrecorded", malformed };
+  const forRelease = recorded.filter(([, entry]) => entry.releaseId === releaseId && entry.releaseHash === releaseHash);
+  if (forRelease.length > 0) {
+    const platforms = forRelease.map(([platform]) => platform);
+    const failedPlatforms = forRelease.filter(([, entry]) => entry.status === "failed").map(([platform]) => platform);
+    return failedPlatforms.length > 0 ? { ...base, state: "failed", platforms, failedPlatforms, malformed } : { ...base, state: "recorded", platforms, malformed };
+  }
+  if (recorded.some(([, entry]) => entry.releaseId === releaseId)) {
+    return {
+      ...base,
+      state: "outdated",
+      platforms: recorded.filter(([, entry]) => entry.releaseId === releaseId).map(([platform]) => platform),
+      malformed
+    };
+  }
+  const otherReleases = [...new Set(recorded.map(([, entry]) => entry.releaseId).filter((id) => !!id))].sort();
+  return {
+    ...base,
+    state: "stale",
+    otherReleases,
+    unstamped: recorded.filter(([, entry]) => !entry.releaseId).length,
+    malformed
+  };
+}
+var NATIVE_PLATFORMS, NATIVE_AUDIT_STATUSES;
 var init_ux_evidence = __esm({
   "src/ux-evidence.ts"() {
     init_mobile_ux_contract();
+    NATIVE_PLATFORMS = ["ios", "android"];
+    NATIVE_AUDIT_STATUSES = ["passed", "failed", "unavailable"];
   }
 });
 
@@ -13853,7 +13964,7 @@ var init_app_module = __esm({
 // ../../core/dev-server/src/app-assets.ts
 import { createHash as createHash8 } from "node:crypto";
 import { existsSync as existsSync27, lstatSync as lstatSync4, readFileSync as readFileSync29, readdirSync as readdirSync14, realpathSync as realpathSync3, statSync as statSync5 } from "node:fs";
-import { join as join31, resolve as resolve19, sep as sep5 } from "node:path";
+import { join as join31, resolve as resolve19, sep as sep6 } from "node:path";
 function contentTypeFor(name) {
   const ext = name.includes(".") ? name.slice(name.lastIndexOf(".") + 1).toLowerCase() : "";
   return CONTENT_TYPES[ext] ?? "application/octet-stream";
@@ -13874,7 +13985,7 @@ function findAssetsDir(appDir2) {
   const rel = typeof declared === "string" && declared ? declared : "assets";
   const abs = resolve19(appDir2, rel);
   const base = resolve19(appDir2);
-  if (abs !== base && !abs.startsWith(base + sep5)) return null;
+  if (abs !== base && !abs.startsWith(base + sep6)) return null;
   return existsSync27(abs) && statSync5(abs).isDirectory() ? abs : null;
 }
 function readOtaOnlyPatterns(appDir2) {
@@ -13932,7 +14043,7 @@ function collectBrandFonts(appDir2) {
     try {
       if (!lstatSync4(path).isFile()) continue;
       const root = realpathSync3(appDir2);
-      if (!realpathSync3(path).startsWith(root + sep5)) continue;
+      if (!realpathSync3(path).startsWith(root + sep6)) continue;
     } catch {
       continue;
     }
@@ -15153,7 +15264,7 @@ function redactBodyString(body) {
     } catch {
     }
   }
-  return body.replace(FORM_ASSIGNMENT_RE, (m, name) => isSensitiveBodyName(name) ? `${name}=${REDACTED}` : m).replace(JSON_FIELD_RE, (m, name, sep11) => isSensitiveBodyName(name) ? `"${name}"${sep11}"${REDACTED}"` : m).replace(JSON_FIELD_UNTERMINATED_RE, (m, name, sep11) => isSensitiveBodyName(name) ? `"${name}"${sep11}"${REDACTED}` : m).replace(BEARER_RE, (_m, s) => `${s} ${REDACTED}`);
+  return body.replace(FORM_ASSIGNMENT_RE, (m, name) => isSensitiveBodyName(name) ? `${name}=${REDACTED}` : m).replace(JSON_FIELD_RE, (m, name, sep12) => isSensitiveBodyName(name) ? `"${name}"${sep12}"${REDACTED}"` : m).replace(JSON_FIELD_UNTERMINATED_RE, (m, name, sep12) => isSensitiveBodyName(name) ? `"${name}"${sep12}"${REDACTED}` : m).replace(BEARER_RE, (_m, s) => `${s} ${REDACTED}`);
 }
 function createWixGatewayState() {
   return { mode: "record", rules: [], traffic: [], clients: /* @__PURE__ */ new Set(), seq: 0, ruleSeq: 0 };
@@ -22257,8 +22368,8 @@ function ensureCountUpRuntime() {
     const neg = fixed.startsWith("-");
     const body = neg ? fixed.slice(1) : fixed;
     const [int, frac] = body.split(".");
-    const sep11 = int.replace(/\B(?=(\d{3})+(?!\d))/g, ",");
-    return (neg ? "-" : "") + sep11 + (frac ? "." + frac : "");
+    const sep12 = int.replace(/\B(?=(\d{3})+(?!\d))/g, ",");
+    return (neg ? "-" : "") + sep12 + (frac ? "." + frac : "");
   };
   const animateEl = (el) => {
     const node = el;
@@ -30109,7 +30220,7 @@ var init_config2 = __esm({
 import { createServer as createServer2 } from "node:http";
 import { readFile as readFile2, stat } from "node:fs/promises";
 import { readFileSync as readFileSync40, existsSync as existsSync35, watch, mkdirSync as mkdirSync16, rmSync as rmSync4 } from "node:fs";
-import { extname, resolve as resolve30, normalize as normalize3, sep as sep6, join as join34 } from "node:path";
+import { extname, resolve as resolve30, normalize as normalize3, sep as sep7, join as join34 } from "node:path";
 import { createHash as createHash15, randomUUID } from "node:crypto";
 import { fileURLToPath as fileURLToPath18, pathToFileURL as pathToFileURL6 } from "node:url";
 function deepMerge2(base, patch) {
@@ -30927,7 +31038,7 @@ function startBundleSourceWatcher(state) {
   try {
     const watcher = watch(TIER5_SRC_ROOT, { persistent: false, recursive: true }, (_event, filename) => {
       if (!filename || !/\.(ts|tsx)$/.test(filename)) return;
-      const name = filename.split(sep6)[0];
+      const name = filename.split(sep7)[0];
       if (!name || name.startsWith(".")) return;
       pending.add(name);
       if (timer) clearTimeout(timer);
@@ -31102,7 +31213,7 @@ function startAppSourceWatcher(state) {
   try {
     const watcher = watch(appDir2, { persistent: false, recursive: true }, (_event, filename) => {
       if (!filename) return;
-      const rel = String(filename).split(sep6).join("/");
+      const rel = String(filename).split(sep7).join("/");
       if (!classifyAppPath(rel)) return;
       pending.add(rel);
       if (timer) clearTimeout(timer);
@@ -34619,8 +34730,8 @@ async function crawlScreenInner(opts) {
       if (linkable.has(target)) return true;
       for (const base of linkable) {
         if (target.length > base.length + 1 && target.startsWith(base)) {
-          const sep11 = target[base.length];
-          if (sep11 === "_" || sep11 === "/") return true;
+          const sep12 = target[base.length];
+          if (sep12 === "_" || sep12 === "/") return true;
         }
       }
       return false;
@@ -38281,7 +38392,7 @@ __export(app_source_exports, {
   zipAppSource: () => zipAppSource
 });
 import { readdirSync as readdirSync21, lstatSync as lstatSync5, readFileSync as readFileSync44, mkdirSync as mkdirSync18, writeFileSync as writeFileSync14, existsSync as existsSync38, rmSync as rmSync6, rmdirSync, unlinkSync as unlinkSync3 } from "node:fs";
-import { join as join37, resolve as resolve34, sep as sep7 } from "node:path";
+import { join as join37, resolve as resolve34, sep as sep8 } from "node:path";
 import { deflateRawSync as deflateRawSync2 } from "node:zlib";
 function walkSourceTree(dir, file, skip) {
   const walk2 = (relDir) => {
@@ -38301,7 +38412,7 @@ function walkSourceTree(dir, file, skip) {
         skip(rel);
         continue;
       }
-      file(rel.split(sep7).join("/"), rel);
+      file(rel.split(sep8).join("/"), rel);
     }
   };
   walk2("");
@@ -38442,7 +38553,7 @@ function pruneEmptyDirs(root, rel, needed, removed) {
   let empty = true;
   for (const name of readdirSync21(rel ? join37(root, rel) : root)) {
     const childRel = rel ? join37(rel, name) : name;
-    const posix3 = childRel.split(sep7).join("/");
+    const posix3 = childRel.split(sep8).join("/");
     const kind = lstatSync5(join37(root, childRel));
     if (kind.isSymbolicLink() || !kind.isDirectory() || isDeniedSourceDir(name, rel)) {
       empty = false;
@@ -38509,7 +38620,7 @@ __export(eject_exports, {
   ejectModule: () => ejectModule
 });
 import { existsSync as existsSync39, mkdirSync as mkdirSync19, readFileSync as readFileSync45, writeFileSync as writeFileSync15 } from "node:fs";
-import { dirname as dirname25, join as join38, relative as relative5, resolve as resolve35, sep as sep8 } from "node:path";
+import { dirname as dirname25, join as join38, relative as relative5, resolve as resolve35, sep as sep9 } from "node:path";
 function locate(repoRootOrCwd, mod) {
   const inRepo = resolve35(repoRootOrCwd, `stdlib/flows/${mod}/src/index.ts`);
   if (existsSync39(inRepo)) return { path: inRepo, form: "source" };
@@ -38562,7 +38673,7 @@ function ejectModule(opts) {
     if (!existsSync39(f)) continue;
     const before = readFileSync45(f, "utf8");
     let after = before;
-    let rel = relative5(dirname25(f), target).split(sep8).join("/").replace(/\.tsx?$/, ".js");
+    let rel = relative5(dirname25(f), target).split(sep9).join("/").replace(/\.tsx?$/, ".js");
     if (!rel.startsWith(".")) rel = `./${rel}`;
     for (const spec of specifiersFor(mod)) {
       const q = spec.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
@@ -41352,7 +41463,7 @@ import { createServer as createServer3 } from "node:http";
 import { createHash as createHash22, randomBytes as randomBytes5 } from "node:crypto";
 import { execFile as execFile2 } from "node:child_process";
 import { existsSync as existsSync44, readdirSync as readdirSync22, readFileSync as readFileSync50, realpathSync as realpathSync4, statSync as statSync8 } from "node:fs";
-import { join as join43, posix, relative as relative6, sep as sep9 } from "node:path";
+import { join as join43, posix, relative as relative6, sep as sep10 } from "node:path";
 function humanMs(ms) {
   return ms < 1e3 ? `${ms}ms` : `${Math.round(ms / 1e3)}s`;
 }
@@ -41402,7 +41513,7 @@ function collectRelativeSiblings(files, tier5SrcDir) {
       ];
       for (const abs of candidates) {
         if (!existsSync44(abs) || statSync8(abs).isDirectory()) continue;
-        const key2 = `/tier5-src/${relative6(tier5SrcDir, abs).split(sep9).join(posix.sep)}`;
+        const key2 = `/tier5-src/${relative6(tier5SrcDir, abs).split(sep10).join(posix.sep)}`;
         if (key2 in files) break;
         files[key2] = readFileSync50(abs, "utf8");
         queue.push(key2);
@@ -48867,7 +48978,7 @@ var init_home_descriptor = __esm({
 
 // src/site-analyze/apply.ts
 import { existsSync as existsSync52, readFileSync as readFileSync62, writeFileSync as writeFileSync21, copyFileSync as copyFileSync3, mkdirSync as mkdirSync24, realpathSync as realpathSync5, statSync as statSync11 } from "node:fs";
-import { dirname as dirname29, isAbsolute as isAbsolute4, join as join50, resolve as resolve40, sep as sep10 } from "node:path";
+import { dirname as dirname29, isAbsolute as isAbsolute4, join as join50, resolve as resolve40, sep as sep11 } from "node:path";
 import { spawnSync as spawnSync4 } from "node:child_process";
 function unpairedRedirectNote(overlay) {
   if (!overlay.wix?.clientId || overlay.wix.redirectUri) return void 0;
@@ -48962,7 +49073,7 @@ function buildMergedContract(options) {
   const contained = (root, rel) => {
     if (isAbsolute4(rel)) return void 0;
     const full = resolve40(root, rel);
-    return full.startsWith(resolve40(root) + sep10) ? full : void 0;
+    return full.startsWith(resolve40(root) + sep11) ? full : void 0;
   };
   const stage = (fromRel, toRel) => {
     const from = contained(packageDir, fromRel);
@@ -48976,7 +49087,7 @@ function buildMergedContract(options) {
     } catch {
       return void 0;
     }
-    if (!real.startsWith(realRoot + sep10)) return void 0;
+    if (!real.startsWith(realRoot + sep11)) return void 0;
     try {
       if (!statSync11(real).isFile()) return void 0;
       mkdirSync24(dirname29(to), { recursive: true });
@@ -55265,7 +55376,7 @@ var COMMANDS = [
   { name: "eject", summary: "take ownership of a shared flow module (auth|checkout|screens): copies its source into tier5-src/_ejected/ and repoints this package's imports at the copy. The ladder's escape hatch \u2014 for when the factory options cannot express what you need. AFTER EJECTING, CENTRAL FIXES NO LONGER REACH THIS APP, including future corrections to the fail-closed entitlement and confirm-the-session rules. The floor is unaffected: credentials still go only through the native auth actions and money only through the cashier rail.", required: "<auth|checkout|screens> [--content <dir>]" },
   { name: "validate", summary: "lint the content package (no push): fixed-contract TypeScript checking for tier5-src workers + schema/proto + token refs + component allowlist + $service bindings + strict-enum foot-guns + capability decls + worker boot dry-run + build/compile failures. Type issues are advisory warnings normally; --strict promotes them and hygiene warnings to errors. Exit 1 on any error. Off-repo, the CLI supplies its compiler and SDK declarations.", required: "[--content <dir>] [--strict]" },
   { name: "test", summary: "DERIVED behavioral crawl \u2014 no test files to write. Boots each screen's real worker, replays its onAppear, then dispatches every action site the descriptor already declares (each from a fresh boot) and checks a narrow set of unambiguous invariants: no uncaught throw, never stuck on `loading`, no `$service:` token or NaN/undefined/[object Object] reaching visible text, no binding aimed at state the worker never defines, no nav param silently dropped, no dead nav target. Catches what validate/render cannot \u2014 a WRONG REDUCER passes both with ok:true. Runs the worker in a DEVICE-PARITY sandbox (no setTimeout/setInterval, as on JSC/WebView) so a bundle that would die on device fails here. Crawls on HERMETIC DATA by default \u2014 zero network: the app's own dev/net-mocks.json first (--net <file> overrides), then the config's CMS `seed:` rows, then a canned fixture of frozen real vertical responses shipped with the CLI (exercises code paths, not API-contract truth) \u2014 so the success/mapping path runs, not just the failure UI. Two boot-only HOSTILE passes (every list empty; every request 500) then check the failure/empty day \u2014 corpus-proven, so their findings are errors. NAV EDGES are followed too: every himi.open site ($item params resolved against row 0 of its list, $service against worker state) and worker _nav directive is harvested, and each baked target is booted WITH the params the source sends \u2014 detail screens finally run their content arm; edge findings are errors tagged edge:<source>. --native runs both real native decoders over the exact baked JSON bytes; --native ios or --native android selects one. It is explicit, runs only from a Himalaya SOURCE CHECKOUT (the decoder oracles are that tree's own HimalayaCore Swift package and Gradle harness, which a published install does not ship), and needs the selected local toolchain. From a source checkout, default runs name missing native verdicts in the coverage ledger; elsewhere the ledger stays silent about them rather than offering a remedy you cannot run. --offline strips all data layers (the old reject-all axis); --no-hostile skips the hostile passes. --record (EXPLICIT, never automatic) refreshes dev/net-mocks.json from live reads first: it boots each screen (load + root onAppear ONLY \u2014 nothing the app's own launch would not fire; the tap/action crawl never sees the live transport, and only proven reads \u2014 GET, or read-shaped POST (/query, /search, /list*, \u2026) \u2014 ever reach the wire), freezes the round-trips, then crawls hermetically on the fresh fixture; record against a QA site, not production. Nodes it cannot judge headless are named, never silently passed. Exit 1 on any error. The CLI carries the authoring runtime off-repo; no per-app SDK install is required. --browser runs the whole loop (build \u2192 extract \u2192 validate \u2192 render \u2192 test) in YOUR browser instead of here, so the source never reaches a pod: it opens one screen's worker + descriptor in a tab, blocks on a loopback callback, and prints the report it posts back. One screen per tab \u2014 name it. --no-open prints the URL rather than launching a browser (SSH); --timeout <ms> bounds the wait, which matters because the page has no execution deadline of its own and an infinite loop in the worker hangs the tab.", required: "[<screenId>\u2026] [--content <dir>] [--net <file>] [--budget <ms>] [--deep [n]] [--record] [--native [ios|android]] [--no-shots] [--offline] [--no-hostile] [--all-screens] [--browser] [--no-open] [--timeout <ms>] [--json]" },
-  { name: "ux", summary: "audit or record mobile UX evidence. `audit` checks the public-safe contract, seeded blank acceptance rows, placeholder source copy, app-specific mocks, and iOS/Android audit records. It warns while existing templates migrate; `--strict` promotes missing evidence. `record` stores checks performed on an installed iOS/Android build: `--checks` is required for passed|failed and empty for unavailable, which drove nothing and needs a `--reason` instead.", required: "audit [--content <dir>] [--strict] [--json] | record --content <dir> --platform ios|android --status passed|failed|unavailable [--checks back,keyboard,...] [--reason <text>]" },
+  { name: "ux", summary: "audit or record mobile UX evidence. `audit` checks the public-safe contract, seeded blank acceptance rows, placeholder source copy, app-specific mocks, and iOS/Android audit records. It warns while existing templates migrate; `--strict` promotes missing evidence. `record` stores checks performed on an installed iOS/Android build: `--checks` is required for passed|failed and empty for unavailable, which drove nothing and needs a `--reason` instead. `record` also STAMPS the release it audited (it resolves the id from a build, the same one a push would publish), because an unforced `himi push` of a release with screens refuses when no drive is recorded for THAT release, or when the one that is recorded says `failed` \u2014 the artifact stopped being advice nobody reads (#2955). `unavailable` still satisfies it; only `failed` refuses.", required: "audit [--content <dir>] [--strict] [--json] | record --content <dir> --platform ios|android --status passed|failed|unavailable [--checks back,keyboard,...] [--reason <text>]" },
   { name: "render", summary: "headless-render a screen with the web renderer: render-tree + diagnostics, optional --png <file>, --scheme dark, --tree. Local by default (content dir). Pass --remote --app --release to hit GET /v1/preview/\u2026/render on the serve plane after a push. Boots the worker so $service bindings resolve; --static raw bindings; --live real Wix fetches; --state injects worker state; --input supplies pre-boot navigation params; --width/--device for PNG viewport; --rtl for dir=rtl + $env:isRTL.", required: "<screenId> [--content <dir>] [--scheme dark] [--png <file>] [--tree] [--static] [--live] [--state <json|file>] [--input <json|file>] [--width <px> | --device <name>] [--rtl] [--remote --app --release]" },
   { name: "cms", summary: "inspect, provision, and seed the app's CMS (Wix Data) collections, declared as a `cms` block in config.ts. `query` is READ-ONLY \u2014 by default it fetches ONLY the physical ids `config.ts` declares (item counts + schema drift) and reports `siteScanned: false`; it never lists the site, so an UNDECLARED `himi_<app>_` collection this app owns is not reached without `--all`. `--all` SCANS the site \u2014 every collection it lists is labeled `owned`/`adopted`/`foreign`, and it describes every foreign one \u2014 id, non-system field keys + types, and the site's read role \u2014 so an app with no `cms` block at all can DISCOVER a collection to adopt (`--all` is the one case where a missing `cms` block is not 'nothing to do'). Foreign collections are described up to a cap and are never item-counted (`itemCount: null`), because a row count is one query each; `foreign.count` is how many the scan LISTED \u2014 NOT a site census, since the list endpoint is paged and is not followed \u2014 and `foreign.truncated` says how many of those went undescribed, while `unlistedCollections` reports what the listing itself left out (from Wix's own paging total). The scan is OWNER-scoped \u2014 it needs --site <msid> (or WIX_SITE_ID) plus owner auth via `wix token`; a visitor token can read a public collection's ITEMS but can never enumerate collections, so there is no tokenless discovery path. `create` reconciles declared collections additively and idempotently, and REFUSES any id outside the app's `himi_<app>_` namespace, so it is safe against a production site. It reports declared-vs-site `permissionDrift` but never writes permissions unless asked with --apply-permissions, so a field reconcile can't silently revert access control changed in the dashboard. `seed` inserts the git-tracked `seed:` rows into EMPTY owned collections (--force to add to a non-empty one; --ai writes placeholder rows plus a refinement prompt for you to review). Adopted collections (declared by `wixId`) are read-only and never created, updated, or seeded. Site from --site <msid> or WIX_SITE_ID; owner auth via `wix token`.", required: "query|create|seed [--site <msid>] [--content <dir>] [--all] [--dry-run] [--force] [--allow-destructive] [--apply-permissions] [--only <id>] [--ai] [--count <n>]" },
   { name: "data", summary: "read-only plan or explicitly provision/seed declared KV and Wix Data resources. Preview and production are separate scopes. KV administration uses the operator data plane from --data-url/HIMI_DATA_ADMIN_URL and HIMI_DATA_ADMIN_TOKEN; draft push never calls it.", required: "plan|create|seed [--site <msid>] [--environment preview|production] [--content <dir>] [--data-url <url>] [--dry-run] [--force]" },
@@ -55276,7 +55387,7 @@ var COMMANDS = [
   { name: "notify", summary: "send a REAL push notification to registered devices (RFC 0036 WS5) \u2014 APNs .p8 over HTTP/2 for iOS, FCM v1 for Android. `send` targets one device (--token / --device <registry id>) or fans out to the whole app (--all), reporting a per-device result; a token the provider says is uninstalled (APNs 410 Unregistered / FCM UNREGISTERED) is auto-disabled, while a merely wrong one (BadDeviceToken \u2014 usually a sandbox/production mismatch) is recorded and left enabled. `tokens` lists the registry. `doctor` reports which credentials resolve and which sender each platform picks. Talks to the admin-server via --admin-url/HIMALAYA_ADMIN_URL, authenticated with --admin-session/HIMALAYA_ADMIN_SESSION. NOT `himi push`, which publishes a release.", required: "send|tokens|doctor [--app <a>] [--token <t> | --device <id> | --all] [--title <s>] [--body <s>] [--link <url>] [--platform ios|android] [--badge <n>] [--channel <id>] [--silent] [--dry-run] [--json]" },
   { name: "site", summary: "PROVISION a Wix project, then analyze or extend it. `create` is the one door that makes a site instead of adopting one: POST /funnel/projects/v1/create (a WIX site, a HEADLESS backend, a BRANDED_APP, or a VIBE site), and for headless it also mints the OAuth app whose id IS the `WIX_CLIENT_ID` \u2014 with --redirect-uri already registered, because an unregistered redirect makes native signInDirect fail at redirect-session. `--verticals` REQUIRES `--template` and reports only what Wix confirmed: measured, installing onto a project created from the BLANK template returns 200 while the vertical's API answers 428 forever, because there is no site for it to attach to (the install asks for CREATE_SITE). A real template ships its verticals installed and populated, and installing further ones onto it then works. It MUTATES your Wix account and needs --yes; without it you get the payload and nothing is sent. `list` is the read-only site inventory (with each site's premium flag, which is what explains an installed-but-428 vertical). Auth is the account-scoped `wix token` (no --site), falling back to WIX_API_KEY + WIX_ACCOUNT_ID. `client` is the same OAuth step for a site that ALREADY exists \u2014 the one himi adopts rather than makes \u2014 so a clientId no longer means a manual trip to the dashboard: `client ensure` reconciles rather than mints, because a site can already carry a client with NO registered redirect and shipping that one fails signInDirect at runtime; it reuses a client that carries --redirect-uri, PATCHes one that matches --name but lacks it (unioning, so another app's redirect survives), and creates only when there is nothing to reuse (--new forces a fresh one). --name IS the idempotency key. `client` auth is SITE-scoped (`wix token --site <msid>`) because minting is owner-of-that-site, and --yes gates every mutation: a site that already has a usable client resolves WITHOUT it, but --write-provisioning needs --yes even then because it rewrites a manifest on disk (the dry run's planned.provisioning names the file). That query endpoint IGNORES paging (measured: query.paging, cursorPaging and a top-level paging all return the full set at offset 0), so when pagingMetadata reports more clients than came back, `ensure` REFUSES EVERY outcome \u2014 create, patch AND reuse. Reuse is refused too because pickClient breaks ties on the OLDEST createdDate, so an omitted older same-named MOBILE client with the same redirect would win a later complete read and the resolved clientId would MOVE between runs \u2014 and it gets written into provisioning.json; convergence is what makes a stray duplicate untidy rather than harmful. --name cannot narrow the query (no filter is sent; the name is applied to the rows after they arrive), so the recovery is the dashboard; --new skips the query entirely. A `patched` result is verified against what Wix STORED \u2014 the redirect list is read off the PATCH response, and re-read via GET when the response omits it, because falling back to the list himi just sent would make the guard pass vacuously on exactly the silent no-op it exists to catch. `client list` is the read-only inventory of a site's clients: act on its `shippable` column rather than `hasRedirect`, because shippable ALSO requires a MOBILE applicationType and `ensure` refuses a WEB client \u2014 a WEB one carrying your redirect looks right and is not a candidate \u2014 and it sets truncated:true rather than presenting a partial inventory as the whole one. `analyze` reads a site into a business-book folder (REST + Playwright, GET-only); `site function` list/get/put/publish/invokes that site's Velo HTTP functions and is not `himi functions` (serve-plane jobs). `--apply` bakes into a brandable shell; `--shell` takes the same names `himi dist --shell` does and defaults to branded-lite, and a shell that ships no prepare:customer is refused by name rather than half-built. It never overwrites any shell's committed apps/<shell>/provisioning.json.", required: "create <name> [--type wix|headless|branded-app|vibe] [--template <guid>] [--verticals stores,bookings,\u2026] [--folder <guid>] [--redirect-uri <uri>] [--no-oauth-app] [--write-provisioning [path]] [--site-url <url>] [--json] --yes | list | client ensure|list --site <msid> [--name <name>] [--redirect-uri <uri>] [--new] [--write-provisioning [path]] [--site-url <url>] [--json] --yes | analyze --client-id <id> --url <https> [--site <msid>] [--out <dir>] [--redirect-uri <uri>] [--apply] [--shell " + SHELL_ENUM + "] | function list|get|put|publish|invoke [--site <msid>] [--url <https>] [--method GET|POST|PUT|DELETE] [--source <file>] [--body <json>] [--publish] [--invoke]" },
   { name: "doctor", summary: 'off-VPN preflight in one shot: probe the PUBLIC data plane (www.wixapis.com \u2014 reachable off-VPN), the INTERNAL serve/control plane for the selected target (VPN-gated), and report auth state (dpx session on disk + stored control token). --dpx additionally runs a live, EADDRINUSE-safe `dpx whoami`. Machine-readable with --json (an unreachable internal plane carries a stable code:"needs_vpn" so an agent branches on the code, not the prose); human-readable otherwise. Never loads the authoring runtime \u2014 works off-repo.', required: "[--target <t>] [--dpx] [--json] [--data-url <url>] [--timeout <ms>]" },
-  { name: "push", summary: "lint + build + publish a release (default --draft: servable via ?release= but NOT live). REFUSES to publish on any error unless --force; --strict promotes warnings. Prints preview + agent-context links. The published slug is NAMESPACED per owner \u2014 `<ownerPrefix>.<name>`, composed offline from the control token's ownerPrefix claim, so the CLI and the server derive the identical string and himalaya.content.json is NOT rewritten (a legacy token with no ownerPrefix publishes the declared name unchanged). An app claimed BEFORE namespacing is stored under its FLAT slug, so the claim route answers `alreadyYours` naming that slug and the push adopts it \u2014 reported as `adoptedSlug`, since it is the one case where the published `app` differs from what the declared name composes to. `--no-claim` skips the route that knows this, so a pre-namespacing app pushed with it still fails 403. --visibility public|unlisted decides whether /v1/apps + the Himi Player picker LIST the app (also declarable as `visibility` in himalaya.content.json; omitted, a brand-new app defaults to unlisted \u2014 share by link \u2014 and an existing setting is kept). --account-id names which account owns the app when your Wix session resolves to several (the auto-claim of a brand-new slug cannot guess \u2014 without it a multi-account caller is told to name one). `--only <selector>` publishes ONLY the named items as a release derived from the app's live release (or `--from-release <id>`), running the identical build + lint gate and simply publishing less; it requires an explicit `--draft` or `--live` because a partial publish has no safe default, never claims a slug, and carries neither attestation nor a source archive (both named in the output). Selectors are comma-separated in ONE `--only`: `changed` (every item whose bytes differ from the base), `screen:<id|glob>`, `bundle:<name>`, `tokens`, `app-config` \u2014 a selected screen drags its logic bundle and a selected bundle drags the screens that reference it, or the bytes land under a key nothing reads.", required: "[--content <dir>] [--draft] [--live] [--force] [--strict] [--label] [--notes] [--visibility public|unlisted] [--no-claim] [--account-id <id>] [--only <selector>] [--from-release <id>] [--accept-regressions] [--accept-partial] [--app <slug>] (--control-token)" },
+  { name: "push", summary: "lint + build + publish a release (default --draft: servable via ?release= but NOT live). REFUSES to publish on any error unless --force; --strict promotes warnings. Prints preview + agent-context links. The published slug is NAMESPACED per owner \u2014 `<ownerPrefix>.<name>`, composed offline from the control token's ownerPrefix claim, so the CLI and the server derive the identical string and himalaya.content.json is NOT rewritten (a legacy token with no ownerPrefix publishes the declared name unchanged). An app claimed BEFORE namespacing is stored under its FLAT slug, so the claim route answers `alreadyYours` naming that slug and the push adopts it \u2014 reported as `adoptedSlug`, since it is the one case where the published `app` differs from what the declared name composes to. `--no-claim` skips the route that knows this, so a pre-namespacing app pushed with it still fails 403. --visibility public|unlisted decides whether /v1/apps + the Himi Player picker LIST the app (also declarable as `visibility` in himalaya.content.json; omitted, a brand-new app defaults to unlisted \u2014 share by link \u2014 and an existing setting is kept). --account-id names which account owns the app when your Wix session resolves to several (the auto-claim of a brand-new slug cannot guess \u2014 without it a multi-account caller is told to name one). An unforced push of a release that carries SCREENS also refuses when `.himi/ux-audit.json` holds no mobile-UX drive recorded against that release id, or holds one that FAILED (#2955) \u2014 a published draft IS the owner-private preview a customer is handed; `himi ux record` clears it and `--force` overrides it, which the success envelope then says out loud. `--only <selector>` publishes ONLY the named items as a release derived from the app's live release (or `--from-release <id>`), running the identical build + lint gate and simply publishing less; it requires an explicit `--draft` or `--live` because a partial publish has no safe default, never claims a slug, and carries neither attestation nor a source archive (both named in the output). Selectors are comma-separated in ONE `--only`: `changed` (every item whose bytes differ from the base), `screen:<id|glob>`, `bundle:<name>`, `tokens`, `app-config` \u2014 a selected screen drags its logic bundle and a selected bundle drags the screens that reference it, or the bytes land under a key nothing reads.", required: "[--content <dir>] [--draft] [--live] [--force] [--strict] [--label] [--notes] [--visibility public|unlisted] [--no-claim] [--account-id <id>] [--only <selector>] [--from-release <id>] [--accept-regressions] [--accept-partial] [--app <slug>] (--control-token)" },
   { name: "pack", summary: 'write the app as a PORTABLE .himi package \u2014 one self-contained file holding every screen, Tier-5 bundle, token, asset and the app-config. Packages are live by default: they boot bundled content offline, then Himi Player follows its baked-in production ring. `--snapshot` makes a frozen reproducible artifact. Package-supplied origins are never trusted. --format dir writes the same tree unpacked, for static hosting. --release-dir packs a release that is already built on disk (needs --app) instead of rebuilding from source. `--dev` ALSO packs the app\'s own source archive into the container (`source.zip`, header `contents: "dev"`), so the package can be handed to someone who then keeps building \u2014 collected through the same deny-list and secret scan `himi push` uses, and refused if a credential is found. A dev package IS the codebase: it is never the default, it is always labelled in the output, the web host will not open one, and it is archive-only (`--dev --format dir` is refused, because a reader that predates the source entry drops it before hashing a directory and would call the package tampered). Restore one with `himi import`.', required: "[--content <dir>] [--out <file|dir>] [--format zip|dir] [--name <title>] [--snapshot] [--dev] [--strict] [--force] | --release-dir <dir> --app <a>" },
   { name: "import", summary: "turn a `himi pack --dev` package back into an EDITABLE content package \u2014 the file counterpart of `himi pull`, needing no serve plane, no account and no network. Refuses an invalid package before trusting its source (the content hash is recomputed over every entry on open, so the archive is provably the one that was packed), and refuses a RELEASE package by naming the absence \u2014 it carries built output and no source \u2014 rather than reporting a missing file. `--name` re-substitutes the app identity through the same machinery `himi init --template` and `himi pull --name` use, so the restored tree is a NEW app rather than a second copy of the same one, and a name that also appears inside a screen id (`feed` in `feed_home`) is left alone. A non-empty destination needs `--force` and a merge NAMES what it kept (`left: [...]`), the same contract `himi pull --force` has; a shape clash is refused whole before anything is written. There is no `--replace`: a file you were handed is not the plane's per-release archive.", required: "<package.himi> [--out <dir>] [--name <new>] [--force]" },
   { name: "pull", summary: "restore an app's SOURCE from the serve plane into an editable content package \u2014 the other half of `himi push`, which archives the content package on every publish. Owner or editor only: a viewer may download the RUNNING app (`/v1/owner/package`), not the codebase it was built from. A PAST release is restorable, not only the live one \u2014 the archive is content-addressed per release, so it has none of the live-single-slot problem that makes `himi pack` current-release-only. Two limits: pruning keeps the current release plus the newest ~20 ledger entries, and a release id identifies OTA content rather than source, so a source-only change republishes under the same id and replaces that id's archive. A release published before source archiving shipped, or pushed with `--no-source`, answers `app_source_not_found` and says to push again. `--name` re-substitutes the app identity through the same machinery `himi init --template` uses, so the restored tree is a NEW app rather than a second copy of the same one, and a name that also appears inside a screen id (`feed` in `feed_home`) is left alone. A NON-EMPTY destination is refused unless you say which you mean: `--force` merges and reports \u2014 it writes the archive over what is there, keeps files the release does not carry, and NAMES every one of them as `left: [...]` so a tree that matches no release is never silent (#2437); `--replace` restores exactly what was archived, removing those files first (`removed: [...]`) and implying `--force`. `--replace` refuses a directory holding no `himalaya.content.json`, so a mistyped `--out` cannot clear an unrelated tree, and the only paths it can remove are the ones a push would have archived \u2014 `.git/`, `node_modules/`, `dist/`, `.env*` and key material are out of its reach by construction.", required: "<app> [--release <id>] [--out <dir>] [--name <new>] [--force|--replace] [--target <t>]" },
@@ -56246,6 +56357,102 @@ function resolutionFlag(flags) {
   if (flags.source === true) return "source";
   return void 0;
 }
+async function recordedReleaseForContentQuietly(dir, resolution) {
+  let out;
+  try {
+    const { buildContentForValidation: buildContentForValidation2, buildReleaseFromContent: buildReleaseFromContent2, bundleOutDir: bundleOutDir2 } = await loadAuthoring();
+    const bres = await buildContentForValidation2(dir, { resolution });
+    out = mkdtempSync4(join51(tmpdir5(), "himi-ux-record-"));
+    const hasWorkers = bres.built.length > 0 || bres.compileErrors.length > 0;
+    const outcome = await buildReleaseFromContent2(dir, out, { ...hasWorkers ? { tier5Dir: bundleOutDir2(dir) } : {} });
+    const releaseId = outcome.manifest.releaseId;
+    if (!releaseId) return null;
+    const payload = await loadReleaseFromDir(await appId(dir), out);
+    return { releaseId, releaseHash: releasePayloadHash(payload) };
+  } catch {
+    return null;
+  } finally {
+    if (out !== void 0) try {
+      rmSync9(out, { recursive: true, force: true });
+    } catch {
+    }
+  }
+}
+function releaseCarriesScreens(...manifests) {
+  return manifests.some((m) => Object.keys(m?.screens ?? {}).length > 0);
+}
+function handoverUxEvidence(dir, release) {
+  if (!release.carriesScreens) return null;
+  const evidence = uxEvidenceForRelease(dir, { releaseId: release.releaseId, releaseHash: release.releaseHash });
+  const inheritedScreens = release.inheritedScreens ?? [];
+  if (inheritedScreens.length > 0) {
+    return {
+      ...evidence,
+      state: "inherited-screens",
+      inheritedScreens,
+      ...evidence.state === "recorded" ? {} : { supersededState: evidence.state }
+    };
+  }
+  return evidence;
+}
+function uxRecordCommands(contentPath) {
+  return `\`himi ux audit --content ${contentPath} --strict\`, then \`himi ux record --content ${contentPath} --platform ios --status unavailable --reason "<why you could not drive it>"\``;
+}
+function uxRestoreCommands(ctx) {
+  const pinned = ctx.baseReleaseId ? ` --release ${ctx.quote(ctx.baseReleaseId)}` : "";
+  return {
+    pull: `\`himi pull ${ctx.quote(ctx.app)}${pinned}${ctx.planeArgs ?? ""} --out ${ctx.quote("<dir>")}\``,
+    record: uxRecordCommands(ctx.quote(`<dir>/${ctx.app}`))
+  };
+}
+function uxHandoverHint(uxEvidence, ctx) {
+  const { releaseId, contentArg, door, app } = ctx;
+  const uxRemedy = uxRecordCommands(contentArg);
+  if (uxEvidence.state === "inherited-screens") {
+    const restore = uxRestoreCommands(ctx);
+    const alsoOwed = uxEvidence.supersededState === "failed" ? ` This package's own drive FAILED on ${uxEvidence.failedPlatforms.join(", ")}, so you owe more than a record: fix what that drive found, then record it against the restored tree \u2014 re-recording a failure without changing the app writes the same failure down again.` : uxEvidence.supersededState ? ` This package also has no drive this gate accepts (${uxEvidence.supersededState}), so the record below is a step you take AFTER the restore, not instead of it.` : "";
+    return `this patch derives from ${ctx.baseReleaseId ?? "a release"} which carries ${uxEvidence.inheritedScreens.length} screen(s) your package does not contain (${uxEvidence.inheritedScreens.join(", ")}), so a drive of this package cannot have seen them \u2014 and a tokens or bundle patch changes how they look or behave.${alsoOwed} Restore THAT release's source with ${restore.pull}, apply the change you were publishing to the restored tree, and only THEN record the drive there: ${restore.record} \u2014 the record is matched against the package you push, so one taken before your change describes the base and goes stale the moment you apply it. Patch from that tree. Do NOT run a full \`himi push\` from this package instead: publishing a tree that is missing those screens REMOVES them. If that release was published with \`--no-source\`, or before source archiving shipped, the pull answers \`app_source_not_found\` and the override below is the only exit. Only if you intend to hand a customer a preview of screens nobody drove, \`${door} --force\`.`;
+  }
+  if (uxEvidence.state === "failed") {
+    return `a mobile-UX drive for ${releaseId} IS recorded, and it FAILED on ${uxEvidence.failedPlatforms.join(", ")} (see ${uxEvidence.path}). A published draft is the owner-private preview a customer is handed, so fix what the drive found and re-record it \u2014 re-running \`himi ux record\` over a failure without changing the app just writes the same failure down again. Only if you intend to hand a customer a preview whose UX you already know is broken, \`${door} --force\`.`;
+  }
+  const why = uxEvidence.state === "absent" ? `nothing has recorded a mobile-UX drive for this package \u2014 ${uxEvidence.path} does not exist` : uxEvidence.state === "unrecorded" ? uxEvidence.malformed.length ? `${uxEvidence.path} has no iOS or Android drive this gate can read \u2014 ${uxEvidence.malformed.join("; ")}` : `${uxEvidence.path} exists but carries no iOS or Android drive (\`himi render\` writes its rendered widths there too, so the file alone proves nothing)` : uxEvidence.state === "unreadable" ? `${ctx.quote(`${ctx.contentDir}/${uxEvidence.path}`)} ${uxEvidence.error} \u2014 delete it and record again, because re-recording over a malformed store does not repair it` : uxEvidence.state === "outdated" ? `a mobile-UX drive for ${releaseId} is recorded, but the package has CHANGED since \u2014 in-repo a release id is the git HEAD sha, so editing a screen, a token or a worker without committing keeps the id while replacing everything it identifies (a record written before this check also lands here, carrying no content hash)` : `the recorded mobile-UX drive is about ${uxEvidence.otherReleases.length ? `release ${uxEvidence.otherReleases.join(", ")}` : "no release at all"}, not ${releaseId}${uxEvidence.unstamped ? ` (${uxEvidence.unstamped} drive(s) name no release)` : ""}`;
+  return `${why}. A published draft is the owner-private preview a customer is handed, so record the drive first: ${uxRemedy}, then re-run \`${door}\`. Only if you intend to hand over a preview WITHOUT that evidence, \`${door} --force\`.`;
+}
+var SECRET_FLAG_NAME = /token|secret|password|cookie|auth/i;
+var NEVER_ECHOED = /* @__PURE__ */ new Set(["force", "help", "json"]);
+var PLANE_FLAGS = ["target", "serve-url", "control-url"];
+function planeArguments(flags, quote) {
+  return PLANE_FLAGS.filter((name) => typeof flags[name] === "string").map((name) => ` --${name} ${quote(String(flags[name]))}`).join("");
+}
+function echoedInvocation(command, flags, quote) {
+  const parts = [command];
+  for (const [name, value] of Object.entries(flags)) {
+    if (value === false || NEVER_ECHOED.has(name) || SECRET_FLAG_NAME.test(name)) continue;
+    parts.push(`--${name}`);
+    if (typeof value === "string") parts.push(quote(value));
+  }
+  return parts.join(" ");
+}
+function uxHandoverNote(uxEvidence, ctx) {
+  if (uxEvidence.state === "inherited-screens") {
+    const restore = uxRestoreCommands(ctx);
+    const drive = uxEvidence.supersededState === "failed" ? `A drive of this package is on record and it FAILED on ${uxEvidence.failedPlatforms.join(", ")} \u2014 and it could not have covered them in any case:` : uxEvidence.supersededState ? `No drive of this package is one this gate accepts (${uxEvidence.supersededState}), and a drive of this package could not have covered them in any case:` : "A drive of this package IS recorded, and it could not have covered them in any case:";
+    return `published over ${uxEvidence.inheritedScreens.length} screen(s) nobody drove (${uxEvidence.inheritedScreens.join(", ")}). ${drive} the preview link below renders them from ${ctx.baseReleaseId ?? "the base release"}, and this package does not contain them. Restore that release's source with ${restore.pull}, apply the change you just published to it, drive and record there (${restore.record}), and publish again from that tree before a customer reviews it \u2014 the record is matched against the package you push, so one taken before your change describes the base.`;
+  }
+  if (uxEvidence.state === "failed") {
+    return `published over a FAILED mobile-UX drive for ${ctx.releaseId} on ${uxEvidence.failedPlatforms.join(", ")}. The preview link below is the owner-private handover \u2014 fix what the drive found and publish again before a customer reviews it.`;
+  }
+  if (uxEvidence.state === "unreadable") {
+    const why = uxEvidence.error ? ` \u2014 ${uxEvidence.path} ${uxEvidence.error}` : ` \u2014 ${uxEvidence.path} could not be read`;
+    return `published with no mobile-UX drive this gate accepts for ${ctx.releaseId} (unreadable)${why}. The preview link below is the owner-private handover \u2014 DELETE ${ctx.quote(`${ctx.contentDir}/${uxEvidence.path}`)} and record the drive again with \`himi ux record --content ${ctx.contentArg} \u2026\`, because re-recording over a malformed store does not repair it, then publish again before a customer reviews it.`;
+  }
+  return `published with no mobile-UX drive this gate accepts for ${ctx.releaseId} (${uxEvidence.state}). The preview link below is the owner-private handover \u2014 record the drive with \`himi ux record --content ${ctx.contentArg} \u2026\` and publish again before a customer reviews it.`;
+}
+function uxHandoverOverride(uxEvidence, ctx) {
+  if (!uxEvidence || uxEvidence.state === "recorded") return {};
+  return { uxEvidence, uxEvidenceNote: uxHandoverNote(uxEvidence, ctx) };
+}
 async function loadAuthoring() {
   try {
     const [b, rb, crawl, vd, cov, wml, nf, rd] = await Promise.all([
@@ -56462,6 +56669,17 @@ himi v${cliVersion()} \u2014 run \`himi --version\`. If authoring commands fail
       }, null, 2));
       return 1;
     }
+    if (opts.ux?.evidence && opts.ux.evidence.state !== "recorded" && !opts.force) {
+      print({
+        ok: false,
+        patched: false,
+        pushed: false,
+        uxEvidence: opts.ux.evidence,
+        releaseId: opts.ux.ctx.releaseId,
+        hint: uxHandoverHint(opts.ux.evidence, opts.ux.ctx)
+      });
+      return 1;
+    }
     let r;
     try {
       r = await client2().releasePatch(app, base.releaseId, plan.patch, draft, opts.meta, opts.force);
@@ -56504,6 +56722,9 @@ himi v${cliVersion()} \u2014 run \`himi --version\`. If authoring commands fail
       ...opts.lintNote ? { lintNote: opts.lintNote } : {},
       attestation: honesty.attestation,
       source: honesty.source,
+      // Spread through the SAME helper the full push uses, so a forced partial publish records
+      // the override in the same words rather than in a second phrasing that can drift.
+      ...opts.ux ? uxHandoverOverride(opts.ux.evidence, opts.ux.ctx) : {},
       ...out.visibility ? { visibility: out.visibility } : {},
       ...opts.extra ?? {},
       previewWeb: sbase && release ? previewWebUrl(sbase, app, release, void 0, void 0, previewToken) : null,
@@ -57134,17 +57355,27 @@ himi v${cliVersion()} \u2014 run \`himi --version\`. If authoring commands fail
         const platform = str5(flags.platform);
         const status = str5(flags.status);
         const checks = str5(flags.checks)?.split(",").map((item) => item.trim()).filter(Boolean) ?? [];
-        if (platform !== "ios" && platform !== "android") {
-          io.error(JSON.stringify({ error: "ux record needs --platform ios|android" }));
+        if (!isNativePlatform(platform)) {
+          io.error(JSON.stringify({ error: `ux record needs --platform ${NATIVE_PLATFORMS.join("|")}` }));
           return 2;
         }
-        if (status !== "passed" && status !== "failed" && status !== "unavailable") {
-          io.error(JSON.stringify({ error: "ux record needs --status passed|failed|unavailable" }));
+        if (!isNativeAuditStatus(status)) {
+          io.error(JSON.stringify({ error: `ux record needs --status ${NATIVE_AUDIT_STATUSES.join("|")}` }));
           return 2;
         }
+        const uxRelease = await recordedReleaseForContentQuietly(dir, resolutionFlag(flags));
         try {
-          const evidence = recordNativeAudit(dir, platform, status, checks, str5(flags.reason));
-          print({ ok: true, content: dir, evidence });
+          const evidence = recordNativeAudit(dir, platform, status, checks, str5(flags.reason), uxRelease ?? void 0);
+          print({
+            ok: true,
+            content: dir,
+            releaseId: uxRelease?.releaseId ?? null,
+            releaseHash: uxRelease?.releaseHash ?? null,
+            ...uxRelease ? {} : {
+              releaseIdNote: "this package could not be built, so the record names no release \u2014 `himi push` will not accept it as evidence for the release it publishes. Fix the build and re-run this command."
+            },
+            evidence
+          });
           return 0;
         } catch (error) {
           io.error(JSON.stringify({ error: error.message }));
@@ -58163,6 +58394,9 @@ default (no --glyph): ${DEFAULT_GLYPH2} \u2014 a mark that claims nothing, so th
         const kvDeclared = declaredData?.kv?.length ?? 0;
         const manifest = outcome.manifest;
         const payload = await loadReleaseFromDir(declared, out);
+        const { shellQuote: shellQuote2 } = await Promise.resolve().then(() => (init_shell_quote(), shell_quote_exports));
+        const contentRel = relative8(process.cwd(), dir) || ".";
+        const contentArg = shellQuote2(contentRel);
         if (onlyRaw !== void 0) {
           const onlyDraft = flags.draft === true;
           const { parseOnlySelectors: parseOnlySelectors2, planFromPayload: planFromPayload2, notCarriedDiff: notCarriedDiff2 } = await Promise.resolve().then(() => (init_release_patch(), release_patch_exports));
@@ -58195,6 +58429,28 @@ default (no --glyph): ${DEFAULT_GLYPH2} \u2014 a mark that claims nothing, so th
             io.error(JSON.stringify(planned.error, null, 2));
             return 1;
           }
+          const patchUxEvidence = handoverUxEvidence(dir, {
+            releaseId: manifest.releaseId,
+            releaseHash: releasePayloadHash(payload),
+            carriesScreens: releaseCarriesScreens(baseForPlan.manifest, manifest),
+            // The screens this publish will serve whose BYTES this package has never held: the
+            // derived release materializes them from the base. Local-only screens are not here —
+            // those the author does have, and a drive of this package covers them.
+            inheritedScreens: Object.keys(baseForPlan.manifest.screens ?? {}).filter((id) => !(id in (manifest.screens ?? {}))).sort()
+          });
+          const patchUxContext = {
+            releaseId: manifest.releaseId,
+            contentArg,
+            contentDir: contentRel,
+            app,
+            door: echoedInvocation("himi push", flags, shellQuote2),
+            // The base this plan DERIVES from, not the live release: under `--from-release` those
+            // differ, and a restore of the wrong one hands the author a different set of screen
+            // bytes to drive.
+            baseReleaseId: baseForPlan.manifest.releaseId,
+            quote: shellQuote2,
+            planeArgs: planeArguments(flags, shellQuote2)
+          };
           let localSourceHash;
           try {
             const { collectAppSource: collectAppSource2, zipAppSource: zipAppSource2 } = await Promise.resolve().then(() => (init_app_source(), app_source_exports));
@@ -58223,6 +58479,9 @@ default (no --glyph): ${DEFAULT_GLYPH2} \u2014 a mark that claims nothing, so th
             },
             ...localSourceHash ? { localSourceHash } : {},
             notCarried: notCarriedDiff2(manifest, baseForPlan.manifest),
+            // Reaches here ONLY on the forced path — the refusal above returned otherwise — so
+            // this is the override's record, and the case it matters most for is a `failed` drive.
+            ux: { evidence: patchUxEvidence, ctx: patchUxContext },
             lint: report.counts ?? null,
             extra: { selector: parsed.selectors.map((s) => "pattern" in s ? `${s.kind}:${s.pattern}` : s.kind) }
           });
@@ -58271,8 +58530,6 @@ default (no --glyph): ${DEFAULT_GLYPH2} \u2014 a mark that claims nothing, so th
         const evidence = pushEvidence(dir, manifest.releaseId, (candidate) => verifyAttestation(candidate, dir, payload.screens, declared, payload, { requireLatest: false }).ok);
         const staleEvidence = evidence.staleRuns > 0 ? `${evidence.staleRuns} retained attestation(s) carry this release id, but none describes the content being published` : null;
         const attestationErrors = [...evidence.historyErrors, ...staleEvidence ? [staleEvidence] : []];
-        const { shellQuote: shellQuote2 } = await Promise.resolve().then(() => (init_shell_quote(), shell_quote_exports));
-        const contentArg = shellQuote2(relative8(process.cwd(), dir) || ".");
         if (attestationErrors.length > 0 && !force) {
           const remedy = `\`himi test --content ${contentArg}\` (or \`himi render <screenId> --content ${contentArg}\`)`;
           const corrupted = evidence.historyErrors.length > 0;
@@ -58294,6 +58551,35 @@ default (no --glyph): ${DEFAULT_GLYPH2} \u2014 a mark that claims nothing, so th
           });
           return 1;
         }
+        const uxEvidence = handoverUxEvidence(dir, {
+          releaseId: manifest.releaseId,
+          releaseHash: releasePayloadHash(payload),
+          // On a FULL push the package is the release, so its own screens are the whole answer —
+          // and nothing is inherited, which is why the limitation above is a `--only` concern only.
+          carriesScreens: releaseCarriesScreens(manifest),
+          inheritedScreens: []
+        });
+        if (uxEvidence && uxEvidence.state !== "recorded" && !force) {
+          print({
+            ok: false,
+            pushed: false,
+            uxEvidence,
+            releaseId: manifest.releaseId,
+            // The SAME reconstruction as the patch door: this one dropped `--content` too, and
+            // fixing one door and not the other is the drift the shared helpers exist to stop.
+            hint: uxHandoverHint(uxEvidence, {
+              releaseId: manifest.releaseId,
+              contentArg,
+              contentDir: contentRel,
+              app,
+              door: echoedInvocation("himi push", flags, shellQuote2),
+              quote: shellQuote2
+              // No `baseReleaseId`: a full push derives from nothing, so it can never produce
+              // `inherited-screens` and has no base to restore.
+            })
+          });
+          return 1;
+        }
         const forcedPastErrors = force && (hasErrors || attestation?.report.ok === false || attestationErrors.length > 0);
         const assetRecords = Object.entries(manifest.assets ?? {});
         const assetSummary = assetRecords.length ? {
@@ -58436,6 +58722,18 @@ default (no --glyph): ${DEFAULT_GLYPH2} \u2014 a mark that claims nothing, so th
           ...!attestation && staleEvidence ? {
             attestationNote: `${staleEvidence} \u2014 this release was published with no headless evidence of its own, and the serve plane may still show the attestation from an earlier push of this release id. Re-attest with \`himi test --content ${contentArg}\` and push again to attach one.`
           } : {},
+          // #2955 — the SAME honesty for the UX gate: `--force` published this preview without a
+          // recorded mobile-UX drive for it, and the link below goes to a customer. Say it in the
+          // success envelope, because a refusal nobody reads and an override nobody records are
+          // the same silence one step apart.
+          ...uxHandoverOverride(uxEvidence, {
+            releaseId: manifest.releaseId,
+            contentArg,
+            contentDir: contentRel,
+            app,
+            door: echoedInvocation("himi push", flags, shellQuote2),
+            quote: shellQuote2
+          }),
           ...loopbackWarning ? { warning: loopbackWarning } : {},
           ...report.issues ? { issues: report.issues } : {},
           previewWeb: base ? previewWebUrl(base, app, release, void 0, void 0, previewToken) : null,
@@ -61149,6 +61447,9 @@ export {
   switchFlagErrors,
   templatePreviewAuthError,
   tokenCoversApp,
+  uxHandoverHint,
+  uxHandoverNote,
+  uxHandoverOverride,
   wixPlayerTestflightTarget,
   writePngFile
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wix/himalaya-cli",
-  "version": "0.968.0",
+  "version": "0.969.0",
   "type": "module",
   "description": "himi — the Himalaya authoring CLI. Build a Tier-5 content package on your own machine, push it to the production serve as a draft, preview it on web + Himi Player, and read worker logs back — the remote dev loop. Wraps @himalaya/serve-cli for transport.",
   "keywords": [
@@ -64,5 +64,5 @@
       "artifactId": "himalaya-cli"
     }
   },
-  "falconPackageHash": "e41cecd4c127159c9493293586554a07f307be67bc66163cb13683c0"
+  "falconPackageHash": "5fbb2ba85ad9db892370efbcc59a8ef6629609514feb06522e4ea1f2"
 }