fix: Delete ai comments

src/widgets/DevicesTable/index.tsx

@@ -11,8 +11,8 @@ import {
   devicesStore,
   Modal,
   snapshotStore,
-  vehicleStore, // Not directly used in this component's rendering logic anymore
-} from "@shared"; // Assuming @shared exports these
+  vehicleStore,
+} from "@shared";
 import { useEffect, useState } from "react";
 import { observer } from "mobx-react-lite";
 import { Button, Checkbox, Typography } from "@mui/material";
@@ -23,12 +23,10 @@ import { useNavigate } from "react-router-dom";
 export type ConnectedDevice = string;
 
 interface Snapshot {
-  ID: string; // Assuming ID is string based on usage
+  ID: string;
   Name: string;
-  // Add other snapshot properties if needed
 }
 
-// --- HELPER FUNCTIONS ---
 const formatDate = (dateString: string | null) => {
   if (!dateString) return "Нет данных";
   try {
@@ -76,12 +74,7 @@ function createData(
   };
 }
 
-// This function transforms the raw device data (which includes vehicle and device_status)
-// into the format expected by the table. It now filters for devices that have a UUID.
-const transformDevicesToRows = (
-  vehicles: Vehicle[]
-  // devices: ConnectedDevice[]
-): TableRowData[] => {
+const transformDevicesToRows = (vehicles: Vehicle[]): TableRowData[] => {
   return vehicles.map((vehicle) => {
     const uuid = vehicle.vehicle.uuid;
     if (!uuid)
@@ -115,26 +108,21 @@ export const DevicesTable = observer(() => {
   } = devicesStore;
 
   const { snapshots, getSnapshots } = snapshotStore;
-  const { getVehicles, vehicles } = vehicleStore; // Removed as devicesStore.devices should be the source of truth
+  const { getVehicles, vehicles } = vehicleStore;
   const { devices } = devicesStore;
   const navigate = useNavigate();
   const [selectedDeviceUuids, setSelectedDeviceUuids] = useState<string[]>([]);
 
-  // Transform the raw devices data into rows suitable for the table
-  // This will also filter out devices without a UUID, as those cannot be acted upon.
-  const currentTableRows = transformDevicesToRows(
-    vehicles.data as Vehicle[]
-    //  devices as ConnectedDevice[]
-  );
+  const currentTableRows = transformDevicesToRows(vehicles.data as Vehicle[]);
 
   useEffect(() => {
     const fetchData = async () => {
-      await getVehicles(); // Not strictly needed if devicesStore.devices is populated correctly by getDevices
-      await getDevices(); // This should fetch the combined vehicle/device_status data
+      await getVehicles();
+      await getDevices();
       await getSnapshots();
     };
     fetchData();
-  }, [getDevices, getSnapshots]); // Added dependencies
+  }, [getDevices, getSnapshots]);
 
   const isAllSelected =
     currentTableRows.length > 0 &&
@@ -144,7 +132,6 @@ export const DevicesTable = observer(() => {
     if (isAllSelected) {
       setSelectedDeviceUuids([]);
     } else {
-      // Select all device UUIDs from the *currently visible and selectable* rows
       setSelectedDeviceUuids(
         currentTableRows.map((row) => row.device_uuid ?? "")
       );
@@ -171,14 +158,13 @@ export const DevicesTable = observer(() => {
   };
 
   const handleReloadStatus = async (uuid: string) => {
-    setSelectedDevice(uuid); // Sets the device in the store, useful for context elsewhere
+    setSelectedDevice(uuid);
     try {
       await authInstance.post(`/devices/${uuid}/request-status`);
       await getVehicles();
-      await getDevices(); // Refresh devices to show updated status
+      await getDevices();
     } catch (error) {
       console.error(`Error requesting status for device ${uuid}:`, error);
-      // Optionally: show a user-facing error message
     }
   };
 
@@ -200,22 +186,16 @@ export const DevicesTable = observer(() => {
       }
     };
     try {
-      // Create an array of promises for all snapshot requests
       const snapshotPromises = selectedDeviceUuids.map((deviceUuid) => {
         return send(deviceUuid);
       });
 
-      // Wait for all promises to settle (either resolve or reject)
       await Promise.allSettled(snapshotPromises);
 
-      // After all requests are attempted
-      await getDevices(); // Refresh the device list
-      setSelectedDeviceUuids([]); // Clear the selection
-      toggleSendSnapshotModal(); // Close the modal
+      await getDevices();
+      setSelectedDeviceUuids([]);
+      toggleSendSnapshotModal();
     } catch (error) {
-      // This catch block might not be hit if Promise.allSettled is used,
-      // as it doesn't reject on individual promise failures.
-      // Individual errors should be handled if needed within the .map or by checking results.
       console.error("Error in snapshot sending process:", error);
     }
   };
@@ -235,7 +215,7 @@ export const DevicesTable = observer(() => {
         </div>
         <div className="flex justify-end p-3 gap-2">
           <Button
-            variant="outlined" // Changed to outlined for distinction
+            variant="outlined"
             onClick={handleSelectAllDevices}
             size="small"
           >
@@ -286,7 +266,6 @@ export const DevicesTable = observer(() => {
                 )}
                 selected={selectedDeviceUuids.includes(row.device_uuid ?? "")}
                 onClick={(event) => {
-                  // Allow clicking row to toggle checkbox, if not clicking on button
                   if (
                     (event.target as HTMLElement).closest("button") === null &&
                     (event.target as HTMLElement).closest(
@@ -308,7 +287,6 @@ export const DevicesTable = observer(() => {
                       }
                     }
 
-                    // Only toggle checkbox if Shift key is not pressed
                     if (!event.shiftKey) {
                       handleSelectDevice(
                         {
@@ -317,7 +295,7 @@ export const DevicesTable = observer(() => {
                               row.device_uuid ?? ""
                             ),
                           },
-                        } as React.ChangeEvent<HTMLInputElement>, // Simulate event
+                        } as React.ChangeEvent<HTMLInputElement>,
                         row.device_uuid ?? ""
                       );
                     }
@@ -445,7 +423,7 @@ export const DevicesTable = observer(() => {
           </strong>
         </Typography>
         <div className="mt-2 flex flex-col gap-2 max-h-[300px] overflow-y-auto pr-2">
-          {snapshots && (snapshots as Snapshot[]).length > 0 ? ( // Cast snapshots
+          {snapshots && (snapshots as Snapshot[]).length > 0 ? (
             (snapshots as Snapshot[]).map((snapshot) => (
               <Button
                 variant="outlined"