@typescript-eslint/eslint-plugin 8.70.1-alpha.20 → 8.70.1-alpha.22

package/dist/index.d.ts

@@ -994,7 +994,7 @@ declare const _default: {
         'no-useless-constructor': import("@typescript-eslint/utils/ts-eslint").RuleModule<"noUselessConstructor" | "removeConstructor", [], import("../rules").ESLintPluginDocs, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
             name: string;
         };
-        'no-useless-default-assignment': import("@typescript-eslint/utils/ts-eslint").RuleModule<"noStrictNullCheck" | "preferOptionalSyntax" | "uselessDefaultAssignment" | "uselessUndefined", [{
+        'no-useless-default-assignment': import("@typescript-eslint/utils/ts-eslint").RuleModule<"noStrictNullCheck" | "preferOptionalSyntax" | "removeDefaultAssignment" | "useOptionalSyntax" | "uselessDefaultAssignment" | "uselessUndefined", [{
             allowRuleToRunWithoutStrictNullChecksIKnowWhatIAmDoing?: boolean;
         }], import("../rules").ESLintPluginDocs, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
             name: string;

package/dist/raw-plugin.d.ts

@@ -1017,7 +1017,7 @@ declare const _default: {
             'no-useless-constructor': TSESLint.RuleModule<"noUselessConstructor" | "removeConstructor", [], import("../rules").ESLintPluginDocs, TSESLint.RuleListener> & {
                 name: string;
             };
-            'no-useless-default-assignment': TSESLint.RuleModule<"noStrictNullCheck" | "preferOptionalSyntax" | "uselessDefaultAssignment" | "uselessUndefined", [{
+            'no-useless-default-assignment': TSESLint.RuleModule<"noStrictNullCheck" | "preferOptionalSyntax" | "removeDefaultAssignment" | "useOptionalSyntax" | "uselessDefaultAssignment" | "uselessUndefined", [{
                 allowRuleToRunWithoutStrictNullChecksIKnowWhatIAmDoing?: boolean;
             }], import("../rules").ESLintPluginDocs, TSESLint.RuleListener> & {
                 name: string;

package/dist/rules/index.d.ts

@@ -315,7 +315,7 @@ declare const rules: {
     'no-useless-constructor': import("@typescript-eslint/utils/ts-eslint").RuleModule<"noUselessConstructor" | "removeConstructor", [], import("../../rules").ESLintPluginDocs, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
         name: string;
     };
-    'no-useless-default-assignment': import("@typescript-eslint/utils/ts-eslint").RuleModule<"noStrictNullCheck" | "preferOptionalSyntax" | "uselessDefaultAssignment" | "uselessUndefined", [{
+    'no-useless-default-assignment': import("@typescript-eslint/utils/ts-eslint").RuleModule<"noStrictNullCheck" | "preferOptionalSyntax" | "removeDefaultAssignment" | "useOptionalSyntax" | "uselessDefaultAssignment" | "uselessUndefined", [{
         allowRuleToRunWithoutStrictNullChecksIKnowWhatIAmDoing?: boolean;
     }], import("../../rules").ESLintPluginDocs, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {
         name: string;

package/dist/rules/no-useless-default-assignment.d.ts

@@ -1,5 +1,5 @@
 import type { TSESLint } from '@typescript-eslint/utils';
-type MessageId = 'noStrictNullCheck' | 'preferOptionalSyntax' | 'uselessDefaultAssignment' | 'uselessUndefined';
+type MessageId = 'noStrictNullCheck' | 'preferOptionalSyntax' | 'removeDefaultAssignment' | 'uselessDefaultAssignment' | 'uselessUndefined' | 'useOptionalSyntax';
 type Options = [
     {
         allowRuleToRunWithoutStrictNullChecksIKnowWhatIAmDoing?: boolean;

package/dist/rules/no-useless-default-assignment.js

@@ -46,12 +46,14 @@ exports.default = (0, util_1.createRule)({
             recommended: 'strict',
             requiresTypeChecking: true,
         },
-        fixable: 'code',
+        hasSuggestions: true,
         messages: {
             noStrictNullCheck: 'This rule requires the `strictNullChecks` compiler option to be turned on to function correctly.',
-            preferOptionalSyntax: 'Using `= undefined` to make a parameter optional adds unnecessary runtime logic. Use the `?` optional syntax instead.',
+            preferOptionalSyntax: 'Using `= undefined` to make a parameter optional adds unnecessary runtime logic.',
+            removeDefaultAssignment: 'Remove the default value.',
             uselessDefaultAssignment: 'Default value is useless because the {{ type }} is not optional.',
             uselessUndefined: 'Default value is useless because it is undefined. Optional {{ type }}s are already undefined by default.',
+            useOptionalSyntax: 'Use the `?` optional syntax instead.',
         },
         schema: [
             {
@@ -286,7 +288,7 @@ exports.default = (0, util_1.createRule)({
                 node: node.right,
                 messageId: 'uselessDefaultAssignment',
                 data: { type },
-                fix: fixer => removeDefault(fixer, node),
+                suggest: [removeDefaultSuggestion(node)],
             });
         }
         function reportUselessUndefined(node, type) {
@@ -294,13 +296,19 @@ exports.default = (0, util_1.createRule)({
                 node: node.right,
                 messageId: 'uselessUndefined',
                 data: { type },
-                fix: fixer => removeDefault(fixer, node),
+                suggest: [removeDefaultSuggestion(node)],
             });
         }
         function reportPreferOptionalSyntax(node) {
             context.report({
                 node: node.right,
                 messageId: 'preferOptionalSyntax',
+                suggest: [useOptionalSyntaxSuggestion(node)],
+            });
+        }
+        function useOptionalSyntaxSuggestion(node) {
+            return {
+                messageId: 'useOptionalSyntax',
                 *fix(fixer) {
                     yield removeDefault(fixer, node);
                     const { left } = node;
@@ -308,7 +316,13 @@ exports.default = (0, util_1.createRule)({
                         yield fixer.insertTextAfterRange([left.range[0], left.range[0] + left.name.length], '?');
                     }
                 },
-            });
+            };
+        }
+        function removeDefaultSuggestion(node) {
+            return {
+                messageId: 'removeDefaultAssignment',
+                fix: fixer => removeDefault(fixer, node),
+            };
         }
         function removeDefault(fixer, node) {
             const start = node.left.range[1];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@typescript-eslint/eslint-plugin",
-  "version": "8.70.1-alpha.20",
+  "version": "8.70.1-alpha.22",
   "description": "TypeScript plugin for ESLint",
   "files": [
     "dist",
@@ -47,10 +47,10 @@
   ],
   "dependencies": {
     "@eslint-community/regexpp": "^4.12.2",
-    "@typescript-eslint/scope-manager": "8.70.1-alpha.20",
-    "@typescript-eslint/type-utils": "8.70.1-alpha.20",
-    "@typescript-eslint/utils": "8.70.1-alpha.20",
-    "@typescript-eslint/visitor-keys": "8.70.1-alpha.20",
+    "@typescript-eslint/scope-manager": "8.70.1-alpha.22",
+    "@typescript-eslint/type-utils": "8.70.1-alpha.22",
+    "@typescript-eslint/utils": "8.70.1-alpha.22",
+    "@typescript-eslint/visitor-keys": "8.70.1-alpha.22",
     "ignore": "^7.0.5",
     "natural-compare": "^1.4.0",
     "ts-api-utils": "^2.5.0"
@@ -60,8 +60,8 @@
     "@types/mdast": "^4.0.4",
     "@types/natural-compare": "^1.4.3",
     "@types/react": "^18.3.21",
-    "@typescript-eslint/rule-schema-to-typescript-types": "8.70.1-alpha.20",
-    "@typescript-eslint/rule-tester": "8.70.1-alpha.20",
+    "@typescript-eslint/rule-schema-to-typescript-types": "8.70.1-alpha.22",
+    "@typescript-eslint/rule-tester": "8.70.1-alpha.22",
     "@typescript/native": "npm:typescript@^7.0.2",
     "@vitest/coverage-v8": "^4.0.18",
     "ajv": "^6.12.6",
@@ -81,7 +81,7 @@
     "vitest": "^4.0.18"
   },
   "peerDependencies": {
-    "@typescript-eslint/parser": "^8.70.1-alpha.20",
+    "@typescript-eslint/parser": "^8.70.1-alpha.22",
     "eslint": "^8.57.0 || ^9.0.0 || ^10.0.0",
     "typescript": ">=4.8.4 <6.1.0"
   },