@itstandu/code-style 0.1.3 → 0.1.6

package/README.md

@@ -142,6 +142,46 @@ Or add to `package.json`:
 cp node_modules/@itstandu/code-style/.prettierignore .prettierignore
 ```
 
+### Setup Scripts
+
+Add these scripts to your `package.json` for convenient formatting and linting:
+
+```json
+{
+  "scripts": {
+    "format": "prettier --write \"**/*.{js,jsx,ts,tsx,json,css,scss,md}\"",
+    "format:check": "prettier --check \"**/*.{js,jsx,ts,tsx,json,css,scss,md}\"",
+    "lint": "eslint .",
+    "lint:fix": "eslint . --fix"
+  }
+}
+```
+
+**Usage:**
+
+- `npm run format` - Format all files with Prettier
+- `npm run format:check` - Check formatting without modifying files (useful for CI)
+- `npm run lint` - Check code for linting errors
+- `npm run lint:fix` - Auto-fix linting issues (including import sorting)
+
+**Recommended workflow:**
+
+Run both format and lint:fix to ensure consistent code style:
+
+```bash
+npm run format && npm run lint:fix
+```
+
+Or create a combined script:
+
+```json
+{
+  "scripts": {
+    "style": "npm run format && npm run lint:fix"
+  }
+}
+```
+
 ### Ensuring Consistent Formatting
 
 This package ensures **100% Prettier coverage** for all formatting:
@@ -177,6 +217,47 @@ This package ensures **100% Prettier coverage** for all formatting:
 - Uses Prettier as default formatter with `formatOnSave`
 - Runs ESLint auto-fix on save (`source.fixAll.eslint`) for import sorting
 
+### Editor Configuration
+
+#### VS Code / Cursor
+
+Add to your workspace `.vscode/settings.json` or user settings:
+
+```json
+{
+  "editor.defaultFormatter": "esbenp.prettier-vscode",
+  "editor.formatOnSave": true,
+  "editor.codeActionsOnSave": {
+    "source.fixAll.eslint": "explicit"
+  },
+  "[javascript]": {
+    "editor.defaultFormatter": "esbenp.prettier-vscode"
+  },
+  "[javascriptreact]": {
+    "editor.defaultFormatter": "esbenp.prettier-vscode"
+  },
+  "[typescript]": {
+    "editor.defaultFormatter": "esbenp.prettier-vscode"
+  },
+  "[typescriptreact]": {
+    "editor.defaultFormatter": "esbenp.prettier-vscode"
+  }
+}
+```
+
+**What this does:**
+
+- ✅ **Prettier** formats code on save (whitespace, quotes, etc.)
+- ✅ **ESLint auto-fix** runs on save, which will:
+  - Remove unused imports (`unused-imports/no-unused-imports`)
+  - Sort imports (`simple-import-sort/imports`)
+  - Fix other auto-fixable ESLint issues
+
+**Required extensions:**
+
+- [Prettier - Code formatter](https://marketplace.visualstudio.com/items?itemName=esbenp.prettier-vscode)
+- [ESLint](https://marketplace.visualstudio.com/items?itemName=dbaeumer.vscode-eslint)
+
 ## Presets
 
 ### `base`

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.3",
+  "version": "0.1.6",
   "description": "Production-ready shared ESLint + Prettier configuration for JavaScript and TypeScript projects",
   "type": "commonjs",
   "main": "index.js",
@@ -45,15 +45,15 @@
     "@angular-eslint/eslint-plugin": "^19.0.0",
     "@angular-eslint/eslint-plugin-template": "^19.0.0",
     "@angular-eslint/template-parser": "^19.0.0",
-    "@typescript-eslint/eslint-plugin": "^8.0.0",
-    "@typescript-eslint/parser": "^8.0.0",
+    "@typescript-eslint/eslint-plugin": "^8.53.0",
+    "@typescript-eslint/parser": "^8.53.0",
     "eslint": "^9.0.0",
-    "eslint-config-prettier": "^10.0.0",
-    "eslint-plugin-boundaries": "^5.0.0",
+    "eslint-config-prettier": "^10.1.8",
+    "eslint-plugin-boundaries": "^5.3.1",
     "eslint-plugin-import": "^2.31.0",
     "eslint-plugin-jsx-a11y": "^6.10.0",
     "eslint-plugin-n": "^17.0.0",
-    "eslint-plugin-promise": "^7.0.0",
+    "eslint-plugin-promise": "^7.2.0",
     "eslint-plugin-react": "^7.37.0",
     "eslint-plugin-react-hooks": "^5.0.0",
     "eslint-plugin-simple-import-sort": "^12.0.0",
@@ -61,9 +61,9 @@
     "eslint-plugin-unicorn": "^56.0.0",
     "eslint-plugin-unused-imports": "^4.0.0",
     "eslint-plugin-vue": "^9.32.0",
-    "@prettier/plugin-oxc": "^0.0.11",
-    "prettier": "^3.4.0",
-    "prettier-plugin-tailwindcss": "^0.6.0",
+    "@prettier/plugin-oxc": "^0.1.0",
+    "prettier": "^3.8.0",
+    "prettier-plugin-tailwindcss": "^0.7.0",
     "vue-eslint-parser": "^9.4.0"
   },
   "peerDependencies": {
@@ -77,4 +77,4 @@
   "scripts": {
     "test": "echo \"Error: no test specified\" && exit 1"
   }
-}
+}