v2.0.1

Author: bwya77

DockAnchor.app/Contents/CodeResources

@@ -23,13 +23,13 @@
 	<key>CFBundlePackageType</key>
 	<string>APPL</string>
 	<key>CFBundleShortVersionString</key>
-	<string>2.0</string>
+	<string>2.0.1</string>
 	<key>CFBundleSupportedPlatforms</key>
 	<array>
 		<string>MacOSX</string>
 	</array>
 	<key>CFBundleVersion</key>
-	<string>2.0</string>
+	<string>2.0.1</string>
 	<key>DTCompiler</key>
 	<string>com.apple.compilers.llvm.clang.1_0</string>
 	<key>DTPlatformBuild</key>

DockAnchor.app/Contents/Info.plist

@@ -407,7 +407,7 @@
 				CODE_SIGN_STYLE = Automatic;
 				COMBINE_HIDPI_IMAGES = YES;
 				CONFIGURATION_BUILD_DIR = build;
-				CURRENT_PROJECT_VERSION = 2.0;
+				CURRENT_PROJECT_VERSION = 2.0.1;
 				DEAD_CODE_STRIPPING = YES;
 				DEVELOPMENT_TEAM = 8772WA9289;
 				ENABLE_APP_SANDBOX = YES;
@@ -422,7 +422,7 @@
 					"$(inherited)",
 					"@executable_path/../Frameworks",
 				);
-				MARKETING_VERSION = 2.0;
+				MARKETING_VERSION = 2.0.1;
 				PRODUCT_BUNDLE_IDENTIFIER = bwyatt.DockAnchor;
 				PRODUCT_NAME = "$(TARGET_NAME)";
 				REGISTER_APP_GROUPS = YES;
@@ -443,7 +443,7 @@
 				CODE_SIGN_STYLE = Automatic;
 				COMBINE_HIDPI_IMAGES = YES;
 				CONFIGURATION_BUILD_DIR = build;
-				CURRENT_PROJECT_VERSION = 2.0;
+				CURRENT_PROJECT_VERSION = 2.0.1;
 				DEAD_CODE_STRIPPING = YES;
 				DEVELOPMENT_TEAM = 8772WA9289;
 				ENABLE_APP_SANDBOX = YES;
@@ -458,7 +458,7 @@
 					"$(inherited)",
 					"@executable_path/../Frameworks",
 				);
-				MARKETING_VERSION = 2.0;
+				MARKETING_VERSION = 2.0.1;
 				PRODUCT_BUNDLE_IDENTIFIER = bwyatt.DockAnchor;
 				PRODUCT_NAME = "$(TARGET_NAME)";
 				REGISTER_APP_GROUPS = YES;

DockAnchor.app/Contents/MacOS/DockAnchor

