eslint-plugin-mobx 0.0.7 → 0.0.8

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # eslint-plugin-mobx
 
+## 0.0.8
+
+### Patch Changes
+
+-   [`aafda613`](https://github.com/mobxjs/mobx/commit/aafda6136afd107c6cbdd73d9bab57e45a2eb9f5) [#3256](https://github.com/mobxjs/mobx/pull/3256) Thanks [@urugator](https://github.com/urugator)! - fix `no-anonymous-observer` autofix for arrow functions without BlockStatement body
+
 ## 0.0.7
 
 ### Patch Changes

package/dist/index.js

@@ -433,21 +433,27 @@ function create(context) {
       var fix = function fix(fixer) {
         var _observer$parent;
 
-        // Use name from variable for autofix 
-        var name = ((_observer$parent = observer.parent) === null || _observer$parent === void 0 ? void 0 : _observer$parent.type) === 'VariableDeclarator' ? observer.parent.id.name : undefined;
+        // Use name from variable for autofix
+        var name = ((_observer$parent = observer.parent) === null || _observer$parent === void 0 ? void 0 : _observer$parent.type) === "VariableDeclarator" ? observer.parent.id.name : undefined;
         if (!name) return;
 
-        if (cmp.type === 'ArrowFunctionExpression') {
+        if (cmp.type === "ArrowFunctionExpression") {
           var arrowToken = sourceCode.getTokenBefore(cmp.body);
-          return [fixer.replaceText(arrowToken, ''), fixer.insertTextBefore(cmp, "function ".concat(name))];
+          var fixes = [fixer.replaceText(arrowToken, ""), fixer.insertTextBefore(cmp, "function ".concat(name))];
+
+          if (cmp.body.type !== "BlockStatement") {
+            fixes.push(fixer.insertTextBefore(cmp.body, "{ return "), fixer.insertTextAfter(cmp.body, " }"));
+          }
+
+          return fixes;
         }
 
-        if (cmp.type === 'FunctionExpression') {
+        if (cmp.type === "FunctionExpression") {
           var functionToken = sourceCode.getFirstToken(cmp);
           return fixer.replaceText(functionToken, "function ".concat(name));
         }
 
-        if (cmp.type === 'ClassExpression') {
+        if (cmp.type === "ClassExpression") {
           var classToken = sourceCode.getFirstToken(cmp);
           return fixer.replaceText(classToken, "class ".concat(name));
         }
@@ -455,7 +461,7 @@ function create(context) {
 
       context.report({
         node: cmp,
-        messageId: 'observerComponentMustHaveName',
+        messageId: "observerComponentMustHaveName",
         fix: fix
       });
     }
@@ -464,10 +470,10 @@ function create(context) {
 
 var noAnonymousObserver$1 = {
   meta: {
-    type: 'problem',
-    fixable: 'code',
+    type: "problem",
+    fixable: "code",
     docs: {
-      description: 'forbids anonymous functions or classes as `observer` components',
+      description: "forbids anonymous functions or classes as `observer` components",
       recommended: true
     },
     messages: {

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "eslint-plugin-mobx",
-    "version": "0.0.7",
+    "version": "0.0.8",
     "description": "ESLint rules for MobX",
     "main": "dist/index.js",
     "repository": {

package/src/no-anonymous-observer.js

@@ -1,57 +1,65 @@
-'use strict';
+"use strict"
 
 function create(context) {
-  const sourceCode = context.getSourceCode();
+    const sourceCode = context.getSourceCode()
 
-  return {
-    'CallExpression[callee.name="observer"]': observer => {
-      const cmp = observer.arguments[0];
-      if (!cmp) return;
-      if (cmp?.id?.name) return;
+    return {
+        'CallExpression[callee.name="observer"]': observer => {
+            const cmp = observer.arguments[0]
+            if (!cmp) return
+            if (cmp?.id?.name) return
 
-      const fix = fixer => {
-        // Use name from variable for autofix 
-        const name = observer.parent?.type === 'VariableDeclarator'
-          ? observer.parent.id.name
-          : undefined;
+            const fix = fixer => {
+                // Use name from variable for autofix
+                const name =
+                    observer.parent?.type === "VariableDeclarator"
+                        ? observer.parent.id.name
+                        : undefined
 
-        if (!name) return;
-        if (cmp.type === 'ArrowFunctionExpression') {
-          const arrowToken = sourceCode.getTokenBefore(cmp.body);
-          return [
-            fixer.replaceText(arrowToken, ''),
-            fixer.insertTextBefore(cmp, `function ${name}`),
-          ]
+                if (!name) return
+                if (cmp.type === "ArrowFunctionExpression") {
+                    const arrowToken = sourceCode.getTokenBefore(cmp.body)
+                    const fixes = [
+                        fixer.replaceText(arrowToken, ""),
+                        fixer.insertTextBefore(cmp, `function ${name}`)
+                    ]
+                    if (cmp.body.type !== "BlockStatement") {
+                        fixes.push(
+                            fixer.insertTextBefore(cmp.body, `{ return `),
+                            fixer.insertTextAfter(cmp.body, ` }`)
+                        )
+                    }
+                    return fixes
+                }
+                if (cmp.type === "FunctionExpression") {
+                    const functionToken = sourceCode.getFirstToken(cmp)
+                    return fixer.replaceText(functionToken, `function ${name}`)
+                }
+                if (cmp.type === "ClassExpression") {
+                    const classToken = sourceCode.getFirstToken(cmp)
+                    return fixer.replaceText(classToken, `class ${name}`)
+                }
+            }
+            context.report({
+                node: cmp,
+                messageId: "observerComponentMustHaveName",
+                fix
+            })
         }
-        if (cmp.type === 'FunctionExpression') {
-          const functionToken = sourceCode.getFirstToken(cmp);
-          return fixer.replaceText(functionToken, `function ${name}`);
-        }
-        if (cmp.type === 'ClassExpression') {
-          const classToken = sourceCode.getFirstToken(cmp);
-          return fixer.replaceText(classToken, `class ${name}`);
-        }
-      }
-      context.report({
-        node: cmp,
-        messageId: 'observerComponentMustHaveName',
-        fix,
-      })
-    },
-  };
+    }
 }
 
 module.exports = {
-  meta: {
-    type: 'problem',
-    fixable: 'code',
-    docs: {
-      description: 'forbids anonymous functions or classes as `observer` components',
-      recommended: true,
-    },
-    messages: {
-      observerComponentMustHaveName: "`observer` component must have a name.",
+    meta: {
+        type: "problem",
+        fixable: "code",
+        docs: {
+            description: "forbids anonymous functions or classes as `observer` components",
+            recommended: true
+        },
+        messages: {
+            observerComponentMustHaveName: "`observer` component must have a name."
+        }
     },
-  },
-  create,
-};
+    create
+}