@yoo-digital/eslint-plugin-angular 2.0.8 → 3.0.2

package/README.md

@@ -1,10 +1,10 @@
 # YOO ESLint plugin Angular
 
-## Custom lint purpose
+## Aim
 
-Here should live all custom Angular lint rules that eslint does not already provide.
+Here should live all ***custom Angular lint rules*** that eslint does not already provide.
 
-## Linting
+## Use
 
 Wrong code is yellow/red underlined in VScode, it can also be raised running : `npm run lint`, autofixing them with : `npm run lint:fix`
 
@@ -15,6 +15,7 @@ TypeScript rule that enforces `booleanAttribute` transform on boolean inputs
 ### Setting 
 ```json
 {
+  "files": ["**/*.ts"],
   "rules": {
     "@yoo-digital/eslint-plugin-angular/boolean-attribute-shorthand": "error"
   }
@@ -47,6 +48,7 @@ HTML rule that enforces shorthand syntax for `[attr]="true"` bindings
 ### Setting 
 ```json
 {
+  "files": ["**/*.html"],
   "rules": {
     "@yoo-digital/eslint-plugin-angular/boolean-attribute-shorthand": "error"
   }
@@ -56,7 +58,7 @@ HTML rule that enforces shorthand syntax for `[attr]="true"` bindings
 #### True value 
 ```html
 <mealComponent [isVegan]="true" />
-<!-- Lint issue enforcing to be :  --> 
+<!-- Lint issue, must become :  --> 
 <mealComponent isVegan />
 ```
 
@@ -66,7 +68,7 @@ HTML rule that enforces shorthand syntax for `[attr]="true"` bindings
 <!-- No lint issue, to be able to address false value for a default true input -->
 ```
 
-### Default value 
+### Right usage 
 #### Default true 
 
 If boolean input is by default **true**

package/dist/rules/boolean-attribute-shorthand.js

@@ -29,6 +29,7 @@ exports.preferBooleanAttributeShorthandRule = {
         docs: {
             description: 'Prefer boolean input attribute shorthand when binding to true (e.g., use "disabled" instead of [disabled]="true").',
         },
+        fixable: 'code',
         schema: [],
         messages: {
             preferTrue: 'Use attribute shorthand "{{attr}}" instead of [{{attr}}]="true".',
@@ -50,10 +51,13 @@ exports.preferBooleanAttributeShorthandRule = {
                     if (ast.value === true) {
                         const attrName = node.name;
                         const loc = parserServices.convertNodeSourceSpanToLoc(node.sourceSpan);
+                        const start = node.sourceSpan.start.offset;
+                        const end = node.sourceSpan.end.offset;
                         context.report({
                             loc,
                             messageId: 'preferTrue',
                             data: { attr: attrName },
+                            fix: (fixer) => fixer.replaceTextRange([start, end], attrName),
                         });
                     }
                     // [attr]="false" is explicitly ignored - no warning

package/dist/rules/boolean-input.js

@@ -19,6 +19,7 @@ exports.requireBooleanAttributeTransformRule = {
         docs: {
             description: 'Require booleanAttribute transform on boolean @Input() properties and input() signals.',
         },
+        fixable: 'code',
         schema: [],
         messages: {
             requireTransformDecorator: 'Boolean @Input() "{{name}}" must use transform: booleanAttribute',
@@ -27,6 +28,7 @@ exports.requireBooleanAttributeTransformRule = {
     },
     defaultOptions: [],
     create(context) {
+        const sourceCode = context.sourceCode || context.getSourceCode();
         return {
             // Handle both @Input() decorator syntax and input() signal syntax
             PropertyDefinition(node) {
@@ -62,6 +64,35 @@ exports.requireBooleanAttributeTransformRule = {
                         node: node.key,
                         messageId: 'requireTransformDecorator',
                         data: { name: propertyName },
+                        fix(fixer) {
+                            const decoratorNode = inputDecorator?.expression;
+                            if (!decoratorNode || decoratorNode.type !== 'CallExpression') {
+                                return null;
+                            }
+                            const decoratorStart = decoratorNode.range[0];
+                            const decoratorEnd = decoratorNode.range[1];
+                            // Generate the fixed decorator
+                            let newDecorator;
+                            if (decoratorNode.arguments.length === 0) {
+                                // @Input() → @Input({ transform: booleanAttribute })
+                                newDecorator = '@Input({ transform: booleanAttribute })';
+                            }
+                            else if (decoratorNode.arguments[0].type === 'ObjectExpression') {
+                                // @Input({ ... }) → @Input({ ..., transform: booleanAttribute })
+                                const objExpr = decoratorNode.arguments[0];
+                                const objText = sourceCode.getText(objExpr);
+                                const closingBrace = objText.lastIndexOf('}');
+                                const existingProps = objText.substring(1, closingBrace).trim();
+                                const separator = existingProps ? ', ' : '';
+                                newDecorator = `@Input({ ${existingProps}${separator}transform: booleanAttribute })`;
+                            }
+                            else {
+                                // @Input('alias') → @Input({ alias: 'alias', transform: booleanAttribute })
+                                const aliasArg = sourceCode.getText(decoratorNode.arguments[0]);
+                                newDecorator = `@Input({ alias: ${aliasArg}, transform: booleanAttribute })`;
+                            }
+                            return fixer.replaceTextRange([decoratorStart, decoratorEnd], newDecorator);
+                        },
                     });
                     return;
                 }
@@ -109,6 +140,15 @@ exports.requireBooleanAttributeTransformRule = {
                     node: node.key,
                     messageId: 'requireTransformSignal',
                     data: { name: propertyName },
+                    fix(fixer) {
+                        const callStart = callExpr.range[0];
+                        const callEnd = callExpr.range[1];
+                        // Get the default value argument
+                        const defaultValue = callExpr.arguments[0] ? sourceCode.getText(callExpr.arguments[0]) : 'false';
+                        // Build the new call expression
+                        const newCall = `input<boolean, BooleanInput>(${defaultValue}, {\\n    transform: booleanAttribute,\\n  })`;
+                        return fixer.replaceTextRange([callStart, callEnd], newCall);
+                    },
                 });
             },
             // Handle input() signal syntax (standalone variable declarations - rare)

package/package.json

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