@@ -344,6 +344,7 @@ struct ContentView: View {
     @State private var showingNewProfile = false
     @State private var editingProfile: DockProfile?
     @State private var newProfileName = ""
+    @State private var showingPermissionHelp = false
 
     private var statusColor: Color {
         if dockMonitor.statusMessage.contains("Blocked") {
@@ -393,7 +394,21 @@ struct ContentView: View {
                     Spacer()
                 }
 
-                if !dockMonitor.isActive {
+                if dockMonitor.needsPermissionReset || dockMonitor.statusMessage.lowercased().contains("permission") {
+                    HStack {
+                        Image(systemName: "exclamationmark.triangle.fill")
+                            .foregroundColor(.orange)
+                        Text("Accessibility permission required")
+                            .font(.caption)
+                            .foregroundColor(.secondary)
+                        Spacer()
+                        Button("Help") {
+                            showingPermissionHelp = true
+                        }
+                        .font(.caption)
+                        .buttonStyle(.bordered)
+                    }
+                } else if !dockMonitor.isActive {
                     HStack {
                         Image(systemName: "exclamationmark.triangle")
                             .foregroundColor(.orange)
@@ -533,9 +548,21 @@ struct ContentView: View {
                 .environmentObject(appSettings)
                 .preferredColorScheme(appSettings.appTheme.colorScheme)
         }
+        .sheet(isPresented: $showingPermissionHelp) {
+            PermissionHelpSheet(dockMonitor: dockMonitor)
+                .preferredColorScheme(appSettings.appTheme.colorScheme)
+        }
         .onAppear {
-            // Request permissions on startup
-            _ = dockMonitor.requestAccessibilityPermissions()
+            // Check permissions on startup and show help if not granted
+            let hasPermissions = dockMonitor.requestAccessibilityPermissions()
+            if !hasPermissions {
+                // Small delay to let the UI appear first
+                DispatchQueue.main.asyncAfter(deadline: .now() + 0.5) {
+                    showingPermissionHelp = true
+                }
+                // Don't perform any operations that require accessibility permissions
+                return
+            }
             // Update available displays
             dockMonitor.updateAvailableDisplays()
             // Set the anchor display from settings (using UUID for stable identification)
@@ -700,6 +727,79 @@ struct SettingsView: View {
     }
 }
 
+struct PermissionHelpSheet: View {
+    @ObservedObject var dockMonitor: DockMonitor
+    @Environment(\.dismiss) var dismiss
+
+    var body: some View {
+        VStack(spacing: 20) {
+            HStack {
+                Text("Accessibility Permission")
+                    .font(.title2)
+                    .fontWeight(.bold)
+                Spacer()
+                Button("Done") {
+                    dismiss()
+                }
+                .buttonStyle(.bordered)
+            }
+            .padding(.horizontal)
+            .padding(.top, 16)
+
+            VStack(alignment: .leading, spacing: 16) {
+                Text("DockAnchor requires Accessibility permission to monitor mouse movement and keep your dock anchored.")
+                    .font(.body)
+
+                Divider()
+
+                Text("How to enable:")
+                    .font(.headline)
+
+                VStack(alignment: .leading, spacing: 8) {
+                    Label("Click \"Open Accessibility Settings\" below", systemImage: "1.circle.fill")
+                    Label("Find DockAnchor in the list", systemImage: "2.circle.fill")
+                    Label("Toggle it ON", systemImage: "3.circle.fill")
+                }
+                .font(.body)
+
+                Divider()
+
+                Text("If permission doesn't work after an update:")
+                    .font(.headline)
+
+                VStack(alignment: .leading, spacing: 8) {
+                    Label("Remove DockAnchor from the list (- button)", systemImage: "1.circle")
+                    Label("Re-add it (+ button)", systemImage: "2.circle")
+                    Label("Toggle it ON", systemImage: "3.circle")
+                }
+                .font(.body)
+                .foregroundColor(.secondary)
+
+                Spacer()
+
+                Button(action: {
+                    dockMonitor.openAccessibilityPreferences()
+                }) {
+                    HStack {
+                        Image(systemName: "gearshape.fill")
+                        Text("Open Accessibility Settings")
+                    }
+                    .frame(maxWidth: .infinity)
+                    .padding(.vertical, 10)
+                    .background(Color.blue)
+                    .foregroundColor(.white)
+                    .cornerRadius(8)
+                }
+                .buttonStyle(.plain)
+            }
+            .padding(.horizontal)
+            .padding(.bottom, 16)
+        }
+        .frame(width: 380, height: 420)
+        .background(.background)
+    }
+}
+
 #Preview {
     ContentView()
         .environment(\.managedObjectContext, PersistenceController.preview.container.viewContext)

DockAnchor.xcodeproj/project.pbxproj

@@ -115,23 +115,27 @@ class ApplicationDelegate: NSObject, NSApplicationDelegate, ObservableObject {
         // This is the most critical piece - ensures menu bar icon appears
         menuBarManager.setup(appSettings: appSettings, dockMonitor: dockMonitor, updateChecker: updateChecker)
 
-        // Set the anchor display from settings (using UUID for stable identification)
-        dockMonitor.changeAnchorDisplay(toUUID: appSettings.selectedDisplayUUID)
-
         // Set initial activation policy
         updateActivationPolicy()
 
-        // Auto-start monitoring if enabled (with a small delay for system stability)
-        if appSettings.runInBackground {
-            DispatchQueue.main.asyncAfter(deadline: .now() + 1.0) { [weak self] in
-                self?.dockMonitor.startMonitoring()
+        // Only perform accessibility-dependent operations if permissions are granted
+        let hasPermissions = dockMonitor.requestAccessibilityPermissions()
+        if hasPermissions {
+            // Set the anchor display from settings (using UUID for stable identification)
+            dockMonitor.changeAnchorDisplay(toUUID: appSettings.selectedDisplayUUID)
+
+            // Auto-start monitoring if enabled (with a small delay for system stability)
+            if appSettings.runInBackground {
+                DispatchQueue.main.asyncAfter(deadline: .now() + 1.0) { [weak self] in
+                    self?.dockMonitor.startMonitoring()
+                }
             }
-        }
 
-        // Auto-relocate dock to anchored display on launch if enabled
-        if appSettings.autoRelocateDock {
-            DispatchQueue.main.asyncAfter(deadline: .now() + 1.5) { [weak self] in
-                self?.dockMonitor.relocateDockToAnchoredDisplay()
+            // Auto-relocate dock to anchored display on launch if enabled
+            if appSettings.autoRelocateDock {
+                DispatchQueue.main.asyncAfter(deadline: .now() + 1.5) { [weak self] in
+                    self?.dockMonitor.relocateDockToAnchoredDisplay()
+                }
             }
         }
 

DockAnchor.zip

@@ -90,13 +90,15 @@ class DockMonitor: NSObject, ObservableObject {
     @Published var anchoredDisplay: String = "Primary"
     @Published var statusMessage = "Dock Anchor Ready"
     @Published var availableDisplays: [DisplayInfo] = []
+    @Published var needsPermissionReset = false
 
     private var eventTap: CFMachPort?
     private var runLoopSource: CFRunLoopSource?
     private var isMonitoring = false
     private var anchorDisplayUUID: String = ""  // Hardware UUID for stable anchor tracking
     private var dockPosition: DockPosition = .bottom
     private var cancellables = Set<AnyCancellable>()
+    private var permissionCheckTimer: Timer?
 
     /// Gets the current anchor display ID (derived from UUID)
     private var anchorDisplayID: CGDirectDisplayID {
@@ -373,18 +375,80 @@ class DockMonitor: NSObject, ObservableObject {
     }
 
     func requestAccessibilityPermissions() -> Bool {
-        let options = [kAXTrustedCheckOptionPrompt.takeRetainedValue(): true]
-        let trusted = AXIsProcessTrustedWithOptions(options as CFDictionary)
-        
+        // Check if already trusted (without prompting)
+        let trusted = AXIsProcessTrusted()
+
         if !trusted {
             DispatchQueue.main.async { [weak self] in
                 self?.statusMessage = "Accessibility permissions required"
             }
+        } else {
+            DispatchQueue.main.async { [weak self] in
+                self?.needsPermissionReset = false
+            }
         }
-        
+
         return trusted
     }
-    
+
+    /// Prompts for accessibility permissions by opening System Preferences
+    /// Note: On modern macOS, the system dialog often just opens System Preferences
+    /// without actually adding the app - users must manually add it with the + button
+    func promptForAccessibilityPermissions() {
+        // Use the string key directly to avoid takeRetainedValue() issues
+        let options = [kAXTrustedCheckOptionPrompt.takeUnretainedValue() as String: true] as CFDictionary
+        AXIsProcessTrustedWithOptions(options)
+    }
+
+    /// Checks accessibility permissions without prompting
+    private func checkAccessibilityPermissions() -> Bool {
+        return AXIsProcessTrusted()
+    }
+
+    /// Opens System Preferences to the Accessibility pane
+    func openAccessibilityPreferences() {
+        if let url = URL(string: "x-apple.systempreferences:com.apple.preference.security?Privacy_Accessibility") {
+            NSWorkspace.shared.open(url)
+        }
+    }
+
+    /// Starts the timer that periodically checks if permissions are still valid
+    private func startPermissionMonitoring() {
+        // Check every 2 seconds for permission changes
+        permissionCheckTimer = Timer.scheduledTimer(withTimeInterval: 2.0, repeats: true) { [weak self] _ in
+            self?.verifyPermissionsAndTapValidity()
+        }
+    }
+
+    /// Stops the permission monitoring timer
+    private func stopPermissionMonitoring() {
+        permissionCheckTimer?.invalidate()
+        permissionCheckTimer = nil
+    }
+
+    /// Verifies that accessibility permissions are still granted and event tap is valid
+    private func verifyPermissionsAndTapValidity() {
+        guard isMonitoring else { return }
+
+        // Check if accessibility permissions are still granted
+        if !checkAccessibilityPermissions() {
+            DispatchQueue.main.async { [weak self] in
+                self?.statusMessage = "Accessibility permissions revoked - stopping monitoring"
+                self?.stopMonitoring()
+            }
+            return
+        }
+
+        // Check if the event tap is still valid
+        if let tap = eventTap, !CFMachPortIsValid(tap) {
+            DispatchQueue.main.async { [weak self] in
+                self?.statusMessage = "Event tap invalidated - stopping monitoring"
+                self?.stopMonitoring()
+            }
+            return
+        }
+    }
+
     func startMonitoring() {
         guard requestAccessibilityPermissions() else {
             statusMessage = "Please grant accessibility permissions in System Preferences"
@@ -426,7 +490,11 @@ class DockMonitor: NSObject, ObservableObject {
         )
 
         guard let eventTap = eventTap else {
-            statusMessage = "Failed to create event tap"
+            // Event tap creation failed even though permissions appeared granted.
+            // This usually means the permission entry is stale (app was updated).
+            // The user needs to remove and re-add the app in Accessibility settings.
+            needsPermissionReset = true
+            statusMessage = "Permission needs reset - remove and re-add app in Accessibility settings"
             return
         }
 
@@ -435,13 +503,15 @@ class DockMonitor: NSObject, ObservableObject {
         CGEvent.tapEnable(tap: eventTap, enable: true)
 
         isMonitoring = true
+        startPermissionMonitoring()
         DispatchQueue.main.async { [weak self] in
             self?.isActive = true
             self?.statusMessage = "Dock Anchor Active - Monitoring mouse movement"
         }
     }
-    
+
     func stopMonitoring() {
+        stopPermissionMonitoring()
         guard isMonitoring else { return }
 
         isMonitoring = false