perf(diagnostics): use incremental updates with debounce

Author: 9romise

package.json

@@ -115,6 +115,7 @@
     "module-replacements": "^2.11.0",
     "nano-staged": "^0.9.0",
     "ofetch": "^2.0.0-alpha.3",
+    "perfect-debounce": "^2.1.0",
     "reactive-vscode": "^0.4.1",
     "tsdown": "^0.20.1",
     "typescript": "^5.9.3",

pnpm-lock.yaml

@@ -5,6 +5,7 @@ import type { Diagnostic, TextDocument } from 'vscode'
 import { basename } from 'node:path'
 import { config, logger } from '#state'
 import { getPackageInfo } from '#utils/api/package'
+import { debounce } from 'perfect-debounce'
 import { computed, useActiveTextEditor, useDocumentText, watch } from 'reactive-vscode'
 import { languages } from 'vscode'
 import { displayName } from '../../generated-meta'
@@ -44,35 +45,35 @@ export function registerDiagnosticCollection(mapping: Record<string, Extractor |
     const dependencies = extractor.getDependenciesInfo(root)
     const diagnostics: Diagnostic[] = []
 
-    await Promise.all(
-      dependencies.map(async (dep) => {
-        try {
-          const pkg = await getPackageInfo(dep.name)
-          if (!pkg)
-            return
+    const flush = debounce(() => {
+      diagnosticCollection.set(document.uri, [...diagnostics])
+    }, 100)
 
-          for (const rule of enabledRules.value) {
-            const diagnostic = await rule(dep, pkg)
+    dependencies.forEach(async (dep) => {
+      try {
+        const pkg = await getPackageInfo(dep.name)
+        if (!pkg)
+          return
 
-            if (diagnostic) {
-              diagnostics.push({
-                source: displayName,
-                message: diagnostic.message,
-                severity: diagnostic.severity,
-                code: diagnostic.code,
-                range: extractor.getNodeRange(document, diagnostic.node),
-              })
-            }
-          }
-        } catch (err) {
-          logger.warn(`Failed to check ${dep.name}: ${err}`)
-        }
-      }),
-    )
+        enabledRules.value.forEach(async (rule) => {
+          const diagnostic = await rule(dep, pkg)
+
+          if (diagnostic) {
+            diagnostics.push({
+              source: displayName,
+              message: diagnostic.message,
+              severity: diagnostic.severity,
+              code: diagnostic.code,
+              range: extractor.getNodeRange(document, diagnostic.node),
+            })
 
-    if (diagnostics.length > 0)
-      logger.info(`${diagnostics.length} diagnostic collected in ${document.fileName}.`)
-    diagnosticCollection.set(document.uri, diagnostics)
+            flush()
+          }
+        })
+      } catch (err) {
+        logger.warn(`Failed to check ${dep.name}: ${err}`)
+      }
+    })
   }
 
   watch(activeDocumentText, async () => {

src/providers/diagnostics/index.ts

@@ -16,6 +16,7 @@ export default defineConfig({
     'yaml',
     'ofetch',
     'fast-npm-meta',
+    'perfect-debounce',
   ],
   minify: 'dce-only',
 })