fix: Avoid excessive sendDiagnostic calls (#183)

Signed-off-by: Arunprasad Rajkumar <[email protected]>
fabric8-analytics · Feb 5, 2021 · 158b1b1 · 158b1b1
1 parent 88969e1
commit 158b1b1
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/release.config.js b/release.config.js
@@ -29,7 +29,7 @@ module.exports = {
         {
           path: "analytics-lsp-linux",
           name: "analytics-lsp-linux",
-          label: "analytics-lsp-server"
+          label: "analytics-lsp-linux"
         },
         {
           path: "analytics-lsp-macos",
@@ -44,7 +44,7 @@ module.exports = {
         {
           path: "analytics-lsp-linux.sha256",
           name: "analytics-lsp-linux.sha256",
-          label: "analytics-lsp-server.sha256"
+          label: "analytics-lsp-linux.sha256"
         },
         {
           path: "analytics-lsp-macos.sha256",

diff --git a/src/server.ts b/src/server.ts
@@ -238,8 +238,9 @@ function runPipeline(response, diagnostics, packageAggregator, diagnosticFilePat
             totalCount.advisoryCount += secEng.advisoryCount;
             totalCount.exploitCount += secEng.exploitCount;
         }
-        connection.sendDiagnostics({ uri: diagnosticFilePath, diagnostics: diagnostics });
     });
+    connection.sendDiagnostics({ uri: diagnosticFilePath, diagnostics: diagnostics });
+    connection.console.log(`sendDiagnostics: ${diagnostics?.length}`);
 }
 
 /* Slice payload in each chunk size of @batchSize */