eslint-plugin-rxjs-x 0.2.2 → 0.2.3

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.2.2";
+var version = "0.2.3";
 
 // src/configs/recommended.ts
 var createRecommendedConfig = (plugin2) => ({
@@ -1412,6 +1412,7 @@ var noIgnoredObservableRule = ruleCreator({
 });
 
 // src/rules/no-ignored-replay-buffer.ts
+
 var noIgnoredReplayBufferRule = ruleCreator({
   defaultOptions: [],
   meta: {
@@ -1427,6 +1428,14 @@ var noIgnoredReplayBufferRule = ruleCreator({
   },
   name: "no-ignored-replay-buffer",
   create: (context) => {
+    function checkShareReplayConfig(node, shareReplayConfigArg) {
+      if (!shareReplayConfigArg.properties.some((p) => p.type === _utils.AST_NODE_TYPES.Property && p.key.type === _utils.AST_NODE_TYPES.Identifier && p.key.name === "bufferSize")) {
+        context.report({
+          messageId: "forbidden",
+          node
+        });
+      }
+    }
     function checkNode(node, { arguments: args }) {
       if (!args || args.length === 0) {
         context.report({
@@ -1434,6 +1443,12 @@ var noIgnoredReplayBufferRule = ruleCreator({
           node
         });
       }
+      if (node.name === "shareReplay" && (args == null ? void 0 : args.length) === 1) {
+        const arg = args[0];
+        if (arg.type === _utils.AST_NODE_TYPES.ObjectExpression) {
+          checkShareReplayConfig(node, arg);
+        }
+      }
     }
     return {
       "NewExpression > Identifier[name='ReplaySubject']": (node) => {
@@ -1448,6 +1463,11 @@ var noIgnoredReplayBufferRule = ruleCreator({
       "CallExpression > Identifier[name=/^(publishReplay|shareReplay)$/]": (node) => {
         const callExpression = node.parent;
         checkNode(node, callExpression);
+      },
+      "CallExpression > MemberExpression > Identifier[name=/^(publishReplay|shareReplay)$/]": (node) => {
+        const memberExpression = node.parent;
+        const callExpression = memberExpression.parent;
+        checkNode(node, callExpression);
       }
     };
   }

package/dist/index.mjs

@@ -1,6 +1,6 @@
 // package.json
 var name = "eslint-plugin-rxjs-x";
-var version = "0.2.2";
+var version = "0.2.3";
 
 // src/configs/recommended.ts
 var createRecommendedConfig = (plugin2) => ({
@@ -1412,6 +1412,7 @@ var noIgnoredObservableRule = ruleCreator({
 });
 
 // src/rules/no-ignored-replay-buffer.ts
+import { AST_NODE_TYPES as AST_NODE_TYPES6 } from "@typescript-eslint/utils";
 var noIgnoredReplayBufferRule = ruleCreator({
   defaultOptions: [],
   meta: {
@@ -1427,6 +1428,14 @@ var noIgnoredReplayBufferRule = ruleCreator({
   },
   name: "no-ignored-replay-buffer",
   create: (context) => {
+    function checkShareReplayConfig(node, shareReplayConfigArg) {
+      if (!shareReplayConfigArg.properties.some((p) => p.type === AST_NODE_TYPES6.Property && p.key.type === AST_NODE_TYPES6.Identifier && p.key.name === "bufferSize")) {
+        context.report({
+          messageId: "forbidden",
+          node
+        });
+      }
+    }
     function checkNode(node, { arguments: args }) {
       if (!args || args.length === 0) {
         context.report({
@@ -1434,6 +1443,12 @@ var noIgnoredReplayBufferRule = ruleCreator({
           node
         });
       }
+      if (node.name === "shareReplay" && (args == null ? void 0 : args.length) === 1) {
+        const arg = args[0];
+        if (arg.type === AST_NODE_TYPES6.ObjectExpression) {
+          checkShareReplayConfig(node, arg);
+        }
+      }
     }
     return {
       "NewExpression > Identifier[name='ReplaySubject']": (node) => {
@@ -1448,6 +1463,11 @@ var noIgnoredReplayBufferRule = ruleCreator({
       "CallExpression > Identifier[name=/^(publishReplay|shareReplay)$/]": (node) => {
         const callExpression = node.parent;
         checkNode(node, callExpression);
+      },
+      "CallExpression > MemberExpression > Identifier[name=/^(publishReplay|shareReplay)$/]": (node) => {
+        const memberExpression = node.parent;
+        const callExpression = memberExpression.parent;
+        checkNode(node, callExpression);
       }
     };
   }
@@ -1575,7 +1595,7 @@ var noIgnoredTakewhileValueRule = ruleCreator({
 
 // src/rules/no-implicit-any-catch.ts
 import {
-  AST_NODE_TYPES as AST_NODE_TYPES6
+  AST_NODE_TYPES as AST_NODE_TYPES7
 } from "@typescript-eslint/utils";
 function isParenthesised(sourceCode, node) {
   const before = sourceCode.getTokenBefore(node);
@@ -1631,7 +1651,7 @@ var noImplicitAnyCatchRule = ruleCreator({
           const {
             typeAnnotation: { type }
           } = typeAnnotation;
-          if (type === AST_NODE_TYPES6.TSAnyKeyword) {
+          if (type === AST_NODE_TYPES7.TSAnyKeyword) {
             let fix2 = function(fixer) {
               return fixer.replaceText(typeAnnotation, ": unknown");
             };
@@ -1650,7 +1670,7 @@ var noImplicitAnyCatchRule = ruleCreator({
                 }
               ]
             });
-          } else if (type !== AST_NODE_TYPES6.TSUnknownKeyword) {
+          } else if (type !== AST_NODE_TYPES7.TSUnknownKeyword) {
             let fix2 = function(fixer) {
               return fixer.replaceText(typeAnnotation, ": unknown");
             };
@@ -1945,7 +1965,7 @@ function isExpressionObserver(expressionStatement, couldBeType2) {
 }
 
 // src/rules/no-sharereplay.ts
-import { AST_NODE_TYPES as AST_NODE_TYPES7 } from "@typescript-eslint/utils";
+import { AST_NODE_TYPES as AST_NODE_TYPES8 } from "@typescript-eslint/utils";
 var defaultOptions7 = [];
 var noSharereplayRule = ruleCreator({
   defaultOptions: defaultOptions7,
@@ -1976,7 +1996,7 @@ var noSharereplayRule = ruleCreator({
       "CallExpression[callee.name='shareReplay']": (node) => {
         let report = true;
         if (allowConfig) {
-          report = node.arguments.length !== 1 || node.arguments[0].type !== AST_NODE_TYPES7.ObjectExpression;
+          report = node.arguments.length !== 1 || node.arguments[0].type !== AST_NODE_TYPES8.ObjectExpression;
         }
         if (report) {
           context.report({
@@ -2780,7 +2800,7 @@ function isValidArgText(argText) {
 }
 
 // src/rules/suffix-subjects.ts
-import { AST_NODE_TYPES as AST_NODE_TYPES8, ESLintUtils as ESLintUtils5 } from "@typescript-eslint/utils";
+import { AST_NODE_TYPES as AST_NODE_TYPES9, ESLintUtils as ESLintUtils5 } from "@typescript-eslint/utils";
 var defaultOptions13 = [];
 var suffixSubjectsRule = ruleCreator({
   defaultOptions: defaultOptions13,
@@ -2864,7 +2884,7 @@ var suffixSubjectsRule = ruleCreator({
         if (!found) {
           return;
         }
-        if (!validate.variables && found.type === AST_NODE_TYPES8.VariableDeclarator) {
+        if (!validate.variables && found.type === AST_NODE_TYPES9.VariableDeclarator) {
           return;
         }
         if (!validate.parameters) {
@@ -2931,7 +2951,7 @@ var suffixSubjectsRule = ruleCreator({
         if (!found) {
           return;
         }
-        if (!validate.variables && found.type === AST_NODE_TYPES8.VariableDeclarator) {
+        if (!validate.variables && found.type === AST_NODE_TYPES9.VariableDeclarator) {
           return;
         }
         if (!validate.parameters) {

package/docs/rules/no-ignored-replay-buffer.md

@@ -15,6 +15,11 @@ import { ReplaySubject } from "rxjs";
 const subject = new ReplaySubject<number>();
 ```
 
+```ts
+import { of, shareReplay } from "rxjs";
+of(42).pipe(shareReplay({ refCount: true }));
+```
+
 Examples of **correct** code for this rule:
 
 ```ts
@@ -26,3 +31,8 @@ const subject = new ReplaySubject<number>(1);
 import { ReplaySubject } from "rxjs";
 const subject = new ReplaySubject<number>(Infinity);
 ```
+
+```ts
+import { of, shareReplay } from "rxjs";
+of(42).pipe(shareReplay({ refCount: true, bufferSize: 1 }));
+```

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "eslint-plugin-rxjs-x",
   "type": "commonjs",
-  "version": "0.2.2",
+  "version": "0.2.3",
   "packageManager": "yarn@4.5.1+sha512.341db9396b6e289fecc30cd7ab3af65060e05ebff4b3b47547b278b9e67b08f485ecd8c79006b405446262142c7a38154445ef7f17c1d5d1de7d90bf9ce7054d",
   "description": "ESLint v9+ plugin for RxJS",
   "author": "Jason Weinzierl <weinzierljason@gmail.com>",
@@ -65,11 +65,16 @@
     "rxjs": ">=7.0.0",
     "typescript": ">=4.7.4"
   },
+  "peerDependenciesMeta": {
+    "rxjs": {
+      "optional": true
+    }
+  },
   "devDependencies": {
     "@eslint/js": "^9.14.0",
     "@stylistic/eslint-plugin": "^2.10.1",
     "@types/common-tags": "^1.8.4",
-    "@types/node": "^18.18.0",
+    "@types/node": "~18.18.0",
     "@typescript-eslint/rule-tester": "^8.13.0",
     "@typescript/vfs": "^1.6.0",
     "@vitest/coverage-v8": "^2.1.4",
@@ -83,7 +88,7 @@
     "eslint-plugin-import-x": "^4.4.0",
     "eslint-plugin-n": "^17.13.1",
     "markdownlint-cli2": "^0.14.0",
-    "rxjs": "^7.0.0",
+    "rxjs": "^7.8.1",
     "tsup": "^8.3.5",
     "tsx": "^4.19.2",
     "typescript": "~5.6.3",