Pre-release 0.32.111

Author: actions-user

Diff for: Copilot for Xcode/App.swift

@@ -13,18 +13,28 @@ struct VisualEffect: NSViewRepresentable {
 }
 
 class AppDelegate: NSObject, NSApplicationDelegate {
+    private var permissionAlertShown = false
+    
     // Launch modes supported by the app
     enum LaunchMode {
         case chat
         case settings
     }
 
     func applicationDidFinishLaunching(_ notification: Notification) {
+        if #available(macOS 13.0, *) {
+            checkBackgroundPermissions()
+        }
+        
         let launchMode = determineLaunchMode()
         handleLaunchMode(launchMode)
     }
 
     func applicationShouldHandleReopen(_ sender: NSApplication, hasVisibleWindows flag: Bool) -> Bool {
+        if #available(macOS 13.0, *) {
+            checkBackgroundPermissions()
+        }
+        
         let launchMode = determineLaunchMode()
         handleLaunchMode(launchMode)
         return true
@@ -73,6 +83,28 @@ class AppDelegate: NSObject, NSApplicationDelegate {
         }
     }
 
+    @available(macOS 13.0, *)
+    private func checkBackgroundPermissions() {
+        Task {
+            // Direct check of permission status
+            let launchAgentManager = LaunchAgentManager()
+            let isPermissionGranted = await launchAgentManager.isBackgroundPermissionGranted()
+            
+            if !isPermissionGranted {
+                // Only show alert if permission isn't granted
+                DispatchQueue.main.async {
+                    if !self.permissionAlertShown {
+                        showBackgroundPermissionAlert()
+                        self.permissionAlertShown = true
+                    }
+                }
+            } else {
+                // Permission is granted, reset flag
+                self.permissionAlertShown = false
+            }
+        }
+    }
+    
     // MARK: - Application Termination
 
     func applicationShouldTerminate(_ sender: NSApplication) -> NSApplication.TerminateReply {

Diff for: Core/Sources/ConversationTab/ChatPanel.swift

@@ -761,11 +761,24 @@ struct ChatPanelInputArea: View {
             return result
         }
         func subscribeToActiveDocumentChangeEvent() {
-            XcodeInspector.shared.$activeDocumentURL.receive(on: DispatchQueue.main)
-                .sink { newDocURL in
-                    if supportedFileExtensions.contains(newDocURL?.pathExtension ?? "") {
-                        let currentEditor = FileReference(url: newDocURL!, isCurrentEditor: true)
-                        chat.send(.setCurrentEditor(currentEditor))
+            Publishers.CombineLatest(
+                XcodeInspector.shared.$latestActiveXcode,
+                XcodeInspector.shared.$activeDocumentURL
+                    .removeDuplicates()
+                )
+                .receive(on: DispatchQueue.main)
+                .sink { newXcode, newDocURL in
+                    // First check for realtimeWorkspaceURL if activeWorkspaceURL is nil
+                    if let realtimeURL = newXcode?.realtimeDocumentURL, newDocURL == nil {
+                        if supportedFileExtensions.contains(realtimeURL.pathExtension) {
+                            let currentEditor = FileReference(url: realtimeURL, isCurrentEditor: true)
+                            chat.send(.setCurrentEditor(currentEditor))
+                        }
+                    } else {
+                        if supportedFileExtensions.contains(newDocURL?.pathExtension ?? "") {
+                            let currentEditor = FileReference(url: newDocURL!, isCurrentEditor: true)
+                            chat.send(.setCurrentEditor(currentEditor))
+                        }
                     }
                 }
                 .store(in: &cancellable)

Diff for: Core/Sources/HostApp/General.swift

@@ -58,7 +58,7 @@ public struct General {
                                 .setupLaunchAgentForTheFirstTimeIfNeeded()
                         } catch {
                             Logger.ui.error("Failed to setup launch agent. \(error.localizedDescription)")
-                            toast(error.localizedDescription, .error)
+                            toast("Operation failed: permission denied. This may be due to missing background permissions.", .error)
                         }
                         await send(.reloadStatus)
                     }
@@ -103,7 +103,7 @@ public struct General {
                     } catch let error as XPCCommunicationBridgeError {
                         Logger.ui.error("Failed to reach communication bridge. \(error.localizedDescription)")
                         toast(
-                            "Failed to reach communication bridge. \(error.localizedDescription)",
+                            "Unable to connect to the communication bridge. The helper application didn't respond. This may be due to missing background permissions.",
                             .error
                         )
                         await send(.failedReloading)

Diff for: Core/Sources/HostApp/LaunchAgentManager.swift

@@ -1,7 +1,7 @@
 import Foundation
 import LaunchAgentManager
 
-extension LaunchAgentManager {
+public extension LaunchAgentManager {
     init() {
         self.init(
             serviceIdentifier: Bundle.main

Diff for: Core/Sources/LaunchAgentManager/LaunchAgentManager.swift

@@ -33,6 +33,14 @@ public struct LaunchAgentManager {
             await removeObsoleteLaunchAgent()
         }
     }
+    
+    @available(macOS 13.0, *)
+    public func isBackgroundPermissionGranted() async -> Bool {
+        // On macOS 13+, check SMAppService status
+        let bridgeLaunchAgent = SMAppService.agent(plistName: "bridgeLaunchAgent.plist")
+        let status = bridgeLaunchAgent.status
+        return status != .requiresApproval
+    }
 
     public func setupLaunchAgent() async throws {
         if #available(macOS 13, *) {

Diff for: Core/Sources/Service/Service.swift

@@ -114,15 +114,28 @@ public final class Service {
                 }.store(in: &cancellable)
 
             // Combine both workspace and auth status changes into a single stream
-            await Publishers.CombineLatest(
+            await Publishers.CombineLatest3(
+                XcodeInspector.shared.safe.$latestActiveXcode,
                 XcodeInspector.shared.safe.$activeWorkspaceURL
                     .removeDuplicates(),
                 StatusObserver.shared.$authStatus
                     .removeDuplicates()
                 )
                 .receive(on: DispatchQueue.main)
-                .sink { [weak self] newURL, newStatus in
-                    self?.onNewActiveWorkspaceURLOrAuthStatus(newURL: newURL, newStatus: newStatus)
+                .sink { [weak self] newXcode, newURL, newStatus in
+                    // First check for realtimeWorkspaceURL if activeWorkspaceURL is nil
+                    if let realtimeURL = newXcode?.realtimeWorkspaceURL, newURL == nil {
+                        self?.onNewActiveWorkspaceURLOrAuthStatus(
+                            newURL: realtimeURL,
+                            newStatus: newStatus
+                        )
+                    } else if let newURL = newURL {
+                        // Then use activeWorkspaceURL if available
+                        self?.onNewActiveWorkspaceURLOrAuthStatus(
+                            newURL: newURL,
+                            newStatus: newStatus
+                        )
+                    }
                 }
                 .store(in: &cancellable)
         }
@@ -137,7 +150,7 @@ public final class Service {
 
     private func getDisplayNameOfXcodeWorkspace(url: URL) -> String {
         var name = url.lastPathComponent
-        let suffixes = [".xcworkspace", ".xcodeproj"]
+        let suffixes = [".xcworkspace", ".xcodeproj", ".playground"]
         for suffix in suffixes {
             if name.hasSuffix(suffix) {
                 name = String(name.dropLast(suffix.count))

Diff for: ExtensionService/XPCController.swift

@@ -39,19 +39,38 @@ final class XPCController: XPCServiceDelegate {
     func createPingTask() {
         pingTask?.cancel()
         pingTask = Task { [weak self] in
+            var consecutiveFailures = 0
+            var backoffDelay = 1_000_000_000 // Start with 1 second
+            
             while !Task.isCancelled {
                 guard let self else { return }
                 do {
                     try await self.bridge.updateServiceEndpoint(self.xpcListener.endpoint)
-                    try await Task.sleep(nanoseconds: 60_000_000_000)
+                    // Reset on success
+                    consecutiveFailures = 0
+                    backoffDelay = 1_000_000_000
+                    try await Task.sleep(nanoseconds: 60_000_000_000) // 60 seconds between successful pings
                 } catch {
-                    try await Task.sleep(nanoseconds: 1_000_000_000)
+                    consecutiveFailures += 1
+                    // Log only on 1st, 5th (31 sec), 10th failures, etc. to avoid flooding
+                    let shouldLog = consecutiveFailures == 1 || consecutiveFailures % 5 == 0
+                    
                     #if DEBUG
                     // No log, but you should run CommunicationBridge, too.
                     #else
-                    Logger.service
-                        .error("Failed to connect to bridge: \(error.localizedDescription)")
+                    if consecutiveFailures == 5 {
+                        if #available(macOS 13.0, *) {
+                            showBackgroundPermissionAlert()
+                        }
+                    }
+                    if shouldLog {
+                        Logger.service.error("Failed to connect to bridge (\(consecutiveFailures) consecutive failures): \(error.localizedDescription)")
+                    }
                     #endif
+                    
+                    // Exponential backoff with a cap
+                    backoffDelay = min(backoffDelay * 2, 120_000_000_000) // Cap at 120 seconds
+                    try await Task.sleep(nanoseconds: UInt64(backoffDelay))
                 }
             }
         }

Diff for: Tool/Sources/GitHubCopilotService/LanguageServer/CopilotLocalProcessServer.swift

@@ -311,7 +311,7 @@ extension CustomJSONRPCLanguageServer {
                             .updateCLSStatus(
                                 payload.kind.clsStatus,
                                 busy: payload.busy,
-                                message: payload.message
+                                message: payload.message ?? ""
                             )
                     }
                 }

Diff for: Tool/Sources/GitHubCopilotService/LanguageServer/GitHubCopilotRequest.swift

@@ -423,7 +423,7 @@ public enum GitHubCopilotNotification {
 
         public var kind: StatusKind
         public var busy: Bool
-        public var message: String
+        public var message: String?
 
         public static func decode(fromParams params: JSONValue?) -> StatusNotification? {
             try? JSONDecoder().decode(Self.self, from: (try? JSONEncoder().encode(params)) ?? Data())

Diff for: Tool/Sources/GitHubCopilotService/LanguageServer/GitHubCopilotService.swift

@@ -207,7 +207,18 @@ public class GitHubCopilotBaseService {
             server.defaultTimeout = 60
             server.initializeParamsProvider = {
                 let capabilities = ClientCapabilities(
-                    workspace: nil,
+                    workspace: .init(
+                        applyEdit: false,
+                        workspaceEdit: nil,
+                        didChangeConfiguration: nil,
+                        didChangeWatchedFiles: nil,
+                        symbol: nil,
+                        executeCommand: nil,
+                        /// enable for "watchedFiles capability", set others to default value
+                        workspaceFolders: true,
+                        configuration: nil,
+                        semanticTokens: nil
+                    ),
                     textDocument: nil,
                     window: nil,
                     general: nil,
@@ -726,13 +737,13 @@ public final class GitHubCopilotService:
     private func updateServiceAuthStatus(_ status: GitHubCopilotRequest.CheckStatus.Response) async {
         Logger.gitHubCopilot.info("check status response: \(status)")
         if status.status == .ok || status.status == .maybeOk {
+            await Status.shared.updateAuthStatus(.loggedIn, username: status.user)
             if !CopilotModelManager.hasLLMs() {
                 let models = try? await models()
                 if let models = models, !models.isEmpty {
                     CopilotModelManager.updateLLMs(models)
                 }
             }
-            await Status.shared.updateAuthStatus(.loggedIn, username: status.user)
             await unwatchAuthStatus()
         } else if status.status == .notAuthorized {
             await Status.shared

Diff for: Tool/Sources/XPCShared/XPCCommunicationBridge.swift

@@ -1,5 +1,6 @@
 import Foundation
 import Logger
+import AppKit
 
 public enum XPCCommunicationBridgeError: Swift.Error, LocalizedError {
     case failedToCreateXPCConnection
@@ -79,3 +80,18 @@ extension XPCCommunicationBridge {
     }
 }
 
+@available(macOS 13.0, *)
+public func showBackgroundPermissionAlert() {
+    let alert = NSAlert()
+    alert.messageText = "Background Permission Required"
+    alert.informativeText = "GitHub Copilot for Xcode needs permission to run in the background. Without this permission, features won't work correctly."
+    alert.alertStyle = .warning
+    
+    alert.addButton(withTitle: "Open Settings")
+    alert.addButton(withTitle: "Later")
+    
+    let response = alert.runModal()
+    if response == .alertFirstButtonReturn {
+        NSWorkspace.shared.open(URL(string: "x-apple.systempreferences:com.apple.LoginItems-Settings.extension")!)
+    }
+}