@burdenoff/microfe-movethewheels 2026.925.2 → 2026.925.3

package/src/movethewheels/hooks/useInventory.ts

@@ -30,7 +30,11 @@ export interface CreateWarehouseInput {
     postalCode: string;
     country: string;
   };
+  /** Free-text on the backend, e.g. "ACTIVE", "INACTIVE". Defaults to ACTIVE. */
+  status?: string;
+  contactInfo?: { email?: string; phone?: string };
   totalCapacity?: number;
+  usedCapacity?: number;
 }
 
 interface UseInventoryResult {
@@ -54,6 +58,7 @@ interface UseInventoryResult {
   createWarehouse: (input: CreateWarehouseInput) => Promise<Warehouse>;
   updateProduct: (id: string, input: CreateProductInput) => Promise<Product>;
   updateWarehouse: (id: string, input: CreateWarehouseInput) => Promise<Warehouse>;
+  deleteWarehouse: (id: string) => Promise<void>;
 }
 
 // Wire shape of MtwInventoryItem (wspace-movethewheels-svc:
@@ -100,6 +105,9 @@ interface ApiWarehouse {
   code: string;
   type: string;
   address: Record<string, unknown>;
+  status: string;
+  contactInfo?: Record<string, unknown> | null;
+  operatingHours?: Record<string, unknown> | null;
   totalCapacity?: number | null;
   usedCapacity?: number | null;
   createdAt: string;
@@ -163,6 +171,9 @@ const LIST_WAREHOUSES_QUERY = `
       code
       type
       address
+      status
+      contactInfo
+      operatingHours
       totalCapacity
       usedCapacity
       createdAt
@@ -200,6 +211,9 @@ const GET_WAREHOUSE_QUERY = `
       code
       type
       address
+      status
+      contactInfo
+      operatingHours
       totalCapacity
       usedCapacity
       createdAt
@@ -293,6 +307,9 @@ const CREATE_WAREHOUSE_MUTATION = `
       code
       type
       address
+      status
+      contactInfo
+      operatingHours
       totalCapacity
       usedCapacity
       createdAt
@@ -300,6 +317,12 @@ const CREATE_WAREHOUSE_MUTATION = `
   }
 `;
 
+const DELETE_WAREHOUSE_MUTATION = `
+  mutation DeleteWarehouse($id: ID!) {
+    deleteWarehouse(id: $id)
+  }
+`;
+
 const UPDATE_WAREHOUSE_MUTATION = `
   mutation UpdateWarehouse($id: ID!, $input: UpdateWarehouseInput!) {
     updateWarehouse(id: $id, input: $input) {
@@ -308,6 +331,9 @@ const UPDATE_WAREHOUSE_MUTATION = `
       code
       type
       address
+      status
+      contactInfo
+      operatingHours
       totalCapacity
       usedCapacity
       createdAt
@@ -363,6 +389,16 @@ function mapProduct(product: ApiProduct): Product {
 function mapWarehouse(warehouse: ApiWarehouse): Warehouse {
   const totalSpace = warehouse.totalCapacity ?? 0;
   const usedSpace = warehouse.usedCapacity ?? 0;
+  const operatingHoursRaw = warehouse.operatingHours ?? {};
+  const operatingHours: Warehouse['operatingHours'] = {};
+  for (const [day, hours] of Object.entries(operatingHoursRaw)) {
+    if (hours && typeof hours === 'object' && 'open' in hours && 'close' in hours) {
+      operatingHours[day] = { open: String(hours.open), close: String(hours.close) };
+    } else {
+      operatingHours[day] = null;
+    }
+  }
+  const contactInfoRaw = warehouse.contactInfo ?? undefined;
   return {
     id: warehouse.id,
     name: warehouse.name,
@@ -376,28 +412,19 @@ function mapWarehouse(warehouse: ApiWarehouse): Warehouse {
       ...(warehouse.address ?? {}),
     },
     type: (warehouse.type?.toLowerCase() ?? 'distribution') as Warehouse['type'],
+    status: warehouse.status ?? 'ACTIVE',
     capacity: {
       totalSpace,
       usedSpace,
-      totalPallets: 0,
-      usedPallets: 0,
-      specialAreas: [],
-    },
-    operatingHours: {},
-    contacts: [],
-    zones: [],
-    equipment: [],
-    staff: [],
-    metrics: {
-      throughput: 0,
-      accuracy: 0,
-      productivity: 0,
-      utilization: totalSpace > 0 ? (usedSpace / totalSpace) * 100 : 0,
-      ordersFulfilled: 0,
-      averageProcessingTime: 0,
-      lastUpdated: new Date(),
     },
-    isActive: true,
+    operatingHours,
+    contactInfo: contactInfoRaw
+      ? {
+          email: (contactInfoRaw.email as string | undefined) ?? undefined,
+          phone: (contactInfoRaw.phone as string | undefined) ?? undefined,
+        }
+      : undefined,
+    utilization: totalSpace > 0 ? (usedSpace / totalSpace) * 100 : 0,
     createdAt: new Date(warehouse.createdAt),
   };
 }
@@ -410,20 +437,12 @@ function mapInventory(item: ApiInventoryItem): Inventory {
     productSku: item.cargoItem?.sku,
     warehouseId: item.warehouseId,
     warehouseName: item.warehouse?.name,
-    zoneId: '',
     quantity: item.onHand,
     reservedQuantity: item.allocated,
     availableQuantity: item.available,
-    reorderLevel: item.reorderPoint ?? 0,
-    maxLevel: Math.max(item.onHand * 2, item.available),
+    damagedQuantity: item.damaged,
+    reorderLevel: item.reorderPoint ?? undefined,
     lastRestocked: new Date(item.updatedAt),
-    location: {
-      aisle: '',
-      shelf: '',
-      position: '',
-      level: 0,
-    },
-    movements: [],
     createdAt: new Date(item.createdAt),
     updatedAt: new Date(item.updatedAt),
   };
@@ -541,7 +560,11 @@ export function useInventory(options: UseInventoryOptions = {}): UseInventoryRes
             onHand: current.quantity + quantity,
             available: current.availableQuantity + quantity,
             allocated: current.reservedQuantity,
-            damaged: 0,
+            // Pass the current value through — the resolver spreads this input
+            // straight into a Prisma update, so hardcoding 0 here would
+            // silently zero out any real damaged-unit count on every
+            // adjustment (BOFF-7909 follow-up).
+            damaged: current.damagedQuantity,
           },
         }
       );
@@ -614,7 +637,10 @@ export function useInventory(options: UseInventoryOptions = {}): UseInventoryRes
             name: input.name,
             type: input.type,
             address: input.address,
+            status: input.status,
+            contactInfo: input.contactInfo,
             totalCapacity: input.totalCapacity,
+            usedCapacity: input.usedCapacity,
           },
         }
       );
@@ -660,7 +686,10 @@ export function useInventory(options: UseInventoryOptions = {}): UseInventoryRes
             name: input.name,
             type: input.type,
             address: input.address,
+            status: input.status,
+            contactInfo: input.contactInfo,
             totalCapacity: input.totalCapacity,
+            usedCapacity: input.usedCapacity,
           },
         }
       );
@@ -671,6 +700,18 @@ export function useInventory(options: UseInventoryOptions = {}): UseInventoryRes
     [graphQLOptions]
   );
 
+  const deleteWarehouse = useCallback(
+    async (id: string): Promise<void> => {
+      await executeGraphQL<{ deleteWarehouse: boolean }>(
+        graphQLOptions,
+        DELETE_WAREHOUSE_MUTATION,
+        { id }
+      );
+      setWarehouses((previous) => previous.filter((item) => item.id !== id));
+    },
+    [graphQLOptions]
+  );
+
   return {
     inventory,
     lowStockItems,
@@ -687,6 +728,7 @@ export function useInventory(options: UseInventoryOptions = {}): UseInventoryRes
     createWarehouse,
     updateProduct,
     updateWarehouse,
+    deleteWarehouse,
   };
 }
 
@@ -702,6 +744,8 @@ interface UseProductsOptions extends UseListOptions {
 
 interface UseWarehousesOptions extends UseListOptions {
   search?: string;
+  /** Raw backend value, e.g. "ACTIVE"/"INACTIVE". */
+  status?: string;
 }
 
 export function useProducts(options: UseProductsOptions = {}) {
@@ -826,7 +870,7 @@ export function useProduct(productId: string | undefined) {
 }
 
 export function useWarehouses(options: UseWarehousesOptions = {}) {
-  const { limit = 20, offset = 0, search } = options;
+  const { limit = 20, offset = 0, search, status } = options;
   const graphQLOptions = useGraphqlOptions();
   const [warehouses, setWarehouses] = useState<Warehouse[]>([]);
   const [totalWarehouses, setTotalWarehouses] = useState(0);
@@ -842,7 +886,7 @@ export function useWarehouses(options: UseWarehousesOptions = {}) {
         warehouses: ApiWarehouse[];
         warehousesCount: number;
       }>(graphQLOptions, LIST_WAREHOUSES_QUERY, {
-        filter: { search: search || undefined },
+        filter: { search: search || undefined, status: status || undefined },
         pagination: { limit, offset },
       });
       setTotalWarehouses(data.warehousesCount);
@@ -853,7 +897,7 @@ export function useWarehouses(options: UseWarehousesOptions = {}) {
     } finally {
       setIsLoading(false);
     }
-  }, [graphQLOptions, limit, offset, search]);
+  }, [graphQLOptions, limit, offset, search, status]);
 
   useEffect(() => {
     void load();

package/src/movethewheels/hooks/useRoutes.ts

@@ -226,14 +226,18 @@ export function mapRoute(route: ApiRoute): Route {
     vehicleId: route.vehicleId ?? '',
     status: mapRouteStatus(route.status),
     stops: mapWaypoints(route.waypoints),
-    distance: route.estimatedDistance ?? 0,
+    // Prefer the real post-drive distance once it exists; the backend fetches
+    // it (`actualDistance`) but it used to be silently discarded here.
+    distance: route.actualDistance ?? route.estimatedDistance ?? 0,
     estimatedDuration: route.estimatedDuration ?? 0,
     fuelCost: route.fuelCost ?? undefined,
     actualDuration: route.actualDuration ?? undefined,
-    startTime: undefined,
-    endTime: undefined,
-    optimized: false,
-    trafficConditions: [],
+    // No `optimized` flag exists on the backend — `optimizeRoute` is the only
+    // path that computes a real `estimatedDistance` (create/update never
+    // collect one), so a positive value is a reasonably honest proxy. This
+    // used to be hardcoded `false` and reset on every reload even for a
+    // route that really had been optimized (BOFF-7909 follow-up).
+    optimized: (route.estimatedDistance ?? 0) > 0,
     createdAt: new Date(route.createdAt),
   };
 }
@@ -311,11 +315,13 @@ export function useRoutes(options: UseRoutesOptions = {}): UseRoutesResult {
             vehicleId: route.vehicleId,
             driverId: route.driverId,
             waypoints: route.stops ?? [],
-            estimatedDistance: route.distance ?? 0,
-            estimatedDuration: route.estimatedDuration ?? 0,
+            // The create form never collects a distance — leave estimated/actual
+            // unset (not a forced 0) so a fresh route honestly shows as
+            // "not yet optimized" until `optimizeRoute` computes a real one.
+            // `distance` is never sent as `actualDistance`: that field means a
+            // real post-drive distance, which this form has no way to know.
+            estimatedDuration: route.estimatedDuration,
             fuelCost: route.fuelCost,
-            actualDistance: route.distance,
-            actualDuration: route.actualDuration,
           },
         }
       );
@@ -341,11 +347,10 @@ export function useRoutes(options: UseRoutesOptions = {}): UseRoutesResult {
             vehicleId: updates.vehicleId,
             driverId: updates.driverId,
             waypoints: updates.stops,
-            estimatedDistance: updates.distance,
+            // See createRoute — the edit form doesn't collect a distance
+            // either, and `distance` must never masquerade as `actualDistance`.
             estimatedDuration: updates.estimatedDuration,
             fuelCost: updates.fuelCost,
-            actualDistance: updates.distance,
-            actualDuration: updates.actualDuration,
           },
         }
       );