Remove natural order comparison utility and simplify duplicate exports/imports handling

Author: Rel1cx

packages/tsl-module/src/rules/no-duplicate-exports.ts

@@ -1,8 +1,6 @@
 import { type AST, defineRule } from "tsl";
 import ts from "typescript";
 
-import { naturalCompare } from "../utils";
-
 export const messages = {
   noDuplicateExports: (p: { source: string }) => `Duplicate export from module ${p.source}.`,
 } as const;
@@ -77,7 +75,7 @@ function buildSuggestions(a: ReExportDeclaration, b: ReExportDeclaration) {
       && ts.isNamedExports(b.exportClause): {
       const aElements = a.exportClause.elements.map((el) => el.getText());
       const bElements = b.exportClause.elements.map((el) => el.getText());
-      const parts = Array.from(new Set([...aElements, ...bElements])).toSorted(naturalCompare);
+      const parts = Array.from(new Set([...aElements, ...bElements]));
       return [
         {
           message: "Merge duplicate exports",

packages/tsl-module/src/rules/no-duplicate-imports.test.ts

@@ -81,7 +81,7 @@ test("no-duplicate-import", () => {
               {
                 message: "Merge duplicate imports",
                 output: tsx`
-                  import foo, { baz, quux, type bar, type qux } from 'module';\n
+                  import foo, { type bar, baz, type qux, quux } from 'module';\n
                   import foo3, { corge } from 'module';
                 `,
               },
@@ -94,7 +94,7 @@ test("no-duplicate-import", () => {
               {
                 message: "Merge duplicate imports",
                 output: tsx`
-                  import foo, { baz, corge, type bar } from 'module';
+                  import foo, { type bar, baz, corge } from 'module';
                   import foo2, { type qux, quux } from 'module';\n
                 `,
               },

packages/tsl-module/src/rules/no-duplicate-imports.ts

@@ -2,7 +2,6 @@ import { unit } from "@let/eff";
 import { P, match } from "ts-pattern";
 import { type AST, defineRule } from "tsl";
 import ts from "typescript";
-import { naturalCompare } from "../utils";
 
 export const messages = {
   noDuplicateImports: (p: { source: string }) => `Duplicate import from module ${p.source}.`,
@@ -136,7 +135,7 @@ function buildMergedImport(a: ImportInfo, b: ImportInfo): string {
     parts.push(`* as ${b.namespaceImport}`);
   }
   // Named imports
-  const namedImports = Array.from(new Set([...a.namedImports, ...b.namedImports])).toSorted(naturalCompare);
+  const namedImports = Array.from(new Set([...a.namedImports, ...b.namedImports]));
   // Construct named imports part
   if (namedImports.length > 0) {
     parts.push(`{ ${namedImports.join(", ")} }`);