Merge pull request #255 from microsoft/code-quality

refactor: clean up imports and remove unused code across multiple components

Author: Roopan-Microsoft

src/frontend/src/commonComponents/ProgressModal/progressModal.tsx

@@ -272,10 +272,6 @@ const ProgressModal: React.FC<ProgressModalProps> = ({
                       const actionsMatch = raw.match(/📊\s*(\d+)\s*actions?/);
                       const actionCount = actionsMatch ? parseInt(actionsMatch[1]) : 0;
 
-                      // Extract blocking info from 🚧 segment
-                      const blockingMatch = raw.match(/🚧\s*Blocking\s*(\d+)/);
-                      const blockingCount = blockingMatch ? parseInt(blockingMatch[1]) : 0;
-
                       // Special handling for Coordinator: parse routing info from message
                       let coordinatorTarget = '';
                       let coordinatorInstruction = '';

src/frontend/src/components/bottomBar.tsx

@@ -1,6 +1,5 @@
 import { Button, Card, Dropdown, DropdownProps, Option } from "@fluentui/react-components"
 import React from "react"
-import { useNavigate } from "react-router-dom"
 
 // Define possible upload states
 const UploadState = {

src/frontend/src/main.jsx

@@ -106,22 +106,6 @@ const Main = () => {
 
     initMsal(); // Call the async function inside useEffect
   }, []);
-  async function checkConnection() {
-    if (!config) return;
-
-    const baseURL = config.API_URL.replace(/\/api$/, ''); // Remove '/api' if it appears at the end
-    console.log('Checking connection to:', baseURL);
-    try {
-      const response = await fetch(`${baseURL}/health`);
-      if (response.ok) {
-        console.log('========> Backend connection successful');
-      } else {
-        console.log('========> Backend responded with error status:', response.status);
-      }
-    } catch (error) {
-      console.log('========> Backend not available (this is okay for frontend-only testing):', error.message);
-    }
-  }
 
   // useEffect(() => {
   //   if (config) {

src/frontend/src/pages/batchView.tsx

@@ -38,7 +38,7 @@ import PanelRight from "../components/Panels/PanelRight";
 import PanelRightToolbar from "../components/Panels/PanelRightToolbar";
 import BatchHistoryPanel from "../components/batchHistoryPanel";
 import ConfirmationDialog from "../commonComponents/ConfirmationDialog/confirmationDialogue";
-import { determineFileStatus, filesLogsBuilder, renderErrorSection, useStyles, renderFileError, filesErrorCounter, completedFiles, hasFiles, fileErrorCounter, BatchSummary, fileWarningCounter } from "../api/utils";
+import { renderErrorSection, useStyles, renderFileError, BatchSummary } from "../api/utils";
 export const History = bundleIcon(HistoryFilled, HistoryRegular);
 
 

src/frontend/src/pages/landingPage.tsx

@@ -1,4 +1,4 @@
-import React, { useEffect, useRef, useState } from "react";
+import React, { useEffect } from "react";
 import { useSelector, useDispatch } from "react-redux";
 import { RootState } from "../store/store";
 import { togglePanel, closePanel } from "../slices/historyPanelSlice";
@@ -32,10 +32,8 @@ export const LandingPage = (): JSX.Element => {
     dispatch(resetState());
   }, [dispatch]);
 
-  const [uploadState, setUploadState] = useState<"IDLE" | "UPLOADING" | "COMPLETED">('IDLE');
-
-  const handleUploadStateChange = (state) => {
-    setUploadState(state);
+  const handleUploadStateChange = (_state) => {
+    // State change handler - currently unused but kept for future use
   };
 
   const handleStartTranslating = async () => {

src/frontend/src/pages/modernizationPage.tsx

@@ -32,7 +32,7 @@ import { vs } from "react-syntax-highlighter/dist/esm/styles/hljs"
 const unwrapHljs = (mod: any) => (typeof mod === "function" ? mod : mod.default);
 SyntaxHighlighter.registerLanguage("sql", unwrapHljs(sqlLang))
 import { useNavigate, useParams } from "react-router-dom"
-import { useState, useEffect, useCallback, useRef } from "react"
+import { useState, useEffect } from "react"
 import { getApiUrl, headerBuilder } from '../api/config';
 import BatchHistoryPanel from "../components/batchHistoryPanel"
 import PanelRight from "../components/Panels/PanelRight";
@@ -412,25 +412,6 @@ const fetchBatchSummary = async (batchId: string): Promise<any> => {
   }
 };
 
-
-enum ProcessingStage {
-  NotStarted = 1,
-  Queued = 10,
-  Starting = 20,
-  Parsing = 40,
-  Processing = 60,
-  FinalChecks = 95,
-  Completed = 100
-}
-
-enum Agents {
-  Verifier = "Semantic Verifier agent",
-  Checker = "Syntax Checker agent",
-  Picker = "Picker agent",
-  Migrator = "Migrator agent",
-  Agents = "Agent"
-}
-
 const getPrintFileStatus = (status: string): string => {
   switch (status) {
     case "completed":
@@ -481,7 +462,7 @@ const ModernizationPage = () => {
         const selectedFile = files.find((f) => f.id === selectedFileId);
         if (!selectedFile || !selectedFile.translatedCode) {
           setFileLoading(true);
-          const newFileUpdate = await fetchFileFromAPI(selectedFile?.fileId || "");
+          await fetchFileFromAPI(selectedFile?.fileId || "");
           setFileLoading(false);
         }
 

src/frontend/src/pages/processPage.tsx

@@ -3,10 +3,6 @@ import { useSelector, useDispatch } from "react-redux";
 import { useNavigate, useParams } from "react-router-dom";
 import { RootState } from "../store/store";
 import { togglePanel, closePanel } from "../slices/historyPanelSlice";
-import {
-  Button,
-  Tooltip,
-} from "@fluentui/react-components";
 import { MessageBar, MessageBarType } from "@fluentui/react";
 import Header from "../components/Header/Header";
 import HeaderTools from "../components/Header/HeaderTools";