update AGLint to v4.0.0-alpha.7 and use type guards for fix commands

- Update AGLint dependency from v4.0.0-alpha.6 to v4.0.0-alpha.7
- Update minimum supported AGLint version to v4.0.0-alpha.7
- Remove LinterSuggestion type import
- Use AGLint's built-in type guards (isLinterFixCommand, isLinterSuggestions) instead of type assertions
- Refactor code action handling to validate fix and suggestion types before processing

Author: totadavid95

pnpm-lock.yaml

@@ -4,4 +4,4 @@ packages:
   - test/static/aglint
 
 catalog:
-  '@adguard/aglint': ^4.0.0-alpha.6
+  '@adguard/aglint': ^4.0.0-alpha.7

pnpm-workspace.yaml

@@ -18,7 +18,6 @@ import type {
     LinterPositionRange,
     LinterResult,
     LinterRunOptions,
-    LinterSuggestion,
 } from '@adguard/aglint/linter';
 import { CommentMarker, type ConfigCommentRule, ConfigCommentRuleParser } from '@adguard/agtree';
 import debounce from 'debounce';
@@ -538,34 +537,10 @@ connection.onCodeAction((params) => {
 
         if (diagnostic.data?.fix) {
             // eslint-disable-next-line prefer-destructuring
-            const fix = diagnostic.data.fix as LinterFixCommand;
-            const actionFix = CodeAction.create(`Fix AGLint rule '${code}'`, CodeActionKind.QuickFix);
+            const fix = diagnostic.data.fix;
 
-            actionFix.edit = {
-                documentChanges: [
-                    TextDocumentEdit.create(
-                        { uri: textDocument.uri, version: textDocument.version },
-                        [
-                            convertAglintFixToVsCodeCodeEdit(
-                                textDocument,
-                                fix,
-                            ),
-                        ],
-                    ),
-                ],
-            };
-            actions.push(actionFix);
-        }
-
-        if (diagnostic.data?.suggestions) {
-            // eslint-disable-next-line prefer-destructuring
-            const suggestions: LinterSuggestion[] = diagnostic.data.suggestions;
-
-            for (const suggestion of suggestions) {
-                const actionFix = CodeAction.create(
-                    `Apply suggestion '${suggestion.message}' from AGLint rule '${code}'`,
-                    CodeActionKind.QuickFix,
-                );
+            if (aglint?.linter.isLinterFixCommand(fix)) {
+                const actionFix = CodeAction.create(`Fix AGLint rule '${code}'`, CodeActionKind.QuickFix);
 
                 actionFix.edit = {
                     documentChanges: [
@@ -574,7 +549,7 @@ connection.onCodeAction((params) => {
                             [
                                 convertAglintFixToVsCodeCodeEdit(
                                     textDocument,
-                                    suggestion.fix as LinterFixCommand,
+                                    fix,
                                 ),
                             ],
                         ),
@@ -584,6 +559,35 @@ connection.onCodeAction((params) => {
             }
         }
 
+        if (diagnostic.data?.suggestions) {
+            // eslint-disable-next-line prefer-destructuring
+            const suggestions = diagnostic.data.suggestions;
+
+            if (aglint?.linter.isLinterSuggestions(suggestions)) {
+                for (const suggestion of suggestions) {
+                    const actionFix = CodeAction.create(
+                        `Apply suggestion '${suggestion.message}' from AGLint rule '${code}'`,
+                        CodeActionKind.QuickFix,
+                    );
+
+                    actionFix.edit = {
+                        documentChanges: [
+                            TextDocumentEdit.create(
+                                { uri: textDocument.uri, version: textDocument.version },
+                                [
+                                    convertAglintFixToVsCodeCodeEdit(
+                                        textDocument,
+                                        suggestion.fix,
+                                    ),
+                                ],
+                            ),
+                        ],
+                    };
+                    actions.push(actionFix);
+                }
+            }
+        }
+
         // If we are here, it means that we have a linter rule name,
         // so we need to suggest disabling this rule for the line
         const action = CodeAction.create(`Disable AGLint rule '${code}' for this line`, CodeActionKind.QuickFix);

server/src/server.ts

@@ -119,7 +119,7 @@ export class LoadedAglint {
  * Minimum version of the external AGLint module that is supported by the VSCode extension.
  * If the version is lower than this, the extension cannot use the external AGLint module.
  */
-const MIN_AGLINT_VERSION = '4.0.0-alpha.4';
+const MIN_AGLINT_VERSION = '4.0.0-alpha.7';
 
 /**
  * Load the installed AGLint module.