fix: bootstrap screenAreas on first run after upgrade

dlamanna · claude · dlamanna · commit 76656abd1806 · 2026-04-01T18:47:53.000-07:00
WriteSettings(false) only logged — screenAreas was never written to
disk until a setting was manually changed. Now writes to file on
startup if screenAreas is missing, so the new setting gets seeded
immediately.

Co-Authored-By: Claude Opus 4.6 &lt;noreply@anthropic.com&gt;
diff --git a/DesktopShell/Settings.cs b/DesktopShell/Settings.cs
@@ -181,7 +181,13 @@ public static void ScanSettings()
                 EnableTCPServer = true;
                 GlobalVar.Log("^^^ Settings::ScanSettings() - enableTCPServer missing; defaulting to true");
             }
-            WriteSettings(false);
+
+            // Write to file if screenAreas is missing (bootstraps new setting),
+            // otherwise just log.
+            bool needsRewrite = ScreenAreas.Count == 0;
+            WriteSettings(needsRewrite);
+            if (needsRewrite)
+                GlobalVar.Log("^^^ Settings::ScanSettings() - screenAreas missing; wrote settings to bootstrap");
         }
         catch (IOException e)
         {

Original file line number	Diff line number	Diff line change
`@@ -181,7 +181,13 @@ public static void ScanSettings()`
`181`	`181`	`EnableTCPServer = true;`
`182`	`182`	`GlobalVar.Log("^^^ Settings::ScanSettings() - enableTCPServer missing; defaulting to true");`
`183`	`183`	`}`
`184`		`- WriteSettings(false);`
	`184`	`+`
	`185`	`+ // Write to file if screenAreas is missing (bootstraps new setting),`
	`186`	`+ // otherwise just log.`
	`187`	`+ bool needsRewrite = ScreenAreas.Count == 0;`
	`188`	`+ WriteSettings(needsRewrite);`
	`189`	`+ if (needsRewrite)`
	`190`	`+ GlobalVar.Log("^^^ Settings::ScanSettings() - screenAreas missing; wrote settings to bootstrap");`
`185`	`191`	`}`
`186`	`192`	`catch (IOException e)`
`187`	`193`	`{`