Fix SaneBar wake recovery release proof

Author: SJ

.outreach.yml

@@ -99,7 +99,7 @@ launch_calendar:
   next_eligible_launch:
     status: "not_scheduled"
     trigger: "Major release only, such as a visible v2.2/v3 workflow change, new Touch ID/security story, or substantial user proof."
-    earliest_recommendation: "After the current v2.1.53 customer UI reliability patch is published and support noise is stable."
+    earliest_recommendation: "After the current customer UI reliability patch is published and support noise is stable."
   scheduled:
     - cadence: "weekly"
       channel: "Opportunity monitoring"
@@ -108,15 +108,16 @@ launch_calendar:
       gate: "No Product Hunt relaunch, no Show HN repost, and no generic standalone launch posts."
       success_metric: "0-2 high-fit replies per week or a recorded no-go."
   last_launch_readiness:
-    date: "2026-05-20"
+    date: "2026-05-24"
     status: "no_go"
     launch_readiness_exit: 1
     blocker_summary:
-      - "Fresh Mini launch_readiness on 2026-05-20 stayed red because SaneBar already had the real Product Hunt/Hacker News launch and a repost without a major new story would be weak."
+      - "Canonical launch_readiness rerun at 2026-05-24 09:32 EDT exited nonzero and the wrapper reported 'Mac mini is unreachable. Falling back to local execution.'"
+      - "SaneBar remains launch no-go because it already had the real Product Hunt/Hacker News launch and a repost without a major new story would be weak."
       - "Open support issues remain around layout stability and icon movement; do not amplify broadly until the current patch is stable."
-      - "Mini launch_readiness reported the latest release_preflight as failed with 1 issue and 4 warnings on 2026-05-20."
-      - "Global validation at 2026-05-20 09:02 EDT is still red for SaneBar because Lemon Squeezy hosted file `2.1.56` lags the live appcast/website `2.1.57`, and customer UI proof is stale."
-    next_date: "2026-05-25"
+      - "Local fallback launch_readiness still reported the latest release_preflight as passed but carrying 5 warnings, and the shared validation report at 2026-05-24 09:31 EDT still says BROKEN RELEASE PIPELINE because SaneBar customer UI proof is stale."
+      - "No weekly opportunity reply was executed on 2026-05-24 because the app gate remained red and the schedule only allows targeted disclosed replies, not a relaunch."
+    next_date: "2026-05-29"
 
 x_search_keywords:
   - "\"Bartender\" (alternative OR replacement OR acquired OR privacy) lang:en -is:retweet"

.sane/customer_ui_action_receipt.json

@@ -1,3 +1,4 @@
+#if !DEBUG
 import AppIntents
 import Foundation
 
@@ -145,3 +146,5 @@ struct SaneBarAppShortcuts: AppShortcutsProvider {
         )
     }
 }
+
+#endif

Core/AppIntents/SaneBarAppIntents.swift

@@ -783,6 +783,36 @@ final class StatusBarController: StatusBarControllerProtocol {
         return (main: safeMainLimit, separator: safeMainLimit + preservedGap)
     }
 
+    nonisolated static func launchSafePreferredSeparatorGap(for screenWidth: Double) -> Double {
+        guard screenWidth > 0 else { return 120.0 }
+        // Recovery is allowed to move SaneBar back beside Control Center, but it
+        // must not collapse the user's visible lane so far that leftmost shown
+        // items wake up hidden. Keep a moderate lane on external displays while
+        // bounding it on smaller screens.
+        return min(240.0, max(180.0, screenWidth * 0.09))
+    }
+
+    private nonisolated static func migratedLegacyNarrowRecoveryPair(
+        mainPosition: Double,
+        separatorPosition: Double,
+        screenWidth: Double,
+        screenHasTopSafeAreaInset: Bool
+    ) -> (main: Double, separator: Double)? {
+        let safeMain = launchSafePreferredMainPositionLimit(
+            for: screenWidth,
+            screenHasTopSafeAreaInset: screenHasTopSafeAreaInset
+        )
+        let targetGap = launchSafePreferredSeparatorGap(for: screenWidth)
+        guard targetGap > 0 else { return nil }
+        guard abs(mainPosition - safeMain) < 0.5 else { return nil }
+        let currentGap = separatorPosition - mainPosition
+        guard currentGap < targetGap - 0.5 else { return nil }
+
+        let widenedSeparator = min(screenWidth - 24.0, safeMain + targetGap)
+        guard widenedSeparator > safeMain else { return nil }
+        return (main: safeMain, separator: widenedSeparator)
+    }
+
     nonisolated static func launchSafeCurrentDisplayRecoveryPair(
         screenWidth: Double,
         screenHasTopSafeAreaInset: Bool
@@ -793,7 +823,10 @@ final class StatusBarController: StatusBarControllerProtocol {
             for: screenWidth,
             screenHasTopSafeAreaInset: screenHasTopSafeAreaInset
         )
-        let safeSeparator = min(screenWidth - 24.0, safeMain + 120.0)
+        let safeSeparator = min(
+            screenWidth - 24.0,
+            safeMain + launchSafePreferredSeparatorGap(for: screenWidth)
+        )
         guard safeSeparator > safeMain else { return nil }
         return (main: safeMain, separator: safeSeparator)
     }
