fix: address PR review — defer notify, localize strings, add welcome trigger, changelog

Author: datlechin

CHANGELOG.md

@@ -9,6 +9,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Added
 
+- Pro license gating for Safe Mode (Touch ID) and XLSX export
+- License activation dialog
+
 - Reusable SSH tunnel profiles: save SSH configurations once and select them across multiple connections
 - Ctrl+HJKL navigation as arrow key alternative for keyboards without dedicated arrow keys
 - Amazon DynamoDB database support with PartiQL queries, AWS IAM/Profile/SSO authentication, GSI/LSI browsing, table scanning, capacity display, and DynamoDB Local support

TablePro/Core/Services/Licensing/LicenseManager.swift

@@ -247,26 +247,23 @@ final class LicenseManager {
     /// Evaluate current license status based on expiration, grace period, and signature validity
     private func evaluateStatus() {
         let previousStatus = status
+        defer { notifyIfChanged(from: previousStatus) }
 
         guard let license else {
             status = .unlicensed
-            notifyIfChanged(from: previousStatus)
             return
         }
 
         // Check server-reported status
         switch license.status {
         case .suspended:
             status = .suspended
-            notifyIfChanged(from: previousStatus)
             return
         case .expired:
             status = .expired
-            notifyIfChanged(from: previousStatus)
             return
         case .deactivated:
             status = .deactivated
-            notifyIfChanged(from: previousStatus)
             return
         default:
             break
@@ -275,19 +272,16 @@ final class LicenseManager {
         // Check local expiration
         if license.isExpired {
             status = .expired
-            notifyIfChanged(from: previousStatus)
             return
         }
 
         // Check grace period
         if license.daysSinceLastValidation > gracePeriodDays {
             status = .validationFailed
-            notifyIfChanged(from: previousStatus)
             return
         }
 
         status = .active
-        notifyIfChanged(from: previousStatus)
     }
 
     private func notifyIfChanged(from previousStatus: LicenseStatus) {

TablePro/Views/Connection/ConnectionFormView.swift

@@ -377,7 +377,7 @@ struct ConnectionFormView: View { // swiftlint:disable:this type_body_length
                     }
                     Button(String(localized: "OK"), role: .cancel) {}
                 } message: {
-                    Text("Safe Mode, Safe Mode (Full), and Read-Only require a Pro license.")
+                    Text(String(localized: "Safe Mode, Safe Mode (Full), and Read-Only require a Pro license."))
                 }
             }
         }

TablePro/Views/Connection/WelcomeWindowView.swift

@@ -170,6 +170,19 @@ struct WelcomeWindowView: View {
                     Text("Version \(Bundle.main.appVersion)")
                         .font(.system(size: ThemeEngine.shared.activeTheme.typography.medium))
                         .foregroundStyle(.secondary)
+
+                    if LicenseManager.shared.status.isValid {
+                        Label("Pro", systemImage: "checkmark.seal.fill")
+                            .font(.system(size: ThemeEngine.shared.activeTheme.typography.small, weight: .medium))
+                            .foregroundStyle(.green)
+                    } else {
+                        Button(action: { showActivationSheet = true }) {
+                            Text("Activate License")
+                                .font(.system(size: ThemeEngine.shared.activeTheme.typography.small))
+                        }
+                        .buttonStyle(.plain)
+                        .foregroundStyle(.secondary)
+                    }
                 }
             }
 

TablePro/Views/Toolbar/SafeModeBadgeView.swift

@@ -66,7 +66,7 @@ struct SafeModeBadgeView: View {
             }
             Button(String(localized: "OK"), role: .cancel) {}
         } message: {
-            Text("Safe Mode, Safe Mode (Full), and Read-Only require a Pro license.")
+            Text(String(localized: "Safe Mode, Safe Mode (Full), and Read-Only require a Pro license."))
         }
         .sheet(isPresented: $showActivationSheet) {
             LicenseActivationSheet()