feature: @putout/plugin-eslint: remove-useless-define-config: add

Author: coderaiser

packages/plugin-eslint/eslint.config.mjs

@@ -1,4 +1,3 @@
 import {safeAlign} from 'eslint-plugin-putout';
-import {defineConfig} from 'eslint/config';
 
-export default defineConfig([safeAlign, {}]);
+export default safeAlign;

packages/plugin-eslint/lib/index.js

@@ -1,3 +1,4 @@
+import * as removeUselessDefineConfig from './remove-useless-define-config/index.js';
 import * as convertRcToFlat from './convert-rc-to-flat/index.js';
 import * as declare from './declare/index.js';
 import * as removeNoMissing from './remove-no-missing/index.js';
@@ -54,4 +55,5 @@ export const rules = {
     'remove-create-eslint-config-with-one-argument': removeCreateEslintConfigWithOneArgument,
     'remove-useless-match-to-flat': removeUselessMatchToFlat,
     'apply-define-config': applyDefineConfig,
+    'remove-useless-define-config': removeUselessDefineConfig,
 };

packages/plugin-eslint/lib/remove-useless-define-config/fixture/empty-fix.js

@@ -0,0 +1 @@
+export default safeAlign;

packages/plugin-eslint/lib/remove-useless-define-config/fixture/empty.js

@@ -0,0 +1 @@
+export default defineConfig([safeAlign, {}]);

packages/plugin-eslint/lib/remove-useless-define-config/fixture/remove-useless-define-config-fix.js

@@ -0,0 +1 @@
+export default safeAlign;

packages/plugin-eslint/lib/remove-useless-define-config/fixture/remove-useless-define-config.js

@@ -0,0 +1 @@
+  export default defineConfig([safeAlign])

packages/plugin-eslint/lib/remove-useless-define-config/index.js

@@ -0,0 +1,6 @@
+export const report = () => `Avoid useless defineConfig()`;
+
+export const replace = () => ({
+    'export default defineConfig([__identifier])': 'export default __identifier',
+    'export default defineConfig([__identifier, {}])': 'export default __identifier',
+});

packages/plugin-eslint/lib/remove-useless-define-config/index.spec.js

@@ -0,0 +1,23 @@
+import {createTest} from '@putout/test';
+import * as plugin from './index.js';
+
+const test = createTest(import.meta.url, {
+    plugins: [
+        ['remove-useless-define-config', plugin],
+    ],
+});
+
+test('eslint: remove-useless-define-config: report', (t) => {
+    t.report('remove-useless-define-config', `Avoid useless defineConfig()`);
+    t.end();
+});
+
+test('eslint: remove-useless-define-config: transform', (t) => {
+    t.transform('remove-useless-define-config');
+    t.end();
+});
+
+test('eslint: remove-useless-define-config: transform: empty', (t) => {
+    t.transform('empty');
+    t.end();
+});

packages/plugin-eslint/test/eslint.js

@@ -146,3 +146,8 @@ test('plugin-eslint: transform: apply-define-config', (t) => {
     t.transform('apply-define-config');
     t.end();
 });
+
+test('plugin-eslint: transform: remove-useless-define-config', (t) => {
+    t.transform('remove-useless-define-config');
+    t.end();
+});

packages/plugin-eslint/test/fixture/remove-useless-define-config-fix.js

@@ -0,0 +1,3 @@
+import {safeAlign} from 'eslint-plugin-putout';
+
+export default safeAlign;

packages/plugin-eslint/test/fixture/remove-useless-define-config.js

@@ -0,0 +1 @@
+  export default defineConfig([safeAlign])

packages/plugin-eslint/test/fixture/remove-useless-match-to-flat-fix.js

@@ -1,4 +1,4 @@
 import {safeAlign} from 'eslint-plugin-putout';
 import {defineConfig} from 'eslint/config';
 
-export default defineConfig([safeAlign]);
+export default safeAlign;