diff --git a/Jarvis/Networking/WebSocketManager.swift b/Jarvis/Networking/WebSocketManager.swift index 242a415..7560c4d 100644 --- a/Jarvis/Networking/WebSocketManager.swift +++ b/Jarvis/Networking/WebSocketManager.swift @@ -60,13 +60,16 @@ final class WebSocketManager: ObservableObject { } private func listen() { - task?.receive { [weak self] result in - guard let self else { return } + // Capture task locally so the Sendable receive closure doesn't reference + // the @MainActor-isolated property directly. + guard let t = task else { return } + t.receive { [weak self] result in switch result { case .failure: - Task { @MainActor in self.handleDisconnect() } + Task { @MainActor [weak self] in self?.handleDisconnect() } case .success(let message): - Task { @MainActor in + Task { @MainActor [weak self] in + guard let self else { return } self.handle(message: message) self.listen() } @@ -99,9 +102,13 @@ final class WebSocketManager: ObservableObject { private func startPing() { pingTimer?.invalidate() pingTimer = Timer.scheduledTimer(withTimeInterval: 30, repeats: true) { [weak self] _ in - self?.task?.sendPing { error in - if error != nil { - Task { @MainActor in self?.handleDisconnect() } + // Hop to MainActor before touching the isolated `task` property. + Task { @MainActor [weak self] in + guard let self else { return } + self.task?.sendPing { [weak self] error in + if error != nil { + Task { @MainActor [weak self] in self?.handleDisconnect() } + } } } } @@ -119,10 +126,10 @@ final class WebSocketManager: ObservableObject { let delay = min(pow(2.0, Double(attempt - 1)), maxBackoff) connectionState = .reconnecting(attempt: attempt) - reconnectTask = Task { + reconnectTask = Task { [weak self] in try? await Task.sleep(nanoseconds: UInt64(delay * 1_000_000_000)) guard !Task.isCancelled else { return } - await MainActor.run { self.openConnection() } + await MainActor.run { self?.openConnection() } } } }