@@ -929,16 +962,22 @@ final class StatusBarController: StatusBarControllerProtocol {
                 screenWidth: width,
                 screenHasTopSafeAreaInset: currentScreenHasTopSafeAreaInset
             ) {
-                setPreferredPosition(reanchored.main, forAutosaveName: mainAutosaveName)
-                setPreferredPosition(reanchored.separator, forAutosaveName: separatorAutosaveName)
-                saveDisplayPositionBackupIfNeeded(
-                    for: width,
+                let restoredPair = migratedLegacyNarrowRecoveryPair(
                     mainPosition: reanchored.main,
                     separatorPosition: reanchored.separator,
+                    screenWidth: width,
+                    screenHasTopSafeAreaInset: currentScreenHasTopSafeAreaInset
+                ) ?? reanchored
+                setPreferredPosition(restoredPair.main, forAutosaveName: mainAutosaveName)
+                setPreferredPosition(restoredPair.separator, forAutosaveName: separatorAutosaveName)
+                saveDisplayPositionBackupIfNeeded(
+                    for: width,
+                    mainPosition: restoredPair.main,
+                    separatorPosition: restoredPair.separator,
                     referenceScreen: referenceScreen
                 )
                 logger.warning(
-                    "Display validation: reanchored unsafe backup for width \(width, privacy: .public) (main=\(mainBackup, privacy: .public) -> \(reanchored.main, privacy: .public), separator=\(separatorBackup, privacy: .public) -> \(reanchored.separator, privacy: .public))"
+                    "Display validation: reanchored unsafe backup for width \(width, privacy: .public) (main=\(mainBackup, privacy: .public) -> \(restoredPair.main, privacy: .public), separator=\(separatorBackup, privacy: .public) -> \(restoredPair.separator, privacy: .public))"
                 )
                 return true
             }
@@ -950,6 +989,26 @@ final class StatusBarController: StatusBarControllerProtocol {
             return false
         }
 
+        if let migrated = migratedLegacyNarrowRecoveryPair(
+            mainPosition: mainBackup,
+            separatorPosition: separatorBackup,
+            screenWidth: width,
+            screenHasTopSafeAreaInset: currentScreenHasTopSafeAreaInset
+        ) {
+            setPreferredPosition(migrated.main, forAutosaveName: mainAutosaveName)
+            setPreferredPosition(migrated.separator, forAutosaveName: separatorAutosaveName)
+            setDisplayPositionBackup(
+                for: width,
+                mainPosition: migrated.main,
+                separatorPosition: migrated.separator,
+                referenceScreen: resolvedReferenceScreen
+            )
+            logger.warning(
+                "Display validation: widened legacy narrow recovery backup for width \(width, privacy: .public) (separator=\(separatorBackup, privacy: .public) -> \(migrated.separator, privacy: .public))"
+            )
+            return true
+        }
+
         setPreferredPosition(mainBackup, forAutosaveName: mainAutosaveName)
         setPreferredPosition(separatorBackup, forAutosaveName: separatorAutosaveName)
         logger.info("Display validation: restored backup positions for width \(width)")

Core/Controllers/StatusBarController.swift

@@ -661,6 +661,26 @@ private func scriptListingZonesForCommand() -> [ScriptZonedIcon] {
     )
 }
 
+@MainActor
+private func authoritativeScriptListingZonesForCommand() -> [ScriptZonedIcon] {
+    AccessibilityService.shared.invalidateMenuBarItemPositionsCache()
+    let result = ScriptResultBox<ScriptClassifiedApps?>(nil)
+    Task { @MainActor in
+        result.value = await SearchService.shared.refreshClassifiedApps()
+    }
+
+    let deadline = Date().addingTimeInterval(5.0)
+    while result.value == nil, Date() < deadline {
+        _ = RunLoop.current.run(mode: .default, before: Date().addingTimeInterval(0.05))
+    }
+
+    if let classified = result.value {
+        return sortedScriptZones(zones(from: classified))
+    }
+
+    return []
+}
+
 @MainActor
 private func runScriptMove(timeoutSeconds: TimeInterval = 9.0, operation: @escaping @MainActor () async -> Bool) -> Bool? {
     let box = ScriptResultBox<Bool?>(nil)
@@ -914,6 +934,32 @@ final class ListIconZonesCommand: SaneBarScriptCommand {
     }
 }
 
