eslint-plugin-raflint 1.4.1 → 1.5.0

package/dist/index.d.ts

@@ -0,0 +1,3 @@
+import type { ESLint } from 'eslint';
+declare const plugin: ESLint.Plugin;
+export default plugin;

package/{index.js → dist/index.js}

@@ -1,9 +1,9 @@
 import noConditionalObjectSpread from './rules/noConditionalObjectSpread.js';
 import noPathJoin from './rules/noPathJoin.js';
-
-export default {
+const plugin = {
     rules: {
         'no-conditional-object-spread': noConditionalObjectSpread,
         'no-path-join': noPathJoin,
     },
 };
+export default plugin;

package/dist/rules/noConditionalObjectSpread.d.ts

@@ -0,0 +1,3 @@
+import type { Rule } from 'eslint';
+declare const rule: Rule.RuleModule;
+export default rule;

package/dist/rules/noConditionalObjectSpread.js

@@ -0,0 +1,13 @@
+const rule = {
+    create: (context) => ({
+        SpreadElement(node) {
+            if (node.argument.type === 'ConditionalExpression' || node.argument.type === 'LogicalExpression') {
+                context.report({
+                    node,
+                    message: 'Conditional object spread is not allowed.',
+                });
+            }
+        },
+    }),
+};
+export default rule;

package/dist/rules/noPathJoin.d.ts

@@ -0,0 +1,3 @@
+import type { Rule } from 'eslint';
+declare const rule: Rule.RuleModule;
+export default rule;

package/dist/rules/noPathJoin.js

@@ -0,0 +1,34 @@
+const rule = {
+    meta: {
+        type: 'suggestion',
+        docs: {
+            description: 'Disallow path.join() in favor of template strings',
+            recommended: true,
+        },
+        fixable: 'code',
+        schema: [],
+    },
+    create(context) {
+        return {
+            CallExpression(node) {
+                if (!(node.callee.type === 'MemberExpression' && node.callee.object.name === 'path' && node.callee.property.name === 'join' && node.arguments.length >= 2)) {
+                    return;
+                }
+                const sourceCode = context.sourceCode ?? context.getSourceCode();
+                const parts = node.arguments.map((argument) => {
+                    if (argument.type === 'Literal' && typeof argument.value === 'string') {
+                        return argument.value;
+                    }
+                    return `\${${sourceCode.getText(argument)}}`;
+                });
+                const replacement = `\`${parts.join('/')}\``;
+                context.report({
+                    node,
+                    message: 'Utilisez des template strings au lieu de path.join() pour la concaténation de chemins simples',
+                    fix: (fixer) => fixer.replaceText(node, replacement),
+                });
+            },
+        };
+    },
+};
+export default rule;

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "eslint-plugin-raflint",
-    "version": "1.4.1",
+    "version": "1.5.0",
     "private": false,
     "description": "",
     "keywords": [],
@@ -11,38 +11,48 @@
     "license": "ISC",
     "author": "Raphael Piccolo",
     "type": "module",
-    "exports": "./index.js",
+    "main": "./dist/index.js",
+    "exports": "./dist/index.js",
+    "types": "./dist/index.d.ts",
+    "files": [
+        "dist"
+    ],
     "scripts": {
+        "build": "tsc",
         "cov": "c8 npm run test",
         "prepare": "husky",
-        "test": "SILENT=1 mocha --timeout 15000 {,lib/**/,test/**/}*.test.js"
+        "prepublishOnly": "npm run build",
+        "test": "SILENT=1 mocha --timeout 60000 {,lib/**/,test/**/}*.test.ts"
     },
     "dependencies": {
-        "@babel/eslint-parser": "^8.0.0",
-        "eslint-plugin-n": "^18.1.0",
-        "eslint-plugin-sonarjs": "^4.0.3",
-        "eslint-plugin-unicorn": "^67.0.0"
+        "@babel/eslint-parser": "^8.0.1",
+        "eslint-plugin-n": "^18.2.1",
+        "eslint-plugin-sonarjs": "^4.1.0",
+        "eslint-plugin-unicorn": "^69.0.0"
     },
     "devDependencies": {
         "@eslint/js": "^10.0.1",
         "@stoplight/spectral-cli": "^6.16.0",
         "c8": "^11.0.0",
         "eslint": "^10.5.0",
-        "eslint-plugin-import-x": "^4.16.2",
+        "eslint-plugin-import-x": "^4.17.1",
         "eslint-plugin-promise": "^7.3.0",
-        "eslint-plugin-raflint": "^1.4.0",
-        "globals": "^17.6.0",
+        "eslint-plugin-raflint": "^1.4.1",
+        "globals": "^17.7.0",
         "husky": "^9.1.7",
-        "lint-staged": "^17.0.7",
+        "lint-staged": "^17.0.8",
         "markdownlint": "^0.41.0",
         "mocha": "^11.7.6",
-        "prettier": "^3.8.4",
-        "ts-api-utils": "^2.5.0"
+        "prettier": "^3.9.1",
+        "ts-api-utils": "^2.5.0",
+        "tsx": "^4.22.4",
+        "typescript": "^6.0.3"
     },
     "engines": {
         "node": ">=21.0.0"
     },
     "allowScripts": {
-        "unrs-resolver": true
+        "unrs-resolver": true,
+        "esbuild": true
     }
 }

