@burdenoff/microfe-movethewheels 2026.929.2 → 2026.929.3

package/dist/hooks/useFleet.js

@@ -90,7 +90,7 @@ function E(e) {
 		phone: e.phone ?? "",
 		employeeId: e.employeeId,
 		licenseNumber: e.licenseNumber ?? "",
-		licenseType: e.licenseClass ?? "standard",
+		licenseType: e.licenseClass ?? "",
 		licenseExpiry: e.licenseExpiry ? new Date(e.licenseExpiry) : void 0,
 		vehicleIds: e.vehicleId ? [e.vehicleId] : [],
 		assignedVehicle: e.vehicleId ?? void 0,

package/dist/movethewheels/hooks/useFleet.d.ts

@@ -44,8 +44,8 @@ export interface CreateDriverInput {
     bonusYtd?: number;
     payCurrency?: string;
 }
-/** Superset of `CreateDriverInput` with fields that only make sense on an existing driver. */
-export interface UpdateDriverInput extends CreateDriverInput {
+/** Fields accepted when updating an existing driver; employee IDs are immutable. */
+export interface UpdateDriverInput extends Omit<CreateDriverInput, 'employeeId'> {
     onLeaveUntil?: string | null;
     terminatedAt?: string;
     terminationReason?: string;

package/dist/pages/DriverDetailsPage.js

@@ -25,12 +25,12 @@ import { EntityCombobox as re } from "../components/EntityCombobox.js";
 import { useEffect as ie, useState as z } from "react";
 import { useLocation as B, useParams as V } from "react-router";
 import { Fragment as H, jsx as U, jsxs as W } from "react/jsx-runtime";
-import { EmphasisPanel as ae, NextSteps as G, PagePurpose as K, ResponsiveDialog as oe, ResponsiveDialogContent as se, ResponsiveDialogDescription as ce, ResponsiveDialogFooter as le, ResponsiveDialogHeader as ue, ResponsiveDialogTitle as de } from "@burdenoff/fe-libs/ui";
+import { EmphasisPanel as ae, NextSteps as G, PagePurpose as oe, ResponsiveDialog as se, ResponsiveDialogContent as ce, ResponsiveDialogDescription as le, ResponsiveDialogFooter as ue, ResponsiveDialogHeader as de, ResponsiveDialogTitle as fe } from "@burdenoff/fe-libs/ui";
 //#region src/movethewheels/pages/DriverDetailsPage.tsx
-var fe = Object.entries(N).map(([e, t]) => ({
+var pe = Object.entries(N).map(([e, t]) => ({
 	value: e,
 	label: t
-})), q = (e) => e ? e.toISOString().slice(0, 10) : "", pe = (e) => e ? `${e}T12:00:00.000Z` : void 0, me = {
+})), K = (e) => e ? e.toISOString().slice(0, 10) : "", q = (e) => e ? `${e}T12:00:00.000Z` : void 0, me = {
 	employeeId: "",
 	firstName: "",
 	lastName: "",
@@ -65,36 +65,39 @@ function J({ editId: t }) {
 			phone: j.phone || "",
 			licenseNumber: j.licenseNumber || "",
 			licenseClass: j.licenseType || "",
-			licenseExpiry: q(j.licenseExpiry),
+			licenseExpiry: K(j.licenseExpiry),
 			status: j.status,
 			vehicleId: j.assignedVehicle || "",
 			rating: j.rating ? String(j.rating) : "",
 			notes: j.notes || "",
-			hireDate: q(j.hireDate),
+			hireDate: K(j.hireDate),
 			baseSalary: j.baseSalary === void 0 ? "" : String(j.baseSalary),
 			bonusYtd: j.bonusYtd === void 0 ? "" : String(j.bonusYtd),
 			payCurrency: j.payCurrency || "",
-			onLeaveUntil: q(j.onLeaveUntil)
+			onLeaveUntil: K(j.onLeaveUntil)
 		});
 	}, [t, j]);
-	let we = () => i?.(d(r, "fleet/drivers")), $ = () => ({
-		employeeId: P.employeeId.trim(),
+	let we = () => i?.(d(r, "fleet/drivers")), $ = (e = !0) => ({
 		firstName: P.firstName.trim(),
 		lastName: P.lastName.trim(),
 		email: P.email.trim() || void 0,
 		phone: P.phone.trim() || void 0,
 		licenseNumber: P.licenseNumber.trim() || void 0,
 		licenseClass: P.licenseClass.trim() || void 0,
-		licenseExpiry: pe(P.licenseExpiry),
+		licenseExpiry: q(P.licenseExpiry),
 		status: p(P.status),
 		vehicleId: P.vehicleId || void 0,
-		rating: P.rating ? Number(P.rating) : void 0,
+		rating: P.rating.trim() ? Number(P.rating) : void 0,
 		notes: P.notes.trim() || void 0,
-		hireDate: pe(P.hireDate),
-		baseSalary: P.baseSalary ? Number(P.baseSalary) : void 0,
-		bonusYtd: P.bonusYtd ? Number(P.bonusYtd) : void 0,
-		payCurrency: P.payCurrency.trim() || void 0
-	}), Te = async (e) => {
+		hireDate: q(P.hireDate),
+		baseSalary: P.baseSalary.trim() ? Number(P.baseSalary) : void 0,
+		bonusYtd: P.bonusYtd.trim() ? Number(P.bonusYtd) : void 0,
+		payCurrency: P.payCurrency.trim() || void 0,
+		...e ? { onLeaveUntil: q(P.onLeaveUntil) ?? null } : {}
+	}), Te = () => ({
+		employeeId: P.employeeId.trim(),
+		...$(!1)
+	}), Ee = async (e) => {
 		if (e.preventDefault(), !P.employeeId.trim() || !P.firstName.trim() || !P.lastName.trim()) {
 			G("Employee ID, first name and last name are required.");
 			return;
@@ -109,13 +112,13 @@ function J({ editId: t }) {
 				let e = await s(t, $());
 				i?.(d(r, `fleet/drivers/${e.id}`));
 			} else {
-				let e = await o($());
+				let e = await o(Te());
 				i?.(d(r, `fleet/drivers/${e.id}`));
 			}
 		} catch (e) {
 			console.error("Failed to save driver:", e), G(e instanceof Error ? e.message : "Failed to save driver. Please try again."), B(!1);
 		}
-	}, Ee = async () => {
+	}, De = async () => {
 		if (!t || !X.trim()) {
 			Q("A reason is required to dismiss a driver.");
 			return;
@@ -132,7 +135,7 @@ function J({ editId: t }) {
 		} finally {
 			ge(!1);
 		}
-	}, De = N && !!j?.terminatedAt;
+	}, Oe = N && !!j?.terminatedAt;
 	return /* @__PURE__ */ W(M, {
 		title: N ? "Edit Driver" : "Add Driver",
 		subtitle: "Add a driver to your workforce",
@@ -145,7 +148,7 @@ function J({ editId: t }) {
 				disabled: R,
 				children: [/* @__PURE__ */ U(e, { size: 16 }), " Cancel"]
 			}), /* @__PURE__ */ W(v, {
-				onClick: Te,
+				onClick: Ee,
 				disabled: R,
 				children: [
 					/* @__PURE__ */ U(a, { size: 16 }),
@@ -155,11 +158,11 @@ function J({ editId: t }) {
 			})]
 		}),
 		children: [
-			/* @__PURE__ */ U(K, {
+			/* @__PURE__ */ U(oe, {
 				icon: /* @__PURE__ */ U(u, { size: 18 }),
 				children: "Add a driver so they can be assigned to orders and routes. An employee ID, name and phone get them on the roster — add licensing, a vehicle and HR details to keep the record complete."
 			}),
-			De && j?.terminatedAt && /* @__PURE__ */ W("div", {
+			Oe && j?.terminatedAt && /* @__PURE__ */ W("div", {
 				className: "rounded-card border border-status-error-border bg-status-error-bg-subtle p-4 text-sm text-status-error-text",
 				children: [/* @__PURE__ */ W("p", {
 					className: "font-medium",
@@ -174,7 +177,7 @@ function J({ editId: t }) {
 				})]
 			}),
 			/* @__PURE__ */ W("form", {
-				onSubmit: Te,
+				onSubmit: Ee,
 				className: "max-w-3xl space-y-6",
 				children: [
 					/* @__PURE__ */ U(F, { message: V }),
@@ -191,10 +194,15 @@ function J({ editId: t }) {
 										className: "space-y-2",
 										children: [/* @__PURE__ */ W(w, {
 											htmlFor: "employeeId",
-											children: ["Employee ID ", /* @__PURE__ */ U("span", {
-												className: "text-status-error-text",
-												children: "*"
-											})]
+											children: [
+												"Employee ID ",
+												N ? "(immutable)" : "",
+												" ",
+												/* @__PURE__ */ U("span", {
+													className: "text-status-error-text",
+													children: !N && "*"
+												})
+											]
 										}), /* @__PURE__ */ U(C, {
 											id: "employeeId",
 											value: P.employeeId,
@@ -203,7 +211,8 @@ function J({ editId: t }) {
 												employeeId: e.target.value
 											})),
 											placeholder: "EMP-001",
-											required: !0
+											required: !0,
+											disabled: N
 										})]
 									}),
 									/* @__PURE__ */ W("div", {
@@ -298,7 +307,7 @@ function J({ editId: t }) {
 											children: [/* @__PURE__ */ U(O, {
 												id: "status",
 												children: /* @__PURE__ */ U(k, {})
-											}), /* @__PURE__ */ U(E, { children: fe.map((e) => /* @__PURE__ */ U(D, {
+											}), /* @__PURE__ */ U(E, { children: pe.map((e) => /* @__PURE__ */ U(D, {
 												value: e.value,
 												children: e.label
 											}, e.value)) })]
@@ -441,9 +450,21 @@ function J({ editId: t }) {
 									}),
 									/* @__PURE__ */ W("div", {
 										className: "space-y-2",
-										children: [/* @__PURE__ */ U(w, {
-											htmlFor: "onLeaveUntil",
-											children: "On Leave Until"
+										children: [/* @__PURE__ */ W("div", {
+											className: "flex items-center justify-between gap-2",
+											children: [/* @__PURE__ */ U(w, {
+												htmlFor: "onLeaveUntil",
+												children: "On Leave Until"
+											}), P.onLeaveUntil && /* @__PURE__ */ U(v, {
+												type: "button",
+												variant: "ghost",
+												size: "sm",
+												onClick: () => I((e) => ({
+													...e,
+													onLeaveUntil: ""
+												})),
+												children: "Clear"
+											})]
 										}), /* @__PURE__ */ U(C, {
 											id: "onLeaveUntil",
 											type: "date",
@@ -530,7 +551,7 @@ function J({ editId: t }) {
 						placeholder: "Performance notes, disciplinary history, anything worth tracking for this driver...",
 						rows: 4
 					}) })] }),
-					N && !De && /* @__PURE__ */ W(y, {
+					N && !Oe && /* @__PURE__ */ W(y, {
 						className: "border-status-error-border",
 						children: [/* @__PURE__ */ U(x, { children: /* @__PURE__ */ W(S, {
 							className: "flex items-center gap-2 text-status-error-text",
@@ -548,11 +569,11 @@ function J({ editId: t }) {
 					})
 				]
 			}),
-			/* @__PURE__ */ U(oe, {
+			/* @__PURE__ */ U(se, {
 				open: J,
 				onOpenChange: Y,
-				children: /* @__PURE__ */ W(se, { children: [
-					/* @__PURE__ */ W(ue, { children: [/* @__PURE__ */ U(de, { children: "Dismiss this driver?" }), /* @__PURE__ */ U(ce, { children: "This records the driver as dismissed and requires a reason. It cannot be undone from here." })] }),
+				children: /* @__PURE__ */ W(ce, { children: [
+					/* @__PURE__ */ W(de, { children: [/* @__PURE__ */ U(fe, { children: "Dismiss this driver?" }), /* @__PURE__ */ U(le, { children: "This records the driver as dismissed and requires a reason. It cannot be undone from here." })] }),
 					/* @__PURE__ */ W("div", {
 						className: "space-y-2 px-1",
 						children: [
@@ -573,14 +594,14 @@ function J({ editId: t }) {
 							/* @__PURE__ */ U(F, { message: _e })
 						]
 					}),
-					/* @__PURE__ */ W(le, { children: [/* @__PURE__ */ U(v, {
+					/* @__PURE__ */ W(ue, { children: [/* @__PURE__ */ U(v, {
 						variant: "outline",
 						onClick: () => Y(!1),
 						disabled: Z,
 						children: "Cancel"
 					}), /* @__PURE__ */ U(v, {
 						className: "bg-status-error-text text-text-inverse hover:opacity-90",
-						onClick: Ee,
+						onClick: De,
 						disabled: Z,
 						children: Z ? "Dismissing..." : "Confirm dismissal"
 					})] })
@@ -650,7 +671,7 @@ function Y() {
 			]
 		}),
 		children: [
-			/* @__PURE__ */ U(K, {
+			/* @__PURE__ */ U(oe, {
 				icon: /* @__PURE__ */ U(u, { size: 18 }),
 				children: "A driver's full profile — licensing, assigned vehicle, documents and HR record. Use it to confirm someone is road-ready and compliant before assigning them an order or route."
 			}),
@@ -788,9 +809,9 @@ function Y() {
 						}) }), /* @__PURE__ */ W(b, { children: [/* @__PURE__ */ U("p", {
 							className: "text-sm",
 							children: _.currentLocation.address || "Location updating..."
-						}), /* @__PURE__ */ W("p", {
+						}), /* @__PURE__ */ U("p", {
 							className: "text-xs text-text-secondary mt-1",
-							children: ["Updated ", L(_.currentLocation.timestamp || /* @__PURE__ */ new Date(), "relative")]
+							children: _.currentLocation.timestamp ? `Updated ${L(_.currentLocation.timestamp, "relative")}` : "Update time not recorded"
 						})] })] })
 					]
 				})]

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@burdenoff/microfe-movethewheels",
-  "version": "2026.929.2",
+  "version": "2026.929.3",
   "description": "Burdenoff MoveTheWheels Microfrontend - Logistics & Supply Chain Management Platform",
   "type": "module",
   "files": [

package/src/movethewheels/hooks/useFleet.ts

@@ -54,8 +54,8 @@ export interface CreateDriverInput {
   payCurrency?: string;
 }
 
-/** Superset of `CreateDriverInput` with fields that only make sense on an existing driver. */
-export interface UpdateDriverInput extends CreateDriverInput {
+/** Fields accepted when updating an existing driver; employee IDs are immutable. */
+export interface UpdateDriverInput extends Omit<CreateDriverInput, 'employeeId'> {
   onLeaveUntil?: string | null;
   terminatedAt?: string;
   terminationReason?: string;
@@ -602,7 +602,7 @@ export function mapDriver(driver: ApiDriver): Driver {
     phone: driver.phone ?? '',
     employeeId: driver.employeeId,
     licenseNumber: driver.licenseNumber ?? '',
-    licenseType: driver.licenseClass ?? 'standard',
+    licenseType: driver.licenseClass ?? '',
     licenseExpiry: driver.licenseExpiry ? new Date(driver.licenseExpiry) : undefined,
     vehicleIds: driver.vehicleId ? [driver.vehicleId] : [],
     assignedVehicle: driver.vehicleId ?? undefined,

package/src/movethewheels/pages/DriverDetailsPage.tsx

@@ -53,6 +53,7 @@ import {
   useVehicle,
   useVehicles,
   toApiDriverStatus,
+  type CreateDriverInput,
   type UpdateDriverInput,
 } from '../hooks/useFleet';
 import { useDebouncedValue } from '../hooks/useDebouncedValue';
@@ -163,8 +164,7 @@ function NewDriverForm({ editId }: { editId?: string }) {
 
   const goBack = () => navigate?.(joinPath(basePath, 'fleet/drivers'));
 
-  const buildPayload = (): UpdateDriverInput => ({
-    employeeId: form.employeeId.trim(),
+  const buildPayload = (includeLeaveFields = true): UpdateDriverInput => ({
     firstName: form.firstName.trim(),
     lastName: form.lastName.trim(),
     email: form.email.trim() || undefined,
@@ -174,12 +174,18 @@ function NewDriverForm({ editId }: { editId?: string }) {
     licenseExpiry: fromDateInput(form.licenseExpiry),
     status: toApiDriverStatus(form.status),
     vehicleId: form.vehicleId || undefined,
-    rating: form.rating ? Number(form.rating) : undefined,
+    rating: form.rating.trim() ? Number(form.rating) : undefined,
     notes: form.notes.trim() || undefined,
     hireDate: fromDateInput(form.hireDate),
-    baseSalary: form.baseSalary ? Number(form.baseSalary) : undefined,
-    bonusYtd: form.bonusYtd ? Number(form.bonusYtd) : undefined,
+    baseSalary: form.baseSalary.trim() ? Number(form.baseSalary) : undefined,
+    bonusYtd: form.bonusYtd.trim() ? Number(form.bonusYtd) : undefined,
     payCurrency: form.payCurrency.trim() || undefined,
+    ...(includeLeaveFields ? { onLeaveUntil: fromDateInput(form.onLeaveUntil) ?? null } : {}),
+  });
+
+  const buildCreatePayload = (): CreateDriverInput => ({
+    employeeId: form.employeeId.trim(),
+    ...buildPayload(false),
   });
 
   const handleSubmit = async (e: React.FormEvent) => {
@@ -199,7 +205,7 @@ function NewDriverForm({ editId }: { editId?: string }) {
         const updated = await updateDriver(editId, buildPayload());
         navigate?.(joinPath(basePath, `fleet/drivers/${updated.id}`));
       } else {
-        const created = await createDriver(buildPayload());
+        const created = await createDriver(buildCreatePayload());
         navigate?.(joinPath(basePath, `fleet/drivers/${created.id}`));
       }
     } catch (err) {
@@ -280,7 +286,8 @@ function NewDriverForm({ editId }: { editId?: string }) {
             <div className="grid grid-cols-1 md:grid-cols-3 gap-4">
               <div className="space-y-2">
                 <Label htmlFor="employeeId">
-                  Employee ID <span className="text-status-error-text">*</span>
+                  Employee ID {isEditing ? '(immutable)' : ''}{' '}
+                  <span className="text-status-error-text">{!isEditing && '*'}</span>
                 </Label>
                 <Input
                   id="employeeId"
@@ -288,6 +295,7 @@ function NewDriverForm({ editId }: { editId?: string }) {
                   onChange={(e) => setForm((prev) => ({ ...prev, employeeId: e.target.value }))}
                   placeholder="EMP-001"
                   required
+                  disabled={isEditing}
                 />
               </div>
               <div className="space-y-2">
@@ -445,7 +453,19 @@ function NewDriverForm({ editId }: { editId?: string }) {
                 />
               </div>
               <div className="space-y-2">
-                <Label htmlFor="onLeaveUntil">On Leave Until</Label>
+                <div className="flex items-center justify-between gap-2">
+                  <Label htmlFor="onLeaveUntil">On Leave Until</Label>
+                  {form.onLeaveUntil && (
+                    <Button
+                      type="button"
+                      variant="ghost"
+                      size="sm"
+                      onClick={() => setForm((prev) => ({ ...prev, onLeaveUntil: '' }))}
+                    >
+                      Clear
+                    </Button>
+                  )}
+                </div>
                 <Input
                   id="onLeaveUntil"
                   type="date"
@@ -823,7 +843,9 @@ export default function DriverDetailsPage() {
                   {driver.currentLocation.address || 'Location updating...'}
                 </p>
                 <p className="text-xs text-text-secondary mt-1">
-                  Updated {formatDate(driver.currentLocation.timestamp || new Date(), 'relative')}
+                  {driver.currentLocation.timestamp
+                    ? `Updated ${formatDate(driver.currentLocation.timestamp, 'relative')}`
+                    : 'Update time not recorded'}
                 </p>
               </CardContent>
             </Card>