adjust suggestion locations

Author: fasttime

lib/processor.js

@@ -315,6 +315,13 @@ function postprocessMessages
         return true;
     }
 
+    function adjustFixRange(fix, eslintEnvInfos)
+    {
+        const range = convertToOriginalRange(fix.range, eslintEnvInfos);
+        if (range) fix.range = range;
+        return range;
+    }
+
     const postprocessedMessages = [];
     for (const message of messages)
     {
@@ -342,14 +349,20 @@ function postprocessMessages
             adjustLocation(message, 'endLine', 'endColumn')
         )
         {
-            const { fix } = message;
-            if (fix)
+            const { fix, suggestions } = message;
+            if (fix && !adjustFixRange(fix, eslintEnvInfos)) delete message.fix;
+            if (suggestions)
             {
-                const range = convertToOriginalRange(fix.range, eslintEnvInfos);
-                if (range)
-                    fix.range = range;
+                const processedSuggestions = [];
+                for (const suggestion of suggestions)
+                {
+                    if (adjustFixRange(suggestion.fix, eslintEnvInfos))
+                        processedSuggestions.push(suggestion);
+                }
+                if (processedSuggestions.length)
+                    message.suggestions = processedSuggestions;
                 else
-                    delete message.fix;
+                    delete message.suggestions;
             }
             postprocessedMessages.push(message);
         }

test/processor.js

@@ -724,5 +724,39 @@ describe
         );
 
         // #endregion
+
+        // #region Suggestions
+
+        it
+        (
+            'adjusts suggestion locations',
+            () =>
+            {
+                const code = '/* eslint-env jquery */ !a in b';
+                const config = { rules: { 'no-unsafe-negation': 'error' } };
+                const lintMessages = verifyWithProcessor(code, config);
+                assert.equal(lintMessages.length, 1);
+                assert.equal(lintMessages[0].ruleId, 'no-unsafe-negation');
+                assert.equal(lintMessages[0].suggestions.length, 2);
+                assert.deepEqual(lintMessages[0].suggestions[0].fix.range, [25, 31]);
+                assert.deepEqual(lintMessages[0].suggestions[1].fix.range, [24, 26]);
+            },
+        );
+
+        it
+        (
+            'suppresses suggestions',
+            () =>
+            {
+                const code = 'foo /* eslint-env jquery */ .isPrototypeOf(bar);';
+                const config = { rules: { 'no-prototype-builtins': 'error' } };
+                const lintMessages = verifyWithProcessor(code, config);
+                assert.equal(lintMessages.length, 1);
+                assert.equal(lintMessages[0].ruleId, 'no-prototype-builtins');
+                assert.equal(lintMessages[0].suggestions, undefined);
+            },
+        );
+
+        // #endregion
     },
 );