@optifye/dashboard-core 6.9.2 → 6.9.4

package/dist/index.js

@@ -2088,6 +2088,59 @@ var workspaceService = {
       console.error("Error updating shift configuration:", error);
       throw error;
     }
+  },
+  /**
+   * Fetch bulk targets data for multiple lines and shifts in a single API call
+   * This replaces 120+ individual API calls with 1 optimized call
+   *
+   * @param params - Parameters for the bulk fetch
+   * @returns Promise with complete targets data for all lines and shifts
+   */
+  async fetchBulkTargets(params) {
+    try {
+      const token = await getAuthToken();
+      const apiUrl = getBackendUrl();
+      const {
+        companyId,
+        lineIds,
+        date,
+        shifts = [0, 1],
+        includeSkus = true,
+        includeActions = true
+      } = params;
+      const queryParams = new URLSearchParams({
+        company_id: companyId,
+        line_ids: lineIds.join(","),
+        date,
+        shifts: shifts.join(","),
+        include_skus: includeSkus.toString(),
+        include_actions: includeActions.toString()
+      });
+      const response = await fetch(
+        `${apiUrl}/api/targets/bulk-load?${queryParams}`,
+        {
+          method: "GET",
+          headers: {
+            "Authorization": `Bearer ${token}`,
+            "Content-Type": "application/json"
+          }
+        }
+      );
+      if (!response.ok) {
+        const errorText = await response.text();
+        throw new Error(`Backend API error (${response.status}): ${errorText}`);
+      }
+      const data = await response.json();
+      console.log("[fetchBulkTargets] Success:", {
+        lineCount: data.metadata?.line_count,
+        totalWorkspaces: data.metadata?.total_workspaces,
+        timestamp: data.metadata?.timestamp
+      });
+      return data;
+    } catch (error) {
+      console.error("Error fetching bulk targets:", error);
+      throw error;
+    }
   }
 };
 var WorkspaceHealthService = class _WorkspaceHealthService {
@@ -5822,60 +5875,32 @@ var UserManagementService = class {
     }
   }
   /**
-   * Deactivate a user
-   * @param input - Deactivation input
+   * Permanently delete a user
+   * @param userId - The ID of the user to delete
+   * @returns Promise with deletion response
    */
-  async deactivateUser(input) {
+  async deleteUser(userId) {
     try {
       const token = await this.getAuthToken();
       const backendUrl = this.getBackendUrl();
       const response = await fetch(
-        `${backendUrl}/api/users/deactivate`,
-        {
-          method: "POST",
-          headers: {
-            "Authorization": `Bearer ${token}`,
-            "Content-Type": "application/json"
-          },
-          body: JSON.stringify(input)
-        }
-      );
-      if (!response.ok) {
-        const errorData = await response.json();
-        throw new Error(`Failed to deactivate user: ${errorData.detail || response.statusText}`);
-      }
-      console.log("[UserManagementService] User deactivated successfully");
-    } catch (error) {
-      console.error("[UserManagementService] Error deactivating user:", error);
-      throw error;
-    }
-  }
-  /**
-   * Reactivate a user
-   * @param input - Reactivation input
-   */
-  async reactivateUser(input) {
-    try {
-      const token = await this.getAuthToken();
-      const backendUrl = this.getBackendUrl();
-      const response = await fetch(
-        `${backendUrl}/api/users/reactivate`,
+        `${backendUrl}/api/users/${userId}`,
         {
-          method: "POST",
+          method: "DELETE",
           headers: {
-            "Authorization": `Bearer ${token}`,
-            "Content-Type": "application/json"
-          },
-          body: JSON.stringify(input)
+            "Authorization": `Bearer ${token}`
+          }
         }
       );
       if (!response.ok) {
         const errorData = await response.json();
-        throw new Error(`Failed to reactivate user: ${errorData.detail || response.statusText}`);
+        throw new Error(`Failed to delete user: ${errorData.detail || response.statusText}`);
       }
-      console.log("[UserManagementService] User reactivated successfully");
+      const result = await response.json();
+      console.log("[UserManagementService] User permanently deleted:", result);
+      return result;
     } catch (error) {
-      console.error("[UserManagementService] Error reactivating user:", error);
+      console.error("[UserManagementService] Error deleting user:", error);
       throw error;
     }
   }
@@ -7688,7 +7713,7 @@ var useLeaderboardMetrics = (date, shiftId, limit = 10, filter2 = "all") => {
     refetch: fetchLeaderboard
   };
 };
