eslint-plugin-rxjs-x 0.9.2 → 0.9.4

package/dist/index.d.mts

@@ -258,9 +258,10 @@ declare const allRules: {
         description: "Disallow importing operators from `rxjs/operators`.";
         recommended: "recommended";
     }, TSESLint.RuleListener>;
-    'suffix-subjects': TSESLint.RuleModule<"forbidden", readonly {
+    'suffix-subjects': TSESLint.RuleModule<"forbidden" | "forbiddenProperty", readonly {
         parameters?: boolean;
         properties?: boolean;
+        objects?: boolean;
         suffix?: string;
         types?: Record<string, boolean>;
         variables?: boolean;

package/dist/index.d.ts

@@ -258,9 +258,10 @@ declare const allRules: {
         description: "Disallow importing operators from `rxjs/operators`.";
         recommended: "recommended";
     }, TSESLint.RuleListener>;
-    'suffix-subjects': TSESLint.RuleModule<"forbidden", readonly {
+    'suffix-subjects': TSESLint.RuleModule<"forbidden" | "forbiddenProperty", readonly {
         parameters?: boolean;
         properties?: boolean;
+        objects?: boolean;
         suffix?: string;
         types?: Record<string, boolean>;
         variables?: boolean;

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.9.2";
+var version = "0.9.4";
 
 // src/configs/recommended.ts
 var createRecommendedConfig = (plugin2) => ({
@@ -117,7 +117,10 @@ function couldBeType(type, name2, qualified) {
   if (tsutils.isTypeReference(type)) {
     type = type.target;
   }
-  if (isType(type, name2, qualified)) {
+  if (isType(type.symbol, name2, qualified)) {
+    return true;
+  }
+  if (type.aliasSymbol && isType(type.aliasSymbol, name2, qualified)) {
     return true;
   }
   if (tsutils.isUnionOrIntersectionType(type)) {
@@ -132,16 +135,16 @@ function couldBeType(type, name2, qualified) {
   }
   return false;
 }
-function isType(type, name2, qualified) {
-  if (!type.symbol) {
+function isType(symbol, name2, qualified) {
+  if (!symbol) {
     return false;
   }
   if (qualified && !qualified.name.test(
-    qualified.typeChecker.getFullyQualifiedName(type.symbol)
+    qualified.typeChecker.getFullyQualifiedName(symbol)
   )) {
     return false;
   }
-  return typeof name2 === "string" ? type.symbol.name === name2 : Boolean(type.symbol.name.match(name2));
+  return typeof name2 === "string" ? symbol.name === name2 : Boolean(symbol.name.match(name2));
 }
 function couldImplement(type, name2, qualified) {
   const { symbol } = type;
@@ -409,52 +412,6 @@ 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 MULTIPLE_OBSERVABLE_ACCEPTING_STATIC_OBSERVABLE_CREATORS = [
-  "combineLatest",
-  "forkJoin",
-  "merge",
-  "zip",
-  "concat",
-  "race"
-];
-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";
@@ -752,6 +709,9 @@ var finnishRule = ruleCreator({
       },
       "PropertyDefinition[computed=false]": (node) => {
         if (validate.properties) {
+          if (node.override) {
+            return;
+          }
           checkNode(node.key);
         }
       },
@@ -781,19 +741,44 @@ var finnishRule = ruleCreator({
       },
       "MethodDefinition[kind='get'][computed=false]": (node) => {
         if (validate.properties) {
+          if (node.override) {
+            return;
+          }
           checkNode(node.key, node);
         }
       },
       "MethodDefinition[kind='method'][computed=false]": (node) => {
         if (validate.methods) {
+          if (node.override) {
+            return;
+          }
           checkNode(node.key, node);
         }
       },
       "MethodDefinition[kind='set'][computed=false]": (node) => {
         if (validate.properties) {
+          if (node.override) {
+            return;
+          }
           checkNode(node.key, node);
         }
       },
+      "TSAbstractMethodDefinition[computed=false]": (node) => {
+        if (node.override) {
+          return;
+        }
+        if (validate.methods && node.kind === "method") {
+          checkNode(node.key, node);
+        }
+        if (validate.properties && (node.kind === "get" || node.kind === "set")) {
+          checkNode(node.key, node);
+        }
+        if (validate.parameters) {
+          node.value.params.forEach((param) => {
+            checkNode(param);
+          });
+        }
+      },
       "ObjectExpression > Property[computed=false] > Identifier": (node) => {
         if (!validate.objects) {
           return;
@@ -1147,6 +1132,49 @@ var noCreateRule = ruleCreator({
 
 
 
+
+// src/constants.ts
+var defaultObservable = String.raw`[Aa]ction(s|s\$|\$)$`;
+var SOURCES_OBJECT_ACCEPTING_STATIC_OBSERVABLE_CREATORS = [
+  "combineLatest",
+  "forkJoin"
+];
+var MULTIPLE_OBSERVABLE_ACCEPTING_STATIC_OBSERVABLE_CREATORS = [
+  "combineLatest",
+  "forkJoin",
+  "merge",
+  "zip",
+  "concat",
+  "race"
+];
+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/rules/no-cyclic-action.ts
 function isTypeReference2(type) {
   return Boolean(type.target);
 }
@@ -3389,7 +3417,7 @@ var noUnboundMethodsRule = ruleCreator({
     const { couldBeObservable, couldBeSubscription } = getTypeServices(context);
     const nodeMap = /* @__PURE__ */ new WeakMap();
     const [config = {}] = context.options;
-    const { allowTypes = [] } = config;
+    const { allowTypes = DEFAULT_UNBOUND_ALLOWED_TYPES } = config;
     function mapArguments(node) {
       node.arguments.filter(isMemberExpression).forEach((arg) => {
         const argType = getTypeAtLocation(arg);
@@ -3711,6 +3739,16 @@ var noUnsafeSubjectNext = ruleCreator({
 // src/rules/no-unsafe-switchmap.ts
 
 var _decamelize = require('decamelize'); var _decamelize2 = _interopRequireDefault(_decamelize);
+var DEFAULT_DISALLOW = [
+  "add",
+  "create",
+  "delete",
+  "post",
+  "put",
+  "remove",
+  "set",
+  "update"
+];
 var defaultOptions15 = [];
 var noUnsafeSwitchmapRule = ruleCreator({
   defaultOptions: defaultOptions15,
@@ -3737,7 +3775,8 @@ var noUnsafeSwitchmapRule = ruleCreator({
             oneOf: [
               { type: "string", description: "A regular expression string." },
               { type: "array", items: { type: "string" }, description: "An array of words." }
-            ]
+            ],
+            default: DEFAULT_DISALLOW
           },
           observable: {
             type: "string",
@@ -3760,29 +3799,15 @@ var noUnsafeSwitchmapRule = ruleCreator({
   name: "no-unsafe-switchmap",
   create: (context) => {
     var _a, _b, _c;
-    const defaultDisallow = [
-      "add",
-      "create",
-      "delete",
-      "post",
-      "put",
-      "remove",
-      "set",
-      "update"
-    ];
-    let allowRegExp;
-    let disallowRegExp;
-    let observableRegExp;
+    let allowRegExp = void 0;
+    let disallowRegExp = void 0;
     const [config = {}] = context.options;
-    if (config.allow || config.disallow) {
+    if (config.allow) {
       allowRegExp = createRegExpForWords((_a = config.allow) != null ? _a : []);
-      disallowRegExp = createRegExpForWords((_b = config.disallow) != null ? _b : []);
-      observableRegExp = new RegExp((_c = config.observable) != null ? _c : defaultObservable);
     } else {
-      allowRegExp = void 0;
-      disallowRegExp = createRegExpForWords(defaultDisallow);
-      observableRegExp = new RegExp(defaultObservable);
+      disallowRegExp = createRegExpForWords((_b = config.disallow) != null ? _b : DEFAULT_DISALLOW);
     }
+    const observableRegExp = new RegExp((_c = config.observable) != null ? _c : defaultObservable);
     const { couldBeObservable } = getTypeServices(context);
     function shouldDisallow(args) {
       const names = args.map((arg) => {
@@ -4139,6 +4164,7 @@ var preferRootOperatorsRule = ruleCreator({
 // src/rules/suffix-subjects.ts
 
 var defaultOptions18 = [];
+var baseShouldHaveSuffix = 'Subject identifiers must end with "{{suffix}}".';
 var suffixSubjectsRule = ruleCreator({
   defaultOptions: defaultOptions18,
   meta: {
@@ -4147,13 +4173,15 @@ var suffixSubjectsRule = ruleCreator({
       requiresTypeChecking: true
     },
     messages: {
-      forbidden: `Subject identifiers must end with "{{suffix}}".`
+      forbidden: baseShouldHaveSuffix,
+      forbiddenProperty: `${baseShouldHaveSuffix} Add a type annotation, assertion, or 'satisfies' to silence this rule.`
     },
     schema: [
       {
         properties: {
           parameters: { type: "boolean", description: "Require for parameters." },
-          properties: { type: "boolean", description: "Require for properties." },
+          properties: { type: "boolean", description: 'Require for properties, except object literal keys (see "objects" option).' },
+          objects: { type: "boolean", description: "Require for object literal keys." },
           suffix: { type: "string", description: "The suffix to enforce." },
           types: { type: "object", description: "Enforce for specific types. Keys are a RegExp, values are a boolean." },
           variables: { type: "boolean", description: "Require for variables." }
@@ -4171,6 +4199,7 @@ var suffixSubjectsRule = ruleCreator({
     const validate = {
       parameters: true,
       properties: true,
+      objects: true,
       variables: true,
       ...config
     };
@@ -4192,7 +4221,7 @@ var suffixSubjectsRule = ruleCreator({
       String.raw`${escapeRegExp(suffix)}\$?$`,
       "i"
     );
-    function checkNode(nameNode, typeNode) {
+    function checkNode(nameNode, typeNode, shouldMessage = "forbidden") {
       const tsNode = esTreeNodeToTSNodeMap.get(nameNode);
       const text = tsNode.getText();
       if (!suffixRegex.test(text) && couldBeType2(typeNode != null ? typeNode : nameNode, "Subject")) {
@@ -4205,7 +4234,7 @@ var suffixSubjectsRule = ruleCreator({
         context.report({
           data: { suffix },
           loc: getLoc(tsNode),
-          messageId: "forbidden"
+          messageId: shouldMessage
         });
       }
     }
@@ -4221,7 +4250,7 @@ var suffixSubjectsRule = ruleCreator({
         if (!found) {
           return;
         }
-        if (!validate.variables && found.type === _utils.AST_NODE_TYPES.VariableDeclarator) {
+        if (!validate.variables && isVariableDeclarator(found)) {
           return;
         }
         if (!validate.parameters) {
@@ -4240,6 +4269,9 @@ var suffixSubjectsRule = ruleCreator({
       "PropertyDefinition[computed=false]": (node) => {
         const anyNode = node;
         if (validate.properties) {
+          if (node.override) {
+            return;
+          }
           checkNode(anyNode.key);
         }
       },
@@ -4261,21 +4293,58 @@ var suffixSubjectsRule = ruleCreator({
       },
       "MethodDefinition[kind='get'][computed=false]": (node) => {
         if (validate.properties) {
+          if (node.override) {
+            return;
+          }
           checkNode(node.key, node);
         }
       },
       "MethodDefinition[kind='set'][computed=false]": (node) => {
         if (validate.properties) {
+          if (node.override) {
+            return;
+          }
           checkNode(node.key, node);
         }
       },
+      "TSAbstractMethodDefinition[computed=false]": (node) => {
+        if (node.override) {
+          return;
+        }
+        if (validate.properties && (node.kind === "get" || node.kind === "set")) {
+          checkNode(node.key, node);
+        }
+        if (validate.parameters) {
+          node.value.params.forEach((param) => {
+            checkNode(param);
+          });
+        }
+      },
       "ObjectExpression > Property[computed=false] > Identifier": (node) => {
-        if (validate.properties) {
-          const parent = node.parent;
-          if (node === parent.key && !isSourcesObjectAcceptingStaticObservableCreator(parent.parent.parent)) {
-            checkNode(node);
+        if (!validate.objects) {
+          return;
+        }
+        const parent = node.parent;
+        if (node !== parent.key) {
+          return;
+        }
+        const found = findParent(
+          node,
+          "CallExpression",
+          "VariableDeclarator",
+          "TSSatisfiesExpression",
+          "TSAsExpression"
+        );
+        if (found) {
+          if (isCallExpression(found)) {
+            return;
+          } else if (isVariableDeclarator(found) && !!found.id.typeAnnotation) {
+            return;
+          } else if (isTSAsExpression(found) || isTSSatisfiesExpression(found)) {
+            return;
           }
         }
+        checkNode(node, void 0, "forbiddenProperty");
       },
       "ObjectPattern > Property > Identifier": (node) => {
         const found = findParent(
@@ -4288,7 +4357,7 @@ var suffixSubjectsRule = ruleCreator({
         if (!found) {
           return;
         }
-        if (!validate.variables && found.type === _utils.AST_NODE_TYPES.VariableDeclarator) {
+        if (!validate.variables && isVariableDeclarator(found)) {
           return;
         }
         if (!validate.parameters) {

package/dist/index.mjs

@@ -1,6 +1,6 @@
 // package.json
 var name = "eslint-plugin-rxjs-x";
-var version = "0.9.2";
+var version = "0.9.4";
 
 // src/configs/recommended.ts
 var createRecommendedConfig = (plugin2) => ({
@@ -117,7 +117,10 @@ function couldBeType(type, name2, qualified) {
   if (tsutils.isTypeReference(type)) {
     type = type.target;
   }
-  if (isType(type, name2, qualified)) {
+  if (isType(type.symbol, name2, qualified)) {
+    return true;
+  }
+  if (type.aliasSymbol && isType(type.aliasSymbol, name2, qualified)) {
     return true;
   }
   if (tsutils.isUnionOrIntersectionType(type)) {
@@ -132,16 +135,16 @@ function couldBeType(type, name2, qualified) {
   }
   return false;
 }
-function isType(type, name2, qualified) {
-  if (!type.symbol) {
+function isType(symbol, name2, qualified) {
+  if (!symbol) {
     return false;
   }
   if (qualified && !qualified.name.test(
-    qualified.typeChecker.getFullyQualifiedName(type.symbol)
+    qualified.typeChecker.getFullyQualifiedName(symbol)
   )) {
     return false;
   }
-  return typeof name2 === "string" ? type.symbol.name === name2 : Boolean(type.symbol.name.match(name2));
+  return typeof name2 === "string" ? symbol.name === name2 : Boolean(symbol.name.match(name2));
 }
 function couldImplement(type, name2, qualified) {
   const { symbol } = type;
@@ -409,52 +412,6 @@ 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 MULTIPLE_OBSERVABLE_ACCEPTING_STATIC_OBSERVABLE_CREATORS = [
-  "combineLatest",
-  "forkJoin",
-  "merge",
-  "zip",
-  "concat",
-  "race"
-];
-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";
@@ -752,6 +709,9 @@ var finnishRule = ruleCreator({
       },
       "PropertyDefinition[computed=false]": (node) => {
         if (validate.properties) {
+          if (node.override) {
+            return;
+          }
           checkNode(node.key);
         }
       },
@@ -781,19 +741,44 @@ var finnishRule = ruleCreator({
       },
       "MethodDefinition[kind='get'][computed=false]": (node) => {
         if (validate.properties) {
+          if (node.override) {
+            return;
+          }
           checkNode(node.key, node);
         }
       },
       "MethodDefinition[kind='method'][computed=false]": (node) => {
         if (validate.methods) {
+          if (node.override) {
+            return;
+          }
           checkNode(node.key, node);
         }
       },
       "MethodDefinition[kind='set'][computed=false]": (node) => {
         if (validate.properties) {
+          if (node.override) {
+            return;
+          }
           checkNode(node.key, node);
         }
       },
+      "TSAbstractMethodDefinition[computed=false]": (node) => {
+        if (node.override) {
+          return;
+        }
+        if (validate.methods && node.kind === "method") {
+          checkNode(node.key, node);
+        }
+        if (validate.properties && (node.kind === "get" || node.kind === "set")) {
+          checkNode(node.key, node);
+        }
+        if (validate.parameters) {
+          node.value.params.forEach((param) => {
+            checkNode(param);
+          });
+        }
+      },
       "ObjectExpression > Property[computed=false] > Identifier": (node) => {
         if (!validate.objects) {
           return;
@@ -1147,6 +1132,49 @@ var noCreateRule = ruleCreator({
 import { ESLintUtils as ESLintUtils4 } from "@typescript-eslint/utils";
 import { stripIndent as stripIndent3 } from "common-tags";
 import ts4 from "typescript";
+
+// src/constants.ts
+var defaultObservable = String.raw`[Aa]ction(s|s\$|\$)$`;
+var SOURCES_OBJECT_ACCEPTING_STATIC_OBSERVABLE_CREATORS = [
+  "combineLatest",
+  "forkJoin"
+];
+var MULTIPLE_OBSERVABLE_ACCEPTING_STATIC_OBSERVABLE_CREATORS = [
+  "combineLatest",
+  "forkJoin",
+  "merge",
+  "zip",
+  "concat",
+  "race"
+];
+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/rules/no-cyclic-action.ts
 function isTypeReference2(type) {
   return Boolean(type.target);
 }
@@ -3389,7 +3417,7 @@ var noUnboundMethodsRule = ruleCreator({
     const { couldBeObservable, couldBeSubscription } = getTypeServices(context);
     const nodeMap = /* @__PURE__ */ new WeakMap();
     const [config = {}] = context.options;
-    const { allowTypes = [] } = config;
+    const { allowTypes = DEFAULT_UNBOUND_ALLOWED_TYPES } = config;
     function mapArguments(node) {
       node.arguments.filter(isMemberExpression).forEach((arg) => {
         const argType = getTypeAtLocation(arg);
@@ -3711,6 +3739,16 @@ var noUnsafeSubjectNext = ruleCreator({
 // src/rules/no-unsafe-switchmap.ts
 import { stripIndent as stripIndent7 } from "common-tags";
 import decamelize from "decamelize";
+var DEFAULT_DISALLOW = [
+  "add",
+  "create",
+  "delete",
+  "post",
+  "put",
+  "remove",
+  "set",
+  "update"
+];
 var defaultOptions15 = [];
 var noUnsafeSwitchmapRule = ruleCreator({
   defaultOptions: defaultOptions15,
@@ -3737,7 +3775,8 @@ var noUnsafeSwitchmapRule = ruleCreator({
             oneOf: [
               { type: "string", description: "A regular expression string." },
               { type: "array", items: { type: "string" }, description: "An array of words." }
-            ]
+            ],
+            default: DEFAULT_DISALLOW
           },
           observable: {
             type: "string",
@@ -3760,29 +3799,15 @@ var noUnsafeSwitchmapRule = ruleCreator({
   name: "no-unsafe-switchmap",
   create: (context) => {
     var _a, _b, _c;
-    const defaultDisallow = [
-      "add",
-      "create",
-      "delete",
-      "post",
-      "put",
-      "remove",
-      "set",
-      "update"
-    ];
-    let allowRegExp;
-    let disallowRegExp;
-    let observableRegExp;
+    let allowRegExp = void 0;
+    let disallowRegExp = void 0;
     const [config = {}] = context.options;
-    if (config.allow || config.disallow) {
+    if (config.allow) {
       allowRegExp = createRegExpForWords((_a = config.allow) != null ? _a : []);
-      disallowRegExp = createRegExpForWords((_b = config.disallow) != null ? _b : []);
-      observableRegExp = new RegExp((_c = config.observable) != null ? _c : defaultObservable);
     } else {
-      allowRegExp = void 0;
-      disallowRegExp = createRegExpForWords(defaultDisallow);
-      observableRegExp = new RegExp(defaultObservable);
+      disallowRegExp = createRegExpForWords((_b = config.disallow) != null ? _b : DEFAULT_DISALLOW);
     }
+    const observableRegExp = new RegExp((_c = config.observable) != null ? _c : defaultObservable);
     const { couldBeObservable } = getTypeServices(context);
     function shouldDisallow(args) {
       const names = args.map((arg) => {
@@ -4137,8 +4162,9 @@ var preferRootOperatorsRule = ruleCreator({
 });
 
 // src/rules/suffix-subjects.ts
-import { AST_NODE_TYPES as AST_NODE_TYPES8, ESLintUtils as ESLintUtils12 } from "@typescript-eslint/utils";
+import { ESLintUtils as ESLintUtils12 } from "@typescript-eslint/utils";
 var defaultOptions18 = [];
+var baseShouldHaveSuffix = 'Subject identifiers must end with "{{suffix}}".';
 var suffixSubjectsRule = ruleCreator({
   defaultOptions: defaultOptions18,
   meta: {
@@ -4147,13 +4173,15 @@ var suffixSubjectsRule = ruleCreator({
       requiresTypeChecking: true
     },
     messages: {
-      forbidden: `Subject identifiers must end with "{{suffix}}".`
+      forbidden: baseShouldHaveSuffix,
+      forbiddenProperty: `${baseShouldHaveSuffix} Add a type annotation, assertion, or 'satisfies' to silence this rule.`
     },
     schema: [
       {
         properties: {
           parameters: { type: "boolean", description: "Require for parameters." },
-          properties: { type: "boolean", description: "Require for properties." },
+          properties: { type: "boolean", description: 'Require for properties, except object literal keys (see "objects" option).' },
+          objects: { type: "boolean", description: "Require for object literal keys." },
           suffix: { type: "string", description: "The suffix to enforce." },
           types: { type: "object", description: "Enforce for specific types. Keys are a RegExp, values are a boolean." },
           variables: { type: "boolean", description: "Require for variables." }
@@ -4171,6 +4199,7 @@ var suffixSubjectsRule = ruleCreator({
     const validate = {
       parameters: true,
       properties: true,
+      objects: true,
       variables: true,
       ...config
     };
@@ -4192,7 +4221,7 @@ var suffixSubjectsRule = ruleCreator({
       String.raw`${escapeRegExp(suffix)}\$?$`,
       "i"
     );
-    function checkNode(nameNode, typeNode) {
+    function checkNode(nameNode, typeNode, shouldMessage = "forbidden") {
       const tsNode = esTreeNodeToTSNodeMap.get(nameNode);
       const text = tsNode.getText();
       if (!suffixRegex.test(text) && couldBeType2(typeNode != null ? typeNode : nameNode, "Subject")) {
@@ -4205,7 +4234,7 @@ var suffixSubjectsRule = ruleCreator({
         context.report({
           data: { suffix },
           loc: getLoc(tsNode),
-          messageId: "forbidden"
+          messageId: shouldMessage
         });
       }
     }
@@ -4221,7 +4250,7 @@ var suffixSubjectsRule = ruleCreator({
         if (!found) {
           return;
         }
-        if (!validate.variables && found.type === AST_NODE_TYPES8.VariableDeclarator) {
+        if (!validate.variables && isVariableDeclarator(found)) {
           return;
         }
         if (!validate.parameters) {
@@ -4240,6 +4269,9 @@ var suffixSubjectsRule = ruleCreator({
       "PropertyDefinition[computed=false]": (node) => {
         const anyNode = node;
         if (validate.properties) {
+          if (node.override) {
+            return;
+          }
           checkNode(anyNode.key);
         }
       },
@@ -4261,21 +4293,58 @@ var suffixSubjectsRule = ruleCreator({
       },
       "MethodDefinition[kind='get'][computed=false]": (node) => {
         if (validate.properties) {
+          if (node.override) {
+            return;
+          }
           checkNode(node.key, node);
         }
       },
       "MethodDefinition[kind='set'][computed=false]": (node) => {
         if (validate.properties) {
+          if (node.override) {
+            return;
+          }
           checkNode(node.key, node);
         }
       },
+      "TSAbstractMethodDefinition[computed=false]": (node) => {
+        if (node.override) {
+          return;
+        }
+        if (validate.properties && (node.kind === "get" || node.kind === "set")) {
+          checkNode(node.key, node);
+        }
+        if (validate.parameters) {
+          node.value.params.forEach((param) => {
+            checkNode(param);
+          });
+        }
+      },
       "ObjectExpression > Property[computed=false] > Identifier": (node) => {
-        if (validate.properties) {
-          const parent = node.parent;
-          if (node === parent.key && !isSourcesObjectAcceptingStaticObservableCreator(parent.parent.parent)) {
-            checkNode(node);
+        if (!validate.objects) {
+          return;
+        }
+        const parent = node.parent;
+        if (node !== parent.key) {
+          return;
+        }
+        const found = findParent(
+          node,
+          "CallExpression",
+          "VariableDeclarator",
+          "TSSatisfiesExpression",
+          "TSAsExpression"
+        );
+        if (found) {
+          if (isCallExpression(found)) {
+            return;
+          } else if (isVariableDeclarator(found) && !!found.id.typeAnnotation) {
+            return;
+          } else if (isTSAsExpression(found) || isTSSatisfiesExpression(found)) {
+            return;
           }
         }
+        checkNode(node, void 0, "forbiddenProperty");
       },
       "ObjectPattern > Property > Identifier": (node) => {
         const found = findParent(
@@ -4288,7 +4357,7 @@ var suffixSubjectsRule = ruleCreator({
         if (!found) {
           return;
         }
-        if (!validate.variables && found.type === AST_NODE_TYPES8.VariableDeclarator) {
+        if (!validate.variables && isVariableDeclarator(found)) {
           return;
         }
         if (!validate.parameters) {

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "eslint-plugin-rxjs-x",
   "type": "commonjs",
-  "version": "0.9.2",
+  "version": "0.9.4",
   "packageManager": "yarn@4.12.0+sha512.f45ab632439a67f8bc759bf32ead036a1f413287b9042726b7cc4818b7b49e14e9423ba49b18f9e06ea4941c1ad062385b1d8760a8d5091a1a31e5f6219afca8",
   "description": "Modern ESLint plugin for RxJS",
   "author": "Jason Weinzierl <weinzierljason@gmail.com>",
@@ -89,7 +89,7 @@
     "eslint-config-flat-gitignore": "^2.1.0",
     "eslint-doc-generator": "^2.4.0",
     "eslint-import-resolver-typescript": "^4.4.4",
-    "eslint-plugin-eslint-plugin": "^7.3.0",
+    "eslint-plugin-eslint-plugin": "patch:eslint-plugin-eslint-plugin@npm%3A7.3.0#~/.yarn/patches/eslint-plugin-eslint-plugin-npm-7.3.0-d800475165.patch",
     "eslint-plugin-import-x": "^4.16.1",
     "eslint-plugin-n": "^17.23.2",
     "markdownlint-cli2": "~0.19.0",