chore: enhance

Author: Shinigami92

src/rules/order.ts

@@ -46,14 +46,22 @@ const categories = {
 type Category = keyof typeof categories
 
 const defaultGroups = [
-  'private-import',
   'builtin',
   'external',
   'parent',
   'sibling',
   'index',
 ] as const
 
+const defaultGroupsOfficialOrganizeImports = [
+  'private-import',
+  'external',
+  'builtin',
+  'parent',
+  'sibling',
+  'index',
+] as const
+
 // REPORTING AND FIXING
 
 function reverse(array: ImportEntryWithRank[]): ImportEntryWithRank[] {
@@ -1398,8 +1406,8 @@ export default createRule<[Options?], MessageId>({
       const { groups, omittedTypes } = convertGroupsToRanks(
         options.groups ||
           (options.privateImportsFeatureFlag
-            ? defaultGroups
-            : defaultGroups.slice(1)),
+            ? defaultGroupsOfficialOrganizeImports
+            : defaultGroups),
       )
       ranks = {
         groups,

test/rules/order.spec.ts

@@ -4813,9 +4813,9 @@ describe('TypeScript', () => {
         tValid({
           code: `import { internA } from "#a";
 import { scopeA } from "@a/a";
+import fs from 'node:fs';
+import path from "path";
 import { localA } from "./a";
-
-console.log({ internA, scopeA, localA });
 `,
           ...parserConfig,
           options: [
@@ -5207,6 +5207,30 @@ console.log({ internA, scopeA, localA });
             // { message: '`node:fs/promises` import should occur before import of `express`' },
           ],
         }),
+        // By default it should order same as TS LSP (issue-286)
+        tInvalid({
+          code: `import { scopeA } from "@a/a";
+import fs from 'node:fs';
+import path from "path";
+import { localA } from "./a";
+import { internA } from "#a";
+`,
+          output: `import { internA } from "#a";
+import { scopeA } from "@a/a";
+import fs from 'node:fs';
+import path from "path";
+import { localA } from "./a";
+`,
+          ...parserConfig,
+          options: [
+            {
+              privateImportsFeatureFlag: true,
+            },
+          ],
+          errors: [
+            createOrderError(['`#a` import', 'before', 'import of `@a/a`']),
+          ],
+        }),
       ],
     })
   }