Update behaviour for external modules

src/rules/orderedImportsRule.ts

@@ -118,6 +118,10 @@ const TRANSFORMS = new Map<string, Transform>([
     ["lowercase-last", (x) => x],
     ["full-path", (x) => x],
     ["module-name", (x) => {
+        if (!(ts as any as {isExternalModuleNameRelative(m: string): boolean}).isExternalModuleNameRelative(x)) {
+            return x;
+        }
+
         const splitIndex = x.lastIndexOf("/");
         if (splitIndex === -1) {
             return x;
@@ -147,8 +151,8 @@ function parseOptions(ruleArguments: any[]): Options {
     } = optionSet === undefined ? {} : optionSet;
     return {
         importSourcesOrderTransform: TRANSFORMS.get(sources)!,
-        namedImportsOrderTransform: TRANSFORMS.get(named)!,
         moduleSourcePath: TRANSFORMS.get(path)!,
+        namedImportsOrderTransform: TRANSFORMS.get(named)!,
     };
 }
 

test/rules/ordered-imports/module-source-path/test.ts.fix

@@ -12,3 +12,8 @@ import c = require('./baz/c');
 import a from '../../foo/a';
 import b from './bar/b';
 import c = require('./baz/c');
+
+// contains scoped import
+import a from 'foo/a';
+import b = require('foo/b');
+import c from 'foo/c';

test/rules/ordered-imports/module-source-path/test.ts.lint

@@ -15,3 +15,9 @@ import c = require('./baz/c');
 import a from '../../foo/a';
 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~ [Import sources within a group must be alphabetized.]
 import b from './bar/b';
+
+// contains scoped import
+import b = require('foo/b');
+import a from 'foo/a';
+~~~~~~~~~~~~~~~~~~~~~~ [Import sources within a group must be alphabetized.]
+import c from 'foo/c';