eslint-plugin-rxjs-x 0.8.2 → 0.8.4

package/README.md

@@ -21,12 +21,12 @@ but has since introduced new features which involve breaking changes.
     + import rxjsX from 'eslint-plugin-rxjs-x';
     ```
 
-3. If you previously used the `plugin:rxjs/recommended` shared config, add `rxjsX.configs.recommended` to your `configs` block:
+3. If you previously used the `plugin:rxjs/recommended` shared config, add `rxjsX.configs.recommended` to your `extends` block:
 
     ```diff
-    configs: {
+    extends: [
     +   rxjsX.configs.recommended,
-    },
+    ],
     ```
 
     - Note: `eslint-plugin-rxjs-x` provides a `strict` shared config, so consider using `rxjsX.configs.strict` instead.

package/dist/index.d.mts

@@ -159,7 +159,9 @@ declare const allRules: {
         description: "Disallow unsafe `shareReplay` usage.";
         recommended: "recommended";
     }, TSESLint.RuleListener>;
-    'no-sharereplay-before-takeuntil': TSESLint.RuleModule<"forbidden", [], {
+    'no-sharereplay-before-takeuntil': TSESLint.RuleModule<"forbidden", readonly {
+        takeUntilAlias?: string[];
+    }[], {
         description: "Disallow using `shareReplay({ refCount: false })` before `takeUntil`.";
         recommended: "strict";
     }, TSESLint.RuleListener>;
@@ -350,6 +352,7 @@ declare const rxjsX: {
     meta: {
         name: string;
         version: string;
+        namespace: string;
     };
     /** Compatibility with `defineConfig` until https://github.com/typescript-eslint/typescript-eslint/issues/11543 is addressed. */
     rules: { [K in keyof typeof allRules]: (typeof allRules)[K] & Rule.RuleModule; };

package/dist/index.d.ts

@@ -159,7 +159,9 @@ declare const allRules: {
         description: "Disallow unsafe `shareReplay` usage.";
         recommended: "recommended";
     }, TSESLint.RuleListener>;
-    'no-sharereplay-before-takeuntil': TSESLint.RuleModule<"forbidden", [], {
+    'no-sharereplay-before-takeuntil': TSESLint.RuleModule<"forbidden", readonly {
+        takeUntilAlias?: string[];
+    }[], {
         description: "Disallow using `shareReplay({ refCount: false })` before `takeUntil`.";
         recommended: "strict";
     }, TSESLint.RuleListener>;
@@ -350,6 +352,7 @@ declare const rxjsX: {
     meta: {
         name: string;
         version: string;
+        namespace: string;
     };
     /** Compatibility with `defineConfig` until https://github.com/typescript-eslint/typescript-eslint/issues/11543 is addressed. */
     rules: { [K in keyof typeof allRules]: (typeof allRules)[K] & Rule.RuleModule; };

package/dist/index.js

@@ -1,6 +1,6 @@
 "use strict";Object.defineProperty(exports, "__esModule", {value: true}); function _interopRequireWildcard(obj) { if (obj && obj.__esModule) { return obj; } else { var newObj = {}; if (obj != null) { for (var key in obj) { if (Object.prototype.hasOwnProperty.call(obj, key)) { newObj[key] = obj[key]; } } } newObj.default = obj; return newObj; } } function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }// package.json
 var name = "eslint-plugin-rxjs-x";
-var version = "0.8.2";
+var version = "0.8.4";
 
 // src/configs/recommended.ts
 var createRecommendedConfig = (plugin2) => ({
@@ -2839,29 +2839,39 @@ var noSharereplayRule = ruleCreator({
 });
 
 // src/rules/no-sharereplay-before-takeuntil.ts
+var defaultOptions11 = [];
 var noSharereplayBeforeTakeuntilRule = ruleCreator({
-  defaultOptions: [],
+  defaultOptions: defaultOptions11,
   meta: {
     docs: {
       description: "Disallow using `shareReplay({ refCount: false })` before `takeUntil`.",
       recommended: "strict"
     },
     messages: {
-      forbidden: "shareReplay before takeUntil is forbidden unless 'refCount: true' is specified."
+      forbidden: "shareReplay before '{{takeUntilAlias}}' is forbidden unless 'refCount: true' is specified."
     },
-    schema: [],
+    schema: [{
+      properties: {
+        takeUntilAlias: { type: "array", description: "List of operators to treat as takeUntil.", default: ["takeUntilDestroyed"] }
+      },
+      type: "object"
+    }],
     type: "problem"
   },
   name: "no-sharereplay-before-takeuntil",
   create: (context) => {
+    const [config = {}] = context.options;
+    const { takeUntilAlias = ["takeUntilDestroyed"] } = config;
     function checkCallExpression(node) {
       const pipeCallExpression = node.parent;
       if (!pipeCallExpression.arguments || !isMemberExpression(pipeCallExpression.callee) || !isIdentifier(pipeCallExpression.callee.property) || pipeCallExpression.callee.property.name !== "pipe") {
         return;
       }
+      const allTakeUntilAlias = ["takeUntil", ...takeUntilAlias];
+      const takeUntilRegex = new RegExp(`^(${allTakeUntilAlias.join("|")})$`);
       const { isOrderValid, operatorNode: takeUntilNode } = findIsLastOperatorOrderValid(
         pipeCallExpression,
-        /^takeUntil$/,
+        takeUntilRegex,
         DEFAULT_VALID_POST_COMPLETION_OPERATORS
       );
       if (!isOrderValid || !takeUntilNode) {
@@ -2874,6 +2884,7 @@ var noSharereplayBeforeTakeuntilRule = ruleCreator({
       if (!shareReplayConfig || !isObjectExpression(shareReplayConfig)) {
         context.report({
           messageId: "forbidden",
+          data: { takeUntilAlias: isIdentifier(takeUntilNode) ? takeUntilNode.name : "takeUntil" },
           node: node.callee
         });
         return;
@@ -2882,6 +2893,7 @@ var noSharereplayBeforeTakeuntilRule = ruleCreator({
       if (!refCountElement || isLiteral(refCountElement.value) && refCountElement.value.value === false) {
         context.report({
           messageId: "forbidden",
+          data: { takeUntilAlias: isIdentifier(takeUntilNode) ? takeUntilNode.name : "takeUntil" },
           node: node.callee
         });
       }
@@ -3232,9 +3244,9 @@ import { ${functionName} } from ${quote}rxjs${quote};`
 
 // src/rules/no-unbound-methods.ts
 