package/TESTING.md

@@ -1,18 +0,0 @@
-# Comment tester la règle no-path-join
-## Test manuel avec le fichier d'exemple
-
-Un fichier d'exemple `assets/testExample.js` a été créé pour démontrer le fonctionnement de la règle. Pour tester manuellement:
-
-1. Vérifier que la règle détecte les problèmes:
-
-```bash
-npx eslint -c eslinttest.config.js assets/testExample.js
-```
-
-Vous devriez voir des erreurs pour les lignes contenant `path.join(directory, filename)` et `path.join(process.cwd(), 'file.txt')`.
-
-2. Vérifier que la règle corrige automatiquement le code:
-
-```bash
-npx eslint -c eslinttest.config.js assets/testExample.js --fix
-```

package/assets/testExample.js

@@ -1,22 +0,0 @@
-// Exemple de code utilisant path.join() qui sera détecté par notre règle
-import path from 'node:path';
-
-// Définition des variables pour éviter les erreurs ESLint
-const directory = '/home/user';
-const filename = 'file.txt';
-const root = '/';
-
-// Ce code sera détecté et corrigé par notre règle
-const filePath1 = `${directory}/${filename}`;
-
-// Ce code sera détecté et corrigé par notre règle
-const filePath2 = `${process.cwd()}/file.txt`;
-
-// Ce code ne sera pas détecté car il a plus de 2 arguments
-const filePath3 = path.join(root, directory, filename);
-
-// Ce code est déjà au format recommandé
-const filePath4 = `${directory}/${filename}`;
-
-// Pour éviter les erreurs "unused variable"
-console.log(filePath1, filePath2, filePath3, filePath4);

package/eslint.config.js

