eslint-plugin-crisp 1.7.7 → 1.7.8

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-crisp",
-  "version": "1.7.7",
+  "version": "1.7.8",
   "description": "Custom ESLint Rules for Crisp",
   "author": "Crisp IM SAS",
   "main": "index.js",

package/rules/logger-arguments.js

@@ -94,6 +94,31 @@ function isReference(node) {
   );
 }
 
+// Checks an inline object only contains references (no computed values)
+function isReferenceObject(node) {
+  node = unwrapTransparentExpression(node);
+
+  if (!node || node.type !== "ObjectExpression") {
+    return false;
+  }
+
+  return node.properties.every((property) => {
+    if (property.type === "SpreadElement") {
+      return isReference(property.argument);
+    }
+
+    if (property.type !== "Property" || property.kind !== "init") {
+      return false;
+    }
+
+    if (property.computed && !isReference(property.key)) {
+      return false;
+    }
+
+    return isReference(property.value) || isReferenceObject(property.value);
+  });
+}
+
 export default {
   meta: {
     type: "suggestion",
@@ -107,7 +132,7 @@ export default {
       message:
         "Logger messages must be single-line, double-quoted string literals.",
       reference:
-        "Logger interpolation arguments must be references, not computed expressions."
+        "Logger interpolation arguments must be references or inline objects of references, not computed expressions."
     }
   },
 
@@ -136,7 +161,7 @@ export default {
         }
 
         for (const value of values) {
-          if (!isReference(value)) {
+          if (!isReference(value) && !isReferenceObject(value)) {
             context.report({
               node: value,
               messageId: "reference"

package/tests/rules/logger-arguments.test.js

@@ -15,6 +15,9 @@ ruleTester.run("logger-arguments", rule, {
   value
 );`,
     "App.logger.info(\"This intentionally long logger message may exceed the usual source line length when necessary.\");",
+    "App.logger.warn(\"Shutting down\", { bufferedCount });",
+    "App.logger.warn(\"Shutting down\", { count: this.count, ...meta });",
+    "App.logger.warn(\"Shutting down\", { nested: { bufferedCount } }, logMeta);",
     "App.logger.child({ namespace: \"worker\" });",
     "console.info(`Unrelated logger calls are ignored: ${value}`);"
   ],
@@ -93,7 +96,15 @@ ruleTester.run("logger-arguments", rule, {
       ]
     },
     {
-      code: "App.logger.debug(\"Loaded value: %s\", { value });",
+      code: "App.logger.debug(\"Loaded value: %s\", { value: value.trim() });",
+      errors: [
+        {
+          messageId: "reference"
+        }
+      ]
+    },
+    {
+      code: "App.logger.debug(\"Loaded value: %s\", { value: 3 });",
       errors: [
         {
           messageId: "reference"