Remove unused type definitions and improve error handling in API utility functions

Author: Pavan-Microsoft

src/ContentProcessorWeb/package.json

@@ -61,7 +61,7 @@
     "@types/node": "^22.15.0",
     "@types/react": "^19.2.14",
     "@types/react-dom": "^19.2.3",
-    "@types/react-router-dom": "^5.3.3",
+
     "@types/react-virtualized": "^9.22.3",
     "@types/react-window": "^2.0.0",
     "customize-cra": "^1.0.0",

src/ContentProcessorWeb/pnpm-lock.yaml

@@ -183,7 +183,7 @@ const PanelCenter: React.FC<PanelCenterProps> = ({ togglePanel }) => {
       }
     }
     // Only fetch for document selection
-    if (store.selectionType === 'document' && (store.activeProcessId != null || store.activeProcessId !== '') && !status.includes(store.selectedItem.status as string) && store.selectedItem?.process_id === store.activeProcessId) {
+    if (store.selectionType === 'document' && (store.activeProcessId != null && store.activeProcessId !== '') && !status.includes(store.selectedItem.status as string) && store.selectedItem?.process_id === store.activeProcessId) {
       fetchContent();
     }
   }, [store.activeProcessId, store.selectedItem, store.selectionType, store.refreshTrigger])
@@ -291,7 +291,7 @@ const PanelCenter: React.FC<PanelCenterProps> = ({ togglePanel }) => {
     try {
       dispatch(startLoader("1"));
       dispatch(setUpdateComments(comment))
-      const result = await dispatch(saveContentJson({ 'processId': store.activeProcessId, 'contentJson': store.modified_result as unknown as string, 'comments': comment, 'savedComments': store.comments }))
+      const result = await dispatch(saveContentJson({ 'processId': store.activeProcessId, 'contentJson': JSON.stringify(store.modified_result), 'comments': comment, 'savedComments': store.comments }))
       if (result?.type === 'SaveContentJSON-Comments/fulfilled') {
         dispatch(setRefreshGrid(true));
       }

src/ContentProcessorWeb/src/Pages/DefaultPage/PanelCenter.tsx

@@ -52,14 +52,14 @@ export const handleApiThunk = async <T>(
     if (response.status === 200 || response.status === 202) {
       return response.data as T;
     } else {
-      return rejectWithValue(`${errorMessage}. Status: ${response.status}`) as T;
+      return rejectWithValue(`${errorMessage}. Status: ${response.status}`);
     }
   } catch (error: unknown) {
     const apiError = error as ApiError;
     if (apiError.status === 415 || apiError.status === 404) {
-      return rejectWithValue(apiError.data?.message || `Unexpected error: ${errorMessage}`) as T;
+      return rejectWithValue(apiError.data?.message || `Unexpected error: ${errorMessage}`);
     }
-    return rejectWithValue(apiError.message || `Unexpected error: ${errorMessage}`) as T;
+    return rejectWithValue(apiError.message || `Unexpected error: ${errorMessage}`);
   }
 };
 

src/ContentProcessorWeb/src/Services/httpUtility.ts

@@ -491,9 +491,6 @@ def validate_source_document_visible(self):
         logger.info("Starting Source Document pane validation...")
 
         logger.info("Validating Source Document pane has content...")
-        _source_doc_pane = self.page.locator(
-            "//div[contains(text(),'Source Document')]/ancestor::div[1]/following-sibling::*"
-        )
 
         # Verify "No document available" is NOT shown
         no_data = self.page.locator(self.SOURCE_DOC_NO_DATA)
@@ -1139,7 +1136,7 @@ def upload_files_with_mismatched_schemas(self):
             "repair_estimate.pdf": "Auto Insurance Claim Form",
         }
 
-        _dialog = self.open_import_dialog_with_files()
+        self.open_import_dialog_with_files()
 
         # Select mismatched schemas for each file
         files = self.get_testdata_files()
@@ -1195,7 +1192,6 @@ def validate_schema_dropdown_after_file_removal(self):
 
         # Try to remove the first file using the delete/remove button next to it
         logger.info("Attempting to remove first file from the list...")
-        _file_labels = dialog.locator("strong")
         first_file_name = os.path.basename(files[0])
 
         # Look for a delete/remove button near the first file entry