fix: Invalidate detected Package Manager cache when a lock file is updated, created or deleted

This way it is possible to guarantee changes in the Package Manager that may have occurred (eg. changed from NPM to PNPM)

Author: david-rodrigues

src/Diagnostic.ts

@@ -32,6 +32,7 @@ import {
   getPackageManager,
   getPackagesAdvisories,
   PackageManager,
+  packageManagerCaches,
   PackagesAdvisories,
   packagesInstalledCaches,
 } from "./PackageManager"
@@ -82,19 +83,24 @@ export const diagnosticSubscribe = (
   )
 
   // Trigger when any file in the workspace is modified.
-  // Our interest here is to know about package-lock.json.
-  context.subscriptions.push(
-    workspace
-      .createFileSystemWatcher("**/{package-lock.json,pnpm-lock.yaml}")
-      .onDidChange((uri: Uri) => {
-        packagesInstalledCaches.get(getWorkspacePath(uri))?.invalidate()
-
-        window.visibleTextEditors.forEach((editor) =>
-          handleChange(editor.document)
-        )
-      })
+  // Our interest here is to know about package-lock.json or pnpm-lock.yaml.
+  const lockerWatcher = workspace.createFileSystemWatcher(
+    "**/{package-lock.json,pnpm-lock.yaml}"
   )
 
+  const lockerUpdated = (uri: Uri) => {
+    const workspacePath = getWorkspacePath(uri)
+
+    packageManagerCaches.get(workspacePath)?.invalidate()
+    packagesInstalledCaches.get(workspacePath)?.invalidate()
+
+    window.visibleTextEditors.forEach((editor) => handleChange(editor.document))
+  }
+
+  context.subscriptions.push(lockerWatcher.onDidCreate(lockerUpdated))
+  context.subscriptions.push(lockerWatcher.onDidChange(lockerUpdated))
+  context.subscriptions.push(lockerWatcher.onDidDelete(lockerUpdated))
+
   // Trigger when the active document is closed, removing the current document from the diagnostic collection.
   context.subscriptions.push(
     workspace.onDidCloseTextDocument((document: TextDocument) => {

src/PackageManager.ts

@@ -111,7 +111,7 @@ const supportsPackageManager = async (
   })
 }
 
-const packageManagerCaches = new Map<
+export const packageManagerCaches = new Map<
   string,
   Cache<PackageManager | undefined>
 >()