tools: custom eslint autofix for inspector-check.js

test/parallel/test-eslint-inspector-check.js

@@ -12,12 +12,18 @@ const message = 'Please add a skipIfInspectorDisabled() call to allow this ' +
 new RuleTester().run('inspector-check', rule, {
   valid: [
     'foo;',
-    'common.skipIfInspectorDisabled(); require("inspector");'
+    'require("common")\n' +
+    'common.skipIfInspectorDisabled();\n' +
+    'require("inspector")'
   ],
   invalid: [
     {
-      code: 'require("inspector")',
-      errors: [{ message }]
+      code: 'require("common")\n' +
+            'require("inspector")',
+      errors: [{ message }],
+      output: 'require("common")\n' +
+              'common.skipIfInspectorDisabled();\n' +
+              'require("inspector")'
     }
   ]
 });

tools/eslint-rules/inspector-check.js

@@ -15,12 +15,17 @@ const msg = 'Please add a skipIfInspectorDisabled() call to allow this ' +
 
 module.exports = function(context) {
   const missingCheckNodes = [];
+  var commonModuleNode = null;
   var hasInspectorCheck = false;
 
   function testInspectorUsage(context, node) {
     if (utils.isRequired(node, ['inspector'])) {
       missingCheckNodes.push(node);
     }
+
+    if (utils.isCommonModule(node)) {
+      commonModuleNode = node;
+    }
   }
 
   function checkMemberExpression(context, node) {
@@ -32,7 +37,18 @@ module.exports = function(context) {
   function reportIfMissing(context) {
     if (!hasInspectorCheck) {
       missingCheckNodes.forEach((node) => {
-        context.report(node, msg);
+        context.report({
+          node,
+          message: msg,
+          fix: (fixer) => {
+            if (commonModuleNode) {
+              return fixer.insertTextAfter(
+                commonModuleNode,
+                '\ncommon.skipIfInspectorDisabled();'
+              );
+            }
+          }
+        });
       });
     }
   }

tools/eslint-rules/rules-utils.js

@@ -24,6 +24,16 @@ module.exports.isBinding = function(node, modules) {
   }
 };
 
+/**
+ * Return true if common module is required
+ * in AST Node under inspection
+ */
+var commonModuleRegExp = new RegExp(/^(\.\.\/)*common(\.js)?$/);
+module.exports.isCommonModule = function(node) {
+  return node.callee.name === 'require' &&
+         commonModuleRegExp.test(node.arguments[0].value);
+};
+
 /**
  * Returns true is the node accesses any property in the properties
  * array on the 'common' object.