Ignore noop dependencies

src/visitors/dependencies.js

@@ -38,7 +38,8 @@ module.exports = {
       callee.name === 'require' &&
       args.length === 1 &&
       types.isStringLiteral(args[0]) &&
-      !hasBinding(ancestors, 'require');
+      !hasBinding(ancestors, 'require') &&
+      !hasNoOpParent(ancestors);
 
     if (isRequire) {
       let optional = ancestors.some(a => types.isTryStatement(a)) || undefined;
@@ -89,6 +90,19 @@ module.exports = {
   }
 };
 
+function hasNoOpParent(ancestors) {
+  return ancestors.some(node => {
+    if (
+      types.isIfStatement(node) &&
+      types.isLiteral(node.test) &&
+      !node.test.value
+    ) {
+      // only ignore if it's in the consequent branch
+      return ancestors.indexOf(node.consequent) !== -1;
+    }
+  });
+}
+
 function hasBinding(node, name) {
   if (Array.isArray(node)) {
     return node.some(ancestor => hasBinding(ancestor, name));

test/integration/optional-dep/index.js

@@ -1,5 +1,19 @@
 try {
-  require('optional-dep');
+  require('try-optional-dep');
 } catch (err) {
   module.exports = err;
 }
+
+if(false) {
+  require('if-false-optional-dep');
+}
+
+if(false) {
+  globalStuff(() =>
+    require('if-false-optional-dep-deep')
+  )
+}
+
+if('') {
+  require('if-falsy-optional-dep');
+}