kaafil-react-uikit 0.1.0 → 0.3.0

package/dist/{chunk-JMWXQTE3.js → chunk-VLP323AD.js}

@@ -1,4 +1,4 @@
-import { OfflineEngineContext } from './chunk-2SRBJ4H7.js';
+import { OfflineEngineContext } from './chunk-A4Z2YBEF.js';
 import { useContext, useState, useRef, useEffect } from 'react';
 
 var TICK_INTERVAL_MS = 1e3;
@@ -12,6 +12,9 @@ function useServerTime() {
       return;
     }
     const unsubscribe = engine.events.on("snapshot.updated", (event) => {
+      if (event.serverTime === void 0) {
+        return;
+      }
       observedAtPerfRef.current = performance.now();
       setLastServerTime(event.serverTime);
     });
@@ -34,5 +37,5 @@ function useServerTime() {
 }
 
 export { useServerTime };
-//# sourceMappingURL=chunk-JMWXQTE3.js.map
-//# sourceMappingURL=chunk-JMWXQTE3.js.map
+//# sourceMappingURL=chunk-VLP323AD.js.map
+//# sourceMappingURL=chunk-VLP323AD.js.map

package/dist/chunk-VLP323AD.js.map

@@ -0,0 +1 @@
+{"version":3,"sources":["../src/core/hooks/useServerTime.ts"],"names":[],"mappings":";;;AA8BA,IAAM,gBAAA,GAAmB,GAAA;AAElB,SAAS,aAAA,GAA+B;AAC7C,EAAA,MAAM,MAAA,GAAS,WAAW,oBAAoB,CAAA;AAC9C,EAAA,MAAM,CAAC,cAAA,EAAgB,iBAAiB,CAAA,GAAI,SAA6B,MAAS,CAAA;AAClF,EAAA,MAAM,iBAAA,GAAoB,OAA2B,MAAS,CAAA;AAC9D,EAAA,MAAM,GAAG,SAAS,CAAA,GAAI,SAAS,CAAC,CAAA;AAEhC,EAAA,SAAA,CAAU,MAAM;AACd,IAAA,IAAI,WAAW,IAAA,EAAM;AACnB,MAAA;AAAA,IACF;AAKA,IAAA,MAAM,cAAc,MAAA,CAAO,MAAA,CAAO,EAAA,CAAG,kBAAA,EAAoB,CAAC,KAAA,KAAU;AASlE,MAAA,IAAI,KAAA,CAAM,eAAe,MAAA,EAAW;AAClC,QAAA;AAAA,MACF;AACA,MAAA,iBAAA,CAAkB,OAAA,GAAU,YAAY,GAAA,EAAI;AAC5C,MAAA,iBAAA,CAAkB,MAAM,UAAU,CAAA;AAAA,IACpC,CAAC,CAAA;AACD,IAAA,OAAO,WAAA;AAAA,EACT,CAAA,EAAG,CAAC,MAAM,CAAC,CAAA;AAEX,EAAA,SAAA,CAAU,MAAM;AACd,IAAA,MAAM,EAAA,GAAK,YAAY,MAAM,SAAA,CAAU,CAAC,IAAA,KAAS,IAAA,GAAO,CAAC,CAAA,EAAG,gBAAgB,CAAA;AAC5E,IAAA,OAAO,MAAM,cAAc,EAAE,CAAA;AAAA,EAC/B,CAAA,EAAG,EAAE,CAAA;AAEL,EAAA,SAAS,GAAA,GAAc;AACrB,IAAA,IAAI,cAAA,KAAmB,MAAA,IAAa,iBAAA,CAAkB,OAAA,KAAY,MAAA,EAAW;AAY3E,MAAA,MAAM,IAAI,KAAA;AAAA,QACR;AAAA,OAEF;AAAA,IACF;AACA,IAAA,MAAM,SAAA,GAAY,WAAA,CAAY,GAAA,EAAI,GAAI,iBAAA,CAAkB,OAAA;AACxD,IAAA,OAAO,IAAI,KAAK,IAAA,CAAK,KAAA,CAAM,cAAc,CAAA,GAAI,SAAS,EAAE,WAAA,EAAY;AAAA,EACtE;AAEA,EAAA,OAAO,EAAE,GAAA,EAAK,aAAA,EAAe,cAAA,EAAe;AAC9C","file":"chunk-VLP323AD.js","sourcesContent":["'use client';\n\n// `08-core-hooks.md §1.6`. The single place that turns the stream of\n// `meta.serverTime` stamps into a usable clock, so no composite reads\n// `meta.serverTime` off an individual response by hand.\n//\n// Hard rule #6 (\"Zero `Date.now()`/`new Date()` local getters in\n// `src/core`\") bans exactly the two calls that fabricate \"now\" from the\n// local wall clock: `Date.now()` and no-argument `new Date()`. This file\n// never calls either. Advancing a SERVER-supplied timestamp by an elapsed\n// delta necessarily needs SOME measure of \"how much local time passed\" —\n// this file measures that with `performance.now()`, a monotonic clock with\n// no relationship to wall-clock \"now\" (it cannot be read as a date, only as\n// a duration), and applies the delta with `new Date(<epoch-ms>)` /\n// `Date.parse(<iso-string>)` — both single-argument forms that operate on a\n// value already anchored to `meta.serverTime`, never on the local clock.\n\nimport { useContext, useEffect, useRef, useState } from 'react';\nimport { OfflineEngineContext } from '../internal/contexts';\n\nexport interface ServerTimeApi {\n  now: () => string;\n  sinceLastSync: string | undefined;\n}\n\n/** Re-renders any hook consumer often enough for a mounted countdown to look\n * live, without pretending the underlying value moved forward on its own\n * merits (`08-core-hooks.md §1.6`'s sandbox time-travel caveat: this timer\n * forces a re-render, it never advances the stamp past what the last real\n * response justified). */\nconst TICK_INTERVAL_MS = 1000;\n\nexport function useServerTime(): ServerTimeApi {\n  const engine = useContext(OfflineEngineContext);\n  const [lastServerTime, setLastServerTime] = useState<string | undefined>(undefined);\n  const observedAtPerfRef = useRef<number | undefined>(undefined);\n  const [, forceTick] = useState(0);\n\n  useEffect(() => {\n    if (engine === null) {\n      return;\n    }\n    // `snapshot.updated` is the one event in `OfflineEventMap` that carries\n    // `serverTime` (`kaafil-js/src/offline/events.ts`, verified this\n    // session) — the engine's other five events do not, so this is the only\n    // subscription this hook can build without inventing a stamp.\n    const unsubscribe = engine.events.on('snapshot.updated', (event) => {\n      // `serverTime` is ABSENT on a change that never involved the server —\n      // an optimistic create painted while offline, or an op the host\n      // discarded (`kaafil-js/src/offline/events.ts`). Those are real snapshot\n      // changes and a list must repaint for them, but they carry no reading,\n      // and taking `undefined` here would erase the last real one this\n      // session had: `now()` would start throwing again mid-session, on a\n      // device that HAS heard from the server, because the manager ticked\n      // something off out of signal.\n      if (event.serverTime === undefined) {\n        return;\n      }\n      observedAtPerfRef.current = performance.now();\n      setLastServerTime(event.serverTime);\n    });\n    return unsubscribe;\n  }, [engine]);\n\n  useEffect(() => {\n    const id = setInterval(() => forceTick((tick) => tick + 1), TICK_INTERVAL_MS);\n    return () => clearInterval(id);\n  }, []);\n\n  function now(): string {\n    if (lastServerTime === undefined || observedAtPerfRef.current === undefined) {\n      // No response has ever landed in this session yet. Fabricating a\n      // value from the local wall clock is exactly what hard rule #6\n      // forbids — this is the one case this hook cannot silently satisfy\n      // its own non-optional return type, so it says so loudly instead.\n      // CALLERS: this is a real, expected window — React runs an effect\n      // (which is how this hook's own subscription above starts) strictly\n      // AFTER the render that needs `now()`'s result has already committed,\n      // so a component's very FIRST render can hit this throw even when\n      // wired up correctly. Guard the call site, matching\n      // `useStaleness.ts`'s own precedent, rather than reaching for a\n      // device-clock fallback here (hard rule #6 forbids exactly that).\n      throw new Error(\n        'useServerTime().now() was called before any response carrying meta.serverTime ' +\n          'had been observed — there is no server timestamp yet to advance from.',\n      );\n    }\n    const elapsedMs = performance.now() - observedAtPerfRef.current;\n    return new Date(Date.parse(lastServerTime) + elapsedMs).toISOString();\n  }\n\n  return { now, sinceLastSync: lastServerTime };\n}\n"]}

