eslint-plugin-rxjs-x 0.4.0 → 0.5.0

package/README.md

@@ -17,6 +17,7 @@ There are some breaking changes:
   - If you need to continue using this old format, use the original `eslint-plugin-rxjs` or a different fork.
 - The plugin namespace specified in the `recommended` config was changed from `rxjs` to `rxjs-x`.
   - e.g. In your ESLint config, `rxjs/no-subject-value` should be renamed to `rxjs-x/no-subject-value`.
+- The rule `rxjs/no-ignored-observable` is renamed to `rxjs-x/no-floating-observables`.
 
 A complete description of all changes are documented in the [CHANGELOG](CHANGELOG.md) file.
 
@@ -88,10 +89,10 @@ The package includes the following rules.
 | [no-explicit-generics](docs/rules/no-explicit-generics.md)             | Disallow unnecessary explicit generic type arguments.                                                     | 🔒   |    |    |    |    |
 | [no-exposed-subjects](docs/rules/no-exposed-subjects.md)               | Disallow public and protected subjects.                                                                   | 🔒   |    |    | 💭 |    |
 | [no-finnish](docs/rules/no-finnish.md)                                 | Disallow Finnish notation.                                                                                |      |    |    | 💭 |    |
+| [no-floating-observables](docs/rules/no-floating-observables.md)       | Require Observables to be handled appropriately.                                                          | 🔒   |    |    | 💭 |    |
 | [no-ignored-default-value](docs/rules/no-ignored-default-value.md)     | Disallow using `firstValueFrom`, `lastValueFrom`, `first`, and `last` without specifying a default value. | 🔒   |    |    | 💭 |    |
 | [no-ignored-error](docs/rules/no-ignored-error.md)                     | Disallow calling `subscribe` without specifying an error handler.                                         | 🔒   |    |    | 💭 |    |
 | [no-ignored-notifier](docs/rules/no-ignored-notifier.md)               | Disallow observables not composed from the `repeatWhen` or `retryWhen` notifier.                          | ✅ 🔒 |    |    | 💭 |    |
-| [no-ignored-observable](docs/rules/no-ignored-observable.md)           | Disallow ignoring observables returned by functions.                                                      | 🔒   |    |    | 💭 |    |
 | [no-ignored-replay-buffer](docs/rules/no-ignored-replay-buffer.md)     | Disallow using `ReplaySubject`, `publishReplay` or `shareReplay` without specifying the buffer size.      | ✅ 🔒 |    |    |    |    |
 | [no-ignored-subscribe](docs/rules/no-ignored-subscribe.md)             | Disallow calling `subscribe` without specifying arguments.                                                |      |    |    | 💭 |    |
 | [no-ignored-subscription](docs/rules/no-ignored-subscription.md)       | Disallow ignoring the subscription returned by `subscribe`.                                               |      |    |    | 💭 |    |
@@ -107,16 +108,16 @@ The package includes the following rules.
 | [no-subject-value](docs/rules/no-subject-value.md)                     | Disallow accessing the `value` property of a `BehaviorSubject` instance.                                  |      |    |    | 💭 |    |
 | [no-subscribe-handlers](docs/rules/no-subscribe-handlers.md)           | Disallow passing handlers to `subscribe`.                                                                 |      |    |    | 💭 |    |
 | [no-tap](docs/rules/no-tap.md)                                         | Disallow the `tap` operator.                                                                              |      |    |    |    | ❌  |
-| [no-topromise](docs/rules/no-topromise.md)                             | Disallow use of the `toPromise` method.                                                                   | 🔒   |    | 💡 | 💭 |    |
+| [no-topromise](docs/rules/no-topromise.md)                             | Disallow use of the `toPromise` method.                                                                   | ✅ 🔒 |    | 💡 | 💭 |    |
 | [no-unbound-methods](docs/rules/no-unbound-methods.md)                 | Disallow passing unbound methods.                                                                         | ✅ 🔒 |    |    | 💭 |    |
 | [no-unsafe-catch](docs/rules/no-unsafe-catch.md)                       | Disallow unsafe `catchError` usage in effects and epics.                                                  |      |    |    | 💭 |    |
 | [no-unsafe-first](docs/rules/no-unsafe-first.md)                       | Disallow unsafe `first`/`take` usage in effects and epics.                                                |      |    |    | 💭 |    |
 | [no-unsafe-subject-next](docs/rules/no-unsafe-subject-next.md)         | Disallow unsafe optional `next` calls.                                                                    | ✅ 🔒 |    |    | 💭 |    |
 | [no-unsafe-switchmap](docs/rules/no-unsafe-switchmap.md)               | Disallow unsafe `switchMap` usage in effects and epics.                                                   |      |    |    | 💭 |    |
 | [no-unsafe-takeuntil](docs/rules/no-unsafe-takeuntil.md)               | Disallow applying operators after `takeUntil`.                                                            | ✅ 🔒 |    |    | 💭 |    |
