fixed policy violation not triggering when manifest file is already o…

…pened
debricked · Sep 26, 2024 · 6443e9d · 6443e9d
1 parent eb91bbd
commit 6443e9d
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 2 deletions.
diff --git a/src/extension.ts b/src/extension.ts
@@ -36,7 +36,6 @@ export async function activate(context: vscode.ExtensionContext) {
 
                 await debrickedCommand.commands(context);
                 await providers.registerHover(context);
-                await providers.registerDependencyPolicyProvider(context);
 
                 const debCommandsProvider = new DebrickedCommandsTreeDataProvider();
                 vscode.window.registerTreeDataProvider(Organization.debrickedCommand, debCommandsProvider);
@@ -64,6 +63,7 @@ export async function activate(context: vscode.ExtensionContext) {
 
                 // Add file watcher for all files found from 'debricked files find'
                 await watchers.registerWatcher(context);
+                await providers.registerDependencyPolicyProvider(context); // after adding watcher and scanning we should add the policy provider
 
                 progress.report({ message: "Debricked extension is ready to use", increment: 100 - progressCount });
                 await new Promise((resolve) => setTimeout(resolve, 1000)); // added for showing the last progress info

diff --git a/src/providers/index.ts b/src/providers/index.ts
@@ -37,6 +37,10 @@ class Providers {
                 context.subscriptions.push(diagnosticCollection);
 
                 const provider = new DependencyPolicyProvider(diagnosticCollection);
+                //added to activate the policy violation provider when the manifest file is already open
+                if (vscode.window.activeTextEditor?.document) {
+                    provider.checkPolicyViolation(vscode.window.activeTextEditor?.document);
+                }
 
                 context.subscriptions.push(
                     vscode.languages.registerCodeActionsProvider({ scheme: "file" }, provider, {

diff --git a/src/services/scanService.ts b/src/services/scanService.ts
@@ -54,7 +54,7 @@ export class ScanService {
                 }
             }
 
-            vscode.window.withProgress(
+            await vscode.window.withProgress(
                 {
                     location: vscode.ProgressLocation.Window,
                     title: Organization.nameCaps,