Fix the issue with the missing diagnostic.data

elm-tooling · Feb 4, 2024 · 3ac9d6e · 3ac9d6e
1 parent edd6813
commit 3ac9d6e
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/src/common/providers/codeActionProvider.ts b/src/common/providers/codeActionProvider.ts
@@ -293,6 +293,12 @@ export class CodeActionProvider {
     // and the fix all code action for that error if there are other diagnostics with
     // the same error code
     (<IDiagnostic[]>params.context.diagnostics).forEach((diagnostic) => {
+      // We have type casted diagnostic to `IDiagnostic`, however the original
+      // type is `Diagnostic`, the `.data` property may be missing in some cases
+      if (!diagnostic.data) {
+        return;
+      }
+
       const registrations =
         CodeActionProvider.errorCodeToRegistrationMap.getAll(
           diagnostic.data.code,