fix: use path.posix instead of path module

Author: zacharygolba

package.json

@@ -43,6 +43,8 @@
     "nyc": "^5.6.0",
     "rimraf": "^2.5.2",
     "rollup": "^0.25.4",
-    "rollup-babel-lib-bundler": "^2.2.4"
-  }
+    "rollup-babel-lib-bundler": "^2.2.4",
+    "slash": "^1.0.0"
+  },
+  "dependencies": {}
 }

src/index.js

@@ -1,6 +1,12 @@
-import path from 'path';
+import { posix as path } from 'path';
+import { platform } from 'os';
 import fs from 'fs';
 
+import slash from 'slash';
+
+const VOLUME = /^([A-Z]:)/;
+const IS_WINDOWS = platform() === 'win32';
+
 // Helper functions
 const noop = () => null;
 const matches = (key, importee) => {
@@ -15,7 +21,7 @@ const matches = (key, importee) => {
   return importeeStartsWithKey && importeeHasSlashAfterKey;
 };
 const endsWith = (needle, haystack) => haystack.slice(-needle.length) === needle;
-const isFilePath = id => new RegExp(`^\\.?\\${path.sep}`).test(id);
+const isFilePath = id => /^\.?\//.test(id);
 const exists = uri => {
   try {
     return fs.statSync(uri).isFile();
@@ -24,6 +30,24 @@ const exists = uri => {
   }
 };
 
+const getVolume = id => {
+  let volume = '';
+
+  if (IS_WINDOWS && typeof id === 'string') {
+    [volume] = id.match(VOLUME) || [''];
+  }
+
+  return volume;
+};
+
+const normalizeId = id => {
+  if (IS_WINDOWS && typeof id === 'string') {
+    return slash(id.replace(VOLUME, ''));
+  }
+
+  return id;
+};
+
 export default function alias(options = {}) {
   const hasResolve = Array.isArray(options.resolve);
   const resolve = hasResolve ? options.resolve : ['.js'];
@@ -39,39 +63,43 @@ export default function alias(options = {}) {
 
   return {
     resolveId(importee, importer) {
+      const volume = getVolume(importee) || getVolume(importer);
+      const importeeId = normalizeId(importee);
+      const importerId = normalizeId(importer);
+
       // First match is supposed to be the correct one
-      const toReplace = aliasKeys.find(key => matches(key, importee));
+      const toReplace = aliasKeys.find(key => matches(key, importeeId));
 
       if (!toReplace) {
         return null;
       }
 
       const entry = options[toReplace];
 
-      const updatedId = importee.replace(toReplace, entry);
+      const updatedId = importeeId.replace(toReplace, entry);
 
       if (isFilePath(updatedId)) {
-        const directory = path.dirname(importer);
+        const directory = path.dirname(importerId);
 
         // Resolve file names
         const filePath = path.resolve(directory, updatedId);
         const match = resolve.map(ext => `${filePath}${ext}`)
                             .find(exists);
 
         if (match) {
-          return match;
+          return path.join(volume, match);
         }
 
         // To keep the previous behaviour we simply return the file path
         // with extension
         if (endsWith('.js', filePath)) {
-          return filePath;
+          return path.join(volume, filePath);
         }
 
-        return filePath + '.js';
+        return path.join(volume, filePath + '.js');
       }
 
-      return updatedId;
+      return path.join(volume, updatedId);
     },
   };
 }

test/index.js

@@ -2,6 +2,7 @@ import test from 'ava';
 import path from 'path';
 import { rollup } from 'rollup';
 import alias from '../dist/rollup-plugin-alias';
+import slash from 'slash';
 
 test(t => {
   t.is(typeof alias, 'function');
@@ -90,9 +91,12 @@ test('Test for the resolve property', t => {
     resolve: ['.js', '.jsx'],
   });
 
-  const resolved = result.resolveId('ember', path.resolve(__dirname, './files/index.js'));
+  const resolved = result.resolveId(
+    'ember',
+    slash(path.resolve(__dirname, './files/index.js'))
+  );
 
-  t.is(resolved, path.resolve(__dirname, './files/folder/hipster.jsx'));
+  t.is(resolved, slash(path.resolve(__dirname, './files/folder/hipster.jsx')));
 });
 
 test(t => {
@@ -110,9 +114,15 @@ test(t => {
     resolve: './i/am/a/local/file',
   });
 
-  const resolved = result.resolveId('resolve', path.resolve(__dirname, './files/index.js'));
+  const resolved = result.resolveId(
+    'resolve',
+    slash(path.resolve(__dirname, './files/index.js'))
+  );
 
-  t.is(resolved, path.resolve(__dirname, './files/i/am/a/local/file.js'));
+  t.is(
+    resolved,
+    slash(path.resolve(__dirname, './files/i/am/a/local/file.js'))
+  );
 });
 
 // Tests in Rollup

yarn.lock

@@ -1466,10 +1466,6 @@ esprima@2.7.x, esprima@^2.6.0, esprima@^2.7.1:
   version "2.7.3"
   resolved "https://registry.yarnpkg.com/esprima/-/esprima-2.7.3.tgz#96e3b70d5779f6ad49cd032673d1c312767ba581"
 
-esprima@^3.1.1:
-  version "3.1.3"
-  resolved "https://registry.yarnpkg.com/esprima/-/esprima-3.1.3.tgz#fdca51cee6133895e3c88d535ce49dbff62a4633"
-
 espurify@^1.6.0:
   version "1.7.0"
   resolved "https://registry.yarnpkg.com/espurify/-/espurify-1.7.0.tgz#1c5cf6cbccc32e6f639380bd4f991fab9ba9d226"
@@ -2184,20 +2180,13 @@ js-tokens@^3.0.0:
   version "3.0.1"
   resolved "https://registry.yarnpkg.com/js-tokens/-/js-tokens-3.0.1.tgz#08e9f132484a2c45a30907e9dc4d5567b7f114d7"
 
-js-yaml@3.6.1:
+js-yaml@3.6.1, js-yaml@3.x, js-yaml@^3.5.1:
   version "3.6.1"
   resolved "https://registry.yarnpkg.com/js-yaml/-/js-yaml-3.6.1.tgz#6e5fe67d8b205ce4d22fad05b7781e8dadcc4b30"
   dependencies:
     argparse "^1.0.7"
     esprima "^2.6.0"
 
-js-yaml@3.x, js-yaml@^3.5.1:
-  version "3.8.2"
-  resolved "https://registry.yarnpkg.com/js-yaml/-/js-yaml-3.8.2.tgz#02d3e2c0f6beab20248d412c352203827d786721"
-  dependencies:
-    argparse "^1.0.7"
-    esprima "^3.1.1"
-
 jsbn@~0.1.0:
   version "0.1.1"
   resolved "https://registry.yarnpkg.com/jsbn/-/jsbn-0.1.1.tgz#a5e654c2e5a2deb5f201d96cefbca80c0ef2f513"