Release 0.31.0

actions-user · actions-user · commit 1dd4c218dcf3 · 2025-02-12T14:31:28.000Z
diff --git a/Tool/Sources/AXNotificationStream/AXNotificationStream.swift b/Tool/Sources/AXNotificationStream/AXNotificationStream.swift
@@ -110,6 +110,7 @@ public final class AXNotificationStream: AsyncSequence {
             )
             var pendingRegistrationNames = Set(notificationNames)
             var retry = 0
+            var shouldLogAXDisabledEvent: Bool = true
             while !pendingRegistrationNames.isEmpty, retry < 100 {
                 guard let self else { return }
                 retry += 1
@@ -125,14 +126,18 @@ public final class AXNotificationStream: AsyncSequence {
                     }
                     switch e {
                     case .success:
+                        shouldLogAXDisabledEvent = true
                         pendingRegistrationNames.remove(name)
                         await Status.shared.updateAXStatus(.granted)
                     case .actionUnsupported:
                         Logger.service.error("AXObserver: Action unsupported: \(name)")
                         pendingRegistrationNames.remove(name)
                     case .apiDisabled:
-                        Logger.service
-                            .error("AXObserver: Accessibility API disabled, will try again later")
+                        if shouldLogAXDisabledEvent { // Avoid keeping log AX disabled too many times
+                            shouldLogAXDisabledEvent = false
+                            Logger.service
+                                .error("AXObserver: Accessibility API disabled, will try again later")
+                        }
                         retry -= 1
                         await Status.shared.updateAXStatus(.notGranted)
                     case .invalidUIElement:
diff --git a/Tool/Sources/BuiltinExtension/BuiltinExtensionTelemetryServiceProvider.swift b/Tool/Sources/BuiltinExtension/BuiltinExtensionTelemetryServiceProvider.swift
@@ -43,11 +43,11 @@ public final class BuiltinExtensionTelemetryServiceProvider<
 
     public func sendError(_ request: TelemetryExceptionRequest) async throws {
         guard let telemetryService else {
-            Logger.service.error("Builtin telemetry service not found.")
+            print("Builtin telemetry service not found.")
             throw BuiltinExtensionTelemetryServiceNotFoundError()
         }
         guard let workspaceInfo = await activeWorkspace() else {
-            Logger.service.error("Builtin active workspace info not found.")
+            print("Builtin active workspace info not found.")
             throw BuiltinExtensionActiveWorkspaceInfoNotFoundError()
         }
         
diff --git a/Tool/Sources/XcodeInspector/XcodeInspector+TriggerCommand.swift b/Tool/Sources/XcodeInspector/XcodeInspector+TriggerCommand.swift
@@ -7,8 +7,33 @@ public extension XcodeAppInstanceInspector {
     func triggerCopilotCommand(name: String, activateXcode: Bool = true) async throws {
         let bundleName = Bundle.main
             .object(forInfoDictionaryKey: "EXTENSION_BUNDLE_NAME") as! String
+        
+        guard await isBundleEnabled(bundleName: bundleName) else {
+            print("Bundle \(bundleName) is not enabled")
+            return
+        }
+        
         try await triggerMenuItem(path: ["Editor", bundleName, name], activateApp: activateXcode)
     }
+    
+    private func isBundleEnabled(bundleName: String) async -> Bool {
+        let app = AXUIElementCreateApplication(runningApplication.processIdentifier)
+        
+        guard let menuBar = app.menuBar,
+              let editorMenu = menuBar.child(title: "Editor") else {
+            return false
+        }
+        
+        if let bundleMenuItem = editorMenu.child(title: bundleName, role: "AXMenuItem") {
+            var enabled: CFTypeRef?
+            let error = AXUIElementCopyAttributeValue(bundleMenuItem, kAXEnabledAttribute as CFString, &enabled)
+            if error == .success, let isEnabled = enabled as? Bool {
+                return isEnabled
+            }
+        }
+        
+        return false
+    }
 }
 
 public extension AppInstanceInspector {

Original file line number	Diff line number	Diff line change
`@@ -43,11 +43,11 @@ public final class BuiltinExtensionTelemetryServiceProvider<`
`43`	`43`
`44`	`44`	`public func sendError(_ request: TelemetryExceptionRequest) async throws {`
`45`	`45`	`guard let telemetryService else {`
`46`		`- Logger.service.error("Builtin telemetry service not found.")`
	`46`	`+ print("Builtin telemetry service not found.")`
`47`	`47`	`throw BuiltinExtensionTelemetryServiceNotFoundError()`
`48`	`48`	`}`
`49`	`49`	`guard let workspaceInfo = await activeWorkspace() else {`
`50`		`- Logger.service.error("Builtin active workspace info not found.")`
	`50`	`+ print("Builtin active workspace info not found.")`
`51`	`51`	`throw BuiltinExtensionActiveWorkspaceInfoNotFoundError()`
`52`	`52`	`}`
`53`	`53`