fix(sparkle): Fix Settings update button and improve release notes formatting

.github/workflows/update-appcast.yml

@@ -21,10 +21,10 @@ jobs:
           ref: main
           token: ${{ secrets.APPCAST_TOKEN }}
 
-      - name: Install jq
+      - name: Install dependencies
         run: |
           sudo apt-get update
-          sudo apt-get install -y jq
+          sudo apt-get install -y jq pandoc
 
       - name: Generate appcast files
         env:

.github/workflows/validate-scripts.yml

@@ -23,7 +23,7 @@ jobs:
       - name: Install dependencies
         run: |
           sudo apt-get update -qq
-          sudo apt-get install -y -qq jq curl
+          sudo apt-get install -y -qq jq curl pandoc
 
       - name: Create output directories
         run: mkdir -p Docs/signatures

Sources/TermQ/TermQApp.swift

@@ -328,6 +328,9 @@ class TermQAppDelegate: NSObject, NSApplicationDelegate, NSWindowDelegate {
     /// Sparkle updater controller for automatic updates
     let updaterController: SPUStandardUpdaterController
 
+    /// Updater view model for SwiftUI
+    let updaterViewModel: UpdaterViewModel
+
     /// Reference to the main window (first window created)
     private var mainWindow: NSWindow?
 
@@ -339,6 +342,10 @@ class TermQAppDelegate: NSObject, NSApplicationDelegate, NSWindowDelegate {
             updaterDelegate: sparkleDelegate,
             userDriverDelegate: nil
         )
+        updaterViewModel = UpdaterViewModel(
+            updater: updaterController.updater,
+            controller: updaterController
+        )
         super.init()
     }
 
@@ -636,6 +643,7 @@ struct TermQApp: App {
 
         Settings {
             SettingsView()
+                .environmentObject(appDelegate.updaterViewModel)
         }
 
         Window("TermQ Help", id: "help") {
@@ -675,15 +683,16 @@ struct IdentifiableURL: Identifiable {
 /// Observable wrapper for Sparkle's updater to use in SwiftUI
 @MainActor
 final class UpdaterViewModel: ObservableObject {
-    /// Shared instance - initialized lazily when first accessed
+    /// Shared instance - accessed from app delegate
     static var shared: UpdaterViewModel? {
         guard let appDelegate = NSApp.delegate as? TermQAppDelegate else {
             return nil
         }
-        return UpdaterViewModel(updater: appDelegate.updaterController.updater)
+        return appDelegate.updaterViewModel
     }
 
     private let updater: SPUUpdater
+    private let controller: SPUStandardUpdaterController?
     private var cancellables = Set<AnyCancellable>()
 
     /// Whether automatic update checks are enabled (defaults to true)
@@ -704,8 +713,9 @@ final class UpdaterViewModel: ObservableObject {
         }
     }
 
-    init(updater: SPUUpdater) {
+    init(updater: SPUUpdater, controller: SPUStandardUpdaterController? = nil) {
         self.updater = updater
+        self.controller = controller
         // Default to true for automatic checks if not previously set
         let hasExistingPreference = UserDefaults.standard.object(forKey: "SUAutomaticallyChecksForUpdates") != nil
         if !hasExistingPreference {
@@ -726,7 +736,12 @@ final class UpdaterViewModel: ObservableObject {
 
     /// Manually check for updates
     func checkForUpdates() {
-        updater.checkForUpdates()
+        // Use the controller's action method if available (same path as Menu Bar)
+        if let controller = controller {
+            controller.checkForUpdates(nil)
+        } else {
+            updater.checkForUpdates()
+        }
     }
 }
 

Sources/TermQ/Views/Settings/SettingsGeneralView.swift

@@ -121,7 +121,6 @@ struct SettingsGeneralView: View {
                 Button(Strings.Settings.checkForUpdates) {
                     updaterViewModel.checkForUpdates()
                 }
-                .disabled(!updaterViewModel.canCheckForUpdates)
             }
         } header: {
             Text(Strings.Settings.sectionUpdates)

Sources/TermQ/Views/SettingsView.swift

@@ -56,16 +56,8 @@ struct SettingsView: View {
     // Data directory
     @State private var dataDirectory: String = DataDirectoryManager.displayPath
 
-    // Updater (Sparkle)
-    @StateObject private var updaterViewModel =
-        UpdaterViewModel.shared
-        ?? UpdaterViewModel(
-            updater: SPUStandardUpdaterController(
-                startingUpdater: false,
-                updaterDelegate: nil,
-                userDriverDelegate: nil
-            ).updater
-        )
+    // Updater (Sparkle) - injected from app level
+    @EnvironmentObject var updaterViewModel: UpdaterViewModel
 
     enum SettingsTab: CaseIterable {
         case general

scripts/generate-appcast.sh

@@ -36,6 +36,12 @@ check_dependencies() {
         log_info "Install with: brew install ${missing[*]}"
         exit 1
     fi
+
+    # Check for pandoc (optional, for Markdown to HTML conversion)
+    if ! command -v pandoc &> /dev/null; then
+        log_warn "pandoc not found - release notes will display as plain Markdown"
+        log_info "For better formatting, install with: brew install pandoc"
+    fi
 }
 
 # Fetch releases from GitHub API
@@ -101,6 +107,22 @@ is_prerelease() {
     echo "false"
 }
 
+# Convert Markdown to HTML if pandoc is available
+markdown_to_html() {
+    local markdown="$1"
+
+    if command -v pandoc &> /dev/null; then
+        # Use pandoc to convert Markdown to HTML
+        # --from=gfm: GitHub-flavored Markdown
+        # --to=html: Convert to HTML
+        # --wrap=none: Don't wrap lines
+        echo "$markdown" | pandoc --from=gfm --to=html --wrap=none 2>/dev/null || echo "$markdown"
+    else
+        # Fallback: escape HTML entities for plain text display
+        echo "$markdown" | sed 's/&/\&amp;/g; s/</\&lt;/g; s/>/\&gt;/g'
+    fi
+}
+
 # Generate XML for a single release item
 generate_item() {
     local tag="$1"
@@ -114,9 +136,9 @@ generate_item() {
     local version
     version=$(extract_version "$tag")
 
-    # Escape HTML entities in body for CDATA
+    # Convert Markdown to HTML (or escape for plain text if pandoc unavailable)
     local description
-    description=$(echo "$body" | sed 's/&/\&amp;/g; s/</\&lt;/g; s/>/\&gt;/g')
+    description=$(markdown_to_html "$body")
 
     cat << EOF
         <item>