Improve the logic to ensure templateSettings.json is free of dirty data

Planet/Entities/MyPlanetModel.swift

@@ -308,6 +308,13 @@ class MyPlanetModel: Equatable, Hashable, Identifiable, ObservableObject, Codabl
                     touched = true
                 }
             }
+            // Remove settings not in default
+            for (key, value) in currentSettings {
+                if defaultSettings[key] == nil {
+                    currentSettings.removeValue(forKey: key)
+                    touched = true
+                }
+            }
             if touched {
                 let data = try JSONSerialization.data(withJSONObject: currentSettings, options: [.prettyPrinted, .sortedKeys])
                 try data.write(to: templateSettingsPath)
@@ -323,13 +330,27 @@ class MyPlanetModel: Equatable, Hashable, Identifiable, ObservableObject, Codabl
         do {
             // Read current settings
             var currentSettings: [String: String] = templateSettings()
+            // Default settings
+            let defaultSettings: [String: String] = self.template?.defaultSettings() ?? [:]
+            // Remove settings not in default
+            for (key, value) in currentSettings {
+                debugPrint("Template Settings: Checking setting \(key)")
+                if defaultSettings[key] == nil {
+                    currentSettings.removeValue(forKey: key)
+                    debugPrint("Template Settings: Removing setting \(key)")
+                } else {
+                    debugPrint("Template Settings: Keeping setting \(key)")
+                }
+            }
             // Update settings
             for (key, value) in settings {
                 currentSettings[key] = value
             }
             // Write settings
             let data = try JSONSerialization.data(withJSONObject: currentSettings, options: [.prettyPrinted, .sortedKeys])
             try data.write(to: templateSettingsPath)
+
+            try self.copyTemplateSettings()
         }
         catch {
             debugPrint("Error updating template settings: \(error) \(settings)")

Planet/Views/My/MyPlanetEditView.swift

@@ -800,7 +800,6 @@ struct MyPlanetEditView: View {
                         planet.filebaseAPIToken = filebaseAPIToken
                         Task {
                             planet.updateTemplateSettings(settings: userSettings)
-                            try? planet.copyTemplateSettings()
                             try planet.save()
                             Task(priority: .background) {
                                 try await planet.rebuild()

Planet/Views/My/MyPlanetTemplateSettingsView.swift

@@ -187,7 +187,6 @@ struct MyPlanetTemplateSettingsView: View {
                     Button {
                         Task.detached(priority: .userInitiated) {
                             await planet.updateTemplateSettings(settings: userSettings)
-                            try? await planet.copyTemplateSettings()
                             try await planet.save()
                             Task(priority: .background) {
                                 try await planet.rebuild()

Planet/versioning.xcconfig

@@ -1 +1 @@
-CURRENT_PROJECT_VERSION = 2248
+CURRENT_PROJECT_VERSION = 2249

PlanetLite/Croptop/MintSettings.swift

@@ -102,7 +102,6 @@ struct MintSettings: View {
                         debugPrint("Template-level user settings: \(userSettings)")
                         Task {
                             planet.updateTemplateSettings(settings: userSettings)
-                            try? planet.copyTemplateSettings()
                             try await planet.publish()
                         }
                         dismiss()