refactor: Make ConfigUpdater handle a null config so the supplied function doesn't have to.

Author: jswalden

src/config.ts

@@ -22,8 +22,8 @@ export const DebugLoggingOptionId = 'debugLogging'
  * logs, to make it easier to debug the module in case of error.  Add a default
  * value for that option to older configs.
  */
-export function tryUpdateConfigWithDebugLogging(config: RawConfig | null): boolean {
-	if (config !== null && !(DebugLoggingOptionId in config)) {
+export function tryUpdateConfigWithDebugLogging(config: RawConfig): boolean {
+	if (!(DebugLoggingOptionId in config)) {
 		config[DebugLoggingOptionId] = false
 		return true
 	}

src/upgrades.ts

@@ -20,11 +20,11 @@ function ActionUpdater(
 	}
 }
 
-function ConfigUpdater(tryUpdate: (config: RawConfig | null) => boolean): CompanionStaticUpgradeScript<RawConfig> {
+function ConfigUpdater(tryUpdate: (config: RawConfig) => boolean): CompanionStaticUpgradeScript<RawConfig> {
 	return (_context: CompanionUpgradeContext<RawConfig>, props: CompanionStaticUpgradeProps<RawConfig>) => {
 		return {
 			updatedActions: [],
-			updatedConfig: tryUpdate(props.config) ? props.config : null,
+			updatedConfig: props.config !== null && tryUpdate(props.config) ? props.config : null,
 			updatedFeedbacks: [],
 		}
 	}
@@ -35,4 +35,4 @@ export const UpgradeScripts = [
 	ConfigUpdater(tryUpdateConfigWithDebugLogging),
 	ActionUpdater(tryUpdateRecallSetPresetActions),
 	ActionUpdater(tryUpdatePresetAndSpeedEncodingsInActions),
-]
+] satisfies CompanionStaticUpgradeScript<RawConfig>[]