@itstandu/code-style 0.1.2 → 0.1.5

package/eslint/base.js

@@ -1,4 +1,4 @@
-const prettierConfig = require('eslint-config-prettier')
+const prettierConfig = require('eslint-config-prettier');
 
 module.exports = {
   ignores: [
@@ -60,7 +60,14 @@ module.exports = {
     'simple-import-sort/imports': [
       'error',
       {
-        groups: [['^\\u0000'], ['^node:'], ['^@?\\w'], ['^@/'], ['^\\.\\.'], ['^\\.']],
+        groups: [
+          ['^\\u0000'],
+          ['^node:'],
+          ['^@?\\w'],
+          ['^@/'],
+          ['^\\.\\.'],
+          ['^\\.'],
+        ],
       },
     ],
     'simple-import-sort/exports': 'error',
@@ -77,12 +84,22 @@ module.exports = {
     ],
     'no-unused-vars': 'off',
 
-    // STRICT: No eslint-disable comments allowed
-    'no-warning-comments': [
+    // STRICT: Dangerous patterns prevention
+    // Note: To enforce no eslint-disable comments, add @eslint-community/eslint-plugin-eslint-comments
+    'no-restricted-syntax': [
       'error',
       {
-        terms: ['eslint-disable', 'eslint-disable-line', 'eslint-disable-next-line'],
-        location: 'anywhere',
+        selector: 'ForInStatement',
+        message:
+          'for...in loops iterate over the entire prototype chain. Use Object.keys() or for...of instead.',
+      },
+      {
+        selector: 'LabeledStatement',
+        message: 'Labels are rarely needed. Consider refactoring.',
+      },
+      {
+        selector: 'WithStatement',
+        message: '`with` is disallowed in strict mode.',
       },
     ],
 
@@ -91,4 +108,4 @@ module.exports = {
     'import/newline-after-import': 'error',
     'import/no-duplicates': 'error',
   },
-}
+};

package/eslint/nest.js

@@ -1,4 +1,4 @@
-const node = require('./node')
+const node = require('./node');
 
 module.exports = {
   ...node,
@@ -22,4 +22,4 @@ module.exports = {
     // Allow any for DTOs and decorators
     '@typescript-eslint/no-explicit-any': 'off',
   },
-}
+};

package/eslint/react.js

@@ -1,4 +1,4 @@
-const ts = require('./typescript')
+const ts = require('./typescript');
 
 module.exports = {
   ...ts,
@@ -45,6 +45,7 @@ module.exports = {
     'react/no-direct-mutation-state': 'error',
     'react/no-unescaped-entities': 'off',
     'react/no-unknown-property': 'error',
+    'react/no-unstable-nested-components': ['warn', { allowAsProps: true }],
     'react/self-closing-comp': 'off',
 
     'react/jsx-boolean-value': 'off',
@@ -98,4 +99,4 @@ module.exports = {
     'jsx-a11y/scope': 'warn',
     'jsx-a11y/tabindex-no-positive': 'warn',
   },
-}
+};

package/eslint/typescript.js

@@ -1,4 +1,4 @@
-const base = require('./base')
+const base = require('./base');
 
 module.exports = {
   ...base,
@@ -58,13 +58,22 @@ module.exports = {
     '@typescript-eslint/no-shadow': 'warn',
     '@typescript-eslint/no-require-imports': 'warn',
     '@typescript-eslint/ban-ts-comment': [
-      'warn',
+      'error',
       {
         'ts-expect-error': 'allow-with-description',
-        'ts-ignore': true,
+        'ts-ignore': 'allow-with-description',
         'ts-nocheck': true,
         'ts-check': false,
+        minimumDescriptionLength: 10,
       },
     ],
+
+    // Strict: Prevent common mistakes
+    '@typescript-eslint/no-array-delete': 'error',
+    '@typescript-eslint/no-duplicate-enum-values': 'error',
+    '@typescript-eslint/no-duplicate-type-constituents': 'warn',
+    '@typescript-eslint/no-redundant-type-constituents': 'warn',
+    '@typescript-eslint/no-unsafe-declaration-merging': 'error',
+    '@typescript-eslint/no-useless-empty-export': 'warn',
   },
-}
+};

package/eslint/vue.js

@@ -1,32 +1,24 @@
-const base = require('./base')
+const ts = require('./typescript');
 
 module.exports = {
-  ...base,
+  ...ts,
   languageOptions: {
-    ...base.languageOptions,
+    ...ts.languageOptions,
     parser: require('vue-eslint-parser'),
     parserOptions: {
       parser: require('@typescript-eslint/parser'),
       ecmaVersion: 'latest',
       sourceType: 'module',
       project: true,
+      extraFileExtensions: ['.vue'],
     },
   },
   plugins: {
-    ...base.plugins,
+    ...ts.plugins,
     vue: require('eslint-plugin-vue'),
-    '@typescript-eslint': require('@typescript-eslint/eslint-plugin'),
   },
   rules: {
-    ...base.rules,
-    'no-unused-vars': 'off',
-    '@typescript-eslint/no-unused-vars': 'off',
-    '@typescript-eslint/no-floating-promises': 'error',
-    '@typescript-eslint/no-misused-promises': 'error',
-    '@typescript-eslint/consistent-type-imports': [
-      'error',
-      { prefer: 'type-imports', fixStyle: 'inline-type-imports' },
-    ],
+    ...ts.rules,
 
     'vue/component-name-in-template-casing': ['warn', 'PascalCase'],
     'vue/component-definition-name-casing': ['warn', 'PascalCase'],
@@ -98,4 +90,4 @@ module.exports = {
     'vue/valid-v-slot': 'error',
     'vue/valid-v-text': 'error',
   },
-}
+};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@itstandu/code-style",
-  "version": "0.1.2",
+  "version": "0.1.5",
   "description": "Production-ready shared ESLint + Prettier configuration for JavaScript and TypeScript projects",
   "type": "commonjs",
   "main": "index.js",