@@ -1,286 +0,0 @@
-import sonarjs from 'eslint-plugin-sonarjs';
-import unicorn from 'eslint-plugin-unicorn';
-import promise from 'eslint-plugin-promise';
-import n from 'eslint-plugin-n';
-import raflint from 'eslint-plugin-raflint';
-import pluginimport from 'eslint-plugin-import-x';
-import babelParser from '@babel/eslint-parser';
-import js from '@eslint/js';
-import globals from 'globals';
-
-export default [
-    // eslint:recommended,
-    js.configs.recommended,
-
-    // unicorn
-    unicorn.configs['flat/recommended'],
-
-    // n
-    n.configs['flat/recommended'],
-
-    // sonarjs
-    sonarjs.configs.recommended,
-
-    // promise
-    promise.configs['flat/recommended'],
-
-    // plugin:import/recommended,
-    pluginimport.flatConfigs.recommended,
-
-    {
-        ignores: ['legacy/', 'dist/', 'node_modules/', 'web/', 'public/', 'coverage/', 'data/', 'vendor/', 'react/', 'js/', 'assets/', '**/old/'],
-    },
-    {
-        languageOptions: {
-            globals: {
-                ...globals.browser,
-                ...globals.node,
-                ...globals.mocha,
-            },
-            parser: babelParser,
-            parserOptions: {
-                ecmaVersion: 'latest',
-                sourceType: 'module',
-                requireConfigFile: false,
-            },
-        },
-        plugins: {
-            raflint,
-        },
-        rules: {
-            'sonarjs/no-empty-function': 0,
-            'sonarjs/no-unused-expressions': 0,
-            'sonarjs/no-unsafe-unzip': 0,
-            'sonarjs/os-command': 0,
-            'sonarjs/cors': 0,
-            'sonarjs/publicly-writable-directories': 0,
-            'sonarjs/file-permissions': 0,
-            'sonarjs/no-empty-test-file': 0,
-            'import-x/no-named-as-default-member': 0,
-            'import-x/no-unresolved': [
-                'error',
-                {
-                    ignore: [
-                        '^yargs/helpers$',
-                        '^p-queue$',
-                        '^@vitejs/plugin-react$',
-                        '^public-ip$',
-                        '^@modelcontextprotocol/sdk/',
-                        '^@maizzle/framework$',
-                        '^file-type$',
-                        '^vite$',
-                        '^windows-1252$',
-                        '^csv-parse/sync$'
-                    ],
-                },
-            ],
-            'sonarjs/regex-complexity': 0,
-            'sonarjs/no-clear-text-protocols': 0,
-            'sonarjs/no-infinite-loop': 0,
-            'sonarjs/no-hardcoded-ip': 0,
-            'sonarjs/pluginRules-of-hooks': 0,
-            'sonarjs/sonar-no-fallthrough': 0,
-            'sonarjs/no-ignored-exceptions': 0,
-            'sonarjs/slow-regex': 0,
-            'sonarjs/concise-regex': 0,
-            'sonarjs/hashing': 0,
-            'sonarjs/no-dead-store': 0,
-            'sonarjs/no-unused-vars': 0,
-            'sonarjs/no-hardcoded-passwords': 0,
-            'sonarjs/no-hardcoded-secrets': 0,
-            'sonarjs/unnecessary-character-escapes': 0,
-            'sonarjs/pseudo-random': 0,
-            'sonarjs/no-os-command-from-path': 0,
-            'sonarjs/anchor-precedence': 0,
-            'sonarjs/no-nested-functions': 0,
-            'sonarjs/new-cap': 0,
-            'sonarjs/sql-queries': 0,
-            'sonarjs/content-length': 0,
-            'sonarjs/no-commented-code': 0,
-            'raflint/no-conditional-object-spread': ['error'],
-            'raflint/no-path-join': ['error'],
-            'unicorn/prefer-event-target': 0,
-            'unicorn/prefer-structured-clone': 0,
-            'unicorn/prefer-string-raw': 0,
-            'unicorn/no-anonymous-default-export': 0,
-            'unicorn/better-regex': 0,
-            'unicorn/catch-error-name': 0,
-            'unicorn/consistent-function-scoping': 0,
-            'unicorn/expiring-todo-comments': 0,
-            'unicorn/explicit-length-check': 0,
-            'unicorn/import-style': 0,
-            'unicorn/no-array-for-each': 0,
-            'unicorn/prefer-single-call': 0,
-            'unicorn/no-array-reduce': 0,
-            'unicorn/no-lonely-if': 0,
-            'unicorn/no-negated-condition': 0,
-            'unicorn/no-null': 0,
-            'unicorn/no-process-exit': 0,
-            'unicorn/no-this-assignment': 0,
-            'unicorn/no-unnecessary-await': 0,
-            'unicorn/no-unused-properties': 0,
-            'unicorn/prefer-code-point': 0,
-            'unicorn/prefer-default-parameters': 0,
-            'unicorn/prefer-export-from': 0,
-            'unicorn/prefer-native-coercion-functions': 0,
-            'unicorn/prefer-number-properties': 0,
-            'unicorn/prefer-set-has': 0,
-            'unicorn/prefer-set-size': 0,
-            'unicorn/prefer-spread': 0,
-            'unicorn/prefer-string-replace-all': 0,
-            'unicorn/prefer-string-slice': 0,
-            'unicorn/prefer-switch': 0,
-            'unicorn/prefer-ternary': 0,
-            'unicorn/prefer-top-level-await': 0,
-            'unicorn/prevent-abbreviations': 0,
-            'unicorn/consistent-compound-words': 0,
-            // ne pas retirer `undefined` passe en argument (souvent requis par la signature -> casse tsc)
-            'unicorn/no-useless-undefined': ['error', { checkArguments: false }],
-            'unicorn/relative-url-style': 0,
-            'unicorn/switch-case-braces': 0,
-            'unicorn/template-indent': 0,
-            'unicorn/text-encoding-identifier-case': 0,
-            'unicorn/prefer-regexp-test': 0,
-            'unicorn/prefer-optional-catch-binding': 0,
-            'unicorn/filename-case': 0,
-            'sonarjs/cognitive-complexity': 0,
-            'sonarjs/no-nested-template-literals': 0,
-            'sonarjs/no-collapsible-if': 0,
-            'sonarjs/no-duplicate-string': 0,
-            'sonarjs/prefer-immediate-return': 0,
-            'sonarjs/no-identical-functions': 0,
-            'sonarjs/prefer-object-literal': 0,
-            'sonarjs/prefer-single-boolean-return': 0,
-            'n/no-unpublished-import': 0,
-            'n/no-unsupported-features/es-syntax': 0,
-            'n/no-unsupported-features/node-builtins': 0,
-            'n/no-missing-import': 0,
-            'n/no-process-exit': 0,
-
-            // si on l'active ça pose pb dans check / automerge
-            'n/shebang': 0,
-
-            'one-var-declaration-per-line': ['error', 'always'],
-            'one-var': ['error', 'never'],
-            'no-unused-expressions': 'error',
-            'no-nested-ternary': 'error',
-            'no-use-before-define': [
-                'error',
-                {
-                    functions: false,
-                    classes: false,
-                    variables: true,
-                    allowNamedExports: false,
-                },
-            ],
-            'no-new-native-nonconstructor': 'error',
-            'no-empty-static-block': 'error',
-            'symbol-description': 'error',
-            'require-yield': 'error',
-            'prefer-spread': 'error',
-            'prefer-rest-params': 'error',
-            'prefer-object-spread': 'error',
-            'prefer-object-has-own': 'error',
-            'prefer-numeric-literals': 'error',
-            'prefer-exponentiation-operator': 'error',
-            'operator-assignment': 'error',
-            'no-useless-rename': 'error',
-            'no-useless-constructor': 'error',
-            'no-useless-computed-key': 'error',
-            'no-unused-labels': 'error',
-            'no-script-url': 'error',
-            'no-new-object': 'error',
-            'no-multi-assign': 'error',
-            'no-loop-func': 'error',
-            'no-inline-comments': 'error',
-            'no-implicit-coercion': 'error',
-            'no-extra-boolean-cast': 'error',
-            'no-confusing-arrow': 'error',
-            'no-case-declarations': 'error',
-            'no-bitwise': 'error',
-            'no-array-constructor': 'error',
-            'no-alert': 'error',
-            'logical-assignment-operators': 'error',
-            'grouped-accessor-pairs': 'error',
-            'func-name-matching': 'error',
-            'dot-notation': 'error',
-            'consistent-this': ['error', 'self'],
-            'accessor-pairs': 'error',
-            'no-unused-private-class-members': 'error',
-            'no-promise-executor-return': 'error',
-            'no-duplicate-imports': 'error',
-            'no-constant-binary-expression': 'error',
-            'no-undef-init': 'error',
-            'no-label-var': 'error',
-            'init-declarations': ['error', 'always'],
-            'wrap-iife': ['error', 'inside'],
-            yoda: 'error',
-            'vars-on-top': 'error',
-            radix: ['error', 'as-needed'],
-            'prefer-regex-literals': 'error',
-            'prefer-promise-reject-errors': 'error',
-            'promise/catch-or-return': ['error', { terminationMethod: ['catch', 'finally'] }],
-            'no-warning-comments': 'error',
-            'no-void': 'error',
-            'no-useless-return': 'error',
-            'no-useless-concat': 'error',
-            'no-useless-call': 'error',
-            'no-throw-literal': 'error',
-            'no-sequences': 'error',
-            'no-self-compare': 'error',
-            'no-return-assign': 'error',
-            'no-proto': 'error',
-            'no-octal-escape': 'error',
-            'no-nonoctal-decimal-escape': 'error',
-            'no-new-wrappers': 'error',
-            'no-new-func': 'error',
-            'no-new': 'error',
-            'no-multi-str': 'error',
-            'no-multi-spaces': 'error',
-            'no-lone-blocks': 'error',
-            'no-labels': 'error',
-            'no-iterator': 'error',
-            'no-empty-function': 'error',
-            'block-scoped-var': 'error',
-            'prefer-arrow-callback': 'error',
-            'no-implied-eval': 'error',
-            'no-implicit-globals': 'error',
-            'no-floating-decimal': 'error',
-            'no-extra-label': 'error',
-            'no-extra-bind': 'error',
-            'no-extend-native': [
-                'error',
-                {
-                    exceptions: ['Object'],
-                },
-            ],
-            'no-eval': 'error',
-            'no-constant-condition': ['error', { checkLoops: false }],
-            'no-eq-null': 'error',
-            'no-else-return': 'error',
-            'no-unused-vars': 0,
-            'no-console': 0,
-            'no-empty': 0,
-            'no-redeclare': 0,
-            'no-useless-escape': 0,
-            'no-var': 'error',
-            'object-shorthand': ['error', 'properties'],
-            'prefer-template': 'error',
-            'no-div-regex': 'error',
-            'no-constructor-return': 'error',
-            'no-caller': 'error',
-            'max-classes-per-file': 'error',
-            'default-param-last': 'error',
-            'default-case-last': 'error',
-            'default-case': 'error',
-            'array-callback-return': 'error',
-            'no-unsafe-optional-chaining': 'error',
-            'no-unreachable-loop': 'error',
-            'no-template-curly-in-string': 'error',
-            'no-loss-of-precision': 'error',
-            'no-inner-declarations': 0,
-            'no-process-exit': 0,
-            'prefer-const': ['error', { destructuring: 'all' }],
-        },
-    },
-];

