kaafil-react-uikit 0.11.6 → 0.11.8

package/dist/manager/index.js

@@ -1,9 +1,9 @@
 import { useManagerMe, useConflicts, useNotifications } from '../chunk-U6IE2UYL.js';
-import { FORM_FIELD_KIND_SINGLE_BOOLEAN, useChecklists, useManagers, useCloseout, useFiles, useFloat, useForms, useParties, useItinerary, useVendors, useExpenses, useCollections, useManagerToday, usePickups, useRooming, useTravellerProfile, useSeating } from '../chunk-63JQKLMH.js';
+import { FORM_FIELD_KIND_SINGLE_BOOLEAN, useChecklists, useManagers, useCloseout, useFiles, useFloat, useForms, useParties, useItinerary, useVendors, useExpenses, useCollections, useManagerToday, usePickups, useRooming, useTravellerProfile, useSeating } from '../chunk-XGPX7TYV.js';
 import { useOfflineEngine } from '../chunk-RCWSMWQD.js';
 import { OfflineFallbackScreen } from '../chunk-LOLQNRIK.js';
 export { BootErrorScreen, LockedTripScreen, NotFoundScreen, OfflineFallbackScreen, SessionExpiredScreen } from '../chunk-LOLQNRIK.js';
-import { createRegistry, useKaafilChrome, useSession, tokenStyle, useOutboxStatus, useKaafilBrand } from '../chunk-YJJ4DI42.js';
+import { createRegistry, useKaafilChrome, useSession, tokenStyle, useOutboxStatus, useOfflineEvent, useKaafilBrand } from '../chunk-YJJ4DI42.js';
 import { useState, useRef, useEffect, useMemo, useId } from 'react';
 import { jsx, jsxs, Fragment } from 'react/jsx-runtime';
 
@@ -6042,7 +6042,37 @@ function ReadyManagerMoneySection({
           issuedMinor: floatRow.issuedMinor,
           currency: floatRow.currency
         } : void 0,
-        spend: expensesReady !== void 0 ? { amountMinor: sumSpendMinor(active), currency } : void 0,
+        spend: (
+          /*
+           * THE SERVER'S OWN ROLLUP FIRST, the local sum only as a fallback.
+           *
+           * This summed `active` — the snapshot-backed rows — and nothing
+           * else. `useExpenses` deliberately enqueues its writes WITHOUT an
+           * optimistic row (its own DECISION comment: a fabricated
+           * `ExpenseResponse` would have to guess `floatMovementId`,
+           * `receiptRequired` and the negative-float guard, none of which a
+           * client can compute). That decision is right, and its consequence
+           * is that a freshly logged expense is absent from `expenses[]` until
+           * a pull brings it down — so this tile read ₹0 with the money
+           * already spent, on the one screen a manager checks it on.
+           *
+           * `totals.spendTotalMinor` is the server's own spend rollup and the
+           * hook already re-reads it on every `sync.drained`, so it reflects
+           * the write as soon as the queue reaches the server. Using it here
+           * is also simply more correct than summing whichever rows this
+           * device happens to hold.
+           *
+           * The local sum stays as the fallback rather than the primary. Its
+           * own doc says `spendTotalMinor` is `undefined` until a read lands
+           * and must never be drawn as zero — offline, where that read cannot
+           * happen, a sum of the rows actually held is the honest answer, and
+           * it is what offline-first means here.
+           */
+          expensesReady !== void 0 ? {
+            amountMinor: expensesReady.totals.spendTotalMinor ?? sumSpendMinor(active),
+            currency
+          } : void 0
+        ),
         collect: collectionsReady !== void 0 ? {
           amountMinor: balancesDueRows.reduce((sum, row) => sum + row.outstandingMinor, 0),
           travellerCount: balancesDueRows.length,
@@ -8003,7 +8033,6 @@ function ReadyManagerPickupsSection({
   style
 }) {
   const pickups = usePickups(tripRef);
-  const outbox = useOutboxStatus();
   const StopListUI = components?.StopList ?? StopList;
   const BoardingViewUI = components?.BoardingView ?? BoardingView;
   const [kind, setKind] = useState("PICKUP");
@@ -8027,27 +8056,13 @@ function ReadyManagerPickupsSection({
       cancelled = true;
     };
   }, [ready, selectedStopId, kind]);
-  const drainedFrom = useRef(outbox.pending);
-  useEffect(() => {
-    const hadWork = drainedFrom.current > 0;
-    drainedFrom.current = outbox.pending;
-    if (!hadWork || outbox.pending > 0) {
-      return;
-    }
+  useOfflineEvent("sync.drained", () => {
     if (pickups.status !== "ready" || selectedStopId === void 0) {
       return;
     }
-    let cancelled = false;
-    void pickups.manifestByPickup({ kind }).then((result) => {
-      if (!cancelled) {
-        setManifest(result);
-      }
-    });
+    void pickups.manifestByPickup({ kind }).then(setManifest);
     void pickups.refreshLive();
-    return () => {
-      cancelled = true;
-    };
-  }, [outbox.pending, selectedStopId, kind]);
+  });
   if (pickups.status === "loading") {
     return /* @__PURE__ */ jsxs("div", { className: "kf-manager-pickups__skeleton", style: tokenStyle(style), children: [
       /* @__PURE__ */ jsx("div", { className: "kf-manager-pickups__skeleton-line" }),