Sort baselines (#10)

graphile · Oct 29, 2024 · 7427c43 · 7427c43
2 parents ee18d4a + 85e97ab
commit 7427c43
Show file tree

Hide file tree

Showing 3 changed files with 61 additions and 10 deletions.
diff --git a/src/baseline.ts b/src/baseline.ts
@@ -1,5 +1,7 @@
 import type {
   Baseline,
+  BaselineOperations,
+  BaselineOperationsIgnoreCoordinatesByRule,
   CheckDocumentOutput,
   CheckOperationsResult,
   SourceResultsBySourceName,
@@ -41,7 +43,51 @@ export function generateBaseline(
     }
   }
 
-  return baseline;
+  return sortBaseline(baseline);
+}
+
+function sortIgnoreCoordinatesByRule(
+  icbr: BaselineOperationsIgnoreCoordinatesByRule,
+): BaselineOperationsIgnoreCoordinatesByRule {
+  return Object.fromEntries(
+    Object.entries(icbr)
+      .sort((a, z) => {
+        return a[0].localeCompare(z[0], "en-US");
+      })
+      .map(([key, arr]) => [key, arr != null ? [...arr].sort() : arr]),
+  );
+}
+
+function sortOperations(ops: BaselineOperations): BaselineOperations {
+  return Object.fromEntries(
+    Object.entries(ops)
+      .sort((a, z) => a[0].localeCompare(z[0], "en-US"))
+      .map(([opName, val]) => {
+        if (val != null) {
+          const { ignoreCoordinatesByRule, ...rest } = val;
+          return [
+            opName,
+            {
+              ignoreCoordinatesByRule: sortIgnoreCoordinatesByRule(
+                ignoreCoordinatesByRule,
+              ),
+              ...rest,
+            },
+          ];
+        } else {
+          return [opName, val];
+        }
+      }),
+  );
+}
+
+export function sortBaseline(baseline: Baseline): Baseline {
+  const { version, operations, ...rest } = baseline;
+  return {
+    version,
+    operations: sortOperations(operations),
+    ...rest,
+  };
 }
 
 function filterOutput(

diff --git a/src/index.ts b/src/index.ts
@@ -13,6 +13,7 @@ export {
   filterBaseline,
   filterOnlyErrors,
   generateBaseline,
+  sortBaseline,
 } from "./baseline.js";
 export {
   Baseline,

diff --git a/src/interfaces.ts b/src/interfaces.ts
@@ -68,17 +68,21 @@ export interface CheckOperationsResult {
   filtered: number;
 }
 
+export interface BaselineOperationsIgnoreCoordinatesByRule {
+  [infraction: string]: string[] | undefined;
+}
+
+export interface BaselineOperations {
+  [operationName: string]:
+    | {
+        ignoreCoordinatesByRule: BaselineOperationsIgnoreCoordinatesByRule;
+      }
+    | undefined;
+}
+
 export interface Baseline {
   version: 1;
-  operations: {
-    [operationName: string]:
-      | {
-          ignoreCoordinatesByRule: {
-            [infraction: string]: string[] | undefined;
-          };
-        }
-      | undefined;
-  };
+  operations: BaselineOperations;
 }
 
 export interface CheckDocumentEvent {