@dartess/eslint-plugin 0.9.0 → 0.10.0

package/CHANGELOG.md

@@ -2,6 +2,13 @@
 
 [//]: # (https://keepachangelog.com/en/1.1.0/)
 
+## [0.10.0] - 2026-02-23
+
+- add rule `@dartess/mobx-no-action-bound`
+- add `@dartess/mobx-no-action-bound` to `mobx` config
+- change `no-sequences` option `allowInParentheses` from `true` to `false`
+- allow `mobx-require-observer` without settings because `mobx` can be used without hooks
+
 ## [0.9.0] - 2026-02-03
 
 - add `eslint-plugin-complete` to `recommended` config

package/README.md

@@ -70,6 +70,7 @@ npm i -D eslint-plugin-storybook
 Edit or create `eslint.config.ts` (or `eslint.config.mts`). You probably have to install `jiti` for it.
 
 ```ts
+import type { TSESLint } from '@typescript-eslint/utils';
 import dartessEslintPluginRecommended from '@dartess/eslint-plugin/recommended';
 import dartessEslintPluginRecommendedPostFormat from '@dartess/eslint-plugin/recommended-post-format';
 
@@ -110,7 +111,7 @@ export default [
   // @see `Fine Tuning` -> `Formatters` section below 
   
   ...dartessEslintPluginRecommendedPostFormat,
-]
+] satisfies TSESLint.FlatConfig.ConfigArray;
 
 ```
 
@@ -154,6 +155,8 @@ This plugin requires manual setup for you build tools.
 
 If you're using Mobx with legacy decorators, you have to enable rule `mobx/missing-make-observable` manually.
 
+if you're using Mobx stores with your custom hooks, set `settings.mobx.storeHooks` [by docs](docs/rules/mobx-require-observer.md). 
+
 ## Supported Rules
 
 Each rule has emojis denoting:
@@ -179,6 +182,7 @@ Each rule has emojis denoting:
 | [mobx-require-observer](docs/rules/mobx-require-observer.md)                                                 | Components using the stores must be wrapped in an `observer`                  | ✅ | 🔧 |    |
 | [mobx-sync-autorun](docs/rules/mobx-sync-autorun.md)                                                         | Enforce synchronous autorun callback                                          | ✅ |    |    |
 | [mobx-sync-action](docs/rules/mobx-sync-action.md)                                                           | Enforce synchronous actions                                                   | ✅ |    |    |
+| [mobx-no-action-bound](docs/rules/mobx-no-action-bound.md)                                                   | Enforce using arrow functions for binging `this` to actions                   | ✅ |    |    |
 
 ## Code Reuse Policy
 

package/dist/configs/mobx.js

@@ -9,6 +9,7 @@ const config = [
             'mobx/missing-make-observable': 'off', // useless with modern decorators syntax. TODO check original plugin?
             '@dartess/mobx-strict-observable-components-declaration': 'error',
             '@dartess/mobx-require-observer': 'error',
+            '@dartess/mobx-no-action-bound': 'error',
             '@dartess/mobx-sync-autorun': 'error', // TODO implement it by types?
             '@dartess/mobx-sync-action': 'error', // TODO implement it by types?
         },

package/dist/configs/recommended.js

@@ -110,6 +110,8 @@ const config = [
                     methods: 'above',
                 },
             ],