package/eslinttest.config.js

@@ -1,13 +0,0 @@
-import raflintPlugin from './index.js';
-
-export default [
-    {
-        files: ['**/*.js'],
-        plugins: {
-            raflint: raflintPlugin,
-        },
-        rules: {
-            'raflint/no-path-join': 'error',
-        },
-    },
-];

package/rules/noConditionalObjectSpread.js

@@ -1,14 +0,0 @@
-export default {
-    create(context) {
-        return {
-            SpreadElement(node) {
-                if (node.argument.type === 'ConditionalExpression' || node.argument.type === 'LogicalExpression') {
-                    context.report({
-                        node,
-                        message: 'Conditional object spread is not allowed.',
-                    });
-                }
-            },
-        };
-    },
-};

package/rules/noPathJoin.js

@@ -1,43 +0,0 @@
-export default {
-    meta: {
-        type: 'suggestion',
-        docs: {
-            description: 'Disallow path.join() in favor of template strings',
-            category: 'Stylistic Issues',
-            recommended: true,
-        },
-        fixable: 'code',
-        schema: [],
-    },
-    create(context) {
-        return {
-            CallExpression(node) {
-                if (
-                    node.callee.type === 'MemberExpression' &&
-                    node.callee.object.name === 'path' &&
-                    node.callee.property.name === 'join' &&
-                    node.arguments.length >= 2
-                ) {
-                    const sourceCode = context.sourceCode ?? context.getSourceCode();
-
-                    const parts = node.arguments.map((arg) => {
-                        if (arg.type === 'Literal' && typeof arg.value === 'string') {
-                            return arg.value;
-                        }
-                        return `\${${sourceCode.getText(arg)}}`;
-                    });
-
-                    const replacement = `\`${parts.join('/')}\``;
-
-                    context.report({
-                        node,
-                        message: 'Utilisez des template strings au lieu de path.join() pour la concaténation de chemins simples',
-                        fix(fixer) {
-                            return fixer.replaceText(node, replacement);
-                        },
-                    });
-                }
-            },
-        };
-    },
-};

