eslint-plugin-rxjs-x 0.7.3 → 0.7.5

package/README.md

@@ -120,6 +120,6 @@ The package includes the following rules.
 | [prefer-observer](docs/rules/prefer-observer.md)                       | Disallow passing separate handlers to `subscribe` and `tap`.                                              | ✅ 🔒 | 🔧 | 💡 | 💭 |    |
 | [prefer-root-operators](docs/rules/prefer-root-operators.md)           | Disallow importing operators from `rxjs/operators`.                                                       | ✅ 🔒 | 🔧 | 💡 |    |    |
 | [suffix-subjects](docs/rules/suffix-subjects.md)                       | Enforce the use of a suffix in subject identifiers.                                                       |      |    |    | 💭 |    |
-| [throw-error](docs/rules/throw-error.md)                               | Enforce passing only `Error` values to `throwError`.                                                      | ✅ 🔒 |    |    | 💭 |    |
+| [throw-error](docs/rules/throw-error.md)                               | Enforce passing only `Error` values to `throwError` or `Subject.error`.                                   | ✅ 🔒 |    |    | 💭 |    |
 
 <!-- end auto-generated rules list -->

package/dist/index.d.mts

@@ -150,7 +150,9 @@ declare const rxjsX: {
         'no-subscribe-in-pipe': TSESLint.RuleModule<"forbidden", [], RxjsXRuleDocs, TSESLint.RuleListener>;
         'no-tap': TSESLint.RuleModule<"forbidden", [], RxjsXRuleDocs, TSESLint.RuleListener>;
         'no-topromise': TSESLint.RuleModule<"forbidden" | "suggestLastValueFromWithDefault" | "suggestLastValueFrom" | "suggestFirstValueFrom", [], RxjsXRuleDocs, TSESLint.RuleListener>;
-        'no-unbound-methods': TSESLint.RuleModule<"forbidden", [], RxjsXRuleDocs, TSESLint.RuleListener>;
+        'no-unbound-methods': TSESLint.RuleModule<"forbidden", readonly {
+            allowTypes?: string[];
+        }[], RxjsXRuleDocs, TSESLint.RuleListener>;
         'no-unsafe-catch': TSESLint.RuleModule<"forbidden", readonly {
             observable?: string;
         }[], RxjsXRuleDocs, TSESLint.RuleListener>;

package/dist/index.d.ts

@@ -150,7 +150,9 @@ declare const rxjsX: {
         'no-subscribe-in-pipe': TSESLint.RuleModule<"forbidden", [], RxjsXRuleDocs, TSESLint.RuleListener>;
         'no-tap': TSESLint.RuleModule<"forbidden", [], RxjsXRuleDocs, TSESLint.RuleListener>;
         'no-topromise': TSESLint.RuleModule<"forbidden" | "suggestLastValueFromWithDefault" | "suggestLastValueFrom" | "suggestFirstValueFrom", [], RxjsXRuleDocs, TSESLint.RuleListener>;
-        'no-unbound-methods': TSESLint.RuleModule<"forbidden", [], RxjsXRuleDocs, TSESLint.RuleListener>;
+        'no-unbound-methods': TSESLint.RuleModule<"forbidden", readonly {
+            allowTypes?: string[];
+        }[], RxjsXRuleDocs, TSESLint.RuleListener>;
         'no-unsafe-catch': TSESLint.RuleModule<"forbidden", readonly {
             observable?: string;
         }[], RxjsXRuleDocs, TSESLint.RuleListener>;

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.7.3";
+var version = "0.7.5";
 
 // src/configs/recommended.ts
 var createRecommendedConfig = (plugin2) => ({
@@ -284,6 +284,9 @@ function isPropertyDefinition(node) {
 function isUnaryExpression(node) {
   return node.type === _utils.AST_NODE_TYPES.UnaryExpression;
 }
+function isUnionType(node) {
+  return node.type === _utils.AST_NODE_TYPES.TSUnionType;
+}
 
 // src/etc/get-type-services.ts
 function getTypeServices(context) {
@@ -389,6 +392,44 @@ function findIsLastOperatorOrderValid(pipeCallExpression, operatorsRegExp, allow
   return { isOrderValid, operatorNode };
 }
 
+// src/constants.ts
+var defaultObservable = String.raw`[Aa]ction(s|s\$|\$)$`;
+var SOURCES_OBJECT_ACCEPTING_STATIC_OBSERVABLE_CREATORS = [
+  "combineLatest",
+  "forkJoin"
+];
+var DEFAULT_UNBOUND_ALLOWED_TYPES = [
+  "Signal"
+];
+var DEFAULT_VALID_POST_COMPLETION_OPERATORS = [
+  "count",
+  "defaultIfEmpty",
+  "endWith",
+  "every",
+  "finalize",
+  "finally",
+  "isEmpty",
+  "last",
+  "max",
+  "min",
+  "publish",
+  "publishBehavior",
+  "publishLast",
+  "publishReplay",
+  "reduce",
+  "share",
+  "shareReplay",
+  "skipLast",
+  "takeLast",
+  "throwIfEmpty",
+  "toArray"
+];
+
+// src/utils/is-sources-object-accepting-static-observable-creator.ts
+function isSourcesObjectAcceptingStaticObservableCreator(expression) {
+  return isCallExpression(expression) && isIdentifier(expression.callee) && SOURCES_OBJECT_ACCEPTING_STATIC_OBSERVABLE_CREATORS.includes(expression.callee.name);
+}
+
 // src/utils/rule-creator.ts
 
 var REPO_URL = "https://github.com/JasonWeinzierl/eslint-plugin-rxjs-x";
@@ -717,7 +758,7 @@ var finnishRule = ruleCreator({
       "ObjectExpression > Property[computed=false] > Identifier": (node) => {
         if (validate.properties) {
           const parent = node.parent;
-          if (node === parent.key) {
+          if (node === parent.key && !isSourcesObjectAcceptingStaticObservableCreator(parent.parent.parent)) {
             checkNode(node);
           }
         }
@@ -894,7 +935,7 @@ var noAsyncSubscribeRule = ruleCreator({
       requiresTypeChecking: true
     },
     messages: {
-      forbidden: "Passing async functions to subscribe is forbidden."
+      forbidden: "Passing async functions to subscribe is forbidden. Use operators like `concatMap` or `switchMap` to avoid race conditions."
     },
     schema: [],
     type: "problem"
@@ -902,27 +943,39 @@ var noAsyncSubscribeRule = ruleCreator({
   name: "no-async-subscribe",
   create: (context) => {
     const { couldBeObservable } = getTypeServices(context);
-    function checkNode(node) {
-      const parentNode = node.parent;
+    function report(node) {
+      const { loc } = node;
+      const asyncLoc = {
+        ...loc,
+        end: {
+          ...loc.start,
+          column: loc.start.column + 5
+        }
+      };
+      context.report({
+        messageId: "forbidden",
+        loc: asyncLoc
+      });
+    }
+    function checkSingleNext(funcNode) {
+      const parentNode = funcNode.parent;
       const callee = parentNode.callee;
       if (couldBeObservable(callee.object)) {
-        const { loc } = node;
-        const asyncLoc = {
-          ...loc,
-          end: {
-            ...loc.start,
-            column: loc.start.column + 5
-          }
-        };
-        context.report({
-          messageId: "forbidden",
-          loc: asyncLoc
-        });
+        report(funcNode);
+      }
+    }
+    function checkObserverObjectProp(propNode) {
+      const parentNode = propNode.parent.parent;
+      const callee = parentNode.callee;
+      if (couldBeObservable(callee.object) && (isArrowFunctionExpression(propNode.value) || isFunctionExpression(propNode.value))) {
+        report(propNode.value);
       }
     }
     return {
-      "CallExpression[callee.property.name='subscribe'] > FunctionExpression[async=true]": checkNode,
-      "CallExpression[callee.property.name='subscribe'] > ArrowFunctionExpression[async=true]": checkNode
+      "CallExpression[callee.property.name='subscribe'] > FunctionExpression[async=true]": checkSingleNext,
+      "CallExpression[callee.property.name='subscribe'] > ArrowFunctionExpression[async=true]": checkSingleNext,
+      "CallExpression[callee.property.name='subscribe'] > ObjectExpression > Property[key.name='next'][value.type=\"ArrowFunctionExpression\"][value.async=true]": checkObserverObjectProp,
+      "CallExpression[callee.property.name='subscribe'] > ObjectExpression > Property[key.name='next'][value.type=\"FunctionExpression\"][value.async=true]": checkObserverObjectProp
     };
   }
 });
@@ -1028,34 +1081,6 @@ var noCreateRule = ruleCreator({
 
 
 
-
-// src/constants.ts
-var defaultObservable = String.raw`[Aa]ction(s|s\$|\$)$`;
-var DEFAULT_VALID_POST_COMPLETION_OPERATORS = [
-  "count",
-  "defaultIfEmpty",
-  "endWith",
-  "every",
-  "finalize",
-  "finally",
-  "isEmpty",
-  "last",
-  "max",
-  "min",
-  "publish",
-  "publishBehavior",
-  "publishLast",
-  "publishReplay",
-  "reduce",
-  "share",
-  "shareReplay",
-  "skipLast",
-  "takeLast",
-  "throwIfEmpty",
-  "toArray"
-];
-
-// src/rules/no-cyclic-action.ts
 function isTypeReference2(type) {
   return Boolean(type.target);
 }
@@ -1180,21 +1205,25 @@ var noExplicitGenericsRule = ruleCreator({
       });
     }
     function checkBehaviorSubjects(node) {
+      var _a;
       const parent = node.parent;
       const {
         arguments: [value]
       } = parent;
-      if (isArrayExpression(value) || isObjectExpression(value)) {
+      const typeArgs = (_a = parent.typeArguments) == null ? void 0 : _a.params[0];
+      if (isArrayExpression(value) || isObjectExpression(value) || typeArgs && isUnionType(typeArgs)) {
         return;
       }
       report(node);
     }
     function checkNotifications(node) {
+      var _a;
       const parent = node.parent;
       const {
         arguments: [, value]
       } = parent;
-      if (isArrayExpression(value) || isObjectExpression(value)) {
+      const typeArgs = (_a = parent.typeArguments) == null ? void 0 : _a.params[0];
+      if (isArrayExpression(value) || isObjectExpression(value) || typeArgs && isUnionType(typeArgs)) {
         return;
       }
       report(node);
@@ -3102,8 +3131,9 @@ import { ${functionName} } from ${quote}rxjs${quote};`
 
 // src/rules/no-unbound-methods.ts
 
+var defaultOptions11 = [];
 var noUnboundMethodsRule = ruleCreator({
-  defaultOptions: [],
+  defaultOptions: defaultOptions11,
   meta: {
     docs: {
       description: "Disallow passing unbound methods.",
@@ -3113,7 +3143,14 @@ var noUnboundMethodsRule = ruleCreator({
     messages: {
       forbidden: "Unbound methods are forbidden."
     },
-    schema: [],
+    schema: [
+      {
+        properties: {
+          allowTypes: { type: "array", items: { type: "string" }, description: "An array of function types that are allowed to be passed unbound.", default: DEFAULT_UNBOUND_ALLOWED_TYPES }
+        },
+        type: "object"
+      }
+    ],
     type: "problem"
   },
   name: "no-unbound-methods",
@@ -3121,9 +3158,14 @@ var noUnboundMethodsRule = ruleCreator({
     const { getTypeAtLocation } = _utils.ESLintUtils.getParserServices(context);
     const { couldBeObservable, couldBeSubscription } = getTypeServices(context);
     const nodeMap = /* @__PURE__ */ new WeakMap();
+    const [config = {}] = context.options;
+    const { allowTypes = [] } = config;
     function mapArguments(node) {
       node.arguments.filter(isMemberExpression).forEach((arg) => {
         const argType = getTypeAtLocation(arg);
+        if (allowTypes.some((t) => couldBeType(argType, t))) {
+          return;
+        }
         if (argType.getCallSignatures().length > 0) {
           nodeMap.set(arg);
         }
@@ -3166,9 +3208,9 @@ var noUnboundMethodsRule = ruleCreator({
 
 // src/rules/no-unsafe-catch.ts
 
-var defaultOptions11 = [];
+var defaultOptions12 = [];
 var noUnsafeCatchRule = ruleCreator({
-  defaultOptions: defaultOptions11,
+  defaultOptions: defaultOptions12,
   meta: {
     docs: {
       description: "Disallow unsafe `catchError` usage in effects and epics.",
@@ -3227,9 +3269,9 @@ var noUnsafeCatchRule = ruleCreator({
 
 // src/rules/no-unsafe-first.ts
 
-var defaultOptions12 = [];
+var defaultOptions13 = [];
 var noUnsafeFirstRule = ruleCreator({
-  defaultOptions: defaultOptions12,
+  defaultOptions: defaultOptions13,
   meta: {
     docs: {
       description: "Disallow unsafe `first`/`take` usage in effects and epics.",
@@ -3351,9 +3393,9 @@ var noUnsafeSubjectNext = ruleCreator({
 // src/rules/no-unsafe-switchmap.ts
 
 var _decamelize = require('decamelize'); var _decamelize2 = _interopRequireDefault(_decamelize);
-var defaultOptions13 = [];
+var defaultOptions14 = [];
 var noUnsafeSwitchmapRule = ruleCreator({
-  defaultOptions: defaultOptions13,
+  defaultOptions: defaultOptions14,
   meta: {
     docs: {
       description: "Disallow unsafe `switchMap` usage in effects and epics.",
@@ -3476,9 +3518,9 @@ var noUnsafeSwitchmapRule = ruleCreator({
 
 // src/rules/no-unsafe-takeuntil.ts
 
-var defaultOptions14 = [];
+var defaultOptions15 = [];
 var noUnsafeTakeuntilRule = ruleCreator({
-  defaultOptions: defaultOptions14,
+  defaultOptions: defaultOptions15,
   meta: {
     docs: {
       description: "Disallow applying operators after `takeUntil`.",
@@ -3539,9 +3581,9 @@ var noUnsafeTakeuntilRule = ruleCreator({
 });
 
 // src/rules/prefer-observer.ts
-var defaultOptions15 = [];
+var defaultOptions16 = [];
 var preferObserverRule = ruleCreator({
-  defaultOptions: defaultOptions15,
+  defaultOptions: defaultOptions16,
   meta: {
     docs: {
       description: "Disallow passing separate handlers to `subscribe` and `tap`.",
@@ -3777,9 +3819,9 @@ var preferRootOperatorsRule = ruleCreator({
 
 // src/rules/suffix-subjects.ts
 
-var defaultOptions16 = [];
+var defaultOptions17 = [];
 var suffixSubjectsRule = ruleCreator({
-  defaultOptions: defaultOptions16,
+  defaultOptions: defaultOptions17,
   meta: {
     docs: {
       description: "Enforce the use of a suffix in subject identifiers.",
@@ -3911,7 +3953,7 @@ var suffixSubjectsRule = ruleCreator({
       "ObjectExpression > Property[computed=false] > Identifier": (node) => {
         if (validate.properties) {
           const parent = node.parent;
-          if (node === parent.key) {
+          if (node === parent.key && !isSourcesObjectAcceptingStaticObservableCreator(parent.parent.parent)) {
             checkNode(node);
           }
         }
@@ -3976,12 +4018,12 @@ var suffixSubjectsRule = ruleCreator({
 // src/rules/throw-error.ts
 
 
-var defaultOptions17 = [];
+var defaultOptions18 = [];
 var throwErrorRule = ruleCreator({
-  defaultOptions: defaultOptions17,
+  defaultOptions: defaultOptions18,
   meta: {
     docs: {
-      description: "Enforce passing only `Error` values to `throwError`.",
+      description: "Enforce passing only `Error` values to `throwError` or `Subject.error`.",
       recommended: {
         recommended: true,
         strict: [{ allowThrowingAny: false, allowThrowingUnknown: false }]
@@ -4005,14 +4047,14 @@ var throwErrorRule = ruleCreator({
   name: "throw-error",
   create: (context) => {
     const { esTreeNodeToTSNodeMap, program, getTypeAtLocation } = _utils.ESLintUtils.getParserServices(context);
-    const { couldBeObservable } = getTypeServices(context);
+    const { couldBeObservable, couldBeSubject } = getTypeServices(context);
     const [config = {}] = context.options;
     const { allowThrowingAny = true, allowThrowingUnknown = true } = config;
-    function checkThrowArgument(node) {
+    function checkThrowArgument(node, noFunction = false) {
       var _a;
       let type = getTypeAtLocation(node);
       let reportNode = node;
-      if (couldBeFunction(type)) {
+      if (!noFunction && couldBeFunction(type)) {
         reportNode = (_a = node.body) != null ? _a : node;
         const tsNode = esTreeNodeToTSNodeMap.get(node);
         const annotation = tsNode.type;
@@ -4033,7 +4075,7 @@ var throwErrorRule = ruleCreator({
         node: reportNode
       });
     }
-    function checkNode(node) {
+    function checkThrowError(node) {
       if (couldBeObservable(node)) {
         const [arg] = node.arguments;
         if (arg) {
@@ -4041,13 +4083,18 @@ var throwErrorRule = ruleCreator({
         }
       }
     }
-    return {
-      "CallExpression[callee.name='throwError']": (node) => {
-        checkNode(node);
-      },
-      "CallExpression[callee.property.name='throwError']": (node) => {
-        checkNode(node);
+    function checkSubjectError(node) {
+      if (isMemberExpression(node.callee) && couldBeSubject(node.callee.object)) {
+        const [arg] = node.arguments;
+        if (arg) {
+          checkThrowArgument(arg, true);
+        }
       }
+    }
+    return {
+      "CallExpression[callee.name='throwError']": checkThrowError,
+      "CallExpression[callee.property.name='throwError']": checkThrowError,
+      "CallExpression[callee.property.name='error']": checkSubjectError
     };
   }
 });

package/dist/index.mjs

@@ -1,6 +1,6 @@
 // package.json
 var name = "eslint-plugin-rxjs-x";
-var version = "0.7.3";
+var version = "0.7.5";
 
 // src/configs/recommended.ts
 var createRecommendedConfig = (plugin2) => ({
@@ -284,6 +284,9 @@ function isPropertyDefinition(node) {
 function isUnaryExpression(node) {
   return node.type === AST_NODE_TYPES.UnaryExpression;
 }
+function isUnionType(node) {
+  return node.type === AST_NODE_TYPES.TSUnionType;
+}
 
 // src/etc/get-type-services.ts
 function getTypeServices(context) {
@@ -389,6 +392,44 @@ function findIsLastOperatorOrderValid(pipeCallExpression, operatorsRegExp, allow
   return { isOrderValid, operatorNode };
 }
 
+// src/constants.ts
+var defaultObservable = String.raw`[Aa]ction(s|s\$|\$)$`;
+var SOURCES_OBJECT_ACCEPTING_STATIC_OBSERVABLE_CREATORS = [
+  "combineLatest",
+  "forkJoin"
+];
+var DEFAULT_UNBOUND_ALLOWED_TYPES = [
+  "Signal"
+];
+var DEFAULT_VALID_POST_COMPLETION_OPERATORS = [
+  "count",
+  "defaultIfEmpty",
+  "endWith",
+  "every",
+  "finalize",
+  "finally",
+  "isEmpty",
+  "last",
+  "max",
+  "min",
+  "publish",
+  "publishBehavior",
+  "publishLast",
+  "publishReplay",
+  "reduce",
+  "share",
+  "shareReplay",
+  "skipLast",
+  "takeLast",
+  "throwIfEmpty",
+  "toArray"
+];
+
+// src/utils/is-sources-object-accepting-static-observable-creator.ts
+function isSourcesObjectAcceptingStaticObservableCreator(expression) {
+  return isCallExpression(expression) && isIdentifier(expression.callee) && SOURCES_OBJECT_ACCEPTING_STATIC_OBSERVABLE_CREATORS.includes(expression.callee.name);
+}
+
 // src/utils/rule-creator.ts
 import { ESLintUtils as ESLintUtils2 } from "@typescript-eslint/utils";
 var REPO_URL = "https://github.com/JasonWeinzierl/eslint-plugin-rxjs-x";
@@ -717,7 +758,7 @@ var finnishRule = ruleCreator({
       "ObjectExpression > Property[computed=false] > Identifier": (node) => {
         if (validate.properties) {
           const parent = node.parent;
-          if (node === parent.key) {
+          if (node === parent.key && !isSourcesObjectAcceptingStaticObservableCreator(parent.parent.parent)) {
             checkNode(node);
           }
         }
@@ -894,7 +935,7 @@ var noAsyncSubscribeRule = ruleCreator({
       requiresTypeChecking: true
     },
     messages: {
-      forbidden: "Passing async functions to subscribe is forbidden."
+      forbidden: "Passing async functions to subscribe is forbidden. Use operators like `concatMap` or `switchMap` to avoid race conditions."
     },
     schema: [],
     type: "problem"
@@ -902,27 +943,39 @@ var noAsyncSubscribeRule = ruleCreator({
   name: "no-async-subscribe",
   create: (context) => {
     const { couldBeObservable } = getTypeServices(context);
-    function checkNode(node) {
-      const parentNode = node.parent;
+    function report(node) {
+      const { loc } = node;
+      const asyncLoc = {
+        ...loc,
+        end: {
+          ...loc.start,
+          column: loc.start.column + 5
+        }
+      };
+      context.report({
+        messageId: "forbidden",
+        loc: asyncLoc
+      });
+    }
+    function checkSingleNext(funcNode) {
+      const parentNode = funcNode.parent;
       const callee = parentNode.callee;
       if (couldBeObservable(callee.object)) {
-        const { loc } = node;
-        const asyncLoc = {
-          ...loc,
-          end: {
-            ...loc.start,
-            column: loc.start.column + 5
-          }
-        };
-        context.report({
-          messageId: "forbidden",
-          loc: asyncLoc
-        });
+        report(funcNode);
+      }
+    }
+    function checkObserverObjectProp(propNode) {
+      const parentNode = propNode.parent.parent;
+      const callee = parentNode.callee;
+      if (couldBeObservable(callee.object) && (isArrowFunctionExpression(propNode.value) || isFunctionExpression(propNode.value))) {
+        report(propNode.value);
       }
     }
     return {
-      "CallExpression[callee.property.name='subscribe'] > FunctionExpression[async=true]": checkNode,
-      "CallExpression[callee.property.name='subscribe'] > ArrowFunctionExpression[async=true]": checkNode
+      "CallExpression[callee.property.name='subscribe'] > FunctionExpression[async=true]": checkSingleNext,
+      "CallExpression[callee.property.name='subscribe'] > ArrowFunctionExpression[async=true]": checkSingleNext,
+      "CallExpression[callee.property.name='subscribe'] > ObjectExpression > Property[key.name='next'][value.type=\"ArrowFunctionExpression\"][value.async=true]": checkObserverObjectProp,
+      "CallExpression[callee.property.name='subscribe'] > ObjectExpression > Property[key.name='next'][value.type=\"FunctionExpression\"][value.async=true]": checkObserverObjectProp
     };
   }
 });
@@ -1028,34 +1081,6 @@ var noCreateRule = ruleCreator({
 import { ESLintUtils as ESLintUtils4 } from "@typescript-eslint/utils";
 import { stripIndent as stripIndent3 } from "common-tags";
 import ts5 from "typescript";
-
-// src/constants.ts
-var defaultObservable = String.raw`[Aa]ction(s|s\$|\$)$`;
-var DEFAULT_VALID_POST_COMPLETION_OPERATORS = [
-  "count",
-  "defaultIfEmpty",
-  "endWith",
-  "every",
-  "finalize",
-  "finally",
-  "isEmpty",
-  "last",
-  "max",
-  "min",
-  "publish",
-  "publishBehavior",
-  "publishLast",
-  "publishReplay",
-  "reduce",
-  "share",
-  "shareReplay",
-  "skipLast",
-  "takeLast",
-  "throwIfEmpty",
-  "toArray"
-];
-
-// src/rules/no-cyclic-action.ts
 function isTypeReference2(type) {
   return Boolean(type.target);
 }
@@ -1180,21 +1205,25 @@ var noExplicitGenericsRule = ruleCreator({
       });
     }
     function checkBehaviorSubjects(node) {
+      var _a;
       const parent = node.parent;
       const {
         arguments: [value]
       } = parent;
-      if (isArrayExpression(value) || isObjectExpression(value)) {
+      const typeArgs = (_a = parent.typeArguments) == null ? void 0 : _a.params[0];
+      if (isArrayExpression(value) || isObjectExpression(value) || typeArgs && isUnionType(typeArgs)) {
         return;
       }
       report(node);
     }
     function checkNotifications(node) {
+      var _a;
       const parent = node.parent;
       const {
         arguments: [, value]
       } = parent;
-      if (isArrayExpression(value) || isObjectExpression(value)) {
+      const typeArgs = (_a = parent.typeArguments) == null ? void 0 : _a.params[0];
+      if (isArrayExpression(value) || isObjectExpression(value) || typeArgs && isUnionType(typeArgs)) {
         return;
       }
       report(node);
@@ -3102,8 +3131,9 @@ import { ${functionName} } from ${quote}rxjs${quote};`
 
 // src/rules/no-unbound-methods.ts
 import { ESLintUtils as ESLintUtils9 } from "@typescript-eslint/utils";
+var defaultOptions11 = [];
 var noUnboundMethodsRule = ruleCreator({
-  defaultOptions: [],
+  defaultOptions: defaultOptions11,
   meta: {
     docs: {
       description: "Disallow passing unbound methods.",
@@ -3113,7 +3143,14 @@ var noUnboundMethodsRule = ruleCreator({
     messages: {
       forbidden: "Unbound methods are forbidden."
     },
-    schema: [],
+    schema: [
+      {
+        properties: {
+          allowTypes: { type: "array", items: { type: "string" }, description: "An array of function types that are allowed to be passed unbound.", default: DEFAULT_UNBOUND_ALLOWED_TYPES }
+        },
+        type: "object"
+      }
+    ],
     type: "problem"
   },
   name: "no-unbound-methods",
@@ -3121,9 +3158,14 @@ var noUnboundMethodsRule = ruleCreator({
     const { getTypeAtLocation } = ESLintUtils9.getParserServices(context);
     const { couldBeObservable, couldBeSubscription } = getTypeServices(context);
     const nodeMap = /* @__PURE__ */ new WeakMap();
+    const [config = {}] = context.options;
+    const { allowTypes = [] } = config;
     function mapArguments(node) {
       node.arguments.filter(isMemberExpression).forEach((arg) => {
         const argType = getTypeAtLocation(arg);
+        if (allowTypes.some((t) => couldBeType(argType, t))) {
+          return;
+        }
         if (argType.getCallSignatures().length > 0) {
           nodeMap.set(arg);
         }
@@ -3166,9 +3208,9 @@ var noUnboundMethodsRule = ruleCreator({
 
 // src/rules/no-unsafe-catch.ts
 import { stripIndent as stripIndent5 } from "common-tags";
-var defaultOptions11 = [];
+var defaultOptions12 = [];
 var noUnsafeCatchRule = ruleCreator({
-  defaultOptions: defaultOptions11,
+  defaultOptions: defaultOptions12,
   meta: {
     docs: {
       description: "Disallow unsafe `catchError` usage in effects and epics.",
@@ -3227,9 +3269,9 @@ var noUnsafeCatchRule = ruleCreator({
 
 // src/rules/no-unsafe-first.ts
 import { stripIndent as stripIndent6 } from "common-tags";
-var defaultOptions12 = [];
+var defaultOptions13 = [];
 var noUnsafeFirstRule = ruleCreator({
-  defaultOptions: defaultOptions12,
+  defaultOptions: defaultOptions13,
   meta: {
     docs: {
       description: "Disallow unsafe `first`/`take` usage in effects and epics.",
@@ -3351,9 +3393,9 @@ var noUnsafeSubjectNext = ruleCreator({
 // src/rules/no-unsafe-switchmap.ts
 import { stripIndent as stripIndent7 } from "common-tags";
 import decamelize from "decamelize";
-var defaultOptions13 = [];
+var defaultOptions14 = [];
 var noUnsafeSwitchmapRule = ruleCreator({
-  defaultOptions: defaultOptions13,
+  defaultOptions: defaultOptions14,
   meta: {
     docs: {
       description: "Disallow unsafe `switchMap` usage in effects and epics.",
@@ -3476,9 +3518,9 @@ var noUnsafeSwitchmapRule = ruleCreator({
 
 // src/rules/no-unsafe-takeuntil.ts
 import { stripIndent as stripIndent8 } from "common-tags";
-var defaultOptions14 = [];
+var defaultOptions15 = [];
 var noUnsafeTakeuntilRule = ruleCreator({
-  defaultOptions: defaultOptions14,
+  defaultOptions: defaultOptions15,
   meta: {
     docs: {
       description: "Disallow applying operators after `takeUntil`.",
@@ -3539,9 +3581,9 @@ var noUnsafeTakeuntilRule = ruleCreator({
 });
 
 // src/rules/prefer-observer.ts
-var defaultOptions15 = [];
+var defaultOptions16 = [];
 var preferObserverRule = ruleCreator({
-  defaultOptions: defaultOptions15,
+  defaultOptions: defaultOptions16,
   meta: {
     docs: {
       description: "Disallow passing separate handlers to `subscribe` and `tap`.",
@@ -3777,9 +3819,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 defaultOptions16 = [];
+var defaultOptions17 = [];
 var suffixSubjectsRule = ruleCreator({
-  defaultOptions: defaultOptions16,
+  defaultOptions: defaultOptions17,
   meta: {
     docs: {
       description: "Enforce the use of a suffix in subject identifiers.",
@@ -3911,7 +3953,7 @@ var suffixSubjectsRule = ruleCreator({
       "ObjectExpression > Property[computed=false] > Identifier": (node) => {
         if (validate.properties) {
           const parent = node.parent;
-          if (node === parent.key) {
+          if (node === parent.key && !isSourcesObjectAcceptingStaticObservableCreator(parent.parent.parent)) {
             checkNode(node);
           }
         }
@@ -3976,12 +4018,12 @@ var suffixSubjectsRule = ruleCreator({
 // src/rules/throw-error.ts
 import { ESLintUtils as ESLintUtils12 } from "@typescript-eslint/utils";
 import * as tsutils4 from "ts-api-utils";
-var defaultOptions17 = [];
+var defaultOptions18 = [];
 var throwErrorRule = ruleCreator({
-  defaultOptions: defaultOptions17,
+  defaultOptions: defaultOptions18,
   meta: {
     docs: {
-      description: "Enforce passing only `Error` values to `throwError`.",
+      description: "Enforce passing only `Error` values to `throwError` or `Subject.error`.",
       recommended: {
         recommended: true,
         strict: [{ allowThrowingAny: false, allowThrowingUnknown: false }]
@@ -4005,14 +4047,14 @@ var throwErrorRule = ruleCreator({
   name: "throw-error",
   create: (context) => {
     const { esTreeNodeToTSNodeMap, program, getTypeAtLocation } = ESLintUtils12.getParserServices(context);
-    const { couldBeObservable } = getTypeServices(context);
+    const { couldBeObservable, couldBeSubject } = getTypeServices(context);
     const [config = {}] = context.options;
     const { allowThrowingAny = true, allowThrowingUnknown = true } = config;
-    function checkThrowArgument(node) {
+    function checkThrowArgument(node, noFunction = false) {
       var _a;
       let type = getTypeAtLocation(node);
       let reportNode = node;
-      if (couldBeFunction(type)) {
+      if (!noFunction && couldBeFunction(type)) {
         reportNode = (_a = node.body) != null ? _a : node;
         const tsNode = esTreeNodeToTSNodeMap.get(node);
         const annotation = tsNode.type;
@@ -4033,7 +4075,7 @@ var throwErrorRule = ruleCreator({
         node: reportNode
       });
     }
-    function checkNode(node) {
+    function checkThrowError(node) {
       if (couldBeObservable(node)) {
         const [arg] = node.arguments;
         if (arg) {
@@ -4041,13 +4083,18 @@ var throwErrorRule = ruleCreator({
         }
       }
     }
-    return {
-      "CallExpression[callee.name='throwError']": (node) => {
-        checkNode(node);
-      },
-      "CallExpression[callee.property.name='throwError']": (node) => {
-        checkNode(node);
+    function checkSubjectError(node) {
+      if (isMemberExpression(node.callee) && couldBeSubject(node.callee.object)) {
+        const [arg] = node.arguments;
+        if (arg) {
+          checkThrowArgument(arg, true);
+        }
       }
+    }
+    return {
+      "CallExpression[callee.name='throwError']": checkThrowError,
+      "CallExpression[callee.property.name='throwError']": checkThrowError,
+      "CallExpression[callee.property.name='error']": checkSubjectError
     };
   }
 });

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "eslint-plugin-rxjs-x",
   "type": "commonjs",
-  "version": "0.7.3",
+  "version": "0.7.5",
   "packageManager": "yarn@4.9.1+sha512.f95ce356460e05be48d66401c1ae64ef84d163dd689964962c6888a9810865e39097a5e9de748876c2e0bf89b232d583c33982773e9903ae7a76257270986538",
   "description": "ESLint v9+ plugin for RxJS",
   "author": "Jason Weinzierl <weinzierljason@gmail.com>",
@@ -72,28 +72,29 @@
     }
   },
   "devDependencies": {
-    "@eslint/js": "^9.27.0",
+    "@eslint/js": "^9.28.0",
     "@stylistic/eslint-plugin": "^4.4.0",
     "@types/common-tags": "^1.8.4",
     "@types/node": "~18.18.0",
-    "@typescript-eslint/rule-tester": "^8.32.1",
+    "@typescript-eslint/rule-tester": "^8.33.1",
     "@typescript/vfs": "^1.6.1",
-    "@vitest/coverage-v8": "^3.1.4",
+    "@vitest/coverage-v8": "^3.2.0",
     "@vitest/eslint-plugin": "^1.2.1",
     "bumpp": "^10.1.1",
-    "eslint": "^9.27.0",
+    "eslint": "^9.28.0",
     "eslint-config-flat-gitignore": "^2.1.0",
     "eslint-doc-generator": "^2.1.2",
-    "eslint-import-resolver-typescript": "^4.4.0",
+    "eslint-import-resolver-typescript": "^4.4.2",
     "eslint-plugin-eslint-plugin": "^6.4.0",
-    "eslint-plugin-import-x": "^4.13.1",
-    "eslint-plugin-n": "^17.18.0",
+    "eslint-plugin-import-x": "^4.15.0",
+    "eslint-plugin-n": "^17.19.0",
     "markdownlint-cli2": "^0.18.1",
     "rxjs": "^7.8.2",
     "tsup": "^8.5.0",
     "typescript": "~5.8.3",
-    "typescript-eslint": "^8.32.1",
-    "vitest": "^3.1.4"
+    "typescript-eslint": "^8.33.1",
+    "vite": "^6.3.5",
+    "vitest": "^3.2.0"
   },
   "engines": {
     "node": "^18.18.0 || ^20.9.0 || >= 21.1.0"