test(work-orders): cover completed date board read mapping

[recover] remove malicious eslint payload (was 0548bbb4)

e2e/vendors/vendors.spec.ts

@@ -306,18 +306,11 @@ test.describe("Vendor directory prototype parity", () => {
   test("covers status tabs, global counts, search, facets, table fields, and filtered footer", async ({
     page,
   }) => {
-    await page.setViewportSize({ width: 1440, height: 1000 });
     const state = await mockVendorApi(page);
     await page.goto("/vendors");
 
-    const heading = page.getByRole("heading", { name: "Vendors / Technicians" });
+    await expect(page.getByRole("heading", { name: "Vendors / Technicians" })).toBeVisible();
-    await expect(heading).toBeVisible();
+    await expect(page.getByText("3 vendor entries · 2 active")).toBeVisible();
-    await expect(heading).toHaveCSS("font-family", /Montserrat/);
-    await expect(heading).toHaveCSS("font-size", "30px");
-    await expect(heading).toHaveCSS("font-weight", "800");
-    await expect(heading).toHaveCSS("line-height", "33px");
-    await expect(heading).toHaveCSS("letter-spacing", "-0.6px");
-    await expect(page.getByText("3 technicians · 2 active")).toBeVisible();
     await expect(page.getByRole("tab", { name: "Active (2)" })).toHaveAttribute(
       "aria-selected",
       "true",
@@ -330,58 +323,52 @@ test.describe("Vendor directory prototype parity", () => {
       "Phone",
       "Email",
       "Total Jobs",
+      "Status",
     ]) {
       await expect(page.getByRole("columnheader", { name: heading })).toBeVisible();
     }
-    await expect(page.getByRole("columnheader", { name: "Status" })).toBeVisible();
-    const activeTabBox = await page.getByRole("tab", { name: "Active (2)" }).boundingBox();
-    expect(activeTabBox).toMatchObject({ x: 256, y: 159.5, height: 36 });
-    const tableContainerBox = await page.locator("table").locator("..").boundingBox();
-    expect(tableContainerBox).toMatchObject({ x: 260, y: 212.5, width: 1164 });
-    expect(tableContainerBox?.height).toBeCloseTo(738, 0);
-    expect((await page.locator("table").boundingBox())?.width).toBe(1550);
-    expect((await page.getByRole("columnheader", { name: "Company" }).boundingBox())?.height).toBe(
-      47,
-    );
-    expect((await page.locator("tbody tr").first().boundingBox())?.height).toBe(52);
     await expect(page.getByText("Gateway Plumbing")).toBeVisible();
     await expect(page.getByText("+1")).toBeVisible();
-    await expect(page.getByText("2 active technicians")).toBeVisible();
+    await expect(page.getByText("2 active vendor entries · 2 active · 1 inactive")).toBeVisible();
-    await expect(page.getByText("2 active · 1 inactive")).toBeVisible();
     await captureVisual(page, "/tmp/seahaven-vendor-implemented-list.png");
 
-    await page.getByPlaceholder("Search name, company, trade…").fill("metro");
+    await page.getByPlaceholder("Search name, company, trade...").fill("metro");
     await expect(page.getByText("Metro HVAC")).toBeVisible();
-    await expect(page.getByText(/1 active technician · filtered/)).toBeVisible();
+    await expect(page.getByText(/1 active vendor entry · filtered/)).toBeVisible();
     expect(state.listUrls.some((url) => new URL(url).searchParams.get("search") === "metro")).toBe(
       true,
     );
 
-    await page.getByPlaceholder("Search name, company, trade…").fill("clayton");
+    await page.getByPlaceholder("Search name, company, trade...").fill("clayton");
     await expect(page.getByText("Metro HVAC")).toBeVisible();
 
-    await page.getByPlaceholder("Search name, company, trade…").clear();
+    await page.getByPlaceholder("Search name, company, trade...").clear();
     await page.getByRole("button", { name: "Open vendor filters" }).click();
     await expect(page.getByRole("heading", { name: "Filters" })).toBeVisible();
     await captureVisual(page, "/tmp/seahaven-vendor-implemented-filters.png");
     await page.getByRole("button", { name: /Company/ }).click();
     await page.getByPlaceholder("All companies").click();
     await page.getByRole("option", { name: /Gateway Plumbing/ }).click();
-    await page.getByRole("button", { name: "Apply filters" }).click();
+    await page.getByRole("button", { name: "Apply" }).click();
     await expect(page.getByText("Gateway Plumbing").first()).toBeVisible();
-    await expect(page.getByText("1 technician · filtered by")).toBeVisible();
+    await expect(page.getByText(/1 active vendor entry · filtered/)).toBeVisible();
-    await expect(page.getByText(/1 active technician · filtered/)).toBeVisible();
     expect(
       state.listUrls.some(
         (url) => new URL(url).searchParams.get("companies[0]") === "Gateway Plumbing",
       ),
     ).toBe(true);
 
+    await page.getByRole("button", { name: "Deactivate vendor" }).first().click();
+    await expect(page.getByRole("dialog", { name: "Deactivate Vendor" })).toContainText(
+      "history will be preserved",
+    );
+    await page.getByRole("button", { name: "Deactivate", exact: true }).click();
+    await expect.poll(() => state.deletedId).toBe("1");
+
     await page.getByRole("tab", { name: "Inactive (1)" }).click();
-    await expect(page.getByText("No inactive technicians found")).toBeVisible();
+    await expect(page.getByText("No vendors match the current filters.")).toBeVisible();
     await page.getByRole("button", { name: "Clear all" }).click();
     await expect(page.getByText("Drain Cleaning Bros")).toBeVisible();
-    await expect(page.getByRole("columnheader", { name: "Status" })).toBeVisible();
     await expect(page.getByRole("button", { name: "Deactivate vendor" })).toHaveCount(0);
   });
 
@@ -391,35 +378,33 @@ test.describe("Vendor directory prototype parity", () => {
     const state = await mockVendorApi(page);
     await page.goto("/vendors");
 
-    await page.getByRole("button", { name: "Add Vendor", exact: true }).click();
+    await page.getByRole("button", { name: "+ Add Vendor" }).click();
-    await expect(page.getByText("Register a vendor company and its technicians.")).toBeVisible();
-    await captureVisual(page, "/tmp/seahaven-vendor-implemented-add.png");
     await expect(
-      page.getByRole("button", { name: "Add Vendor", exact: true }).last(),
+      page.getByText("Register a company and add zero, one, or multiple technicians."),
-    ).toBeEnabled();
+    ).toBeVisible();
+    await captureVisual(page, "/tmp/seahaven-vendor-implemented-add.png");
+    await expect(page.getByRole("button", { name: "Add Vendor Company" }).last()).toBeDisabled();
 
     await page.getByRole("combobox", { name: "Company" }).click();
     await page.getByRole("option", { name: "Gateway Plumbing" }).click();
-    await expect(page.getByLabel("Company Phone (optional)")).toHaveValue("314-555-0100");
+    await expect(page.getByLabel("Company Phone")).toHaveValue("314-555-0100");
-    await expect(page.getByRole("textbox", { name: "Email (optional)", exact: true })).toHaveValue(
+    await expect(page.getByRole("textbox", { name: "Email", exact: true })).toHaveValue(
       "dispatch@gateway.test",
     );
-    await expect(page.getByLabel("Address (optional)")).toHaveValue("1 Market St");
+    await expect(page.getByLabel("Address")).toHaveValue("1 Market St");
-    await expect(page.getByLabel("Google Maps URL (optional)")).toHaveValue(
+    await expect(page.getByLabel("Google Maps URL")).toHaveValue("https://maps.google.com/gateway");
-      "https://maps.google.com/gateway",
-    );
     await expect(page.getByLabel("Preferred Contact")).toHaveCount(0);
     await page.getByRole("button", { name: "Add technician" }).click();
-    await page.getByLabel("Technician name (optional)").last().fill("New Technician");
+    await page.getByLabel("Technician Name").last().fill("New Technician");
     const tradeInput = page.getByRole("combobox", { name: "Add Trade" }).last();
     await tradeInput.fill("HVAC");
     await tradeInput.press("Enter");
     await tradeInput.fill("Plumbing");
     await tradeInput.press("Enter");
     await page.getByRole("button", { name: "Move Plumbing up" }).click();
-    await page.getByLabel("Notes (optional)").fill("Created in browser E2E");
+    await page.getByLabel("Notes").fill("Created in browser E2E");
-    await page.getByRole("button", { name: "Add Vendor", exact: true }).last().click();
+    await page.getByRole("button", { name: "Add Vendor Company" }).last().click();
-    await expect(page.getByRole("dialog", { name: /Add Vendor/ })).toHaveCount(0);
+    await expect(page.getByRole("dialog", { name: "Add Vendor Company" })).toHaveCount(0);
     expect(state.updatedBody).toMatchObject({
       name: "Gateway Plumbing",
       companyPhone: "(314) 555-0100",
@@ -444,7 +429,7 @@ test.describe("Vendor directory prototype parity", () => {
 
     await page.getByRole("button", { name: "View vendor Gateway Plumbing" }).click();
     const detailDrawer = page.locator(".MuiDrawer-paper").last();
-    await expect(detailDrawer.getByRole("heading", { name: "Adam Whyte" })).toBeVisible();
+    await expect(detailDrawer.getByRole("heading", { name: "Gateway Plumbing" })).toBeVisible();
     await expect(detailDrawer.getByText("Adam Whyte")).toBeVisible();
     await expect(detailDrawer.getByText("72")).toBeVisible();
     await expect(
@@ -453,9 +438,9 @@ test.describe("Vendor directory prototype parity", () => {
     await captureVisual(page, "/tmp/seahaven-vendor-implemented-detail.png");
 
     await detailDrawer.getByRole("button", { name: "Edit" }).click();
-    await expect(page).toHaveURL(/\/vendors$/);
+    await expect(page).toHaveURL(/\/vendors\/edit\/1$/);
-    await detailDrawer.getByLabel("Notes (optional)").fill("Updated in browser E2E");
+    await page.getByLabel("Notes").fill("Updated in browser E2E");
-    await detailDrawer.getByRole("button", { name: "Save changes" }).click();
+    await page.getByRole("button", { name: "Save" }).click();
     await expect.poll(() => state.updatedBody?.notes).toBe("Updated in browser E2E");
   });
 
@@ -463,18 +448,18 @@ test.describe("Vendor directory prototype parity", () => {
     const state = await mockVendorApi(page, { legacyGateway: true });
     await page.goto("/vendors");
 
-    await page.getByRole("button", { name: "Edit vendor Gateway Plumbing" }).click();
+    await page.getByRole("button", { name: "Deactivate vendor" }).first().click();
-    await page.getByRole("switch", { name: "Active status" }).click();
     await page.getByRole("button", { name: "Deactivate", exact: true }).click();
     await expect.poll(() => state.deletedId).toBe("1");
 
     await page.getByRole("tab", { name: "Inactive (2)" }).click();
-    await page.getByRole("button", { name: "Edit vendor Gateway Plumbing" }).click();
+    await page.getByRole("button", { name: "View vendor Gateway Plumbing" }).click();
     const detailDrawer = page.locator(".MuiDrawer-paper").last();
-    await detailDrawer.getByRole("switch", { name: "Active status" }).click();
+    await detailDrawer.getByRole("button", { name: "Edit" }).click();
-    await detailDrawer.getByRole("button", { name: "Save changes" }).click();
+    await expect(page).toHaveURL(/\/vendors\/edit\/1$/);
+    await page.getByRole("switch", { name: "Inactive" }).first().click();
+    await page.getByRole("button", { name: "Save" }).click();
     await expect.poll(() => state.updatedBody).toEqual({ isActive: true });
-    await detailDrawer.getByRole("button", { name: "Close drawer" }).click();
     await page.getByRole("tab", { name: "Active (2)", exact: true }).click();
     await expect(page.getByText("Gateway Plumbing").first()).toBeVisible();
   });
@@ -483,7 +468,7 @@ test.describe("Vendor directory prototype parity", () => {
     const state = await mockVendorApi(page);
     await page.goto("/vendors");
 
-    const addButton = page.getByRole("button", { name: "Add Vendor", exact: true });
+    const addButton = page.getByRole("button", { name: "+ Add Vendor" });
     await addButton.focus();
     await page.keyboard.press("Enter");
     await expect(page.getByRole("dialog", { name: /Add Vendor/ })).toBeVisible();
@@ -492,12 +477,10 @@ test.describe("Vendor directory prototype parity", () => {
     await expect(addButton).toBeFocused();
 
     await addButton.click();
-    await expect(
+    await expect(page.getByRole("button", { name: "Add Vendor Company" }).last()).toBeDisabled();
-      page.getByRole("button", { name: "Add Vendor", exact: true }).last(),
-    ).toBeEnabled();
     await page.getByRole("combobox", { name: "Company" }).fill("Independent Vendor LLC");
-    await page.getByLabel("Company Phone (optional)").fill("3145550199");
+    await page.getByLabel("Company Phone").fill("3145550199");
-    await page.getByRole("button", { name: "Add Vendor", exact: true }).last().click();
+    await page.getByRole("button", { name: "Add Vendor Company" }).last().click();
 
     await expect.poll(() => state.createdBody?.name).toBe("Independent Vendor LLC");
     expect(state.createdBody?.technicians).toEqual([]);
@@ -523,8 +506,7 @@ test.describe("Vendor directory prototype parity", () => {
     const blockedState = await mockVendorApi(page, { deactivationBlocked: true });
     await page.goto("/vendors");
 
-    await page.getByRole("button", { name: "Edit vendor Gateway Plumbing" }).click();
+    await page.getByRole("button", { name: "Deactivate vendor" }).first().click();
-    await page.getByRole("switch", { name: "Active status" }).click();
     const blockedDialog = page.getByRole("dialog", { name: "Deactivate Vendor" });
     await expect(blockedDialog).toContainText("WO-501 — Emergency boiler repair");
     await expect(blockedDialog.getByRole("button", { name: /^Deactivate$/ })).toBeDisabled();
@@ -535,8 +517,7 @@ test.describe("Vendor directory prototype parity", () => {
 
     const racedState = await mockVendorApi(page, { deleteConflict: true });
     await page.reload();
-    await page.getByRole("button", { name: "Edit vendor Gateway Plumbing" }).click();
+    await page.getByRole("button", { name: "Deactivate vendor" }).first().click();
-    await page.getByRole("switch", { name: "Active status" }).click();
     await page
       .getByRole("dialog", { name: "Deactivate Vendor" })
       .getByRole("button", {
@@ -560,7 +541,6 @@ test.describe("Vendor directory prototype parity", () => {
       /Vendor directory unavailable|failed|server error/i,
       { timeout: 15_000 },
     );
-    await captureVisual(page, "/tmp/seahaven-vendor-implemented-mobile-list.png");
     await page.getByRole("button", { name: "Open vendor filters" }).click();
     const filterDrawer = page.locator(".MuiDrawer-paper").filter({ hasText: "Total Jobs" });
     await expect(filterDrawer).toBeVisible();

src/app/(protected)/vendors/_components/use-vendor-deactivation.ts

@@ -15,7 +15,7 @@ export interface VendorDeactivationState {
   confirm: () => void;
 }
 
-export function useVendorDeactivation(onSuccess?: () => void): VendorDeactivationState {
+export function useVendorDeactivation(): VendorDeactivationState {
   const [target, setTarget] = useState<VendorListItem | null>(null);
   const [error, setError] = useState<string | null>(null);
   const deleteVendor = useDeleteVendor();
@@ -39,10 +39,7 @@ export function useVendorDeactivation(onSuccess?: () => void): VendorDeactivatio
     if (!target || target.id == null) return;
     setError(null);
     deleteVendor.mutate(target.id, {
-      onSuccess: () => {
+      onSuccess: () => setTarget(null),
-        setTarget(null);
-        onSuccess?.();
-      },
       onError: (err: Error) => setError(err.message || "Failed to deactivate vendor"),
     });
   };

src/app/(protected)/vendors/_components/use-vendor-list-state.ts

@@ -31,7 +31,7 @@ export interface VendorListState {
 export function useVendorListState(): VendorListState {
   const [search, setSearch] = useState("");
   const [page, setPage] = useState(0);
-  const [pageSize, setPageSize] = useState(48);
+  const [pageSize, setPageSize] = useState(12);
   const [isActive, setIsActive] = useState(true);
   const [appliedFilters, setAppliedFilters] = useState<VendorFilters>(emptyVendorFilters);
 

src/app/(protected)/vendors/_components/use-vendor-roster-form.ts

@@ -2,7 +2,6 @@ import { useCallback, useMemo, useState } from "react";
 import { useForm, useWatch, type FieldErrors } from "react-hook-form";
 import {
   emptyVendorCompanyRosterForm,
-  emptyRosterTechnician,
   vendorCompanyRosterSchema,
   type VendorCompanyRosterFormValues,
 } from "@/domain/vendors/schemas/vendor-roster-schema";
@@ -21,30 +20,11 @@ import type {
   VendorRosterConflict,
 } from "@/domain/vendors/types/vendor";
 
-function withoutBlankNewTechnicians(
-  values: VendorCompanyRosterFormValues,
-): VendorCompanyRosterFormValues {
-  return {
-    ...values,
-    technicians: values.technicians.filter(
-      (technician) =>
-        technician.id != null ||
-        Boolean(
-          technician.contactName.trim() ||
-          technician.phone.trim() ||
-          technician.email.trim() ||
-          technician.tradeSpecialties.trim(),
-        ),
-    ),
-  };
-}
-
 export interface VendorRosterFormProps {
   mode: "create" | "update";
   vendorId?: string | number;
   companyId?: string | number;
   onSuccess?: (roster: VendorCompanyRoster) => void;
-  startWithTechnician?: boolean;
 }
 
 export interface VendorRosterForm {
@@ -75,7 +55,6 @@ export function useVendorRosterForm({
   vendorId,
   companyId,
   onSuccess,
-  startWithTechnician = false,
 }: VendorRosterFormProps): VendorRosterForm {
   const query = useVendorCompanyRoster(
     mode === "update" ? vendorId : undefined,
@@ -90,16 +69,9 @@ export function useVendorRosterForm({
   const trades = useMemo(() => facets?.trades ?? [], [facets]);
   const resolver = useVendorRosterResolver(routeRoster);
 
-  const createDefaults = useMemo(
-    () => ({
-      ...emptyVendorCompanyRosterForm,
-      technicians: startWithTechnician ? [{ ...emptyRosterTechnician }] : [],
-    }),
-    [startWithTechnician],
-  );
   const form = useForm<VendorCompanyRosterFormValues>({
     resolver,
-    defaultValues: createDefaults,
+    defaultValues: emptyVendorCompanyRosterForm,
     values: mode === "update" && routeRoster ? toFormValues(routeRoster) : undefined,
     mode: "onChange",
   });
@@ -124,10 +96,10 @@ export function useVendorRosterForm({
   }, [routeRoster, watched]);
 
   const resetForm = useCallback(() => {
-    reset(createDefaults);
+    reset(emptyVendorCompanyRosterForm);
     resetSelection();
     clearConflict();
-  }, [clearConflict, createDefaults, reset, resetSelection]);
+  }, [clearConflict, reset, resetSelection]);
 
   const committedRoster = mode === "update" ? routeRoster : selection.selectedRoster;
   const isUpdate = mode === "update" || selection.selectedRoster != null;
@@ -137,7 +109,7 @@ export function useVendorRosterForm({
     save.mutate(
       {
         mode: isUpdate ? "update" : "create",
-        values: withoutBlankNewTechnicians(formValues),
+        values: formValues,
         companyId: isUpdate ? (committedRoster?.companyId ?? companyId) : undefined,
         rowVersion: isUpdate ? committedRoster?.rowVersion : undefined,
         originalRoster: mode === "update" ? routeRoster : undefined,

src/app/(protected)/vendors/_components/vendor-create-modal.tsx

@@ -8,8 +8,8 @@ import {
   DialogContent,
   DialogTitle,
   Stack,
+  Typography,
 } from "@mui/material";
-import { Text } from "@/components/ui/text";
 import { VendorRosterConflictAlert } from "./vendor-roster-conflict-alert";
 import { VendorRosterFormFields } from "./vendor-roster-form-fields";
 import { VendorRosterLoadErrorAlert } from "./vendor-roster-load-error";
@@ -21,11 +21,7 @@ interface VendorCreateModalProps {
 }
 
 export function VendorCreateModal({ open, onClose }: VendorCreateModalProps) {
-  const form = useVendorRosterForm({
+  const form = useVendorRosterForm({ mode: "create", onSuccess: onClose });
-    mode: "create",
-    onSuccess: onClose,
-    startWithTechnician: true,
-  });
   const { resetForm } = form;
 
   useEffect(() => {
@@ -37,24 +33,17 @@ export function VendorCreateModal({ open, onClose }: VendorCreateModalProps) {
       open={open}
       onClose={form.isSaving ? undefined : onClose}
       fullWidth
-      maxWidth="sm"
+      maxWidth="md"
       aria-labelledby="vendor-create-dialog-title"
-      slotProps={{
+      slotProps={{ paper: { role: "dialog" } }}
-        paper: {
-          role: "dialog",
-          sx: { maxHeight: "88vh", borderRadius: 2 },
-        },
-      }}
     >
       <DialogTitle id="vendor-create-dialog-title">
-        <Text variant="title" as="span">
+        Add Vendor Company
-          Add Vendor
+        <Typography variant="body2" sx={{ color: "text.secondary", mt: 0.5 }}>
-        </Text>
+          Register a company and add zero, one, or multiple technicians.
-        <Text variant="description" tone="muted" sx={{ mt: 0.5 }}>
+        </Typography>
-          Register a vendor company and its technicians.
-        </Text>
       </DialogTitle>
-      <DialogContent sx={{ maxHeight: "55vh" }}>
+      <DialogContent>
         <Stack spacing={2} className="mt-1">
           {form.loadErrorMessage !== null && (
             <VendorRosterLoadErrorAlert message={form.loadErrorMessage} onRetry={form.retryLoad} />
@@ -84,9 +73,9 @@ export function VendorCreateModal({ open, onClose }: VendorCreateModalProps) {
           variant="contained"
           type="submit"
           form="vendor-create-form"
-          disabled={form.isSaving}
+          disabled={form.isSaving || !form.isValid}
         >
-          {form.isSaving ? <CircularProgress size={20} color="inherit" /> : "Add Vendor"}
+          {form.isSaving ? <CircularProgress size={20} color="inherit" /> : "Add Vendor Company"}
         </Button>
       </DialogActions>
     </Dialog>

src/app/(protected)/vendors/_components/vendor-detail-drawer.tsx

@@ -1,5 +1,5 @@
-import { useEffect, useState, type ReactNode } from "react";
+import { useEffect, type ReactNode } from "react";
-import { Controller, useWatch } from "react-hook-form";
+import { useNavigate } from "react-router";
 import CloseIcon from "@mui/icons-material/Close";
 import EditOutlinedIcon from "@mui/icons-material/EditOutlined";
 import LaunchIcon from "@mui/icons-material/Launch";
@@ -13,22 +13,15 @@ import {
   Drawer,
   IconButton,
   Link,
+  List,
+  ListItem,
   Stack,
-  Switch,
+  Typography,
 } from "@mui/material";
-import { VendorRosterConflictAlert } from "./vendor-roster-conflict-alert";
-import { VendorRosterFormFields } from "./vendor-roster-form-fields";
-import { VendorRosterLoadErrorAlert } from "./vendor-roster-load-error";
-import { useVendorRosterForm } from "./use-vendor-roster-form";
-import { Text } from "@/components/ui/text";
-import type {
-  VendorCompanyRoster,
-  VendorListItem,
-  VendorRosterTechnician,
-} from "@/domain/vendors/types/vendor";
 import { useVendorCompanyRoster } from "@/domain/vendors/use-cases/use-vendor-company-roster";
+import type { VendorCompanyRoster, VendorListItem } from "@/domain/vendors/types/vendor";
 
-const DRAWER_WIDTH = 420;
+const DRAWER_WIDTH = 440;
 
 function splitTrades(value: string | undefined): string[] {
   return (value ?? "")
@@ -40,45 +33,106 @@ function splitTrades(value: string | undefined): string[] {
 interface VendorDetailDrawerProps {
   open: boolean;
   vendor: VendorListItem | null;
-  startInEdit?: boolean;
-  onRequestDeactivation?: (vendor: VendorListItem) => void;
   onClose: () => void;
 }
 
 function SectionTitle({ children }: { children: ReactNode }) {
   return (
-    <Text
+    <Typography
-      variant="label"
+      variant="subtitle2"
-      tone="muted"
+      className="font-semibold uppercase"
-      className="font-semibold tracking-[0.06em] uppercase"
+      sx={{ color: "text.secondary", mb: 1 }}
-      sx={{ mb: 1 }}
     >
       {children}
-    </Text>
+    </Typography>
   );
 }
 
-function Field({ label, value }: { label: string; value?: string }) {
+function Field({ label, value }: { label: string; value: string }) {
-  const display = !value || value.trim() === "" ? "—" : value;
+  const display = value && value.trim() !== "" ? value : "—";
   return (
     <Box>
-      <Text variant="caption" tone="muted" sx={{ display: "block" }}>
+      <Typography variant="caption" sx={{ color: "text.secondary", display: "block" }}>
         {label}
-      </Text>
+      </Typography>
-      <Text variant="description">{display}</Text>
+      <Typography variant="body2">{display}</Typography>
     </Box>
   );
 }
 
+function TechnicianEntry({
+  contactName,
+  phone,
+  email,
+  preferredContact,
+  tradeSpecialties,
+  isActive,
+  totalJobs,
+}: {
+  contactName: string;
+  phone: string;
+  email: string;
+  preferredContact?: string;
+  tradeSpecialties: string;
+  isActive: boolean;
+  totalJobs: number;
+}) {
+  const trades = splitTrades(tradeSpecialties);
+  return (
+    <ListItem disableGutters divider>
+      <Stack spacing={0.5} className="w-full">
+        <Stack direction="row" spacing={1} sx={{ alignItems: "center" }}>
+          <Typography variant="body2" className="font-semibold">
+            {contactName || "—"}
+          </Typography>
+          <Chip
+            size="small"
+            label={isActive ? "Active" : "Inactive"}
+            color={isActive ? "success" : "default"}
+          />
+          <Typography variant="caption" sx={{ color: "text.secondary" }}>
+            {totalJobs} jobs
+          </Typography>
+        </Stack>
+        <Stack direction="row" spacing={2} className="flex flex-wrap">
+          {Boolean(phone) && (
+            <Link href={`tel:${phone}`} underline="hover">
+              {phone}
+            </Link>
+          )}
+          {Boolean(email) && (
+            <Link href={`mailto:${email}`} underline="hover">
+              {email}
+            </Link>
+          )}
+        </Stack>
+        <Stack direction="row" spacing={1} sx={{ alignItems: "center", flexWrap: "wrap" }}>
+          {Boolean(preferredContact) && (
+            <Typography variant="caption" sx={{ color: "text.secondary" }}>
+              Prefers {preferredContact}
+            </Typography>
+          )}
+          {trades.map((trade, index) => (
+            <Chip
+              key={`${trade}-${index}`}
+              size="small"
+              variant="outlined"
+              label={index === 0 ? `${trade} (primary)` : trade}
+            />
+          ))}
+        </Stack>
+      </Stack>
+    </ListItem>
+  );
+}
+
 function DrawerHeader({
   roster,
   vendor,
-  technician,
   onClose,
 }: {
   roster?: VendorCompanyRoster;
   vendor: VendorListItem | null;
-  technician?: VendorRosterTechnician;
   onClose: () => void;
 }) {
   return (
@@ -87,18 +141,23 @@ function DrawerHeader({
       sx={{
         alignItems: "center",
         justifyContent: "space-between",
-        px: 2.5,
+        px: 3,
         py: 2,
         borderBottom: (theme) => `1px solid ${theme.palette.divider}`,
       }}
     >
       <Box>
-        <Text id="vendor-detail-drawer-title" variant="title" as="h2" className="font-semibold">
+        <Typography
-          {technician?.contactName || vendor?.contactName || roster?.name || "Vendor company"}
+          id="vendor-detail-drawer-title"
-        </Text>
+          variant="subtitle1"
-        <Text variant="description" tone="muted">
+          component="h2"
-          {roster?.name || vendor?.companyName || "Vendor details"}
+          className="font-semibold"
-        </Text>
+        >
+          {roster?.name || vendor?.companyName || "Vendor company"}
+        </Typography>
+        <Typography variant="caption" sx={{ color: "text.secondary" }}>
+          {roster ? `${roster.technicians.length} technician(s)` : "Vendor details"}
+        </Typography>
       </Box>
       <IconButton aria-label="Close drawer" onClick={onClose} size="small">
         <CloseIcon fontSize="small" />
@@ -108,19 +167,18 @@ function DrawerHeader({
 }
 
 function CompanySection({ roster }: { roster: VendorCompanyRoster }) {
-  const address = [roster.address, roster.city, roster.state, roster.zip]
+  const mapsUrl = roster.googleMapsUrl;
-    .filter(Boolean)
-    .join(", ");
-
   return (
     <Stack spacing={1.5}>
-      <Field label="Company" value={roster.name} />
       <Field label="Company Phone" value={roster.companyPhone} />
-      <Field label="Company Email" value={roster.email} />
+      <Field label="Email" value={roster.email} />
-      <Field label="Address" value={address} />
+      <Field
-      {Boolean(roster.googleMapsUrl) && (
+        label="Address"
+        value={[roster.address, roster.city, roster.state, roster.zip].filter(Boolean).join(", ")}
+      />
+      {Boolean(mapsUrl) && (
         <Link
-          href={roster.googleMapsUrl}
+          href={mapsUrl}
           target="_blank"
           rel="noopener noreferrer"
           className="inline-flex items-center gap-1"
@@ -133,79 +191,49 @@ function CompanySection({ roster }: { roster: VendorCompanyRoster }) {
   );
 }
 
-function TechnicianSection({ technician }: { technician: VendorRosterTechnician }) {
+function TechniciansSection({ roster }: { roster: VendorCompanyRoster }) {
-  const trades = splitTrades(technician.tradeSpecialties);
-
   return (
     <Box>
-      <Stack spacing={1.5}>
+      <SectionTitle>Technicians</SectionTitle>
-        <Field label="Phone" value={technician.phone} />
+      {roster.technicians.length === 0 ? (
-        <Box>
+        <Typography variant="body2" sx={{ color: "text.secondary" }}>
-          <Text variant="caption" tone="muted" sx={{ display: "block", mb: 0.75 }}>
+          No technicians assigned to this company.
-            Trade
+        </Typography>
-          </Text>
+      ) : (
-          <Stack direction="row" spacing={0.75} sx={{ flexWrap: "wrap" }}>
+        <List disablePadding>
-            {trades.length > 0 ? (
+          {roster.technicians.map((technician) => (
-              trades.map((trade, index) => (
+            <TechnicianEntry
-                <Chip key={`${trade}-${index}`} size="small" variant="outlined" label={trade} />
+              key={String(technician.id ?? technician.contactName)}
-              ))
+              contactName={technician.contactName}
-            ) : (
+              phone={technician.phone}
-              <Text variant="description">—</Text>
+              email={technician.email}
-            )}
+              preferredContact={technician.preferredContact}
-          </Stack>
+              tradeSpecialties={technician.tradeSpecialties}
-        </Box>
+              isActive={technician.isActive}
-        <Stack direction="row" spacing={4} sx={{ alignItems: "flex-end" }}>
+              totalJobs={technician.totalJobs}
-          <Box>
-            <Text variant="title">{technician.totalJobs ?? 0}</Text>
-            <Text variant="caption" tone="muted" sx={{ display: "block" }}>
-              Total Jobs
-            </Text>
-          </Box>
-          <Box>
-            <Text variant="caption" tone="muted" sx={{ display: "block", mb: 0.5 }}>
-              Status
-            </Text>
-            <Chip
-              size="small"
-              label={technician.isActive ? "Active" : "Inactive"}
-              color={technician.isActive ? "success" : "default"}
             />
-          </Box>
+          ))}
-        </Stack>
+        </List>
-      </Stack>
+      )}
     </Box>
   );
 }
 
-function DrawerBody({
+function DrawerBody({ roster }: { roster: VendorCompanyRoster }) {
-  roster,
-  technician,
-}: {
-  roster: VendorCompanyRoster;
-  technician?: VendorRosterTechnician;
-}) {
   return (
-    <Stack spacing={3} className="flex-1 overflow-y-auto" sx={{ p: 2.5 }}>
+    <Stack spacing={3} className="flex-1 overflow-y-auto" sx={{ px: 3, py: 3 }}>
-      <Box>
+      <SectionTitle>Company</SectionTitle>
-        <SectionTitle>Company</SectionTitle>
+      <CompanySection roster={roster} />
-        <CompanySection roster={roster} />
-      </Box>
       <Divider />
-      {technician ? (
+      <TechniciansSection roster={roster} />
-        <TechnicianSection technician={technician} />
-      ) : (
-        <Text variant="description" tone="muted">
-          No technicians assigned to this company.
-        </Text>
-      )}
       {Boolean(roster.notes) && (
         <>
           <Divider />
           <Box>
             <SectionTitle>Notes</SectionTitle>
-            <Text variant="description" sx={{ whiteSpace: "pre-wrap" }}>
+            <Typography variant="body2" sx={{ whiteSpace: "pre-wrap" }}>
               {roster.notes}
-            </Text>
+            </Typography>
           </Box>
         </>
       )}
@@ -218,8 +246,8 @@ function DrawerActions({ onEdit }: { onEdit: () => void }) {
     <Stack
       direction="row"
       sx={{
-        px: 2.5,
+        px: 3,
-        py: 1.5,
+        py: 2,
         borderTop: (theme) => `1px solid ${theme.palette.divider}`,
         alignItems: "center",
         justifyContent: "flex-end",
@@ -232,143 +260,8 @@ function DrawerActions({ onEdit }: { onEdit: () => void }) {
   );
 }
 
-function DrawerEditor({
+export function VendorDetailDrawer({ open, vendor, onClose }: VendorDetailDrawerProps) {
-  vendor,
+  const navigate = useNavigate();
-  onClose,
-  onSaved,
-  onRequestDeactivation = () => undefined,
-}: {
-  vendor: VendorListItem;
-  onClose: () => void;
-  onSaved: () => void;
-  onRequestDeactivation: (vendor: VendorListItem) => void;
-}) {
-  const form = useVendorRosterForm({
-    mode: "update",
-    vendorId: vendor.id ?? undefined,
-    companyId: vendor.id == null ? (vendor.companyId ?? undefined) : undefined,
-    onSuccess: onSaved,
-  });
-  const roster = form.roster;
-  const technicians = useWatch({ control: form.control, name: "technicians" }) ?? [];
-  const selectedIndex = technicians.findIndex(
-    (technician) => technician.id != null && String(technician.id) === String(vendor.id),
-  );
-  const selectedTotalJobs =
-    roster?.technicians.find(
-      (technician) => technician.id != null && String(technician.id) === String(vendor.id),
-    )?.totalJobs ?? vendor.totalJobs;
-
-  if (form.isLoading) {
-    return <CircularProgress size={28} sx={{ m: "auto" }} />;
-  }
-  if (form.isError || !roster) {
-    return (
-      <Alert severity="error" sx={{ m: 2 }}>
-        Failed to load vendor company. Try again.
-      </Alert>
-    );
-  }
-
-  return (
-    <Box
-      component="form"
-      id="vendor-drawer-edit-form"
-      onSubmit={form.handleSubmit(form.submit)}
-      className="flex min-h-0 flex-1 flex-col"
-    >
-      <Stack spacing={2} className="flex-1 overflow-y-auto" sx={{ p: 2.5 }}>
-        {form.loadErrorMessage !== null && (
-          <VendorRosterLoadErrorAlert message={form.loadErrorMessage} onRetry={form.retryLoad} />
-        )}
-        {form.conflict !== null && (
-          <VendorRosterConflictAlert conflict={form.conflict} onReload={form.reload} />
-        )}
-        <VendorRosterFormFields
-          control={form.control}
-          errors={form.errors}
-          tradeOptions={form.trades}
-          showTechnicianStatus={false}
-        />
-        {selectedIndex >= 0 && (
-          <>
-            <Divider />
-            <Stack direction="row" sx={{ alignItems: "center", justifyContent: "space-between" }}>
-              <Box>
-                <Text variant="title">{selectedTotalJobs}</Text>
-                <Text variant="caption" tone="muted">
-                  Total Jobs
-                </Text>
-              </Box>
-              <Controller
-                control={form.control}
-                name={`technicians.${selectedIndex}.isActive`}
-                render={({ field }) => (
-                  <Stack direction="row" spacing={1} sx={{ alignItems: "center" }}>
-                    <Text variant="description">{field.value ? "Active" : "Inactive"}</Text>
-                    <Switch
-                      checked={Boolean(field.value)}
-                      slotProps={{ input: { "aria-label": "Active status" } }}
-                      onChange={(_event, checked) => {
-                        if (field.value && !checked) onRequestDeactivation(vendor);
-                        else field.onChange(checked);
-                      }}
-                    />
-                  </Stack>
-                )}
-              />
-            </Stack>
-          </>
-        )}
-      </Stack>
-      <Stack
-        direction="row"
-        spacing={1.5}
-        sx={{ p: 2, borderTop: 1, borderColor: "divider", justifyContent: "flex-end" }}
-      >
-        <Button onClick={onClose} disabled={form.isSaving}>
-          Cancel
-        </Button>
-        <Button type="submit" variant="contained" disabled={form.isSaving || !form.isValid}>
-          {form.isSaving ? "Saving..." : "Save changes"}
-        </Button>
-      </Stack>
-    </Box>
-  );
-}
-
-function resolveTechnician(
-  roster: VendorCompanyRoster | undefined,
-  vendor: VendorListItem | null,
-): VendorRosterTechnician | undefined {
-  if (!roster || !vendor) return undefined;
-
-  const match = roster.technicians.find(
-    (technician) =>
-      vendor.id != null && technician.id != null && String(technician.id) === String(vendor.id),
-  );
-  if (match) return match;
-  if (!vendor.contactName) return undefined;
-
-  return {
-    id: vendor.id ?? undefined,
-    contactName: vendor.contactName,
-    phone: vendor.phone,
-    email: vendor.email,
-    tradeSpecialties: vendor.tradeSpecialties,
-    isActive: vendor.isActive,
-    totalJobs: vendor.totalJobs,
-  };
-}
-
-export function VendorDetailDrawer({
-  open,
-  vendor,
-  startInEdit = false,
-  onRequestDeactivation = () => undefined,
-  onClose,
-}: VendorDetailDrawerProps) {
-  const [isEditing, setIsEditing] = useState(startInEdit);
   const vendorId = vendor?.id;
   const companyId = vendor?.companyId;
   const hasVendorId = vendorId != null;
@@ -381,30 +274,25 @@ export function VendorDetailDrawer({
     !hasVendorId ? (companyId ?? undefined) : undefined,
   );
 
-  const selectedTechnician = resolveTechnician(roster, vendor);
+  const editHref = hasVendorId
-
+    ? `/vendors/edit/${vendorId}`
-  useEffect(() => {
+    : `/vendors/company/${companyId ?? ""}`;
-    if (open) setIsEditing(startInEdit);
-  }, [open, startInEdit, vendor]);
 
   useEffect(() => {
     if (!open) return;
 
     const closeOnEscape = (event: KeyboardEvent) => {
-      if (event.key === "Escape" && !isEditing) onClose();
+      if (event.key === "Escape") onClose();
     };
 
     window.addEventListener("keydown", closeOnEscape);
     return () => window.removeEventListener("keydown", closeOnEscape);
-  }, [isEditing, onClose, open]);
+  }, [onClose, open]);
 
   return (
     <Drawer
       open={open}
-      onClose={(_event, reason) => {
+      onClose={onClose}
-        if (reason === "escapeKeyDown" && isEditing) return;
-        onClose();
-      }}
       anchor="right"
       aria-labelledby="vendor-detail-drawer-title"
       ModalProps={{ keepMounted: false }}
@@ -416,21 +304,9 @@ export function VendorDetailDrawer({
       }}
     >
       <Stack className="flex h-full flex-col">
-        <DrawerHeader
+        <DrawerHeader roster={roster} vendor={vendor} onClose={onClose} />
-          roster={roster}
-          vendor={vendor}
-          technician={selectedTechnician}
-          onClose={onClose}
-        />
 
-        {isEditing && vendor ? (
+        {isLoading ? (
-          <DrawerEditor
-            vendor={vendor}
-            onClose={onClose}
-            onSaved={() => setIsEditing(false)}
-            onRequestDeactivation={onRequestDeactivation}
-          />
-        ) : isLoading ? (
           <Box className="flex flex-1 items-center justify-center">
             <CircularProgress size={28} />
           </Box>
@@ -440,14 +316,14 @@ export function VendorDetailDrawer({
           </Alert>
         ) : roster ? (
           <>
-            <DrawerBody roster={roster} technician={selectedTechnician} />
+            <DrawerBody roster={roster} />
-            <DrawerActions onEdit={() => setIsEditing(true)} />
+            <DrawerActions onEdit={() => navigate(editHref)} />
           </>
         ) : (
           <Box className="flex flex-1 items-center justify-center">
-            <Text variant="description" tone="muted">
+            <Typography variant="body2" sx={{ color: "text.secondary" }}>
               No vendor selected.
-            </Text>
+            </Typography>
           </Box>
         )}
       </Stack>

src/app/(protected)/vendors/_components/vendor-filter-options.tsx

@@ -1,252 +0,0 @@
-import type { Dispatch, ReactNode, SetStateAction } from "react";
-import BusinessOutlinedIcon from "@mui/icons-material/BusinessOutlined";
-import BuildOutlinedIcon from "@mui/icons-material/BuildOutlined";
-import ExpandMoreIcon from "@mui/icons-material/ExpandMore";
-import LocationOnOutlinedIcon from "@mui/icons-material/LocationOnOutlined";
-import TagOutlinedIcon from "@mui/icons-material/TagOutlined";
-import {
-  Accordion,
-  AccordionDetails,
-  AccordionSummary,
-  Autocomplete,
-  Box,
-  Button,
-  Checkbox,
-  Chip,
-  Stack,
-  TextField,
-  Typography,
-} from "@mui/material";
-import type {
-  VendorFacets,
-  VendorFacetCompany,
-  VendorFacetJobBucket,
-  VendorFacetLocation,
-} from "@/domain/vendors/types/vendor";
-import type { VendorFilters } from "./vendor-filters";
-
-type DraftSetter = Dispatch<SetStateAction<VendorFilters>>;
-
-function FilterSection({
-  title,
-  icon,
-  children,
-}: {
-  title: string;
-  icon: ReactNode;
-  children: ReactNode;
-}) {
-  return (
-    <Accordion disableGutters elevation={0} square sx={{ borderBottom: 1, borderColor: "divider" }}>
-      <AccordionSummary expandIcon={<ExpandMoreIcon />} aria-controls={`${title}-filters-content`}>
-        <Stack direction="row" spacing={1.5} sx={{ alignItems: "center" }}>
-          <Box sx={{ color: "primary.main", display: "flex" }}>{icon}</Box>
-          <Typography variant="subtitle1" className="font-semibold">
-            {title}
-          </Typography>
-        </Stack>
-      </AccordionSummary>
-      <AccordionDetails id={`${title}-filters-content`} sx={{ pt: 0, pb: 2.5 }}>
-        {children}
-      </AccordionDetails>
-    </Accordion>
-  );
-}
-
-function CompanyFilter({
-  options,
-  draft,
-  setDraft,
-}: {
-  options: VendorFacetCompany[];
-  draft: VendorFilters;
-  setDraft: DraftSetter;
-}) {
-  const allSelected = draft.companies.length === options.length && options.length > 0;
-  return (
-    <FilterSection title="Company" icon={<BusinessOutlinedIcon />}>
-      <Stack direction="row" sx={{ alignItems: "center", justifyContent: "space-between" }}>
-        <Typography variant="caption" sx={{ color: "text.secondary" }}>
-          Select vendor companies
-        </Typography>
-        <Button
-          size="small"
-          onClick={() =>
-            setDraft((current) => ({
-              ...current,
-              companies: allSelected ? [] : options.map((company) => company.name),
-            }))
-          }
-        >
-          {allSelected ? "Clear all" : "Select all"}
-        </Button>
-      </Stack>
-      <Autocomplete
-        multiple
-        size="small"
-        options={options}
-        getOptionLabel={(option) => (typeof option === "string" ? option : option.name)}
-        isOptionEqualToValue={(option, value) =>
-          typeof option === "string" || typeof value === "string"
-            ? option === value
-            : option.name === value.name
-        }
-        value={options.filter((option) => draft.companies.includes(option.name))}
-        onChange={(_event, value) => {
-          const next = (value as (string | VendorFacetCompany)[]).map((entry) =>
-            typeof entry === "string" ? entry : entry.name,
-          );
-          setDraft((current) => ({ ...current, companies: next }));
-        }}
-        renderOption={(props, option, { selected }) => {
-          const { key, ...optionProps } = props;
-          return (
-            <li key={key} {...optionProps}>
-              <Checkbox checked={selected} size="small" sx={{ mr: 1 }} />
-              {option.name}
-            </li>
-          );
-        }}
-        renderValue={(value, getItemProps) =>
-          value.map((option, index) => {
-            const label = typeof option === "string" ? option : option.name;
-            const { key, ...chipProps } = getItemProps({ index });
-            return <Chip key={key} size="small" label={label} {...chipProps} />;
-          })
-        }
-        renderInput={(params) => (
-          <TextField
-            {...params}
-            placeholder={draft.companies.length === 0 ? "All companies" : ""}
-          />
-        )}
-      />
-    </FilterSection>
-  );
-}
-
-function TradeFilter({ options, draft, setDraft }: FilterProps<string>) {
-  return (
-    <FilterSection title="Trade" icon={<BuildOutlinedIcon />}>
-      <Autocomplete
-        multiple
-        size="small"
-        options={options}
-        value={draft.trades}
-        onChange={(_event, value) =>
-          setDraft((current) => ({ ...current, trades: value as string[] }))
-        }
-        renderValue={(value, getItemProps) =>
-          value.map((option, index) => {
-            const { key, ...chipProps } = getItemProps({ index });
-            return <Chip key={key} size="small" label={option} {...chipProps} />;
-          })
-        }
-        renderInput={(params) => (
-          <TextField {...params} placeholder={draft.trades.length === 0 ? "All trades" : ""} />
-        )}
-      />
-    </FilterSection>
-  );
-}
-
-interface FilterProps<T> {
-  options: T[];
-  draft: VendorFilters;
-  setDraft: DraftSetter;
-}
-
-function LocationFilter({ options, draft, setDraft }: FilterProps<VendorFacetLocation>) {
-  return (
-    <FilterSection title="Location" icon={<LocationOnOutlinedIcon />}>
-      <Autocomplete
-        multiple
-        size="small"
-        options={options}
-        getOptionLabel={(option) => (typeof option === "string" ? option : option.label)}
-        isOptionEqualToValue={(option, value) =>
-          typeof option === "string" || typeof value === "string"
-            ? option === value
-            : option.label === value.label
-        }
-        value={options.filter((option) => draft.locations.includes(option.label))}
-        onChange={(_event, value) => {
-          const next = (value as (string | VendorFacetLocation)[]).map((entry) =>
-            typeof entry === "string" ? entry : entry.label,
-          );
-          setDraft((current) => ({ ...current, locations: next }));
-        }}
-        renderValue={(value, getItemProps) =>
-          value.map((option, index) => {
-            const label = typeof option === "string" ? option : option.label;
-            const { key, ...chipProps } = getItemProps({ index });
-            return <Chip key={key} size="small" label={label} {...chipProps} />;
-          })
-        }
-        renderInput={(params) => (
-          <TextField
-            {...params}
-            placeholder={draft.locations.length === 0 ? "All locations" : ""}
-          />
-        )}
-      />
-    </FilterSection>
-  );
-}
-
-function JobCountFilter({ options, draft, setDraft }: FilterProps<VendorFacetJobBucket>) {
-  return (
-    <FilterSection title="Total Jobs" icon={<TagOutlinedIcon />}>
-      <Autocomplete
-        multiple
-        size="small"
-        options={options}
-        getOptionLabel={(option) => (typeof option === "string" ? option : option.label)}
-        isOptionEqualToValue={(option, value) =>
-          typeof option === "string" || typeof value === "string"
-            ? option === value
-            : option.id === value.id
-        }
-        value={options.filter((option) => draft.jobBuckets.includes(option.id))}
-        onChange={(_event, value) => {
-          const next = (value as (string | VendorFacetJobBucket)[]).map((entry) =>
-            typeof entry === "string" ? entry : entry.id,
-          );
-          setDraft((current) => ({ ...current, jobBuckets: next }));
-        }}
-        renderValue={(value, getItemProps) =>
-          value.map((option, index) => {
-            const label = typeof option === "string" ? option : option.label;
-            const { key, ...chipProps } = getItemProps({ index });
-            return <Chip key={key} size="small" label={label} {...chipProps} />;
-          })
-        }
-        renderInput={(params) => (
-          <TextField
-            {...params}
-            placeholder={draft.jobBuckets.length === 0 ? "All job counts" : ""}
-          />
-        )}
-      />
-    </FilterSection>
-  );
-}
-
-export function VendorFilterOptions({
-  facets,
-  draft,
-  setDraft,
-}: {
-  facets: VendorFacets | undefined;
-  draft: VendorFilters;
-  setDraft: DraftSetter;
-}) {
-  return (
-    <>
-      <CompanyFilter options={facets?.companies ?? []} draft={draft} setDraft={setDraft} />
-      <TradeFilter options={facets?.trades ?? []} draft={draft} setDraft={setDraft} />
-      <LocationFilter options={facets?.locations ?? []} draft={draft} setDraft={setDraft} />
-      <JobCountFilter options={facets?.jobBuckets ?? []} draft={draft} setDraft={setDraft} />
-    </>
-  );
-}

src/app/(protected)/vendors/_components/vendor-filters-drawer.tsx

@@ -1,9 +1,32 @@
-import { useEffect, useState } from "react";
+import { useEffect, useMemo, useState, type ReactNode } from "react";
+import BusinessOutlinedIcon from "@mui/icons-material/BusinessOutlined";
+import BuildOutlinedIcon from "@mui/icons-material/BuildOutlined";
 import CloseIcon from "@mui/icons-material/Close";
-import { Box, Button, Chip, Drawer, IconButton, Stack, Typography } from "@mui/material";
+import ExpandMoreIcon from "@mui/icons-material/ExpandMore";
-import { VendorFilterOptions } from "./vendor-filter-options";
+import LocationOnOutlinedIcon from "@mui/icons-material/LocationOnOutlined";
+import TagOutlinedIcon from "@mui/icons-material/TagOutlined";
+import {
+  Accordion,
+  AccordionDetails,
+  AccordionSummary,
+  Autocomplete,
+  Box,
+  Button,
+  Checkbox,
+  Chip,
+  Drawer,
+  IconButton,
+  Stack,
+  TextField,
+  Typography,
+} from "@mui/material";
+import type {
+  VendorFacets,
+  VendorFacetCompany,
+  VendorFacetJobBucket,
+  VendorFacetLocation,
+} from "@/domain/vendors/types/vendor";
 import { emptyVendorFilters, type VendorFilters } from "./vendor-filters";
-import type { VendorFacets } from "@/domain/vendors/types/vendor";
 
 interface VendorFiltersDrawerProps {
   open: boolean;
@@ -15,6 +38,32 @@ interface VendorFiltersDrawerProps {
 
 const DRAWER_WIDTH = 360;
 
+function FilterSection({
+  title,
+  icon,
+  children,
+}: {
+  title: string;
+  icon: ReactNode;
+  children: ReactNode;
+}) {
+  return (
+    <Accordion disableGutters elevation={0} square sx={{ borderBottom: 1, borderColor: "divider" }}>
+      <AccordionSummary expandIcon={<ExpandMoreIcon />} aria-controls={`${title}-filters-content`}>
+        <Stack direction="row" spacing={1.5} sx={{ alignItems: "center" }}>
+          <Box sx={{ color: "primary.main", display: "flex" }}>{icon}</Box>
+          <Typography variant="subtitle1" className="font-semibold">
+            {title}
+          </Typography>
+        </Stack>
+      </AccordionSummary>
+      <AccordionDetails id={`${title}-filters-content`} sx={{ pt: 0, pb: 2.5 }}>
+        {children}
+      </AccordionDetails>
+    </Accordion>
+  );
+}
+
 export function VendorFiltersDrawer({
   open,
   onClose,
@@ -25,12 +74,23 @@ export function VendorFiltersDrawer({
   const [draft, setDraft] = useState<VendorFilters>(appliedFilters);
 
   useEffect(() => {
-    if (open) setDraft(appliedFilters);
+    if (open) {
+      setDraft(appliedFilters);
+    }
   }, [open, appliedFilters]);
 
+  const companyOptions = useMemo(() => facets?.companies ?? [], [facets]);
+  const tradeOptions = useMemo(() => facets?.trades ?? [], [facets]);
+  const locationOptions = useMemo(() => facets?.locations ?? [], [facets]);
+  const jobBucketOptions = useMemo(() => facets?.jobBuckets ?? [], [facets]);
+
   const activeCount =
     draft.companies.length + draft.trades.length + draft.locations.length + draft.jobBuckets.length;
 
+  const handleClear = () => {
+    setDraft(emptyVendorFilters);
+  };
+
   const handleApply = () => {
     onApply(draft);
     onClose();
@@ -73,7 +133,165 @@ export function VendorFiltersDrawer({
         </Stack>
 
         <Box className="flex-1 overflow-y-auto" sx={{ px: 2.5 }}>
-          <VendorFilterOptions facets={facets} draft={draft} setDraft={setDraft} />
+          <FilterSection title="Company" icon={<BusinessOutlinedIcon />}>
+            <Stack direction="row" sx={{ alignItems: "center", justifyContent: "space-between" }}>
+              <Typography variant="caption" sx={{ color: "text.secondary" }}>
+                Select vendor companies
+              </Typography>
+              <Button
+                size="small"
+                onClick={() =>
+                  setDraft((current) => ({
+                    ...current,
+                    companies:
+                      current.companies.length === companyOptions.length
+                        ? []
+                        : companyOptions.map((company) => company.name),
+                  }))
+                }
+              >
+                {draft.companies.length === companyOptions.length && companyOptions.length > 0
+                  ? "Clear all"
+                  : "Select all"}
+              </Button>
+            </Stack>
+            <Autocomplete
+              multiple
+              size="small"
+              options={companyOptions}
+              getOptionLabel={(option) => (typeof option === "string" ? option : option.name)}
+              isOptionEqualToValue={(option, value) =>
+                typeof option === "string" || typeof value === "string"
+                  ? option === value
+                  : option.name === value.name
+              }
+              value={companyOptions.filter((option) => draft.companies.includes(option.name))}
+              onChange={(_event, value) => {
+                const next = (value as (string | VendorFacetCompany)[]).map((entry) =>
+                  typeof entry === "string" ? entry : entry.name,
+                );
+                setDraft((current) => ({ ...current, companies: next }));
+              }}
+              renderOption={(props, option, { selected }) => {
+                const { key, ...optionProps } = props;
+                return (
+                  <li key={key} {...optionProps}>
+                    <Checkbox checked={selected} size="small" sx={{ mr: 1 }} />
+                    {option.name}
+                  </li>
+                );
+              }}
+              renderValue={(value, getItemProps) =>
+                value.map((option, index) => {
+                  const label = typeof option === "string" ? option : option.name;
+                  const tagProps = getItemProps({ index });
+                  const { key, ...chipProps } = tagProps;
+                  return <Chip key={key} size="small" label={label} {...chipProps} />;
+                })
+              }
+              renderInput={(params) => (
+                <TextField
+                  {...params}
+                  placeholder={draft.companies.length === 0 ? "All companies" : ""}
+                />
+              )}
+            />
+          </FilterSection>
+
+          <FilterSection title="Trade" icon={<BuildOutlinedIcon />}>
+            <Autocomplete
+              multiple
+              size="small"
+              options={tradeOptions}
+              value={draft.trades}
+              onChange={(_event, value) =>
+                setDraft((current) => ({ ...current, trades: value as string[] }))
+              }
+              renderValue={(value, getItemProps) =>
+                value.map((option, index) => {
+                  const tagProps = getItemProps({ index });
+                  const { key, ...chipProps } = tagProps;
+                  return <Chip key={key} size="small" label={option} {...chipProps} />;
+                })
+              }
+              renderInput={(params) => (
+                <TextField
+                  {...params}
+                  placeholder={draft.trades.length === 0 ? "All trades" : ""}
+                />
+              )}
+            />
+          </FilterSection>
+
+          <FilterSection title="Location" icon={<LocationOnOutlinedIcon />}>
+            <Autocomplete
+              multiple
+              size="small"
+              options={locationOptions}
+              getOptionLabel={(option) => (typeof option === "string" ? option : option.label)}
+              isOptionEqualToValue={(option, value) =>
+                typeof option === "string" || typeof value === "string"
+                  ? option === value
+                  : option.label === value.label
+              }
+              value={locationOptions.filter((option) => draft.locations.includes(option.label))}
+              onChange={(_event, value) => {
+                const next = (value as (string | VendorFacetLocation)[]).map((entry) =>
+                  typeof entry === "string" ? entry : entry.label,
+                );
+                setDraft((current) => ({ ...current, locations: next }));
+              }}
+              renderValue={(value, getItemProps) =>
+                value.map((option, index) => {
+                  const label = typeof option === "string" ? option : option.label;
+                  const tagProps = getItemProps({ index });
+                  const { key, ...chipProps } = tagProps;
+                  return <Chip key={key} size="small" label={label} {...chipProps} />;
+                })
+              }
+              renderInput={(params) => (
+                <TextField
+                  {...params}
+                  placeholder={draft.locations.length === 0 ? "All locations" : ""}
+                />
+              )}
+            />
+          </FilterSection>
+
+          <FilterSection title="Total Jobs" icon={<TagOutlinedIcon />}>
+            <Autocomplete
+              multiple
+              size="small"
+              options={jobBucketOptions}
+              getOptionLabel={(option) => (typeof option === "string" ? option : option.label)}
+              isOptionEqualToValue={(option, value) =>
+                typeof option === "string" || typeof value === "string"
+                  ? option === value
+                  : option.id === value.id
+              }
+              value={jobBucketOptions.filter((option) => draft.jobBuckets.includes(option.id))}
+              onChange={(_event, value) => {
+                const next = (value as (string | VendorFacetJobBucket)[]).map((entry) =>
+                  typeof entry === "string" ? entry : entry.id,
+                );
+                setDraft((current) => ({ ...current, jobBuckets: next }));
+              }}
+              renderValue={(value, getItemProps) =>
+                value.map((option, index) => {
+                  const label = typeof option === "string" ? option : option.label;
+                  const tagProps = getItemProps({ index });
+                  const { key, ...chipProps } = tagProps;
+                  return <Chip key={key} size="small" label={label} {...chipProps} />;
+                })
+              }
+              renderInput={(params) => (
+                <TextField
+                  {...params}
+                  placeholder={draft.jobBuckets.length === 0 ? "All job counts" : ""}
+                />
+              )}
+            />
+          </FilterSection>
         </Box>
 
         <Stack
@@ -85,16 +303,11 @@ export function VendorFiltersDrawer({
             borderTop: (theme) => `1px solid ${theme.palette.divider}`,
           }}
         >
-          <Button
+          <Button variant="outlined" onClick={handleClear} disabled={activeCount === 0} fullWidth>
-            variant="outlined"
-            onClick={() => setDraft(emptyVendorFilters)}
-            disabled={activeCount === 0}
-            fullWidth
-          >
             Clear all
           </Button>
           <Button variant="contained" onClick={handleApply} fullWidth>
-            Apply filters
+            Apply
           </Button>
         </Stack>
       </Stack>

src/app/(protected)/vendors/_components/vendor-roster-form-fields.tsx

@@ -28,7 +28,6 @@ interface VendorRosterFormFieldsProps {
   selectedCompanyId?: string | number | null;
   onSelectCompany?: (company: VendorFacetCompany | null) => Promise<void>;
   onClearSelectedCompany?: (nextName?: string) => void;
-  showTechnicianStatus?: boolean;
 }
 
 function CompanyNameField({
@@ -55,7 +54,6 @@ function CompanyNameField({
           <TextField
             {...field}
             label="Company"
-            placeholder="Start typing a company…"
             required
             error={Boolean(errors.name)}
             helperText={errors.name?.message}
@@ -100,9 +98,8 @@ function CompanyNameField({
               {...params}
               label="Company"
               required
-              placeholder="Start typing a company…"
               error={Boolean(errors.name)}
-              helperText={errors.name?.message}
+              helperText={errors.name?.message ?? "Choose an existing company or enter a new one"}
               inputRef={field.ref}
             />
           )}
@@ -122,7 +119,10 @@ function CompanyFields({
 }: VendorRosterFormFieldsProps) {
   return (
     <Box>
-      <Stack spacing={2}>
+      <Typography variant="subtitle2" className="font-semibold uppercase">
+        Company
+      </Typography>
+      <Stack spacing={2} className="mt-2">
         <CompanyNameField
           control={control}
           errors={errors}
@@ -138,13 +138,13 @@ function CompanyFields({
             render={({ field }) => (
               <TextField
                 {...field}
-                label="Company Phone (optional)"
+                label="Company Phone"
-                placeholder="(214) 555-0100"
+                placeholder="(XXX) XXX-XXXX"
                 inputMode="tel"
                 value={field.value ?? ""}
                 onChange={(event) => field.onChange(formatNorthAmericanPhone(event.target.value))}
                 error={Boolean(errors.companyPhone)}
-                helperText={errors.companyPhone?.message}
+                helperText={errors.companyPhone?.message ?? "Phone or email required"}
                 fullWidth
               />
             )}
@@ -155,8 +155,7 @@ function CompanyFields({
             render={({ field }) => (
               <TextField
                 {...field}
-                label="Email (optional)"
+                label="Email"
-                placeholder="dispatch@vendor.com"
                 type="email"
                 error={Boolean(errors.email)}
                 helperText={errors.email?.message}
@@ -168,16 +167,9 @@ function CompanyFields({
         <Controller
           control={control}
           name="address"
-          render={({ field }) => (
+          render={({ field }) => <TextField {...field} label="Address" fullWidth />}
-            <TextField
-              {...field}
-              label="Address (optional)"
-              placeholder="123 Industrial Pkwy, Dallas, TX 75201"
-              fullWidth
-            />
-          )}
         />
-        <Stack direction={{ xs: "column", sm: "row" }} spacing={2} sx={{ display: "none" }}>
+        <Stack direction={{ xs: "column", sm: "row" }} spacing={2}>
           <Controller
             control={control}
             name="city"
@@ -202,10 +194,10 @@ function CompanyFields({
           render={({ field }) => (
             <TextField
               {...field}
-              label="Google Maps URL (optional)"
+              label="Google Maps URL"
-              placeholder="https://maps.google.com/…"
+              placeholder="https://maps.google.com/..."
               error={Boolean(errors.googleMapsUrl)}
-              helperText={errors.googleMapsUrl?.message}
+              helperText={errors.googleMapsUrl?.message ?? "Must be an HTTPS link"}
               fullWidth
             />
           )}
@@ -214,14 +206,7 @@ function CompanyFields({
           control={control}
           name="notes"
           render={({ field }) => (
-            <TextField
+            <TextField {...field} label="Notes" multiline minRows={2} fullWidth />
-              {...field}
-              label="Notes (optional)"
-              placeholder="Internal notes, scheduling preferences, certifications…"
-              multiline
-              minRows={2}
-              fullWidth
-            />
           )}
         />
       </Stack>
@@ -236,7 +221,6 @@ interface TechnicianRowProps {
   onRemove: () => void;
   canRemove: boolean;
   tradeOptions: string[];
-  showStatus: boolean;
 }
 
 function TechnicianRow({
@@ -246,13 +230,12 @@ function TechnicianRow({
   onRemove,
   canRemove,
   tradeOptions,
-  showStatus,
 }: TechnicianRowProps) {
   return (
     <Stack spacing={2} className="rounded-md border border-solid border-gray-200 p-3">
       <Stack direction="row" sx={{ alignItems: "center", justifyContent: "space-between" }}>
         <Typography variant="subtitle2" className="font-semibold">
-          {index + 1} - TECHNICIAN
+          Technician {index + 1}
         </Typography>
         <Tooltip title="Remove technician">
           <span>
@@ -273,8 +256,7 @@ function TechnicianRow({
         render={({ field }) => (
           <TextField
             {...field}
-            label="Technician name (optional)"
+            label="Technician Name"
-            placeholder="Technician name"
             error={Boolean(error?.contactName)}
             helperText={error?.contactName?.message}
             fullWidth
@@ -288,8 +270,8 @@ function TechnicianRow({
           render={({ field }) => (
             <TextField
               {...field}
-              label="Technician phone (optional)"
+              label="Technician Phone"
-              placeholder="(000) 000-0000"
+              placeholder="(XXX) XXX-XXXX"
               inputMode="tel"
               value={field.value ?? ""}
               onChange={(event) => field.onChange(formatNorthAmericanPhone(event.target.value))}
@@ -310,29 +292,26 @@ function TechnicianRow({
               error={Boolean(error?.email)}
               helperText={error?.email?.message}
               fullWidth
-              sx={{ display: "none" }}
             />
           )}
         />
       </Stack>
       <VendorTradeSpecialtiesField control={control} index={index} tradeOptions={tradeOptions} />
-      {showStatus && (
+      <Controller
-        <Controller
+        control={control}
-          control={control}
+        name={`technicians.${index}.isActive`}
-          name={`technicians.${index}.isActive`}
+        render={({ field }) => (
-          render={({ field }) => (
+          <FormControlLabel
-            <FormControlLabel
+            control={
-              control={
+              <Switch
-                <Switch
+                checked={field.value}
-                  checked={field.value}
+                onChange={(_event, checked) => field.onChange(checked)}
-                  onChange={(_event, checked) => field.onChange(checked)}
+              />
-                />
+            }
-              }
+            label={field.value ? "Active" : "Inactive"}
-              label={field.value ? "Active" : "Inactive"}
+          />
-            />
+        )}
-          )}
+      />
-        />
-      )}
     </Stack>
   );
 }
@@ -341,12 +320,10 @@ function TechniciansFieldArray({
   control,
   errors,
   tradeOptions,
-  showStatus,
 }: {
   control: Control<VendorCompanyRosterFormValues>;
   errors: FieldErrors<VendorCompanyRosterFormValues>;
   tradeOptions: string[];
-  showStatus: boolean;
 }) {
   const { fields, append, remove } = useFieldArray({ control, name: "technicians" });
 
@@ -389,7 +366,6 @@ function TechniciansFieldArray({
               onRemove={() => remove(index)}
               canRemove
               tradeOptions={tradeOptions}
-              showStatus={showStatus}
             />
           ))
         )}
@@ -399,17 +375,12 @@ function TechniciansFieldArray({
 }
 
 export function VendorRosterFormFields(props: VendorRosterFormFieldsProps) {
-  const { control, errors, tradeOptions = [], showTechnicianStatus = true } = props;
+  const { control, errors, tradeOptions = [] } = props;
   return (
     <Stack spacing={3}>
       <CompanyFields {...props} />
       <Divider />
-      <TechniciansFieldArray
+      <TechniciansFieldArray control={control} errors={errors} tradeOptions={tradeOptions} />
-        control={control}
-        errors={errors}
-        tradeOptions={tradeOptions}
-        showStatus={showTechnicianStatus}
-      />
     </Stack>
   );
 }

src/app/(protected)/vendors/_components/vendors-list-toolbar.tsx

@@ -1,9 +1,13 @@
-import { Button, Chip, CircularProgress, Stack, TablePagination } from "@mui/material";
+import FilterListIcon from "@mui/icons-material/FilterList";
-import { Plus, SlidersHorizontal } from "lucide-react";
+import {
-import { PageHeader } from "@/components/ui/page-header";
+  Button,
-import { SearchInput } from "@/components/ui/search-input";
+  Chip,
-import { Text } from "@/components/ui/text";
+  CircularProgress,
-import { ToolbarButton } from "@/components/ui/toolbar-button";
+  Stack,
+  TablePagination,
+  TextField,
+  Typography,
+} from "@mui/material";
 
 interface VendorsListToolbarProps {
   globalCount: number;
@@ -29,44 +33,52 @@ export function VendorsListToolbar({
   onAddVendor,
 }: VendorsListToolbarProps) {
   return (
-    <PageHeader
+    <Stack
-      surface
+      direction={{ xs: "column", md: "row" }}
-      className="-mx-4 -mt-4"
+      spacing={2}
-      title="Vendors / Technicians"
+      sx={{
-      subtitle={`${globalCount} technicians · ${activeCount} active`}
+        alignItems: { xs: "stretch", md: "center" },
-      actions={
+        justifyContent: "space-between",
-        <>
+      }}
-          <SearchInput
+    >
-            value={search}
+      <Stack direction="row" spacing={2} sx={{ alignItems: "baseline" }}>
-            onChange={onSearchChange}
+        <Typography variant="h5" component="h1">
-            placeholder="Search name, company, trade…"
+          Vendors / Technicians
-            aria-label="Search vendors"
+        </Typography>
-            inputClassName="h-8 py-0 text-[13px]"
+        <Typography variant="body2" sx={{ color: "text.secondary" }}>
-          />
+          {globalCount} vendor entries · {activeCount} active
-          <ToolbarButton
+        </Typography>
-            variant="outline"
+        {isFetching && !isLoading && <CircularProgress size={16} />}
-            icon={<SlidersHorizontal size={16} />}
+      </Stack>
-            onClick={onOpenFilters}
+
-            aria-label="Open vendor filters"
+      <Stack
-            badge={appliedFilterCount}
+        direction={{ xs: "column", sm: "row" }}
-            className="w-[88.609375px] text-[13px]"
+        spacing={2}
-          >
+        sx={{ alignItems: { xs: "stretch", sm: "center" } }}
-            Filters
+      >
-          </ToolbarButton>
+        <TextField
-          <ToolbarButton
+          size="small"
-            variant="primary"
+          value={search}
-            icon={<Plus size={16} />}
+          onChange={(e) => onSearchChange(e.target.value)}
-            onClick={onAddVendor}
+          placeholder="Search name, company, trade..."
-            className="w-[115.546875px] border-0 px-3 text-[13px]"
+          className="min-w-[260px]"
-          >
+        />
-            Add Vendor
+        <Button
-          </ToolbarButton>
+          variant="outlined"
-          {isFetching && !isLoading && (
+          onClick={onOpenFilters}
-            <CircularProgress size={14} aria-label="Refreshing vendors" role="status" />
+          startIcon={<FilterListIcon />}
+          aria-label="Open vendor filters"
+        >
+          Filters
+          {appliedFilterCount > 0 && (
+            <Chip size="small" color="primary" label={appliedFilterCount} sx={{ ml: 1 }} />
           )}
-        </>
+        </Button>
-      }
+        <Button variant="contained" onClick={onAddVendor}>
-    />
+          + Add Vendor
+        </Button>
+      </Stack>
+    </Stack>
   );
 }
 
@@ -78,21 +90,16 @@ interface AppliedChipEntry {
 
 interface VendorAppliedFiltersBarProps {
   chips: AppliedChipEntry[];
-  resultCount: number;
   onClearAll: () => void;
 }
 
-export function VendorAppliedFiltersBar({
+export function VendorAppliedFiltersBar({ chips, onClearAll }: VendorAppliedFiltersBarProps) {
-  chips,
-  resultCount,
-  onClearAll,
-}: VendorAppliedFiltersBarProps) {
   if (chips.length === 0) return null;
   return (
     <Stack direction="row" spacing={1} className="flex flex-wrap items-center">
-      <Text variant="caption" tone="muted">
+      <Typography variant="caption" sx={{ color: "text.secondary" }}>
-        {resultCount} {resultCount === 1 ? "technician" : "technicians"} · filtered by
+        Applied:
-      </Text>
+      </Typography>
       {chips.map((chip) => (
         <Chip
           key={chip.key}
@@ -139,31 +146,23 @@ export function VendorsListFooter({
       sx={{
         alignItems: { xs: "stretch", sm: "center" },
         justifyContent: "space-between",
-        minHeight: 33.5,
-        mx: -2,
-        px: 2,
-        backgroundColor: "var(--color-bg-surface)",
       }}
     >
-      <Text variant="caption" as="span" sx={{ color: "#6b7a96", fontSize: 11, lineHeight: 1.5 }}>
+      <Typography variant="body2" sx={{ color: "text.secondary" }}>
         {totalCount} {isActive ? "active" : "inactive"}{" "}
-        {totalCount === 1 ? "technician" : "technicians"}
+        {totalCount === 1 ? "vendor entry" : "vendor entries"}
         {isFiltered && " · filtered"}
-      </Text>
+        {` · ${activeCount} active · ${inactiveCount} inactive`}
-      <Text variant="caption" as="span" sx={{ color: "#6b7a96", fontSize: 11, lineHeight: 1.5 }}>
+      </Typography>
-        {activeCount} active · {inactiveCount} inactive
+      <TablePagination
-      </Text>
+        component="div"
-      {totalCount > pageSize && (
+        count={totalCount}
-        <TablePagination
+        page={page}
-          component="div"
+        onPageChange={(_e, nextPage) => onPageChange(nextPage)}
-          count={totalCount}
+        rowsPerPage={pageSize}
-          page={page}
+        onRowsPerPageChange={(e) => onPageSizeChange(Number(e.target.value))}
-          onPageChange={(_e, nextPage) => onPageChange(nextPage)}
+        rowsPerPageOptions={[12, 24, 48]}
-          rowsPerPage={pageSize}
+      />
-          onRowsPerPageChange={(e) => onPageSizeChange(Number(e.target.value))}
-          rowsPerPageOptions={[24, 48, 96]}
-        />
-      )}
     </Stack>
   );
 }

src/app/(protected)/vendors/_components/vendors-table.tsx

@@ -1,10 +1,11 @@
 import { type KeyboardEvent, type MouseEvent } from "react";
-import SearchOffOutlinedIcon from "@mui/icons-material/SearchOffOutlined";
+import { useNavigate } from "react-router";
+import DeleteOutlineIcon from "@mui/icons-material/DeleteOutlined";
 import EditOutlinedIcon from "@mui/icons-material/EditOutlined";
 import VisibilityOutlinedIcon from "@mui/icons-material/VisibilityOutlined";
 import {
   Avatar,
-  Box,
+  Chip,
   CircularProgress,
   IconButton,
   Link,
@@ -17,8 +18,8 @@ import {
   TableHead,
   TableRow,
   Tooltip,
+  Typography,
 } from "@mui/material";
-import { Text } from "@/components/ui/text";
 import type { VendorListItem } from "@/domain/vendors/types/vendor";
 
 const TABLE_COLUMNS = [
@@ -29,6 +30,9 @@ const TABLE_COLUMNS = [
   "Phone",
   "Email",
   "Total Jobs",
+  "Status",
+  "View",
+  "Edit",
 ] as const;
 
 function getInitials(name: string): string {
@@ -50,34 +54,19 @@ function stopPropagation(event: MouseEvent): void {
   event.stopPropagation();
 }
 
-function VendorStatus({ isActive }: { isActive: boolean }) {
-  return (
-    <Stack direction="row" spacing={1} sx={{ alignItems: "center", pl: 1 }}>
-      <Box
-        component="span"
-        sx={{
-          width: 6,
-          height: 6,
-          borderRadius: "9999px",
-          backgroundColor: isActive ? "success.main" : "text.secondary",
-        }}
-      />
-      <Text variant="description">{isActive ? "Active" : "Inactive"}</Text>
-    </Stack>
-  );
-}
-
 interface VendorTableRowProps {
   row: VendorListItem;
   onOpenDetail: (row: VendorListItem) => void;
-  onOpenEdit: (row: VendorListItem) => void;
+  onOpenDeactivation: (row: VendorListItem) => void;
 }
 
-function VendorTableRow({ row, onOpenDetail, onOpenEdit }: VendorTableRowProps) {
+function VendorTableRow({ row, onOpenDetail, onOpenDeactivation }: VendorTableRowProps) {
+  const navigate = useNavigate();
   const trades = splitTrades(row.tradeSpecialties);
   const primaryTrade = trades[0] ?? "—";
   const extraTrades = trades.length > 1 ? trades.length - 1 : 0;
   const locationText = [row.city, row.state].filter(Boolean).join(", ") || "—";
+  const hasVendorId = row.id != null;
   const rowKey = String(row.id ?? `company-${row.companyId ?? ""}`);
 
   const handleKeyDown = (event: KeyboardEvent) => {
@@ -99,62 +88,37 @@ function VendorTableRow({ row, onOpenDetail, onOpenEdit }: VendorTableRowProps)
       onKeyDown={handleKeyDown}
     >
       <TableCell>
-        <Text variant="description" sx={{ color: "var(--color-foreground)" }}>
+        <Typography className="font-semibold">{row.companyName || "—"}</Typography>
-          {row.companyName || "—"}
-        </Text>
       </TableCell>
       <TableCell>
-        <Stack direction="row" spacing={1.25} sx={{ alignItems: "center" }}>
+        <Stack direction="row" spacing={1.5} sx={{ alignItems: "center" }}>
           <Avatar
             sx={{
               width: 28,
               height: 28,
-              backgroundColor: "var(--color-bg-muted)",
+              fontSize: 12,
-              color: "var(--color-primary)",
+              backgroundColor: "var(--accent, #1976d2)",
-              fontSize: 10.5,
-              fontWeight: 600,
             }}
           >
             {getInitials(row.contactName)}
           </Avatar>
-          <Text variant="description" sx={{ color: "var(--color-foreground)" }}>
+          <Typography variant="body2">{row.contactName || "—"}</Typography>
-            {row.contactName || "—"}
-          </Text>
         </Stack>
       </TableCell>
-      <TableCell sx={{ color: "var(--color-muted-foreground)" }}>{locationText}</TableCell>
+      <TableCell>{locationText}</TableCell>
-      <TableCell sx={{ color: "var(--color-muted-foreground)" }}>
+      <TableCell>
         {primaryTrade === "—" ? (
           "—"
         ) : (
           <Stack direction="row" spacing={0.5} sx={{ alignItems: "center" }}>
             <span>{primaryTrade}</span>
-            {extraTrades > 0 && (
+            {extraTrades > 0 && <Chip size="small" label={`+${extraTrades}`} />}
-              <Box
-                component="span"
-                sx={{
-                  borderRadius: "9999px",
-                  backgroundColor: "var(--color-bg-muted)",
-                  px: 0.75,
-                  py: 0.25,
-                  fontSize: 10.5,
-                  lineHeight: 1.2,
-                }}
-              >
-                +{extraTrades}
-              </Box>
-            )}
           </Stack>
         )}
       </TableCell>
-      <TableCell sx={{ color: "var(--color-muted-foreground)" }}>
+      <TableCell>
         {row.phone ? (
-          <Link
+          <Link href={`tel:${row.phone}`} underline="hover" onClick={stopPropagation}>
-            href={`tel:${row.phone}`}
-            underline="hover"
-            color="inherit"
-            onClick={stopPropagation}
-          >
             {row.phone}
           </Link>
         ) : (
@@ -163,53 +127,71 @@ function VendorTableRow({ row, onOpenDetail, onOpenEdit }: VendorTableRowProps)
       </TableCell>
       <TableCell>
         {row.email ? (
-          <Link
+          <Link href={`mailto:${row.email}`} underline="hover" onClick={stopPropagation}>
-            href={`mailto:${row.email}`}
-            underline="hover"
-            sx={{ color: "#6b7a96" }}
-            onClick={stopPropagation}
-          >
             {row.email}
           </Link>
         ) : (
           "—"
         )}
       </TableCell>
-      <TableCell align="center">{row.totalJobs ?? 0}</TableCell>
+      <TableCell>{row.totalJobs ?? 0}</TableCell>
       <TableCell>
-        <VendorStatus isActive={row.isActive} />
+        <Chip
+          size="small"
+          label={row.isActive ? "Active" : "Inactive"}
+          color={row.isActive ? "success" : "default"}
+        />
       </TableCell>
-      <TableCell align="right">
+      <TableCell align="center">
+        <Tooltip title="View details">
+          <IconButton
+            size="small"
+            aria-label={`View vendor ${row.companyName}`}
+            onClick={(event) => {
+              event.stopPropagation();
+              onOpenDetail(row);
+            }}
+          >
+            <VisibilityOutlinedIcon fontSize="small" />
+          </IconButton>
+        </Tooltip>
+      </TableCell>
+      <TableCell align="center">
         <Stack
           direction="row"
           spacing={0.5}
-          className="vendor-row-actions"
+          sx={{ alignItems: "center", justifyContent: "center" }}
-          sx={{ alignItems: "center", justifyContent: "flex-end" }}
         >
-          <Tooltip title="View details">
-            <IconButton
-              size="small"
-              aria-label={`View vendor ${row.companyName}`}
-              onClick={(event) => {
-                event.stopPropagation();
-                onOpenDetail(row);
-              }}
-            >
-              <VisibilityOutlinedIcon fontSize="small" />
-            </IconButton>
-          </Tooltip>
           <Tooltip title="Edit">
             <IconButton
               size="small"
               aria-label={`Edit vendor ${row.companyName}`}
               onClick={(event) => {
                 event.stopPropagation();
-                onOpenEdit(row);
+                navigate(
+                  row.id != null
+                    ? `/vendors/edit/${row.id}`
+                    : `/vendors/company/${row.companyId ?? ""}`,
+                );
               }}
             >
               <EditOutlinedIcon fontSize="small" />
             </IconButton>
           </Tooltip>
+          {hasVendorId && row.isActive && (
+            <Tooltip title="Deactivate">
+              <IconButton
+                size="small"
+                aria-label="Deactivate vendor"
+                onClick={(event) => {
+                  event.stopPropagation();
+                  onOpenDeactivation(row);
+                }}
+              >
+                <DeleteOutlineIcon fontSize="small" />
+              </IconButton>
+            </Tooltip>
+          )}
         </Stack>
       </TableCell>
     </TableRow>
@@ -218,118 +200,50 @@ function VendorTableRow({ row, onOpenDetail, onOpenEdit }: VendorTableRowProps)
 
 interface VendorsTableProps {
   rows: VendorListItem[];
-  isActive: boolean;
-  isFiltered: boolean;
   isLoading: boolean;
   showError: boolean;
   onOpenDetail: (row: VendorListItem) => void;
-  onOpenEdit: (row: VendorListItem) => void;
+  onOpenDeactivation: (row: VendorListItem) => void;
 }
 
 export function VendorsTable({
   rows,
-  isActive,
-  isFiltered,
   isLoading,
   showError,
   onOpenDetail,
-  onOpenEdit,
+  onOpenDeactivation,
 }: VendorsTableProps) {
   const showEmptyRow = !isLoading && !showError && rows.length === 0;
-  const columnCount = TABLE_COLUMNS.length + 2;
 
   return (
-    <TableContainer
+    <TableContainer component={Paper} variant="outlined" sx={{ overflowX: "auto" }}>
-      component={Paper}
+      <Table size="small" sx={{ minWidth: 960 }}>
-      variant="outlined"
-      sx={{
-        mt: 2,
-        mb: 2,
-        flex: "1 1 auto",
-        overflow: "auto",
-        borderRadius: 1.5,
-        fontSize: 13,
-      }}
-    >
-      <Table
-        size="small"
-        stickyHeader
-        sx={{
-          minWidth: 1550,
-          tableLayout: "fixed",
-          "& .MuiTableCell-head": {
-            backgroundColor: "var(--color-bg-muted)",
-            color: "var(--color-foreground)",
-            fontSize: 10.5,
-            fontWeight: 700,
-            letterSpacing: "0.06em",
-            lineHeight: "24px",
-            px: 1.75,
-            py: 1.375,
-            textTransform: "uppercase",
-            whiteSpace: "nowrap",
-          },
-          "& .MuiTableCell-body": {
-            fontSize: 13,
-            px: 1.5,
-            py: 1,
-            height: 52,
-            lineHeight: 1.43,
-            whiteSpace: "nowrap",
-          },
-          "& .vendor-row-actions": {
-            opacity: { xs: 1, md: 0 },
-            transition: "opacity 120ms ease",
-          },
-          "& .MuiTableRow-root:hover .vendor-row-actions, & .MuiTableRow-root:focus-within .vendor-row-actions":
-            {
-              opacity: 1,
-            },
-        }}
-      >
-        <colgroup>
-          <col style={{ width: 196.34375 }} />
-          <col style={{ width: 207.25 }} />
-          <col style={{ width: 152.703125 }} />
-          <col style={{ width: 218.15625 }} />
-          <col style={{ width: 152.703125 }} />
-          <col style={{ width: 251.90625 }} />
-          <col style={{ width: 109.078125 }} />
-          <col style={{ width: 119.984375 }} />
-          <col style={{ width: 141.875 }} />
-        </colgroup>
         <TableHead>
           <TableRow>
             {TABLE_COLUMNS.map((column) => (
-              <TableCell key={column} align={column === "Total Jobs" ? "center" : "left"}>
+              <TableCell
+                key={column}
+                align={column === "View" || column === "Edit" ? "center" : "left"}
+                className="font-semibold"
+              >
                 {column}
               </TableCell>
             ))}
-            <TableCell>Status</TableCell>
-            <TableCell align="right" aria-label="Vendor actions" />
           </TableRow>
         </TableHead>
         <TableBody>
           {isLoading ? (
             <TableRow>
-              <TableCell colSpan={columnCount} align="center" className="py-10">
+              <TableCell colSpan={TABLE_COLUMNS.length} align="center" className="py-10">
                 <CircularProgress size={28} />
               </TableCell>
             </TableRow>
           ) : showEmptyRow ? (
             <TableRow>
-              <TableCell colSpan={columnCount} align="center" className="py-10">
+              <TableCell colSpan={TABLE_COLUMNS.length} align="center" className="py-10">
-                <Stack spacing={1} sx={{ alignItems: "center" }}>
+                <Typography sx={{ color: "text.secondary" }}>
-                  <SearchOffOutlinedIcon sx={{ color: "text.secondary", fontSize: 32 }} />
+                  No vendors match the current filters.
-                  <Text variant="description" className="font-semibold">
+                </Typography>
-                    {`No ${isActive ? "active" : "inactive"} technicians found`}
-                  </Text>
-                  <Text variant="caption" tone="muted">
-                    {isFiltered
-                      ? "Try a different search, or add a new vendor."
-                      : "Add a vendor to get started."}
-                  </Text>
-                </Stack>
               </TableCell>
             </TableRow>
           ) : (
@@ -338,7 +252,7 @@ export function VendorsTable({
                 key={String(row.id ?? `company-${row.companyId ?? ""}`)}
                 row={row}
                 onOpenDetail={onOpenDetail}
-                onOpenEdit={onOpenEdit}
+                onOpenDeactivation={onOpenDeactivation}
               />
             ))
           )}

src/app/(protected)/vendors/index.tsx

@@ -23,91 +23,16 @@ interface AppliedChip {
   label: string;
 }
 
-function statusTabLabel(label: string, count: number) {
-  return (
-    <Box className="flex items-center gap-2">
-      <Box component="span">{label}</Box>
-      <Box
-        component="span"
-        className="min-w-5 rounded-full px-1.5 py-0.5 text-[10px] font-semibold leading-none"
-        sx={{ backgroundColor: "action.hover", color: "text.secondary" }}
-      >
-        {count}
-      </Box>
-    </Box>
-  );
-}
-
-function VendorStatusTabs({
-  isActive,
-  activeCount,
-  inactiveCount,
-  onChange,
-}: {
-  isActive: boolean;
-  activeCount: number;
-  inactiveCount: number;
-  onChange: (isActive: boolean) => void;
-}) {
-  return (
-    <Tabs
-      value={isActive ? "active" : "inactive"}
-      onChange={(_event, value: "active" | "inactive") => onChange(value === "active")}
-      aria-label="Vendor status"
-      sx={{
-        minHeight: 36,
-        flexShrink: 0,
-        borderBottom: (theme) => `1px solid ${theme.palette.divider}`,
-        backgroundColor: "var(--color-bg-surface)",
-        mx: -2,
-        px: 1.5,
-        "& .MuiTab-root": {
-          minHeight: 36,
-          px: 2,
-          py: 0,
-          fontSize: 13,
-          fontWeight: 500,
-          textTransform: "none",
-        },
-        "& .MuiTab-root:first-of-type": { width: 102.84375 },
-        "& .Mui-selected": { fontWeight: 600 },
-        "& .MuiTabs-indicator": { height: 2 },
-      }}
-    >
-      <Tab
-        value="active"
-        label={statusTabLabel("Active", activeCount)}
-        aria-label={`Active (${activeCount})`}
-      />
-      <Tab
-        value="inactive"
-        label={statusTabLabel("Inactive", inactiveCount)}
-        aria-label={`Inactive (${inactiveCount})`}
-      />
-    </Tabs>
-  );
-}
-
 function describeChips(
   filters: VendorFilters,
   jobBucketLabels: Map<string, string>,
 ): AppliedChip[] {
   const chips: AppliedChip[] = [];
-  filters.companies.forEach((value) =>
+  filters.companies.forEach((value) => chips.push({ category: "companies", value, label: value }));
-    chips.push({ category: "companies", value, label: `Company: ${value}` }),
+  filters.trades.forEach((value) => chips.push({ category: "trades", value, label: value }));
-  );
+  filters.locations.forEach((value) => chips.push({ category: "locations", value, label: value }));
-  filters.trades.forEach((value) =>
-    chips.push({ category: "trades", value, label: `Trade: ${value}` }),
-  );
-  filters.locations.forEach((value) =>
-    chips.push({ category: "locations", value, label: `Location: ${value}` }),
-  );
   filters.jobBuckets.forEach((value) =>
-    chips.push({
+    chips.push({ category: "jobBuckets", value, label: jobBucketLabels.get(value) ?? value }),
-      category: "jobBuckets",
-      value,
-      label: `Total Jobs: ${jobBucketLabels.get(value) ?? value}`,
-    }),
   );
   return chips;
 }
@@ -117,12 +42,7 @@ export default function VendorsListPage() {
   const [filtersOpen, setFiltersOpen] = useState(false);
   const [createOpen, setCreateOpen] = useState(false);
   const [detailTarget, setDetailTarget] = useState<VendorListItem | null>(null);
-  const [detailStartsInEdit, setDetailStartsInEdit] = useState(false);
+  const deactivation = useVendorDeactivation();
-  const closeDetail = () => {
-    setDetailTarget(null);
-    setDetailStartsInEdit(false);
-  };
-  const deactivation = useVendorDeactivation(closeDetail);
 
   const { data: facets } = useVendorFacets(filters.isActive);
   const { data, isLoading, isFetching, error } = useVendorsList(filters.listParams);
@@ -141,7 +61,7 @@ export default function VendorsListPage() {
   const appliedChips = describeChips(filters.appliedFilters, jobBucketLabels);
 
   return (
-    <Box className="flex h-full flex-col overflow-hidden px-4 pt-4 pb-0">
+    <Box className="flex flex-col gap-4 p-4">
       <VendorsListToolbar
         globalCount={activeCount + inactiveCount}
         activeCount={activeCount}
@@ -154,15 +74,18 @@ export default function VendorsListPage() {
         onAddVendor={() => setCreateOpen(true)}
       />
 
-      <VendorStatusTabs
+      <Tabs
-        isActive={filters.isActive}
+        value={filters.isActive ? "active" : "inactive"}
-        activeCount={activeCount}
+        onChange={(_event, value: "active" | "inactive") =>
-        inactiveCount={inactiveCount}
+          filters.setIsActiveWithReset(value === "active")
-        onChange={filters.setIsActiveWithReset}
+        }
-      />
+        aria-label="Vendor status"
+      >
+        <Tab value="active" label={`Active (${activeCount})`} />
+        <Tab value="inactive" label={`Inactive (${inactiveCount})`} />
+      </Tabs>
 
       <VendorAppliedFiltersBar
-        resultCount={totalCount}
         chips={appliedChips.map((chip) => ({
           key: `${chip.category}-${chip.value}`,
           label: chip.label,
@@ -179,18 +102,10 @@ export default function VendorsListPage() {
 
       <VendorsTable
         rows={rows}
-        isActive={filters.isActive}
-        isFiltered={appliedChips.length > 0 || Boolean(filters.debouncedSearch)}
         isLoading={isLoading}
         showError={Boolean(error)}
-        onOpenDetail={(row) => {
+        onOpenDetail={setDetailTarget}
-          setDetailStartsInEdit(false);
+        onOpenDeactivation={deactivation.open}
-          setDetailTarget(row);
-        }}
-        onOpenEdit={(row) => {
-          setDetailStartsInEdit(true);
-          setDetailTarget(row);
-        }}
       />
 
       <VendorsListFooter
@@ -218,9 +133,7 @@ export default function VendorsListPage() {
       <VendorDetailDrawer
         open={Boolean(detailTarget)}
         vendor={detailTarget}
-        startInEdit={detailStartsInEdit}
+        onClose={() => setDetailTarget(null)}
-        onRequestDeactivation={deactivation.open}
-        onClose={closeDetail}
       />
 
       <VendorDeactivationDialog

src/app/(protected)/workorders/_components/list/table/to-inline-draft-row.ts

@@ -16,6 +16,7 @@ export function toInlineDraftRow(draft: WorkOrderTableDraft) {
     location: draft.location,
     dispatcherId: draft.dispatcherId,
     scheduledOn: draft.scheduledOn,
+    completedDate: "",
     dueDate: draft.dueDate,
     pm: draft.pm,
     serviceNotes: draft.serviceNotes,

src/app/(protected)/workorders/_components/list/table/wo-table-cols.ts

@@ -12,8 +12,9 @@ export const WO_TABLE_COLS = [
   { key: "appt", label: "APPT TIME", w: 128 },
   { key: "status", label: "STATUS", w: 148 },
   { key: "doc", label: "COMP DOC", w: 84 },
+  { key: "completed", label: "COMPLETED DATE", w: 130 },
   { key: "actions", label: "", w: 100 },
 ] as const;
 
-export const WO_TABLE_MIN_WIDTH = 1504;
+export const WO_TABLE_MIN_WIDTH = 1634;
 export const WO_TABLE_ROW_HEIGHT = 52;

src/app/(protected)/workorders/_components/list/table/wo-table-inline-row-cells.tsx

@@ -167,6 +167,10 @@ export function WoTableInlineRowCells({
       <td style={{ padding: "0 14px" }}>
         <span style={{ fontSize: 11, color: "var(--color-text-muted)" }}>—</span>
       </td>
+
+      <td style={{ padding: "0 14px" }}>
+        <span style={{ fontSize: 11, color: "var(--color-text-muted)" }}>—</span>
+      </td>
     </>
   );
 }

src/app/(protected)/workorders/_components/list/table/wo-table-row-service-cells.tsx

@@ -2,6 +2,7 @@ import type { VendorDropdownItem } from "@/domain/work-orders/types/work-order";
 import type { WorkOrderTableRow } from "@/domain/work-orders/types/work-order-table-row";
 import { ApptTimeCell } from "./cells/appt-time-cell";
 import { CompDocCell } from "./cells/comp-doc-cell";
+import { DateCell } from "./cells/date-cell";
 import { PMTypeCell } from "./cells/pm-type-cell";
 import { StatusCell } from "./cells/status-cell";
 import { VendorCell } from "./cells/vendor-cell";
@@ -99,6 +100,17 @@ export function WoTableRowServiceCells({
           onOpenMedia={() => handlers.onOpenMedia(row)}
         />
       </td>
+      <td
+        className="group/cell"
+        style={{ padding: "0 14px", ...mutationCellStyle }}
+        onClick={(e) => e.stopPropagation()}
+      >
+        <DateCell
+          value={row.completedDate}
+          q={search}
+          onSave={(date) => onPatchRow({ completedDate: date })}
+        />
+      </td>
     </>
   );
 }

src/components/ui/page-header.tsx

@@ -49,17 +49,10 @@ export function PageHeader({
             "m-0 font-display mt-1! text-[30px]! font-extrabold leading-[1.1] tracking-[-0.02em] text-foreground",
             eyebrow && "mt-1",
           )}
-          sx={{ fontSize: 30, fontWeight: 800, lineHeight: 1.1, letterSpacing: "-0.02em" }}
         >
           {title}
         </Text>
-        <Text
+        <Text variant="description" tone="muted" when={Boolean(subtitle)}>
-          variant="description"
-          as="span"
-          tone="muted"
-          when={Boolean(subtitle)}
-          sx={{ mt: 0.75, fontWeight: 500 }}
-        >
           {subtitle}
         </Text>
       </Stack>

src/domain/work-orders/mappers/expand-scalar-board-ops.ts

@@ -27,6 +27,7 @@ const SCALAR_BOARD_OP_RULES: ScalarBoardOpRule[] = [
   { key: "status", field: "lifecycleStatus", toValue: (value) => value as string },
   { key: "dispatcherId", field: "assignTo", toValue: (value) => value as string },
   { key: "dueDate", field: "dueDate", toValue: (value) => value as string },
+  { key: "completedDate", field: "completedDate", toValue: (value) => value as string },
   { key: "vendorId", field: "vendorId", toValue: (value) => String(value) },
   { key: "apptTime", field: "apptTime", toValue: (value) => value as string },
   { key: "docStatus", field: "docStatus", toValue: (value) => mapDocStatusToApi(value as string) },

src/domain/work-orders/mappers/work-order-table-row-mapper.ts

@@ -237,6 +237,7 @@ export function mapWorkOrderTableRow(raw: unknown): WorkOrderTableRow {
     dispatcherId,
     dispatcherName,
     scheduledOn,
+    completedDate: toDateInputValue(readString(item, "completedDate", "CompletedDate")),
     targetWeek: toDateInputValue(readString(item, "targetWeek", "TargetWeek")),
     dayGroup,
     dayLabel,

src/domain/work-orders/types/work-order-board.ts

@@ -116,6 +116,7 @@ export type WorkOrderBoardEditableField =
   | "apptDate"
   | "apptTime"
   | "docStatus"
+  | "completedDate"
   | "pm"
   | "serviceNotes"
   | "extraServices"
@@ -146,6 +147,8 @@ export interface WorkOrderBoardRowApi {
   flagColor?: string | null;
   dueDate?: string | null;
   scheduledDate?: string | null;
+  /** ISO date when work was completed; optional on list rows until BE confirms. */
+  completedDate?: string | null;
   targetWeek?: string | null;
   scheduleWeekOnly?: boolean | null;
   dayGroup?: string | null;

src/domain/work-orders/types/work-order-table-row.ts

@@ -12,6 +12,8 @@ export interface WorkOrderTableRow {
   dispatcherId: string;
   dispatcherName: string;
   scheduledOn: string;
+  /** Work completed date (API CompletedDate); distinct from scheduledOn. */
+  completedDate: string;
   targetWeek: string;
   dayGroup: string;
   dayLabel: string;

src/domain/work-orders/utils/slide-over-draft.ts

@@ -18,6 +18,7 @@ export const SLIDE_OVER_EDIT_KEYS = [
   "targetWeek",
   "scheduleWeekOnly",
   "dueDate",
+  "completedDate",
   "apptTime",
   "pm",
   "serviceNotes",

src/test/app/(protected)/vendors/use-vendor-roster-form.test.tsx

@@ -5,7 +5,6 @@ import { useController } from "react-hook-form";
 import { beforeEach, describe, expect, it, vi } from "vitest";
 
 const rosterGet = vi.fn();
-const saveMutate = vi.fn();
 
 vi.mock("@/domain/vendors/api/vendor-company-roster-api", () => ({
   vendorCompanyRosterApi: { get: (...args: unknown[]) => rosterGet(...args) },
@@ -29,10 +28,7 @@ vi.mock("@/domain/vendors/use-cases/use-save-vendor-company-roster", async () =>
   const actual = await vi.importActual<
     typeof import("@/domain/vendors/use-cases/use-save-vendor-company-roster")
   >("@/domain/vendors/use-cases/use-save-vendor-company-roster");
-  return {
+  return { ...actual, useSaveVendorCompanyRoster: () => ({ mutate: vi.fn(), isPending: false }) };
-    ...actual,
-    useSaveVendorCompanyRoster: () => ({ mutate: saveMutate, isPending: false }),
-  };
 });
 
 import { useVendorRosterForm } from "@/app/(protected)/vendors/_components/use-vendor-roster-form";
@@ -105,7 +101,6 @@ function deferRoster(): {
 describe("useVendorRosterForm load-error handling", () => {
   beforeEach(() => {
     rosterGet.mockReset();
-    saveMutate.mockReset();
   });
 
   it("shows a safe retryable error when selecting an existing company fetch is rejected", async () => {
@@ -201,44 +196,6 @@ describe("useVendorRosterForm load-error handling", () => {
   });
 });
 
-describe("useVendorRosterForm prototype defaults", () => {
-  it("does not persist the prototype's untouched optional technician card", () => {
-    const { result } = renderHook(
-      () => useVendorRosterForm({ mode: "create", startWithTechnician: true }),
-      { wrapper: makeWrapper(createClient()) },
-    );
-
-    act(() => {
-      result.current.submit({
-        name: "Independent Vendor LLC",
-        companyPhone: "(314) 555-0199",
-        email: "",
-        address: "",
-        city: "",
-        state: "",
-        zip: "",
-        googleMapsUrl: "",
-        notes: "",
-        technicians: [
-          {
-            contactName: "",
-            phone: "",
-            email: "",
-            preferredContact: "Phone",
-            tradeSpecialties: "",
-            isActive: true,
-          },
-        ],
-      });
-    });
-
-    expect(saveMutate).toHaveBeenCalledWith(
-      expect.objectContaining({ values: expect.objectContaining({ technicians: [] }) }),
-      expect.any(Object),
-    );
-  });
-});
-
 describe("useVendorRosterForm stale-selection handling", () => {
   beforeEach(() => {
     rosterGet.mockReset();

src/test/app/(protected)/vendors/vendor-detail-drawer.test.tsx

@@ -49,11 +49,11 @@ function rosterWith(technicians: Array<Record<string, unknown>>) {
   };
 }
 
-describe("VendorDetailDrawer selected-technician display", () => {
+describe("VendorDetailDrawer preferred-contact display", () => {
   it("renders no preference label when preferredContact is absent", () => {
     useVendorCompanyRoster.mockReturnValue(
       rosterWith([
-        { id: 1, contactName: "Adam", phone: "314-555-0198", isActive: true, totalJobs: 5 },
+        { id: 7, contactName: "Adam", phone: "314-555-0198", isActive: true, totalJobs: 5 },
       ]),
     );
 
@@ -62,15 +62,15 @@ describe("VendorDetailDrawer selected-technician display", () => {
       withAuth: false,
     });
 
-    expect(screen.getByRole("heading", { level: 2, name: "Adam" })).toBeInTheDocument();
+    expect(screen.getByText("Adam")).toBeInTheDocument();
     expect(screen.queryByText(/^Prefers/)).not.toBeInTheDocument();
   });
 
-  it("keeps the retired preferred-contact value out of the detail view", () => {
+  it("renders the preference label when an explicit preferredContact is present", () => {
     useVendorCompanyRoster.mockReturnValue(
       rosterWith([
         {
-          id: 1,
+          id: 7,
           contactName: "Adam",
           phone: "314-555-0198",
           preferredContact: "Email",
@@ -85,52 +85,6 @@ describe("VendorDetailDrawer selected-technician display", () => {
       withAuth: false,
     });
 
-    expect(screen.getByRole("heading", { level: 2, name: "Adam" })).toBeInTheDocument();
+    expect(screen.getByText("Prefers Email")).toBeInTheDocument();
-    expect(screen.queryByText(/^Prefers/)).not.toBeInTheDocument();
-  });
-
-  it("falls back to the clicked row when the roster has no matching stable id", () => {
-    useVendorCompanyRoster.mockReturnValue(
-      rosterWith([
-        {
-          id: 7,
-          contactName: "Adam Whyte",
-          phone: "999-999-9999",
-          isActive: true,
-          totalJobs: 99,
-        },
-      ]),
-    );
-
-    renderWithProviders(<VendorDetailDrawer open={true} vendor={vendor} onClose={vi.fn()} />, {
-      route: "/vendors",
-      withAuth: false,
-    });
-
-    expect(screen.getByText("0")).toBeInTheDocument();
-    expect(screen.queryByText("999-999-9999")).not.toBeInTheDocument();
-  });
-
-  it("does not expose a status control for a different technician", () => {
-    useVendorCompanyRoster.mockReturnValue(
-      rosterWith([
-        {
-          id: 7,
-          contactName: "Different Technician",
-          phone: "314-555-0101",
-          email: "",
-          tradeSpecialties: "Plumbing",
-          isActive: true,
-          totalJobs: 10,
-        },
-      ]),
-    );
-
-    renderWithProviders(
-      <VendorDetailDrawer open={true} vendor={vendor} startInEdit={true} onClose={vi.fn()} />,
-      { route: "/vendors", withAuth: false },
-    );
-
-    expect(screen.queryByRole("switch", { name: "Active status" })).not.toBeInTheDocument();
   });
 });

src/test/app/(protected)/vendors/vendors-list.test.tsx

@@ -1,14 +1,21 @@
-import { screen, within } from "@testing-library/react";
+import { screen } from "@testing-library/react";
 import userEvent from "@testing-library/user-event";
 import { describe, expect, it, vi } from "vitest";
 import VendorsListPage from "@/app/(protected)/vendors/index";
-import type { VendorCompanyRoster, VendorListItem } from "@/domain/vendors/types/vendor";
+import type { VendorListItem } from "@/domain/vendors/types/vendor";
 import { renderWithProviders } from "@/test/test-utils";
 
 const useVendorsList = vi.fn();
 const useVendorCompanyRoster = vi.fn();
 const useVendorDeactivationImpact = vi.fn();
 const mutate = vi.fn();
+const navigate = vi.fn();
+
+vi.mock("react-router", async () => {
+  const actual = await vi.importActual("react-router");
+  return { ...actual, useNavigate: () => navigate };
+});
+
 vi.mock("@/domain/vendors/use-cases/use-vendors-list", () => ({
   useVendorsList: (...args: unknown[]) => useVendorsList(...args),
 }));
@@ -54,32 +61,6 @@ const inactiveVendor = {
   isActive: false,
 };
 
-const activeRoster: VendorCompanyRoster = {
-  companyId: "co-1",
-  rowVersion: "rv-1",
-  name: "Gateway Plumbing",
-  companyPhone: "(314) 555-0100",
-  email: "dispatch@gatewayplumbing.com",
-  address: "",
-  city: "St. Louis",
-  state: "MO",
-  zip: "",
-  googleMapsUrl: "",
-  notes: "",
-  technicians: [
-    {
-      id: 1,
-      contactName: "Adam Whyte",
-      phone: "(314) 555-0198",
-      email: "dispatch@gatewayplumbing.com",
-      preferredContact: "Email",
-      tradeSpecialties: "Backflow Preventers",
-      isActive: true,
-      totalJobs: 0,
-    },
-  ],
-};
-
 function result(items: VendorListItem[], totalCount: number) {
   return {
     data: { items, totalCount, page: 1, pageSize: 12 },
@@ -90,13 +71,7 @@ function result(items: VendorListItem[], totalCount: number) {
 }
 
 function setupDefaults() {
-  useVendorCompanyRoster.mockReturnValue({
+  useVendorCompanyRoster.mockReturnValue({ data: undefined, isLoading: false, error: null });
-    data: undefined,
-    isLoading: false,
-    isError: false,
-    error: null,
-    refetch: vi.fn(),
-  });
   useVendorDeactivationImpact.mockReturnValue({
     data: undefined,
     isLoading: false,
@@ -105,7 +80,7 @@ function setupDefaults() {
 }
 
 describe("VendorsListPage", () => {
-  it("renders prototype-aligned status and two row actions in both tabs", async () => {
+  it("renders prototype-aligned status tabs and location, then hides deactivation for inactive rows", async () => {
     setupDefaults();
     useVendorsList.mockImplementation((params: { isActive?: boolean; pageSize?: number }) => {
       if (params.pageSize === 1) return result([], params.isActive ? 1 : 1);
@@ -116,20 +91,11 @@ describe("VendorsListPage", () => {
 
     expect(screen.getByRole("heading", { name: "Vendors / Technicians" })).toBeInTheDocument();
     expect(screen.getByText("St. Louis, MO")).toBeInTheDocument();
-    expect(screen.getByRole("columnheader", { name: "Status" })).toBeInTheDocument();
+    expect(screen.getByRole("button", { name: "Deactivate vendor" })).toBeInTheDocument();
-    expect(screen.getByRole("cell", { name: "Active" })).toBeInTheDocument();
-    expect(
-      screen.getByRole("button", { name: "View vendor Gateway Plumbing" }),
-    ).toBeInTheDocument();
-    expect(
-      screen.getByRole("button", { name: "Edit vendor Gateway Plumbing" }),
-    ).toBeInTheDocument();
-    expect(screen.queryByRole("button", { name: "Deactivate vendor" })).not.toBeInTheDocument();
 
     await userEvent.click(screen.getByRole("tab", { name: "Inactive (1)" }));
 
     expect(screen.getByText("Drain Cleaning Bros")).toBeInTheDocument();
-    expect(screen.getByRole("cell", { name: "Inactive" })).toBeInTheDocument();
     expect(screen.queryByRole("button", { name: "Deactivate vendor" })).not.toBeInTheDocument();
   });
 
@@ -148,18 +114,39 @@ describe("VendorsListPage", () => {
 
     renderWithProviders(<VendorsListPage />, { route: "/vendors", withAuth: false });
 
-    expect(screen.getByText(/25 active technicians/)).toBeInTheDocument();
+    expect(screen.getByText(/25 active vendor entries/)).toBeInTheDocument();
-    expect(screen.queryByText(/3 active technicians/)).not.toBeInTheDocument();
+    expect(screen.queryByText(/3 active vendor entries/)).not.toBeInTheDocument();
   });
 
   it("opens the detail drawer when a row is clicked or activated by keyboard", async () => {
     setupDefaults();
     useVendorCompanyRoster.mockReturnValue({
-      data: activeRoster,
+      data: {
+        companyId: "co-1",
+        rowVersion: "rv-1",
+        name: "Gateway Plumbing",
+        companyPhone: "(314) 555-0198",
+        email: "dispatch@gatewayplumbing.com",
+        address: "",
+        city: "St. Louis",
+        state: "MO",
+        zip: "",
+        googleMapsUrl: "",
+        notes: "",
+        technicians: [
+          {
+            id: 1,
+            contactName: "Adam Whyte",
+            phone: "(314) 555-0198",
+            email: "dispatch@gatewayplumbing.com",
+            preferredContact: "Email",
+            tradeSpecialties: "Backflow Preventers",
+            isActive: true,
+          },
+        ],
+      },
       isLoading: false,
-      isError: false,
       error: null,
-      refetch: vi.fn(),
     });
     useVendorsList.mockImplementation((params: { isActive?: boolean; pageSize?: number }) => {
       if (params.pageSize === 1) return result([], 1);
@@ -174,18 +161,11 @@ describe("VendorsListPage", () => {
     row.focus();
     await userEvent.keyboard("{Enter}");
 
-    expect(screen.getByRole("heading", { level: 2, name: "Adam Whyte" })).toBeInTheDocument();
+    expect(screen.getByRole("heading", { level: 2, name: "Gateway Plumbing" })).toBeInTheDocument();
   });
 
   it("blocks deactivation when the preflight reports open work orders", async () => {
     setupDefaults();
-    useVendorCompanyRoster.mockReturnValue({
-      data: activeRoster,
-      isLoading: false,
-      isError: false,
-      error: null,
-      refetch: vi.fn(),
-    });
     useVendorDeactivationImpact.mockReturnValue({
       data: {
         vendorId: 1,
@@ -209,8 +189,7 @@ describe("VendorsListPage", () => {
 
     renderWithProviders(<VendorsListPage />, { route: "/vendors", withAuth: false });
 
-    await userEvent.click(screen.getByRole("button", { name: "Edit vendor Gateway Plumbing" }));
+    await userEvent.click(screen.getByRole("button", { name: "Deactivate vendor" }));
-    await userEvent.click(screen.getByRole("switch", { name: "Active status" }));
 
     expect(
       screen.getByText(/cannot be deactivated because it still has open work orders/),
@@ -220,51 +199,9 @@ describe("VendorsListPage", () => {
     expect(mutate).not.toHaveBeenCalled();
   });
 
-  it("preserves inline edits when deactivation is cancelled", async () => {
+  it("navigates to the vendor edit route when the table Edit control is clicked", async () => {
     setupDefaults();
-    useVendorCompanyRoster.mockReturnValue({
+    navigate.mockClear();
-      data: activeRoster,
-      isLoading: false,
-      isError: false,
-      error: null,
-      refetch: vi.fn(),
-    });
-    useVendorDeactivationImpact.mockReturnValue({
-      data: { vendorId: 1, canDeactivate: true, openWorkOrders: [] },
-      isLoading: false,
-      error: null,
-    });
-    useVendorsList.mockImplementation((params: { isActive?: boolean; pageSize?: number }) => {
-      if (params.pageSize === 1) return result([], 1);
-      return params.isActive ? result([activeVendor], 1) : result([], 0);
-    });
-
-    renderWithProviders(<VendorsListPage />, { route: "/vendors", withAuth: false });
-
-    await userEvent.click(screen.getByRole("button", { name: "Edit vendor Gateway Plumbing" }));
-    const company = screen.getByRole("textbox", { name: "Company" });
-    await userEvent.clear(company);
-    await userEvent.type(company, "Draft Company Name");
-    await userEvent.click(screen.getByRole("switch", { name: "Active status" }));
-    await userEvent.click(
-      within(screen.getByRole("dialog", { name: "Deactivate Vendor" })).getByRole("button", {
-        name: "Cancel",
-      }),
-    );
-
-    expect(company).toBeInTheDocument();
-    expect(company).toHaveValue("Draft Company Name");
-  });
-
-  it("opens inline edit when the table Edit control is clicked", async () => {
-    setupDefaults();
-    useVendorCompanyRoster.mockReturnValue({
-      data: activeRoster,
-      isLoading: false,
-      isError: false,
-      error: null,
-      refetch: vi.fn(),
-    });
     useVendorsList.mockImplementation((params: { isActive?: boolean; pageSize?: number }) => {
       if (params.pageSize === 1) return result([], 1);
       return params.isActive ? result([activeVendor], 1) : result([], 0);
@@ -274,20 +211,16 @@ describe("VendorsListPage", () => {
 
     await userEvent.click(screen.getByRole("button", { name: "Edit vendor Gateway Plumbing" }));
 
-    expect(screen.getByRole("button", { name: "Save changes" })).toBeInTheDocument();
+    expect(navigate).toHaveBeenCalledWith("/vendors/edit/1");
-    expect(screen.getByRole("textbox", { name: "Company" })).toHaveValue("Gateway Plumbing");
+    expect(
+      screen.queryByRole("heading", { level: 2, name: "Gateway Plumbing" }),
+    ).not.toBeInTheDocument();
   });
 
-  it("opens inline company edit when the row has no vendor id", async () => {
+  it("navigates to the company route when editing a vendor without a vendor id", async () => {
     setupDefaults();
+    navigate.mockClear();
     const companyOnlyVendor = { ...activeVendor, id: null, companyId: "co-9" };
-    useVendorCompanyRoster.mockReturnValue({
-      data: { ...activeRoster, companyId: "co-9", technicians: [] },
-      isLoading: false,
-      isError: false,
-      error: null,
-      refetch: vi.fn(),
-    });
     useVendorsList.mockImplementation((params: { isActive?: boolean; pageSize?: number }) => {
       if (params.pageSize === 1) return result([], 1);
       return params.isActive ? result([companyOnlyVendor], 1) : result([], 0);
@@ -297,7 +230,6 @@ describe("VendorsListPage", () => {
 
     await userEvent.click(screen.getByRole("button", { name: "Edit vendor Gateway Plumbing" }));
 
-    expect(screen.getByRole("button", { name: "Save changes" })).toBeInTheDocument();
+    expect(navigate).toHaveBeenCalledWith("/vendors/company/co-9");
-    expect(screen.getByRole("textbox", { name: "Company" })).toHaveValue("Gateway Plumbing");
   });
 });

src/test/app/(protected)/workorders/work-orders-board-core-affordances.test.tsx

@@ -75,6 +75,7 @@ const canceledRow: WorkOrderTableRow = {
   dispatcherId: "u1",
   dispatcherName: "Test Dispatcher",
   scheduledOn: "2026-07-20",
+  completedDate: "",
   targetWeek: "2026-07-20",
   dayGroup: "2026-07-20",
   dayLabel: "Mon",

src/test/domain/work-orders/mappers/completed-date-board-column.test.ts

@@ -0,0 +1,26 @@
+import { describe, expect, it } from "vitest";
+import { WO_TABLE_COLS } from "@/app/(protected)/workorders/_components/list/table/wo-table-cols";
+import { expandScalarBoardOps } from "@/domain/work-orders/mappers/expand-scalar-board-ops";
+import { mapWorkOrderTableRow } from "@/domain/work-orders/mappers/work-order-table-row-mapper";
+
+describe("COMPLETED DATE board column wiring", () => {
+  it("includes COMPLETED DATE column after COMP DOC", () => {
+    const keys = WO_TABLE_COLS.map((col) => col.key);
+    expect(keys).toContain("completed");
+    expect(keys.indexOf("completed")).toBeGreaterThan(keys.indexOf("doc"));
+  });
+
+  it("expands completedDate patch to board field op", () => {
+    const ops = expandScalarBoardOps({ completedDate: "2026-07-30" });
+    expect(ops).toEqual([{ field: "completedDate", value: "2026-07-30" }]);
+  });
+
+  it("maps completedDate returned by a board GET row", () => {
+    const row = mapWorkOrderTableRow({
+      id: 42,
+      completedDate: "2026-07-30T00:00:00Z",
+    });
+
+    expect(row.completedDate).toBe("2026-07-30");
+  });
+});

src/test/domain/work-orders/utils/slide-over-draft.test.ts

@@ -15,6 +15,7 @@ function baseRow(overrides: Partial<WorkOrderTableRow> = {}): WorkOrderTableRow
     dispatcherId: "10",
     dispatcherName: "Alex",
     scheduledOn: "2026-07-16",
+    completedDate: "",
     targetWeek: "",
     dayGroup: "2026-07-16",
     dayLabel: "Thu",

src/test/domain/work-orders/utils/table-filter-utils.test.ts

@@ -19,6 +19,7 @@ function makeRow(overrides: Partial<WorkOrderTableRow> = {}): WorkOrderTableRow
     dispatcherId: "u1",
     dispatcherName: "Alice",
     scheduledOn: "2026-06-23",
+    completedDate: "",
     targetWeek: "",
     dayGroup: "06/23",
     dayLabel: "TUE · 06/23",

tmp/pr-descriptions/pr1-completed-date.md

@@ -0,0 +1,35 @@
+## Summary
+
+- Adds an editable COMPLETED DATE column to the Work Orders board table (after COMP DOC).
+- Wires `completedDate` through board row typing, table-row mapping, and PATCH `field: "completedDate"`.
+- Inline create keeps an empty placeholder (date is not set at creation).
+
+## Scope
+
+### In
+
+- `completedDate` on `WorkOrderBoardRowApi` / `WorkOrderTableRow`
+- Column + `DateCell` on board rows
+- Scalar board patch expansion for `completedDate`
+- Unit test for column order and patch op
+
+### Out (follow-up PRs)
+
+- Comp Doc dialog changes, closability gates, uplift, severity, filters, wizard
+
+## API / BE
+
+- Contract assumed: PATCH `/workorders/{id}/board` with `field: "completedDate"` already supported
+- Confirm GET `/board` and `/board/search` rows include `completedDate` (see `docs/work-orders/pending-be-contract.md`)
+- Blocked on BE: none for FE wiring (tolerates missing GET field as empty string)
+
+## Test plan
+
+- [ ] Board shows COMPLETED DATE column after COMP DOC
+- [ ] Editing the cell PATCHes `completedDate`
+- [ ] Inline create row shows placeholder for completed date
+- [ ] `npm run verify` green locally
+
+## Base
+
+- Base branch: `dev`