Fixed and cleaned code

Fixed and cleaned code

Author: jorgedlcruz

app/api/vbr/restore-points/route.ts

@@ -75,7 +75,7 @@ export async function GET(request: NextRequest) {
         }
 
         const rpResult = await rpResponse.json();
-        let restorePoints: VeeamRestorePoint[] = rpResult.data;
+        const restorePoints: VeeamRestorePoint[] = rpResult.data;
 
         // Optimize: If we have no restore points, Return early
         if (restorePoints.length === 0) {
@@ -135,7 +135,7 @@ export async function GET(request: NextRequest) {
             const matchingFile = backupFiles.find(file => {
                 // The user JSON shows backupFile has restorePointIds array.
                 // We need to check if VeeamBackupFile type has it, if not we treat it as unknown/any for now
-                const fileWithRps = file as any;
+                const fileWithRps = file as unknown as { restorePointIds?: string[] };
                 return fileWithRps.restorePointIds && fileWithRps.restorePointIds.includes(rp.id);
             });
 

app/vbm/jobs/page.tsx

@@ -2,7 +2,7 @@
 
 export const dynamic = 'force-dynamic'
 
-import { useState, useEffect, useMemo } from "react"
+import { useState, useEffect } from "react"
 import { VBMJobsTable } from "@/components/vbm-jobs-table"
 import { veeamApi } from "@/lib/api/veeam-client"
 import { VBMJob } from "@/lib/types/vbm"

app/vbr/infrastructure/proxies/page.tsx

@@ -4,8 +4,6 @@ import React, { useEffect, useState } from 'react';
 import { VeeamProxy } from '@/lib/types/veeam';
 import { veeamApi as veeamApiClient } from '@/lib/api/veeam-client';
 import { BackupProxiesTable } from '@/components/backup-proxies-table';
-import { Card, CardContent, CardDescription, CardHeader, CardTitle } from '@/components/ui/card';
-import { Server } from 'lucide-react';
 
 export default function BackupProxiesPage() {
     const [proxies, setProxies] = useState<VeeamProxy[]>([]);

app/vbr/inventory/protection-groups/[id]/page.tsx

@@ -1,7 +1,7 @@
 "use client"
 
 import { DiscoveredEntitiesTable } from "@/components/discovered-entities-table"
-import { useParams } from "next/navigation"
+
 import { Button } from "@/components/ui/button"
 import Link from "next/link"
 import { ArrowLeft } from "lucide-react"

app/vbr/jobs/page.tsx

@@ -2,7 +2,7 @@
 
 export const dynamic = 'force-dynamic'
 
-import { useState, useEffect, useMemo, useCallback } from "react"
+import { useState, useEffect, useCallback } from "react"
 import { BackupJobsTable } from "@/components/backup-jobs-table"
 import { veeamApi } from "@/lib/api/veeam-client"
 import { VeeamBackupJob } from "@/lib/types/veeam"

app/vbr/page.tsx

@@ -2,7 +2,7 @@
 
 export const dynamic = 'force-dynamic'
 
-import { useState, useEffect, useMemo } from "react"
+import { useState, useEffect } from "react"
 import { BackupJobsTable } from "@/components/backup-jobs-table"
 import { TransferRateChart } from "@/components/transfer-rate-chart"
 import { veeamApi } from "@/lib/api/veeam-client"

app/vro/page.tsx

@@ -2,7 +2,7 @@
 
 export const dynamic = 'force-dynamic'
 
-import { useState, useEffect, useMemo } from "react"
+import { useState, useEffect } from "react"
 import { RecoveryPlansTable } from "@/components/recovery-plans-table"
 import { veeamApi } from "@/lib/api/veeam-client"
 import { VRORecoveryPlan } from "@/lib/types/veeam"

components/backup-proxies-table.tsx

@@ -33,7 +33,7 @@ import {
     DropdownMenuTrigger,
 } from "@/components/ui/dropdown-menu";
 import { Badge } from "@/components/ui/badge";
-import { MoreHorizontal, ChevronDown, Check, X, Server, Power, PowerOff, Trash2, Edit, Search, RefreshCw, Smartphone, Monitor } from "lucide-react";
+import { MoreHorizontal, ChevronDown, Server, Power, PowerOff, Trash2, Edit, Search, RefreshCw } from "lucide-react";
 import { VeeamProxy } from "@/lib/types/veeam";
 import { toast } from "sonner";
 import { veeamApi as veeamApiClient } from "@/lib/api/veeam-client";
@@ -260,17 +260,7 @@ export function BackupProxiesTable({ data, isLoading, onRefresh }: BackupProxies
         // Add others if known
     ];
 
-    const statusOptions = [
-        { label: "Online", value: "true" }, // isOnline is boolean, faceted filter usually expects string matches or we need to adapt accessor?
-        // Wait, accessor is boolean. Faceted filter usually works with strings.
-        // Let's check filter function.
-        { label: "Offline", value: "false" }
-    ];
 
-    const stateOptions = [
-        { label: "Disabled", value: "true" },
-        { label: "Enabled", value: "false" }
-    ];
 
     return (
         <div className="w-full space-y-4">

components/backup-repositories-table.tsx

@@ -20,10 +20,8 @@ import {
     MoreHorizontal,
     Search,
     RefreshCw,
-    AlertCircle,
-    Database,
-    Trash2,
     RotateCw,
+    Trash2,
 } from "lucide-react";
 
 import { Button } from "@/components/ui/button";

components/discovered-entities-table.tsx

@@ -13,7 +13,7 @@ import {
     getSortedRowModel,
     useReactTable,
 } from "@tanstack/react-table"
-import { ArrowUpDown, ChevronDown, MoreHorizontal, Search, RefreshCw, Smartphone, Laptop, Download, Trash, Ban } from "lucide-react"
+import { ArrowUpDown, ChevronDown, MoreHorizontal, Search, RefreshCw, Laptop, Download, Trash } from "lucide-react"
 import { Button } from "@/components/ui/button"
 import {
     DropdownMenu,