Merge pull request #364 from Checkmarx/feature/secrets_realtime

- Fixed CxFinding window issue-AST-107870

Author: cx-anurag-dalke

src/main/java/com/checkmarx/intellij/devassist/ui/findings/window/CxFindingsWindow.java

@@ -66,8 +66,7 @@
  * Uses a timer to periodically update the tab title with the current problem count.
  * Refactored to have separate drawAuthPanel() and drawMainPanel() following pattern in CxToolWindowPanel.
  */
-public class CxFindingsWindow extends SimpleToolWindowPanel
-        implements Disposable {
+public class CxFindingsWindow extends SimpleToolWindowPanel implements Disposable {
 
     private static final Logger LOGGER = Utils.getLogger(CxFindingsWindow.class);
 
@@ -151,7 +150,7 @@ public void mouseReleased(MouseEvent e) {
         // Trigger initial refresh with existing scan results if any (on EDT)
         SwingUtilities.invokeLater(() -> {
             Map<String, List<ScanIssue>> existingIssues = ProblemHolderService.getInstance(project).getAllIssues();
-            if (existingIssues != null && !existingIssues.isEmpty()) {
+            if (!existingIssues.isEmpty()) {
                 triggerRefreshTree();
             }
         });
@@ -219,12 +218,11 @@ private void drawMainPanel() {
      */
     private void triggerRefreshTree() {
         Map<String, List<ScanIssue>> allIssues = ProblemHolderService.getInstance(project).getAllIssues();
-        if (allIssues == null) {
+        if (allIssues.isEmpty()) {
             return;
         }
 
         Set<Filterable> activeFilters = VulnerabilityFilterState.getInstance().getFilters();
-
         Map<String, List<ScanIssue>> filteredIssues = new HashMap<>();
 
         for (Map.Entry<String, List<ScanIssue>> entry : allIssues.entrySet()) {
@@ -270,6 +268,18 @@ public void refreshTree(Map<String, List<ScanIssue>> issues) {
                     })
                     .collect(Collectors.toList());
 
+            ApplicationManager.getApplication().runReadAction(() ->
+                    createFileNode(filePath, filteredScanDetails, fileName));
+        }
+        ((DefaultTreeModel) tree.getModel()).reload();
+        expandNodesByFilePath();
+    }
+
+    /**
+     * Creating file node
+     */
+    private void createFileNode(String filePath, List<ScanIssue> filteredScanDetails, String fileName) {
+        try {
             VirtualFile virtualFile = LocalFileSystem.getInstance().findFileByPath(filePath);
             Icon icon = virtualFile != null ? virtualFile.getFileType().getIcon() : null;
             PsiFile psiFile = PsiManager.getInstance(project).findFile(virtualFile);
@@ -295,12 +305,10 @@ public void refreshTree(Map<String, List<ScanIssue>> issues) {
             for (ScanIssue detail : filteredScanDetails) {
                 fileNode.add(new DefaultMutableTreeNode(new ScanDetailWithPath(detail, filePath)));
             }
-
             rootNode.add(fileNode);
+        } catch (Exception e) {
+            LOGGER.warn("Failed to create file node for file: " + filePath, e);
         }
-        ((DefaultTreeModel) tree.getModel()).reload();
-
-        expandNodesByFilePath();
     }
 
     /**