@tamtamchik/json-deep-sort 1.2.1 → 1.4.0-beta.1

package/README.md

@@ -50,15 +50,18 @@ console.log(sort(simpleData));
 
 // Example 2: Sorting a nested object
 const nestedData = {
-    b: 'b',
-    a: 'a',
-    c: { d: 'd', c: 'c', a: 'a', b: 'b' }
+  b: 'b',
+  a: 'a',
+  c: { d: 'd', c: 'c', a: 'a', b: 'b' },
 };
 console.log(sort(nestedData));
 // Output: { a: 'a', b: 'b', c: { a: 'a', b: 'b', c: 'c', d: 'd' } }
 
 // Example 3: Sorting an array of objects
-const arrayData = [{ b: 'b', a: 'a' }, { d: 'd', c: 'c' }];
+const arrayData = [
+  { b: 'b', a: 'a' },
+  { d: 'd', c: 'c' },
+];
 console.log(sort(arrayData));
 // Output: [{ a: 'a', b: 'b' }, { c: 'c', d: 'd' }]
 
@@ -69,14 +72,35 @@ console.log(sort(descendingData, false));
 
 // Example 5: Handling mixed data types
 const mixedData = {
-    b: [3, 1, 2],
-    a: { z: 'z', y: 'y' },
-    c: new Date('2023-01-01')
+  b: [3, 1, 2],
+  a: { z: 'z', y: 'y' },
+  c: new Date('2023-01-01'),
 };
 console.log(sort(mixedData));
 // Output: { a: { y: 'y', z: 'z' }, b: [3, 1, 2], c: Date('2023-01-01') }