package/dist/{chunk-7WXE5SUS.js → chunk-WTTGPWMR.js}

@@ -1,4 +1,4 @@
-import { OfflineEngineContext, KaafilClientContext, useCapabilityRows, UNKNOWN_CAPABILITY_TRIPLE, deriveCapabilityTriple, darkReasonFor, useKaafilClient, useOfflineMutation, usePersona, isDevelopment, useOfflineEvent } from './chunk-2SRBJ4H7.js';
+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';
 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,14 @@ function createDomainHook(moduleKey, build) {
 }
 
 // src/core/hooks/useChecklists.ts
+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);
@@ -183,22 +191,31 @@ var useChecklistsInternal = createDomainHook("checklists", (tripRef) => {
     tripRef,
     "checklist"
   );
-  const [aggregate, setAggregate] = useState({
-    title: void 0,
-    subtitle: void 0,
-    progress: void 0,
-    hasOpenMandatoryByPhase: void 0,
-    sections: [],
-    items: [],
-    availableTemplates: [],
-    loaded: false,
-    lastFetchFailed: false,
-    fetchedAt: void 0
+  const aggregateCacheKey = `checklists.read:${tripRef}`;
+  const [aggregate, setAggregate] = useState(() => {
+    const cached = engine?.reads.get(
+      aggregateCacheKey
+    );
+    if (cached !== void 0) {
+      return { ...cached.value, loaded: true, lastFetchFailed: false };
+    }
+    return {
+      title: void 0,
+      subtitle: void 0,
+      progress: void 0,
+      hasOpenMandatoryByPhase: void 0,
+      sections: [],
+      items: [],
+      availableTemplates: [],
+      loaded: false,
+      lastFetchFailed: false,
+      fetchedAt: void 0
+    };
   });
   const refreshAggregate = useCallback(async () => {
     try {
       const response = await client.checklists.read({ tripRef });
-      setAggregate({
+      const value = {
         title: response.title,
         subtitle: response.subtitle,
         progress: response.progress,
@@ -206,14 +223,21 @@ var useChecklistsInternal = createDomainHook("checklists", (tripRef) => {
         sections: response.sections,
         items: response.items,
         availableTemplates: response.availableTemplates,
-        loaded: true,
-        lastFetchFailed: false,
         fetchedAt: response.meta.serverTime
+      };
+      void engine?.reads.set(aggregateCacheKey, value, response.meta.serverTime).catch(() => {
       });
+      setAggregate({ ...value, loaded: true, lastFetchFailed: false });
     } catch {
-      setAggregate((previous) => ({ ...previous, lastFetchFailed: true }));
+      setAggregate((previous) => {
+        if (previous.loaded) {
+          return { ...previous, lastFetchFailed: true };
+        }
+        const cached = engine?.reads.get(aggregateCacheKey);
+        return cached === void 0 ? { ...previous, lastFetchFailed: true } : { ...cached.value, loaded: true, lastFetchFailed: true };
+      });
     }
-  }, [client, tripRef]);
+  }, [client, tripRef, engine, aggregateCacheKey]);
   useEffect(() => {
     void refreshAggregate();
   }, [refreshAggregate]);
@@ -233,7 +257,54 @@ 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: {
+        id: "pending",
+        version: 0,
+        sectionId: aggregate.sections.find((section) => section.key === input.sectionKey)?.id ?? "",
+        key: input.key ?? null,
+        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. The optimistic row shows it unstarred for the moment it
+        // takes the write to land, and the canonical row replaces it.
+        isMandatory: input.mandatory === true,
+        gate: null,
+        dayOffset: null,
+        sortOrder: Number.MAX_SAFE_INTEGER,
+        completedByManagerId: null,
+        performedByKind: null,
+        performedById: null,
+        completedAt: null,
+        updatedAt: null
+      }
     }),
     // ── THE DESK LANE ─────────────────────────────────────────────────
     // All five checklist writes take the admin triple, and a desk console
@@ -375,7 +446,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,
@@ -2150,5 +2221,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-7WXE5SUS.js.map
-//# sourceMappingURL=chunk-7WXE5SUS.js.map
+//# sourceMappingURL=chunk-WTTGPWMR.js.map
+//# sourceMappingURL=chunk-WTTGPWMR.js.map