-| [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`.                                                       | 🔒   | 🔧 | 💡 |    |    |
+| [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`.                                                      | ✅ 🔒 |    |    | 💭 |    |
 
 <!-- end auto-generated rules list -->

package/dist/index.d.mts

@@ -25,9 +25,13 @@ declare const rxjsX: {
                 'rxjs-x/no-redundant-notify': "error";
                 'rxjs-x/no-sharereplay': "error";
                 'rxjs-x/no-subject-unsubscribe': "error";
+                'rxjs-x/no-topromise': "error";
                 'rxjs-x/no-unbound-methods': "error";
                 'rxjs-x/no-unsafe-subject-next': "error";
                 'rxjs-x/no-unsafe-takeuntil': "error";
+                'rxjs-x/prefer-observer': "error";
+                'rxjs-x/prefer-root-operators': "error";
+                'rxjs-x/throw-error': "error";
             };
         };
         strict: {
@@ -39,10 +43,10 @@ declare const rxjsX: {
                 'rxjs-x/no-create': "error";
                 'rxjs-x/no-explicit-generics': "error";
                 'rxjs-x/no-exposed-subjects': "error";
+                'rxjs-x/no-floating-observables': "error";
                 'rxjs-x/no-ignored-default-value': "error";
                 'rxjs-x/no-ignored-error': "error";
                 'rxjs-x/no-ignored-notifier': "error";
-                'rxjs-x/no-ignored-observable': "error";
                 'rxjs-x/no-ignored-replay-buffer': "error";
                 'rxjs-x/no-ignored-takewhile-value': "error";
                 'rxjs-x/no-implicit-any-catch': ["error", {
@@ -99,10 +103,12 @@ declare const rxjsX: {
             allowProtected?: boolean;
         }[], RxjsXRuleDocs, TSESLint.RuleListener>;
         'no-finnish': TSESLint.RuleModule<"forbidden", [], RxjsXRuleDocs, TSESLint.RuleListener>;
+        'no-floating-observables': TSESLint.RuleModule<"forbidden" | "forbiddenNoVoid", readonly {
+            ignoreVoid?: boolean;
+        }[], RxjsXRuleDocs, TSESLint.RuleListener>;
         'no-ignored-default-value': TSESLint.RuleModule<"forbidden", [], RxjsXRuleDocs, TSESLint.RuleListener>;
         'no-ignored-error': TSESLint.RuleModule<"forbidden", [], RxjsXRuleDocs, TSESLint.RuleListener>;
         'no-ignored-notifier': TSESLint.RuleModule<"forbidden", [], RxjsXRuleDocs, TSESLint.RuleListener>;
-        'no-ignored-observable': TSESLint.RuleModule<"forbidden", [], RxjsXRuleDocs, TSESLint.RuleListener>;
         'no-ignored-replay-buffer': TSESLint.RuleModule<"forbidden", [], RxjsXRuleDocs, TSESLint.RuleListener>;
         'no-ignored-subscribe': TSESLint.RuleModule<"forbidden", [], RxjsXRuleDocs, TSESLint.RuleListener>;
         'no-ignored-subscription': TSESLint.RuleModule<"forbidden", [], RxjsXRuleDocs, TSESLint.RuleListener>;

package/dist/index.d.ts

@@ -25,9 +25,13 @@ declare const rxjsX: {
                 'rxjs-x/no-redundant-notify': "error";
                 'rxjs-x/no-sharereplay': "error";
                 'rxjs-x/no-subject-unsubscribe': "error";
+                'rxjs-x/no-topromise': "error";
                 'rxjs-x/no-unbound-methods': "error";
                 'rxjs-x/no-unsafe-subject-next': "error";
                 'rxjs-x/no-unsafe-takeuntil': "error";
+                'rxjs-x/prefer-observer': "error";
+                'rxjs-x/prefer-root-operators': "error";
+                'rxjs-x/throw-error': "error";
             };
         };
         strict: {
@@ -39,10 +43,10 @@ declare const rxjsX: {
                 'rxjs-x/no-create': "error";
                 'rxjs-x/no-explicit-generics': "error";
                 'rxjs-x/no-exposed-subjects': "error";
+                'rxjs-x/no-floating-observables': "error";
                 'rxjs-x/no-ignored-default-value': "error";
                 'rxjs-x/no-ignored-error': "error";
                 'rxjs-x/no-ignored-notifier': "error";
-                'rxjs-x/no-ignored-observable': "error";
                 'rxjs-x/no-ignored-replay-buffer': "error";
                 'rxjs-x/no-ignored-takewhile-value': "error";
                 'rxjs-x/no-implicit-any-catch': ["error", {
@@ -99,10 +103,12 @@ declare const rxjsX: {
             allowProtected?: boolean;
         }[], RxjsXRuleDocs, TSESLint.RuleListener>;
         'no-finnish': TSESLint.RuleModule<"forbidden", [], RxjsXRuleDocs, TSESLint.RuleListener>;
+        'no-floating-observables': TSESLint.RuleModule<"forbidden" | "forbiddenNoVoid", readonly {
+            ignoreVoid?: boolean;
+        }[], RxjsXRuleDocs, TSESLint.RuleListener>;
         'no-ignored-default-value': TSESLint.RuleModule<"forbidden", [], RxjsXRuleDocs, TSESLint.RuleListener>;
         'no-ignored-error': TSESLint.RuleModule<"forbidden", [], RxjsXRuleDocs, TSESLint.RuleListener>;
         'no-ignored-notifier': TSESLint.RuleModule<"forbidden", [], RxjsXRuleDocs, TSESLint.RuleListener>;
-        'no-ignored-observable': TSESLint.RuleModule<"forbidden", [], RxjsXRuleDocs, TSESLint.RuleListener>;
         'no-ignored-replay-buffer': TSESLint.RuleModule<"forbidden", [], RxjsXRuleDocs, TSESLint.RuleListener>;
         'no-ignored-subscribe': TSESLint.RuleModule<"forbidden", [], RxjsXRuleDocs, TSESLint.RuleListener>;
         'no-ignored-subscription': TSESLint.RuleModule<"forbidden", [], 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.4.0";
+var version = "0.5.0";
 
 // src/configs/recommended.ts
 var createRecommendedConfig = (plugin2) => ({
@@ -20,9 +20,13 @@ var createRecommendedConfig = (plugin2) => ({
     "rxjs-x/no-redundant-notify": "error",
     "rxjs-x/no-sharereplay": "error",
     "rxjs-x/no-subject-unsubscribe": "error",
+    "rxjs-x/no-topromise": "error",
     "rxjs-x/no-unbound-methods": "error",
     "rxjs-x/no-unsafe-subject-next": "error",
-    "rxjs-x/no-unsafe-takeuntil": "error"
+    "rxjs-x/no-unsafe-takeuntil": "error",
+    "rxjs-x/prefer-observer": "error",
+    "rxjs-x/prefer-root-operators": "error",
+    "rxjs-x/throw-error": "error"
   }
 });
 
@@ -36,10 +40,10 @@ var createStrictConfig = (plugin2) => ({
     "rxjs-x/no-create": "error",
     "rxjs-x/no-explicit-generics": "error",
     "rxjs-x/no-exposed-subjects": "error",
+    "rxjs-x/no-floating-observables": "error",
     "rxjs-x/no-ignored-default-value": "error",
     "rxjs-x/no-ignored-error": "error",
     "rxjs-x/no-ignored-notifier": "error",
-    "rxjs-x/no-ignored-observable": "error",
     "rxjs-x/no-ignored-replay-buffer": "error",
     "rxjs-x/no-ignored-takewhile-value": "error",
     "rxjs-x/no-implicit-any-catch": ["error", {
@@ -295,6 +299,9 @@ function isBlockStatement(node) {
 function isCallExpression(node) {
   return node.type === _utils.AST_NODE_TYPES.CallExpression;
 }
+function isChainExpression(node) {
+  return node.type === _utils.AST_NODE_TYPES.ChainExpression;
+}
 function isFunctionDeclaration(node) {
   return node.type === _utils.AST_NODE_TYPES.FunctionDeclaration;
 }
@@ -331,6 +338,9 @@ function isProgram(node) {
 function isProperty(node) {
   return node.type === _utils.AST_NODE_TYPES.Property;
 }
+function isUnaryExpression(node) {
+  return node.type === _utils.AST_NODE_TYPES.UnaryExpression;
+}
 
 // src/etc/get-type-services.ts
 function getTypeServices(context) {
@@ -1318,6 +1328,87 @@ var noFinnishRule = ruleCreator({
   }
 });
 
+// src/rules/no-floating-observables.ts
+var defaultOptions6 = [];
+var messageBase = "Observables must be subscribed to, returned, converted to a promise and awaited, or be explicitly marked as ignored with the `void` operator.";
+var messageBaseNoVoid = "Observables must be subscribed to, returned, or converted to a promise and awaited.";
+var noFloatingObservablesRule = ruleCreator({
+  defaultOptions: defaultOptions6,
+  meta: {
+    docs: {
+      description: "Require Observables to be handled appropriately.",
+      recommended: "strict",
+      requiresTypeChecking: true
+    },
+    messages: {
+      forbidden: messageBase,
+      forbiddenNoVoid: messageBaseNoVoid
+    },
+    schema: [
+      {
+        properties: {
+          ignoreVoid: { type: "boolean", default: true, description: "Whether to ignore `void` expressions." }
+        },
+        type: "object"
+      }
+    ],
+    type: "problem"
+  },
+  name: "no-floating-observables",
+  create: (context) => {
+    const { couldBeObservable } = getTypeServices(context);
+    const [config = {}] = context.options;
+    const { ignoreVoid = true } = config;
+    function checkNode(node) {
+      if (couldBeObservable(node)) {
+        context.report({
+          messageId: ignoreVoid ? "forbidden" : "forbiddenNoVoid",
+          node
+        });
+      }
+    }
+    function checkVoid(node) {
+      if (ignoreVoid) return;
+      if (node.operator !== "void") return;
+      let expression = node.argument;
+      if (isChainExpression(expression)) {
+        expression = expression.expression;
+      }
+      if (!isCallExpression(expression)) return;
+      checkNode(expression);
+    }
+    return {
+      "ExpressionStatement > CallExpression": (node) => {
+        checkNode(node);
+      },
+      "ExpressionStatement > UnaryExpression": (node) => {
+        checkVoid(node);
+      },
+      "ExpressionStatement > ChainExpression": (node) => {
+        if (!isCallExpression(node.expression)) return;
+        checkNode(node.expression);
+      },
+      "ExpressionStatement > SequenceExpression": (node) => {
+        node.expressions.forEach((expression) => {
+          if (isCallExpression(expression)) {
+            checkNode(expression);
+          }
+        });
+      },
+      "ExpressionStatement > ArrayExpression": (node) => {
+        node.elements.forEach((expression) => {
+          if (!expression) return;
+          if (isCallExpression(expression)) {
+            checkNode(expression);
+          } else if (isUnaryExpression(expression)) {
+            checkVoid(expression);
+          }
+        });
+      }
+    };
+  }
+});
+
 // src/rules/no-ignored-default-value.ts
 
 var noIgnoredDefaultValueRule = ruleCreator({
@@ -1559,37 +1650,6 @@ var noIgnoredNotifierRule = ruleCreator({
   }
 });
 
-// src/rules/no-ignored-observable.ts
-var noIgnoredObservableRule = ruleCreator({
-  defaultOptions: [],
-  meta: {
-    docs: {
-      description: "Disallow ignoring observables returned by functions.",
-      recommended: "strict",
-      requiresTypeChecking: true
-    },
-    messages: {
-      forbidden: "Ignoring a returned Observable is forbidden."
-    },
-    schema: [],
-    type: "problem"
-  },
-  name: "no-ignored-observable",
-  create: (context) => {
-    const { couldBeObservable } = getTypeServices(context);
-    return {
-      "ExpressionStatement > CallExpression": (node) => {
-        if (couldBeObservable(node)) {
-          context.report({
-            messageId: "forbidden",
-            node
-          });
-        }
-      }
-    };
-  }
-});
-
 // src/rules/no-ignored-replay-buffer.ts
 var noIgnoredReplayBufferRule = ruleCreator({
   defaultOptions: [],
@@ -1780,9 +1840,9 @@ function isParenthesised(sourceCode, node) {
   const after = sourceCode.getTokenAfter(node);
   return before && after && before.value === "(" && before.range[1] <= node.range[0] && after.value === ")" && after.range[0] >= node.range[1];
 }
-var defaultOptions6 = [];
+var defaultOptions7 = [];
 var noImplicitAnyCatchRule = ruleCreator({
-  defaultOptions: defaultOptions6,
+  defaultOptions: defaultOptions7,
   meta: {
     docs: {
       description: "Disallow implicit `any` error parameters in `catchError` operators.",
@@ -2149,9 +2209,9 @@ function isExpressionObserver(expressionStatement, couldBeType2) {
 
 // src/rules/no-sharereplay.ts
 
-var defaultOptions7 = [];
+var defaultOptions8 = [];
 var noSharereplayRule = ruleCreator({
-  defaultOptions: defaultOptions7,
+  defaultOptions: defaultOptions8,
   meta: {
     docs: {
       description: "Disallow unsafe `shareReplay` usage.",
@@ -2385,7 +2445,7 @@ var noTopromiseRule = ruleCreator({
   meta: {
     docs: {
       description: "Disallow use of the `toPromise` method.",
-      recommended: "strict",
+      recommended: "recommended",
       requiresTypeChecking: true
     },
     hasSuggestions: true,
@@ -2539,9 +2599,9 @@ var noUnboundMethodsRule = ruleCreator({
 
 // src/rules/no-unsafe-catch.ts
 
-var defaultOptions8 = [];
+var defaultOptions9 = [];
 var noUnsafeCatchRule = ruleCreator({
-  defaultOptions: defaultOptions8,
+  defaultOptions: defaultOptions9,
   meta: {
     docs: {
       description: "Disallow unsafe `catchError` usage in effects and epics.",
@@ -2600,9 +2660,9 @@ var noUnsafeCatchRule = ruleCreator({
 
 // src/rules/no-unsafe-first.ts
 
-var defaultOptions9 = [];
+var defaultOptions10 = [];
 var noUnsafeFirstRule = ruleCreator({
-  defaultOptions: defaultOptions9,
+  defaultOptions: defaultOptions10,
   meta: {
     docs: {
       description: "Disallow unsafe `first`/`take` usage in effects and epics.",
@@ -2724,9 +2784,9 @@ var noUnsafeSubjectNext = ruleCreator({
 // src/rules/no-unsafe-switchmap.ts
 
 var _decamelize = require('decamelize'); var _decamelize2 = _interopRequireDefault(_decamelize);
-var defaultOptions10 = [];
+var defaultOptions11 = [];
 var noUnsafeSwitchmapRule = ruleCreator({
-  defaultOptions: defaultOptions10,
+  defaultOptions: defaultOptions11,
   meta: {
     docs: {
       description: "Disallow unsafe `switchMap` usage in effects and epics.",
@@ -2849,7 +2909,7 @@ var noUnsafeSwitchmapRule = ruleCreator({
 
 // src/rules/no-unsafe-takeuntil.ts
 
-var defaultOptions11 = [];
+var defaultOptions12 = [];
 var allowedOperators = [
   "count",
   "defaultIfEmpty",
@@ -2874,7 +2934,7 @@ var allowedOperators = [
   "toArray"
 ];
 var noUnsafeTakeuntilRule = ruleCreator({
-  defaultOptions: defaultOptions11,
+  defaultOptions: defaultOptions12,
   meta: {
     docs: {
       description: "Disallow applying operators after `takeUntil`.",
@@ -2953,13 +3013,13 @@ var noUnsafeTakeuntilRule = ruleCreator({
 });
 
 // src/rules/prefer-observer.ts
-var defaultOptions12 = [];
+var defaultOptions13 = [];
 var preferObserverRule = ruleCreator({
-  defaultOptions: defaultOptions12,
+  defaultOptions: defaultOptions13,
   meta: {
     docs: {
       description: "Disallow passing separate handlers to `subscribe` and `tap`.",
-      recommended: "strict",
+      recommended: "recommended",
       requiresTypeChecking: true
     },
     fixable: "code",
@@ -3069,7 +3129,7 @@ var preferRootOperatorsRule = ruleCreator({
   meta: {
     docs: {
       description: "Disallow importing operators from `rxjs/operators`.",
-      recommended: "strict"
+      recommended: "recommended"
     },
     fixable: "code",
     hasSuggestions: true,
@@ -3191,9 +3251,9 @@ var preferRootOperatorsRule = ruleCreator({
 
 // src/rules/suffix-subjects.ts
 
-var defaultOptions13 = [];
+var defaultOptions14 = [];
 var suffixSubjectsRule = ruleCreator({
-  defaultOptions: defaultOptions13,
+  defaultOptions: defaultOptions14,
   meta: {
     docs: {
       description: "Enforce the use of a suffix in subject identifiers.",
@@ -3390,13 +3450,14 @@ var suffixSubjectsRule = ruleCreator({
 // src/rules/throw-error.ts
 
 
-var defaultOptions14 = [];
+var defaultOptions15 = [];
 var throwErrorRule = ruleCreator({
-  defaultOptions: defaultOptions14,
+  defaultOptions: defaultOptions15,
   meta: {
     docs: {
       description: "Enforce passing only `Error` values to `throwError`.",
       recommended: {
+        recommended: true,
         strict: [{ allowThrowingAny: false, allowThrowingUnknown: false }]
       },
       requiresTypeChecking: true
@@ -3482,10 +3543,10 @@ var plugin = {
     "no-explicit-generics": noExplicitGenericsRule,
     "no-exposed-subjects": noExposedSubjectsRule,
     "no-finnish": noFinnishRule,
+    "no-floating-observables": noFloatingObservablesRule,
     "no-ignored-default-value": noIgnoredDefaultValueRule,
     "no-ignored-error": noIgnoredErrorRule,
     "no-ignored-notifier": noIgnoredNotifierRule,
-    "no-ignored-observable": noIgnoredObservableRule,
     "no-ignored-replay-buffer": noIgnoredReplayBufferRule,
     "no-ignored-subscribe": noIgnoredSubscribeRule,
     "no-ignored-subscription": noIgnoredSubscriptionRule,

package/dist/index.mjs

@@ -1,6 +1,6 @@
 // package.json
 var name = "eslint-plugin-rxjs-x";
-var version = "0.4.0";
+var version = "0.5.0";
 
 // src/configs/recommended.ts
 var createRecommendedConfig = (plugin2) => ({
@@ -20,9 +20,13 @@ var createRecommendedConfig = (plugin2) => ({
     "rxjs-x/no-redundant-notify": "error",
     "rxjs-x/no-sharereplay": "error",
     "rxjs-x/no-subject-unsubscribe": "error",
+    "rxjs-x/no-topromise": "error",
     "rxjs-x/no-unbound-methods": "error",
     "rxjs-x/no-unsafe-subject-next": "error",
-    "rxjs-x/no-unsafe-takeuntil": "error"
+    "rxjs-x/no-unsafe-takeuntil": "error",
+    "rxjs-x/prefer-observer": "error",
+    "rxjs-x/prefer-root-operators": "error",
+    "rxjs-x/throw-error": "error"
   }
 });
 
@@ -36,10 +40,10 @@ var createStrictConfig = (plugin2) => ({
     "rxjs-x/no-create": "error",
     "rxjs-x/no-explicit-generics": "error",
     "rxjs-x/no-exposed-subjects": "error",
+    "rxjs-x/no-floating-observables": "error",
     "rxjs-x/no-ignored-default-value": "error",
     "rxjs-x/no-ignored-error": "error",
     "rxjs-x/no-ignored-notifier": "error",
-    "rxjs-x/no-ignored-observable": "error",
     "rxjs-x/no-ignored-replay-buffer": "error",
     "rxjs-x/no-ignored-takewhile-value": "error",
     "rxjs-x/no-implicit-any-catch": ["error", {
@@ -295,6 +299,9 @@ function isBlockStatement(node) {
 function isCallExpression(node) {
   return node.type === AST_NODE_TYPES2.CallExpression;
 }
+function isChainExpression(node) {
+  return node.type === AST_NODE_TYPES2.ChainExpression;
+}
 function isFunctionDeclaration(node) {
   return node.type === AST_NODE_TYPES2.FunctionDeclaration;
 }
@@ -331,6 +338,9 @@ function isProgram(node) {
 function isProperty(node) {
   return node.type === AST_NODE_TYPES2.Property;
 }
+function isUnaryExpression(node) {
+  return node.type === AST_NODE_TYPES2.UnaryExpression;
+}
 
 // src/etc/get-type-services.ts
 function getTypeServices(context) {
@@ -1318,6 +1328,87 @@ var noFinnishRule = ruleCreator({
   }
 });
 
+// src/rules/no-floating-observables.ts
+var defaultOptions6 = [];
+var messageBase = "Observables must be subscribed to, returned, converted to a promise and awaited, or be explicitly marked as ignored with the `void` operator.";
+var messageBaseNoVoid = "Observables must be subscribed to, returned, or converted to a promise and awaited.";
+var noFloatingObservablesRule = ruleCreator({
+  defaultOptions: defaultOptions6,
+  meta: {
+    docs: {
+      description: "Require Observables to be handled appropriately.",
+      recommended: "strict",
+      requiresTypeChecking: true
+    },
+    messages: {
+      forbidden: messageBase,
+      forbiddenNoVoid: messageBaseNoVoid
+    },
+    schema: [
+      {
+        properties: {
+          ignoreVoid: { type: "boolean", default: true, description: "Whether to ignore `void` expressions." }
+        },
+        type: "object"
+      }
+    ],
+    type: "problem"
+  },
+  name: "no-floating-observables",
+  create: (context) => {
+    const { couldBeObservable } = getTypeServices(context);
+    const [config = {}] = context.options;
+    const { ignoreVoid = true } = config;
+    function checkNode(node) {
+      if (couldBeObservable(node)) {
+        context.report({
+          messageId: ignoreVoid ? "forbidden" : "forbiddenNoVoid",
+          node
+        });
+      }
+    }
+    function checkVoid(node) {
+      if (ignoreVoid) return;
+      if (node.operator !== "void") return;
+      let expression = node.argument;
+      if (isChainExpression(expression)) {
+        expression = expression.expression;
+      }
+      if (!isCallExpression(expression)) return;
+      checkNode(expression);
+    }
+    return {
+      "ExpressionStatement > CallExpression": (node) => {
+        checkNode(node);
+      },
+      "ExpressionStatement > UnaryExpression": (node) => {
+        checkVoid(node);
+      },
+      "ExpressionStatement > ChainExpression": (node) => {
+        if (!isCallExpression(node.expression)) return;
+        checkNode(node.expression);
+      },
+      "ExpressionStatement > SequenceExpression": (node) => {
+        node.expressions.forEach((expression) => {
+          if (isCallExpression(expression)) {
+            checkNode(expression);
+          }
+        });
+      },
+      "ExpressionStatement > ArrayExpression": (node) => {
+        node.elements.forEach((expression) => {
+          if (!expression) return;
+          if (isCallExpression(expression)) {
+            checkNode(expression);
+          } else if (isUnaryExpression(expression)) {
+            checkVoid(expression);
+          }
+        });
+      }
+    };
+  }
+});
+
 // src/rules/no-ignored-default-value.ts
 import { ESLintUtils as ESLintUtils6 } from "@typescript-eslint/utils";
 var noIgnoredDefaultValueRule = ruleCreator({
@@ -1559,37 +1650,6 @@ var noIgnoredNotifierRule = ruleCreator({
   }
 });
 
-// src/rules/no-ignored-observable.ts
-var noIgnoredObservableRule = ruleCreator({
-  defaultOptions: [],
-  meta: {
-    docs: {
-      description: "Disallow ignoring observables returned by functions.",
-      recommended: "strict",
-      requiresTypeChecking: true
-    },
-    messages: {
-      forbidden: "Ignoring a returned Observable is forbidden."
-    },
-    schema: [],
-    type: "problem"
-  },
-  name: "no-ignored-observable",
-  create: (context) => {
-    const { couldBeObservable } = getTypeServices(context);
-    return {
-      "ExpressionStatement > CallExpression": (node) => {
-        if (couldBeObservable(node)) {
-          context.report({
-            messageId: "forbidden",
-            node
-          });
-        }
-      }
-    };
-  }
-});
-
 // src/rules/no-ignored-replay-buffer.ts
 var noIgnoredReplayBufferRule = ruleCreator({
   defaultOptions: [],
@@ -1780,9 +1840,9 @@ function isParenthesised(sourceCode, node) {
   const after = sourceCode.getTokenAfter(node);
   return before && after && before.value === "(" && before.range[1] <= node.range[0] && after.value === ")" && after.range[0] >= node.range[1];
 }
-var defaultOptions6 = [];
+var defaultOptions7 = [];
 var noImplicitAnyCatchRule = ruleCreator({
-  defaultOptions: defaultOptions6,
+  defaultOptions: defaultOptions7,
   meta: {
     docs: {
       description: "Disallow implicit `any` error parameters in `catchError` operators.",
@@ -2149,9 +2209,9 @@ function isExpressionObserver(expressionStatement, couldBeType2) {
 
 // src/rules/no-sharereplay.ts
 import { AST_NODE_TYPES as AST_NODE_TYPES6 } from "@typescript-eslint/utils";
-var defaultOptions7 = [];
+var defaultOptions8 = [];
 var noSharereplayRule = ruleCreator({
-  defaultOptions: defaultOptions7,
+  defaultOptions: defaultOptions8,
   meta: {
     docs: {
       description: "Disallow unsafe `shareReplay` usage.",
@@ -2385,7 +2445,7 @@ var noTopromiseRule = ruleCreator({
   meta: {
     docs: {
       description: "Disallow use of the `toPromise` method.",
-      recommended: "strict",
+      recommended: "recommended",
       requiresTypeChecking: true
     },
     hasSuggestions: true,
@@ -2539,9 +2599,9 @@ var noUnboundMethodsRule = ruleCreator({
 
 // src/rules/no-unsafe-catch.ts
 import { stripIndent as stripIndent4 } from "common-tags";
-var defaultOptions8 = [];
+var defaultOptions9 = [];
 var noUnsafeCatchRule = ruleCreator({
-  defaultOptions: defaultOptions8,
+  defaultOptions: defaultOptions9,
   meta: {
     docs: {
       description: "Disallow unsafe `catchError` usage in effects and epics.",
@@ -2600,9 +2660,9 @@ var noUnsafeCatchRule = ruleCreator({
 
 // src/rules/no-unsafe-first.ts
 import { stripIndent as stripIndent5 } from "common-tags";
-var defaultOptions9 = [];
+var defaultOptions10 = [];
 var noUnsafeFirstRule = ruleCreator({
-  defaultOptions: defaultOptions9,
+  defaultOptions: defaultOptions10,
   meta: {
     docs: {
       description: "Disallow unsafe `first`/`take` usage in effects and epics.",
@@ -2724,9 +2784,9 @@ var noUnsafeSubjectNext = ruleCreator({
 // src/rules/no-unsafe-switchmap.ts
 import { stripIndent as stripIndent6 } from "common-tags";
 import decamelize from "decamelize";
-var defaultOptions10 = [];
+var defaultOptions11 = [];
 var noUnsafeSwitchmapRule = ruleCreator({
-  defaultOptions: defaultOptions10,
+  defaultOptions: defaultOptions11,
   meta: {
     docs: {
       description: "Disallow unsafe `switchMap` usage in effects and epics.",
@@ -2849,7 +2909,7 @@ var noUnsafeSwitchmapRule = ruleCreator({
 
 // src/rules/no-unsafe-takeuntil.ts
 import { stripIndent as stripIndent7 } from "common-tags";
-var defaultOptions11 = [];
+var defaultOptions12 = [];
 var allowedOperators = [
   "count",
   "defaultIfEmpty",
@@ -2874,7 +2934,7 @@ var allowedOperators = [
   "toArray"
 ];
 var noUnsafeTakeuntilRule = ruleCreator({
-  defaultOptions: defaultOptions11,
+  defaultOptions: defaultOptions12,
   meta: {
     docs: {
       description: "Disallow applying operators after `takeUntil`.",
@@ -2953,13 +3013,13 @@ var noUnsafeTakeuntilRule = ruleCreator({
 });
 
 // src/rules/prefer-observer.ts
-var defaultOptions12 = [];
+var defaultOptions13 = [];
 var preferObserverRule = ruleCreator({
-  defaultOptions: defaultOptions12,
+  defaultOptions: defaultOptions13,
   meta: {
     docs: {
       description: "Disallow passing separate handlers to `subscribe` and `tap`.",
-      recommended: "strict",
+      recommended: "recommended",
       requiresTypeChecking: true
     },
     fixable: "code",
@@ -3069,7 +3129,7 @@ var preferRootOperatorsRule = ruleCreator({
   meta: {
     docs: {
       description: "Disallow importing operators from `rxjs/operators`.",
-      recommended: "strict"
+      recommended: "recommended"
     },
     fixable: "code",
     hasSuggestions: true,
@@ -3191,9 +3251,9 @@ var preferRootOperatorsRule = ruleCreator({
 
 // src/rules/suffix-subjects.ts
 import { AST_NODE_TYPES as AST_NODE_TYPES7, ESLintUtils as ESLintUtils10 } from "@typescript-eslint/utils";
-var defaultOptions13 = [];
+var defaultOptions14 = [];
 var suffixSubjectsRule = ruleCreator({
-  defaultOptions: defaultOptions13,
+  defaultOptions: defaultOptions14,
   meta: {
     docs: {
       description: "Enforce the use of a suffix in subject identifiers.",
@@ -3390,13 +3450,14 @@ var suffixSubjectsRule = ruleCreator({
 // src/rules/throw-error.ts
 import { ESLintUtils as ESLintUtils11 } from "@typescript-eslint/utils";
 import * as tsutils3 from "ts-api-utils";
-var defaultOptions14 = [];
+var defaultOptions15 = [];
 var throwErrorRule = ruleCreator({
-  defaultOptions: defaultOptions14,
+  defaultOptions: defaultOptions15,
   meta: {
     docs: {
       description: "Enforce passing only `Error` values to `throwError`.",
       recommended: {
+        recommended: true,
         strict: [{ allowThrowingAny: false, allowThrowingUnknown: false }]
       },
       requiresTypeChecking: true
@@ -3482,10 +3543,10 @@ var plugin = {
     "no-explicit-generics": noExplicitGenericsRule,
     "no-exposed-subjects": noExposedSubjectsRule,
     "no-finnish": noFinnishRule,
+    "no-floating-observables": noFloatingObservablesRule,
     "no-ignored-default-value": noIgnoredDefaultValueRule,
     "no-ignored-error": noIgnoredErrorRule,
     "no-ignored-notifier": noIgnoredNotifierRule,
-    "no-ignored-observable": noIgnoredObservableRule,
     "no-ignored-replay-buffer": noIgnoredReplayBufferRule,
     "no-ignored-subscribe": noIgnoredSubscribeRule,
     "no-ignored-subscription": noIgnoredSubscriptionRule,

package/docs/rules/no-floating-observables.md

@@ -0,0 +1,47 @@
+# Require Observables to be handled appropriately (`rxjs-x/no-floating-observables`)
+
+💼 This rule is enabled in the 🔒 `strict` config.
+
+💭 This rule requires [type information](https://typescript-eslint.io/linting/typed-linting).
+
+<!-- end auto-generated rule header -->
+
+A "floating" observable is one that is created without any code set up to handle any errors it might emit.
+Like a floating Promise, floating observables can cause several issues, such as ignored errors, unhandled cold observables, and more.
+
+This rule is like [no-floating-promises](https://typescript-eslint.io/rules/no-floating-promises/) but for Observables.
+This rule will report observable-valued statements that are not treated in one of the following ways:
+
+- Calling its `.subscribe()`
+- `return`ing it
+- Wrapping it in `lastValueFrom` or `firstValueFrom` and `await`ing it
+- [`void`ing it](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/void)
+
+> [!TIP]
+> `no-floating-observables` only detects apparently unhandled observable _statements_.
+
+## Rule details
+
+Examples of **incorrect** code for this rule:
+
+```ts
+import { of } from "rxjs";
+of(42, 54);
+```
+
+Examples of **correct** code for this rule:
+
+```ts
+import { of } from "rxjs";
+const answers = of(42, 54);
+```
+
+## Options
+
+<!-- begin auto-generated rule options list -->
+
+| Name         | Description                           | Type    | Default |
+| :----------- | :------------------------------------ | :------ | :------ |
+| `ignoreVoid` | Whether to ignore `void` expressions. | Boolean | `true`  |
+
+<!-- end auto-generated rule options list -->

package/docs/rules/no-topromise.md

@@ -1,6 +1,6 @@
 # Disallow use of the `toPromise` method (`rxjs-x/no-topromise`)
 
-💼 This rule is enabled in the 🔒 `strict` config.
+💼 This rule is enabled in the following configs: ✅ `recommended`, 🔒 `strict`.
 
 💡 This rule is manually fixable by [editor suggestions](https://eslint.org/docs/latest/use/core-concepts#rule-suggestions).
 

package/docs/rules/prefer-observer.md

@@ -1,6 +1,6 @@
 # Disallow passing separate handlers to `subscribe` and `tap` (`rxjs-x/prefer-observer`)
 
-💼 This rule is enabled in the 🔒 `strict` config.
+💼 This rule is enabled in the following configs: ✅ `recommended`, 🔒 `strict`.
 
 🔧💡 This rule is automatically fixable by the [`--fix` CLI option](https://eslint.org/docs/latest/user-guide/command-line-interface#--fix) and manually fixable by [editor suggestions](https://eslint.org/docs/latest/use/core-concepts#rule-suggestions).
 

package/docs/rules/prefer-root-operators.md

@@ -1,6 +1,6 @@
 # Disallow importing operators from `rxjs/operators` (`rxjs-x/prefer-root-operators`)
 
-💼 This rule is enabled in the 🔒 `strict` config.
+💼 This rule is enabled in the following configs: ✅ `recommended`, 🔒 `strict`.
 
 🔧💡 This rule is automatically fixable by the [`--fix` CLI option](https://eslint.org/docs/latest/user-guide/command-line-interface#--fix) and manually fixable by [editor suggestions](https://eslint.org/docs/latest/use/core-concepts#rule-suggestions).
 

package/docs/rules/throw-error.md

@@ -1,6 +1,6 @@
 # Enforce passing only `Error` values to `throwError` (`rxjs-x/throw-error`)
 
-💼 This rule is enabled in the 🔒 `strict` config.
+💼 This rule is enabled in the following configs: ✅ `recommended`, 🔒 `strict`.
 
 💭 This rule requires [type information](https://typescript-eslint.io/linting/typed-linting).
 

package/package.json

@@ -1,8 +1,8 @@
 {
   "name": "eslint-plugin-rxjs-x",
   "type": "commonjs",
-  "version": "0.4.0",
-  "packageManager": "yarn@4.5.1+sha512.341db9396b6e289fecc30cd7ab3af65060e05ebff4b3b47547b278b9e67b08f485ecd8c79006b405446262142c7a38154445ef7f17c1d5d1de7d90bf9ce7054d",
+  "version": "0.5.0",
+  "packageManager": "yarn@4.5.3+sha512.3003a14012e2987072d244c720506549c1aab73ee728208f1b2580a9fd67b92d61ba6b08fe93f6dce68fd771e3af1e59a0afa28dd242dd0940d73b95fedd4e90",
   "description": "ESLint v9+ plugin for RxJS",
   "author": "Jason Weinzierl <weinzierljason@gmail.com>",
   "license": "MIT",
@@ -47,7 +47,7 @@
     "lint-docs": "markdownlint-cli2 \"**/*.md\" \"#node_modules\"",
     "lint-eslint-docs": "yarn build && eslint-doc-generator --check",
     "docs": "eslint-doc-generator",
-    "release": "bumpp && yarn run build",
+    "release": "bumpp && echo \"Create a new release in GitHub to trigger the publish workflow.\"",
     "test": "vitest",
     "coverage": "vitest run --coverage",
     "typecheck": "tsc --noEmit"
@@ -62,7 +62,7 @@
   },
   "peerDependencies": {
     "eslint": "^8.57.0 || ^9.0.0",
-    "rxjs": ">=7.0.0",
+    "rxjs": ">=7.2.0",
     "typescript": ">=4.7.4"
   },
   "peerDependenciesMeta": {
@@ -72,10 +72,10 @@
   },
   "devDependencies": {
     "@eslint/js": "^9.14.0",
-    "@stylistic/eslint-plugin": "^2.10.1",
+    "@stylistic/eslint-plugin": "^2.11.0",
     "@types/common-tags": "^1.8.4",
     "@types/node": "~18.18.0",
-    "@typescript-eslint/rule-tester": "^8.14.0",
+    "@typescript-eslint/rule-tester": "^8.16.0",
     "@typescript/vfs": "^1.6.0",
     "@vitest/coverage-v8": "^2.1.5",
     "@vitest/eslint-plugin": "^1.1.10",
@@ -86,12 +86,12 @@
     "eslint-import-resolver-typescript": "^3.6.3",
     "eslint-plugin-eslint-plugin": "^6.3.2",
     "eslint-plugin-import-x": "^4.4.2",
-    "eslint-plugin-n": "^17.13.2",
+    "eslint-plugin-n": "^17.14.0",
     "markdownlint-cli2": "^0.15.0",
     "rxjs": "^7.8.1",
     "tsup": "^8.3.5",
-    "typescript": "~5.6.3",
-    "typescript-eslint": "^8.14.0",
+    "typescript": "~5.7.2",
+    "typescript-eslint": "^8.16.0",
     "vitest": "^2.1.5"
   },
   "engines": {

package/docs/rules/no-ignored-observable.md

@@ -1,25 +0,0 @@
-# Disallow ignoring observables returned by functions (`rxjs-x/no-ignored-observable`)
-
-💼 This rule is enabled in the 🔒 `strict` config.
-
-💭 This rule requires [type information](https://typescript-eslint.io/linting/typed-linting).
-
-<!-- end auto-generated rule header -->
-
-The effects failures if an observable returned by a function is neither assigned to a variable or property or passed to a function.
-
-## Rule details
-
-Examples of **incorrect** code for this rule:
-
-```ts
-import { of } from "rxjs";
-of(42, 54);
-```
-
-Examples of **correct** code for this rule:
-
-```ts
-import { of } from "rxjs";
-const answers = of(42, 54);
-```