-var defaultOptions11 = [];
+var defaultOptions12 = [];
 var noUnboundMethodsRule = ruleCreator({
-  defaultOptions: defaultOptions11,
+  defaultOptions: defaultOptions12,
   meta: {
     docs: {
       description: "Disallow passing unbound methods.",
@@ -3397,9 +3409,9 @@ var noUnnecessaryCollectionRule = ruleCreator({
 
 // src/rules/no-unsafe-catch.ts
 
-var defaultOptions12 = [];
+var defaultOptions13 = [];
 var noUnsafeCatchRule = ruleCreator({
-  defaultOptions: defaultOptions12,
+  defaultOptions: defaultOptions13,
   meta: {
     docs: {
       description: "Disallow unsafe `catchError` usage in effects and epics.",
@@ -3458,9 +3470,9 @@ var noUnsafeCatchRule = ruleCreator({
 
 // src/rules/no-unsafe-first.ts
 
-var defaultOptions13 = [];
+var defaultOptions14 = [];
 var noUnsafeFirstRule = ruleCreator({
-  defaultOptions: defaultOptions13,
+  defaultOptions: defaultOptions14,
   meta: {
     docs: {
       description: "Disallow unsafe `first`/`take` usage in effects and epics.",
@@ -3582,9 +3594,9 @@ var noUnsafeSubjectNext = ruleCreator({
 // src/rules/no-unsafe-switchmap.ts
 
 var _decamelize = require('decamelize'); var _decamelize2 = _interopRequireDefault(_decamelize);
-var defaultOptions14 = [];
+var defaultOptions15 = [];
 var noUnsafeSwitchmapRule = ruleCreator({
-  defaultOptions: defaultOptions14,
+  defaultOptions: defaultOptions15,
   meta: {
     docs: {
       description: "Disallow unsafe `switchMap` usage in effects and epics.",
@@ -3707,9 +3719,9 @@ var noUnsafeSwitchmapRule = ruleCreator({
 
 // src/rules/no-unsafe-takeuntil.ts
 
-var defaultOptions15 = [];
+var defaultOptions16 = [];
 var noUnsafeTakeuntilRule = ruleCreator({
-  defaultOptions: defaultOptions15,
+  defaultOptions: defaultOptions16,
   meta: {
     docs: {
       description: "Disallow applying operators after `takeUntil`.",
@@ -3770,9 +3782,9 @@ var noUnsafeTakeuntilRule = ruleCreator({
 });
 
 // src/rules/prefer-observer.ts
-var defaultOptions16 = [];
+var defaultOptions17 = [];
 var preferObserverRule = ruleCreator({
-  defaultOptions: defaultOptions16,
+  defaultOptions: defaultOptions17,
   meta: {
     docs: {
       description: "Disallow passing separate handlers to `subscribe` and `tap`.",
@@ -4008,9 +4020,9 @@ var preferRootOperatorsRule = ruleCreator({
 
 // src/rules/suffix-subjects.ts
 
-var defaultOptions17 = [];
+var defaultOptions18 = [];
 var suffixSubjectsRule = ruleCreator({
-  defaultOptions: defaultOptions17,
+  defaultOptions: defaultOptions18,
   meta: {
     docs: {
       description: "Enforce the use of a suffix in subject identifiers.",
@@ -4207,9 +4219,9 @@ var suffixSubjectsRule = ruleCreator({
 // src/rules/throw-error.ts
 
 
-var defaultOptions18 = [];
+var defaultOptions19 = [];
 var throwErrorRule = ruleCreator({
-  defaultOptions: defaultOptions18,
+  defaultOptions: defaultOptions19,
   meta: {
     docs: {
       description: "Enforce passing only `Error` values to `throwError` or `Subject.error`.",
@@ -4339,7 +4351,11 @@ var allRules = {
   "throw-error": throwErrorRule
 };
 var plugin = {
-  meta: { name, version },
+  meta: {
+    name,
+    version,
+    namespace: "rxjs-x"
+  },
   /** Compatibility with `defineConfig` until https://github.com/typescript-eslint/typescript-eslint/issues/11543 is addressed. */
   rules: allRules
 };

package/dist/index.mjs

@@ -1,6 +1,6 @@
 // package.json
 var name = "eslint-plugin-rxjs-x";
-var version = "0.8.2";
+var version = "0.8.4";
 
 // src/configs/recommended.ts
 var createRecommendedConfig = (plugin2) => ({
@@ -2839,29 +2839,39 @@ var noSharereplayRule = ruleCreator({
 });
 
 // src/rules/no-sharereplay-before-takeuntil.ts
+var defaultOptions11 = [];
 var noSharereplayBeforeTakeuntilRule = ruleCreator({
-  defaultOptions: [],
+  defaultOptions: defaultOptions11,
   meta: {
     docs: {
       description: "Disallow using `shareReplay({ refCount: false })` before `takeUntil`.",
       recommended: "strict"
     },
     messages: {
-      forbidden: "shareReplay before takeUntil is forbidden unless 'refCount: true' is specified."
+      forbidden: "shareReplay before '{{takeUntilAlias}}' is forbidden unless 'refCount: true' is specified."
     },
-    schema: [],
+    schema: [{
+      properties: {
+        takeUntilAlias: { type: "array", description: "List of operators to treat as takeUntil.", default: ["takeUntilDestroyed"] }
+      },
+      type: "object"
+    }],
     type: "problem"
   },
   name: "no-sharereplay-before-takeuntil",
   create: (context) => {
+    const [config = {}] = context.options;
+    const { takeUntilAlias = ["takeUntilDestroyed"] } = config;
     function checkCallExpression(node) {
       const pipeCallExpression = node.parent;
       if (!pipeCallExpression.arguments || !isMemberExpression(pipeCallExpression.callee) || !isIdentifier(pipeCallExpression.callee.property) || pipeCallExpression.callee.property.name !== "pipe") {
         return;
       }
+      const allTakeUntilAlias = ["takeUntil", ...takeUntilAlias];
+      const takeUntilRegex = new RegExp(`^(${allTakeUntilAlias.join("|")})$`);
       const { isOrderValid, operatorNode: takeUntilNode } = findIsLastOperatorOrderValid(
         pipeCallExpression,
-        /^takeUntil$/,
+        takeUntilRegex,
         DEFAULT_VALID_POST_COMPLETION_OPERATORS
       );
       if (!isOrderValid || !takeUntilNode) {
@@ -2874,6 +2884,7 @@ var noSharereplayBeforeTakeuntilRule = ruleCreator({
       if (!shareReplayConfig || !isObjectExpression(shareReplayConfig)) {
         context.report({
           messageId: "forbidden",
+          data: { takeUntilAlias: isIdentifier(takeUntilNode) ? takeUntilNode.name : "takeUntil" },
           node: node.callee
         });
         return;
@@ -2882,6 +2893,7 @@ var noSharereplayBeforeTakeuntilRule = ruleCreator({
       if (!refCountElement || isLiteral(refCountElement.value) && refCountElement.value.value === false) {
         context.report({
           messageId: "forbidden",
+          data: { takeUntilAlias: isIdentifier(takeUntilNode) ? takeUntilNode.name : "takeUntil" },
           node: node.callee
         });
       }
@@ -3232,9 +3244,9 @@ import { ${functionName} } from ${quote}rxjs${quote};`
 
 // src/rules/no-unbound-methods.ts
 import { ESLintUtils as ESLintUtils9 } from "@typescript-eslint/utils";
-var defaultOptions11 = [];
+var defaultOptions12 = [];
 var noUnboundMethodsRule = ruleCreator({
-  defaultOptions: defaultOptions11,
+  defaultOptions: defaultOptions12,
   meta: {
     docs: {
       description: "Disallow passing unbound methods.",
@@ -3397,9 +3409,9 @@ var noUnnecessaryCollectionRule = ruleCreator({
 
 // src/rules/no-unsafe-catch.ts
 import { stripIndent as stripIndent5 } from "common-tags";
-var defaultOptions12 = [];
+var defaultOptions13 = [];
 var noUnsafeCatchRule = ruleCreator({
-  defaultOptions: defaultOptions12,
+  defaultOptions: defaultOptions13,
   meta: {
     docs: {
       description: "Disallow unsafe `catchError` usage in effects and epics.",
@@ -3458,9 +3470,9 @@ var noUnsafeCatchRule = ruleCreator({
 
 // src/rules/no-unsafe-first.ts
 import { stripIndent as stripIndent6 } from "common-tags";
-var defaultOptions13 = [];
+var defaultOptions14 = [];
 var noUnsafeFirstRule = ruleCreator({
-  defaultOptions: defaultOptions13,
+  defaultOptions: defaultOptions14,
   meta: {
     docs: {
       description: "Disallow unsafe `first`/`take` usage in effects and epics.",
@@ -3582,9 +3594,9 @@ var noUnsafeSubjectNext = ruleCreator({
 // src/rules/no-unsafe-switchmap.ts
 import { stripIndent as stripIndent7 } from "common-tags";
 import decamelize from "decamelize";
-var defaultOptions14 = [];
+var defaultOptions15 = [];
 var noUnsafeSwitchmapRule = ruleCreator({
-  defaultOptions: defaultOptions14,
+  defaultOptions: defaultOptions15,
   meta: {
     docs: {
       description: "Disallow unsafe `switchMap` usage in effects and epics.",
@@ -3707,9 +3719,9 @@ var noUnsafeSwitchmapRule = ruleCreator({
 
 // src/rules/no-unsafe-takeuntil.ts
 import { stripIndent as stripIndent8 } from "common-tags";
-var defaultOptions15 = [];
+var defaultOptions16 = [];
 var noUnsafeTakeuntilRule = ruleCreator({
-  defaultOptions: defaultOptions15,
+  defaultOptions: defaultOptions16,
   meta: {
     docs: {
       description: "Disallow applying operators after `takeUntil`.",
@@ -3770,9 +3782,9 @@ var noUnsafeTakeuntilRule = ruleCreator({
 });
 
 // src/rules/prefer-observer.ts
-var defaultOptions16 = [];
+var defaultOptions17 = [];
 var preferObserverRule = ruleCreator({
-  defaultOptions: defaultOptions16,
+  defaultOptions: defaultOptions17,
   meta: {
     docs: {
       description: "Disallow passing separate handlers to `subscribe` and `tap`.",
@@ -4008,9 +4020,9 @@ var preferRootOperatorsRule = ruleCreator({
 
 // src/rules/suffix-subjects.ts
 import { AST_NODE_TYPES as AST_NODE_TYPES9, ESLintUtils as ESLintUtils11 } from "@typescript-eslint/utils";
-var defaultOptions17 = [];
+var defaultOptions18 = [];
 var suffixSubjectsRule = ruleCreator({
-  defaultOptions: defaultOptions17,
+  defaultOptions: defaultOptions18,
   meta: {
     docs: {
       description: "Enforce the use of a suffix in subject identifiers.",
@@ -4207,9 +4219,9 @@ var suffixSubjectsRule = ruleCreator({
 // src/rules/throw-error.ts
 import { ESLintUtils as ESLintUtils12 } from "@typescript-eslint/utils";
 import * as tsutils4 from "ts-api-utils";
-var defaultOptions18 = [];
+var defaultOptions19 = [];
 var throwErrorRule = ruleCreator({
-  defaultOptions: defaultOptions18,
+  defaultOptions: defaultOptions19,
   meta: {
     docs: {
       description: "Enforce passing only `Error` values to `throwError` or `Subject.error`.",
@@ -4339,7 +4351,11 @@ var allRules = {
   "throw-error": throwErrorRule
 };
 var plugin = {
-  meta: { name, version },
+  meta: {
+    name,
+    version,
+    namespace: "rxjs-x"
+  },
   /** Compatibility with `defineConfig` until https://github.com/typescript-eslint/typescript-eslint/issues/11543 is addressed. */
   rules: allRules
 };

package/package.json

@@ -1,9 +1,9 @@
 {
   "name": "eslint-plugin-rxjs-x",
   "type": "commonjs",
-  "version": "0.8.2",
-  "packageManager": "yarn@4.10.3+sha512.c38cafb5c7bb273f3926d04e55e1d8c9dfa7d9c3ea1f36a4868fa028b9e5f72298f0b7f401ad5eb921749eb012eb1c3bb74bf7503df3ee43fd600d14a018266f",
-  "description": "ESLint v9+ plugin for RxJS",
+  "version": "0.8.4",
+  "packageManager": "yarn@4.12.0+sha512.f45ab632439a67f8bc759bf32ead036a1f413287b9042726b7cc4818b7b49e14e9423ba49b18f9e06ea4941c1ad062385b1d8760a8d5091a1a31e5f6219afca8",
+  "description": "Modern ESLint plugin for RxJS",
   "author": "Jason Weinzierl <weinzierljason@gmail.com>",
   "license": "MIT",
   "homepage": "https://github.com/JasonWeinzierl/eslint-plugin-rxjs-x",
@@ -11,6 +11,10 @@
     "type": "git",
     "url": "git+https://github.com/JasonWeinzierl/eslint-plugin-rxjs-x.git"
   },
+  "publishConfig": {
+    "provenance": true,
+    "registry": "https://registry.npmjs.org"
+  },
   "bugs": {
     "url": "https://github.com/JasonWeinzierl/eslint-plugin-rxjs-x/issues"
   },
@@ -72,29 +76,28 @@
     }
   },
   "devDependencies": {
-    "@eslint/config-helpers": "0.4.1",
-    "@eslint/js": "^9.38.0",
-    "@stylistic/eslint-plugin": "^5.5.0",
+    "@eslint/js": "^9.39.2",
+    "@stylistic/eslint-plugin": "^5.6.1",
     "@types/common-tags": "^1.8.4",
     "@types/node": "~18.18.0",
-    "@typescript-eslint/rule-tester": "^8.46.2",
+    "@typescript-eslint/rule-tester": "^8.51.0",
     "@typescript/vfs": "^1.6.2",
     "@vitest/coverage-v8": "^3.2.4",
     "@vitest/eslint-plugin": "^1.4.0",
-    "bumpp": "^10.3.1",
-    "eslint": "^9.38.0",
+    "bumpp": "^10.3.2",
+    "eslint": "^9.39.2",
     "eslint-config-flat-gitignore": "^2.1.0",
-    "eslint-doc-generator": "~2.2.2",
+    "eslint-doc-generator": "^2.4.0",
     "eslint-import-resolver-typescript": "^4.4.4",
     "eslint-plugin-eslint-plugin": "^7.2.0",
     "eslint-plugin-import-x": "^4.16.1",
     "eslint-plugin-n": "^17.23.1",
-    "markdownlint-cli2": "^0.18.1",
+    "markdownlint-cli2": "~0.19.0",
     "rxjs": "^7.8.2",
-    "tsup": "^8.5.0",
+    "tsup": "8.5.0",
     "typescript": "~5.8.3",
-    "typescript-eslint": "^8.46.2",
-    "vite": "^6.3.5",
+    "typescript-eslint": "^8.51.0",
+    "vite": "^6.4.1",
     "vitest": "^3.2.4"
   },
   "engines": {