-var useDashboardMetrics = ({ onLineMetricsUpdate, lineId }) => {
+var useDashboardMetrics = ({ onLineMetricsUpdate, lineId, userAccessibleLineIds }) => {
   const { supabaseUrl, supabaseKey } = useDashboardConfig();
   const entityConfig = useEntityConfig();
   const databaseConfig = useDatabaseConfig();
@@ -7733,7 +7758,7 @@ var useDashboardMetrics = ({ onLineMetricsUpdate, lineId }) => {
     try {
       const currentShiftDetails = getCurrentShift(defaultTimezone, shiftConfig);
       const operationalDate = getOperationalDate(defaultTimezone);
-      const targetLineIds = currentLineIdToUse === (entityConfig.factoryViewId || "factory") ? getConfiguredLineIds(entityConfig) : [currentLineIdToUse];
+      const targetLineIds = currentLineIdToUse === (entityConfig.factoryViewId || "factory") ? userAccessibleLineIds || getConfiguredLineIds(entityConfig) : [currentLineIdToUse];
       if (targetLineIds.length === 0 && currentLineIdToUse === (entityConfig.factoryViewId || "factory")) {
         throw new Error("Factory view selected, but no lines are configured in entityConfig.");
       }
@@ -7859,7 +7884,7 @@ var useDashboardMetrics = ({ onLineMetricsUpdate, lineId }) => {
     }
     const currentShiftDetails = getCurrentShift(defaultTimezone, shiftConfig);
     const operationalDateForSubscription = getOperationalDate(defaultTimezone);
-    const targetLineIds = currentLineIdToUse === (entityConfig.factoryViewId || "factory") ? getConfiguredLineIds(entityConfig) : [currentLineIdToUse];
+    const targetLineIds = currentLineIdToUse === (entityConfig.factoryViewId || "factory") ? userAccessibleLineIds || getConfiguredLineIds(entityConfig) : [currentLineIdToUse];
     if (targetLineIds.length === 0) return;
     const wsMetricsFilter = `line_id=in.(${targetLineIds.map((id3) => `"${id3}"`).join(",")})`;
     const lineMetricsFilter = `line_id=in.(${targetLineIds.map((id3) => `"${id3}"`).join(",")})`;
@@ -7897,7 +7922,8 @@ var useDashboardMetrics = ({ onLineMetricsUpdate, lineId }) => {
     entityConfig?.companyId,
     entityConfig?.factoryViewId,
     defaultTimezone,
-    lineId
+    lineId,
+    userAccessibleLineIds
   ]);
   return {
     workspaceMetrics: metrics2?.workspaceMetrics || [],
@@ -9887,7 +9913,8 @@ var useAllWorkspaceMetrics = (options) => {
     }
     setError(null);
     try {
-      const configuredLineIds = getConfiguredLineIds(entityConfig);
+      const allConfiguredLineIds = getConfiguredLineIds(entityConfig);
+      const configuredLineIds = options?.allowedLineIds ? allConfiguredLineIds.filter((id3) => options.allowedLineIds.includes(id3)) : allConfiguredLineIds;
       let enabledWorkspaceIds = [];
       for (const lineId of configuredLineIds) {
         const workspaces2 = await workspaceService.getWorkspaces(lineId);
@@ -9950,7 +9977,7 @@ var useAllWorkspaceMetrics = (options) => {
       setLoading(false);
       isFetchingRef.current = false;
     }
-  }, [queryDate, queryShiftId, metricsTable, supabase, entityConfig.companyId]);
+  }, [queryDate, queryShiftId, metricsTable, supabase, entityConfig.companyId, entityConfig, options?.allowedLineIds]);
   React23.useEffect(() => {
     if (!initialized) {
       fetchWorkspaceMetrics();
@@ -41449,7 +41476,6 @@ function HomeView({
   factoryName = "Simba Beer - Line 1"
 }) {
   const [isHydrated, setIsHydrated] = React23.useState(false);
-  const availableLineIds = React23.useMemo(() => [factoryViewId, ...allLineIds], [factoryViewId, allLineIds]);
   const [selectedLineId, setSelectedLineId] = React23.useState(factoryViewId);
   const [isChangingFilter, setIsChangingFilter] = React23.useState(false);
   const [errorMessage, setErrorMessage] = React23.useState(null);
@@ -41459,6 +41485,21 @@ function HomeView({
   const entityConfig = useEntityConfig();
   const supabaseClient = useSupabaseClient();
   const { user } = useAuth();
+  const isSupervisor = user?.role_level === "supervisor";
+  const hasMultipleLines = allLineIds.length > 1;
+  const availableLineIds = React23.useMemo(() => {
+    if (isSupervisor && !hasMultipleLines) {
+      return allLineIds;
+    }
+    return [factoryViewId, ...allLineIds];
+  }, [factoryViewId, allLineIds, isSupervisor, hasMultipleLines]);
+  React23.useEffect(() => {
+    if (user) {
+      if (isSupervisor && allLineIds.length > 0) {
+        setSelectedLineId(allLineIds[0]);
+      }
+    }
+  }, [user, isSupervisor, allLineIds]);
   const userCompanyId = React23.useMemo(() => {
     return user?.properties?.company_id || user?.company_id || entityConfig.companyId;
   }, [user, entityConfig.companyId]);
@@ -41519,7 +41560,9 @@ function HomeView({
     refetch: refetchMetrics
   } = useDashboardMetrics({
     lineId: selectedLineId,
-    onLineMetricsUpdate
+    onLineMetricsUpdate,
+    userAccessibleLineIds: allLineIds
+    // Pass user's accessible lines for supervisor filtering
   });
   const {
     activeBreaks: allActiveBreaks,
@@ -43578,7 +43621,8 @@ var LeaderboardDetailView = React23.memo(({
   line1Id = "",
   line2Id = "",
   lineNames = {},
-  className = ""
+  className = "",
+  userAccessibleLineIds
 }) => {
   const navigation = useNavigation();
   const entityConfig = useEntityConfig();
@@ -43618,7 +43662,9 @@ var LeaderboardDetailView = React23.memo(({
     refreshWorkspaces
   } = useAllWorkspaceMetrics({
     initialDate: date,
-    initialShiftId: typeof shift === "number" ? shift : typeof shift === "string" ? parseInt(shift) : void 0
+    initialShiftId: typeof shift === "number" ? shift : typeof shift === "string" ? parseInt(shift) : void 0,
+    allowedLineIds: userAccessibleLineIds
+    // Filter to user's accessible lines only
   });
   const getShiftName = React23.useCallback((shiftId2) => {
     if (shiftId2 === void 0) return "Day";
@@ -44974,12 +45020,17 @@ var ACTION_NAMES = {
 var calculatePPH = (cycleTime, breaks = [], shiftHours = 0) => {
   if (cycleTime === "" || cycleTime === 0) return "";
   const pph = 3600 / cycleTime;
-  return Number(pph.toFixed(1));
+  return Math.round(pph);
 };
 var calculateDayOutput = (pph, shiftHours, breaks = []) => {
   if (pph === "") return "";
   return Math.round(pph * shiftHours);
 };
+var calculateDayOutputFromCycleTime = (cycleTime, shiftHours) => {
+  if (cycleTime === "" || cycleTime === 0 || shiftHours === 0) return "";
+  const dayOutput = 3600 / cycleTime * shiftHours;
+  return Math.round(dayOutput);
+};
 var formatWorkspaceName = (name, lineId) => {
   return getWorkspaceDisplayName(name, lineId);
 };
@@ -44991,20 +45042,6 @@ var getStoredLineState2 = (lineId) => {
     return false;
   }
 };
-var calculateShiftHours2 = (startTime, endTime, breaks = []) => {
-  if (!startTime || !endTime) return 8;
-  const [startHour, startMinute] = startTime.split(":").map(Number);
-  const [endHour, endMinute] = endTime.split(":").map(Number);
-  let startMinutes = startHour * 60 + startMinute;
-  let endMinutes = endHour * 60 + endMinute;
-  if (endMinutes < startMinutes) {
-    endMinutes += 24 * 60;
-  }
-  const safeBreaks = Array.isArray(breaks) ? breaks : [];
-  const totalBreakMinutes = safeBreaks.reduce((total, breakItem) => total + breakItem.duration, 0);
-  const hoursDiff = (endMinutes - startMinutes - totalBreakMinutes) / 60;
-  return Number(hoursDiff.toFixed(1));
-};
 var BulkConfigureModal = ({
   isOpen,
   onClose,
@@ -46212,121 +46249,6 @@ var TargetsView = ({
   const dashboardConfig = useDashboardConfig();
   const { skus, isLoading: skusLoading } = useSKUs(companyId);
   const skuEnabled = dashboardConfig?.skuConfig?.enabled || false;
-  const loadOperatingHours = React23.useCallback(async (lineId, shiftId) => {
-    try {
-      const { data: { session } } = await supabase.auth.getSession();
-      if (!session?.access_token) {
-        console.error("No authentication token available");
-        return {
-          startTime: "08:00",
-          // Default values
-          endTime: "19:00",
-          breaks: []
-        };
-      }
-      const backendUrl = process.env.NEXT_PUBLIC_BACKEND_URL;
-      if (!backendUrl) {
-        console.error("Backend URL is not configured");
-        return {
-          startTime: "08:00",
-          // Default values
-          endTime: "19:00",
-          breaks: []
-        };
-      }
-      const response = await fetch(`${backendUrl}/api/lines/${lineId}/operating-hours?shift_id=${shiftId}`, {
-        headers: {
-          "Authorization": `Bearer ${session.access_token}`,
-          "Content-Type": "application/json"
-        }
-      });
-      if (!response.ok) {
-        console.log(`No operating hours found for line ${lineId}, shift ${shiftId} - using defaults`);
-        return {
-          startTime: "08:00",
-          // Default values
-          endTime: "19:00",
-          breaks: []
-        };
-      }
-      const data = await response.json();
-      let breaks = [];
-      if (data?.breaks) {
-        if (Array.isArray(data.breaks)) {
-          breaks = data.breaks.map((breakItem) => {
-            const startTime = breakItem.start || breakItem.startTime || "00:00";
-            const endTime = breakItem.end || breakItem.endTime || "00:00";
-            const calculateDuration = (start, end) => {
-              const [startHour, startMinute] = start.split(":").map(Number);
-              const [endHour, endMinute] = end.split(":").map(Number);
-              let startMinutes = startHour * 60 + startMinute;
-              let endMinutes = endHour * 60 + endMinute;
-              if (endMinutes < startMinutes) {
-                endMinutes += 24 * 60;
-              }
-              return endMinutes - startMinutes;
-            };
-            return {
-              startTime,
-              endTime,
-              duration: breakItem.duration || calculateDuration(startTime, endTime)
-            };
-          });
-        } else if (typeof data.breaks === "object" && data.breaks.breaks) {
-          breaks = data.breaks.breaks.map((breakItem) => {
-            const startTime = breakItem.start || breakItem.startTime || "00:00";
-            const endTime = breakItem.end || breakItem.endTime || "00:00";
-            const calculateDuration = (start, end) => {
-              const [startHour, startMinute] = start.split(":").map(Number);
-              const [endHour, endMinute] = end.split(":").map(Number);
-              let startMinutes = startHour * 60 + startMinute;
-              let endMinutes = endHour * 60 + endMinute;
-              if (endMinutes < startMinutes) {
-                endMinutes += 24 * 60;
-              }
-              return endMinutes - startMinutes;
-            };
-            return {
-              startTime,
-              endTime,
-              duration: breakItem.duration || calculateDuration(startTime, endTime)
-            };
-          });
-        } else if (typeof data.breaks === "string") {
-          try {
-            const parsedBreaks = JSON.parse(data.breaks);
-            if (Array.isArray(parsedBreaks)) {
-              breaks = parsedBreaks.map((breakItem) => ({
-                startTime: breakItem.start || breakItem.startTime || "00:00",
-                endTime: breakItem.end || breakItem.endTime || "00:00",
-                duration: breakItem.duration || 0
-              }));
-            } else if (parsedBreaks.breaks && Array.isArray(parsedBreaks.breaks)) {
-              breaks = parsedBreaks.breaks.map((breakItem) => ({
-                startTime: breakItem.start || breakItem.startTime || "00:00",
-                endTime: breakItem.end || breakItem.endTime || "00:00",
-                duration: breakItem.duration || 0
-              }));
-            }
-          } catch (e) {
-            console.error("Error parsing breaks data:", e);
-          }
-        }
-      }
-      return {
-        startTime: data?.start_time || "08:00",
-        endTime: data?.end_time || "19:00",
-        breaks
-      };
-    } catch (e) {
-      console.error("Exception when loading operating hours:", e);
-      return {
-        startTime: "08:00",
-        endTime: "19:00",
-        breaks: []
-      };
-    }
-  }, [supabase]);
   React23.useEffect(() => {
     console.log("[TargetsView] Component mounted/re-rendered", {
       timestamp: (/* @__PURE__ */ new Date()).toISOString(),
@@ -46338,221 +46260,108 @@ var TargetsView = ({
     };
   }, []);
   React23.useEffect(() => {
-    let timeoutId;
-    let retryCount = 0;
-    const MAX_RETRIES2 = 2;
-    const LOADING_TIMEOUT = 15e3;
     const fetchInitialData = async () => {
-      if (lineIds.length === 0) return;
-      console.log("[TargetsView] Starting fetchInitialData");
+      if (lineIds.length === 0 || !companyId) return;
+      console.log("[TargetsView] Starting optimized fetchInitialData with bulk endpoint");
       setIsLoading(true);
-      timeoutId = setTimeout(() => {
-        console.error("Loading timeout reached");
-        if (retryCount < MAX_RETRIES2) {
-          retryCount++;
-          console.log(`Retrying... (attempt ${retryCount + 1}/${MAX_RETRIES2 + 1})`);
-          sonner.toast.warning("Loading is taking longer than expected. Retrying...");
-          fetchInitialData();
-        } else {
-          setIsLoading(false);
-          sonner.toast.error("Failed to load data. Please refresh the page.");
-        }
-      }, LOADING_TIMEOUT);
       try {
-        const { data: { session } } = await supabase.auth.getSession();
-        if (!session?.access_token) {
-          throw new Error("No authentication token available");
-        }
-        const backendUrl = process.env.NEXT_PUBLIC_BACKEND_URL;
-        if (!backendUrl) {
-          throw new Error("Backend URL is not configured");
+        const currentDate = getOperationalDate(timezone);
+        const bulkResponse = await workspaceService.fetchBulkTargets({
+          companyId,
+          lineIds,
+          date: currentDate,
+          shifts: [0, 1],
+          // Fetch both shifts at once
+          includeSkus: skuEnabled,
+          includeActions: true
+        });
+        if (!bulkResponse.success) {
+          throw new Error("Failed to fetch bulk targets data");
         }
-        const [linesResponse, actions] = await Promise.all([
-          // Fetch lines data (includes factory_id)
-          Promise.all(lineIds.map(async (lineId) => {
-            try {
-              const response = await fetch(`${backendUrl}/api/lines/${lineId}`, {
-                headers: {
-                  "Authorization": `Bearer ${session.access_token}`,
-                  "Content-Type": "application/json"
-                }
-              });
-              if (!response.ok) {
-                console.error(`Error fetching line data for ${lineId}:`, response.statusText);
-                return { lineId, factoryId: void 0 };
-              }
-              const lineData = await response.json();
-              return { lineId, factoryId: lineData.factory_id };
-            } catch (err) {
-              console.error(`Exception fetching line data for ${lineId}:`, err);
-              return { lineId, factoryId: void 0 };
-            }
-          })),
-          // Fetch action IDs
-          actionService.getActionsByName(
-            [ACTION_NAMES.ASSEMBLY, ACTION_NAMES.PACKAGING],
-            companyId
-          )
-        ]);
-        const factoryResults = linesResponse;
-        const assemblyAction = actions.find((a) => a.action_name === ACTION_NAMES.ASSEMBLY);
-        const packagingAction = actions.find((a) => a.action_name === ACTION_NAMES.PACKAGING);
+        const { data } = bulkResponse;
+        const assemblyAction = Object.values(data.actions).find((a) => a.action_name === ACTION_NAMES.ASSEMBLY);
+        const packagingAction = Object.values(data.actions).find((a) => a.action_name === ACTION_NAMES.PACKAGING);
         if (!assemblyAction || !packagingAction) {
-          throw new Error("Could not find required actions");
+          throw new Error("Could not find required actions in bulk response");
         }
         const actionIdsData = {
           assembly: assemblyAction.id,
           packaging: packagingAction.id
         };
         setActionIds(actionIdsData);
-        const updatedLineWorkspaces = { ...initialLineWorkspaces };
-        factoryResults.forEach((result) => {
-          if (result.factoryId && updatedLineWorkspaces[result.lineId]) {
-            updatedLineWorkspaces[result.lineId].factoryId = result.factoryId;
-          }
-        });
-        const currentDate = getOperationalDate(timezone);
-        for (const lineId of lineIds) {
-          if (!updatedLineWorkspaces[lineId]?.factoryId) {
-            console.warn(`Skipping workspace fetch for line ${lineId} - no factory ID`);
-            continue;
-          }
-          try {
-            const workspacesData = await workspaceService.getWorkspaces(lineId);
-            const enabledWorkspaces = workspacesData.filter((ws) => ws.enable === true);
-            const actionThresholds = await workspaceService.getActionThresholds(
-              lineId,
-              currentDate,
-              0
-              // Always use day shift for initial load
-            );
-            const operatingHoursData = await loadOperatingHours(lineId, 0);
-            if (operatingHoursData) {
-              updatedLineWorkspaces[lineId].shiftStartTime = operatingHoursData.startTime;
-              updatedLineWorkspaces[lineId].shiftEndTime = operatingHoursData.endTime;
-              updatedLineWorkspaces[lineId].breaks = operatingHoursData.breaks;
-              updatedLineWorkspaces[lineId].shiftHours = calculateShiftHours2(
-                operatingHoursData.startTime,
-                operatingHoursData.endTime,
-                operatingHoursData.breaks
-              );
+        const newAllShiftsData = { 0: {}, 1: {} };
+        const newDbValues = { 0: {}, 1: {} };
+        Object.entries(data.lines).forEach(([lineId, lineData]) => {
+          [0, 1].forEach((shiftId) => {
+            const shiftData = lineData.shifts[shiftId.toString()];
+            if (!shiftData) {
+              console.warn(`No shift ${shiftId} data for line ${lineId}`);
+              return;
             }
+            const operatingHours = shiftData.operating_hours || {
+              start_time: "08:00",
+              end_time: "19:00",
+              breaks: [],
+              total_hours: 11
+            };
+            newAllShiftsData[shiftId][lineId] = {
+              productId: "",
+              shiftStartTime: operatingHours.start_time,
+              shiftEndTime: operatingHours.end_time,
+              breaks: operatingHours.breaks,
+              shiftHours: operatingHours.total_hours,
+              workspaces: [],
+              factoryId: lineData.line_info.factory_id
+            };
+            newDbValues[shiftId][lineId] = {};
+            const enabledWorkspaces = lineData.workspaces.filter((ws) => ws.enable === true);
             const mappedWorkspaces = enabledWorkspaces.map((ws) => {
-              const threshold = actionThresholds.find((t) => t.workspace_id === ws.id);
-              if (!dbValues[0][lineId]) {
-                dbValues[0][lineId] = {};
-              }
+              const threshold = shiftData.thresholds.find((t) => t.workspace_id === ws.id);
               if (threshold) {
-                dbValues[0][lineId][ws.id] = {
-                  targetPPH: threshold.pph_threshold,
-                  targetCycleTime: threshold.ideal_cycle_time,
-                  targetDayOutput: threshold.total_day_output
+                newDbValues[shiftId][lineId][ws.id] = {
+                  targetPPH: threshold.pph_threshold ? Math.round(threshold.pph_threshold) : "",
+                  targetCycleTime: threshold.ideal_cycle_time ?? "",
+                  targetDayOutput: threshold.total_day_output ?? ""
                 };
               }
+              let actionType = "assembly";
+              let actionId = actionIdsData.assembly;
+              if (ws.action_id === packagingAction.id || ws.action_type === "packaging") {
+                actionType = "packaging";
+                actionId = packagingAction.id;
+              } else if (ws.action_id === assemblyAction.id || ws.action_type === "assembly") {
+                actionType = "assembly";
+                actionId = assemblyAction.id;
+              }
               return {
                 id: ws.id,
                 name: ws.workspace_id,
-                targetPPH: threshold?.pph_threshold ?? (ws.action_pph_threshold === null ? "" : ws.action_pph_threshold),
-                targetCycleTime: threshold?.ideal_cycle_time ?? (ws.action_cycle_time === null ? "" : ws.action_cycle_time),
-                targetDayOutput: threshold?.total_day_output ?? (ws.action_total_day_output === null ? "" : ws.action_total_day_output),
-                actionType: ws.action_id === actionIdsData.assembly ? "assembly" : ws.action_id === actionIdsData.packaging ? "packaging" : "assembly",
-                actionId: ws.action_id === actionIdsData.assembly ? actionIdsData.assembly : ws.action_id === actionIdsData.packaging ? actionIdsData.packaging : actionIdsData.assembly
+                targetPPH: threshold?.pph_threshold ? Math.round(threshold.pph_threshold) : "",
+                targetCycleTime: threshold?.ideal_cycle_time ?? "",
+                targetDayOutput: threshold?.total_day_output ?? "",
+                actionType,
+                actionId
               };
             }).sort((a, b) => a.name.localeCompare(b.name, void 0, { numeric: true }));
-            updatedLineWorkspaces[lineId].workspaces = mappedWorkspaces;
-          } catch (error) {
-            console.error(`Error fetching workspace data for line ${lineId}:`, error);
-          }
-        }
-        setLineWorkspaces(updatedLineWorkspaces);
-        await fetchAllShiftsData(updatedLineWorkspaces);
+            newAllShiftsData[shiftId][lineId].workspaces = mappedWorkspaces;
+          });
+        });
+        setAllShiftsData(newAllShiftsData);
+        setDbValues(newDbValues);
+        console.log("[TargetsView] Successfully loaded all data with bulk endpoint:", {
+          lineCount: bulkResponse.metadata.line_count,
+          totalWorkspaces: bulkResponse.metadata.total_workspaces,
+          loadTime: "Fast!"
+        });
       } catch (error) {
-        console.error("Error fetching initial data:", error);
-        clearTimeout(timeoutId);
-        if (retryCount < MAX_RETRIES2) {
-          retryCount++;
-          console.log(`Error occurred, retrying... (attempt ${retryCount + 1}/${MAX_RETRIES2 + 1})`);
-          sonner.toast.warning("Error loading data. Retrying...");
-          setTimeout(() => fetchInitialData(), 1e3);
-        } else {
-          sonner.toast.error("Failed to load initial data");
-          setIsLoading(false);
-        }
+        console.error("[TargetsView] Error fetching bulk data:", error);
+        sonner.toast.error("Failed to load targets data. Please refresh the page.");
       } finally {
-        clearTimeout(timeoutId);
-        if (retryCount === 0 || retryCount >= MAX_RETRIES2) {
-          setIsLoading(false);
-        }
+        setIsLoading(false);
       }
     };
     fetchInitialData();
-    return () => {
-      clearTimeout(timeoutId);
-    };
-  }, [lineIds, companyId, loadOperatingHours]);
-  const fetchAllShiftsData = React23.useCallback(async (currentWorkspaces) => {
-    if (!supabase) return;
-    const currentDate = getOperationalDate(timezone);
-    const newAllShiftsData = {
-      0: JSON.parse(JSON.stringify(currentWorkspaces)),
-      // Deep clone for day shift
-      1: JSON.parse(JSON.stringify(currentWorkspaces))
-      // Deep clone for night shift
-    };
-    const newDbValues = { 0: {}, 1: {} };
-    for (const shiftId of [0, 1]) {
-      for (const lineId of lineIds) {
-        try {
-          const operatingHoursData = await loadOperatingHours(lineId, shiftId);
-          if (!operatingHoursData) {
-            console.warn(`No operating hours for line ${lineId}, shift ${shiftId} - using defaults`);
-            continue;
-          }
-          const { startTime, endTime, breaks } = operatingHoursData;
-          const shiftHours = calculateShiftHours2(startTime, endTime, breaks);
-          const actionThresholds = await workspaceService.getActionThresholds(
-            lineId,
-            currentDate,
-            shiftId
-          );
-          if (!newDbValues[shiftId][lineId]) {
-            newDbValues[shiftId][lineId] = {};
-          }
-          const existingLine = newAllShiftsData[shiftId][lineId];
-          if (existingLine) {
-            newAllShiftsData[shiftId][lineId] = {
-              ...existingLine,
-              shiftStartTime: startTime,
-              shiftEndTime: endTime,
-              breaks,
-              shiftHours: Number(shiftHours),
-              workspaces: existingLine.workspaces.map((ws) => {
-                const threshold = actionThresholds.find((t) => t.workspace_id === ws.id);
-                if (threshold) {
-                  newDbValues[shiftId][lineId][ws.id] = {
-                    targetPPH: threshold.pph_threshold,
-                    targetCycleTime: threshold.ideal_cycle_time,
-                    targetDayOutput: threshold.total_day_output
-                  };
-                  return {
-                    ...ws,
-                    targetPPH: threshold.pph_threshold,
-                    targetCycleTime: threshold.ideal_cycle_time,
-                    targetDayOutput: threshold.total_day_output
-                  };
-                }
-                return ws;
-              })
-            };
-          }
-        } catch (error) {
-          console.error(`Error fetching data for line ${lineId}, shift ${shiftId}:`, error);
-        }
-      }
-    }
-    setAllShiftsData(newAllShiftsData);
-    setDbValues(newDbValues);
-  }, [supabase, lineIds, loadOperatingHours]);
+  }, [lineIds, companyId, timezone, skuEnabled]);
   const toggleLineDropdown = React23.useCallback((lineId) => {
     setDropdownStates((prev) => {
       const newIsOpen = !prev[lineId];
@@ -46625,7 +46434,7 @@ var TargetsView = ({
               if (value !== "") {
                 const pph = calculatePPH(value, prev[lineId].breaks, shiftHours);
                 updates.targetPPH = pph;
-                updates.targetDayOutput = calculateDayOutput(pph, shiftHours, prev[lineId].breaks);
+                updates.targetDayOutput = calculateDayOutputFromCycleTime(value, shiftHours);
               } else {
                 updates.targetPPH = "";
                 updates.targetDayOutput = "";
@@ -46743,7 +46552,8 @@ var TargetsView = ({
         action_id: ws.actionId,
         workspace_id: ws.id,
         date: currentDate,
-        pph_threshold: Number(ws.targetPPH) || 0,
+        pph_threshold: ws.targetPPH ? Math.round(Number(ws.targetPPH)) : 0,
+        // Round to whole number
         ideal_cycle_time: Number(ws.targetCycleTime) || 0,
         total_day_output: Number(ws.targetDayOutput) || 0,
         action_name: ws.actionType === "assembly" ? ACTION_NAMES.ASSEMBLY : ACTION_NAMES.PACKAGING,
@@ -46762,7 +46572,8 @@ var TargetsView = ({
         shift_id: selectedShift,
         product_code: lineDataToSave.productId,
         threshold_day_output: packagingWorkspaces.reduce((acc, ws) => acc + (Number(ws.targetDayOutput) || 0), 0),
-        threshold_pph: packagingWorkspaces.reduce((acc, ws) => acc + (Number(ws.targetPPH) || 0), 0),
+        threshold_pph: packagingWorkspaces.reduce((acc, ws) => acc + (ws.targetPPH ? Math.round(Number(ws.targetPPH)) : 0), 0),
+        // Round each PPH value
         ...skuEnabled && lineDataToSave.selectedSKU ? { sku_id: lineDataToSave.selectedSKU.id } : {}
       };
       console.log(`[handleSaveLine] lineThresholdData for upsert on ${lineId}:`, lineThresholdData);
@@ -48514,94 +48325,14 @@ var SupervisorManagementView = ({
   ] });
 };
 var SupervisorManagementView_default = SupervisorManagementView;
-var roleChangeImpacts = {
-  supervisor_to_plant_head: {
-    title: "Supervisor \u2192 Plant Head",
-    warnings: [
-      "Will lose line-specific assignments",
-      "Will need factory assignment",
-      "Will gain ability to manage supervisors",
-      "Will see factory-wide data"
-    ],
-    colorClass: "text-blue-700 bg-blue-50 border-blue-200"
-  },
-  supervisor_to_owner: {
-    title: "Supervisor \u2192 Owner",
-    warnings: [
-      "Will gain full company access",
-      "Will lose line-specific restrictions",
-      "Will be able to manage all users",
-      "Will see all company data"
-    ],
-    colorClass: "text-red-700 bg-red-50 border-red-200"
-  },
-  plant_head_to_supervisor: {
-    title: "Plant Head \u2192 Supervisor",
-    warnings: [
-      "Will lose factory access",
-      "Will lose ability to manage others",
-      "Will need line assignment",
-      "Will only see assigned line data"
-    ],
-    colorClass: "text-orange-700 bg-orange-50 border-orange-200"
-  },
-  plant_head_to_owner: {
-    title: "Plant Head \u2192 Owner",
-    warnings: [
-      "Will gain full company access",
-      "Will lose factory-specific restrictions",
-      "Will be able to manage all users",
-      "Will see all company data"
-    ],
-    colorClass: "text-blue-700 bg-blue-50 border-blue-200"
-  },
-  owner_to_plant_head: {
-    title: "Owner \u2192 Plant Head",
-    warnings: [
-      "Will lose company-wide access",
-      "Will need factory assignment",
-      "Will only manage supervisors",
-      "Will only see factory data"
-    ],
-    colorClass: "text-orange-700 bg-orange-50 border-orange-200"
-  },
-  owner_to_supervisor: {
-    title: "Owner \u2192 Supervisor",
-    warnings: [
-      "Will lose company-wide access",
-      "Will lose ability to manage others",
-      "Will need line assignment",
-      "Will only see assigned line data"
-    ],
-    colorClass: "text-red-700 bg-red-50 border-red-200"
-  },
-  to_optifye: {
-    title: "Promoting to Optifye",
-    warnings: [
-      "Will gain access to ALL companies",
-      "Will be able to manage all users globally",
-      "Will see all data across the platform",
-      "This is the highest privilege level"
-    ],
-    colorClass: "text-purple-700 bg-purple-50 border-purple-200"
-  },
-  from_optifye: {
-    title: "Demoting from Optifye",
-    warnings: [
-      "Will lose access to other companies",
-      "Will only see assigned company data",
-      "Will lose global management capabilities",
-      "Will need appropriate assignments"
-    ],
-    colorClass: "text-red-700 bg-red-50 border-red-200"
-  }
-};
-var getRoleChangeKey = (currentRole, newRole) => {
-  if (currentRole === newRole) return null;
-  if (newRole === "optifye") return "to_optifye";
-  if (currentRole === "optifye") return "from_optifye";
-  const key = `${currentRole}_to_${newRole}`;
-  return roleChangeImpacts[key] ? key : null;
+var getRoleLabel = (role) => {
+  const labels = {
+    "supervisor": "Supervisor",
+    "plant_head": "Plant Head",
+    "owner": "Owner",
+    "optifye": "Optifye Admin"
+  };
+  return labels[role] || role;
 };
 var ChangeRoleDialog = ({
   user,
@@ -48614,8 +48345,6 @@ var ChangeRoleDialog = ({
   const [confirmed, setConfirmed] = React23.useState(false);
   const [isChanging, setIsChanging] = React23.useState(false);
   const roleChanged = selectedRole !== user.role_level;
-  const changeKey = roleChanged ? getRoleChangeKey(user.role_level, selectedRole) : null;
-  const impactInfo = changeKey ? roleChangeImpacts[changeKey] : null;
   const handleNext = () => {
     if (roleChanged) {
       setStep("confirm");
@@ -48635,7 +48364,7 @@ var ChangeRoleDialog = ({
     setStep("select");
     setConfirmed(false);
   };
-  return /* @__PURE__ */ jsxRuntime.jsx("div", { className: "fixed inset-0 z-50 flex items-center justify-center bg-black/50", children: /* @__PURE__ */ jsxRuntime.jsxs("div", { className: "bg-white rounded-lg shadow-xl max-w-md w-full mx-4", children: [
+  return /* @__PURE__ */ jsxRuntime.jsx("div", { className: "fixed inset-0 z-50 flex items-center justify-center bg-black/50 backdrop-blur-sm", children: /* @__PURE__ */ jsxRuntime.jsxs("div", { className: "bg-white rounded-xl shadow-2xl max-w-md w-full mx-4 transform transition-all", children: [
     /* @__PURE__ */ jsxRuntime.jsxs("div", { className: "flex items-center justify-between p-6 border-b border-gray-200", children: [
       /* @__PURE__ */ jsxRuntime.jsxs("div", { className: "flex items-center gap-3", children: [
         /* @__PURE__ */ jsxRuntime.jsx("div", { className: "p-2 bg-blue-100 rounded-lg", children: /* @__PURE__ */ jsxRuntime.jsx(lucideReact.UserCog, { className: "w-5 h-5 text-blue-600" }) }),
@@ -48645,20 +48374,14 @@ var ChangeRoleDialog = ({
         "button",
         {
           onClick: onClose,
-          className: "text-gray-400 hover:text-gray-600 transition-colors",
+          className: "text-gray-400 hover:text-gray-600 transition-colors p-1 hover:bg-gray-100 rounded-lg",
           disabled: isChanging,
+          "aria-label": "Close dialog",
           children: /* @__PURE__ */ jsxRuntime.jsx(lucideReact.X, { className: "w-5 h-5" })
         }
       )
     ] }),
-    /* @__PURE__ */ jsxRuntime.jsx("div", { className: "p-6 space-y-6", children: step === "select" ? /* @__PURE__ */ jsxRuntime.jsxs(jsxRuntime.Fragment, { children: [
-      roleChanged && /* @__PURE__ */ jsxRuntime.jsxs("div", { className: "flex items-start gap-3 p-4 bg-amber-50 border border-amber-200 rounded-lg", children: [
-        /* @__PURE__ */ jsxRuntime.jsx(lucideReact.AlertCircle, { className: "w-5 h-5 text-amber-600 mt-0.5 flex-shrink-0" }),
-        /* @__PURE__ */ jsxRuntime.jsxs("div", { children: [
-          /* @__PURE__ */ jsxRuntime.jsx("p", { className: "font-medium text-amber-900 text-sm", children: "Changing roles affects access" }),
-          /* @__PURE__ */ jsxRuntime.jsx("p", { className: "text-amber-700 text-sm mt-1", children: "Review the impact carefully before confirming." })
-        ] })
-      ] }),
+    /* @__PURE__ */ jsxRuntime.jsx("div", { className: "p-6 space-y-5", children: step === "select" ? /* @__PURE__ */ jsxRuntime.jsxs(jsxRuntime.Fragment, { children: [
       /* @__PURE__ */ jsxRuntime.jsxs("div", { className: "space-y-2", children: [
         /* @__PURE__ */ jsxRuntime.jsx("p", { className: "text-sm text-gray-600", children: "User" }),
         /* @__PURE__ */ jsxRuntime.jsxs("div", { className: "p-3 bg-gray-50 rounded-lg", children: [
@@ -48669,7 +48392,7 @@ var ChangeRoleDialog = ({
           ] })
         ] })
       ] }),
-      /* @__PURE__ */ jsxRuntime.jsxs("div", { className: "space-y-3", children: [
+      /* @__PURE__ */ jsxRuntime.jsxs("div", { className: "space-y-2", children: [
         /* @__PURE__ */ jsxRuntime.jsx("p", { className: "text-sm font-medium text-gray-700", children: "Select new role" }),
         /* @__PURE__ */ jsxRuntime.jsx("div", { className: "space-y-2", children: availableRoles.map((role) => /* @__PURE__ */ jsxRuntime.jsxs(
           "label",
@@ -48693,27 +48416,24 @@ var ChangeRoleDialog = ({
           role
         )) })
       ] }),
-      impactInfo && /* @__PURE__ */ jsxRuntime.jsxs("div", { className: `p-4 border rounded-lg ${impactInfo.colorClass}`, children: [
-        /* @__PURE__ */ jsxRuntime.jsx("p", { className: "font-medium text-sm mb-2", children: "\u2139\uFE0F Impact of this change:" }),
-        /* @__PURE__ */ jsxRuntime.jsx("ul", { className: "space-y-1.5 text-sm", children: impactInfo.warnings.map((warning6, idx) => /* @__PURE__ */ jsxRuntime.jsxs("li", { className: "flex items-start gap-2", children: [
-          /* @__PURE__ */ jsxRuntime.jsx("span", { className: "mt-0.5", children: "\u2022" }),
-          /* @__PURE__ */ jsxRuntime.jsx("span", { children: warning6 })
-        ] }, idx)) })
-      ] })
+      roleChanged && /* @__PURE__ */ jsxRuntime.jsx("div", { className: "px-3 py-2 bg-gray-50 border border-gray-200 rounded-lg", children: /* @__PURE__ */ jsxRuntime.jsxs("p", { className: "text-xs text-gray-600", children: [
+        "Changing from ",
+        /* @__PURE__ */ jsxRuntime.jsx("span", { className: "font-medium text-gray-900", children: getRoleLabel(user.role_level) }),
+        " to ",
+        /* @__PURE__ */ jsxRuntime.jsx("span", { className: "font-medium text-gray-900", children: getRoleLabel(selectedRole) }),
+        " will modify user permissions."
+      ] }) })
     ] }) : /* @__PURE__ */ jsxRuntime.jsx(jsxRuntime.Fragment, { children: /* @__PURE__ */ jsxRuntime.jsxs("div", { className: "space-y-4", children: [
       /* @__PURE__ */ jsxRuntime.jsxs("div", { className: "flex items-center justify-center gap-3 p-4 bg-gray-50 rounded-lg", children: [
         /* @__PURE__ */ jsxRuntime.jsx(RoleBadge, { role: user.role_level }),
         /* @__PURE__ */ jsxRuntime.jsx(lucideReact.ChevronRight, { className: "w-5 h-5 text-gray-400" }),
         /* @__PURE__ */ jsxRuntime.jsx(RoleBadge, { role: selectedRole })
       ] }),
-      impactInfo && /* @__PURE__ */ jsxRuntime.jsxs("div", { className: `p-4 border rounded-lg ${impactInfo.colorClass}`, children: [
-        /* @__PURE__ */ jsxRuntime.jsx("p", { className: "font-medium text-sm mb-3", children: impactInfo.title }),
-        /* @__PURE__ */ jsxRuntime.jsx("ul", { className: "space-y-2 text-sm", children: impactInfo.warnings.map((warning6, idx) => /* @__PURE__ */ jsxRuntime.jsxs("li", { className: "flex items-start gap-2", children: [
-          /* @__PURE__ */ jsxRuntime.jsx("span", { className: "mt-0.5", children: "\u2022" }),
-          /* @__PURE__ */ jsxRuntime.jsx("span", { children: warning6 })
-        ] }, idx)) })
+      /* @__PURE__ */ jsxRuntime.jsxs("div", { className: "space-y-2", children: [
+        /* @__PURE__ */ jsxRuntime.jsx("p", { className: "text-sm text-gray-600", children: "User" }),
+        /* @__PURE__ */ jsxRuntime.jsx("p", { className: "text-sm font-medium text-gray-900", children: user.email })
       ] }),
-      /* @__PURE__ */ jsxRuntime.jsxs("div", { className: "flex items-start gap-3 p-4 border border-gray-200 rounded-lg", children: [
+      /* @__PURE__ */ jsxRuntime.jsxs("div", { className: "flex items-start gap-3 p-3 border border-gray-200 rounded-lg bg-gray-50", children: [
         /* @__PURE__ */ jsxRuntime.jsx(
           "input",
           {
@@ -48724,7 +48444,7 @@ var ChangeRoleDialog = ({
             className: "mt-0.5 h-4 w-4 text-blue-600 border-gray-300 rounded focus:ring-blue-500"
           }
         ),
-        /* @__PURE__ */ jsxRuntime.jsx("label", { htmlFor: "confirm-role-change", className: "text-sm text-gray-700 cursor-pointer", children: "I understand this will change the user's access and responsibilities" })
+        /* @__PURE__ */ jsxRuntime.jsx("label", { htmlFor: "confirm-role-change", className: "text-sm text-gray-700 cursor-pointer", children: "I confirm this role change and understand the permissions will be updated" })
       ] })
     ] }) }) }),
     /* @__PURE__ */ jsxRuntime.jsx("div", { className: "flex items-center justify-end gap-3 p-6 border-t border-gray-200 bg-gray-50", children: step === "select" ? /* @__PURE__ */ jsxRuntime.jsxs(jsxRuntime.Fragment, { children: [
@@ -48767,7 +48487,7 @@ var ChangeRoleDialog = ({
               /* @__PURE__ */ jsxRuntime.jsx("path", { className: "opacity-75", fill: "currentColor", d: "M4 12a8 8 0 018-8V0C5.373 0 0 5.373 0 12h4zm2 5.291A7.962 7.962 0 014 12H0c0 3.042 1.135 5.824 3 7.938l3-2.647z" })
             ] }),
             "Changing..."
-          ] }) : "Change Role"
+          ] }) : "Confirm Change"
         }
       )
     ] }) })
@@ -48790,32 +48510,34 @@ var ConfirmRemoveUserDialog = ({
       setIsRemoving(false);
     }
   };
-  return /* @__PURE__ */ jsxRuntime.jsx("div", { className: "fixed inset-0 z-50 flex items-center justify-center bg-black/50", children: /* @__PURE__ */ jsxRuntime.jsxs("div", { className: "bg-white rounded-lg shadow-xl max-w-md w-full mx-4", children: [
+  return /* @__PURE__ */ jsxRuntime.jsx("div", { className: "fixed inset-0 z-50 flex items-center justify-center bg-black/50 backdrop-blur-sm", children: /* @__PURE__ */ jsxRuntime.jsxs("div", { className: "bg-white rounded-xl shadow-2xl max-w-md w-full mx-4 transform transition-all", children: [
     /* @__PURE__ */ jsxRuntime.jsxs("div", { className: "flex items-center justify-between p-6 border-b border-gray-200", children: [
       /* @__PURE__ */ jsxRuntime.jsxs("div", { className: "flex items-center gap-3", children: [
         /* @__PURE__ */ jsxRuntime.jsx("div", { className: "p-2 bg-red-100 rounded-lg", children: /* @__PURE__ */ jsxRuntime.jsx(lucideReact.Trash2, { className: "w-5 h-5 text-red-600" }) }),
-        /* @__PURE__ */ jsxRuntime.jsx("h2", { className: "text-lg font-semibold text-gray-900", children: "Remove User" })
+        /* @__PURE__ */ jsxRuntime.jsx("h2", { className: "text-lg font-semibold text-gray-900", children: "Permanently Delete User" })
       ] }),
       /* @__PURE__ */ jsxRuntime.jsx(
         "button",
         {
           onClick: onCancel,
-          className: "text-gray-400 hover:text-gray-600 transition-colors",
+          className: "text-gray-400 hover:text-gray-600 transition-colors p-1 hover:bg-gray-100 rounded-lg",
+          disabled: isRemoving,
+          "aria-label": "Close dialog",
           children: /* @__PURE__ */ jsxRuntime.jsx(lucideReact.X, { className: "w-5 h-5" })
         }
       )
     ] }),
-    /* @__PURE__ */ jsxRuntime.jsxs("div", { className: "p-6 space-y-6", children: [
-      /* @__PURE__ */ jsxRuntime.jsxs("div", { className: "flex items-start gap-3 p-4 bg-red-50 border border-red-200 rounded-lg", children: [
-        /* @__PURE__ */ jsxRuntime.jsx(lucideReact.AlertTriangle, { className: "w-5 h-5 text-red-600 mt-0.5 flex-shrink-0" }),
-        /* @__PURE__ */ jsxRuntime.jsxs("div", { children: [
-          /* @__PURE__ */ jsxRuntime.jsx("p", { className: "font-medium text-red-900 text-sm", children: "This action cannot be undone" }),
-          /* @__PURE__ */ jsxRuntime.jsx("p", { className: "text-red-700 text-sm mt-1", children: "The user will immediately lose all access to the platform." })
+    /* @__PURE__ */ jsxRuntime.jsxs("div", { className: "p-6 space-y-5", children: [
+      /* @__PURE__ */ jsxRuntime.jsxs("div", { className: "flex items-start gap-3 p-3 bg-red-50 border border-red-200 rounded-lg", children: [
+        /* @__PURE__ */ jsxRuntime.jsx(lucideReact.AlertTriangle, { className: "w-4 h-4 text-red-600 mt-0.5 flex-shrink-0" }),
+        /* @__PURE__ */ jsxRuntime.jsxs("p", { className: "text-sm text-red-900", children: [
+          /* @__PURE__ */ jsxRuntime.jsx("span", { className: "font-medium", children: "This action cannot be undone." }),
+          " This will permanently delete the user and all related data from the system."
         ] })
       ] }),
-      /* @__PURE__ */ jsxRuntime.jsxs("div", { className: "space-y-3", children: [
-        /* @__PURE__ */ jsxRuntime.jsx("p", { className: "text-sm font-medium text-gray-700", children: "You are about to remove:" }),
-        /* @__PURE__ */ jsxRuntime.jsxs("div", { className: "p-4 bg-gray-50 rounded-lg space-y-2", children: [
+      /* @__PURE__ */ jsxRuntime.jsxs("div", { className: "space-y-2", children: [
+        /* @__PURE__ */ jsxRuntime.jsx("p", { className: "text-sm font-medium text-gray-700", children: "You are about to permanently delete:" }),
+        /* @__PURE__ */ jsxRuntime.jsxs("div", { className: "p-3 bg-gray-50 rounded-lg space-y-2.5", children: [
           /* @__PURE__ */ jsxRuntime.jsxs("div", { className: "flex items-center justify-between", children: [
             /* @__PURE__ */ jsxRuntime.jsx("span", { className: "text-sm text-gray-600", children: "Email" }),
             /* @__PURE__ */ jsxRuntime.jsx("span", { className: "text-sm font-medium text-gray-900", children: user.email })
@@ -48840,28 +48562,7 @@ var ConfirmRemoveUserDialog = ({
           ] })
         ] })
       ] }),
-      /* @__PURE__ */ jsxRuntime.jsxs("div", { className: "space-y-2", children: [
-        /* @__PURE__ */ jsxRuntime.jsx("p", { className: "text-sm font-medium text-gray-700", children: "This will:" }),
-        /* @__PURE__ */ jsxRuntime.jsxs("ul", { className: "space-y-2 text-sm text-gray-600", children: [
-          /* @__PURE__ */ jsxRuntime.jsxs("li", { className: "flex items-start gap-2", children: [
-            /* @__PURE__ */ jsxRuntime.jsx("span", { className: "text-red-500 mt-0.5", children: "\u2022" }),
-            /* @__PURE__ */ jsxRuntime.jsx("span", { children: "Revoke all access immediately" })
-          ] }),
-          /* @__PURE__ */ jsxRuntime.jsxs("li", { className: "flex items-start gap-2", children: [
-            /* @__PURE__ */ jsxRuntime.jsx("span", { className: "text-red-500 mt-0.5", children: "\u2022" }),
-            /* @__PURE__ */ jsxRuntime.jsx("span", { children: "Remove all line and factory assignments" })
-          ] }),
-          /* @__PURE__ */ jsxRuntime.jsxs("li", { className: "flex items-start gap-2", children: [
-            /* @__PURE__ */ jsxRuntime.jsx("span", { className: "text-red-500 mt-0.5", children: "\u2022" }),
-            /* @__PURE__ */ jsxRuntime.jsx("span", { children: "Log the user out of all sessions" })
-          ] }),
-          /* @__PURE__ */ jsxRuntime.jsxs("li", { className: "flex items-start gap-2", children: [
-            /* @__PURE__ */ jsxRuntime.jsx("span", { className: "text-red-500 mt-0.5", children: "\u2022" }),
-            /* @__PURE__ */ jsxRuntime.jsx("span", { children: "User will not be able to access the dashboard" })
-          ] })
-        ] })
-      ] }),
-      /* @__PURE__ */ jsxRuntime.jsxs("div", { className: "flex items-start gap-3 p-4 border border-gray-200 rounded-lg", children: [
+      /* @__PURE__ */ jsxRuntime.jsxs("div", { className: "flex items-start gap-3 p-3 border border-gray-200 rounded-lg bg-gray-50", children: [
         /* @__PURE__ */ jsxRuntime.jsx(
           "input",
           {
@@ -48872,7 +48573,7 @@ var ConfirmRemoveUserDialog = ({
             className: "mt-0.5 h-4 w-4 text-red-600 border-gray-300 rounded focus:ring-red-500"
           }
         ),
-        /* @__PURE__ */ jsxRuntime.jsx("label", { htmlFor: "confirm-remove", className: "text-sm text-gray-700 cursor-pointer", children: "I understand this action is permanent and cannot be undone" })
+        /* @__PURE__ */ jsxRuntime.jsx("label", { htmlFor: "confirm-remove", className: "text-sm text-gray-700 cursor-pointer", children: "I understand this will permanently delete the user and all related data, and this action cannot be undone" })
       ] })
     ] }),
     /* @__PURE__ */ jsxRuntime.jsxs("div", { className: "flex items-center justify-end gap-3 p-6 border-t border-gray-200 bg-gray-50", children: [
@@ -48896,8 +48597,8 @@ var ConfirmRemoveUserDialog = ({
               /* @__PURE__ */ jsxRuntime.jsx("circle", { className: "opacity-25", cx: "12", cy: "12", r: "10", stroke: "currentColor", strokeWidth: "4", fill: "none" }),
               /* @__PURE__ */ jsxRuntime.jsx("path", { className: "opacity-75", fill: "currentColor", d: "M4 12a8 8 0 018-8V0C5.373 0 0 5.373 0 12h4zm2 5.291A7.962 7.962 0 014 12H0c0 3.042 1.135 5.824 3 7.938l3-2.647z" })
             ] }),
-            "Removing..."
-          ] }) : "Remove User"
+            "Deleting..."
+          ] }) : "Delete User"
         }
       )
     ] })
@@ -49320,7 +49021,6 @@ var FactoryAssignmentDropdown = ({
 var UserManagementTable = ({
   users,
   isLoading = false,
-  onUserClick,
   onRoleChange,
   onRemoveUser,
   onLineAssignmentUpdate,
@@ -49346,9 +49046,48 @@ var UserManagementTable = ({
   const [sortField, setSortField] = React23.useState("email");
   const [sortDirection, setSortDirection] = React23.useState("asc");
   const [openActionMenuId, setOpenActionMenuId] = React23.useState(null);
+  const [dropdownPosition, setDropdownPosition] = React23.useState(null);
+  const actionButtonRefs = React23.useRef({});
   const [selectedUser, setSelectedUser] = React23.useState(null);
   const [showChangeRoleDialog, setShowChangeRoleDialog] = React23.useState(false);
   const [showRemoveUserDialog, setShowRemoveUserDialog] = React23.useState(false);
+  const handleOpenActionMenu = (userId) => {
+    const buttonRef = actionButtonRefs.current[userId];
+    if (buttonRef) {
+      const rect = buttonRef.getBoundingClientRect();
+      setDropdownPosition({
+        top: rect.bottom + 8,
+        // 8px below the button
+        left: rect.right - 192
+        // 192px is the dropdown width (w-48 = 12rem = 192px)
+      });
+      setOpenActionMenuId(userId);
+    }
+  };
+  const handleCloseActionMenu = () => {
+    setOpenActionMenuId(null);
+    setDropdownPosition(null);
+  };
+  React23.useEffect(() => {
+    if (openActionMenuId && actionButtonRefs.current[openActionMenuId]) {
+      const updatePosition = () => {
+        const buttonRef = actionButtonRefs.current[openActionMenuId];
+        if (buttonRef) {
+          const rect = buttonRef.getBoundingClientRect();
+          setDropdownPosition({
+            top: rect.bottom + 8,
+            left: rect.right - 192
+          });
+        }
+      };
+      window.addEventListener("scroll", updatePosition, true);
+      window.addEventListener("resize", updatePosition);
+      return () => {
+        window.removeEventListener("scroll", updatePosition, true);
+        window.removeEventListener("resize", updatePosition);
+      };
+    }
+  }, [openActionMenuId]);
   const filteredAndSortedUsers = React23.useMemo(() => {
     let filtered = users;
     filtered = filtered.filter((user) => availableRoles.includes(user.role_level));
@@ -49539,7 +49278,7 @@ var UserManagementTable = ({
                   LineAssignmentDropdown,
                   {
                     userId: user.user_id,
-                    currentLineIds: user.properties?.line_id || [],
+                    currentLineIds: user.properties?.line_ids || [],
                     availableLines: (
                       // Filter lines to only show those from the target user's company
                       (() => {
@@ -49577,69 +49316,24 @@ var UserManagementTable = ({
                   }
                 ) : /* @__PURE__ */ jsxRuntime.jsx("p", { className: "text-sm text-gray-900", children: formatAssignments(user) }) }),
                 /* @__PURE__ */ jsxRuntime.jsx("td", { className: "px-6 py-4 whitespace-nowrap", children: /* @__PURE__ */ jsxRuntime.jsx("p", { className: "text-sm text-gray-500", children: formatDate(user.created_at) }) }),
-                /* @__PURE__ */ jsxRuntime.jsx("td", { className: "px-6 py-4 whitespace-nowrap text-right", children: hasActions && /* @__PURE__ */ jsxRuntime.jsxs("div", { className: "relative", children: [
-                  /* @__PURE__ */ jsxRuntime.jsx(
-                    "button",
-                    {
-                      onClick: () => setOpenActionMenuId(openActionMenuId === user.user_id ? null : user.user_id),
-                      className: "p-2 text-gray-400 hover:text-gray-600 hover:bg-gray-100 rounded-lg transition-colors",
-                      "aria-label": "User actions",
-                      children: /* @__PURE__ */ jsxRuntime.jsx(lucideReact.MoreVertical, { className: "w-4 h-4" })
-                    }
-                  ),
-                  openActionMenuId === user.user_id && /* @__PURE__ */ jsxRuntime.jsxs(jsxRuntime.Fragment, { children: [
-                    /* @__PURE__ */ jsxRuntime.jsx(
-                      "div",
-                      {
-                        className: "fixed inset-0 z-[9998]",
-                        onClick: () => setOpenActionMenuId(null)
+                /* @__PURE__ */ jsxRuntime.jsx("td", { className: "px-6 py-4 whitespace-nowrap text-right", children: hasActions && /* @__PURE__ */ jsxRuntime.jsx("div", { className: "relative", children: /* @__PURE__ */ jsxRuntime.jsx(
+                  "button",
+                  {
+                    ref: (el) => {
+                      actionButtonRefs.current[user.user_id] = el;
+                    },
+                    onClick: () => {
+                      if (openActionMenuId === user.user_id) {
+                        handleCloseActionMenu();
+                      } else {
+                        handleOpenActionMenu(user.user_id);
                       }
-                    ),
-                    /* @__PURE__ */ jsxRuntime.jsx("div", { className: "absolute right-0 mt-2 w-48 bg-white rounded-lg shadow-lg border border-gray-200 z-[9999]", children: /* @__PURE__ */ jsxRuntime.jsxs("div", { className: "py-1", children: [
-                      /* @__PURE__ */ jsxRuntime.jsxs(
-                        "button",
-                        {
-                          onClick: () => {
-                            onUserClick?.(user);
-                            setOpenActionMenuId(null);
-                          },
-                          className: "w-full px-4 py-2 text-sm text-left text-gray-700 hover:bg-gray-100 flex items-center gap-2",
-                          children: [
-                            /* @__PURE__ */ jsxRuntime.jsx(lucideReact.Eye, { className: "w-4 h-4" }),
-                            "View Details"
-                          ]
-                        }
-                      ),
-                      canChangeRole && /* @__PURE__ */ jsxRuntime.jsxs(
-                        "button",
-                        {
-                          onClick: () => handleChangeRole(user),
-                          className: "w-full px-4 py-2 text-sm text-left text-gray-700 hover:bg-gray-100 flex items-center gap-2",
-                          disabled: isCurrentUser,
-                          children: [
-                            /* @__PURE__ */ jsxRuntime.jsx(lucideReact.UserCog, { className: "w-4 h-4" }),
-                            "Change Role"
-                          ]
-                        }
-                      ),
-                      canRemove && /* @__PURE__ */ jsxRuntime.jsxs(jsxRuntime.Fragment, { children: [
-                        /* @__PURE__ */ jsxRuntime.jsx("div", { className: "border-t border-gray-200 my-1" }),
-                        /* @__PURE__ */ jsxRuntime.jsxs(
-                          "button",
-                          {
-                            onClick: () => handleRemoveUser(user),
-                            className: "w-full px-4 py-2 text-sm text-left text-red-600 hover:bg-red-50 flex items-center gap-2",
-                            disabled: isCurrentUser,
-                            children: [
-                              /* @__PURE__ */ jsxRuntime.jsx(lucideReact.Trash2, { className: "w-4 h-4" }),
-                              "Remove User"
-                            ]
-                          }
-                        )
-                      ] })
-                    ] }) })
-                  ] })
-                ] }) })
+                    },
+                    className: "p-2 text-gray-400 hover:text-gray-600 hover:bg-gray-100 rounded-lg transition-colors",
+                    "aria-label": "User actions",
+                    children: /* @__PURE__ */ jsxRuntime.jsx(lucideReact.MoreVertical, { className: "w-4 h-4" })
+                  }
+                ) }) })
               ]
             },
             user.user_id
@@ -49647,6 +49341,75 @@ var UserManagementTable = ({
         }) })
       ] }) }) })
     ] }),
+    openActionMenuId && dropdownPosition && typeof document !== "undefined" && reactDom.createPortal(
+      /* @__PURE__ */ jsxRuntime.jsxs(jsxRuntime.Fragment, { children: [
+        /* @__PURE__ */ jsxRuntime.jsx(
+          "div",
+          {
+            className: "fixed inset-0 z-[9998]",
+            onClick: handleCloseActionMenu
+          }
+        ),
+        /* @__PURE__ */ jsxRuntime.jsx(
+          "div",
+          {
+            className: "fixed w-48 bg-white rounded-lg shadow-lg border border-gray-200 z-[9999]",
+            style: {
+              top: `${dropdownPosition.top}px`,
+              left: `${dropdownPosition.left}px`
+            },
+            children: /* @__PURE__ */ jsxRuntime.jsxs("div", { className: "py-1", children: [
+              (() => {
+                const user = users.find((u) => u.user_id === openActionMenuId);
+                const isCurrentUser = user?.user_id === currentUserId;
+                const canChangeRole = user && permissions.canChangeRole(user);
+                return canChangeRole && /* @__PURE__ */ jsxRuntime.jsxs(
+                  "button",
+                  {
+                    onClick: () => {
+                      if (user) {
+                        handleChangeRole(user);
+                      }
+                    },
+                    className: "w-full px-4 py-2 text-sm text-left text-gray-700 hover:bg-gray-100 flex items-center gap-2 disabled:opacity-50 disabled:cursor-not-allowed",
+                    disabled: isCurrentUser,
+                    children: [
+                      /* @__PURE__ */ jsxRuntime.jsx(lucideReact.UserCog, { className: "w-4 h-4" }),
+                      "Change Role"
+                    ]
+                  }
+                );
+              })(),
+              (() => {
+                const user = users.find((u) => u.user_id === openActionMenuId);
+                const isCurrentUser = user?.user_id === currentUserId;
+                const canRemove = user && permissions.canRemoveUser(user);
+                return canRemove && /* @__PURE__ */ jsxRuntime.jsxs(jsxRuntime.Fragment, { children: [
+                  /* @__PURE__ */ jsxRuntime.jsx("div", { className: "border-t border-gray-200 my-1" }),
+                  /* @__PURE__ */ jsxRuntime.jsxs(
+                    "button",
+                    {
+                      onClick: () => {
+                        if (user) {
+                          handleRemoveUser(user);
+                        }
+                      },
+                      className: "w-full px-4 py-2 text-sm text-left text-red-600 hover:bg-red-50 flex items-center gap-2 disabled:opacity-50 disabled:cursor-not-allowed",
+                      disabled: isCurrentUser,
+                      children: [
+                        /* @__PURE__ */ jsxRuntime.jsx(lucideReact.Trash2, { className: "w-4 h-4" }),
+                        "Delete User"
+                      ]
+                    }
+                  )
+                ] });
+              })()
+            ] })
+          }
+        )
+      ] }),
+      document.body
+    ),
     showChangeRoleDialog && selectedUser && /* @__PURE__ */ jsxRuntime.jsx(
       ChangeRoleDialog,
       {
@@ -50298,15 +50061,12 @@ var TeamManagementView = ({
     if (!supabase || !user) return;
     try {
       const userManagementService = createUserManagementService(supabase);
-      await userManagementService.deactivateUser({
-        user_id: userId,
-        deactivated_by: user.id
-      });
-      sonner.toast.success("User removed successfully");
+      const result = await userManagementService.deleteUser(userId);
+      sonner.toast.success(`User ${result.deleted_user_email} permanently deleted`);
       loadData();
     } catch (err) {
-      console.error("Error removing user:", err);
-      sonner.toast.error("Failed to remove user");
+      console.error("Error deleting user:", err);
+      sonner.toast.error("Failed to delete user");
     }
   }, [supabase, user, loadData]);
   const handleLineAssignmentUpdate = React23.useCallback(async (userId, lineIds) => {
@@ -50400,9 +50160,6 @@ var TeamManagementView = ({
       {
         users,
         currentUserId: user?.id,
-        onUserClick: (clickedUser) => {
-          console.log("View user details:", clickedUser);
-        },
         onRoleChange: handleRoleChange,
         onRemoveUser: handleRemoveUser,
         onLineAssignmentUpdate: handleLineAssignmentUpdate,