@escapenavigator/hooks 2.2.6 → 2.2.8

package/dist/players-and-variation-picker/index.js

@@ -57,6 +57,7 @@ const reduced_price_1 = require("@escapenavigator/utils/dist/reduced-price");
 const react_1 = __importStar(require("react"));
 const react_i18next_1 = require("react-i18next");
 const use_players_and_variation_picker_1 = require("../use-players-and-variation-picker");
+const sync_composition_1 = require("./sync-composition");
 const ticket_row_1 = require("./ticket-row");
 /**
  * Единый пикер «вариация + состав», который используют админские модалки
@@ -163,20 +164,32 @@ const PlayersAndVariationPicker = ({ slot, currentTariffId, currentDuration, loa
      * полях льготные места — одно число).
      */
     const emittedLegacy = (0, react_1.useRef)(null);
+    /** Состав последнего клика, пока хост не вернул его в пропсах. */
+    const pendingComposition = (0, react_1.useRef)(null);
     /**
      * Синхронизация сверху вниз. Состав по категориям приоритетнее старых
-     * полей: их читаем, только пока хост на него не перешёл.
+     * полей: их читаем, только пока хост на него не перешёл. Свежий клик
+     * не затираем предыдущим пропом — иначе цена остаётся на прошлом числе.
      */
     (0, react_1.useEffect)(() => {
-        if (compositionProp && Object.keys(compositionProp).length) {
-            setComposition(compositionProp);
+        const decision = (0, sync_composition_1.decideCompositionSync)({
+            compositionProp,
+            pending: pendingComposition.current,
+            emittedLegacy: emittedLegacy.current,
+            players,
+            child,
+            reducedPlayers,
+        });
+        if (decision.action === 'keep') {
+            if (decision.clearPending)
+                pendingComposition.current = null;
             return;
         }
-        const echo = emittedLegacy.current;
-        if (echo &&
-            echo.players === players &&
-            echo.children === child &&
-            echo.reducedPlayers === reducedPlayers) {
+        if (decision.action === 'adopt') {
+            if (decision.clearPending)
+                pendingComposition.current = null;
+            if (compositionProp)
+                setComposition(compositionProp);
             return;
         }
         setComposition((0, composition_price_1.compositionFromLegacy)({ players, children: child, reducedPlayers }, categories));
@@ -200,6 +213,7 @@ const PlayersAndVariationPicker = ({ slot, currentTariffId, currentDuration, loa
     const baseCategory = categories.find((category) => category.mode === tariff_category_mode_enum_1.TariffCategoryModeEnum.BASE);
     const baseCategoryKey = baseCategory === null || baseCategory === void 0 ? void 0 : baseCategory.key;
     const applyComposition = (next) => {
+        pendingComposition.current = next;
         setComposition(next);
         const legacy = (0, composition_price_1.legacyCountsFromComposition)(next, categories);
         emittedLegacy.current = legacy;

package/dist/players-and-variation-picker/sync-composition.d.ts

@@ -0,0 +1,35 @@
+import { OrderComposition } from '@escapenavigator/types/dist/tariff/tariff-category';
+type LegacyCounts = {
+    players: number;
+    children: number;
+    reducedPlayers: number;
+};
+/**
+ * Что делать с составом, который хост вернул пропсами, пока пикер уже
+ * показал свой клик.
+ *
+ * Цена рисуется из локального состава. Если эффект синхронизации
+ * подставляет предыдущий `composition` (хост уже записал `players`, а
+ * состав ещё старый — или записал оба, но эффект успел раньше), сумма
+ * остаётся на прошлом числе до размонтирования шага.
+ */
+export type CompositionSyncDecision = {
+    action: 'adopt';
+    clearPending: boolean;
+} | {
+    action: 'keep';
+    clearPending: boolean;
+} | {
+    action: 'rebuild';
+};
+export declare const sameCompositionCounts: (left?: OrderComposition | null, right?: OrderComposition | null) => boolean;
+export declare const decideCompositionSync: ({ compositionProp, pending, emittedLegacy, players, child, reducedPlayers, }: {
+    compositionProp?: OrderComposition | null;
+    /** Состав, который пикер только что поставил кликом и ещё не увидел в пропсах. */
+    pending: OrderComposition | null;
+    emittedLegacy: LegacyCounts | null;
+    players: number;
+    child: number;
+    reducedPlayers: number;
+}) => CompositionSyncDecision;
+export {};

package/dist/players-and-variation-picker/sync-composition.js

@@ -0,0 +1,41 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.decideCompositionSync = exports.sameCompositionCounts = void 0;
+const hasComposition = (composition) => !!composition && Object.keys(composition).length > 0;
+const sameCompositionCounts = (left, right) => {
+    const keys = new Set([...Object.keys(left || {}), ...Object.keys(right || {})]);
+    for (const key of keys) {
+        if (((left === null || left === void 0 ? void 0 : left[key]) || 0) !== ((right === null || right === void 0 ? void 0 : right[key]) || 0))
+            return false;
+    }
+    return true;
+};
+exports.sameCompositionCounts = sameCompositionCounts;
+const legacyCaughtUp = (emitted, players, child, reducedPlayers) => !!emitted &&
+    emitted.players === players &&
+    emitted.children === child &&
+    emitted.reducedPlayers === reducedPlayers;
+const decideCompositionSync = ({ compositionProp, pending, emittedLegacy, players, child, reducedPlayers, }) => {
+    const caughtUp = legacyCaughtUp(emittedLegacy, players, child, reducedPlayers);
+    if (pending) {
+        // Хост вернул ровно то, что ушло наверх — можно отпустить локальную копию.
+        if (hasComposition(compositionProp) && (0, exports.sameCompositionCounts)(compositionProp, pending)) {
+            return { action: 'adopt', clearPending: true };
+        }
+        // Легаси-хост состава не хранит, но число игроков уже наше.
+        // Локальный состав богаче денорма, пересобирать его нельзя.
+        if (!hasComposition(compositionProp) && caughtUp) {
+            return { action: 'keep', clearPending: true };
+        }
+        // Проп ещё предыдущий. Не откатывать цену и не отпускать клик,
+        // иначе следующий прогон подставит старый состав.
+        return { action: 'keep', clearPending: false };
+    }
+    if (hasComposition(compositionProp))
+        return { action: 'adopt', clearPending: false };
+    // Эхо собственного денорма: несколько льготных категорий схлопнулись бы в одну.
+    if (caughtUp)
+        return { action: 'keep', clearPending: false };
+    return { action: 'rebuild' };
+};
+exports.decideCompositionSync = decideCompositionSync;

package/dist/players-and-variation-picker/sync-composition.spec.d.ts

@@ -0,0 +1 @@
+export {};

package/dist/players-and-variation-picker/sync-composition.spec.js

@@ -0,0 +1,86 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const sync_composition_1 = require("./sync-composition");
+const adult = (count) => ({ adult: count });
+describe('decideCompositionSync', () => {
+    it('принимает состав хоста, пока своего клика нет', () => {
+        expect((0, sync_composition_1.decideCompositionSync)({
+            compositionProp: adult(5),
+            pending: null,
+            emittedLegacy: null,
+            players: 5,
+            child: 0,
+            reducedPlayers: 0,
+        })).toEqual({ action: 'adopt', clearPending: false });
+    });
+    it('не откатывает клик, если в пропсах ещё прошлый состав', () => {
+        expect((0, sync_composition_1.decideCompositionSync)({
+            compositionProp: adult(5),
+            pending: adult(4),
+            emittedLegacy: { players: 4, children: 0, reducedPlayers: 0 },
+            players: 5,
+            child: 0,
+            reducedPlayers: 0,
+        })).toEqual({ action: 'keep', clearPending: false });
+    });
+    it('не откатывает клик, даже если players уже новый, а composition старый', () => {
+        expect((0, sync_composition_1.decideCompositionSync)({
+            compositionProp: adult(5),
+            pending: adult(4),
+            emittedLegacy: { players: 4, children: 0, reducedPlayers: 0 },
+            players: 4,
+            child: 0,
+            reducedPlayers: 0,
+        })).toEqual({ action: 'keep', clearPending: false });
+    });
+    it('отпускает локальный состав, когда хост вернул тот же набор', () => {
+        expect((0, sync_composition_1.decideCompositionSync)({
+            compositionProp: { adult: 4, student: 0 },
+            pending: adult(4),
+            emittedLegacy: { players: 4, children: 0, reducedPlayers: 0 },
+            players: 4,
+            child: 0,
+            reducedPlayers: 0,
+        })).toEqual({ action: 'adopt', clearPending: true });
+    });
+    it('не пересобирает состав из денорма, пока хост не догнал клик', () => {
+        expect((0, sync_composition_1.decideCompositionSync)({
+            compositionProp: undefined,
+            pending: { adult: 2, student: 2 },
+            emittedLegacy: { players: 4, children: 0, reducedPlayers: 2 },
+            players: 2,
+            child: 0,
+            reducedPlayers: 0,
+        })).toEqual({ action: 'keep', clearPending: false });
+    });
+    it('отпускает клик у легаси-хоста, когда число игроков уже совпало', () => {
+        expect((0, sync_composition_1.decideCompositionSync)({
+            compositionProp: undefined,
+            pending: { adult: 2, student: 2 },
+            emittedLegacy: { players: 4, children: 0, reducedPlayers: 2 },
+            players: 4,
+            child: 0,
+            reducedPlayers: 2,
+        })).toEqual({ action: 'keep', clearPending: true });
+    });
+    it('не схлопывает льготные категории, когда денорм — эхо нашего ответа', () => {
+        expect((0, sync_composition_1.decideCompositionSync)({
+            compositionProp: undefined,
+            pending: null,
+            emittedLegacy: { players: 4, children: 0, reducedPlayers: 2 },
+            players: 4,
+            child: 0,
+            reducedPlayers: 2,
+        })).toEqual({ action: 'keep', clearPending: false });
+    });
+    it('пересобирает состав, если число игроков пришло снаружи', () => {
+        expect((0, sync_composition_1.decideCompositionSync)({
+            compositionProp: undefined,
+            pending: null,
+            emittedLegacy: { players: 4, children: 0, reducedPlayers: 0 },
+            players: 6,
+            child: 1,
+            reducedPlayers: 0,
+        })).toEqual({ action: 'rebuild' });
+    });
+});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@escapenavigator/hooks",
-  "version": "2.2.6",
+  "version": "2.2.8",
   "publishConfig": {
     "access": "public"
   },
@@ -14,9 +14,9 @@
     "test": "jest"
   },
   "dependencies": {
-    "@escapenavigator/services": "^2.2.6",
-    "@escapenavigator/types": "^2.2.5",
-    "@escapenavigator/utils": "^2.2.5",
+    "@escapenavigator/services": "^2.2.8",
+    "@escapenavigator/types": "^2.2.7",
+    "@escapenavigator/utils": "^2.2.7",
     "libphonenumber-js": "^1.12.24",
     "react": "19.2.6",
     "react-phone-input-2": "^2.15.1"
@@ -32,5 +32,5 @@
     "ts-jest": "^29.1.1",
     "typescript": "^5.6"
   },
-  "gitHead": "2a9e5dcd5dd44298d96d7dfa918d4bd86b611979"
+  "gitHead": "821f81560e48bb5554575ada7f5f8b0d7597fdcb"
 }