eslint-plugin-raflint 1.5.2 → 1.5.4

package/README.md

@@ -20,9 +20,20 @@ qui devrait être écrites comme ça:
     const obj = {};
     if (condition) obj.prop = value;
 
+La règle couvre aussi le spread conditionnel dans un **tableau** ou un appel, malgré son nom —
+ne pas la restreindre au parent `ObjectExpression` :
+
+    const args = [target, ...(Array.isArray(command) ? command : [command])];
+
+qui devrait être écrit comme ça:
+
+    const argv = Array.isArray(command) ? command : [command];
+    const args = [target, ...argv];
+
 ## no-path-join
 
-Les utilisations de `path.join()` avec deux arguments qui peuvent être remplacées par des template strings:
+Les utilisations de `path.join()` avec au moins deux arguments, remplacées par des template strings
+(`fixable`, donc réécrites par `eslint --fix`) :
 
     const filePath = path.join(directory, filename);
 
@@ -30,7 +41,10 @@ qui devrait être écrites comme ça:
 
     const filePath = `${directory}/${filename}`;
 
-# tester le plugin sur un projet
+Le fix est laissé de côté quand il casserait l'appel : accès calculé (`path[join](a, b)`) et
+argument spread (`path.join(root, ...parts)`), qui n'est pas interpolable.
+
+## tester le plugin sur un projet
 
 on peut se mettre dans le dossier ou on veut utiliser le plugin pui lancer :
 

package/dist/rules/noConditionalObjectSpread.js

@@ -1,13 +1,21 @@
+const MESSAGE = 'Conditional object spread is not allowed.';
 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.',
-                });
-            }
+    meta: {
+        type: 'suggestion',
+        docs: {
+            description: 'Disallow conditional object spread (`...(cond ? a : b)` / `...(cond && a)`) in favor of explicit branching',
+            recommended: true,
         },
-    }),
+        schema: [],
+    },
+    create(context) {
+        return {
+            SpreadElement(node) {
+                if (node.argument.type === 'ConditionalExpression' || node.argument.type === 'LogicalExpression') {
+                    context.report({ node, message: MESSAGE });
+                }
+            },
+        };
+    },
 };
 export default rule;

package/dist/rules/noPathJoin.js

@@ -1,6 +1,11 @@
 // Le backslash doit etre echappe en premier, sinon il redouble ceux ajoutes ensuite.
 const escapeTemplateLiteral = (value) => value.replaceAll('\\', '\\\\').replaceAll('`', '\\`').replaceAll('${', '\\${');
 const MESSAGE = 'Utilisez des template strings au lieu de path.join() pour la concaténation de chemins simples';
+const isPathJoinCall = (node) => node.callee.type === 'MemberExpression' &&
+    !node.callee.computed &&
+    node.callee.object.name === 'path' &&
+    node.callee.property.name === 'join' &&
+    node.arguments.length >= 2;
 const rule = {
     meta: {
         type: 'suggestion',
@@ -14,10 +19,7 @@ const rule = {
     create(context) {
         return {
             CallExpression(node) {
-                if (!(node.callee.type === 'MemberExpression' &&
-                    node.callee.object.name === 'path' &&
-                    node.callee.property.name === 'join' &&
-                    node.arguments.length >= 2)) {
+                if (!isPathJoinCall(node)) {
                     return;
                 }
                 // Un spread ne peut pas être interpolé (`${...parts}` est une erreur de syntaxe) : on signale sans corriger.
@@ -25,12 +27,11 @@ const rule = {
                     context.report({ node, message: MESSAGE });
                     return;
                 }
-                const sourceCode = context.sourceCode ?? context.getSourceCode();
                 const parts = node.arguments.map((argument) => {
                     if (argument.type === 'Literal' && typeof argument.value === 'string') {
                         return escapeTemplateLiteral(argument.value);
                     }
-                    return `\${${sourceCode.getText(argument)}}`;
+                    return `\${${context.sourceCode.getText(argument)}}`;
                 });
                 const replacement = `\`${parts.join('/')}\``;
                 context.report({

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "eslint-plugin-raflint",
-    "version": "1.5.2",
+    "version": "1.5.4",
     "private": false,
     "description": "",
     "keywords": [],
@@ -26,30 +26,31 @@
     },
     "dependencies": {
         "@babel/eslint-parser": "^8.0.1",
-        "eslint-plugin-n": "^18.2.2",
+        "eslint-plugin-n": "^18.3.0",
         "eslint-plugin-sonarjs": "^4.2.0",
-        "eslint-plugin-unicorn": "^72.0.0"
+        "eslint-plugin-unicorn": "^74.0.0"
     },
     "devDependencies": {
         "@eslint/js": "^10.0.1",
-        "@stoplight/spectral-cli": "^6.16.2",
+        "@stoplight/spectral-cli": "^6.16.3",
         "c8": "^12.0.0",
         "eslint": "^10.5.0",
+        "eslint-config-prettier": "^10.1.8",
         "eslint-import-resolver-typescript": "^4.4.5",
         "eslint-plugin-import-x": "^4.17.1",
         "eslint-plugin-promise": "^7.3.0",
-        "eslint-plugin-raflint": "^1.5.1",
+        "eslint-plugin-raflint": "^1.5.2",
         "eslint-plugin-react-hooks": "^7.1.1",
-        "globals": "^17.8.0",
+        "globals": "^17.11.0",
         "husky": "^9.1.7",
-        "lint-staged": "^17.2.0",
+        "lint-staged": "^17.4.1",
         "markdownlint": "^0.41.1",
-        "mocha": "^11.7.6",
+        "mocha": "^11.8.0",
         "prettier": "^3.9.6",
         "ts-api-utils": "^2.5.0",
-        "tsx": "^4.23.1",
+        "tsx": "^4.23.13",
         "typescript": "^6.0.3",
-        "typescript-eslint": "^8.65.0"
+        "typescript-eslint": "^8.68.0"
     },
     "engines": {
         "node": ">=21.0.0"