@wix/himalaya-cli 0.979.0 → 0.981.0

package/dist/cli.mjs

@@ -7274,9 +7274,10 @@ function pathCrossesOpaqueAuthorJson(path) {
 function nativeRequiredFindings(descriptor) {
   const out = [];
   const shapeOf = (ref) => ref.startsWith("list:") ? "an array" : ref.startsWith("map:") ? "an object keyed by name" : "an object";
+  const kindOf = (value) => Array.isArray(value) ? "an array" : value === null ? "null" : typeof value === "object" ? "an object" : `a ${typeof value} (${JSON.stringify(value)})`;
   const reportKind = (ref, value, path, from) => {
     const cls = bareNestedClass(ref);
-    const got = Array.isArray(value) ? "an array" : value === null ? "null" : typeof value === "object" ? "an object" : `a ${typeof value} (${JSON.stringify(value)})`;
+    const got = kindOf(value);
     const expected = ref.startsWith("list:") ? "Expected JsonArray" : `Expected JsonObject for type with serial name 'io.himalaya.core.${cls}'`;
     out.push({
       path,
@@ -7322,6 +7323,25 @@ function nativeRequiredFindings(descriptor) {
       const inner = own(bag, name);
       if (inner !== void 0 && inner !== null) checkNested(inner, sub, `${path}.${name}`, from);
     }
+    if (cls === "Action") checkActionParams(own(bag, "params"), `${path}.params`, from);
+  };
+  const checkActionParams = (params, path, from) => {
+    if (params === void 0 || params === null) return;
+    const cost = "Native decode rejects it and drops the WHOLE screen from the seed, while web renders it fine";
+    if (typeof params !== "object" || Array.isArray(params)) {
+      out.push({
+        path,
+        message: `Action.params must be an object of strings, but the descriptor has ${kindOf(params)} ("Expected JsonObject"). ${cost} (reached from ${from})`
+      });
+      return;
+    }
+    for (const [key2, value] of Object.entries(params)) {
+      if (typeof value === "string" || value === void 0) continue;
+      out.push({
+        path: `${path}.${key2}`,
+        message: `Action.params values must be strings, but params.${key2} is ${kindOf(value)} ("Expected JsonPrimitive at ${key2}" on Android, typeMismatch on iOS). ${cost}. Flatten it into string keys of params, or move structured data to Action.args (reached from ${from})`
+      });
+    }
   };
   const walk2 = (node, at) => {
     if (Array.isArray(node)) {
@@ -7423,6 +7443,7 @@ function componentPropFindings(descriptor) {
             out.push({
               path,
               kind: "decode",
+              ...inArray ? { authoringOnly: true } : {},
               // TWO different defects share this branch, and they need DIFFERENT prose.
               //
               // For an ordinary typed slot the sentinel really is a screen-killer: kotlinx
@@ -8061,6 +8082,7 @@ function validateScreenDescriptor(screenId, descriptor, opts = {}) {
     if (!node || typeof node !== "object") return;
     for (const [k, val] of Object.entries(node)) {
       if (isOpaqueAuthorJson(k, val, at === "")) continue;
+      if (k === "params" && typeof node.actionId === "string") continue;
       const here = at ? `${at}.${k}` : k;
       const allowed2 = own(NATIVE_PROP_ENUMS, k);
       if (typeof val === "string" && enums.has(k) && DYNAMIC_PLACEHOLDER.test(val)) {
@@ -8068,6 +8090,7 @@ function validateScreenDescriptor(screenId, descriptor, opts = {}) {
           severity: "error",
           pass: "strict-enum",
           path: here,
+          nativeDecode: true,
           message: `enum field "${k}" carries a dynamic placeholder ("${val}") \u2014 native decode fails BEFORE substitution and kills the screen; use a literal (e.g. the spacer leadFill pattern for per-row alignment)`
         });
       } else if (allowed2 && val != null && typeof val !== "string") {
@@ -8075,6 +8098,7 @@ function validateScreenDescriptor(screenId, descriptor, opts = {}) {
           severity: "error",
           pass: "strict-enum",
           path: here,
+          nativeDecode: true,
           message: `enum field "${k}" has non-string value ${JSON.stringify(val)} \u2014 native decode rejects it (allowed: ${[...allowed2].join("|")}) and drops the whole screen`
         });
       } else if (typeof val === "string" && !DYNAMIC_PLACEHOLDER.test(val)) {
@@ -8083,6 +8107,7 @@ function validateScreenDescriptor(screenId, descriptor, opts = {}) {
             severity: "error",
             pass: "strict-enum",
             path: here,
+            nativeDecode: true,
             message: `enum field "${k}" has unknown value "${val}" \u2014 native decode rejects it (allowed: ${[...allowed2].join("|")}) and drops the whole screen; use one of those literals (HStack/VStack alignment: leading = top)`
           });
         }
@@ -8106,6 +8131,7 @@ function validateScreenDescriptor(screenId, descriptor, opts = {}) {
     issue2.pass = "strict-enum";
     issue2.path = finding.path;
     issue2.message = finding.message;
+    if (!("authoringOnly" in finding && finding.authoringOnly)) issue2.nativeDecode = true;
     if (!duplicate) issues.push(issue2);
     for (const extra of alsoAboutThisField) if (!consumed.has(extra)) superseded.add(extra);
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wix/himalaya-cli",
-  "version": "0.979.0",
+  "version": "0.981.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": "16761478845899a9ba0931eb0a135cf4d6e5d8909a3ad2899152b122"
+  "falconPackageHash": "8c6c57df555495fb76153e6101f19971862f487629e302e57aa4fdc0"
 }