package/test/noConditionalObjectSpread.test.js

@@ -1,46 +0,0 @@
-import { RuleTester } from 'eslint';
-import noConditionalObjectSpread from '../rules/noConditionalObjectSpread.js';
-import path from 'node:path';
-import { fileURLToPath } from 'node:url';
-const __dirname = path.dirname(fileURLToPath(import.meta.url));
-
-const ruleTester = new RuleTester({
-    languageOptions: {
-        parser: await import('@babel/eslint-parser'),
-        ecmaVersion: 'latest',
-        sourceType: 'module',
-        parserOptions: {
-            requireConfigFile: false,
-        },
-    },
-});
-
-ruleTester.run('no-conditional-object-spread', noConditionalObjectSpread, {
-    valid: [
-        {
-            code: `
-                const condition = true;
-                const obj = {};
-                if (condition) obj.prop = value;
-            `,
-        },
-    ],
-    invalid: [
-        {
-            code: `
-                const obj = {
-                    ...(condition ? { prop: 'value' } : {}),
-                };
-            `,
-            errors: [{ message: 'Conditional object spread is not allowed.' }],
-        },
-        {
-            code: `
-                const obj = {
-                    ...(condition && { prop: 'value' }),
-                };
-            `,
-            errors: [{ message: 'Conditional object spread is not allowed.' }],
-        },
-    ],
-});

