Merge pull request #51 from airbnb/shar--refactor-index

Remove dependencies on babel-template and babel-plugin-syntax-dynamic-import

package.json

@@ -54,9 +54,5 @@
     "rimraf": "^2.6.2",
     "safe-publish-latest": "^1.1.1",
     "tape": "^4.9.0"
-  },
-  "dependencies": {
-    "babel-plugin-syntax-dynamic-import": "^6.18.0",
-    "babel-template": "^6.26.0"
   }
 }

src/index.js

@@ -1,23 +1,26 @@
-import template from 'babel-template';
-import syntax from 'babel-plugin-syntax-dynamic-import';
-
-const buildImport = template(`
+export default ({ template }) => {
+  const buildImport = template(`
   (new Promise((resolve) => {
     require.ensure([], (require) => {
       resolve(require(SOURCE));
     });
   }))
 `);
 
-export default () => ({
-  inherits: syntax,
-
-  visitor: {
-    Import(path) {
-      const newImport = buildImport({
-        SOURCE: path.parentPath.node.arguments,
-      });
-      path.parentPath.replaceWith(newImport);
+  return {
+    // NOTE: Once we drop support for Babel <= v6 we should
+    // update this to import from @babel/plugin-syntax-dynamic-import.
+    // https://www.npmjs.com/package/@babel/plugin-syntax-dynamic-import
+    manipulateOptions(opts, parserOpts) {
+      parserOpts.plugins.push('dynamicImport');
+    },
+    visitor: {
+      Import(path) {
+        const newImport = buildImport({
+          SOURCE: path.parentPath.node.arguments,
+        });
+        path.parentPath.replaceWith(newImport);
+      },
     },
-  },
-});
+  };
+};