eslint-plugin-raflint 1.4.0 → 1.4.1

package/eslint.config.js

@@ -3,7 +3,7 @@ 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';
+import pluginimport from 'eslint-plugin-import-x';
 import babelParser from '@babel/eslint-parser';
 import js from '@eslint/js';
 import globals from 'globals';
@@ -28,7 +28,7 @@ export default [
     pluginimport.flatConfigs.recommended,
 
     {
-        ignores: ['node_modules/', 'web/', 'public/', 'coverage/', 'data/', 'vendor/', 'cypress/', 'react/', 'js/', 'assets/', '**/old/'],
+        ignores: ['legacy/', 'dist/', 'node_modules/', 'web/', 'public/', 'coverage/', 'data/', 'vendor/', 'react/', 'js/', 'assets/', '**/old/'],
     },
     {
         languageOptions: {
@@ -56,8 +56,8 @@ export default [
             'sonarjs/publicly-writable-directories': 0,
             'sonarjs/file-permissions': 0,
             'sonarjs/no-empty-test-file': 0,
-            'import/no-named-as-default-member': 0,
-            'import/no-unresolved': [
+            'import-x/no-named-as-default-member': 0,
+            'import-x/no-unresolved': [
                 'error',
                 {
                     ignore: [
@@ -66,7 +66,11 @@ export default [
                         '^@vitejs/plugin-react$',
                         '^public-ip$',
                         '^@modelcontextprotocol/sdk/',
-                        '^@maizzle/framework$'
+                        '^@maizzle/framework$',
+                        '^file-type$',
+                        '^vite$',
+                        '^windows-1252$',
+                        '^csv-parse/sync$'
                     ],
                 },
             ],
@@ -129,18 +133,16 @@ export default [
             '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': [
-                'error',
-                {
-                    case: 'camelCase',
-                },
-            ],
+            'unicorn/filename-case': 0,
             'sonarjs/cognitive-complexity': 0,
             'sonarjs/no-nested-template-literals': 0,
             'sonarjs/no-collapsible-if': 0,
@@ -151,6 +153,7 @@ export default [
             '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,
 
@@ -216,6 +219,7 @@ export default [
             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',

package/index.js

@@ -7,47 +7,3 @@ export default {
         'no-path-join': noPathJoin,
     },
 };
-
-/*
-import path from 'node:path';
-import { fileURLToPath } from 'node:url';
-const __dirname = path.dirname(fileURLToPath(import.meta.url));
-import fs from 'node:fs';
-const { name, version } = JSON.parse(await fs.promises.readFile(`${__dirname}/package.json`));
-
-import noConditionalObjectSpread from './rules/noConditionalObjectSpread.js';
-import noPathJoin from './rules/noPathJoin.js';
-
-const rules = {
-    'no-conditional-object-spread': noConditionalObjectSpread,
-    'no-path-join': noPathJoin,
-};
-
-export default {
-    meta: {
-        name,
-        version,
-    },
-    rules,
-    configs: {
-        'flat/recommended': {
-            plugins: {
-                raflint: {
-                    meta: {
-                        name,
-                        version,
-                    },
-                    rules: {
-                        'raflint/no-conditional-object-spread': "error",
-                        'raflint/no-path-join': "error"
-                    },
-                }
-            },
-            rules: {
-                'raflint/no-conditional-object-spread': "error",
-                'raflint/no-path-join': "error"
-            },
-        },
-    }
-};
-*/

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "eslint-plugin-raflint",
-    "version": "1.4.0",
+    "version": "1.4.1",
     "private": false,
     "description": "",
     "keywords": [],
@@ -14,35 +14,35 @@
     "exports": "./index.js",
     "scripts": {
         "cov": "c8 npm run test",
-        "jest": "SILENT=1 node --no-warnings --experimental-vm-modules ./node_modules/.bin/jest",
         "prepare": "husky",
-        "test": "SILENT=1 mocha --timeout 15000 {lib,test}/**/*.test.js"
+        "test": "SILENT=1 mocha --timeout 15000 {,lib/**/,test/**/}*.test.js"
     },
     "dependencies": {
-        "@babel/eslint-parser": "^7.28.6",
-        "eslint-plugin-n": "^17.24.0",
-        "eslint-plugin-sonarjs": "^4.0.2",
-        "eslint-plugin-unicorn": "^64.0.0"
+        "@babel/eslint-parser": "^8.0.0",
+        "eslint-plugin-n": "^18.1.0",
+        "eslint-plugin-sonarjs": "^4.0.3",
+        "eslint-plugin-unicorn": "^67.0.0"
     },
     "devDependencies": {
-        "@eslint/js": "^9.39.4",
-        "@stoplight/spectral-cli": "^6.15.0",
+        "@eslint/js": "^10.0.1",
+        "@stoplight/spectral-cli": "^6.16.0",
         "c8": "^11.0.0",
-        "eslint": "^9.39.4",
-        "eslint-plugin-import": "^2.32.0",
-        "eslint-plugin-promise": "^7.2.1",
-        "eslint-plugin-raflint": "^1.3.4",
-        "globals": "^17.4.0",
-        "html-validate": "^10.11.2",
+        "eslint": "^10.5.0",
+        "eslint-plugin-import-x": "^4.16.2",
+        "eslint-plugin-promise": "^7.3.0",
+        "eslint-plugin-raflint": "^1.4.0",
+        "globals": "^17.6.0",
         "husky": "^9.1.7",
-        "jest": "^30.3.0",
-        "lint-staged": "^16.4.0",
-        "markdownlint": "^0.40.0",
-        "mocha": "^11.7.5",
-        "prettier": "^3.8.1",
+        "lint-staged": "^17.0.7",
+        "markdownlint": "^0.41.0",
+        "mocha": "^11.7.6",
+        "prettier": "^3.8.4",
         "ts-api-utils": "^2.5.0"
     },
     "engines": {
         "node": ">=21.0.0"
+    },
+    "allowScripts": {
+        "unrs-resolver": true
     }
 }

package/todo.txt

@@ -0,0 +1,96 @@
+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.

package/jest.config.js

@@ -1,4 +0,0 @@
-export default {
-    transform: {},
-    testTimeout: 60_000,
-};