package/test/noPathJoin.test.js

@@ -1,78 +0,0 @@
-import { RuleTester } from 'eslint';
-import noPathJoin from '../rules/noPathJoin.js';
-import path from 'node:path';
-import { fileURLToPath } from 'node:url';
-const __dirname = path.dirname(fileURLToPath(import.meta.url));
-
-const ruleTester = new RuleTester({
-    languageOptions: {
-        parser: await import('@babel/eslint-parser'),
-        ecmaVersion: 'latest',
-        sourceType: 'module',
-        parserOptions: {
-            requireConfigFile: false,
-        },
-    },
-});
-
-ruleTester.run('no-path-join', noPathJoin, {
-    valid: [
-        {
-            code: `
-                const filePath = \`\${directory}/\${filename}\`;
-            `,
-        },
-        {
-            // Cas avec un seul argument (non concerné)
-            code: `
-                const filePath = path.join(directory);
-            `,
-        },
-        {
-            // Cas avec un autre objet que path
-            code: `
-                const result = utils.join(a, b);
-            `,
-        },
-    ],
-    invalid: [
-        {
-            code: `
-                const filePath = path.join(directory, filename);
-            `,
-            errors: [{ message: 'Utilisez des template strings au lieu de path.join() pour la concaténation de chemins simples' }],
-            output: `
-                const filePath = \`\${directory}/\${filename}\`;
-            `,
-        },
-        {
-            code: `
-                const filePath = path.join(process.cwd(), 'file.txt');
-            `,
-            errors: [{ message: 'Utilisez des template strings au lieu de path.join() pour la concaténation de chemins simples' }],
-            output: `
-                const filePath = \`\${process.cwd()}/file.txt\`;
-            `,
-        },
-        {
-            // 3 arguments
-            code: `
-                const filePath = path.join(root, directory, filename);
-            `,
-            errors: [{ message: 'Utilisez des template strings au lieu de path.join() pour la concaténation de chemins simples' }],
-            output: `
-                const filePath = \`\${root}/\${directory}/\${filename}\`;
-            `,
-        },
-        {
-            // Mix variables et littéraux
-            code: `
-                const filePath = path.join(root, 'uploads', filename);
-            `,
-            errors: [{ message: 'Utilisez des template strings au lieu de path.join() pour la concaténation de chemins simples' }],
-            output: `
-                const filePath = \`\${root}/uploads/\${filename}\`;
-            `,
-        },
-    ],
-});

package/todo.txt

