Fix hoisting of optional require calls

src/packagers/JSConcatPackager.js

@@ -94,11 +94,13 @@ class JSConcatPackager extends Packager {
       let [source, name] = asset.cacheData.imports[identifier];
       let dep = asset.depAssets.get(asset.dependencies.get(source));
 
-      if (name === '*') {
-        this.markUsedExports(dep);
-      }
+      if (dep) {
+        if (name === '*') {
+          this.markUsedExports(dep);
+        }
 
-      this.markUsed(dep, name);
+        this.markUsed(dep, name);
+      }
     }
   }
 

test/integration/scope-hoisting/commonjs/wrap-optional/a.js

@@ -0,0 +1,6 @@
+try {
+    output = require('noop')
+}
+catch(_) {
+    output = 42
+}

test/scope-hoisting.js

@@ -1053,5 +1053,17 @@ describe('scope hoisting', function() {
       let output = await run(b);
       assert.deepEqual(output, 9);
     });
+
+    it('should support optional requires', async function() {
+      let b = await bundle(
+        path.join(
+          __dirname,
+          '/integration/scope-hoisting/commonjs/wrap-optional/a.js'
+        )
+      );
+
+      let output = await run(b);
+      assert.deepEqual(output, 42);
+    });
   });
 });