support scoped modules containing hyphens

import-js · Feb 14, 2017 · 2fb246d · 2fb246d
1 parent 0800951
commit 2fb246d
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -13,6 +13,7 @@ This change log adheres to standards from [Keep a CHANGELOG](http://keepachangel
 
 ### Fixed
 - attempt to fix crash in [`no-mutable-exports`]. ([#660])
+- support scoped modules containing non word characters
 
 
 ## [2.2.0] - 2016-11-07

diff --git a/src/core/importType.js b/src/core/importType.js
@@ -27,7 +27,7 @@ function isExternalModule(name, settings, path) {
   return externalModuleRegExp.test(name) && isExternalPath(path, name, settings)
 }
 
-const scopedRegExp = /^@\w+\/\w+/
+const scopedRegExp = /^@[^\/]+\/[^\/]+/
 function isScoped(name) {
   return scopedRegExp.test(name)
 }

diff --git a/tests/src/core/importType.js b/tests/src/core/importType.js
@@ -31,6 +31,7 @@ describe('importType(name)', function () {
   it("should return 'external' for scopes packages", function() {
     expect(importType('@cycle/core', context)).to.equal('external')
     expect(importType('@cycle/dom', context)).to.equal('external')
+    expect(importType('@some-thing/something', context)).to.equal('external')
   })
 
   it("should return 'internal' for non-builtins resolved outside of node_modules", function () {