@superfan-app/spotify-auth 0.1.79 → 0.1.81

package/ios/SpotifyAuthAuth.swift

@@ -492,26 +492,40 @@ final class SpotifyAuthAuth: NSObject, SPTSessionManagerDelegate {
     // URLComponents.percentEncodedQuery properly encodes the values
     request.httpBody = components.percentEncodedQuery?.data(using: .utf8)
     
+    print("[SpotifyDebug] exchangeCodeForToken: sending request to \(url)")
     let task = URLSession.shared.dataTask(with: request) { [weak self] data, response, error in
+        print("[SpotifyDebug] exchangeCodeForToken: dataTask callback fired, self=\(self != nil), error=\(String(describing: error))")
+        if let http = response as? HTTPURLResponse {
+            print("[SpotifyDebug] exchangeCodeForToken: HTTP status \(http.statusCode)")
+        }
+        if let data = data, let body = String(data: data, encoding: .utf8) {
+            print("[SpotifyDebug] exchangeCodeForToken: response body = \(body)")
+        }
         do {
             let responseData = try self?.validateHTTPResponse(data: data, response: response, error: error)
-            guard let responseData = responseData else { return }
+            guard let responseData = responseData else {
+                print("[SpotifyDebug] exchangeCodeForToken: responseData is nil (self may be nil)")
+                return
+            }
             let parsed = try Self.parseTokenJSON(from: responseData)
-            
+            print("[SpotifyDebug] exchangeCodeForToken: parsed token OK, hasRefreshToken=\(parsed.refreshToken != nil)")
+
             guard let refreshToken = parsed.refreshToken else {
                 throw SpotifyAuthError.tokenError("Missing refresh_token in response")
             }
-            
+
             let expirationDate = Date(timeIntervalSinceNow: parsed.expiresIn)
             let sessionData = SpotifySessionData(accessToken: parsed.accessToken, refreshToken: refreshToken, expirationDate: expirationDate, scope: parsed.scope)
             DispatchQueue.main.async {
+                print("[SpotifyDebug] exchangeCodeForToken: setting currentSession on main thread")
                 self?.currentSession = sessionData
             }
         } catch {
+            print("[SpotifyDebug] exchangeCodeForToken: caught error: \(error)")
             self?.handleError(error, context: "token_exchange")
         }
     }
-    
+
     task.resume()
   }
   

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@superfan-app/spotify-auth",
-  "version": "0.1.79",
+  "version": "0.1.81",
   "description": "Spotify OAuth module for Expo",
   "main": "src/index.tsx",
   "types": "build/index.d.ts",

package/plugin/build/index.js

@@ -111,8 +111,16 @@ const withSpotifyAndroidManifest = (config, props) => {
         if (!mainActivity['intent-filter']) {
             mainActivity['intent-filter'] = [];
         }
-        // Check if we already have a Spotify redirect intent filter
-        const hasSpotifyIntentFilter = mainActivity['intent-filter'].some((filter) => filter.data?.some((d) => d.$?.['android:scheme'] === props.scheme && d.$?.['android:host'] === callbackHost));
+        // Check if we already have an intent filter covering the Spotify redirect.
+        // A broad scheme-only filter (no android:host) already covers superfan://callback,
+        // so we must treat it as sufficient to avoid registering two overlapping filters
+        // (which causes Android to show the app twice in the disambiguation dialog).
+        const hasSpotifyIntentFilter = mainActivity['intent-filter'].some((filter) => filter.data?.some((d) => {
+            const scheme = d.$?.['android:scheme'];
+            const host = d.$?.['android:host'];
+            // Matches if same scheme + matching host, OR same scheme with no host restriction
+            return scheme === props.scheme && (!host || host === callbackHost);
+        }));
         if (!hasSpotifyIntentFilter) {
             // Build the data element: android:host must be the hostname only (no path).
             // If the callback has a path component (e.g. "auth/spotify"), add android:pathPrefix.

package/plugin/src/index.ts

@@ -128,12 +128,18 @@ const withSpotifyAndroidManifest: ConfigPlugin<SpotifyConfig> = (config, props)
       mainActivity['intent-filter'] = [];
     }
 
-    // Check if we already have a Spotify redirect intent filter
+    // Check if we already have an intent filter covering the Spotify redirect.
+    // A broad scheme-only filter (no android:host) already covers superfan://callback,
+    // so we must treat it as sufficient to avoid registering two overlapping filters
+    // (which causes Android to show the app twice in the disambiguation dialog).
     const hasSpotifyIntentFilter = mainActivity['intent-filter'].some(
       (filter: any) =>
-        filter.data?.some(
-          (d: any) => d.$?.['android:scheme'] === props.scheme && d.$?.['android:host'] === callbackHost
-        )
+        filter.data?.some((d: any) => {
+          const scheme = d.$?.['android:scheme'];
+          const host = d.$?.['android:host'];
+          // Matches if same scheme + matching host, OR same scheme with no host restriction
+          return scheme === props.scheme && (!host || host === callbackHost);
+        })
     );
 
     if (!hasSpotifyIntentFilter) {