Remove hook that filters out Drupal dependencies, breaks externals

Author: danielamorse

lib/DrupalLibrariesPlugin.js

@@ -49,19 +49,6 @@ class DrupalLibrariesPlugin {
    *   The webpack compiler.
    */
   apply(compiler) {
-    // Prevents drupal library dependencies from being included.
-    compiler.hooks.normalModuleFactory.tap('DrupalLibrariesPlugin', cmf => {
-      cmf.hooks.factorize.tapAsync("DrupalLibrariesPlugin", (data, callback) => {
-        const result = this.opts.requirePattern.exec(data.request)
-        if (result) {
-          callback(null, new DrupalLibraryModule(result[1]))
-        }
-        else {
-          return callback()
-        }
-      })
-    })
-
     // Analyzes the final chunks to determine library dependencies.
     compiler.hooks.done.tapPromise('DrupalLibrariesPlugin', async stats => {
       this.metadata = await this._generateLibraryMetadata(stats.compilation)

test/library-dependency.test.js

@@ -6,6 +6,9 @@ test('Generates a library entry for @drupal(core/drupal)', async () => {
     entry: {
       'require-drupal': path.resolve(__dirname, './fixtures/require-drupal.es6.js'),
     },
+    externals: {
+      '@drupal(drupal/core)': 'Drupal'
+    }
   })).result
 
   expect(result).toEqual({