fix: remove empty overridden list

CyanSalt · CyanSalt · commit 547bb742f409 · 2024-12-06T10:32:15.000+08:00
diff --git a/src/main/lib/settings.ts b/src/main/lib/settings.ts
@@ -101,10 +101,11 @@ function resolveOverridden(values: string[]) {
 
 function createOverridden(previous: string[], current: string[]) {
   const values = resolveOverridden(previous)
-  return [
+  const overridden = [
     ...difference(values, current).map(value => `!${value}`),
     ...difference(current, values).map(value => value),
   ]
+  return overridden.length ? overridden : undefined
 }
 
 let oldSettings: Settings | undefined

Original file line number	Diff line number	Diff line change
`@@ -101,10 +101,11 @@ function resolveOverridden(values: string[]) {`
`101`	`101`
`102`	`102`	`function createOverridden(previous: string[], current: string[]) {`
`103`	`103`	`const values = resolveOverridden(previous)`
`104`		`- return [`
	`104`	`+ const overridden = [`
`105`	`105`	...difference(values, current).map(value => `!${value}`),
`106`	`106`	`...difference(current, values).map(value => value),`
`107`	`107`	`]`
	`108`	`+ return overridden.length ? overridden : undefined`
`108`	`109`	`}`
`109`	`110`
`110`	`111`	`let oldSettings: Settings \| undefined`