@@ -1,96 +0,0 @@
-REVIEW DE CODE — 2026-04-26
-============================================
-
-Plugin ESLint custom maison. Expose 2 regles :
-- `no-conditional-object-spread` : interdit `{...(cond ? {x:1} : {})}`
-  et `{...(cond && {x:1})}`
-- `no-path-join` : interdit `path.join(a, b)` au profit de
-  `\`${a}/${b}\``  (avec autofix).
-
-13+43 lignes de code metier dans rules/, 124 lignes de tests, 53
-lignes (dont 42 commentees) dans index.js.
-
-BUGS / FRAGILITE
-----------------
-[ ] index.js : 42 lignes commentees (CRITIQUE de lisibilite)
-    index.js:11-53 : tout un bloc commente avec une version
-    "améliorée" du plugin (avec `meta.name`, `meta.version`,
-    `configs.flat/recommended`). C'est probablement la version
-    "qui marche" sur ESLint 9 flat config, mais elle est commentee.
-    L'export actuel (lignes 1-9) est minimal et ne suit pas la
-    convention ESLint `meta` (manque name/version). Si ESLint
-    detecte le plugin sans meta, certains checks (cache, dedup)
-    sont degrades.
-    A clarifier : soit reactiver le bloc commente (et garder une
-    seule version), soit delete le bloc inutile.
-
-[ ] noPathJoin : autofix n'introduit pas de slash safe
-    rules/noPathJoin.js:25-27 : `parts.join('/')`. Si l'argument
-    est `path.join(__dirname, '..', 'file.js')`, le fix produit
-    `\`${__dirname}/../file.js\``. OK.
-    Mais si l'argument contient un slash final/début, doublon :
-    `path.join('a/', 'b')` => `\`a//b\`` (autofix incorrect).
-    Path.join normalise les slashs, le fix non. Edge case rare
-    mais reel.
-
-[ ] noPathJoin : detection MemberExpression chain
-    rules/noPathJoin.js:13 : `node.callee.object.name === 'path'`.
-    Si import `import { join } from 'node:path'` et utilise
-    `join(a, b)`, pas detecte (pas de `path.join`). Fragmentation
-    incomplete.
-
-[ ] noPathJoin : detection seulement "path.join(>=2 args)"
-    rules/noPathJoin.js:14 : `node.arguments.length >= 2`. Si
-    `path.join(a)` (1 seul arg), pas declenche. Mais `path.join
-    (a)` est en pratique inutile (path.join('foo') === 'foo'),
-    une regle pourrait suggerer de retirer l'appel.
-
-[ ] noPathJoin : pas de skip pour les cas corrects
-    Si l'utilisateur a deliberement choisi `path.join` pour la
-    portabilité Windows (`\\` vs `/`), le fix le casse.
-    Convention de la stack : tout est sur Linux donc OK, mais
-    a documenter (cf README).
-
-[ ] noConditionalObjectSpread : detection trop large
-    rules/noConditionalObjectSpread.js:5 : `node.argument.type
-    === 'ConditionalExpression' || 'LogicalExpression'`.
-    Detecte aussi :
-        const obj = { ...(a || b) };
-    qui est un usage VALIDE (a est preferable, sinon b). Faux
-    positif. A affiner : check si une des branches est `{}` ou
-    null/undefined.
-
-[ ] noConditionalObjectSpread : pas d'autofix
-    Pas `fixable: 'code'`. Le user doit refactor a la main. Bon
-    point puisque le refactor change la semantique (variable de
-    sortie modifiee), mais le message d'erreur ne propose pas
-    d'alternative claire.
-
-[ ] noConditionalObjectSpread : pas de meta
-    rules/noConditionalObjectSpread.js : pas de `meta` block.
-    Manque `meta.docs.description`, `meta.type`, etc. ESLint
-    plant pas, mais best practice.
-
-CODE MORT / POLLUTION
----------------------
-[ ] assets/testExample.js
-    Fichier de test manuel. OK pour la documentation.
-
-[ ] TESTING.md
-    Documentation manuelle separee. A consolider dans README.md.
-
-CONVENTIONS
------------
-[ ] Mauvaise pratique : meta block manquant sur noConditionalObjectSpread
-    Cf bugs.
-
-[ ] Pas de versioning des rules
-    1.4.0 mais pas de breaking change tracking. Si on rend la
-    detection plus stricte, casse les CI des consommateurs.
-
-DECISION SUGGEREE
------------------
-[ ] Trancher index.js : la version commentee semble plus complete.
-    Migrer vers la version commentee (avec meta + configs/flat/
-    recommended), tester sur 2-3 projets consommateurs, release
-    1.5.0.