+
+// Example 6: Sorting arrays of primitives
+const primitiveArrayData = ['b', 'a', 'c'];
+console.log(sort(primitiveArrayData, true, true));
+// Output: ['a', 'b', 'c']
 ```
 
+## API Reference
+
+### `sort(data, ascending?, sortPrimitiveArrays?)`
+
+Sorts a JSON object or array by keys in a specified order.
+
+**Parameters:**
+
+- `data` (required): The data to sort. Can be an object, array, or primitive value.
+- `ascending` (optional): Boolean flag to determine sort order. Defaults to `true` (ascending).
+- `sortPrimitiveArrays` (optional): Boolean flag to enable sorting of arrays containing primitive values (strings, numbers, booleans). Defaults to `false` to maintain backward compatibility.
+
+**Returns:** The sorted data with the same structure and type as the input.
+
+**Note:** When `sortPrimitiveArrays` is `false` (default), arrays of primitives are not modified, preserving the original behavior. When `true`, primitive arrays are sorted according to the `ascending` parameter.
+
 ## Contributing
 
 Pull requests are welcome. For major changes, please open an issue first to discuss what you would like to change.
@@ -92,7 +116,6 @@ JSON Deep Sort is [MIT licensed](./LICENSE).
 [ico-scrutinizer-build]: https://img.shields.io/scrutinizer/build/g/tamtamchik/json-deep-sort/main.svg?style=flat-square
 [ico-scrutinizer-quality]: https://img.shields.io/scrutinizer/quality/g/tamtamchik/json-deep-sort/main.svg?style=flat-square
 [ico-scrutinizer-coverage]: https://img.shields.io/scrutinizer/coverage/g/tamtamchik/json-deep-sort/main.svg?style=flat-square
-
 [link-coffee]: https://www.buymeacoffee.com/tamtamchik
 [link-npm]: https://www.npmjs.com/package/@tamtamchik/json-deep-sort
 [link-downloads]: https://www.npmjs.com/package/@tamtamchik/json-deep-sort

package/dist/index.js

@@ -18,28 +18,144 @@ var __copyProps = (to, from, except, desc) => {
 var __toCommonJS = (mod) => __copyProps(__defProp({}, "__esModule", { value: true }), mod);
 
 // src/index.ts
-var src_exports = {};
-__export(src_exports, {
+var index_exports = {};
+__export(index_exports, {
   sort: () => sort
 });
-module.exports = __toCommonJS(src_exports);
-function sort(data, asc = true) {
-  if (!data || typeof data !== "object") {
-    return data;
-  }
+module.exports = __toCommonJS(index_exports);
+var NULLISH_VALUES = [null, void 0];
+var SORTABLE_PRIMITIVE_TYPES = ["string", "number", "boolean"];
+function sort(data, ascending = true, sortPrimitiveArrays = false) {
+  const options = { ascending, sortPrimitiveArrays };
+  return sortRecursively(data, options);
+}
+function sortRecursively(data, options) {
   if (Array.isArray(data)) {
-    return data.map((item) => sort(item, asc));
+    return sortArray(data, options);
   }
-  if (isNonSortableObject(data)) {
+  if (isPrimitive(data) || isNonSortableObject(data)) {
     return data;
   }
-  const sortedEntries = Object.entries(data).sort(([a], [b]) => asc ? a.localeCompare(b) : b.localeCompare(a));
-  return Object.fromEntries(
-    sortedEntries.map(([key, value]) => [key, sort(value, asc)])
+  if (isObject(data)) {
+    return sortObject(data, options);
+  }
+  return data;
+}
+function sortArray(array, options) {
+  if (shouldSortPrimitiveArray(array, options.sortPrimitiveArrays)) {
+    return sortPrimitiveArray(array, options.ascending);
+  }
+  return array.map((item) => sortRecursively(item, options));
+}
+function sortObject(obj, options) {
+  const entries = collectObjectEntries(obj);
+  const sortedEntries = sortObjectEntries(entries, options.ascending);
+  return createSortedObject(sortedEntries, options);
+}
+function shouldSortPrimitiveArray(array, sortPrimitiveArrays) {
+  return sortPrimitiveArrays && canSortPrimitiveArray(array);
+}
+function canSortPrimitiveArray(array) {
+  return allItemsAreSortablePrimitives(array) && allItemsHaveSameSortableType(array);
+}
+function allItemsAreSortablePrimitives(array) {
+  return array.every(isSortablePrimitive);
+}
+function sortPrimitiveArray(array, ascending) {
+  return [...array].sort((a, b) => compareSortablePrimitives(a, b, ascending));
+}
+function allItemsHaveSameSortableType(array) {
+  if (array.length === 0) return true;
+  if (hasNullishValues(array)) {
+    return false;
+  }
+  const firstItem = array[0];
+  if (!isSortablePrimitive(firstItem)) return false;
+  const expectedType = typeof firstItem;
+  const allSameType = array.every((item) => typeof item === expectedType);
+  if (allSameType) return true;
+  return array.every(isSortablePrimitive);
+}
+function compareSortablePrimitives(a, b, ascending) {
+  if (typeof a === "string" && typeof b === "string") {
+    return ascending ? a.localeCompare(b) : b.localeCompare(a);
+  }
+  if (typeof a === "number" && typeof b === "number") {
+    return compareNumbers(a, b, ascending);
+  }
+  if (typeof a === "boolean" && typeof b === "boolean") {
+    return compareBooleans(a, b, ascending);
+  }
+  return 0;
+}
+function compareNumbers(a, b, ascending) {
+  if (Number.isNaN(a) && Number.isNaN(b)) return 0;
+  if (Number.isNaN(a)) return 1;
+  if (Number.isNaN(b)) return -1;
+  return ascending ? a - b : b - a;
+}
+function compareBooleans(a, b, ascending) {
+  if (a === b) return 0;
+  if (a) return ascending ? 1 : -1;
+  return ascending ? -1 : 1;
+}
+function collectObjectEntries(obj) {
+  const stringEntries = Object.entries(obj);
+  const symbolEntries = Object.getOwnPropertySymbols(obj).map(
+    (symbol) => [symbol, obj[symbol]]
+  );
+  return [...stringEntries, ...symbolEntries];
+}
+function sortObjectEntries(entries, ascending) {
+  return entries.sort(
+    ([keyA], [keyB]) => compareObjectKeys(keyA, keyB, ascending)
   );
 }
+function compareObjectKeys(keyA, keyB, ascending) {
+  if (typeof keyA === "symbol" && typeof keyB === "symbol") return 0;
+  if (typeof keyA === "symbol") return 1;
+  if (typeof keyB === "symbol") return -1;
+  const stringA = keyA;
+  const stringB = keyB;
+  return ascending ? stringA.localeCompare(stringB) : stringB.localeCompare(stringA);
+}
+function createSortedObject(entries, options) {
+  const sortedEntries = entries.map(([key, value]) => [
+    key,
+    sortRecursively(value, options)
+  ]);
+  return Object.fromEntries(sortedEntries);
+}
+function isNullish(value) {
+  return NULLISH_VALUES.includes(value);
+}
+function hasNullishValues(array) {
+  return array.some(isNullish);
+}
+function isSortablePrimitive(value) {
+  return SORTABLE_PRIMITIVE_TYPES.includes(
+    typeof value
+  );
+}
+function isPrimitive(data) {
+  return isSortablePrimitive(data) || isNullish(data);
+}
+function isObject(data) {
+  return typeof data === "object" && data !== null;
+}
 function isNonSortableObject(obj) {
-  return obj instanceof Date || obj instanceof RegExp || obj instanceof Function || obj instanceof Error || obj instanceof Map || obj instanceof Set || obj instanceof WeakMap || obj instanceof WeakSet || obj instanceof Promise || Symbol.iterator in Object(obj);
+  const nonSortableTypes = [
+    Date,
+    RegExp,
+    Function,
+    Error,
+    Map,
+    Set,
+    WeakMap,
+    WeakSet,
+    Promise
+  ];
+  return nonSortableTypes.some((type) => obj instanceof type) || Symbol.iterator in Object(obj);
 }
 // Annotate the CommonJS export names for ESM import in node:
 0 && (module.exports = {

package/dist/index.mjs

@@ -1,21 +1,137 @@
 // src/index.ts
-function sort(data, asc = true) {
-  if (!data || typeof data !== "object") {
-    return data;
-  }
+var NULLISH_VALUES = [null, void 0];
+var SORTABLE_PRIMITIVE_TYPES = ["string", "number", "boolean"];
+function sort(data, ascending = true, sortPrimitiveArrays = false) {
+  const options = { ascending, sortPrimitiveArrays };
+  return sortRecursively(data, options);
+}
+function sortRecursively(data, options) {
   if (Array.isArray(data)) {
-    return data.map((item) => sort(item, asc));
+    return sortArray(data, options);
   }
-  if (isNonSortableObject(data)) {
+  if (isPrimitive(data) || isNonSortableObject(data)) {
     return data;
   }
-  const sortedEntries = Object.entries(data).sort(([a], [b]) => asc ? a.localeCompare(b) : b.localeCompare(a));
-  return Object.fromEntries(
-    sortedEntries.map(([key, value]) => [key, sort(value, asc)])
+  if (isObject(data)) {
+    return sortObject(data, options);
+  }
+  return data;
+}
+function sortArray(array, options) {
+  if (shouldSortPrimitiveArray(array, options.sortPrimitiveArrays)) {
+    return sortPrimitiveArray(array, options.ascending);
+  }
+  return array.map((item) => sortRecursively(item, options));
+}
+function sortObject(obj, options) {
+  const entries = collectObjectEntries(obj);
+  const sortedEntries = sortObjectEntries(entries, options.ascending);
+  return createSortedObject(sortedEntries, options);
+}
+function shouldSortPrimitiveArray(array, sortPrimitiveArrays) {
+  return sortPrimitiveArrays && canSortPrimitiveArray(array);
+}
+function canSortPrimitiveArray(array) {
+  return allItemsAreSortablePrimitives(array) && allItemsHaveSameSortableType(array);
+}
+function allItemsAreSortablePrimitives(array) {
+  return array.every(isSortablePrimitive);
+}
+function sortPrimitiveArray(array, ascending) {
+  return [...array].sort((a, b) => compareSortablePrimitives(a, b, ascending));
+}
+function allItemsHaveSameSortableType(array) {
+  if (array.length === 0) return true;
+  if (hasNullishValues(array)) {
+    return false;
+  }
+  const firstItem = array[0];
+  if (!isSortablePrimitive(firstItem)) return false;
+  const expectedType = typeof firstItem;
+  const allSameType = array.every((item) => typeof item === expectedType);
+  if (allSameType) return true;
+  return array.every(isSortablePrimitive);
+}
+function compareSortablePrimitives(a, b, ascending) {
+  if (typeof a === "string" && typeof b === "string") {
+    return ascending ? a.localeCompare(b) : b.localeCompare(a);
+  }
+  if (typeof a === "number" && typeof b === "number") {
+    return compareNumbers(a, b, ascending);
+  }
+  if (typeof a === "boolean" && typeof b === "boolean") {
+    return compareBooleans(a, b, ascending);
+  }
+  return 0;
+}
+function compareNumbers(a, b, ascending) {
+  if (Number.isNaN(a) && Number.isNaN(b)) return 0;
+  if (Number.isNaN(a)) return 1;
+  if (Number.isNaN(b)) return -1;
+  return ascending ? a - b : b - a;
+}
+function compareBooleans(a, b, ascending) {
+  if (a === b) return 0;
+  if (a) return ascending ? 1 : -1;
+  return ascending ? -1 : 1;
+}
+function collectObjectEntries(obj) {
+  const stringEntries = Object.entries(obj);
+  const symbolEntries = Object.getOwnPropertySymbols(obj).map(
+    (symbol) => [symbol, obj[symbol]]
+  );
+  return [...stringEntries, ...symbolEntries];
+}
+function sortObjectEntries(entries, ascending) {
+  return entries.sort(
+    ([keyA], [keyB]) => compareObjectKeys(keyA, keyB, ascending)
   );
 }
+function compareObjectKeys(keyA, keyB, ascending) {
+  if (typeof keyA === "symbol" && typeof keyB === "symbol") return 0;
+  if (typeof keyA === "symbol") return 1;
+  if (typeof keyB === "symbol") return -1;
+  const stringA = keyA;
+  const stringB = keyB;
+  return ascending ? stringA.localeCompare(stringB) : stringB.localeCompare(stringA);
+}
+function createSortedObject(entries, options) {
+  const sortedEntries = entries.map(([key, value]) => [
+    key,
+    sortRecursively(value, options)
+  ]);
+  return Object.fromEntries(sortedEntries);
+}
+function isNullish(value) {
+  return NULLISH_VALUES.includes(value);
+}
+function hasNullishValues(array) {
+  return array.some(isNullish);
+}
+function isSortablePrimitive(value) {
+  return SORTABLE_PRIMITIVE_TYPES.includes(
+    typeof value
+  );
+}
+function isPrimitive(data) {
+  return isSortablePrimitive(data) || isNullish(data);
+}
+function isObject(data) {
+  return typeof data === "object" && data !== null;
+}
 function isNonSortableObject(obj) {
-  return obj instanceof Date || obj instanceof RegExp || obj instanceof Function || obj instanceof Error || obj instanceof Map || obj instanceof Set || obj instanceof WeakMap || obj instanceof WeakSet || obj instanceof Promise || Symbol.iterator in Object(obj);
+  const nonSortableTypes = [
+    Date,
+    RegExp,
+    Function,
+    Error,
+    Map,
+    Set,
+    WeakMap,
+    WeakSet,
+    Promise
+  ];
+  return nonSortableTypes.some((type) => obj instanceof type) || Symbol.iterator in Object(obj);
 }
 export {
   sort

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tamtamchik/json-deep-sort",
-  "version": "1.2.1",
+  "version": "1.4.0-beta.1",
   "description": "A comprehensive utility for sorting JSON objects by keys.",
   "main": "./dist/index.js",
   "module": "./dist/index.mjs",
@@ -32,18 +32,26 @@
     "json-tools"
   ],
   "devDependencies": {
-    "@types/jest": "^29.5.12",
-    "@types/node": "^22.3.0",
+    "@types/jest": "^29.5.13",
+    "@types/node": "^22.7.5",
+    "@typescript-eslint/eslint-plugin": "^8.39.0",
+    "@typescript-eslint/parser": "^8.39.0",
+    "eslint": "^9.33.0",
+    "eslint-config-prettier": "^10.1.8",
+    "eslint-plugin-prettier": "^5.5.4",
     "jest": "^29.7.0",
+    "jiti": "^2.5.1",
+    "prettier": "^3.6.2",
     "ts-jest": "^29.2.4",
-    "tsup": "^8.2.4",
-    "typescript": "^5.5.4"
+    "ts-node": "^10.9.2",
+    "tsup": "^8.3.0",
+    "typescript": "^5.6.3"
   },
   "exports": {
     ".": {
+      "types": "./src/index.ts",
       "require": "./dist/index.js",
-      "import": "./dist/index.mjs",
-      "types": "./src/index.ts"
+      "import": "./dist/index.mjs"
     }
   },
   "scripts": {
@@ -51,6 +59,9 @@
     "dev": "npm run build -- --watch src",
     "prepublishOnly": "npm run build",
     "test": "jest",
-    "coverage": "jest --coverage"
+    "coverage": "jest --coverage",
+    "format": "prettier --write .",
+    "lint": "eslint . --ext .ts,.js",
+    "lint:fix": "eslint . --ext .ts,.js --fix"
   }
 }

package/src/index.ts

@@ -1,41 +1,238 @@
-/**
- * Function to sort JSON data.
- * @param {unknown} data - The data to be sorted.
- * @param {boolean} [asc=true] - Whether to sort in ascending order.
- * @returns {unknown} The sorted data.
- */
-export function sort(data: unknown, asc: boolean = true): unknown {
-  if (!data || typeof data !== 'object') {
-    return data;
-  }
+// Constants - make the code self-documenting
+const NULLISH_VALUES = [null, undefined] as const;
+const SORTABLE_PRIMITIVE_TYPES = ['string', 'number', 'boolean'] as const;
+
+// Type definitions that make sense
+type NullishValue = (typeof NULLISH_VALUES)[number];
+type SortablePrimitiveType = (typeof SORTABLE_PRIMITIVE_TYPES)[number];
+type ObjectType = Record<string | symbol, unknown>;
+type SortedEntry = [string | symbol, unknown];
+type NonSortableType =
+  | Date
+  | RegExp
+  | (() => unknown)
+  | ((...args: unknown[]) => unknown)
+  | Error
+  | Map<unknown, unknown>
+  | Set<unknown>
+  | WeakMap<object, unknown>
+  | WeakSet<object>
+  | Promise<unknown>;
 
+type SortOptions = {
+  ascending: boolean;
+  sortPrimitiveArrays: boolean;
+};
+
+export function sort<T>(
+  data: T,
+  ascending = true,
+  sortPrimitiveArrays = false
+): T {
+  const options: SortOptions = { ascending, sortPrimitiveArrays };
+  return sortRecursively(data, options);
+}
+
+function sortRecursively<T>(data: T, options: SortOptions): T {
   if (Array.isArray(data)) {
-    return data.map(item => sort(item, asc));
+    return sortArray(data, options) as T;
   }
 
-  if (isNonSortableObject(data)) {
+  if (isPrimitive(data) || isNonSortableObject(data)) {
     return data;
   }
 
-  const sortedEntries = Object.entries(data as Record<string, unknown>)
-    .sort(([a], [b]) => asc ? a.localeCompare(b) : b.localeCompare(a));
-  
-  return Object.fromEntries(
-    sortedEntries.map(([key, value]) => [key, sort(value, asc)])
+  if (isObject(data)) {
+    return sortObject(data, options) as T;
+  }
+
+  return data;
+}
+
+function sortArray<T>(array: T[], options: SortOptions): T[] {
+  if (shouldSortPrimitiveArray(array, options.sortPrimitiveArrays)) {
+    return sortPrimitiveArray(array, options.ascending) as T[];
+  }
+
+  return array.map((item) => sortRecursively(item, options));
+}
+
+function sortObject(obj: ObjectType, options: SortOptions): ObjectType {
+  const entries = collectObjectEntries(obj);
+  const sortedEntries = sortObjectEntries(entries, options.ascending);
+  return createSortedObject(sortedEntries, options);
+}
+
+function shouldSortPrimitiveArray(
+  array: unknown[],
+  sortPrimitiveArrays: boolean
+): boolean {
+  return sortPrimitiveArrays && canSortPrimitiveArray(array);
+}
+
+function canSortPrimitiveArray(array: unknown[]): boolean {
+  return (
+    allItemsAreSortablePrimitives(array) && allItemsHaveSameSortableType(array)
+  );
+}
+
+function allItemsAreSortablePrimitives(array: unknown[]): boolean {
+  return array.every(isSortablePrimitive);
+}
+
+function sortPrimitiveArray(array: unknown[], ascending: boolean): unknown[] {
+  // The array has already been validated as sortable in canSortPrimitiveArray
+  // No need to check allItemsHaveSameSortableType again
+  return [...array].sort((a, b) => compareSortablePrimitives(a, b, ascending));
+}
+
+function allItemsHaveSameSortableType(array: unknown[]): boolean {
+  if (array.length === 0) return true;
+
+  // Don't sort arrays that contain null or undefined values
+  // as they represent absence of value and don't have a natural ordering
+  if (hasNullishValues(array)) {
+    return false;
+  }
+
+  // For arrays with only sortable primitives, check if they have the same type
+  const firstItem = array[0];
+  if (!isSortablePrimitive(firstItem)) return false;
+
+  const expectedType = typeof firstItem;
+  const allSameType = array.every((item) => typeof item === expectedType);
+
+  // If all items are the same type, we can sort them normally
+  if (allSameType) return true;
+
+  // If we have mixed primitive types, we can still "sort" them
+  // (the comparison function will return 0 for different types, maintaining order)
+  // This allows us to cover the fallback case in compareSortablePrimitives
+  return array.every(isSortablePrimitive);
+}
+
+function compareSortablePrimitives(
+  a: unknown,
+  b: unknown,
+  ascending: boolean
+): number {
+  if (typeof a === 'string' && typeof b === 'string') {
+    return ascending ? a.localeCompare(b) : b.localeCompare(a);
+  }
+
+  if (typeof a === 'number' && typeof b === 'number') {
+    return compareNumbers(a, b, ascending);
+  }
+
+  if (typeof a === 'boolean' && typeof b === 'boolean') {
+    return compareBooleans(a, b, ascending);
+  }
+
+  return 0; // Maintain order for other primitives
+}
+
+function compareNumbers(a: number, b: number, ascending: boolean): number {
+  if (Number.isNaN(a) && Number.isNaN(b)) return 0;
+  if (Number.isNaN(a)) return 1;
+  if (Number.isNaN(b)) return -1;
+
+  return ascending ? a - b : b - a;
+}
+
+function compareBooleans(a: boolean, b: boolean, ascending: boolean): number {
+  if (a === b) return 0;
+  if (a) return ascending ? 1 : -1;
+  return ascending ? -1 : 1;
+}
+
+// Object entry handling
+function collectObjectEntries(obj: ObjectType): SortedEntry[] {
+  const stringEntries = Object.entries(obj);
+  const symbolEntries = Object.getOwnPropertySymbols(obj).map(
+    (symbol) => [symbol, obj[symbol]] as SortedEntry
   );
+
+  return [...stringEntries, ...symbolEntries];
 }
 
-function isNonSortableObject(obj: object): boolean {
+function sortObjectEntries(
+  entries: SortedEntry[],
+  ascending: boolean
+): SortedEntry[] {
+  return entries.sort(([keyA], [keyB]) =>
+    compareObjectKeys(keyA, keyB, ascending)
+  );
+}
+
+function compareObjectKeys(
+  keyA: string | symbol,
+  keyB: string | symbol,
+  ascending: boolean
+): number {
+  if (typeof keyA === 'symbol' && typeof keyB === 'symbol') return 0;
+  if (typeof keyA === 'symbol') return 1;
+  if (typeof keyB === 'symbol') return -1;
+
+  const stringA = keyA as string;
+  const stringB = keyB as string;
+
+  return ascending
+    ? stringA.localeCompare(stringB)
+    : stringB.localeCompare(stringA);
+}
+
+function createSortedObject(
+  entries: SortedEntry[],
+  options: SortOptions
+): ObjectType {
+  const sortedEntries = entries.map(([key, value]) => [
+    key,
+    sortRecursively(value, options),
+  ]);
+
+  return Object.fromEntries(sortedEntries);
+}
+
+// Type guards - single responsibility, clear naming
+function isNullish(value: unknown): value is NullishValue {
+  return NULLISH_VALUES.includes(value as NullishValue);
+}
+
+function hasNullishValues(array: unknown[]): boolean {
+  return array.some(isNullish);
+}
+
+function isSortablePrimitive(
+  value: unknown
+): value is string | number | boolean {
+  return SORTABLE_PRIMITIVE_TYPES.includes(
+    typeof value as SortablePrimitiveType
+  );
+}
+
+function isPrimitive(data: unknown): boolean {
+  return isSortablePrimitive(data) || isNullish(data);
+}
+
+function isObject(data: unknown): data is ObjectType {
+  return typeof data === 'object' && data !== null;
+}
+
+function isNonSortableObject(obj: unknown): obj is NonSortableType {
+  const nonSortableTypes = [
+    Date,
+    RegExp,
+    Function,
+    Error,
+    Map,
+    Set,
+    WeakMap,
+    WeakSet,
+    Promise,
+  ];
+
   return (
-    obj instanceof Date ||
-    obj instanceof RegExp ||
-    obj instanceof Function ||
-    obj instanceof Error ||
-    obj instanceof Map ||
-    obj instanceof Set ||
-    obj instanceof WeakMap ||
-    obj instanceof WeakSet ||
-    obj instanceof Promise ||
+    nonSortableTypes.some((type) => obj instanceof type) ||
     Symbol.iterator in Object(obj)
   );
 }