fix: DH-18542: Remove duplicate and invalid ruff quick fixes

packages/console/src/monaco/MonacoProviders.tsx

@@ -251,15 +251,17 @@
       };
     }
 
-    const diagnostics = MonacoProviders.getDiagnostics(model).filter(d => {
-      const diagnosticRange = new monaco.Range(
-        d.location.row,
-        d.location.column,
-        d.end_location.row,
-        d.end_location.column
-      );
-      return diagnosticRange.intersectRanges(range);
-    });
+    const diagnostics = MonacoProviders.getDiagnostics(model)
+      .filter(d => d.code != null) // Syntax errors have no code and can't be fixed/disabled
+      .filter(d => {
+        const diagnosticRange = new monaco.Range(
+          d.location.row,
+          d.location.column,
+          d.end_location.row,
+          d.end_location.column
+        );
+        return diagnosticRange.intersectRanges(range);
+      });
 
     const fixActions: monaco.languages.CodeAction[] = diagnostics
       .filter(({ fix }) => fix != null)
@@ -272,6 +274,7 @@
             title = `Fix ${d.code}`;
           }
         }
+        const b = 4;
         return {
           title,
           id: `fix-${d.code}`,
@@ -296,8 +299,23 @@
         };
       });
 
-    const disableActions: monaco.languages.CodeAction[] = diagnostics
+    const seenCodes = new Set<string>();
+    const duplicateCodes = new Set<string>();
+    diagnostics.forEach(d => {
+      if (d.code == null) {
+        return;
+      }
+      if (seenCodes.has(d.code)) {
+        duplicateCodes.add(d.code);
+      }
+      seenCodes.add(d.code);
+    });
+
+    const disableLineActions: monaco.languages.CodeAction[] = diagnostics
       .map(d => {
+        if (d.code == null) {
+          return [];
+        }
         const line = model.getLineContent(d.location.row);
         const lastToken = monaco.editor
           .tokenize(line, model.getLanguageId())[0]
@@ -327,7 +345,11 @@
         }
         return [
           {
-            title: `Disable ${d.code} for this line`,
+            title: `Disable ${d.code} for ${
+              duplicateCodes.has(d.code)
+                ? `line ${d.location.row}`
+                : 'this line'
+            }`,
             kind: 'quickfix',
             edit: {
               edits: [
@@ -339,33 +361,40 @@
               ],
             },
           },
+        ];
+      })
+      .flat()
+      .filter(
+        // Remove actions with duplicate titles as you can't disable the same rule on a line twice
+        (action, i, arr) => arr.find(a => a.title === action.title) === action
+      );
+
+    const disableGlobalActions: monaco.languages.CodeAction[] = [
+      ...seenCodes,
+    ].map(code => ({
+      title: `Disable ${code} for this file`,
+      kind: 'quickfix',
+      edit: {
+        edits: [
           {
-            title: `Disable ${d.code} for this file`,
-            kind: 'quickfix',
-            edit: {
-              edits: [
-                {
-                  resource: model.uri,
-                  versionId: model.getVersionId(),
-                  textEdit: {
-                    range: {
-                      startLineNumber: 1,
-                      startColumn: 1,
-                      endLineNumber: 1,
-                      endColumn: 1,
-                    },
-                    text: `# ruff: noqa: ${d.code}\n`,
-                  },
-                },
-              ],
+            resource: model.uri,
+            versionId: model.getVersionId(),
+            textEdit: {
+              range: {
+                startLineNumber: 1,
+                startColumn: 1,
+                endLineNumber: 1,
+                endColumn: 1,
+              },
+              text: `# ruff: noqa: ${code}\n`,
             },
           },
-        ];
-      })
-      .flat();
+        ],
+      },
+    }));
 
     return {
-      actions: [...fixActions, ...disableActions],
+      actions: [...fixActions, ...disableLineActions, ...disableGlobalActions],
       dispose: () => {
         /* no-op */
       },