kaafil-react-uikit 0.2.0 → 0.3.1

package/dist/{chunk-RSC2ZJUP.js → chunk-JHMN6Q5R.js}

@@ -1,5 +1,5 @@
 import { OfflineEngineContext, KaafilClientContext, useCapabilityRows, UNKNOWN_CAPABILITY_TRIPLE, deriveCapabilityTriple, darkReasonFor, useKaafilClient, useOfflineMutation, usePersona, isDevelopment, useOfflineEvent } from './chunk-A4Z2YBEF.js';
-import { UnsatisfiableSchemeError, isKaafilError, SyncOpMethod } from 'kaafil-js/client';
+import { UnsatisfiableSchemeError, isKaafilError, SyncOpMethod, normalizeDateTimeInput } from 'kaafil-js/client';
 export { FALLBACK_GLYPH, SHADES_PER_FAMILY, TONE_FAMILIES, parseRoomingTone, roomingHeuristicSeverity } from 'kaafil-js/client';
 import { useContext, useState, useRef, useCallback, useEffect } from 'react';
 
@@ -176,6 +176,51 @@ function createDomainHook(moduleKey, build) {
 }
 
 // src/core/hooks/useChecklists.ts
+function optimisticChecklistItem(input, sections, fetchedAt) {
+  return {
+    // Replaced by the id the engine mints — this hook never invents identity.
+    id: "pending",
+    // Cannot beat any server row in the snapshot's version-guarded merge.
+    version: 0,
+    // The write addresses a section by KEY; the board groups rows by ID. An
+    // unknown section leaves the row ungrouped rather than filing it under a
+    // heading that does not exist.
+    sectionId: sections.find((section) => section.key === input.sectionKey)?.id ?? "",
+    // The server derives the key from the title when one is not sent. Blank
+    // rather than a guessed slug: a key that disagrees with the server's is
+    // worse than an obviously-absent one, and nothing renders it.
+    key: input.key ?? "",
+    title: input.title,
+    subLine: null,
+    status: "OPEN",
+    // `=== true`, never `?? false` (hard rule #5): an omitted `mandatory` is
+    // the SERVER's default to apply, not this hook's to guess.
+    isMandatory: input.mandatory === true,
+    // `'NONE'`, never null. The add request carries no `gate` at all — the
+    // engine derives it from `phase` — and `'NONE'` is the one value that
+    // cannot make a pending item look like it is blocking the trip.
+    gate: "NONE",
+    dayOffset: null,
+    // Appended. The server's own ordering arrives with the canonical row.
+    sortOrder: Number.MAX_SAFE_INTEGER,
+    completedByManagerId: null,
+    performedByKind: null,
+    performedById: null,
+    completedAt: null,
+    // The last reading the SERVER gave this hook, never a device clock (hard
+    // rule #6). Blank when nothing has been fetched yet, which is the only
+    // honest answer and which the board already prefers `fetchedAt` over.
+    updatedAt: fetchedAt ?? ""
+  };
+}
+function appendPendingRows(fromAggregate, fromSnapshot) {
+  if (fromSnapshot.length === 0) {
+    return fromAggregate;
+  }
+  const known = new Set(fromAggregate.map((row) => row.id));
+  const pending = fromSnapshot.filter((row) => !known.has(row.id));
+  return pending.length === 0 ? fromAggregate : [...fromAggregate, ...pending];
+}
 var useChecklistsInternal = createDomainHook("checklists", (tripRef) => {
   const client = useKaafilClient();
   const engine = useContext(OfflineEngineContext);
@@ -249,7 +294,33 @@ var useChecklistsInternal = createDomainHook("checklists", (tripRef) => {
       method: SyncOpMethod.Post,
       path: `/api/v1/trips/${tripRef}/checklist/items`,
       operationId: "addChecklistItem",
-      body: input
+      body: input,
+      // ── THE OPTIMISTIC CREATE ──────────────────────────────────────
+      // Without this the item a manager adds out of signal does not
+      // appear until it syncs. It was queued, durable and landed on
+      // reconnect the whole time — and from where the manager is
+      // standing, work that does not show up is work that was lost, so
+      // they add it again.
+      //
+      // `creates` makes the ENGINE mint the row's temporary id (this
+      // hook never invents identifiers) and swap it for the server's
+      // canonical row the moment the write lands, in one
+      // `snapshot.updated`, so the item is never rendered twice.
+      creates: true,
+      entity: { listName: "checklist" },
+      // Shaped as a real `ChecklistItemDeltaRow` so the board renders it
+      // through the same path as every other row rather than a special
+      // "pending" branch. `id` is a placeholder the engine replaces with
+      // the id it mints; `version: 0` cannot beat any server row in the
+      // snapshot's version-guarded merge.
+      //
+      // The write addresses a section by its `sectionKey`, but the board
+      // groups rows by `sectionId`, so the id is resolved from the sections
+      // this hook already holds. Adding to a section that does not exist
+      // yet resolves to `''` and the row sits ungrouped until the write
+      // lands — correct rather than clever: inventing a section id would
+      // put the item under a heading that does not exist.
+      optimistic: optimisticChecklistItem(input, aggregate.sections, aggregate.fetchedAt)
     }),
     // ── THE DESK LANE ─────────────────────────────────────────────────
     // All five checklist writes take the admin triple, and a desk console
@@ -391,7 +462,7 @@ var useChecklistsInternal = createDomainHook("checklists", (tripRef) => {
     (input) => client.checklists.agencyTemplates.publish(input),
     [client]
   );
-  const items = syncedAt === void 0 ? aggregate.items : snapshotItems;
+  const items = syncedAt === void 0 ? appendPendingRows(aggregate.items, snapshotItems) : snapshotItems;
   return {
     items,
     syncedAt,
@@ -484,6 +555,34 @@ var useCloseoutInternal = createDomainHook("closing-day", (tripRef) => {
 function useCloseout(tripRef) {
   return useCloseoutInternal(tripRef);
 }
+function optimisticCollection(input, syncedAt) {
+  const collectedAt = input.collectedAt === void 0 ? "" : normalizeDateTimeInput(input.collectedAt, "collectedAt");
+  return {
+    // Replaced by the id the engine mints.
+    id: "pending",
+    version: 0,
+    travellerId: input.travellerId,
+    amountMinor: input.amountMinor,
+    // The request's own default, restated: this tenant's money unit.
+    currency: input.currency ?? "INR",
+    mode: input.mode,
+    reference: input.reference ?? null,
+    note: input.note ?? null,
+    collectedByManagerId: null,
+    performedByKind: null,
+    performedById: null,
+    collectedAt,
+    reportedAt: null,
+    voidedAt: null,
+    voidReason: null,
+    // The server's arithmetic, deliberately not guessed — see above.
+    outstandingMinor: null,
+    // The last reading the SERVER gave this list, never a device clock
+    // (hard rule #6).
+    createdAt: syncedAt ?? "",
+    updatedAt: syncedAt ?? ""
+  };
+}
 var useCollectionsInternal = createDomainHook("collections", (tripRef) => {
   const client = useKaafilClient();
   const {
@@ -533,7 +632,12 @@ var useCollectionsInternal = createDomainHook("collections", (tripRef) => {
       // hardcoded literal, verified against
       // `kaafil-js/src/generated/security.ts` this session.
       operationId: "recordCollection",
-      body: input
+      body: input,
+      // A payment recorded out of signal appears at once — see
+      // `optimisticCollection` for what it deliberately does not claim.
+      creates: true,
+      entity: { listName: "collections" },
+      optimistic: optimisticCollection(input, syncedAt)
     }),
     // The desk lane. `recordCollection` takes the admin triple, and a desk
     // console mounts no `OfflineShell` — without this the write has no
@@ -1521,6 +1625,34 @@ function useParties(tripRef) {
     remove
   };
 }
+function optimisticPickupStop(input, syncedAt) {
+  return {
+    // Replaced by the id the engine mints.
+    id: "pending",
+    version: 0,
+    // The request's own default, restated rather than guessed: the SDK's
+    // `kind` is optional and the engine treats an absent one as a pickup.
+    kind: input.kind ?? "PICKUP",
+    name: input.name,
+    locationLabel: input.locationLabel ?? null,
+    lat: input.lat ?? null,
+    lng: input.lng ?? null,
+    scheduledTime: normalizeDateTimeInput(input.scheduledTime, "scheduledTime"),
+    // Appended; the server's ordering arrives with the canonical row.
+    sortOrder: input.sortOrder ?? Number.MAX_SAFE_INTEGER,
+    status: "OPEN",
+    closedAt: null,
+    closedByManagerId: null,
+    performedByKind: null,
+    performedById: null,
+    rollup: "not_started",
+    boardedCount: 0,
+    expectedCount: 0,
+    // The last reading the SERVER gave this list, never a device clock
+    // (hard rule #6).
+    updatedAt: syncedAt ?? ""
+  };
+}
 var usePickupsInternal = createDomainHook("pickup-points", (tripRef) => {
   const client = useKaafilClient();
   const {
@@ -1544,7 +1676,14 @@ var usePickupsInternal = createDomainHook("pickup-points", (tripRef) => {
       // literal below is checked against `kaafil-js/src/generated/
       // security.ts`'s own operation-id table.
       operationId: "createPickupStop",
-      body: input
+      body: input,
+      // A stop added out of signal appears at once, keyed on an id the
+      // ENGINE mints and swapped for the server's row when the write lands
+      // — see `optimisticPickupStop` for why every field here is one the
+      // client can honestly know.
+      creates: true,
+      entity: { listName: "pickups" },
+      optimistic: optimisticPickupStop(input, syncedAt)
     }),
     // The desk lane. `board`, `close` and `reopen` below deliberately get
     // NONE — they are `managerAuth`-only on the wire, so a direct call from
@@ -1690,6 +1829,24 @@ var usePickupsInternal = createDomainHook("pickup-points", (tripRef) => {
 function usePickups(tripRef) {
   return usePickupsInternal(tripRef);
 }
+function optimisticStayWindow(input, syncedAt) {
+  return {
+    // Replaced by the id the engine mints.
+    id: "pending",
+    version: 0,
+    label: input.label,
+    startDate: normalizeDateTimeInput(input.startDate, "startDate"),
+    endDate: normalizeDateTimeInput(input.endDate, "endDate"),
+    // Appended; the server's ordering arrives with the canonical row.
+    sortOrder: input.sortOrder ?? Number.MAX_SAFE_INTEGER,
+    // A window created by hand has no CRM segment behind it. This is a fact
+    // about the write, not a placeholder.
+    sourceSegmentRef: null,
+    sourceUpdatedAt: null,
+    // The last reading the SERVER gave this list, never a device clock.
+    updatedAt: syncedAt ?? ""
+  };
+}
 var useRoomingInternal = createDomainHook("rooming", (tripRef) => {
   const client = useKaafilClient();
   const {
@@ -1777,7 +1934,14 @@ var useRoomingInternal = createDomainHook("rooming", (tripRef) => {
       method: SyncOpMethod.Post,
       path: `/api/v1/trips/${tripRef}/rooming/stay-windows`,
       operationId: "createRoomingStayWindow",
-      body: input
+      body: input,
+      // A window added out of signal appears at once. Every field of a
+      // stay window is either given by the caller or genuinely absent —
+      // which is exactly what `createRoom` below cannot say, and why only
+      // this one of the two gets an optimistic row.
+      creates: true,
+      entity: { listName: "stayWindows" },
+      optimistic: optimisticStayWindow(input, windowsSyncedAt)
     }),
     direct: (liveClient, input) => liveClient.rooming.stayWindows.create({ tripRef, ...input })
   });
@@ -2166,5 +2330,5 @@ function useVendors(tripRef, agencyRef) {
 }
 
 export { ERROR_HANDLING_PATH, FORM_FIELD_KIND_SINGLE_BOOLEAN, classifyError, createDomainHook, useChecklists, useCloseout, useCollections, useExpenses, useFiles, useFloat, useForms, useItinerary, useManagerToday, useManagers, useParties, usePickups, useRooming, useSeating, useSnapshotList, useTravellerProfile, useVendors };
-//# sourceMappingURL=chunk-RSC2ZJUP.js.map
-//# sourceMappingURL=chunk-RSC2ZJUP.js.map
+//# sourceMappingURL=chunk-JHMN6Q5R.js.map
+//# sourceMappingURL=chunk-JHMN6Q5R.js.map