feat: Aliasing a npm module without 'npm:' (#73)

Author: tleunen

README.md

@@ -32,7 +32,8 @@ Specify the plugin in your `.babelrc` with the custom root or alias. Here's an e
       ["module-resolver", {
         "root": ["./src"],
         "alias": {
-          "test": "./test"
+          "test": "./test",
+          "underscore": "lodash"
         }
       }]
     ]

src/index.js

@@ -57,7 +57,15 @@ export function mapModule(source, file, pluginOpts) {
         return null;
     }
 
+    // remove legacy "npm:" prefix for npm packages
+    aliasPath = aliasPath.replace(/^(npm:)/, '');
     const newPath = source.replace(moduleSplit.join('/'), aliasPath);
+
+    // alias to npm module don't need relative mapping
+    if (aliasPath[0] !== '.') {
+        return newPath;
+    }
+    // relative alias
     return mapToRelative(file, newPath);
 }
 

src/mapToRelative.js

@@ -20,12 +20,6 @@ export default function mapToRelative(currentFile, module) {
 
     moduleMapped = toPosixPath(moduleMapped);
 
-    // Support npm modules instead of directories
-    if (moduleMapped.indexOf('npm:') !== -1) {
-        const [, npmModuleName] = moduleMapped.split('npm:');
-        return npmModuleName;
-    }
-
     if (moduleMapped[0] !== '.') moduleMapped = `./${moduleMapped}`;
 
     return moduleMapped;

test/index.js

@@ -68,7 +68,8 @@ describe('alias', () => {
                 alias: {
                     utils: './src/mylib/subfolder/utils',
                     'awesome/components': './src/components',
-                    abstract: 'npm:concrete'
+                    abstract: 'npm:concrete',
+                    underscore: 'lodash'
                 }
             }]
         ]
@@ -138,11 +139,19 @@ describe('alias', () => {
         });
     });
 
-    describe('should support remapping to node modules with "npm:"', () => {
+    describe('(legacy) should support aliasing a node module with "npm:"', () => {
         testRequireImport(
             'abstract/thing',
             'concrete/thing',
             transformerOpts
         );
     });
+
+    describe('should support aliasing a node modules', () => {
+        testRequireImport(
+            'underscore/map',
+            'lodash/map',
+            transformerOpts
+        );
+    });
 });