@depup/angular-eslint__eslint-plugin 22.2.0-depup.0 → 22.3.0-depup.0

package/dist/rules/require-localize-metadata.js

@@ -44,8 +44,8 @@ exports.default = (0, create_eslint_rule_1.createESLintRule)({
             requireLocalizeMeaning: `$localize tagged messages should contain a meaning. See more at ${STYLE_GUIDE_LINK_METADATA_FOR_TRANSLATION}`,
             requireLocalizeCustomId: `$localize tagged messages should contain a custom id{{patternMessage}}. See more at ${STYLE_GUIDE_LINK_METADATA_FOR_TRANSLATION}`,
         },
+        defaultOptions: [DEFAULT_OPTIONS],
     },
-    defaultOptions: [DEFAULT_OPTIONS],
     create(context, [{ requireDescription, requireMeaning, requireCustomId }]) {
         return {
             TaggedTemplateExpression(taggedTemplateExpression) {

package/dist/rules/runtime-localize.js

@@ -15,8 +15,8 @@ exports.default = (0, create_eslint_rule_1.createESLintRule)({
         messages: {
             runtimeLocalize: `$localize could be called before translations are loaded`,
         },
+        defaultOptions: [],
     },
-    defaultOptions: [],
     create(context) {
         return {
             TaggedTemplateExpression(taggedTemplateExpression) {

package/dist/rules/sort-keys-in-type-decorator.js

@@ -103,8 +103,8 @@ exports.default = (0, create_eslint_rule_1.createESLintRule)({
         messages: {
             incorrectOrder: 'Keys in @{{decorator}} decorator should be ordered: {{expectedOrder}}',
         },
+        defaultOptions: [DEFAULT_ORDER],
     },
-    defaultOptions: [DEFAULT_ORDER],
     create(context, [orderConfig]) {
         function checkContext(node, decoratorName) {
             const expectedOrder = orderConfig[decoratorName];

package/dist/rules/sort-lifecycle-methods.js

@@ -15,8 +15,8 @@ exports.default = (0, create_eslint_rule_1.createESLintRule)({
         messages: {
             lifecycleMethodsNotSorted: `Lifecycle Methods are not declared in order of execution`,
         },
+        defaultOptions: [],
     },
-    defaultOptions: [],
     create(context) {
         const isBefore = (method1, method2) => {
             const methodIndex1 = utils_1.ASTUtils.angularLifecycleMethodsOrdered.indexOf(utils_1.ASTUtils.getMethodName(method1));

package/dist/rules/use-component-selector.js

@@ -16,8 +16,8 @@ exports.default = (0, create_eslint_rule_1.createESLintRule)({
         messages: {
             useComponentSelector: 'The selector of the component is mandatory',
         },
+        defaultOptions: [],
     },
-    defaultOptions: [],
     create(context) {
         return {
             [utils_1.Selectors.COMPONENT_CLASS_DECORATOR](node) {

package/dist/rules/use-component-view-encapsulation.js

@@ -18,8 +18,8 @@ exports.default = (0, create_eslint_rule_1.createESLintRule)({
             useComponentViewEncapsulation: `Using \`${VIEW_ENCAPSULATION_NONE}\` makes your styles global, which may have an unintended effect`,
             suggestRemoveViewEncapsulationNone: `Remove \`${VIEW_ENCAPSULATION_NONE}\``,
         },
+        defaultOptions: [],
     },
-    defaultOptions: [],
     create(context) {
         const sourceCode = context.sourceCode;
         return {

package/dist/rules/use-injectable-provided-in.d.ts

@@ -1,6 +1,9 @@
-export type Options = [{
-    readonly ignoreClassNamePattern?: string;
-}];
+export type Options = [
+    {
+        readonly ignoreClassNamePattern?: string;
+        readonly allowProvidedInNull?: boolean;
+    }
+];
 export type MessageIds = 'useInjectableProvidedIn' | 'suggestInjector';
 export declare const RULE_NAME = "use-injectable-provided-in";
 declare const _default: import("@typescript-eslint/utils/ts-eslint").RuleModule<MessageIds, Options, import("../utils/create-eslint-rule").RuleDocs, import("@typescript-eslint/utils/ts-eslint").RuleListener> & {

package/dist/rules/use-injectable-provided-in.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"use-injectable-provided-in.d.ts","sourceRoot":"","sources":["../../src/rules/use-injectable-provided-in.ts"],"names":[],"mappings":"AAIA,MAAM,MAAM,OAAO,GAAG,CAAC;IAAE,QAAQ,CAAC,sBAAsB,CAAC,EAAE,MAAM,CAAA;CAAE,CAAC,CAAC;AACrE,MAAM,MAAM,UAAU,GAAG,yBAAyB,GAAG,iBAAiB,CAAC;AACvE,eAAO,MAAM,SAAS,+BAA+B,CAAC;;;;AAGtD,wBA2DG;AAEH,eAAO,MAAM,mBAAmB;;CAE/B,CAAC"}
+{"version":3,"file":"use-injectable-provided-in.d.ts","sourceRoot":"","sources":["../../src/rules/use-injectable-provided-in.ts"],"names":[],"mappings":"AAIA,MAAM,MAAM,OAAO,GAAG;IACpB;QACE,QAAQ,CAAC,sBAAsB,CAAC,EAAE,MAAM,CAAC;QACzC,QAAQ,CAAC,mBAAmB,CAAC,EAAE,OAAO,CAAC;KACxC;CACF,CAAC;AACF,MAAM,MAAM,UAAU,GAAG,yBAAyB,GAAG,iBAAiB,CAAC;AACvE,eAAO,MAAM,SAAS,+BAA+B,CAAC;;;;AAMtD,wBAiEG;AAEH,eAAO,MAAM,mBAAmB;;CAE/B,CAAC"}

package/dist/rules/use-injectable-provided-in.js

@@ -5,6 +5,9 @@ const utils_1 = require("@angular-eslint/utils");
 const create_eslint_rule_1 = require("../utils/create-eslint-rule");
 exports.RULE_NAME = 'use-injectable-provided-in';
 const METADATA_PROPERTY_NAME = 'providedIn';
+const DEFAULT_OPTIONS = {
+    allowProvidedInNull: false,
+};
 exports.default = (0, create_eslint_rule_1.createESLintRule)({
     name: exports.RULE_NAME,
     meta: {
@@ -20,6 +23,10 @@ exports.default = (0, create_eslint_rule_1.createESLintRule)({
                     ignoreClassNamePattern: {
                         type: 'string',
                     },
+                    allowProvidedInNull: {
+                        type: 'boolean',
+                        default: DEFAULT_OPTIONS.allowProvidedInNull,
+                    },
                 },
                 additionalProperties: false,
             },
@@ -28,16 +35,18 @@ exports.default = (0, create_eslint_rule_1.createESLintRule)({
             useInjectableProvidedIn: `The \`${METADATA_PROPERTY_NAME}\` property is mandatory for \`Injectables\``,
             suggestInjector: `Use \`${METADATA_PROPERTY_NAME}: '{{injector}}'\``,
         },
+        defaultOptions: [DEFAULT_OPTIONS],
     },
-    defaultOptions: [{}],
-    create(context, [{ ignoreClassNamePattern }]) {
+    create(context, [{ ignoreClassNamePattern, allowProvidedInNull }]) {
         const injectableClassDecorator = `ClassDeclaration:not([id.name=${ignoreClassNamePattern}]):not(:has(TSClassImplements:matches([expression.property.name='HttpInterceptor'], [expression.name='HttpInterceptor']))) > Decorator[expression.callee.name="Injectable"]`;
         const providedInMetadataProperty = utils_1.Selectors.metadataProperty(METADATA_PROPERTY_NAME);
         const withoutProvidedInDecorator = `${injectableClassDecorator}:matches([expression.arguments.length=0], [expression.arguments.0.type='ObjectExpression']:not(:has(${providedInMetadataProperty})))`;
-        const nullableProvidedInProperty = `${injectableClassDecorator} ${providedInMetadataProperty}:matches([value.type='Identifier'][value.name='undefined'], [value.type='Literal'][value.raw='null'])`;
+        const undefinedProvidedInProperty = `${injectableClassDecorator} ${providedInMetadataProperty}[value.type='Identifier'][value.name='undefined']`;
+        const nullProvidedInProperty = `${injectableClassDecorator} ${providedInMetadataProperty}[value.type='Literal'][value.raw='null']`;
         const selectors = [
             withoutProvidedInDecorator,
-            nullableProvidedInProperty,
+            undefinedProvidedInProperty,
+            ...(allowProvidedInNull ? [] : [nullProvidedInProperty]),
         ].join(',');
         return {
             [selectors](node) {
@@ -59,5 +68,5 @@ exports.default = (0, create_eslint_rule_1.createESLintRule)({
     },
 });
 exports.RULE_DOCS_EXTENSION = {
-    rationale: `Using 'providedIn' in the @Injectable() decorator (like '@Injectable({ providedIn: "root" })') enables tree-shaking, which means Angular can remove the service from your production bundle if it's never actually used. Without 'providedIn', services must be registered in a module's providers array, and Angular must include them in the bundle even if they're unused. This can significantly increase bundle size. Additionally, 'providedIn' makes services easier to use (no need to add them to module providers) and makes circular dependencies less likely. The 'providedIn' syntax is the modern, recommended way to provide services.`,
+    rationale: `Using 'providedIn' in the @Injectable() decorator (like '@Injectable({ providedIn: "root" })') enables tree-shaking, which means Angular can remove the service from your production bundle if it's never actually used. Without 'providedIn', services must be registered in a module's providers array, and Angular must include them in the bundle even if they're unused. This can significantly increase bundle size. Additionally, 'providedIn' makes services easier to use (no need to add them to module providers) and makes circular dependencies less likely. The 'providedIn' syntax is the modern, recommended way to provide services. Angular treats 'providedIn: null' as equivalent to omitting the property, so it is flagged by default. Set 'allowProvidedInNull' to true if you need to permit this pattern.`,
 };

package/dist/rules/use-lifecycle-interface.js

@@ -17,8 +17,8 @@ exports.default = (0, create_eslint_rule_1.createESLintRule)({
             useLifecycleInterface: `Lifecycle interface '{{interfaceName}}' should be implemented for method '{{methodName}}'. (${STYLE_GUIDE_LINK})`,
         },
         fixable: 'code',
+        defaultOptions: [],
     },
-    defaultOptions: [],
     create(context) {
         const angularLifecycleMethodsPattern = (0, utils_1.toPattern)([
             ...utils_1.ASTUtils.ANGULAR_LIFECYCLE_METHODS,

package/dist/rules/use-pipe-transform-interface.js

@@ -18,8 +18,8 @@ exports.default = (0, create_eslint_rule_1.createESLintRule)({
         messages: {
             usePipeTransformInterface: `Pipes should implement \`${PIPE_TRANSFORM}\` interface`,
         },
+        defaultOptions: [],
     },
-    defaultOptions: [],
     create(context) {
         return {
             [`ClassDeclaration:not(:has(TSClassImplements:matches([expression.name='${PIPE_TRANSFORM}'], [expression.property.name='${PIPE_TRANSFORM}']))) > Decorator[expression.callee.name='Pipe']`]({ parent: classDeclaration, }) {

package/dist/utils/create-eslint-rule.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"create-eslint-rule.d.ts","sourceRoot":"","sources":["../../src/utils/create-eslint-rule.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,WAAW,EAAE,MAAM,0BAA0B,CAAC;AAEvD,MAAM,WAAW,QAAQ;IACvB,WAAW,CAAC,EAAE,MAAM,CAAC;CACtB;AAiCD,eAAO,MAAM,gBAAgB;;CAG5B,CAAC"}
+{"version":3,"file":"create-eslint-rule.d.ts","sourceRoot":"","sources":["../../src/utils/create-eslint-rule.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,WAAW,EAAE,MAAM,0BAA0B,CAAC;AAEvD,MAAM,WAAW,QAAQ;IACvB,WAAW,CAAC,EAAE,MAAM,CAAC;CACtB;AAED,eAAO,MAAM,gBAAgB;;CAG5B,CAAC"}

package/dist/utils/create-eslint-rule.js

@@ -2,27 +2,4 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.createESLintRule = void 0;
 const utils_1 = require("@typescript-eslint/utils");
-/**
- * We need to patch the RuleCreator in order to preserve the defaultOptions
- * to use as part of documentation generation.
- */
-const patchedRuleCreator = (urlCreator) => {
-    return function createRule({ name, meta, defaultOptions, create }) {
-        const resolvedDefaultOptions = (defaultOptions ?? []);
-        return {
-            name,
-            meta: Object.assign(Object.assign({}, meta), {
-                docs: Object.assign(Object.assign({}, meta.docs), {
-                    url: urlCreator(name),
-                }),
-            }),
-            defaultOptions: resolvedDefaultOptions,
-            create(context) {
-                const optionsWithDefault = utils_1.ESLintUtils.applyDefault(resolvedDefaultOptions, context.options);
-                return create(context, optionsWithDefault);
-            },
-        };
-    };
-};
-patchedRuleCreator.withoutDocs = utils_1.ESLintUtils.RuleCreator.withoutDocs;
-exports.createESLintRule = patchedRuleCreator((ruleName) => `https://github.com/angular-eslint/angular-eslint/blob/main/packages/eslint-plugin/docs/rules/${ruleName}.md`);
+exports.createESLintRule = utils_1.ESLintUtils.RuleCreator((ruleName) => `https://github.com/angular-eslint/angular-eslint/blob/main/packages/eslint-plugin/docs/rules/${ruleName}.md`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@depup/angular-eslint__eslint-plugin",
-  "version": "22.2.0-depup.0",
+  "version": "22.3.0-depup.0",
   "description": "ESLint plugin for Angular applications, following https://angular.dev/style-guide (with updated dependencies)",
   "license": "MIT",
   "main": "dist/index.js",
@@ -20,12 +20,12 @@
     "changes.json"
   ],
   "dependencies": {
-    "ts-api-utils": "^2.5.0",
-    "@angular-eslint/bundled-angular-compiler": "22.2.0",
-    "@angular-eslint/utils": "22.2.0"
+    "@angular-eslint/bundled-angular-compiler": "22.3.0",
+    "@angular-eslint/utils": "22.3.0",
+    "ts-api-utils": "^2.5.0"
   },
   "devDependencies": {
-    "@angular-eslint/test-utils": "22.2.0"
+    "@angular-eslint/test-utils": "22.3.0"
   },
   "peerDependencies": {
     "@typescript-eslint/utils": "^8.0.0",
@@ -50,8 +50,8 @@
     },
     "depsUpdated": 1,
     "originalPackage": "@angular-eslint/eslint-plugin",
-    "originalVersion": "22.2.0",
-    "processedAt": "2026-08-30T16:08:13.663Z",
+    "originalVersion": "22.3.0",
+    "processedAt": "2026-09-07T08:13:08.078Z",
     "smokeTest": "passed"
   }
 }