feat: expose hover, diagnostics, and completion settings

Author: 9romise

README.md

@@ -32,9 +32,13 @@
 
 <!-- configs -->
 
-| Key                      | Description                                                                                                                                     | Type     | Default             |
-| ------------------------ | ----------------------------------------------------------------------------------------------------------------------------------------------- | -------- | ------------------- |
-| `npmx.versionCompletion` | Version completion behavior: 'all' shows all versions, 'provenance-only' shows only versions with provenance, 'off' disables version completion | `string` | `"provenance-only"` |
+| Key                              | Description                                           | Type      | Default             |
+| -------------------------------- | ----------------------------------------------------- | --------- | ------------------- |
+| `npmx.hover.enabled`             | Enable hover information for packages                 | `boolean` | `true`              |
+| `npmx.completion.version`        | Version completion behavior                           | `string`  | `"provenance-only"` |
+| `npmx.diagnostics.deprecation`   | Show warnings for deprecated packages                 | `boolean` | `true`              |
+| `npmx.diagnostics.replacement`   | Show suggestions for package replacements             | `boolean` | `true`              |
+| `npmx.diagnostics.vulnerability` | Show warnings for packages with known vulnerabilities | `boolean` | `true`              |
 
 <!-- configs -->
 

package.json

@@ -42,18 +42,38 @@
   ],
   "contributes": {
     "configuration": {
-      "title": "Only suggest version",
+      "title": "npmx",
       "properties": {
-        "npmx.versionCompletion": {
+        "npmx.hover.enabled": {
+          "type": "boolean",
+          "default": true,
+          "description": "Enable hover information for packages"
+        },
+        "npmx.completion.version": {
           "type": "string",
           "enum": ["all", "provenance-only", "off"],
           "enumDescriptions": [
-            "show all versions",
-            "show only versions with provenance",
-            "disables version completion"
+            "Show all versions",
+            "Show only versions with provenance",
+            "Disable version completion"
           ],
           "default": "provenance-only",
-          "description": "Version completion behavior: 'all' shows all versions, 'provenance-only' shows only versions with provenance, 'off' disables version completion"
+          "description": "Version completion behavior"
+        },
+        "npmx.diagnostics.deprecation": {
+          "type": "boolean",
+          "default": true,
+          "description": "Show warnings for deprecated packages"
+        },
+        "npmx.diagnostics.replacement": {
+          "type": "boolean",
+          "default": true,
+          "description": "Show suggestions for package replacements"
+        },
+        "npmx.diagnostics.vulnerability": {
+          "type": "boolean",
+          "default": true,
+          "description": "Show warnings for packages with known vulnerabilities"
         }
       }
     }

src/index.ts

@@ -21,18 +21,20 @@ export const { activate, deactivate } = defineExtension((ctx) => {
   const packageJsonExtractor = new PackageJsonExtractor()
   const pnpmWorkspaceYamlExtractor = new PnpmWorkspaceYamlExtractor()
 
-  ctx.subscriptions.push(
-    languages.registerHoverProvider(
-      { pattern: PACKAGE_JSON_PATTERN },
-      new NpmxHoverProvider(packageJsonExtractor),
-    ),
-    languages.registerHoverProvider(
-      { pattern: PNPM_WORKSPACE_PATTERN },
-      new NpmxHoverProvider(pnpmWorkspaceYamlExtractor),
-    ),
-  )
+  if (config.hover.enabled) {
+    ctx.subscriptions.push(
+      languages.registerHoverProvider(
+        { pattern: PACKAGE_JSON_PATTERN },
+        new NpmxHoverProvider(packageJsonExtractor),
+      ),
+      languages.registerHoverProvider(
+        { pattern: PNPM_WORKSPACE_PATTERN },
+        new NpmxHoverProvider(pnpmWorkspaceYamlExtractor),
+      ),
+    )
+  }
 
-  if (config.versionCompletion !== 'off') {
+  if (config.completion.version !== 'off') {
     ctx.subscriptions.push(
       languages.registerCompletionItemProvider(
         { pattern: PACKAGE_JSON_PATTERN },

src/providers/completion-item/version.ts

@@ -34,7 +34,7 @@ export class VersionCompletionItemProvider<T extends Extractor> implements Compl
 
     let versionsKV = Object.values(pkg.versions)
 
-    if (config.versionCompletion === 'provenance-only')
+    if (config.completion.version === 'provenance-only')
       versionsKV = versionsKV.filter(({ hasProvenance }) => hasProvenance)
 
     return versionsKV.map(({ version, tag }) => {

src/providers/diagnostics/index.ts

@@ -3,7 +3,7 @@ import type { ResolvedPackument } from '#utils/npm'
 import type { Awaitable } from 'reactive-vscode'
 import type { Diagnostic, TextDocument } from 'vscode'
 import { basename } from 'node:path'
-import { logger } from '#state'
+import { config, logger } from '#state'
 import { getPackageInfo } from '#utils/npm'
 import { useActiveTextEditor, useDocumentText, watch } from 'reactive-vscode'
 import { languages } from 'vscode'
@@ -17,11 +17,16 @@ export interface NodeDiagnosticInfo extends Pick<Diagnostic, 'message' | 'severi
 }
 export type DiagnosticRule = (dep: DependencyInfo, pkg: ResolvedPackument) => Awaitable<NodeDiagnosticInfo | undefined>
 
-const rules: DiagnosticRule[] = [
-  checkDeprecation,
-  checkReplacement,
-  checkVulnerability,
-]
+function getEnabledRules(): DiagnosticRule[] {
+  const rules: DiagnosticRule[] = []
+  if (config.diagnostics.deprecation)
+    rules.push(checkDeprecation)
+  if (config.diagnostics.replacement)
+    rules.push(checkReplacement)
+  if (config.diagnostics.vulnerability)
+    rules.push(checkVulnerability)
+  return rules
+}
 
 export function registerDiagnosticCollection(mapping: Record<string, Extractor | undefined>) {
   const diagnosticCollection = languages.createDiagnosticCollection(displayName)
@@ -44,7 +49,7 @@ export function registerDiagnosticCollection(mapping: Record<string, Extractor |
         try {
           const pkg = await getPackageInfo(dep.name)
 
-          for (const rule of rules) {
+          for (const rule of getEnabledRules()) {
             const diagnostic = await rule(dep, pkg)
 
             if (diagnostic) {