fix: [import/order] ensure arcane imports do not cause undefined behavior

Xunnamius · Xunnamius · commit d92910813b59 · 2024-12-22T18:12:29.000-08:00
diff --git a/src/rules/order.js b/src/rules/order.js
@@ -533,8 +533,12 @@ function computeRank(context, ranks, importEntry, excludedImportTypes, isSorting
     rank = computePathRank(ranks.groups, ranks.pathGroups, importEntry.value, ranks.maxPosition);
   }
 
-  if (typeof rank === 'undefined') {
+  if (rank === undefined) {
     rank = ranks.groups[impType];
+
+    if(rank === undefined) {
+      return -1;
+    }
   }
 
   if (isTypeOnlyImport && isSortingTypesAmongThemselves) {
diff --git a/tests/src/rules/order.js b/tests/src/rules/order.js
@@ -3115,7 +3115,6 @@ context('TypeScript', function () {
           }),
           // Option alphabetize: {order: 'asc'} with type group & path group
           test({
-            // only: true,
             code: `
               import c from 'Bar';
               import a from 'foo';
@@ -3145,7 +3144,6 @@ context('TypeScript', function () {
           }),
           // Option alphabetize: {order: 'asc'} with path group
           test({
-            // only: true,
             code: `
               import c from 'Bar';
               import type { A } from 'foo';
@@ -3739,6 +3737,36 @@ context('TypeScript', function () {
               }
             ],
           }),
+          // Ensure the rule doesn't choke and die on absolute paths trying to pass NaN around
+          test({
+            code: `
+              import fs from 'node:fs';
+
+              import '@scoped/package';
+              import type { B } from 'node:fs';
+
+              import type { A1 } from '/bad/bad/bad/bad';
+              import './a/b/c';
+              import type { A2 } from '/bad/bad/bad/bad';
+              import type { A3 } from '/bad/bad/bad/bad';
+              import type { D1 } from '/bad/bad/not/good';
+              import type { D2 } from '/bad/bad/not/good';
+              import type { D3 } from '/bad/bad/not/good';
+
+              import type { C } from '@something/else';
+
+              import type { E } from './index.js';
+            `,
+            ...parserConfig,
+            options: [
+              {
+                alphabetize: { order: 'asc' },
+                groups: ['builtin', 'type', 'unknown', 'external'],
+                sortTypesAmongThemselves: true,
+                'newlines-between': 'always'
+              },
+            ],
+          }),
         ),
         invalid: [].concat(
           // Option alphabetize: {order: 'asc'}
@@ -3992,7 +4020,6 @@ context('TypeScript', function () {
               message: '`A` export should occur before export of `B`',
             }],
           }),
-
           // Options: sortTypesAmongThemselves + newlines-between-types example #1 from the documentation (fail)
           test({
             code: `
@@ -4053,7 +4080,6 @@ context('TypeScript', function () {
               },
             ],
           }),
-
           // Options: sortTypesAmongThemselves + newlines-between-types example #2 from the documentation (fail)
           test({
             code: `

Original file line number	Diff line number	Diff line change
`@@ -533,8 +533,12 @@ function computeRank(context, ranks, importEntry, excludedImportTypes, isSorting`
`533`	`533`	`rank = computePathRank(ranks.groups, ranks.pathGroups, importEntry.value, ranks.maxPosition);`
`534`	`534`	`}`
`535`	`535`
`536`		`- if (typeof rank === 'undefined') {`
	`536`	`+ if (rank === undefined) {`
`537`	`537`	`rank = ranks.groups[impType];`
	`538`	`+`
	`539`	`+ if(rank === undefined) {`
	`540`	`+ return -1;`
	`541`	`+ }`
`538`	`542`	`}`
`539`	`543`
`540`	`544`	`if (isTypeOnlyImport && isSortingTypesAmongThemselves) {`