@wix/himalaya-cli 0.1029.0 → 0.1031.0

package/templates/branded-lite/tier5-src/_shared/collection-contract.ts

@@ -14,6 +14,9 @@ export interface CollectionConfig {
   searchPlaceholder?: string;
   sortLabels?: { default: string; az: string; za: string };
   metadata?: string[];
+  /** Numeric 0–5 row field rendered as a Rating (accessible "N out of 5") instead of
+   *  folding a "★★★★☆" glyph string into the metadata line, so screen readers get a value. */
+  rating?: string;
   sectioned?: boolean;
   view: 'list' | 'grid';
   backendQuery?: boolean;
@@ -58,7 +61,7 @@ export const COLLECTIONS: Record<string, CollectionConfig> = {
   himiBrandedCoupons: { rows: 'coupons', title: 'name', subtitle: 'subtitle', view: 'list', primaryVerb: { label: 'Get code' } },
   himiBrandedNotifications: { rows: 'items', title: 'name', subtitle: 'subtitle', image: 'photo', metadata: ['unreadDot'], view: 'list' },
   himiBrandedLoyalty: { rows: 'rewards', title: 'name', subtitle: 'description', view: 'grid', primaryVerb: { label: 'Redeem' } },
-  himiBrandedReviews: { rows: 'reviews', title: 'title', subtitle: 'body', metadata: ['author', 'stars'], view: 'list' },
+  himiBrandedReviews: { rows: 'reviews', title: 'title', subtitle: 'body', metadata: ['author'], rating: 'rating', view: 'list' },
   himiBrandedPortfolio: { rows: 'projects', title: 'title', subtitle: 'subtitle', image: 'image', view: 'grid', gridImageHeight: 180 },
   himiBrandedInvoices: { rows: 'invoices', title: 'title', subtitle: 'subtitle', view: 'list' },
   himiBrandedBadges: { rows: 'badges', title: 'name', subtitle: 'description', view: 'grid' },

package/templates/branded-lite/tier5-src/_shared/owner-load-error.ts

@@ -1,6 +1,12 @@
 import { copyParam } from "./himi-copy.js";
 
-/** Host-owned preview input. Absent on native and older hosts. */
+/** Host-owned preview input. Absent on native and older hosts.
+ *
+ *  `"sample"` — the built-in demo records; `"live"` — the viewer's own business over the
+ *  owner rung; `"fixtures"` — a DEV web host over a sealed hermetic gateway, where
+ *  the dev fixtures are the sample data. `"fixtures"` is neither live nor demo, so a
+ *  worker READS through the bridge as it does on a device; it is still a preview host
+ *  (`isOwnerPreviewHost`), so failure copy may name the web preview. */
 function ownerPreviewMode(): unknown {
   return (globalThis as unknown as { himalaya?: { input?: Record<string, unknown> } }).himalaya?.input?.__himiOwnerPreview;
 }

package/templates/branded-lite/tier5-src/himi-branded-lite-appearance/index.ts

@@ -1,5 +1,6 @@
 import { defineService } from "@himalaya/state";
 import {
+  ACCENT_IDS,
   ACCENT_KEY,
   ACCENT_META,
   DEFAULT_ACCENT,
@@ -29,6 +30,10 @@ interface State {
   accentHex_sage: string;
   accentHex_plum: string;
   accentHex_ocean: string;
+  // Per-accent selection flags for the radio group (`selected_<id>`) and the
+  // check mark on the swatch (`notSelected_<id>`, hiddenWhen hides on TRUE). Flat
+  // fields for the same reason as the hex ones: the swatches are static nodes.
+  [flag: `selected_${string}` | `notSelected_${string}`]: boolean | string;
 }
 
 interface StorageLike {
@@ -50,7 +55,13 @@ function paramValue(arg: unknown, key: string): string {
 }
 
 function build(pendingAccent: string, brandAccent: string): State {
+  const flags: Record<string, boolean> = {};
+  for (const id of ACCENT_IDS) {
+    flags[`selected_${id}`] = id === pendingAccent;
+    flags[`notSelected_${id}`] = id !== pendingAccent;
+  }
   return {
+    ...flags,
     pendingAccent,
     brandAccent,
     accentHex_brand: brandAccent || ACCENT_META.amber.hex,
@@ -85,7 +96,7 @@ const svc = defineService<State, {
     selectAccent: (state, arg) => {
       const id = paramValue(arg, "id");
       if (!isAccentId(id)) return state;
-      return { ...state, pendingAccent: id };
+      return build(id, state.brandAccent);
     },
     // Apply: persist the accent; the home worker re-reads it on launch.
     apply: (state) => {

package/templates/branded-lite/tier5-src/himi-branded-lite-notifsettings/index.ts

@@ -26,7 +26,10 @@ import {
  *  One constant, so the read and the write cannot drift apart. */
 const CHANNEL = "MOBILE";
 
-interface Row { id: string; name: string; subtitle: string; on: boolean; locked: boolean }
+/** `note` is the line under a row's switch — only a locked topic has one. */
+interface Row { id: string; name: string; subtitle: string; note: string; on: boolean; locked: boolean;
+  /** Inverse of `locked` (hiddenWhen hides on TRUE) and the plain On/Off a locked row shows. */
+  notLocked: boolean; stateLabel: string }
 interface S {
   status: "loading" | "ready";
   topics: Row[]; emptyText: string;
@@ -59,7 +62,7 @@ function subtitleFor(on: boolean): string {
 }
 
 function row(id: string, name: string, on: boolean): Row {
-  return { id, name, subtitle: subtitleFor(on), on, locked: false };
+  return { id, name, subtitle: subtitleFor(on), note: "", on, locked: false, notLocked: true, stateLabel: on ? "On" : "Off" };
 }
 
 function norm(t: NotificationTopicDefinition, prefs: PreferenceInfo[]): Row {
@@ -68,7 +71,7 @@ function norm(t: NotificationTopicDefinition, prefs: PreferenceInfo[]): Row {
   const r = row(id, name, isOn(id, prefs));
   // An overridden row says so instead of offering a toggle that cannot take effect.
   return locked(id, prefs)
-    ? { ...r, locked: true, subtitle: "Managed by your channel settings" }
+    ? { ...r, locked: true, notLocked: false, subtitle: "Managed by your channel settings", note: "Managed by your channel settings" }
     : r;
 }
 
@@ -85,7 +88,7 @@ function mockState(s: S, reason: "empty" | "unavailable"): S {
 
 const svc = defineService<S, {
   load: (s: S) => Promise<S>;
-  toggle: (s: S, p?: { id?: string }) => Promise<S>;
+  toggle: (s: S, p?: { id?: string; value?: unknown }) => Promise<S>;
 }>("himiBrandedNotifSettings", {
   state: { status: "loading", topics: [], emptyText: "", isMock: false, notMock: true, notMockEmpty: true, notMockUnavailable: true},
   actions: {
@@ -112,8 +115,10 @@ const svc = defineService<S, {
       let nextOn = true;
       const topics = s.topics.map((r) => {
         if (r.id !== id) return r;
-        nextOn = !r.on;
-        return { ...r, on: nextOn, subtitle: subtitleFor(nextOn) };
+        // A Toggle sends the NEXT state as `value` (action-payload contract); honour it so
+        // a repeated event can never flip the row the wrong way. A bare tap still flips.
+        nextOn = typeof p?.value === "boolean" ? p.value : !r.on;
+        return { ...r, on: nextOn, subtitle: subtitleFor(nextOn), stateLabel: nextOn ? "On" : "Off" };
       });
       if (!s.isMock && !id.startsWith("sample-")) {
         try {

package/templates/branded-lite/tier5-src/himi-branded-lite-reviews/index.ts

@@ -8,7 +8,8 @@
 import { defineService } from "../_shared/collection-service.js";
 import { queryReviews, type WixReview } from "@himalaya/wix-reviews";
 
-interface Row { id: string; author: string; stars: string; title: string; body: string }
+/** `rating` (0–5) feeds the row's Rating control (spoken "N out of 5"); `stars` stays for older descriptors. */
+interface Row { id: string; author: string; stars: string; rating: number; title: string; body: string }
 interface S {
   status: "loading" | "ready";
   reviews: Row[]; emptyText: string;
@@ -20,8 +21,11 @@ interface S {
 }
 
 /** "★★★★☆" from a 0–5 rating (clamped). */
+function clampRating(rating?: number): number {
+  return Math.max(0, Math.min(5, Math.round(typeof rating === "number" && Number.isFinite(rating) ? rating : 0)));
+}
 function starBar(rating?: number): string {
-  const r = Math.max(0, Math.min(5, Math.round(typeof rating === "number" ? rating : 0)));
+  const r = clampRating(rating);
   return "★".repeat(r) + "☆".repeat(5 - r);
 }
 
@@ -31,16 +35,17 @@ function norm(v: WixReview): Row {
     id: v.id || "review",
     author,
     stars: starBar(v.content?.rating),
+    rating: clampRating(v.content?.rating),
     title: (v.content?.title || "").trim(),
     body: (v.content?.body || "").trim(),
   };
 }
 
 const MOCK: Row[] = [
-  { id: "sample-1", author: "Maya L.",   stars: "★★★★★", title: "Fantastic experience", body: "Best in town — friendly staff and top quality. Highly recommend!" },
-  { id: "sample-2", author: "Daniel C.", stars: "★★★★☆", title: "Great value",          body: "Really enjoyed it and will definitely be coming back." },
-  { id: "sample-3", author: "Noa B.",    stars: "★★★★★", title: "Loved it",             body: "Everything was perfect from start to finish." },
-  { id: "sample-4", author: "Amit L.",   stars: "★★★★☆", title: "Solid",                body: "Good quality and quick service. Would order again." },
+  { id: "sample-1", author: "Maya L.",   stars: "★★★★★", rating: 5, title: "Fantastic experience", body: "Best in town — friendly staff and top quality. Highly recommend!" },
+  { id: "sample-2", author: "Daniel C.", stars: "★★★★☆", rating: 4, title: "Great value",          body: "Really enjoyed it and will definitely be coming back." },
+  { id: "sample-3", author: "Noa B.",    stars: "★★★★★", rating: 5, title: "Loved it",             body: "Everything was perfect from start to finish." },
+  { id: "sample-4", author: "Amit L.",   stars: "★★★★☆", rating: 4, title: "Solid",                body: "Good quality and quick service. Would order again." },
 ];
 
 function mockState(s: S, reason: "empty" | "unavailable"): S {

package/templates/branded-lite/tier5-src/himi-store-product/index.ts

@@ -55,6 +55,12 @@ interface S {
   shareUrl: string;
   cartHint: string;
   cartCount: number;
+  /** An add-to-cart write is in flight (pending line under the CTA). */
+  adding: boolean;
+  notAdding: boolean;
+  /** The last add succeeded (confirmation + View cart under the CTA). */
+  added: boolean;
+  notAdded: boolean;
   emptyText: string;
   isLoading: boolean;
   notLoading: boolean;
@@ -145,6 +151,8 @@ function derive(s: S): S {
   return {
     ...s,
     saveLabel: s.saved ? copyParam("product.saved", "Saved") : copyParam("product.save", "Save product"),
+    notAdding: !s.adding,
+    notAdded: !s.added,
     isLoading: s.status === "loading",
     notLoading: s.status !== "loading",
     isErrored: s.status === "error",
@@ -163,6 +171,7 @@ const svc = defineService<S, {
   load: (s: S, arg?: unknown) => Promise<S>;
   setQuantity: (s: S, arg?: unknown) => S;
   addToCart: (s: S) => Promise<S>;
+  beginAdd: (s: S, arg?: unknown) => S;
   share: (s: S) => Promise<S>;
   toggleSave: (s: S) => S;
 }>("himiStoreProduct", {
@@ -179,6 +188,10 @@ const svc = defineService<S, {
     shareUrl: "",
     cartHint: "",
     cartCount: 0,
+    adding: false,
+    notAdding: true,
+    added: false,
+    notAdded: true,
     emptyText: "",
     isLoading: true,
     notLoading: false,
@@ -201,7 +214,7 @@ const svc = defineService<S, {
       // Storage problems must not prevent browsing or buying a product.
       let saved = false;
       try { saved = isProductSaved(id); } catch { /* save action reports a retryable error */ }
-      s = { ...s, saved, saveHint: "" };
+      s = { ...s, saved, saveHint: "", adding: false, added: false };
       // A fresh navigation into the product resets any quantity left over
       // from a previous visit to the same worker instance.
       if (!id) return derive({ ...s, status: "ready", quantity: 1, emptyText: copyParam("product.notFoundText", "Product not found.") });
@@ -254,25 +267,42 @@ const svc = defineService<S, {
       const quantity = parseQuantity(argValue(arg, "value"));
       return quantity ? derive({ ...s, quantity }) : s;
     },
+    // The pending half of addToCart, as its own reducer so it PUBLISHES before the
+    // network write starts: the pending line shows and the top-bar badge ticks up in the same
+    // interaction, instead of the tap looking dead until the round-trip (and the badge re-read)
+    // settle. Called from addToCart, not from the descriptor.
+    beginAdd: (s, arg) => {
+      const quantity = parseQuantity(argValue(arg, "quantity")) ?? 1;
+      return derive({ ...s, adding: true, added: false, cartHint: "", cartCount: s.cartCount + quantity });
+    },
     addToCart: async (s) => {
       const id = inputId();
       if (!id) return derive({ ...s, cartHint: copyParam("product.cartHintNoProduct", "No product to add.") });
+      // A second tap while the first write is in flight must not add the line twice.
+      if (s.adding) return s;
       const quantity = s.quantity > 0 ? s.quantity : 1;
+      const before = s.cartCount;
+      await svc.beginAdd({ quantity: String(quantity) });
+      s = svc.getState() as S;
       // Optimistic local cart line — the cart repaints from cache immediately (offline or not).
       const line: DbEntity = { id: `line-${id}`, productId: id, name: s.name || "product", title: s.name || "product", subtitle: `Qty ${quantity}`, quantity: String(quantity) };
       await cartPut(line);
       try {
         await addToCurrentCart(id, quantity);
-        return derive({ ...s, cartHint: copyTemplate("product.cartHintAdded", "Added {name} to cart.", { name: s.name || "product" }), cartCount: await cartItemCount() });
       } catch (e) {
         if (isBridgeUnavailable(e)) {
           // Offline / no bridge: the optimistic cache line already shows; queue the
           // add to replay (WriteQueue, idempotent) when connectivity returns.
           await queueWrite("tier5.himiStoreProduct.addToCart", { id });
-          return derive({ ...s, cartHint: copyTemplate("product.cartHintQueued", "Added {name} to cart. Will sync when you reconnect.", { name: s.name || "product" }) });
+          return derive({ ...s, adding: false, added: false, cartHint: copyTemplate("product.cartHintQueued", "Added {name} to cart. Will sync when you reconnect.", { name: s.name || "product" }) });
         }
-        return derive({ ...s, cartHint: copyParam("product.cartHintFailed", "Could not add to cart. Try again.") });
+        // Roll the optimistic badge back — nothing was added.
+        return derive({ ...s, adding: false, added: false, cartCount: before, cartHint: copyParam("product.cartHintFailed", "Could not add to cart. Try again.") });
       }
+      // The write LANDED. The badge re-read is outside the write's try on purpose: a failed
+      // count must never be mistaken for a failed add. Server count wins; else keep optimistic.
+      const count = await cartItemCount().catch(() => 0);
+      return derive({ ...s, adding: false, added: true, cartHint: copyTemplate("product.cartHintAdded", "Added {name} to cart.", { name: s.name || "product" }), cartCount: count || s.cartCount });
     },
     toggleSave: (s) => {
       if (s.status !== "ready" || !s.name || !inputId()) return s;
@@ -291,7 +321,8 @@ const svc = defineService<S, {
         await shareSheet({ subject: s.name, text: s.name, ...(s.shareUrl ? { url: s.shareUrl } : {}) });
         return s;
       } catch {
-        return derive({ ...s, cartHint: copyParam("product.shareUnavailable", "Sharing is not available on this device.") });
+        // `added: false` — this hint must not render inside the add-to-cart confirmation.
+        return derive({ ...s, added: false, cartHint: copyParam("product.shareUnavailable", "Sharing is not available on this device.") });
       }
     },
   },

package/templates/branded-lite/tier5-src/wix-spaces-app-language/index.ts

@@ -5,11 +5,12 @@
 import { defineService } from "@himalaya/state";
 import { dispatch } from "@himalaya/dispatch";
 
-interface Row { id: string; name: string; subtitle: string }
+/** `selected` drives the row's radio; `subtitle` stays for older descriptors. */
+interface Row { id: string; name: string; subtitle: string; selected: boolean }
 interface S { status: "ready"; options: Row[] }
 
 /** Locales with catalogs (`tools/l10n/catalog.<tag>.json`) + LocaleStore support. */
-const LOCALE_OPTIONS: Omit<Row, "subtitle">[] = [
+const LOCALE_OPTIONS: Omit<Row, "subtitle" | "selected">[] = [
   { id: "en", name: "English" },
   { id: "es", name: "Español" },
   { id: "he", name: "עברית" },
@@ -19,6 +20,7 @@ function withSelected(selected: string): Row[] {
   return LOCALE_OPTIONS.map((o) => ({
     ...o,
     subtitle: o.id === selected ? "Selected" : "",
+    selected: o.id === selected,
   }));
 }