@dartess/eslint-plugin 0.0.8 → 0.0.9

package/CHANGELOG.md

@@ -2,6 +2,13 @@
 
 [//]: # (https://keepachangelog.com/en/1.1.0/)
 
+## [0.0.9] - 2026-01-02
+
+- remove overriding `@typescript-eslint/return-await` rule
+- cleanup disabled rules
+- add `prefer-object-has-own` to `recomended`: previously was disabled because of old browsers support
+- apply `no-throw-literal` only for `js`-files (for `ts` files already enabled `@typescript-eslint/only-throw-error`)
+
 ## [0.0.8] - 2025-12-28
 
 - update `README`

package/dist/configs/recommended.js

@@ -49,7 +49,6 @@ const config = [
         plugins: {
             unicorn: eslintPluginUnicorn,
             '@dartess': dartessPlugin,
-            // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment -- https://github.com/NullVoxPopuli/eslint-plugin-decorator-position/issues/778
             'decorator-position': eslintPluginDecoratorPosition,
         },
         languageOptions: {
@@ -224,8 +223,16 @@ const config = [
         },
     },
     {
-        files: ['**/*.js'],
+        files: ['**/*.js', '**/*.mjs', '**/*.cjs', '**/*.jsx'],
         ...tsEslint.configs.disableTypeChecked,
     },
+    {
+        name: '@dartess/recommended-js',
+        files: ['**/*.js', '**/*.mjs', '**/*.cjs', '**/*.jsx'],
+        rules: {
+            // enable for js files only
+            'no-throw-literal': 'error',
+        },
+    },
 ];
 export default config;

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

@@ -87,19 +87,17 @@ declare const rules: {
     'no-script-url': "error";
     'no-self-compare': "error";
     'no-sequences': "error";
-    'no-throw-literal': "error";
     'no-useless-concat': "error";
     'no-useless-return': "error";
     'no-void': "error";
     'prefer-promise-reject-errors': ["error", {
         allowEmptyReject: boolean;
     }];
-    'prefer-object-has-own': "off";
+    'prefer-object-has-own': "error";
     'prefer-regex-literals': ["error", {
         disallowRedundantWrapping: boolean;
     }];
     radix: "error";
-    'require-await': "off";
     'vars-on-top': "error";
     yoda: "error";
 };

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

@@ -171,9 +171,6 @@ const rules = {
     // disallow use of comma operator
     // https://eslint.org/docs/rules/no-sequences
     'no-sequences': 'error',
-    // restrict what can be thrown as an exception
-    // https://eslint.org/docs/rules/no-throw-literal
-    'no-throw-literal': 'error',
     // disallow useless string concatenation
     // https://eslint.org/docs/rules/no-useless-concat
     'no-useless-concat': 'error',
@@ -188,8 +185,7 @@ const rules = {
     'prefer-promise-reject-errors': ['error', { allowEmptyReject: true }],
     // Prefer Object.hasOwn() over Object.prototype.hasOwnProperty.call()
     // https://eslint.org/docs/rules/prefer-object-has-own
-    // TODO: enable after browserslist Chrome 93+ and Safari 15.4+ and Firefox 92+
-    'prefer-object-has-own': 'off',
+    'prefer-object-has-own': 'error',
     // https://eslint.org/docs/rules/prefer-regex-literals
     'prefer-regex-literals': [
         'error',
@@ -200,9 +196,6 @@ const rules = {
     // require use of the second argument for parseInt()
     // https://eslint.org/docs/rules/radix
     radix: 'error',
-    // require `await` in `async function` (note: this is a horrible rule that should never be used)
-    // https://eslint.org/docs/rules/require-await
-    'require-await': 'off',
     // requires to declare all vars on top of their containing scope
     // https://eslint.org/docs/rules/vars-on-top
     'vars-on-top': 'error',

package/dist/configs/vendor-rules/es6.d.ts

@@ -2,17 +2,12 @@ declare const rules: {
     'no-restricted-exports': ["error", {
         restrictedNamedExports: string[];
     }];
-    'no-restricted-imports': ["off", {
-        paths: never[];
-        patterns: never[];
-    }];
     'no-useless-computed-key': "error";
     'no-useless-rename': ["error", {
         ignoreDestructuring: boolean;
         ignoreImport: boolean;
         ignoreExport: boolean;
     }];
-    'no-var': "error";
     'object-shorthand': ["error", string, {
         ignoreConstructors: boolean;
         avoidQuotes: boolean;

package/dist/configs/vendor-rules/es6.js

@@ -14,15 +14,6 @@ const rules = {
             ],
         },
     ],
-    // disallow specific imports
-    // https://eslint.org/docs/rules/no-restricted-imports
-    'no-restricted-imports': [
-        'off',
-        {
-            paths: [],
-            patterns: [],
-        },
-    ],
     // disallow useless computed property keys
     // https://eslint.org/docs/rules/no-useless-computed-key
     'no-useless-computed-key': 'error',
@@ -36,8 +27,6 @@ const rules = {
             ignoreExport: false,
         },
     ],
-    // require let or const instead of var
-    'no-var': 'error',
     // require method and property shorthand syntax for object literals
     // https://eslint.org/docs/rules/object-shorthand
     'object-shorthand': [

package/dist/configs/vendor-rules/react.d.ts

@@ -152,9 +152,6 @@ declare const rules: {
         ignoreNonDOM: boolean;
     }];
     'jsx-a11y/aria-unsupported-elements': "error";
-    'jsx-a11y/autocomplete-valid': ["off", {
-        inputComponents: never[];
-    }];
     'jsx-a11y/click-events-have-key-events': "error";
     'jsx-a11y/heading-has-content': ["error", {
         components: string[];

package/dist/configs/vendor-rules/react.js

@@ -16,6 +16,7 @@ const rules = {
             allow: ['__REDUX_DEVTOOLS_EXTENSION_COMPOSE__'],
         },
     ],
+    // TODO check it later
     // Prevent missing displayName in a React component definition
     // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/display-name.md
     'react/display-name': ['off', { ignoreTranspilerName: false }],
@@ -328,14 +329,6 @@ const rules = {
     // properties do not have those attributes.
     // https://github.com/jsx-eslint/eslint-plugin-jsx-a11y/blob/master/docs/rules/aria-unsupported-elements.md
     'jsx-a11y/aria-unsupported-elements': 'error',
-    // Ensure the autocomplete attribute is correct and suitable for the form field it is used with
-    // https://github.com/jsx-eslint/eslint-plugin-jsx-a11y/blob/29c68596b15c4ff0a40daae6d4a2670e36e37d35/docs/rules/autocomplete-valid.md
-    'jsx-a11y/autocomplete-valid': [
-        'off',
-        {
-            inputComponents: [],
-        },
-    ],
     // require onClick be accompanied by onKeyUp/onKeyDown/onKeyPress
     // https://github.com/jsx-eslint/eslint-plugin-jsx-a11y/blob/master/docs/rules/click-events-have-key-events.md
     'jsx-a11y/click-events-have-key-events': 'error',

package/dist/configs/vendor-rules/typescript.d.ts

@@ -21,10 +21,7 @@ declare const rules: {
         classes: boolean;
         variables: boolean;
     }];
-    'require-await': "off";
     '@typescript-eslint/require-await': "off";
-    'no-return-await': "off";
-    '@typescript-eslint/return-await': ["error", string];
     'import-x/extensions': ["error", string, {
         ts: string;
         tsx: string;

package/dist/configs/vendor-rules/typescript.js

@@ -31,13 +31,9 @@ const rules = {
     'no-use-before-define': 'off',
     '@typescript-eslint/no-use-before-define': baseVariablesRules['no-use-before-define'],
     // Replace Airbnb 'require-await' rule with '@typescript-eslint' version
+    // require `await` in `async function` (note: this is a horrible rule that should never be used)
     // https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/require-await.md
-    'require-await': 'off',
-    '@typescript-eslint/require-await': baseBestPracticesRules['require-await'],
-    // Replace Airbnb 'no-return-await' rule with '@typescript-eslint' version
-    // https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/return-await.md
-    'no-return-await': 'off',
-    '@typescript-eslint/return-await': ['error', 'in-try-catch'],
+    '@typescript-eslint/require-await': 'off',
     // Append 'ts' and 'tsx' to Airbnb 'import-x/extensions' rule
     // https://github.com/benmosher/eslint-plugin-import/blob/master/docs/rules/extensions.md
     'import-x/extensions': [

package/package.json

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