kaafil-react-uikit 0.12.0 → 0.13.0

package/dist/manager/index.d.cts

@@ -267,6 +267,41 @@ interface ManagerItinerarySectionProps {
  */
 declare function ManagerItinerarySection(props: ManagerItinerarySectionProps): ReactNode;
 
+/**
+ * The ONE place a free-text `Traveller.dietary` value becomes a bucket.
+ *
+ * ── WHY THIS EXISTS ──
+ * `kaafil-engine`'s `Traveller.dietary` is a free-text `String?` the partner CRM
+ * supplies verbatim (the engine's own published example is `"Vegetarian, no
+ * nuts"`). Three copies of this function previously lived in three families,
+ * each testing raw EXACT EQUALITY against `V|VEG|VEGETARIAN` /
+ * `NV|NON-VEG|NONVEG|NON_VEG` / `J|JAIN` and returning `null` for anything else.
+ *
+ * Case was handled; nothing else was. `"Pure veg"`, `"Vegetarian, no nuts"`,
+ * `"Vegan"`, `"Halal"` and `"Non veg"` (space, no hyphen) all fell to `null`,
+ * which the UI rendered identically to "nothing recorded". A trip with twelve
+ * vegetarians spelled several ways showed a "Veg 4 / 20" stat card, and the
+ * kitchen ordered four meals — and the filter chips silently hid the other
+ * eight rather than offering a bucket they could appear in.
+ *
+ * ── THE RULE ──
+ * `null` is returned ONLY when there is genuinely no value (null, or blank after
+ * trimming). Every other input lands in a bucket, so a count over the buckets
+ * always sums to the roster. That is the property the old version lacked and the
+ * reason `OTHER` exists at all.
+ *
+ * ── WHY `VEGAN` IS `OTHER` AND NOT `V` ──
+ * Folding it into `V` would report a vegan as served by the vegetarian meal and
+ * remove the only signal that anything further is needed. `OTHER` keeps them
+ * counted, visible and distinct. The same argument covers `HALAL` and every
+ * allergy note. `OTHER` means "recorded, and not one of the three the badge
+ * knows" — never "unknown".
+ *
+ * Callers should still render the RAW string alongside the bucket: `"Vegetarian,
+ * no nuts"` buckets as `V`, and the nut allergy lives only in the raw text.
+ */
+type DietBucket = 'V' | 'NV' | 'J' | 'OTHER';
+
 interface TripBookingGroupCardProps {
     readonly group: ManifestBookingGroup;
     readonly onCollect: (groupId: string) => void;
@@ -331,7 +366,7 @@ interface ManagerManifestSectionProps {
     };
     readonly style?: KaafilTokenOverrides;
 }
-type ManifestDiet = 'V' | 'NV' | 'J';
+type ManifestDiet = DietBucket;
 /** `BookingStatus` (`kaafil-engine`'s real Prisma enum) — never a free
  * string; the DTO widens it to `z.string()` on the wire, but this composite
  * only ever recognizes these three real values. */
@@ -732,7 +767,16 @@ interface ManagerSyncCenterSectionProps {
 
 declare function ManagerSyncCenterSection({ tripRef, getEntityLabel, onResolved, style, }: ManagerSyncCenterSectionProps): ReactNode;
 
-type TravellerDiet = 'V' | 'NV' | 'J';
+/**
+ * The bucket a free-text `Traveller.dietary` value lands in.
+ *
+ * One shared implementation in `core/internal/normalizeDiet.ts`, not a local
+ * copy: three copies of this rule existed and all three dropped `"Pure veg"`
+ * and `"Vegan"` to `null`, which rendered as "nothing recorded". See that
+ * file's header.
+ */
+type TravellerDiet = DietBucket;
+
 interface DietaryRow {
     readonly key: string;
     readonly labelKey: string;

package/dist/manager/index.d.ts

@@ -267,6 +267,41 @@ interface ManagerItinerarySectionProps {
  */
 declare function ManagerItinerarySection(props: ManagerItinerarySectionProps): ReactNode;
 
+/**
+ * The ONE place a free-text `Traveller.dietary` value becomes a bucket.
+ *
+ * ── WHY THIS EXISTS ──
+ * `kaafil-engine`'s `Traveller.dietary` is a free-text `String?` the partner CRM
+ * supplies verbatim (the engine's own published example is `"Vegetarian, no
+ * nuts"`). Three copies of this function previously lived in three families,
+ * each testing raw EXACT EQUALITY against `V|VEG|VEGETARIAN` /
+ * `NV|NON-VEG|NONVEG|NON_VEG` / `J|JAIN` and returning `null` for anything else.
+ *
+ * Case was handled; nothing else was. `"Pure veg"`, `"Vegetarian, no nuts"`,
+ * `"Vegan"`, `"Halal"` and `"Non veg"` (space, no hyphen) all fell to `null`,
+ * which the UI rendered identically to "nothing recorded". A trip with twelve
+ * vegetarians spelled several ways showed a "Veg 4 / 20" stat card, and the
+ * kitchen ordered four meals — and the filter chips silently hid the other
+ * eight rather than offering a bucket they could appear in.
+ *
+ * ── THE RULE ──
+ * `null` is returned ONLY when there is genuinely no value (null, or blank after
+ * trimming). Every other input lands in a bucket, so a count over the buckets
+ * always sums to the roster. That is the property the old version lacked and the
+ * reason `OTHER` exists at all.
+ *
+ * ── WHY `VEGAN` IS `OTHER` AND NOT `V` ──
+ * Folding it into `V` would report a vegan as served by the vegetarian meal and
+ * remove the only signal that anything further is needed. `OTHER` keeps them
+ * counted, visible and distinct. The same argument covers `HALAL` and every
+ * allergy note. `OTHER` means "recorded, and not one of the three the badge
+ * knows" — never "unknown".
+ *
+ * Callers should still render the RAW string alongside the bucket: `"Vegetarian,
+ * no nuts"` buckets as `V`, and the nut allergy lives only in the raw text.
+ */
+type DietBucket = 'V' | 'NV' | 'J' | 'OTHER';
+
 interface TripBookingGroupCardProps {
     readonly group: ManifestBookingGroup;
     readonly onCollect: (groupId: string) => void;
@@ -331,7 +366,7 @@ interface ManagerManifestSectionProps {
     };
     readonly style?: KaafilTokenOverrides;
 }
-type ManifestDiet = 'V' | 'NV' | 'J';
+type ManifestDiet = DietBucket;
 /** `BookingStatus` (`kaafil-engine`'s real Prisma enum) — never a free
  * string; the DTO widens it to `z.string()` on the wire, but this composite
  * only ever recognizes these three real values. */
@@ -732,7 +767,16 @@ interface ManagerSyncCenterSectionProps {
 
 declare function ManagerSyncCenterSection({ tripRef, getEntityLabel, onResolved, style, }: ManagerSyncCenterSectionProps): ReactNode;
 
-type TravellerDiet = 'V' | 'NV' | 'J';
+/**
+ * The bucket a free-text `Traveller.dietary` value lands in.
+ *
+ * One shared implementation in `core/internal/normalizeDiet.ts`, not a local
+ * copy: three copies of this rule existed and all three dropped `"Pure veg"`
+ * and `"Vegan"` to `null`, which rendered as "nothing recorded". See that
+ * file's header.
+ */
+type TravellerDiet = DietBucket;
+
 interface DietaryRow {
     readonly key: string;
     readonly labelKey: string;

package/dist/manager/index.js

@@ -1,9 +1,10 @@
-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-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, useOfflineEvent, useKaafilBrand } from '../chunk-YJJ4DI42.js';
+import { useManagerMe, useConflicts, useNotifications } from '../chunk-CJLQ2Z4Q.js';
+import { normalizeDiet } from '../chunk-JOUTGBUJ.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-LW2YB7VC.js';
+import { useOfflineEngine } from '../chunk-IAKK2CKB.js';
+import { OfflineFallbackScreen } from '../chunk-YSJXNKCU.js';
+export { BootErrorScreen, LockedTripScreen, NotFoundScreen, OfflineFallbackScreen, SessionExpiredScreen } from '../chunk-YSJXNKCU.js';
+import { createRegistry, useKaafilChrome, useSession, tokenStyle, useOutboxStatus, useOfflineEvent, useKaafilBrand } from '../chunk-AKJLIH5A.js';
 import { useState, useRef, useEffect, useMemo, useId } from 'react';
 import { jsx, jsxs, Fragment } from 'react/jsx-runtime';
 
@@ -4118,22 +4119,6 @@ var RECOGNIZED_BOOKING_STATUS = /* @__PURE__ */ new Set([
 function normalizeBookingStatus(raw) {
   return RECOGNIZED_BOOKING_STATUS.has(raw) ? raw : "TENTATIVE";
 }
-function normalizeDiet(raw) {
-  if (raw === null) {
-    return null;
-  }
-  const upper = raw.trim().toUpperCase();
-  if (upper === "V" || upper === "VEG" || upper === "VEGETARIAN") {
-    return "V";
-  }
-  if (upper === "NV" || upper === "NON-VEG" || upper === "NONVEG" || upper === "NON_VEG") {
-    return "NV";
-  }
-  if (upper === "J" || upper === "JAIN") {
-    return "J";
-  }
-  return null;
-}
 function toManifestTravellers(roster) {
   return roster.map((row) => ({
     travellerId: row.traveller.travellerId,
@@ -4358,11 +4343,12 @@ function CloseIcon5(props) {
     }
   );
 }
-var DIET_OPTIONS = ["V", "NV", "J"];
+var DIET_OPTIONS = ["V", "NV", "J", "OTHER"];
 var DIET_LABEL_KEY = {
   V: "chrome.managerTrip.manifest.traveller.diet.v",
   NV: "chrome.managerTrip.manifest.traveller.diet.nv",
-  J: "chrome.managerTrip.manifest.traveller.diet.j"
+  J: "chrome.managerTrip.manifest.traveller.diet.j",
+  OTHER: "chrome.managerTrip.manifest.traveller.diet.other"
 };
 var BALANCE_OPTIONS = ["due", "settled"];
 var BALANCE_LABEL_KEY = {
@@ -4551,7 +4537,8 @@ function TripManifestStats({ stats }) {
 var DIET_LABEL_KEY2 = {
   V: "chrome.managerTrip.manifest.traveller.diet.v",
   NV: "chrome.managerTrip.manifest.traveller.diet.nv",
-  J: "chrome.managerTrip.manifest.traveller.diet.j"
+  J: "chrome.managerTrip.manifest.traveller.diet.j",
+  OTHER: "chrome.managerTrip.manifest.traveller.diet.other"
 };
 var STATUS_LABEL_KEY3 = {
   TENTATIVE: "chrome.managerTrip.manifest.traveller.status.tentative",
@@ -4635,7 +4622,8 @@ function ReadyManagerManifestSection({
       diet: {
         V: travellers.filter((row) => row.diet === "V").length,
         NV: travellers.filter((row) => row.diet === "NV").length,
-        J: travellers.filter((row) => row.diet === "J").length
+        J: travellers.filter((row) => row.diet === "J").length,
+        OTHER: travellers.filter((row) => row.diet === "OTHER").length
       },
       balance: {
         due: travellers.filter((row) => row.dueMinor > 0).length,
@@ -9222,7 +9210,6 @@ function UnassignedPoolSection({
         children: [
           /* @__PURE__ */ jsx("span", { className: "kf-manager-rooming__pool-avatar", children: traveller.glyph }),
           /* @__PURE__ */ jsx("span", { className: "kf-manager-rooming__pool-name", children: traveller.fullName }),
-          traveller.dietary !== null ? /* @__PURE__ */ jsx("span", { className: "kf-manager-rooming__pill", "data-tone": "soft", children: traveller.dietary }) : null,
           traveller.medicalFlag ? /* @__PURE__ */ jsx("span", { className: "kf-manager-rooming__pill", "data-tone": "warn", children: t("chrome.managerTrip.rooming.pool.medical") }) : null
         ]
       },
@@ -9988,30 +9975,17 @@ function ManagerSyncCenterSection({
 }
 
 // src/manager/composite/office/ManagerTravellerProfileSection/internal/deriveTravellerProfile.ts
-function normalizeDiet2(raw) {
-  if (raw === null) {
-    return null;
-  }
-  const upper = raw.trim().toUpperCase();
-  if (upper === "V" || upper === "VEG" || upper === "VEGETARIAN") {
-    return "V";
-  }
-  if (upper === "NV" || upper === "NON-VEG" || upper === "NONVEG" || upper === "NON_VEG") {
-    return "NV";
-  }
-  if (upper === "J" || upper === "JAIN") {
-    return "J";
-  }
-  return null;
-}
 function deriveDietaryRows(dietary, mealPreference, dietaryReqs, allergies, medicalFlag, medicalNotes) {
   const rows = [];
-  const diet = normalizeDiet2(dietary);
+  const diet = normalizeDiet(dietary);
   if (mealPreference !== null || diet !== null) {
     rows.push({
       key: "meal",
       labelKey: "chrome.managerTraveller.dietary.meal",
-      value: mealPreference ?? "",
+      // The RAW string when the CRM sent no separate meal preference. The bucket
+      // is a badge and necessarily lossy — `"Vegetarian, no nuts"` buckets as
+      // `V` — so dropping the text left the nut allergy on screen NOWHERE.
+      value: mealPreference ?? dietary ?? "",
       diet,
       isAlert: false
     });
@@ -10357,7 +10331,8 @@ function TravellerContactRow({ fullName, phone }) {
 var DIET_LABEL_KEY3 = {
   V: "chrome.managerTraveller.dietary.diet.v",
   NV: "chrome.managerTraveller.dietary.diet.nv",
-  J: "chrome.managerTraveller.dietary.diet.j"
+  J: "chrome.managerTraveller.dietary.diet.j",
+  OTHER: "chrome.managerTraveller.dietary.diet.other"
 };
 function TravellerDietaryCard({ rows }) {
   const { t } = useKaafilChrome();