Merge pull request #234 from microsoft/psl-fixcopilotcomments-cm

fix: fixed all copilot comments

Author: Roopan-Microsoft

src/backend-api/src/app/libs/base/SKLogicBase.py

@@ -1,5 +1,5 @@
 from abc import ABC, abstractmethod
-from typing import Type, TypeVar, overload
+from typing import Any, Type, TypeVar, overload
 
 from pydantic import BaseModel, Field
 from semantic_kernel.agents import (
@@ -81,7 +81,7 @@ def _init_agent(self):
         """
         raise NotImplementedError("This method should be overridden in subclasses")
 
-    async def execute(self, func_params: dict[str, any]):
+    async def execute(self, func_params: dict[str, Any]):
         raise NotImplementedError("Execute method not implemented")
 
     @overload

src/frontend/src/pages/modernizationPage.tsx

@@ -1000,7 +1000,9 @@ useEffect(() => {
         );
       }
       // Otherwise, show the progress view with summary information
-      const fileIndex = files.findIndex(file => file.fileId === fileId);
+      // selectedFileId/fileId is the internal UI id (e.g. "summary"/"file0"),
+      // so match against file.id rather than the server-side file.fileId.
+      const fileIndex = files.findIndex(file => file.id === fileId);
       const currentFile = files[fileIndex];
       return (
         <>

src/processor/src/services/queue_service.py

@@ -1119,8 +1119,6 @@ async def _process_queue_message(self, worker_id: int, queue_message: QueueMessa
                     execution_time,
                     task_param=task_param,
                 )
-            finally:
-                migration_processor = None  # noqa: F841 — release reference for GC
 
         except asyncio.CancelledError:
             # When cancelled, we assume stop_process has already deleted the message