@wix/himalaya-cli 0.1442.0 → 0.1443.0

package/dist/authoring-runtime/node_modules/@wix/himalaya/dist/_types/core/sdk/state/src/index.d.ts

@@ -15,6 +15,13 @@ export interface ServiceDefinition<S> {
      * Runtimes that honour it set `himalaya.hydrationOptOut`; an older one ignores the option.
      */
     notHydrated?: readonly (keyof S & string)[];
+    /**
+     * Keys that start each VISIT from `state`: when a cached embed shows the screen again (its warm worker
+     * reclaimed), these go back to their declared values before the screen reloads. For the same kind of
+     * per-run UI state as `notHydrated` (an editor's one-shot seed, an open sheet); never data a returning
+     * screen should keep showing. A full app's own screens and an uncached embed ignore it.
+     */
+    perVisit?: readonly (keyof S & string)[];
 }
 export interface ServiceHandle<S> {
     /** Snapshot of the current state. Cheap; safe to call from anywhere. */

package/dist/cli.mjs

@@ -8115,6 +8115,7 @@ function resolveTranslation(value, catalog3, sourceCatalog) {
 var BINDING_SIGILS, BINDING_SIGIL_RE, TRANSLATION_PREFIX, TRANSLATION_KEY_RE;
 var init_bindings = __esm({
   "../../core/ts/src/bindings.ts"() {
+    "use strict";
     BINDING_SIGILS = ["service", "flag", "item", "t"];
     BINDING_SIGIL_RE = new RegExp(
       `^\\$(?:${BINDING_SIGILS.filter((s2) => s2 !== "t").join("|")})\\b|^\\$t:`
@@ -11358,9 +11359,15 @@ async function bootBundleWorker(bundleJs, opts) {
         });
       }
       def.state = clearRuntimeError({ ...def.state, ...stripped });
+      published(def);
       return;
     }
     def.state = clearRuntimeError({ ...def.state, ...rec });
+    published(def);
+  };
+  const published = (def) => {
+    if (!opts.onPublish) return;
+    opts.onPublish(captured.find(({ def: d2 }) => d2 === def)?.name ?? "", clone(def.state));
   };
   function clearRuntimeError(state) {
     delete state._error;
@@ -161506,32 +161513,32 @@ var init_preview_assets_generated = __esm({
   "../../packages/serve/src/preview-assets.generated.ts"() {
     init_preview_assets_format();
     PREVIEW_ASSET_MANIFEST = [
-      { key: "assets/DashboardHost-3-2Jf3cl.js", contentType: "application/javascript; charset=utf-8", offset: 8, length: 2916, sha256: "16067cee02695e0e78e872b92dd41d85dd30c9c29d04f45db43126c4452b0156" },
-      { key: "assets/DevDrawer-CiwSZJ0n.js", contentType: "application/javascript; charset=utf-8", offset: 2924, length: 16744, sha256: "a570bdc5f979ddd8b5c677886d980f41ff8d7c9e54a0461487cd1b9e5c83ad66" },
-      { key: "assets/IdiomCompare-CJAnEFoY.js", contentType: "application/javascript; charset=utf-8", offset: 19668, length: 1892, sha256: "4ede78710504fa2f22e7e4dc75f95c52f64b0d7c8defb53333ce4ed32ae678f6" },
-      { key: "assets/LightboxViewer-BOY9iVs-.js", contentType: "application/javascript; charset=utf-8", offset: 21560, length: 8132, sha256: "0d11ab35256faae44aed69854dc3b5cf42bdcfefaccb6cc833ecb401ee5c6e03" },
-      { key: "assets/PhonePreviewControl-BqaJPdhq.js", contentType: "application/javascript; charset=utf-8", offset: 29692, length: 37729, sha256: "2373e4117a568f4788edec591429b6c81291b572caf11d511e1d17f10701da5a" },
-      { key: "assets/SchemeCompare-j4m0nu1j.js", contentType: "application/javascript; charset=utf-8", offset: 67421, length: 1824, sha256: "4e41c913297e35eda4816f5646d678f83eeb974944378f2627a31ad97c5829df" },
-      { key: "assets/index-Cak7Oz8v.js", contentType: "application/javascript; charset=utf-8", offset: 69245, length: 1266847, sha256: "015168656f2d2f8a22589c77e2f46cc917e7fc2dfa96c8f2f498acb0e4778946" },
-      { key: "assets/pdf-BdBVqwzN.js", contentType: "application/javascript; charset=utf-8", offset: 1336092, length: 488199, sha256: "cfdedd9cca30899c52b4d5d14612b20ead4699afef16253f3d8a1b4995cffff0" },
-      { key: "assets/pdf.worker.min-BQa_JDET.js", contentType: "application/javascript; charset=utf-8", offset: 1824291, length: 73, sha256: "41bcfda4f3c8c0a81c785ea87c69afc8e7d0ed8c333324aee94c273d881bb6aa" },
-      { key: "assets/pdf.worker.min-BmVo14Nb.mjs", contentType: "application/javascript; charset=utf-8", offset: 1824364, length: 1317034, sha256: "a33cfe728c584fdba4fcc1fd54bcdc2f9f2f13889ddbb5b2bd1d0f8cbe49b84e" },
-      { key: "assets/pdfDocument-CWUvZXa-.js", contentType: "application/javascript; charset=utf-8", offset: 3141398, length: 2868, sha256: "d06defc34d8fb99209b44a1f16397d33c2fa0f36c72a76692261ee2900304f9b" },
-      { key: "fonts/Roboto-LICENSE.txt", contentType: "application/octet-stream", offset: 3144266, length: 4610, sha256: "994062dd7af4186a739de55dd9e151c63b6301f8e2cbafc608d35c2e87da0cd2" },
-      { key: "fonts/Roboto-Variable.ttf", contentType: "font/ttf", offset: 3148876, length: 488584, sha256: "d7598e12c5dbef095ff8272cfc55da0250bd07fbdecbac8a530b9b277872a134" },
-      { key: "fonts/WixMadeforDisplay-Bold.ttf", contentType: "font/ttf", offset: 3637460, length: 90924, sha256: "ec4c7d63772d980d6e147b2df6215fd2737e4b04cebe0cf7e11dc695fb63cdcf" },
-      { key: "fonts/WixMadeforDisplay-ExtraBold.ttf", contentType: "font/ttf", offset: 3728384, length: 90984, sha256: "bf4c16a2774edd38a921edf3099469866cd74c094fd596f3fa4e81f54a04dfaa" },
-      { key: "fonts/WixMadeforDisplay-Medium.ttf", contentType: "font/ttf", offset: 3819368, length: 90980, sha256: "368faca44fb9f7207e4af784e94750b1ee59c1098b8689e0f4a918926f244fb5" },
-      { key: "fonts/WixMadeforDisplay-Regular.ttf", contentType: "font/ttf", offset: 3910348, length: 90872, sha256: "97ed8f1a6772abe26e3d8af5cecfc4b02faa94497d585b0398b5053b739a70c1" },
-      { key: "fonts/WixMadeforDisplay-SemiBold.ttf", contentType: "font/ttf", offset: 4001220, length: 90980, sha256: "642feab8a48ddf6c7a89d7c2a7c0edfa0e6e773f671287e460e222703563191e" },
-      { key: "fonts/WixMadeforText-Bold.ttf", contentType: "font/ttf", offset: 4092200, length: 108636, sha256: "37c179b4fefb63ea9637322b84583a2df53aa226c150294eea1675c14b1b8e29" },
-      { key: "fonts/WixMadeforText-ExtraBold.ttf", contentType: "font/ttf", offset: 4200836, length: 109084, sha256: "62e2fe4718129f36ecdb0ae4de385bd6b81ecc876146447167c6c140fad699a9" },
-      { key: "fonts/WixMadeforText-Medium.ttf", contentType: "font/ttf", offset: 4309920, length: 109048, sha256: "4ecf270136af928612c76fe4ebade91d8156975e5ea4c0cf9225501dd1788e02" },
-      { key: "fonts/WixMadeforText-Regular.ttf", contentType: "font/ttf", offset: 4418968, length: 107700, sha256: "dee56f09fd3efc127c1ea95557edc74110c1460a46287da36e0af25efd341ef5" },
-      { key: "fonts/WixMadeforText-SemiBold.ttf", contentType: "font/ttf", offset: 4526668, length: 88108, sha256: "dced97a77624bc5f8b1ae65850360908297f4588e09feb6f22792d29dc7e765b" },
-      { key: "himi-sw.js", contentType: "application/javascript; charset=utf-8", offset: 4614776, length: 6748, sha256: "4a92459977d7cefa1a6a7e020c3a63509737f083b3f2773557c1a5771665811a" },
-      { key: "index.html", contentType: "text/html; charset=utf-8", offset: 4621524, length: 4947, sha256: "3b2cf7b22b8a77eaabfb1634073d256c7aa367b91d0cc70081bea99b12b77fb0" },
-      { key: "t5-frame.html", contentType: "text/html; charset=utf-8", offset: 4626471, length: 17786, sha256: "084107ca997628c248f1b0ba3a7494744850df39fad813546ab8bdb82a6058da" }
+      { key: "assets/DashboardHost-CWbeef06.js", contentType: "application/javascript; charset=utf-8", offset: 8, length: 2916, sha256: "9ab72647e78bee902e76c4672f3d22b18d9b7a79892e0e64ae80ffb574417911" },
+      { key: "assets/DevDrawer-mM0I8ALw.js", contentType: "application/javascript; charset=utf-8", offset: 2924, length: 16744, sha256: "9bbb22f0ee24b6782c4645da75788028c942122748f0149d00e9039cdd09f0b1" },
+      { key: "assets/IdiomCompare-SDHU6iNa.js", contentType: "application/javascript; charset=utf-8", offset: 19668, length: 1892, sha256: "2af535cb7bb80d6f90d99bbb67470e4f9754949b62e08f4767a977a3414ae43e" },
+      { key: "assets/LightboxViewer-YV8gCISo.js", contentType: "application/javascript; charset=utf-8", offset: 21560, length: 8132, sha256: "e3c873007b6790042640966c3fb0d19f16d2f0ca4c41276838d38e90c852391a" },
+      { key: "assets/PhonePreviewControl-Dl_3aZaI.js", contentType: "application/javascript; charset=utf-8", offset: 29692, length: 37729, sha256: "f8d15085d2b83522584fd3a30bc5bead6d68c885e2987e7df12035c45d1aa003" },
+      { key: "assets/SchemeCompare-VEnYs5jY.js", contentType: "application/javascript; charset=utf-8", offset: 67421, length: 1824, sha256: "9f51da5008d043106847efb01ab675ae9c4ed58e6745cba03b1fd0b5f173a520" },
+      { key: "assets/index-zKquXoS4.js", contentType: "application/javascript; charset=utf-8", offset: 69245, length: 1268324, sha256: "4d6fe8fe1939fbc4a50b528233ff482afc6f615a0df92b54847386d0557b837a" },
+      { key: "assets/pdf-ipblMaD0.js", contentType: "application/javascript; charset=utf-8", offset: 1337569, length: 488199, sha256: "f103cd80a3f9a971e1be6162886350314c28bd42c1cb998210147e4a5a23a947" },
+      { key: "assets/pdf.worker.min-BQa_JDET.js", contentType: "application/javascript; charset=utf-8", offset: 1825768, length: 73, sha256: "41bcfda4f3c8c0a81c785ea87c69afc8e7d0ed8c333324aee94c273d881bb6aa" },
+      { key: "assets/pdf.worker.min-BmVo14Nb.mjs", contentType: "application/javascript; charset=utf-8", offset: 1825841, length: 1317034, sha256: "a33cfe728c584fdba4fcc1fd54bcdc2f9f2f13889ddbb5b2bd1d0f8cbe49b84e" },
+      { key: "assets/pdfDocument-BLnEEkur.js", contentType: "application/javascript; charset=utf-8", offset: 3142875, length: 2868, sha256: "3913f6a2cc3e3eda95617b848a6240dfd132d2da9629fc429b144b722401e2f0" },
+      { key: "fonts/Roboto-LICENSE.txt", contentType: "application/octet-stream", offset: 3145743, length: 4610, sha256: "994062dd7af4186a739de55dd9e151c63b6301f8e2cbafc608d35c2e87da0cd2" },
+      { key: "fonts/Roboto-Variable.ttf", contentType: "font/ttf", offset: 3150353, length: 488584, sha256: "d7598e12c5dbef095ff8272cfc55da0250bd07fbdecbac8a530b9b277872a134" },
+      { key: "fonts/WixMadeforDisplay-Bold.ttf", contentType: "font/ttf", offset: 3638937, length: 90924, sha256: "ec4c7d63772d980d6e147b2df6215fd2737e4b04cebe0cf7e11dc695fb63cdcf" },
+      { key: "fonts/WixMadeforDisplay-ExtraBold.ttf", contentType: "font/ttf", offset: 3729861, length: 90984, sha256: "bf4c16a2774edd38a921edf3099469866cd74c094fd596f3fa4e81f54a04dfaa" },
+      { key: "fonts/WixMadeforDisplay-Medium.ttf", contentType: "font/ttf", offset: 3820845, length: 90980, sha256: "368faca44fb9f7207e4af784e94750b1ee59c1098b8689e0f4a918926f244fb5" },
+      { key: "fonts/WixMadeforDisplay-Regular.ttf", contentType: "font/ttf", offset: 3911825, length: 90872, sha256: "97ed8f1a6772abe26e3d8af5cecfc4b02faa94497d585b0398b5053b739a70c1" },
+      { key: "fonts/WixMadeforDisplay-SemiBold.ttf", contentType: "font/ttf", offset: 4002697, length: 90980, sha256: "642feab8a48ddf6c7a89d7c2a7c0edfa0e6e773f671287e460e222703563191e" },
+      { key: "fonts/WixMadeforText-Bold.ttf", contentType: "font/ttf", offset: 4093677, length: 108636, sha256: "37c179b4fefb63ea9637322b84583a2df53aa226c150294eea1675c14b1b8e29" },
+      { key: "fonts/WixMadeforText-ExtraBold.ttf", contentType: "font/ttf", offset: 4202313, length: 109084, sha256: "62e2fe4718129f36ecdb0ae4de385bd6b81ecc876146447167c6c140fad699a9" },
+      { key: "fonts/WixMadeforText-Medium.ttf", contentType: "font/ttf", offset: 4311397, length: 109048, sha256: "4ecf270136af928612c76fe4ebade91d8156975e5ea4c0cf9225501dd1788e02" },
+      { key: "fonts/WixMadeforText-Regular.ttf", contentType: "font/ttf", offset: 4420445, length: 107700, sha256: "dee56f09fd3efc127c1ea95557edc74110c1460a46287da36e0af25efd341ef5" },
+      { key: "fonts/WixMadeforText-SemiBold.ttf", contentType: "font/ttf", offset: 4528145, length: 88108, sha256: "dced97a77624bc5f8b1ae65850360908297f4588e09feb6f22792d29dc7e765b" },
+      { key: "himi-sw.js", contentType: "application/javascript; charset=utf-8", offset: 4616253, length: 6748, sha256: "4a92459977d7cefa1a6a7e020c3a63509737f083b3f2773557c1a5771665811a" },
+      { key: "index.html", contentType: "text/html; charset=utf-8", offset: 4623001, length: 4947, sha256: "ac12b463d6c2a4195ff7f6fb911b79c7e14de48488493bf6480b660866f833f9" },
+      { key: "t5-frame.html", contentType: "text/html; charset=utf-8", offset: 4627948, length: 17786, sha256: "084107ca997628c248f1b0ba3a7494744850df39fad813546ab8bdb82a6058da" }
     ];
   }
 });

package/guides/sdk-reference.json

@@ -8,7 +8,7 @@
         "name": "ServiceDefinition",
         "kind": "interface",
         "jsdoc": "",
-        "decl": "export interface ServiceDefinition<S> {\n  /** Initial state for the service. Must be JSON-serializable. */\n  state: S;\n\n  /**\n   * Actions: pure functions of `(state, ...args) → newState` (or async). The\n   * platform wraps each action so calling it from JS or from `__invokeAction`\n   * (native side) updates the service's state and notifies subscribers.\n   */\n  actions?: Record<\n    string,\n    // `never[]`, not `unknown[]` — same reason as defineService's constraint below:\n    // parameters are contravariant, so `unknown[]` rejects every handler that narrows\n    // its payload. Leaving this one loose would reintroduce the same TS2344 for anyone\n    // who annotates a `ServiceDefinition<S>` explicitly instead of inferring it.\n    (state: S, ...args: never[]) => S | undefined | Promise<S | undefined>\n  >;\n\n  /**\n   * Keys a cold boot takes from `state`, never from the persisted snapshot (cold-boot\n   * hydration). For UI state that must describe THIS run — an editor's one-shot seed, a\n   * \"still loading\" gate, an open sheet — where last run's value would be wrong, not just stale.\n   * Runtimes that honour it set `himalaya.hydrationOptOut`; an older one ignores the option.\n   */\n  notHydrated?: readonly (keyof S & string)[];\n}"
+        "decl": "export interface ServiceDefinition<S> {\n  /** Initial state for the service. Must be JSON-serializable. */\n  state: S;\n\n  /**\n   * Actions: pure functions of `(state, ...args) → newState` (or async). The\n   * platform wraps each action so calling it from JS or from `__invokeAction`\n   * (native side) updates the service's state and notifies subscribers.\n   */\n  actions?: Record<\n    string,\n    // `never[]`, not `unknown[]` — same reason as defineService's constraint below:\n    // parameters are contravariant, so `unknown[]` rejects every handler that narrows\n    // its payload. Leaving this one loose would reintroduce the same TS2344 for anyone\n    // who annotates a `ServiceDefinition<S>` explicitly instead of inferring it.\n    (state: S, ...args: never[]) => S | undefined | Promise<S | undefined>\n  >;\n\n  /**\n   * Keys a cold boot takes from `state`, never from the persisted snapshot (cold-boot\n   * hydration). For UI state that must describe THIS run — an editor's one-shot seed, a\n   * \"still loading\" gate, an open sheet — where last run's value would be wrong, not just stale.\n   * Runtimes that honour it set `himalaya.hydrationOptOut`; an older one ignores the option.\n   */\n  notHydrated?: readonly (keyof S & string)[];\n\n  /**\n   * Keys that start each VISIT from `state`: when a cached embed shows the screen again (its warm worker\n   * reclaimed), these go back to their declared values before the screen reloads. For the same kind of\n   * per-run UI state as `notHydrated` (an editor's one-shot seed, an open sheet); never data a returning\n   * screen should keep showing. A full app's own screens and an uncached embed ignore it.\n   */\n  perVisit?: readonly (keyof S & string)[];\n}"
       },
       {
         "name": "ServiceHandle",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wix/himalaya-cli",
-  "version": "0.1442.0",
+  "version": "0.1443.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": [
@@ -71,5 +71,5 @@
       "artifactId": "himalaya-cli"
     }
   },
-  "falconPackageHash": "a9410b54858e369de3dc999ca67e799366424e27e7a49022ff34cdbf"
+  "falconPackageHash": "8a70f3bc09b20693d0d5f9f3f9412dc5cdbd20ee62407d59ff646f7f"
 }

package/templates/beauty-salon/tier5-src/_shared/collection-query.ts

@@ -27,7 +27,9 @@ export interface CollectionQuerySource<Row> {
   cached?(criteria: CollectionQuery): Promise<CollectionPage<Row> | undefined>;
   id(row: Row): string;
   errorMessage?(error: unknown): string;
-  /** Private collections must discard prior rows when access or identity changes. */
+  /** Private collections must discard prior rows when access or identity changes. A revalidation keeps the
+   *  rows it shows through any other failure: safe because one worker never serves two identities (its pool
+   *  key carries the account, and a purge tears parked workers down). */
   clearOnError?(error: unknown): boolean;
 }
 const normalize = (q: Partial<CollectionQuery>): CollectionQuery => ({
@@ -46,8 +48,17 @@ export function createCollectionQuery<Row>(source: CollectionQuerySource<Row>, p
   let generation = 0, disposed = false;
   let loadedPages = 0, restoringPages = 0;
   let state: CollectionQueryState<Row> = { criteria: normalize({}), items: [], nextCursor: '', status: 'idle', loadingMore: false, pageError: false, complete: false, error: '' };
+  /** The query a revalidation is running under: its snapshots are held back until it is done. */
+  let revalidating = -1;
+  /** What the screen shows: the last published state, restored when a revalidation fails. */
+  let shown: { state: CollectionQueryState<Row>; loadedPages: number } | undefined;
+  let clearedByError = false;
   const snapshot = (): CollectionQueryState<Row> => ({ ...state, criteria: copyQuery(state.criteria), items: [...state.items] });
-  const emit = () => { if (!disposed) publish(snapshot()); };
+  const emit = () => {
+    if (disposed || revalidating === generation) return;
+    shown = { state: snapshot(), loadedPages };
+    publish(snapshot());
+  };
   const merge = (old: Row[], page: Row[]): Row[] => {
     const positions = new Map<string, number>(), result: Row[] = [];
     for (const item of [...old, ...page]) {
@@ -88,6 +99,8 @@ export function createCollectionQuery<Row>(source: CollectionQuerySource<Row>, p
     } catch (error) {
       if (disposed || generation !== revision) return;
       const clear = source.clearOnError?.(error) === true;
+      // Revoked access always ends what a revalidation showed, whether or not the source clears its rows.
+      clearedByError = clear || /HTTP 40[13]\b/.test(String(error));
       if (clear) loadedPages = 0;
       state = { ...state, ...(clear ? { items: [], nextCursor: '', total: undefined, metadata: undefined } : {}),
         status: append && !clear ? 'ready' : 'error', loadingMore: false, pageError: append && !clear, complete: false,
@@ -97,13 +110,18 @@ export function createCollectionQuery<Row>(source: CollectionQuerySource<Row>, p
   };
   return {
     getState: snapshot,
-    async query(criteria: Partial<CollectionQuery>, restorePages = false): Promise<void> {
+    /** [revalidate]: a screen coming back re-queries the rows it shows. They stay on screen until every
+     *  page is in, and a failure keeps them; revoked access still clears them, and other criteria reset. */
+    async query(criteria: Partial<CollectionQuery>, restorePages = false, revalidate = false): Promise<void> {
       if (disposed) return Promise.resolve();
       const next = normalize(criteria);
-      const count = restorePages && JSON.stringify(next) === JSON.stringify(state.criteria)
-        ? Math.max(loadedPages, restoringPages, 1) : 1;
+      const same = JSON.stringify(next) === JSON.stringify(state.criteria);
+      const count = restorePages && same ? Math.max(loadedPages, restoringPages, 1) : 1;
+      const showing = shown && shown.state.status === 'ready' && shown.state.items.length > 0 && !shown.state.pageError
+        && !shown.state.loadingMore && JSON.stringify(shown.state.criteria) === JSON.stringify(next) ? shown : undefined;
       restoringPages = count;
       state = { ...state, criteria: next };
+      if (revalidate && showing) { revalidating = generation + 1; clearedByError = false; }
       const pending = fetch(false);
       const revision = generation;
       await pending;
@@ -112,6 +130,15 @@ export function createCollectionQuery<Row>(source: CollectionQuerySource<Row>, p
       while (!disposed && generation === revision && loadedPages < count && state.status === 'ready'
         && !state.pageError && !state.loadingMore && state.nextCursor) await fetch(true);
       if (generation === revision) restoringPages = 0;
+      if (revalidating === revision) {
+        revalidating = -1;
+        if (generation !== revision) return;
+        if ((state.status !== 'ready' || state.pageError) && showing && !clearedByError) {
+          state = { ...showing.state, criteria: next };
+          loadedPages = showing.loadedPages;
+        }
+        emit();
+      }
     },
     refresh: () => { restoringPages = 0; return fetch(false); },
     loadMore: () => fetch(true),

package/templates/beauty-salon/tier5-src/_shared/collection-service.ts

@@ -299,7 +299,8 @@ export function decorateCollection(name: string, def: { state: State; actions?:
     actions.load = (_s, arg) => {
       stashCopy(arg); syncSaved(); applyNavigationCriteria(arg, _s);
       backend.onLoad?.(arg);
-      return backend.controller.query(criteria(), backend.restorePagesOnLoad).then(() => undefined);
+      // A load of what the screen already shows is the screen coming back: it revalidates behind its rows.
+      return backend.controller.query(criteria(), backend.restorePagesOnLoad, true).then(() => undefined);
     };
     actions.refresh = (_s, arg) => {
       stashCopy(arg); syncSaved();

package/templates/beauty-salon-owner/tier5-src/_shared/collection-query.ts

@@ -27,7 +27,9 @@ export interface CollectionQuerySource<Row> {
   cached?(criteria: CollectionQuery): Promise<CollectionPage<Row> | undefined>;
   id(row: Row): string;
   errorMessage?(error: unknown): string;
-  /** Private collections must discard prior rows when access or identity changes. */
+  /** Private collections must discard prior rows when access or identity changes. A revalidation keeps the
+   *  rows it shows through any other failure: safe because one worker never serves two identities (its pool
+   *  key carries the account, and a purge tears parked workers down). */
   clearOnError?(error: unknown): boolean;
 }
 const normalize = (q: Partial<CollectionQuery>): CollectionQuery => ({
@@ -46,8 +48,17 @@ export function createCollectionQuery<Row>(source: CollectionQuerySource<Row>, p
   let generation = 0, disposed = false;
   let loadedPages = 0, restoringPages = 0;
   let state: CollectionQueryState<Row> = { criteria: normalize({}), items: [], nextCursor: '', status: 'idle', loadingMore: false, pageError: false, complete: false, error: '' };
+  /** The query a revalidation is running under: its snapshots are held back until it is done. */
+  let revalidating = -1;
+  /** What the screen shows: the last published state, restored when a revalidation fails. */
+  let shown: { state: CollectionQueryState<Row>; loadedPages: number } | undefined;
+  let clearedByError = false;
   const snapshot = (): CollectionQueryState<Row> => ({ ...state, criteria: copyQuery(state.criteria), items: [...state.items] });
-  const emit = () => { if (!disposed) publish(snapshot()); };
+  const emit = () => {
+    if (disposed || revalidating === generation) return;
+    shown = { state: snapshot(), loadedPages };
+    publish(snapshot());
+  };
   const merge = (old: Row[], page: Row[]): Row[] => {
     const positions = new Map<string, number>(), result: Row[] = [];
     for (const item of [...old, ...page]) {
@@ -88,6 +99,8 @@ export function createCollectionQuery<Row>(source: CollectionQuerySource<Row>, p
     } catch (error) {
       if (disposed || generation !== revision) return;
       const clear = source.clearOnError?.(error) === true;
+      // Revoked access always ends what a revalidation showed, whether or not the source clears its rows.
+      clearedByError = clear || /HTTP 40[13]\b/.test(String(error));
       if (clear) loadedPages = 0;
       state = { ...state, ...(clear ? { items: [], nextCursor: '', total: undefined, metadata: undefined } : {}),
         status: append && !clear ? 'ready' : 'error', loadingMore: false, pageError: append && !clear, complete: false,
@@ -97,13 +110,18 @@ export function createCollectionQuery<Row>(source: CollectionQuerySource<Row>, p
   };
   return {
     getState: snapshot,
-    async query(criteria: Partial<CollectionQuery>, restorePages = false): Promise<void> {
+    /** [revalidate]: a screen coming back re-queries the rows it shows. They stay on screen until every
+     *  page is in, and a failure keeps them; revoked access still clears them, and other criteria reset. */
+    async query(criteria: Partial<CollectionQuery>, restorePages = false, revalidate = false): Promise<void> {
       if (disposed) return Promise.resolve();
       const next = normalize(criteria);
-      const count = restorePages && JSON.stringify(next) === JSON.stringify(state.criteria)
-        ? Math.max(loadedPages, restoringPages, 1) : 1;
+      const same = JSON.stringify(next) === JSON.stringify(state.criteria);
+      const count = restorePages && same ? Math.max(loadedPages, restoringPages, 1) : 1;
+      const showing = shown && shown.state.status === 'ready' && shown.state.items.length > 0 && !shown.state.pageError
+        && !shown.state.loadingMore && JSON.stringify(shown.state.criteria) === JSON.stringify(next) ? shown : undefined;
       restoringPages = count;
       state = { ...state, criteria: next };
+      if (revalidate && showing) { revalidating = generation + 1; clearedByError = false; }
       const pending = fetch(false);
       const revision = generation;
       await pending;
@@ -112,6 +130,15 @@ export function createCollectionQuery<Row>(source: CollectionQuerySource<Row>, p
       while (!disposed && generation === revision && loadedPages < count && state.status === 'ready'
         && !state.pageError && !state.loadingMore && state.nextCursor) await fetch(true);
       if (generation === revision) restoringPages = 0;
+      if (revalidating === revision) {
+        revalidating = -1;
+        if (generation !== revision) return;
+        if ((state.status !== 'ready' || state.pageError) && showing && !clearedByError) {
+          state = { ...showing.state, criteria: next };
+          loadedPages = showing.loadedPages;
+        }
+        emit();
+      }
     },
     refresh: () => { restoringPages = 0; return fetch(false); },
     loadMore: () => fetch(true),

package/templates/beauty-salon-owner/tier5-src/_shared/collection-service.ts

@@ -299,7 +299,8 @@ export function decorateCollection(name: string, def: { state: State; actions?:
     actions.load = (_s, arg) => {
       stashCopy(arg); syncSaved(); applyNavigationCriteria(arg, _s);
       backend.onLoad?.(arg);
-      return backend.controller.query(criteria(), backend.restorePagesOnLoad).then(() => undefined);
+      // A load of what the screen already shows is the screen coming back: it revalidates behind its rows.
+      return backend.controller.query(criteria(), backend.restorePagesOnLoad, true).then(() => undefined);
     };
     actions.refresh = (_s, arg) => {
       stashCopy(arg); syncSaved();

package/templates/beauty-salon-owner/tier5-src/_shared/owner-editor-value.ts

@@ -91,6 +91,16 @@ export function editorRunKeys<K extends string>(key: K): [`${K}Json`, `${K}Pendi
   return [`${key}Json`, `${key}Pending`, `${key}Dirty`, `${key}Edits`];
 }
 
+/** Does the form already hold `target` with its editor run in progress? A `load` for it then keeps the form as
+ *  it is: on the way back to a warm worker (its refresh and onAppear) a re-read would race an edit in flight and
+ *  reset the reported edit, and the editor, which seeds once, would show another document than the state holds.
+ *  An edit form holds its record once the editor loaded; a create form holds a draft once the owner edited. */
+export function holdsLoadedForm(state: object, target: { id: string; mode: string }, key: string): boolean {
+  const s = state as Record<string, unknown>;
+  if (s.status === 'error' || s.error || s.mode !== target.mode || (s.id ?? '') !== target.id) return false;
+  return target.mode === 'create' ? s[`${key}Dirty`] === true || Number(s[`${key}Edits`] ?? 0) > 0 : s[`${key}Pending`] === false;
+}
+
 /** The same keys on a runtime that ignores `notHydrated` (a binary from before #3644, still served new
  *  bundles over the air): reset them in place, before any action runs. A no-op where the runtime
  *  honours the option (`himalaya.hydrationOptOut`). */

package/templates/beauty-salon-owner/tier5-src/himi-owner-lite-manage/index.ts

@@ -214,6 +214,22 @@ let loadGeneration = 0;
 let newestStarted: State | undefined;
 let newestSettled: { generation: number; state: State } | undefined;
 
+/** A hub already showing its areas re-reads their summaries behind them, never back under the skeleton: the
+ *  screen coming back (a warm worker's refresh and its onAppear, at once) keeps what it shows until they land. */
+function revalidating(state: State, areas: State['areas'] = state.areas): State {
+  // The areas as delivered now (labels, routes), each still showing the summary it showed.
+  const shown = new Map(state.areas.map(area => [area.id, area]));
+  const next = areas.map(area => {
+    const was = shown.get(area.id);
+    return was ? { ...area, metricText: was.metricText, statusText: was.statusText, unavailable: was.unavailable } : area;
+  });
+  return derive({ ...state, areas: next, businessName: currentBusinessName, handoffError: '' }, next);
+}
+
+function sameAreas(shown: State['areas'], next: State['areas']): boolean {
+  return shown.length === next.length && shown.every((area, i) => area.id === next[i].id && area.metricKey === next[i].metricKey);
+}
+
 async function loadMetrics(state: State): Promise<State> {
   const generation = ++loadGeneration;
   newestStarted = state;
@@ -292,12 +308,11 @@ const svc = defineService<State, {
       // it must not settle (that derived "0 business areas" and painted the
       // filtered-empty state); once they are, it re-reads their summaries.
       if (!('areas' in read(arg))) {
-        return state.status === 'ready'
-          ? loadMetrics(derive({ ...state, isLoading: true, notLoading: false, summaryNote: '', handoffError: '' }))
-          : { ...state, businessName: currentBusinessName };
+        return state.status === 'ready' ? loadMetrics(revalidating(state)) : { ...state, businessName: currentBusinessName };
       }
       const parsed = (() => { try { return JSON.parse(str(arg, 'areas') || '[]'); } catch { return []; } })();
       const areas = parseAreas(parsed);
+      if (state.status === 'ready' && sameAreas(state.areas, areas)) return loadMetrics(revalidating(state, areas));
       return loadMetrics(derive({ ...state, status: 'ready', areas, isLoading: true, notLoading: false, summaryNote: '', handoffError: '' }, areas));
     },
     // Refresh re-reads summaries for KNOWN areas. Before the descriptor has

package/templates/beauty-salon-owner/tier5-src/himi-owner-lite-post/index.ts

@@ -340,6 +340,8 @@ const svc = defineService<PostState, {
 }>("himiOwnerLitePost", {
   state: shape({ status: "loading" }),
   notHydrated: [...DELETE_RUN_KEYS],
+  // perVisit: a cached embed's warm return starts this run fresh, as a cold boot does (`__resetRunState`).
+  perVisit: [...DELETE_RUN_KEYS],
   actions: {
     load: async (s0, arg) => {
       const s = disarm(s0);

package/templates/beauty-salon-owner/tier5-src/himi-owner-lite-post-form/index.ts

@@ -4,7 +4,7 @@
 // screen (himiOwnerLitePost.publish). Bound by buildForm ($service:himiOwnerLitePostForm.*).
 
 import { postBodyText, plainPostBody, postCoverUrl } from "../_shared/owner-post-content.js";
-import { editorSubmitValue, queuedDocument, editorChanged, editorRunKeys, resetHydratedEditorOnOlderRuntime, editorAfterSave, liveState, settleOverLive } from "../_shared/owner-editor-value.js";
+import { editorSubmitValue, queuedDocument, editorChanged, editorRunKeys, resetHydratedEditorOnOlderRuntime, editorAfterSave, liveState, settleOverLive, holdsLoadedForm } from "../_shared/owner-editor-value.js";
 import { ricosUploadService } from "@himalaya/wix-rich-content-editor";
 import { wixCall } from "@himalaya/wix-site-media";
 import { wixMediaUrl, WIX_DETAIL_IMAGE } from "../_shared/himi-store.js";
@@ -165,18 +165,24 @@ const svc = defineService<PostFormState, {
   bodyChanged: (s: PostFormState, arg?: unknown) => PostFormState;
   submit: (s: PostFormState, values?: Record<string, unknown>) => Promise<PostFormState>;
 }>("himiOwnerLitePostForm", {
-  state: shape({ mode: startMode, id: initialInput.id ?? "", bodyPending: startMode === "edit" }),
+  // The editor stays hidden until this run's load lands, however the form was opened (an id as input or in the params).
+  state: shape({ mode: startMode, id: initialInput.id ?? "", bodyPending: true }),
   // [rico] #3644 — this run's editor state and UI: a cold boot takes these from `state`, never from
   // the persisted snapshot (the editor seeds once; last run's sheet is not this run's).
   notHydrated: [...editorRunKeys("body"), "detailsOpen"],
+  // perVisit: a cached embed's warm return starts this run fresh, as a cold boot does (`__resetRunState`).
+  perVisit: [...editorRunKeys("body"), "detailsOpen"],
   actions: {
     load: async (s, arg) => {
       stashCopy(arg); // absorb the descriptor's onAppear copy-pack param
       // WS-C — nav-param input is PRIMARY (the canonical create-or-edit shell
       // bakes no id/mode); the onAppear `id` param is the #570 suffixed-build
       // fallback. An id ⇒ edit; no id anywhere ⇒ the create shell.
-      const id = navInputId() || idFrom(arg, s.id);
-      if (!id) return shape({ ...s, ...BODY_LOADED, mode: "create", id: "", bodyJson: "", status: "idle", error: "", demoNote: isOwnerPreviewDemo() ? copyParam("ownerCommon.demoSaveNote", DEMO_SAVE_NOTE) : "" });
+      // The held id is for the host's bare refresh only (no argument): an onAppear without an id is a create screen.
+      const id = navInputId() || idFrom(arg, arg === undefined ? s.id : "");
+      if (holdsLoadedForm(s, { id, mode: id ? "edit" : "create" }, "body")) return s;
+      // A form that held a record (the one just saved) opens blank.
+      if (!id) return shape({ ...(s.id ? {} : s), ...BODY_LOADED, mode: "create", id: "", bodyJson: "", status: "idle", error: "", demoNote: isOwnerPreviewDemo() ? copyParam("ownerCommon.demoSaveNote", DEMO_SAVE_NOTE) : "" });
       if (isOwnerPreviewDemo()) {
           const d = DEMO_POSTS.find((x) => x.id === id);
           if (d) {

package/templates/beauty-salon-owner/tier5-src/himi-owner-lite-product/index.ts

@@ -305,6 +305,8 @@ const svc = defineService<ProductState, {
 }>("himiOwnerLiteProduct", {
   state: shape({ status: "loading" }),
   notHydrated: [...DELETE_RUN_KEYS],
+  // perVisit: a cached embed's warm return starts this run fresh, as a cold boot does (`__resetRunState`).
+  perVisit: [...DELETE_RUN_KEYS],
   actions: {
     load: async (s0, arg) => {
       const s = disarm(s0);

package/templates/beauty-salon-owner/tier5-src/himi-owner-lite-product-form/index.ts

@@ -18,7 +18,7 @@ import {
 import { get as dbGet, upsert as dbUpsert, remove as dbRemove, type DbEntity } from "../_shared/owner-local-db.js";
 import { htmlIsTrivial, htmlPreviewText } from "../_shared/owner-rich-guard.js";
 import { postBodyText, plainPostBody } from "../_shared/owner-post-content.js";
-import { editorSubmitValue, queuedDocument, editorChanged, cachedRichVerdict, editorRunKeys, resetHydratedEditorOnOlderRuntime, editorAfterSave, liveState, settleOverLive } from "../_shared/owner-editor-value.js";
+import { editorSubmitValue, queuedDocument, editorChanged, cachedRichVerdict, editorRunKeys, resetHydratedEditorOnOlderRuntime, editorAfterSave, liveState, settleOverLive, holdsLoadedForm } from "../_shared/owner-editor-value.js";
 import { parseRicosJson } from "@himalaya/wix-rich-content";
 import { ricosUploadService } from "@himalaya/wix-rich-content-editor";
 import { ownerLoadError, isOwnerPreviewDemo, ownerDemoFallback, ownerWriteError } from "../_shared/owner-load-error.js";
@@ -195,17 +195,23 @@ const svc = defineService<ProductFormState, {
   submit: (s: ProductFormState, values?: Record<string, unknown>) => Promise<ProductFormState>;
   remove: (s: ProductFormState) => Promise<ProductFormState>;
 }>("himiOwnerLiteProductForm", {
-  state: shape({ mode: startMode, id: initialInput.id ?? "", descriptionPending: startMode === "edit" }),
+  // The editor stays hidden until this run's load lands, however the form was opened (an id as input or in the params).
+  state: shape({ mode: startMode, id: initialInput.id ?? "", descriptionPending: true }),
   // [rico] #3644 — this run's editor state: a cold boot takes it from `state`, never the snapshot.
   notHydrated: editorRunKeys("description"),
+  // perVisit: a cached embed's warm return starts this run fresh, as a cold boot does (`__resetRunState`).
+  perVisit: editorRunKeys("description"),
   actions: {
     load: async (s, arg) => {
       stashCopy(arg); // absorb the descriptor's onAppear copy-pack param
       // WS-C — nav-param input is PRIMARY (the canonical create-or-edit shell
       // bakes no id/mode); the onAppear `id` param is the #570 suffixed-build
       // fallback. An id ⇒ edit; no id anywhere ⇒ the create shell.
-      const id = navInputId() || idFrom(arg, s.id);
-      if (!id) return shape({ ...s, ...DESCRIPTION_LOADED, mode: "create", id: "", descriptionMode: "plain", descriptionJson: "", status: "idle", error: "", demoNote: isOwnerPreviewDemo() ? copyParam("ownerCommon.demoSaveNote", DEMO_SAVE_NOTE) : "" });
+      // The held id is for the host's bare refresh only (no argument): an onAppear without an id is a create screen.
+      const id = navInputId() || idFrom(arg, arg === undefined ? s.id : "");
+      if (holdsLoadedForm(s, { id, mode: id ? "edit" : "create" }, "description")) return s;
+      // A form that held a record (the one just saved) opens blank.
+      if (!id) return shape({ ...(s.id ? {} : s), ...DESCRIPTION_LOADED, mode: "create", id: "", descriptionMode: "plain", descriptionJson: "", status: "idle", error: "", demoNote: isOwnerPreviewDemo() ? copyParam("ownerCommon.demoSaveNote", DEMO_SAVE_NOTE) : "" });
       if (isOwnerPreviewDemo()) {
           const d = DEMO_PRODUCTS.find((p) => p.id === id);
           if (d) {

package/templates/beauty-salon-owner/tier5-src/himi-owner-lite-search/index.ts

@@ -254,6 +254,8 @@ const IDLE: State = {
 
 /** Rebuilt on `load`, because the allowlist arrives with the descriptor. */
 let coordinator: ReturnType<typeof createGroupedSearch> | undefined;
+/** The allowlist the coordinator was built for. */
+let coordinatorSources = "";
 let published: State = IDLE;
 const memory = localPreferences("ownerSearch", normalizeMemory);
 
@@ -279,6 +281,15 @@ const svc = defineService<State, {
   actions: {
     load: async (_state, arg) => {
       stashCopy(arg);
+      // The screen coming back re-runs its search behind what it shows: a warm worker's refresh calls `load` bare (no allowlist),
+      // and its onAppear repeats the allowlist the coordinator was built for.
+      const sources = str(arg, "sources");
+      if (coordinator && (!sources || sources === coordinatorSources)) {
+        await coordinator.revalidate();
+        remember();
+        return published;
+      }
+      coordinatorSources = sources;
       const allowed = (() => {
         try {
           return JSON.parse(str(arg, "sources") || "[]");

package/templates/beauty-salon-owner/tier5-src/himi-owner-lite-search/logic.ts

@@ -446,7 +446,13 @@ export function createGroupedSearch(
     };
   };
 
-  const emit = () => { if (!disposed) publish(snapshot()); };
+  /** The revalidation running behind the shown rows (0: none): its snapshots are held back until it is done. */
+  let holding = 0, holds = 0;
+  /** The depth the running revalidation restores, so one that overlaps it starts from the owner's depth, not
+   *  from its reset; and the revalidation itself, which a "look further" tapped meanwhile waits for. */
+  let heldDepth: Record<string, number> = {};
+  let revalidation: Promise<void> | undefined;
+  const emit = () => { if (!disposed && !holding) publish(snapshot()); };
 
   /** Identity merge: a re-fetched row REPLACES its predecessor in place, so a
    *  refresh cannot reorder the list under the user's finger or duplicate a row
@@ -604,6 +610,7 @@ export function createGroupedSearch(
      *  the wrong result set. */
     async search(raw: unknown): Promise<void> {
       if (disposed) return;
+      holding = 0;
       const next = normalizeQuery(raw);
       query = next;
       for (const id of order) {
@@ -627,6 +634,7 @@ export function createGroupedSearch(
      */
     async restore(raw: unknown, depth: Record<string, number> = {}): Promise<void> {
       if (disposed) return;
+      holding = 0;
       const next = normalizeQuery(raw);
       query = next;
       for (const id of order) {
@@ -637,8 +645,34 @@ export function createGroupedSearch(
       if (!next) return;
       await fanOut(order, depth);
     },
+    /** A returning screen (a cached embed's warm worker): the same search re-run to the depth each group
+     *  shows, behind the rows it shows, and published once. Rows are a live claim about records, so they
+     *  are re-fetched, never repainted; a search typed meanwhile takes over at once. */
+    async revalidate(): Promise<void> {
+      if (disposed || !query) return;
+      const depth: Record<string, number> = holding ? { ...heldDepth } : {};
+      for (const id of order) { const pages = groups.get(id)?.pages ?? 0; if (pages > Math.max(1, depth[id] ?? 0)) depth[id] = pages; }
+      heldDepth = depth;
+      const held = ++holds;
+      holding = held;
+      for (const id of order) {
+        groups.set(id, idleGroup(byId.get(id)!));
+        sourceGeneration.set(id, (sourceGeneration.get(id) ?? 0) + 1);
+      }
+      const run = (async () => {
+        try {
+          await fanOut(order, depth);
+        } finally {
+          if (holding === held) { holding = 0; heldDepth = {}; emit(); }
+        }
+      })();
+      revalidation = run;
+      await run;
+    },
     /** One group's next page. */
     async loadMore(id: unknown): Promise<void> {
+      // Tapped while a revalidation re-fetches behind the shown rows: page on from what it lands.
+      while (holding && revalidation) await revalidation;
       const adapter = typeof id === "string" ? byId.get(id) : undefined;
       if (!adapter || disposed || !query) return;
       await runSource(adapter, true);

package/templates/beauty-salon-owner/tier5-src/himi-owner-lite-service-form/index.ts

@@ -117,7 +117,8 @@ const svc = defineService<ServiceFormState, {
       stashCopy(arg); // absorb the descriptor's onAppear copy-pack param
       // WS-C — nav-param input is PRIMARY (the canonical form bakes no id); the
       // onAppear `id` param is the #570 suffixed-build fallback.
-      const id = navInputId() || idFrom(arg, s.id);
+      // The held id is for the host's bare refresh only (no argument): an onAppear without an id is a create screen.
+      const id = navInputId() || idFrom(arg, arg === undefined ? s.id : "");
       if (!id) return shape({ ...s, status: "error", error: copyParam("ownerServiceForm.errorNoService", "No service to edit.") });
       if (isOwnerPreviewDemo()) {
           const d = DEMO_SERVICES.find((x) => x.id === id);