feat: sort library and sub-app names to reduce git merge conflicts

Author: xpol

src/lib/library/library.factory.ts

@@ -13,7 +13,7 @@ import {
   url,
 } from '@angular-devkit/schematics';
 import { parse } from 'jsonc-parser';
-import { normalizeToKebabOrSnakeCase } from '../../utils/formatting';
+import { inPlaceSortByKeys, normalizeToKebabOrSnakeCase } from '../../utils';
 import {
   DEFAULT_LANGUAGE,
   DEFAULT_LIB_PATH,
@@ -133,6 +133,8 @@ function updateJestConfig(
   const packageKeyRegex = '^' + packageKey + '(|/.*)$';
   const packageRoot = join('<rootDir>' as Path, distRoot);
   jestOptions.moduleNameMapper[packageKeyRegex] = join(packageRoot, '$1');
+
+  inPlaceSortByKeys(jestOptions.moduleNameMapper);
 }
 
 function updateNpmScripts(
@@ -181,6 +183,8 @@ function updateJestEndToEnd(options: LibraryOptions) {
         const packageRoot = '<rootDir>/../' + distRoot;
         jestOptions.moduleNameMapper[deepPackagePath] = packageRoot + '/$1';
         jestOptions.moduleNameMapper[packageKey] = packageRoot;
+
+        inPlaceSortByKeys(jestOptions.moduleNameMapper);
       },
     );
   };
@@ -238,6 +242,8 @@ function updateTsConfig(
           tsconfig.compilerOptions.paths[deepPackagePath] = [];
         }
         tsconfig.compilerOptions.paths[deepPackagePath].push(distRoot + '/*');
+
+        inPlaceSortByKeys(tsconfig.compilerOptions.paths);
       },
     );
   };
@@ -284,6 +290,8 @@ function addLibraryToCliOptions(
           );
         }
         optionsFile.projects[projectName] = project;
+
+        inPlaceSortByKeys(optionsFile.projects);
       },
     );
   };

src/lib/sub-app/sub-app.factory.ts

@@ -16,7 +16,7 @@ import {
 } from '@angular-devkit/schematics';
 import { existsSync, readFileSync } from 'fs';
 import { parse, stringify } from 'comment-json';
-import { normalizeToKebabOrSnakeCase } from '../../utils/formatting';
+import { inPlaceSortByKeys, normalizeToKebabOrSnakeCase } from '../../utils';
 import {
   DEFAULT_APPS_PATH,
   DEFAULT_APP_NAME,
@@ -150,6 +150,8 @@ function updateTsConfig() {
         if (!tsconfig.compilerOptions.paths) {
           tsconfig.compilerOptions.paths = {};
         }
+
+        inPlaceSortByKeys(tsconfig.compilerOptions.paths);
       },
     );
   };
@@ -324,6 +326,8 @@ function addAppsToCliOptions(
           );
         }
         optionsFile.projects[projectName] = project;
+
+        inPlaceSortByKeys(optionsFile.projects);
       },
     );
   };

src/utils/index.ts

@@ -7,3 +7,4 @@ export * from './name.parser';
 export * from './path.solver';
 export * from './source-root.helpers';
 export * from './formatting';
+export * from './object-sorting';

src/utils/object-sorting.ts

@@ -0,0 +1,17 @@
+/**
+ * In-place sort object entities by their keys so that it can be serialized to json with sorted order.
+ * @param object
+ * @returns The original object with modified entities order.
+ */
+export function inPlaceSortByKeys(object: Record<string, any>): Record<string, any> {
+  const sorted: Record<string, any> = {};
+
+  const keys = Object.keys(object);
+  keys.sort();
+  for (const key of keys) {
+    sorted[key] = object[key];
+    delete object[key];
+  }
+
+  return Object.assign(object, sorted);
+}

test/utils/object-sorting.test.ts

@@ -0,0 +1,14 @@
+import { inPlaceSortByKeys } from '../../src/utils/object-sorting';
+
+
+describe('inPlaceSortByKeys', () => {
+  it('should in-place sort the entities by their keys', () => {
+    const input = { z: 'z', b: 'b', c: 'c', a: 'a', };
+    expect(Object.keys(input)).toEqual(['z', 'b', 'c', 'a']);
+
+    const got = inPlaceSortByKeys(input);
+
+    expect(got).toBe(input); // Same object
+    expect(Object.keys(got)).toEqual(['a', 'b', 'c', 'z']);
+  })
+})