+@objc(ListAuthoritativeIconZonesCommand)
+final class ListAuthoritativeIconZonesCommand: SaneBarScriptCommand {
+    override func performDefaultImplementation() -> Any? {
+        guard checkAccessibilityTrusted() else {
+            setAccessibilityError()
+            return nil
+        }
+
+        let zones: [ScriptZonedIcon] = if Thread.isMainThread {
+            MainActor.assumeIsolated {
+                return authoritativeScriptListingZonesForCommand()
+            }
+        } else {
+            DispatchQueue.main.sync {
+                return authoritativeScriptListingZonesForCommand()
+            }
+        }
+
+        let lines = zones.map { item in
+            let movable = item.app.isUnmovableSystemItem ? "false" : "true"
+            return "\(item.zone.rawValue)\t\(movable)\t\(item.app.bundleId)\t\(item.app.uniqueId)\t\(item.app.name)"
+        }
+        return lines.joined(separator: "\n")
+    }
+}
+
 // MARK: - Layout Snapshot Command
 
 @objc(LayoutSnapshotCommand)

Core/Services/AppleScriptCommands.swift

@@ -92,6 +92,11 @@
             <result type="text" description="Newline-separated lines: zone, movable, bundleId, uniqueId, name"/>
         </command>
 
+        <command name="list authoritative icon zones" code="SBarlaiz" description="List detected menu bar icons with fresh authoritative visibility zones">
+            <cocoa class="ListAuthoritativeIconZonesCommand"/>
+            <result type="text" description="Newline-separated lines: zone, movable, bundleId, uniqueId, name"/>
+        </command>
+
         <!-- Layout Snapshot Command -->
         <command name="layout snapshot" code="SBarlyot" description="Return a JSON snapshot of menu bar launch/layout invariants">
             <cocoa class="LayoutSnapshotCommand"/>

Resources/SaneBar.sdef

@@ -1,5 +1,4 @@
 import AppKit
-import AppIntents
 import KeyboardShortcuts
 import os.log
 import SaneUI