+            // totally disable comma operator
+            'no-sequences': ['error', { allowInParentheses: false }],
         },
     },
     {

package/dist/configs/vendor-rules/best-practices.d.ts

@@ -83,7 +83,6 @@ declare const rules: {
     'no-return-assign': ["error", string];
     'no-script-url': "error";
     'no-self-compare': "error";
-    'no-sequences': "error";
     'no-useless-concat': "error";
     'no-void': "error";
     'prefer-promise-reject-errors': ["error", {

package/dist/configs/vendor-rules/best-practices.js

@@ -165,9 +165,6 @@ const rules = {
     // disallow comparisons where both sides are exactly the same
     // https://eslint.org/docs/rules/no-self-compare
     'no-self-compare': 'error',
-    // disallow use of comma operator
-    // https://eslint.org/docs/rules/no-sequences
-    'no-sequences': 'error',
     // disallow useless string concatenation
     // https://eslint.org/docs/rules/no-useless-concat
     'no-useless-concat': 'error',

package/dist/index.js

@@ -5,6 +5,7 @@ import ruleStoriesExportMeta from "./rules/stories-export-meta.js";
 import ruleStoriesExportTyped from "./rules/stories-export-typed.js";
 import ruleMobxStrictObservableComponentsDeclaration from "./rules/mobx-strict-observable-components-declaration.js";
 import ruleMobxRequireObserver from "./rules/mobx-require-observer.js";
+import ruleMobxNoActionBound from "./rules/mobx-no-action-bound.js";
 import ruleImportsMaxParentDepth from "./rules/imports-max-parent-depth.js";
 import ruleTsNamedTupleElements from "./rules/ts-named-tuple-elements.js";
 import ruleMobxSyncAutorun from "./rules/mobx-sync-autorun.js";
@@ -23,6 +24,7 @@ const plugin = {
         'mobx-sync-action': ruleMobxSyncAction,
         'mobx-sync-autorun': ruleMobxSyncAutorun,
         'mobx-require-observer': ruleMobxRequireObserver,
+        'mobx-no-action-bound': ruleMobxNoActionBound,
         'stories-export-typed': ruleStoriesExportTyped,
         'stories-export-meta': ruleStoriesExportMeta,
         'ts-named-tuple-elements': ruleTsNamedTupleElements,

package/dist/rules/mobx-no-action-bound.d.ts

@@ -0,0 +1,3 @@
+import { ESLintUtils } from '@typescript-eslint/utils';
+declare const _default: ESLintUtils.RuleModule<"noActionBound", [], unknown, ESLintUtils.RuleListener>;
+export default _default;

package/dist/rules/mobx-no-action-bound.js

@@ -0,0 +1,25 @@
+import { ESLintUtils } from '@typescript-eslint/utils';
+export default ESLintUtils.RuleCreator(() => '')({
+    name: 'mobx-no-action-bound',
+    defaultOptions: [],
+    meta: {
+        type: 'problem',
+        docs: {
+            description: 'Enforce using arrow functions for binging `this` to actions',
+        },
+        messages: {
+            noActionBound: 'rewrite action with arrow function for binding `this`',
+        },
+        schema: [],
+    },
+    create(context) {
+        return {
+            "Decorator[expression.object.name='action'][expression.property.name='bound']": function (node) {
+                context.report({
+                    node,
+                    messageId: 'noActionBound',
+                });
+            },
+        };
+    },
+});

package/dist/rules/mobx-require-observer.js

@@ -54,7 +54,7 @@ export default ESLintUtils.RuleCreator(() => '')({
         const mobxSettings = context.settings.mobx;
         const hooks = Array.isArray(mobxSettings?.storeHooks) ? mobxSettings.storeHooks : [];
         if (hooks.length === 0) {
-            throw new Error('Please fill settings.mobx.storeHooks');
+            return {};
         }
         const { sourceCode } = context;
         const program = sourceCode.ast;

package/docs/rules/mobx-no-action-bound.md

@@ -0,0 +1,40 @@
+# Enforce using arrow functions for binging `this` to actions (mobx-no-action-bound)
+
+Decorator `@action.bound` automatically bind methods to class instances to avoid context loss (`this`).
+However, this can also be achieved by declaring methods as an properies with arrow functions.
+
+This is the first problem with bound methods—their functionality is duplicated by native abilities.
+
+In general, for optimization purposes, it's not recommended to bind _everything_; by default,
+class methods should be on the prototype.
+
+To resolve the dilemma of "how to avoid binding unnecessary things and avoid losing context"
+the [`@typescript-eslint/unbound-method`](https://typescript-eslint.io/rules/unbound-method/) rule exists.
+
+However, it can't detect whether a method is bound via an `@action.bound`. Therefore, it's best to avoid
+using `@action.bound` at all and leave only one binding method—arrow functions with `@typescript-eslint/unbound-method`.
+In this case you don't need think about it.
+
+## Rule Details
+
+Examples of **incorrect** code for this rule:
+
+```ts
+class Store {
+  @action.bound
+  reset() {
+    this.value = null;
+  };
+}
+```
+
+Examples of **correct** code for this rule:
+
+```ts
+class Store {
+  @action
+  reset = () => {
+    this.value = null;
+  };
+}
+```

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@dartess/eslint-plugin",
   "type": "module",
-  "version": "0.9.0",
+  "version": "0.10.0",
   "description": "A set of rules and configs for various TypeScript projects",
   "keywords": [
     "eslint",