Move overrides handling to config package and unify logging

src/config/index.ts

@@ -2,8 +2,8 @@ import { merge } from "merge-anything";
 import YAML from "yaml";
 
 import { logger } from "../logging";
-import { applyConfigOverrides } from "../util/config";
 import defaultConfigContents from "./git-that-semver.default.yaml" with { type: "text" };
+import { applyConfigOverrides } from "./overrides";
 import { Config } from "./types";
 
 const configLogger = logger.childLogger("config");
@@ -54,7 +54,6 @@ export const resolveConfig = async (
   if (configOverrides.length > 0) {
     configLogger.trace("Config overrides", configOverrides);
     mergedConfig = applyConfigOverrides(mergedConfig, configOverrides);
-    configLogger.trace("Config after applying overrides", mergedConfig);
   }
 
   configLogger.trace("Merged config before strategy merge", mergedConfig);

src/util/config.ts → src/config/overrides.ts

@@ -2,7 +2,7 @@ import { set } from "lodash-es";
 
 import { logger } from "../logging";
 
-const utilLogger = logger.childLogger("util/config");
+const configLogger = logger.childLogger("config");
 
 function parseConfigOverride(override: string): { path: string; value: any } {
   if (!/^[^=]+=[^=]*$/.test(override)) {
@@ -51,15 +51,11 @@ export function applyConfigOverrides(
 ): Record<string, any> {
   const result = { ...config };
 
-  utilLogger.trace("Overrides", overrides);
-
   for (const override of overrides) {
     const { path, value } = parseConfigOverride(override);
-    utilLogger.trace("Applying config override", { path, value });
+    configLogger.trace("Applying config override", { path, value });
     set(result, path, value);
   }
 
-  utilLogger.trace("Resulting config", result);
-
   return result;
 }