@@ -36,14 +35,25 @@ class SaneBarAppDelegate: NSObject, NSApplicationDelegate {
 
     // No @main - using main.swift instead
 
+    nonisolated static func shouldUpdateAppShortcutParameters(
+        environment: [String: String] = ProcessInfo.processInfo.environment,
+        isRunningTests: Bool = NSClassFromString("XCTestCase") != nil
+    ) -> Bool {
+        guard environment["XCTestConfigurationFilePath"] == nil else { return false }
+        guard !isRunningTests else { return false }
+        return true
+    }
+
     func applicationDidFinishLaunching(_: Notification) {
         appLogger.info("🏁 applicationDidFinishLaunching START")
 
         // Near-instant tooltips (default is ~1000ms)
         UserDefaults.standard.set(100, forKey: "NSInitialToolTipDelay")
-        if ProcessInfo.processInfo.environment["XCTestConfigurationFilePath"] == nil {
+        #if !DEBUG
+        if Self.shouldUpdateAppShortcutParameters() {
             SaneBarAppShortcuts.updateAppShortcutParameters()
         }
+        #endif
 
         // Keep the menu bar process alive across idle periods.
         keepAliveActivity = ProcessInfo.processInfo.beginActivity(

SaneBarApp.swift

@@ -37,6 +37,7 @@ class CustomerUIActionSweep
     ['layout snapshot', /separatorBeforeMain/],
     ['list icons', /./],
     ['list icon zones', /\t/],
+    ['list authoritative icon zones', /\t/],
     ['open icon panel', /true|false/],
     ['quick search "Sane"', /true|false/],
     ['close browse panel', /true|false/],
@@ -58,6 +59,15 @@ class CustomerUIActionSweep
     'search?q=Sane'
   ].freeze
 
+  HEALTH_WARNING_LABELS = [
+    'Needs Action',
+    'Needs Check',
+    'Needs Repair',
+    'Missing Items',
+    'Hidden by macOS',
+    'Detached'
+  ].freeze
+
   STRICT_MINI_EVIDENCE_TYPES = %w[
     mini_click
     mini_automation
@@ -284,7 +294,11 @@ def verify_release_app_running!
   def exercise_settings_tabs
     app_script('open settings window')
     SETTINGS_TABS.each do |tab|
-      text = press_settings_tab(tab[:index])
+      text = if tab[:id] == 'health'
+               wait_for_clean_health_tab(tab[:index])
+             else
+               press_settings_tab(tab[:index])
+             end
       tab[:expected].each do |expected|
         raise "Settings #{tab[:id]} tab missing #{expected.inspect}: #{text}" unless text.include?(expected)
       end
@@ -325,6 +339,27 @@ def press_settings_tab(index)
     end
   end
 
+  def wait_for_clean_health_tab(index, timeout: 8.0)
+    deadline = Time.now + timeout
+    last_text = nil
+    loop do
+      last_text = press_settings_tab(index)
+      warnings = health_tab_warnings(last_text)
+      return last_text if warnings.empty?
+
+      break if Time.now >= deadline
+
+      sleep 0.5
+    end
+
+    raise "Health tab is not release-clean: #{health_tab_warnings(last_text).join(', ')}"
+  end
+
+  def health_tab_warnings(text)
+    value = text.to_s
+    HEALTH_WARNING_LABELS.select { |label| value.include?(label) }
+  end
+
   def exercise_url_routes
     {
       'toggle' => nil,
@@ -586,19 +621,71 @@ def runtime_state_results(report)
 
         Array(item[:paths] || item['paths'] || item[:artifacts] || item['artifacts'] || item[:path] || item['path'])
       end.compact
-      status = (required_types - evidence_types).empty? && evidence_paths.any? ? 'passed' : 'failed'
+      runtime_artifact = runtime_state_artifact(id.to_s)
+      if runtime_artifact
+        evidence_types |= Array(runtime_artifact[:evidence_types])
+        evidence_paths |= Array(runtime_artifact[:evidence_paths])
+      end
+      completed_scenarios = Array(runtime_artifact && runtime_artifact[:completed_scenarios]).map(&:to_s)
+      required_scenarios = Array(row['required_scenarios']).map(&:to_s)
+      status = (required_types - evidence_types).empty? &&
+               (required_scenarios - completed_scenarios).empty? &&
+               evidence_paths.any? ? 'passed' : 'failed'
       {
         id: id.to_s,
         status: status,
         action_ids: action_ids,
         required_evidence_types: required_types,
         evidence_types: evidence_types.uniq,
         evidence_paths: evidence_paths.uniq,
+        completed_scenarios: completed_scenarios.uniq,
         manifest_sha256: report.fetch('manifest_sha256')
       }
     end
   end
 
+  def runtime_state_artifact(id)
+    case id
+    when 'fullscreen_maximize_transition'
+      fullscreen_matrix_artifact
+    when 'wake_visible_zone_persistence'
+      wake_visible_zone_artifact
+    end
+  end
+
+  def fullscreen_matrix_artifact
+    path = '/tmp/sanebar_runtime_fullscreen_matrix.json'
+    return nil unless File.exist?(path) && File.mtime(path) >= @started_at - 30 * 60
+
+    payload = JSON.parse(File.read(path))
+    return nil unless payload['status'] == 'pass'
+
+    {
+      evidence_types: Array(payload['evidence_types']).map(&:to_s),
+      evidence_paths: ([path] + Array(payload['evidence_paths'])).map(&:to_s),
+      completed_scenarios: Array(payload['completed_scenarios']).map(&:to_s)
+    }
+  rescue JSON::ParserError
+    nil
+  end
+
+  def wake_visible_zone_artifact
+    path = '/tmp/sanebar_runtime_wake_probe.json'
+    return nil unless File.exist?(path) && File.mtime(path) >= @started_at - 30 * 60
+
+    payload = JSON.parse(File.read(path))
+    proof = payload['visible_zone_persistence']
+    return nil unless payload['status'] == 'pass' && proof.is_a?(Hash) && proof['status'] == 'pass'
+
+    {
+      evidence_types: %w[mini_runtime screenshot log],
+      evidence_paths: [path, '/tmp/sanebar_runtime_wake_probe.log'].select { |candidate| File.exist?(candidate) },
+      completed_scenarios: Array(proof['completed_scenarios']).map(&:to_s)
+    }
+  rescue JSON::ParserError
+    nil
+  end
+
   def write_failure_artifact(error)
     FileUtils.mkdir_p(OUTPUT_DIR)
     path = File.join(OUTPUT_DIR, "customer-ui-action-sweep-failed-#{@timestamp}.txt")