feat(prefer-import-root-operators): separate message when no fixer provided

Warn about possible renames or partition being gone.

Author: JasonWeinzierl

src/rules/prefer-import-root-operators.ts

@@ -26,11 +26,12 @@ export const preferImportRootOperatorsRule = ruleCreator({
     fixable: 'code',
     hasSuggestions: true,
     messages: {
-      forbidden: 'RxJS imports from `rxjs/operators` are forbidden.',
-      suggest: 'Import from `rxjs` instead.',
+      forbidden: 'RxJS imports from `rxjs/operators` are forbidden; import from `rxjs` instead.',
+      forbiddenWithoutFix: 'RxJS imports from `rxjs/operators` are forbidden; import from `rxjs` instead. Note some operators may have been renamed or deprecated.',
+      suggest: 'Replace with import from `rxjs`.',
     },
     schema: [],
-    type: 'problem',
+    type: 'suggestion',
   },
   name: 'prefer-import-root-operators',
   create: (context) => {
@@ -117,15 +118,15 @@ export const preferImportRootOperatorsRule = ruleCreator({
       const replacement = getSourceReplacement(source.raw);
       if (!replacement || hasDeprecatedOperators(specifiers)) {
         context.report({
-          messageId: 'forbidden',
+          messageId: 'forbiddenWithoutFix',
           node: source,
         });
         return;
       }
 
       if (!specifiers) {
         context.report({
-          messageId: 'forbidden',
+          messageId: 'forbiddenWithoutFix',
           node: source,
           suggest: [{ messageId: 'suggest', fix: (fixer) => fixer.replaceText(source, replacement) }],
         });

tests/rules/prefer-import-root-operators.test.ts

@@ -89,14 +89,14 @@ ruleTester({ types: false }).run('prefer-import-root-operators', preferImportRoo
       stripIndent`
         // import declaration named, deprecated operator
         import { partition } from "rxjs/operators";
-                                  ~~~~~~~~~~~~~~~~ [forbidden]
+                                  ~~~~~~~~~~~~~~~~ [forbiddenWithoutFix]
       `,
     ),
     fromFixture(
       stripIndent`
         // import declaration namespace
         import * as RxOperators from "rxjs/operators";
-                                     ~~~~~~~~~~~~~~~~ [forbidden suggest]
+                                     ~~~~~~~~~~~~~~~~ [forbiddenWithoutFix suggest]
       `,
       {
         suggestions: [
@@ -114,7 +114,7 @@ ruleTester({ types: false }).run('prefer-import-root-operators', preferImportRoo
       stripIndent`
         // import declaration default
         import RxOperators, { map } from "rxjs/operators";
-                                         ~~~~~~~~~~~~~~~~ [forbidden suggest]
+                                         ~~~~~~~~~~~~~~~~ [forbiddenWithoutFix suggest]
       `,
       {
         suggestions: [
@@ -132,7 +132,7 @@ ruleTester({ types: false }).run('prefer-import-root-operators', preferImportRoo
       stripIndent`
         // import expression
         const { concat, merge: m, map } = await import("rxjs/operators");
-                                                       ~~~~~~~~~~~~~~~~ [forbidden suggest]
+                                                       ~~~~~~~~~~~~~~~~ [forbiddenWithoutFix suggest]
       `,
       {
         suggestions: [
@@ -150,7 +150,7 @@ ruleTester({ types: false }).run('prefer-import-root-operators', preferImportRoo
       stripIndent`
         // import expression, separated import
         const opPromise = import("rxjs/operators");
-                                 ~~~~~~~~~~~~~~~~ [forbidden suggest]
+                                 ~~~~~~~~~~~~~~~~ [forbiddenWithoutFix suggest]
         const { concat } = await opPromise;
       `,
       {
@@ -170,7 +170,7 @@ ruleTester({ types: false }).run('prefer-import-root-operators', preferImportRoo
       stripIndent`
         // import expression, deprecated operator
         const { concat, partition } = await import("rxjs/operators");
-                                                   ~~~~~~~~~~~~~~~~ [forbidden suggest]
+                                                   ~~~~~~~~~~~~~~~~ [forbiddenWithoutFix suggest]
       `,
       {
         suggestions: [
@@ -210,14 +210,14 @@ ruleTester({ types: false }).run('prefer-import-root-operators', preferImportRoo
       stripIndent`
         // export named, deprecated operator
         export { concat, partition } from "rxjs/operators";
-                                          ~~~~~~~~~~~~~~~~ [forbidden]
+                                          ~~~~~~~~~~~~~~~~ [forbiddenWithoutFix]
       `,
     ),
     fromFixture(
       stripIndent`
         // export all
         export * from "rxjs/operators";
-                      ~~~~~~~~~~~~~~~~ [forbidden suggest]
+                      ~~~~~~~~~~~~~~~~ [forbiddenWithoutFix suggest]
       `,
       {
         suggestions: [