From 12ef75386cf132e19c3c1c5d897f2bc5c855552f Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Wed, 30 Jul 2025 03:44:07 +0200 Subject: [PATCH] fix: Update Mac app to use path-based URLs for sessions The Mac app was still using the old query parameter format (?session=id) while the web app expects path-based URLs (/session/id). This caused sessions not backed by windows to open with the wrong URL format. Updated DashboardURLBuilder to generate path-based URLs matching the web app's routing expectations. --- mac/VibeTunnel/Core/Utilities/DashboardURLBuilder.swift | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/mac/VibeTunnel/Core/Utilities/DashboardURLBuilder.swift b/mac/VibeTunnel/Core/Utilities/DashboardURLBuilder.swift index 2b5e0e60..2bf7262d 100644 --- a/mac/VibeTunnel/Core/Utilities/DashboardURLBuilder.swift +++ b/mac/VibeTunnel/Core/Utilities/DashboardURLBuilder.swift @@ -14,10 +14,8 @@ enum DashboardURLBuilder { static func dashboardURL(port: String, sessionId: String? = nil) -> URL? { let serverManager = ServerManager.shared if let sessionId { - return serverManager.buildURL( - endpoint: "/", - queryItems: [URLQueryItem(name: "session", value: sessionId)] - ) + // Use path-based URL format: /session/sessionId + return serverManager.buildURL(endpoint: "/session/\(sessionId)") } else { return serverManager.buildURL(endpoint: "/") }