eslint-plugin-rxjs-x 0.2.0 → 0.2.1

package/README.md

@@ -17,7 +17,7 @@ Almost all of these rules require the TypeScript parser for ESLint and are indic
 
 See [typescript-eslint's Getting Started](https://typescript-eslint.io/getting-started) for a full ESLint setup guide.
 
-Then use the `recommended` configuration in your `eslint.config.mjs`:
+Then use the `recommended` configuration in your `eslint.config.mjs` and enable typed linting:
 
 ```js
 // @ts-check
@@ -26,6 +26,7 @@ import rxjsX from 'eslint-plugin-rxjs-x';
 
 export default tseslint.config({
     extends: [
+        ...tseslint.configs.recommended,
         rxjsX.configs.recommended,
     ],
     languageOptions: {
@@ -36,6 +37,10 @@ export default tseslint.config({
 });
 ```
 
+The above example uses `typescript-eslint`'s built-in config to set up the TypeScript parser for us.
+Enabling `projectService` then turns on typed linting.
+See [Linting with Type Information](https://typescript-eslint.io/getting-started/typed-linting/) for details.
+
 ## Configs
 
 <!-- begin auto-generated configs list -->

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.0";
+var version = "0.2.1";
 
 // src/configs/recommended.ts
 var createRecommendedConfig = (plugin2) => ({
@@ -518,7 +518,7 @@ var finnishRule = ruleCreator({
         });
       } : () => {
       };
-      if (couldBeObservable(typeNode || nameNode) || couldReturnObservable(typeNode || nameNode)) {
+      if (couldBeObservable(typeNode != null ? typeNode : nameNode) || couldReturnObservable(typeNode != null ? typeNode : nameNode)) {
         for (const name2 of names) {
           const { regExp, validate: validate2 } = name2;
           if (regExp.test(text) && !validate2) {
@@ -528,7 +528,7 @@ var finnishRule = ruleCreator({
         }
         for (const type of types) {
           const { regExp, validate: validate2 } = type;
-          if ((couldBeType2(typeNode || nameNode, regExp) || couldReturnType(typeNode || nameNode, regExp)) && !validate2) {
+          if ((couldBeType2(typeNode != null ? typeNode : nameNode, regExp) || couldReturnType(typeNode != null ? typeNode : nameNode, regExp)) && !validate2) {
             shouldNotBeFinnish();
             return;
           }
@@ -1201,7 +1201,7 @@ var noFinnishRule = ruleCreator({
     const { esTreeNodeToTSNodeMap } = _utils.ESLintUtils.getParserServices(context);
     const { couldBeObservable, couldReturnObservable } = getTypeServices(context);
     function checkNode(nameNode, typeNode) {
-      if (couldBeObservable(typeNode || nameNode) || couldReturnObservable(typeNode || nameNode)) {
+      if (couldBeObservable(typeNode != null ? typeNode : nameNode) || couldReturnObservable(typeNode != null ? typeNode : nameNode)) {
         const tsNode = esTreeNodeToTSNodeMap.get(nameNode);
         if (/[$]+$/.test(tsNode.getText())) {
           context.report({
@@ -2838,10 +2838,10 @@ var suffixSubjectsRule = ruleCreator({
     function checkNode(nameNode, typeNode) {
       const tsNode = esTreeNodeToTSNodeMap.get(nameNode);
       const text = tsNode.getText();
-      if (!suffixRegex.test(text) && couldBeType2(typeNode || nameNode, "Subject")) {
+      if (!suffixRegex.test(text) && couldBeType2(typeNode != null ? typeNode : nameNode, "Subject")) {
         for (const type of types) {
           const { regExp, validate: validate2 } = type;
-          if (couldBeType2(typeNode || nameNode, regExp) && !validate2) {
+          if (couldBeType2(typeNode != null ? typeNode : nameNode, regExp) && !validate2) {
             return;
           }
         }

package/dist/index.mjs

@@ -1,6 +1,6 @@
 // package.json
 var name = "eslint-plugin-rxjs-x";
-var version = "0.2.0";
+var version = "0.2.1";
 
 // src/configs/recommended.ts
 var createRecommendedConfig = (plugin2) => ({
@@ -518,7 +518,7 @@ var finnishRule = ruleCreator({
         });
       } : () => {
       };
-      if (couldBeObservable(typeNode || nameNode) || couldReturnObservable(typeNode || nameNode)) {
+      if (couldBeObservable(typeNode != null ? typeNode : nameNode) || couldReturnObservable(typeNode != null ? typeNode : nameNode)) {
         for (const name2 of names) {
           const { regExp, validate: validate2 } = name2;
           if (regExp.test(text) && !validate2) {
@@ -528,7 +528,7 @@ var finnishRule = ruleCreator({
         }
         for (const type of types) {
           const { regExp, validate: validate2 } = type;
-          if ((couldBeType2(typeNode || nameNode, regExp) || couldReturnType(typeNode || nameNode, regExp)) && !validate2) {
+          if ((couldBeType2(typeNode != null ? typeNode : nameNode, regExp) || couldReturnType(typeNode != null ? typeNode : nameNode, regExp)) && !validate2) {
             shouldNotBeFinnish();
             return;
           }
@@ -1201,7 +1201,7 @@ var noFinnishRule = ruleCreator({
     const { esTreeNodeToTSNodeMap } = ESLintUtils4.getParserServices(context);
     const { couldBeObservable, couldReturnObservable } = getTypeServices(context);
     function checkNode(nameNode, typeNode) {
-      if (couldBeObservable(typeNode || nameNode) || couldReturnObservable(typeNode || nameNode)) {
+      if (couldBeObservable(typeNode != null ? typeNode : nameNode) || couldReturnObservable(typeNode != null ? typeNode : nameNode)) {
         const tsNode = esTreeNodeToTSNodeMap.get(nameNode);
         if (/[$]+$/.test(tsNode.getText())) {
           context.report({
@@ -2838,10 +2838,10 @@ var suffixSubjectsRule = ruleCreator({
     function checkNode(nameNode, typeNode) {
       const tsNode = esTreeNodeToTSNodeMap.get(nameNode);
       const text = tsNode.getText();
-      if (!suffixRegex.test(text) && couldBeType2(typeNode || nameNode, "Subject")) {
+      if (!suffixRegex.test(text) && couldBeType2(typeNode != null ? typeNode : nameNode, "Subject")) {
         for (const type of types) {
           const { regExp, validate: validate2 } = type;
-          if (couldBeType2(typeNode || nameNode, regExp) && !validate2) {
+          if (couldBeType2(typeNode != null ? typeNode : nameNode, regExp) && !validate2) {
             return;
           }
         }

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "eslint-plugin-rxjs-x",
   "type": "commonjs",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "packageManager": "yarn@4.5.1+sha512.341db9396b6e289fecc30cd7ab3af65060e05ebff4b3b47547b278b9e67b08f485ecd8c79006b405446262142c7a38154445ef7f17c1d5d1de7d90bf9ce7054d",
   "description": "ESLint v9+ plugin for RxJS",
   "author": "Jason Weinzierl <weinzierljason@gmail.com>",
@@ -53,8 +53,8 @@
     "typecheck": "tsc --noEmit"
   },
   "dependencies": {
-    "@typescript-eslint/scope-manager": "^8.12.2",
-    "@typescript-eslint/utils": "^8.12.2",
+    "@typescript-eslint/scope-manager": "^8.1.0",
+    "@typescript-eslint/utils": "^8.1.0",
     "common-tags": "^1.8.0",
     "decamelize": "^5.0.1",
     "ts-api-utils": "^1.3.0",
@@ -66,28 +66,28 @@
     "typescript": ">=4.2.0"
   },
   "devDependencies": {
-    "@eslint/js": "^9.13.0",
+    "@eslint/js": "^9.14.0",
     "@stylistic/eslint-plugin": "^2.10.1",
     "@types/common-tags": "^1.8.4",
     "@types/node": "^18.18.0",
-    "@typescript-eslint/rule-tester": "^8.12.2",
+    "@typescript-eslint/rule-tester": "^8.13.0",
     "@typescript/vfs": "^1.6.0",
     "@vitest/coverage-v8": "^2.1.4",
     "@vitest/eslint-plugin": "^1.1.7",
     "bumpp": "^9.8.0",
-    "eslint": "^9.13.0",
+    "eslint": "^9.14.0",
     "eslint-config-flat-gitignore": "^0.3.0",
     "eslint-doc-generator": "^1.7.1",
     "eslint-import-resolver-typescript": "^3.6.3",
     "eslint-plugin-eslint-plugin": "^6.3.1",
     "eslint-plugin-import-x": "^4.4.0",
-    "eslint-plugin-n": "^17.12.0",
+    "eslint-plugin-n": "^17.13.1",
     "markdownlint-cli2": "^0.14.0",
     "rxjs": "^7.0.0",
     "tsup": "^8.3.5",
     "tsx": "^4.19.2",
     "typescript": "~5.6.3",
-    "typescript-eslint": "^8.12.2",
+    "typescript-eslint": "^8.13.0",
     "vitest": "^2.1.4"
   },
   "engines": {