eslint-config-angular-strict 2.1.3 → 2.1.5

package/index.js

@@ -8,6 +8,15 @@ import tsParser from '@typescript-eslint/parser';
 import { configs, rules } from 'eslint-config-airbnb-extended';
 
 export default [
+  // Global plugins configuration for all files
+  {
+    plugins: {
+      'import-x': importX,
+      '@stylistic': stylistic,
+      '@typescript-eslint': tsEslint,
+    },
+  },
+
   // Airbnb strict rules
   ...configs.base.all,
   rules.base.strict,
@@ -21,9 +30,6 @@ export default [
     },
     plugins: {
       '@angular-eslint': angular,
-      '@stylistic': stylistic,
-      '@typescript-eslint': tsEslint,
-      'import-x': importX,
     },
     rules: {
       // Angular ESLint rules
@@ -80,8 +86,10 @@ export default [
       '@stylistic/indent': ['error', 2],
       '@stylistic/keyword-spacing': ['error', { before: true, after: true }],
       '@stylistic/lines-between-class-members': ['error', 'always', { exceptAfterSingleLine: true }],
+      '@stylistic/max-len': ['error', 180],
       '@stylistic/no-extra-semi': 'error',
       '@stylistic/object-curly-spacing': ['error', 'always'],
+      '@stylistic/padded-blocks': 'off',
       '@stylistic/quotes': ['error', 'single'],
       '@stylistic/semi': ['error', 'always'],
       '@stylistic/space-before-blocks': 'error',
@@ -117,7 +125,6 @@ export default [
       // General rules
       'arrow-parens': ['error', 'as-needed', { requireForBlockBody: true }],
       'class-methods-use-this': ['error', { exceptMethods: ['beforeUnloadHander', 'trackBy', 'transform', 'windowRef'] }],
-      'max-len': ['error', 180],
       'max-lines': ['error', 500],
       'no-param-reassign': ['error', { props: false }],
       'no-plusplus': 'off',
@@ -133,7 +140,6 @@ export default [
           ObjectPattern: { minProperties: 4, multiline: true },
         },
       ],
-      'padded-blocks': 'off',
       'prefer-destructuring': 'off',
       'sort-keys': ['error'],
       radix: ['error', 'as-needed'],

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-config-angular-strict",
-  "version": "2.1.3",
+  "version": "2.1.5",
   "description": "Modern ESLint configuration with strict rules for Angular development.",
   "keywords": [
     "angular",
@@ -31,6 +31,9 @@
   "exports": {
     ".": "./index.js"
   },
+  "scripts": {
+    "test": "cd tests && yarn install && eslint . --format=stylish"
+  },
   "dependencies": {
     "@angular-eslint/builder": "20.3.0",
     "@angular-eslint/eslint-plugin": "20.3.0",

package/tests/eslint.config.js

@@ -0,0 +1,3 @@
+import angularStrict from '../index.js';
+
+export default [...angularStrict];

package/tests/package.json

@@ -0,0 +1,9 @@
+{
+  "name": "eslint-config-angular-strict-tests",
+  "version": "1.0.0",
+  "private": true,
+  "type": "module",
+  "dependencies": {
+    "@angular/core": "^20.3.2"
+  }
+}

package/tests/sample.component.html

@@ -0,0 +1,38 @@
+<!-- Angular template with intentional violations to test ESLint rules -->
+<div class="sample-container">
+  <h2>Sample Test Component</h2>
+
+  <!-- Missing trackBy function in ngFor -->
+  <ul>
+    <li *ngFor="let item of _items">
+      {{ item }}
+    </li>
+  </ul>
+
+  <!-- Attributes not in alphabetical order -->
+  <input
+    type="text"
+    placeholder="Enter name"
+    id="name-field"
+    class="form-control"
+    [(ngModel)]="name">
+
+  <!-- Missing label for accessibility -->
+  <input type="checkbox" id="option1" name="options" value="1">
+  Option 1
+
+  <!-- Positive tabindex (accessibility violation) -->
+  <button tabindex="2" (click)="processItems()">Process</button>
+
+  <!-- Non-self-closing tags -->
+  <br>
+  <hr>
+
+  <!-- Using any type in template -->
+  <div>{{ config.anyProperty }}</div>
+
+  <!-- Missing key events for mouse events -->
+  <div (click)="loadData()" class="clickable">
+    Click to load data
+  </div>
+</div>

package/tests/sample.component.ts

@@ -0,0 +1,34 @@
+import { Component, OnInit, Input } from '@angular/core';
+
+// Test component with intentional violations to validate ESLint rules
+
+@Component({
+  selector: 'app-sample',
+  templateUrl: './sample.component.html',
+  styleUrls: ['./sample.component.css'] // Should trigger consistent-component-styles
+})
+export class SampleComponent implements OnInit {
+  @Input() config: any; // Should trigger no-explicit-any warning
+  private _items!: string[]; // Non-null assertion
+  public name = 'sample';
+
+  constructor() {} // Empty constructor should trigger warning
+
+  ngOnInit(): void {
+    this.loadData();
+  }
+
+  loadData() {
+    this._items = this.config?.items || [];
+  }
+
+  // Method should be after getter based on member-ordering
+  processItems() {
+    return this._items.map(item => item.toUpperCase()); // Missing arrow function parens
+  }
+
+  // Getter should be before methods
+  get itemCount(): number {
+    return this._items?.length || 0;
+  }
+}

package/tests/tsconfig.json

@@ -0,0 +1,25 @@
+{
+  "compilerOptions": {
+    "declaration": false,
+    "esModuleInterop": true,
+    "experimentalDecorators": true,
+    "forceConsistentCasingInFileNames": true,
+    "importHelpers": true,
+    "incremental": true,
+    "lib": ["es2022", "dom"],
+    "module": "es2022",
+    "moduleResolution": "bundler",
+    "noImplicitAny": true,
+    "noImplicitOverride": true,
+    "noUnusedLocals": true,
+    "outDir": "./dist/out-tsc",
+    "sourceMap": true,
+    "strict": true,
+    "target": "es2022"
+  },
+  "angularCompilerOptions": {
+    "strictInjectionParameters": true,
+    "strictStandalone": true,
+    "strictTemplates": true
+  }
+}