support scoped modules containing hyphens

Author: rosswarren

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

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)
 }

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 () {