@yoo-digital/eslint-plugin-angular 1.4.0 → 2.0.1

package/README.md

@@ -12,15 +12,15 @@ Wrong code is yellow/red underlined in VScode, it can also be raised running : `
 
 This feature consists of **two complementary rules**:
 
-1. **`require-boolean-attribute-transform`** - TypeScript rule that enforces `booleanAttribute` transform on boolean inputs
-2. **`prefer-boolean-attribute-shorthand`** - Template rule that enforces shorthand syntax for `[attr]="true"` bindings
+1. **`boolean-attribute-ts`** - TypeScript rule that enforces `booleanAttribute` transform on boolean inputs
+2. **`boolean-attribute-html`** - Template rule that enforces shorthand syntax for `[attr]="true"` bindings
 
 ### Setting 
 ```json
 {
   "rules": {
-    "@yoo-digital/eslint-plugin-angular/require-boolean-attribute-transform": "error",
-    "@yoo-digital/eslint-plugin-angular/prefer-boolean-attribute-shorthand": "error"
+    "@yoo-digital/eslint-plugin-angular/boolean-attribute-ts": "error",
+    "@yoo-digital/eslint-plugin-angular/boolean-attribute-html": "error"
   }
 }
 ```

package/dist/index.d.ts

@@ -3,15 +3,15 @@ export declare const configs: {
     default: {
         plugins: string[];
         rules: {
-            '@yoo-digital/angular/prefer-boolean-attribute-shorthand': string;
-            '@yoo-digital/angular/require-boolean-attribute-transform': string;
+            '@yoo-digital/angular/boolean-attribute-html': string;
+            '@yoo-digital/angular/boolean-attribute-ts': string;
         };
     };
     recommended: {
         plugins: string[];
         rules: {
-            '@yoo-digital/angular/prefer-boolean-attribute-shorthand': string;
-            '@yoo-digital/angular/require-boolean-attribute-transform': string;
+            '@yoo-digital/angular/boolean-attribute-html': string;
+            '@yoo-digital/angular/boolean-attribute-ts': string;
         };
     };
 };

package/dist/index.js

@@ -3,22 +3,22 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.configs = exports.rules = void 0;
 const rules_1 = require("./rules");
 exports.rules = {
-    'prefer-boolean-attribute-shorthand': rules_1.preferBooleanAttributeShorthandRule,
-    'require-boolean-attribute-transform': rules_1.requireBooleanAttributeTransformRule,
+    'boolean-attribute-html': rules_1.preferBooleanAttributeShorthandRule,
+    'boolean-attribute-ts': rules_1.requireBooleanAttributeTransformRule,
 };
 exports.configs = {
     default: {
         plugins: ['@yoo-digital/eslint-plugin-angular'],
         rules: {
-            '@yoo-digital/angular/prefer-boolean-attribute-shorthand': 'warn',
-            '@yoo-digital/angular/require-boolean-attribute-transform': 'warn',
+            '@yoo-digital/angular/boolean-attribute-html': 'warn',
+            '@yoo-digital/angular/boolean-attribute-ts': 'warn',
         },
     },
     recommended: {
         plugins: ['@yoo-digital/eslint-plugin-angular'],
         rules: {
-            '@yoo-digital/angular/prefer-boolean-attribute-shorthand': 'error',
-            '@yoo-digital/angular/require-boolean-attribute-transform': 'error',
+            '@yoo-digital/angular/boolean-attribute-html': 'error',
+            '@yoo-digital/angular/boolean-attribute-ts': 'error',
         },
     },
 };

package/dist/rules/{prefer-boolean-attribute-shorthand.d.ts → boolean-attribute-html.d.ts}

@@ -1,6 +1,6 @@
 import type { TSESLint } from '@typescript-eslint/utils';
 type MessageIds = 'preferTrue' | 'suggestTrue';
-export declare const RULE_NAME = "prefer-boolean-attribute-shorthand";
+export declare const RULE_NAME = "boolean-attribute-html";
 /**
  * This rule enforces shorthand syntax for boolean inputs bound to true.
  *

package/dist/rules/{prefer-boolean-attribute-shorthand.js → boolean-attribute-html.js}

@@ -2,7 +2,7 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.preferBooleanAttributeShorthandRule = exports.RULE_NAME = void 0;
 const utils_1 = require("@angular-eslint/utils");
-exports.RULE_NAME = 'prefer-boolean-attribute-shorthand';
+exports.RULE_NAME = 'boolean-attribute-html';
 /**
  * This rule enforces shorthand syntax for boolean inputs bound to true.
  *

package/dist/rules/{require-boolean-attribute-transform.d.ts → boolean-attribute-ts.d.ts}

@@ -1,6 +1,6 @@
 import type { TSESLint } from '@typescript-eslint/utils';
 type MessageIds = 'requireTransformDecorator' | 'requireTransformSignal';
-export declare const RULE_NAME = "require-boolean-attribute-transform";
+export declare const RULE_NAME = "boolean-attribute-ts";
 /**
  * This rule enforces that boolean @Input() properties and input() signals
  * use the booleanAttribute transform.

package/dist/rules/{require-boolean-attribute-transform.js → boolean-attribute-ts.js}

@@ -1,7 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.requireBooleanAttributeTransformRule = exports.RULE_NAME = void 0;
-exports.RULE_NAME = 'require-boolean-attribute-transform';
+exports.RULE_NAME = 'boolean-attribute-ts';
 /**
  * This rule enforces that boolean @Input() properties and input() signals
  * use the booleanAttribute transform.
@@ -105,19 +105,22 @@ exports.requireBooleanAttributeTransformRule = {
                     return;
                 }
                 const callExpr = node.init;
-                // Check if it's a boolean input
-                if (!callExpr.typeArguments || callExpr.typeArguments.params.length === 0) {
-                    return;
+                // Check if it's a boolean input (with or without type arguments)
+                let isBooleanInput = false;
+                if (callExpr.typeArguments && callExpr.typeArguments.params.length > 0) {
+                    const firstTypeParam = callExpr.typeArguments.params[0];
+                    isBooleanInput = firstTypeParam.type === 'TSBooleanKeyword';
                 }
-                const firstTypeParam = callExpr.typeArguments.params[0];
-                if (firstTypeParam.type !== 'TSBooleanKeyword') {
+                if (!isBooleanInput) {
                     return;
                 }
+                // At this point, we know typeArguments exists and has at least one boolean param
+                const typeArgs = callExpr.typeArguments;
                 // Check if it already has BooleanInput as second type parameter
-                const hasBooleanInput = callExpr.typeArguments.params.length > 1 &&
-                    callExpr.typeArguments.params[1].type === 'TSTypeReference' &&
-                    callExpr.typeArguments.params[1].typeName.type === 'Identifier' &&
-                    callExpr.typeArguments.params[1].typeName.name === 'BooleanInput';
+                const hasBooleanInput = typeArgs.params.length > 1 &&
+                    typeArgs.params[1].type === 'TSTypeReference' &&
+                    typeArgs.params[1].typeName.type === 'Identifier' &&
+                    typeArgs.params[1].typeName.name === 'BooleanInput';
                 // Check if it already has transform in options
                 const hasTransformOption = callExpr.arguments.length > 1 &&
                     callExpr.arguments[1].type === 'ObjectExpression' &&
@@ -126,6 +129,7 @@ exports.requireBooleanAttributeTransformRule = {
                         prop.key.name === 'transform' &&
                         prop.value.type === 'Identifier' &&
                         prop.value.name === 'booleanAttribute');
+                // Both BooleanInput type and transform option are required
                 if (hasBooleanInput && hasTransformOption) {
                     return; // Already correctly configured
                 }

package/dist/rules/index.d.ts

@@ -1,2 +1,2 @@
-export { preferBooleanAttributeShorthandRule, RULE_NAME as PREFER_BOOLEAN_ATTRIBUTE_SHORTHAND_RULE_NAME } from './prefer-boolean-attribute-shorthand';
-export { requireBooleanAttributeTransformRule, RULE_NAME as REQUIRE_BOOLEAN_ATTRIBUTE_TRANSFORM_RULE_NAME } from './require-boolean-attribute-transform';
+export { RULE_NAME as BOOLEAN_ATTRIBUTE_HTML_RULE_NAME, preferBooleanAttributeShorthandRule } from './boolean-attribute-html';
+export { RULE_NAME as BOOLEAN_ATTRIBUTE_TS_RULE_NAME, requireBooleanAttributeTransformRule } from './boolean-attribute-ts';

package/dist/rules/index.js

@@ -1,9 +1,9 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.REQUIRE_BOOLEAN_ATTRIBUTE_TRANSFORM_RULE_NAME = exports.requireBooleanAttributeTransformRule = exports.PREFER_BOOLEAN_ATTRIBUTE_SHORTHAND_RULE_NAME = exports.preferBooleanAttributeShorthandRule = void 0;
-var prefer_boolean_attribute_shorthand_1 = require("./prefer-boolean-attribute-shorthand");
-Object.defineProperty(exports, "preferBooleanAttributeShorthandRule", { enumerable: true, get: function () { return prefer_boolean_attribute_shorthand_1.preferBooleanAttributeShorthandRule; } });
-Object.defineProperty(exports, "PREFER_BOOLEAN_ATTRIBUTE_SHORTHAND_RULE_NAME", { enumerable: true, get: function () { return prefer_boolean_attribute_shorthand_1.RULE_NAME; } });
-var require_boolean_attribute_transform_1 = require("./require-boolean-attribute-transform");
-Object.defineProperty(exports, "requireBooleanAttributeTransformRule", { enumerable: true, get: function () { return require_boolean_attribute_transform_1.requireBooleanAttributeTransformRule; } });
-Object.defineProperty(exports, "REQUIRE_BOOLEAN_ATTRIBUTE_TRANSFORM_RULE_NAME", { enumerable: true, get: function () { return require_boolean_attribute_transform_1.RULE_NAME; } });
+exports.requireBooleanAttributeTransformRule = exports.BOOLEAN_ATTRIBUTE_TS_RULE_NAME = exports.preferBooleanAttributeShorthandRule = exports.BOOLEAN_ATTRIBUTE_HTML_RULE_NAME = void 0;
+var boolean_attribute_html_1 = require("./boolean-attribute-html");
+Object.defineProperty(exports, "BOOLEAN_ATTRIBUTE_HTML_RULE_NAME", { enumerable: true, get: function () { return boolean_attribute_html_1.RULE_NAME; } });
+Object.defineProperty(exports, "preferBooleanAttributeShorthandRule", { enumerable: true, get: function () { return boolean_attribute_html_1.preferBooleanAttributeShorthandRule; } });
+var boolean_attribute_ts_1 = require("./boolean-attribute-ts");
+Object.defineProperty(exports, "BOOLEAN_ATTRIBUTE_TS_RULE_NAME", { enumerable: true, get: function () { return boolean_attribute_ts_1.RULE_NAME; } });
+Object.defineProperty(exports, "requireBooleanAttributeTransformRule", { enumerable: true, get: function () { return boolean_attribute_ts_1.requireBooleanAttributeTransformRule; } });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yoo-digital/eslint-plugin-angular",
-  "version": "1.4.0",
+  "version": "2.0.1",
   "description": "Yoo Digital custom Angular ESLint plugin for enforcing boolean attribute best practices.",
   "type": "commonjs",
   "main": "dist/index.js",