@angular-eslint/eslint-plugin 21.0.2-alpha.10 → 21.0.2-alpha.12

package/README.md

@@ -72,6 +72,7 @@ Please see https://github.com/angular-eslint/angular-eslint for full usage instr
 | [`prefer-on-push-component-change-detection`](https://github.com/angular-eslint/angular-eslint/blob/main/packages/eslint-plugin/docs/rules/prefer-on-push-component-change-detection.md) | Ensures component's `changeDetection` is set to `ChangeDetectionStrategy.OnPush` |  |  | :bulb: |
 | [`prefer-output-emitter-ref`](https://github.com/angular-eslint/angular-eslint/blob/main/packages/eslint-plugin/docs/rules/prefer-output-emitter-ref.md) | Use `OutputEmitterRef` instead of `@Output()` |  |  |  |
 | [`prefer-output-readonly`](https://github.com/angular-eslint/angular-eslint/blob/main/packages/eslint-plugin/docs/rules/prefer-output-readonly.md) | Prefer to declare `@Output`, `OutputEmitterRef` and `OutputRef` as `readonly` since they are not supposed to be reassigned |  |  | :bulb: |
+| [`prefer-signal-model`](https://github.com/angular-eslint/angular-eslint/blob/main/packages/eslint-plugin/docs/rules/prefer-signal-model.md) | Use `model` instead of `input` and `output` for two-way bindings |  | :wrench: |  |
 | [`prefer-signals`](https://github.com/angular-eslint/angular-eslint/blob/main/packages/eslint-plugin/docs/rules/prefer-signals.md) | Use readonly signals instead of `@Input()`, `@ViewChild()` and other legacy decorators |  | :wrench: |  |
 | [`prefer-standalone`](https://github.com/angular-eslint/angular-eslint/blob/main/packages/eslint-plugin/docs/rules/prefer-standalone.md) | Ensures Components, Directives and Pipes do not opt out of standalone. | :white_check_mark: |  | :bulb: |
 | [`relative-url-prefix`](https://github.com/angular-eslint/angular-eslint/blob/main/packages/eslint-plugin/docs/rules/relative-url-prefix.md) | The ./ and ../ prefix is standard syntax for relative URLs; don't depend on Angular's current ability to do without that prefix. |  |  |  |

package/dist/configs/all.json

@@ -34,6 +34,7 @@
     "@angular-eslint/prefer-on-push-component-change-detection": "error",
     "@angular-eslint/prefer-output-emitter-ref": "error",
     "@angular-eslint/prefer-output-readonly": "error",
+    "@angular-eslint/prefer-signal-model": "error",
     "@angular-eslint/prefer-signals": "error",
     "@angular-eslint/prefer-standalone": "error",
     "@angular-eslint/relative-url-prefix": "error",

package/dist/index.d.ts

@@ -36,6 +36,7 @@ declare const _default: {
                 "@angular-eslint/prefer-on-push-component-change-detection": string;
                 "@angular-eslint/prefer-output-emitter-ref": string;
                 "@angular-eslint/prefer-output-readonly": string;
+                "@angular-eslint/prefer-signal-model": string;
                 "@angular-eslint/prefer-signals": string;
                 "@angular-eslint/prefer-standalone": string;
                 "@angular-eslint/relative-url-prefix": string;
@@ -110,6 +111,7 @@ declare const _default: {
         "prefer-inject": import("@typescript-eslint/utils/ts-eslint").RuleModule<"preferInject", [], import("./utils/create-eslint-rule").RuleDocs, import("@typescript-eslint/utils/ts-eslint").RuleListener>;
         "prefer-output-emitter-ref": import("@typescript-eslint/utils/ts-eslint").RuleModule<"preferOutputEmitterRef", [], import("./utils/create-eslint-rule").RuleDocs, import("@typescript-eslint/utils/ts-eslint").RuleListener>;
         "prefer-output-readonly": import("@typescript-eslint/utils/ts-eslint").RuleModule<import("./rules/prefer-output-readonly").MessageIds, [], import("./utils/create-eslint-rule").RuleDocs, import("@typescript-eslint/utils/ts-eslint").RuleListener>;
+        "prefer-signal-model": import("@typescript-eslint/utils/ts-eslint").RuleModule<"preferSignalModel", [], import("./utils/create-eslint-rule").RuleDocs, import("@typescript-eslint/utils/ts-eslint").RuleListener>;
         "relative-url-prefix": import("@typescript-eslint/utils/ts-eslint").RuleModule<"relativeUrlPrefix", [], import("./utils/create-eslint-rule").RuleDocs, import("@typescript-eslint/utils/ts-eslint").RuleListener>;
         "require-lifecycle-on-prototype": import("@typescript-eslint/utils/ts-eslint").RuleModule<"defineOnPrototype", [], import("./utils/create-eslint-rule").RuleDocs, import("@typescript-eslint/utils/ts-eslint").RuleListener>;
         "require-localize-metadata": import("@typescript-eslint/utils/ts-eslint").RuleModule<import("./rules/require-localize-metadata").MessageIds, import("./rules/require-localize-metadata").Options, import("./utils/create-eslint-rule").RuleDocs, import("@typescript-eslint/utils/ts-eslint").RuleListener>;

package/dist/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;AA8IA,kBAsDE"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;AAiJA,kBAuDE"}

package/dist/index.js

@@ -67,6 +67,7 @@ const prefer_host_metadata_property_1 = __importStar(require("./rules/prefer-hos
 const prefer_on_push_component_change_detection_1 = __importStar(require("./rules/prefer-on-push-component-change-detection"));
 const prefer_output_emitter_ref_1 = __importStar(require("./rules/prefer-output-emitter-ref"));
 const prefer_output_readonly_1 = __importStar(require("./rules/prefer-output-readonly"));
+const prefer_signal_model_1 = __importStar(require("./rules/prefer-signal-model"));
 const prefer_inject_1 = __importStar(require("./rules/prefer-inject"));
 const prefer_signals_1 = __importStar(require("./rules/prefer-signals"));
 const prefer_standalone_1 = __importStar(require("./rules/prefer-standalone"));
@@ -122,6 +123,7 @@ module.exports = {
         [prefer_inject_1.RULE_NAME]: prefer_inject_1.default,
         [prefer_output_emitter_ref_1.RULE_NAME]: prefer_output_emitter_ref_1.default,
         [prefer_output_readonly_1.RULE_NAME]: prefer_output_readonly_1.default,
+        [prefer_signal_model_1.RULE_NAME]: prefer_signal_model_1.default,
         [relative_url_prefix_1.RULE_NAME]: relative_url_prefix_1.default,
         [require_lifecycle_on_prototype_1.RULE_NAME]: require_lifecycle_on_prototype_1.default,
         [require_localize_metadata_1.RULE_NAME]: require_localize_metadata_1.default,

package/dist/rules/prefer-signal-model.d.ts

@@ -0,0 +1,8 @@
+export type MessageIds = 'preferSignalModel';
+export declare const RULE_NAME = "prefer-signal-model";
+declare const _default: import("@typescript-eslint/utils/ts-eslint").RuleModule<"preferSignalModel", [], import("../utils/create-eslint-rule").RuleDocs, import("@typescript-eslint/utils/ts-eslint").RuleListener>;
+export default _default;
+export declare const RULE_DOCS_EXTENSION: {
+    rationale: string;
+};
+//# sourceMappingURL=prefer-signal-model.d.ts.map

package/dist/rules/prefer-signal-model.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"prefer-signal-model.d.ts","sourceRoot":"","sources":["../../src/rules/prefer-signal-model.ts"],"names":[],"mappings":"AAUA,MAAM,MAAM,UAAU,GAAG,mBAAmB,CAAC;AAC7C,eAAO,MAAM,SAAS,wBAAwB,CAAC;;AAE/C,wBA0EG;AAEH,eAAO,MAAM,mBAAmB;;CAG/B,CAAC"}

package/dist/rules/prefer-signal-model.js

@@ -0,0 +1,71 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.RULE_DOCS_EXTENSION = exports.RULE_NAME = void 0;
+const utils_1 = require("@angular-eslint/utils");
+const create_eslint_rule_1 = require("../utils/create-eslint-rule");
+exports.RULE_NAME = 'prefer-signal-model';
+exports.default = (0, create_eslint_rule_1.createESLintRule)({
+    name: exports.RULE_NAME,
+    meta: {
+        type: 'suggestion',
+        docs: {
+            description: 'Use `model` instead of `input` and `output` for two-way bindings',
+        },
+        fixable: 'code',
+        schema: [],
+        messages: {
+            preferSignalModel: 'Use `model` for two-way bindings instead of `input()` and `output()`',
+        },
+    },
+    defaultOptions: [],
+    create(context) {
+        const inputs = new Map();
+        const outputs = new Map();
+        return {
+            "PropertyDefinition > CallExpression[callee.name='input']"(node) {
+                const propertyDef = node.parent;
+                const propertyName = utils_1.ASTUtils.getPropertyDefinitionName(propertyDef);
+                inputs.set(propertyName, propertyDef);
+            },
+            "PropertyDefinition > CallExpression[callee.name='output']"(node) {
+                const propertyDef = node.parent;
+                const propertyName = utils_1.ASTUtils.getPropertyDefinitionName(propertyDef);
+                outputs.set(propertyName, propertyDef);
+            },
+            'ClassDeclaration:exit'() {
+                for (const [inputName, inputProperty] of inputs) {
+                    const outputName = `${inputName}Change`;
+                    const outputProperty = outputs.get(outputName);
+                    if (outputProperty) {
+                        // Report on the input property
+                        context.report({
+                            node: inputProperty,
+                            messageId: 'preferSignalModel',
+                            fix: (fixer) => {
+                                const sourceCode = context.sourceCode;
+                                const inputText = sourceCode.getText(inputProperty);
+                                const fixedInputText = inputText.replace(/\binput\b/, 'model');
+                                return [
+                                    utils_1.RuleFixes.getImportAddFix({
+                                        fixer,
+                                        importName: 'model',
+                                        moduleName: '@angular/core',
+                                        node: inputProperty,
+                                    }),
+                                    fixer.replaceText(inputProperty, fixedInputText),
+                                    fixer.remove(outputProperty),
+                                ].filter(utils_1.isNotNullOrUndefined);
+                            },
+                        });
+                    }
+                }
+                // Clear the maps for the next class
+                inputs.clear();
+                outputs.clear();
+            },
+        };
+    },
+});
+exports.RULE_DOCS_EXTENSION = {
+    rationale: "The model() function is Angular's modern API for two-way bindings, combining both input and output into a single signal. When you have an input property paired with an output property that follows the naming pattern of `propertyChange` (e.g., `enabled` input with `enabledChange` output), this is the traditional pattern for two-way binding. The model() function provides a cleaner, more concise way to express this pattern with better type safety and integration with Angular's signal ecosystem. It eliminates the boilerplate of managing separate input and output properties while maintaining the same two-way binding functionality.",
+};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@angular-eslint/eslint-plugin",
-  "version": "21.0.2-alpha.10",
+  "version": "21.0.2-alpha.12",
   "description": "ESLint plugin for Angular applications, following https://angular.dev/style-guide",
   "license": "MIT",
   "main": "dist/index.js",
@@ -19,11 +19,11 @@
   ],
   "dependencies": {
     "ts-api-utils": "^2.1.0",
-    "@angular-eslint/bundled-angular-compiler": "21.0.2-alpha.10",
-    "@angular-eslint/utils": "21.0.2-alpha.10"
+    "@angular-eslint/utils": "21.0.2-alpha.12",
+    "@angular-eslint/bundled-angular-compiler": "21.0.2-alpha.12"
   },
   "devDependencies": {
-    "@angular-eslint/test-utils": "21.0.2-alpha.10"
+    "@angular-eslint/test-utils": "21.0.2-alpha.12"
   },
   "peerDependencies": {
     "@typescript-eslint/utils": "^7.11.0 || ^8.0.0",