Fix Wasm incompatibility in web error processing (#1651)

Replace `is JSObject` runtime check with Wasm-safe `isA<JSObject>()`
in `_processError`. The `is` operator on JS interop types is unreliable
under Wasm compilation due to type erasure differences between JS and
Wasm targets.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: brunovsiqueira

CHANGELOG.md

@@ -3,6 +3,9 @@
 ### ✨ New Features
 * Add setAppstackAttributionParams API (#1671) via Rick (@rickvdl)
 
+### 🐞 Bugfixes
+* Fix Wasm incompatibility: replace `is JSObject` runtime check with `isA<JSObject>()` in web error processing (#1651)
+
 ## 9.13.2
 ## RevenueCat SDK
 ### 📦 Dependency Updates

lib/web/purchases_flutter_web.dart

@@ -11,7 +11,8 @@ import '../purchases_flutter.dart';
 
 class PurchasesFlutterPlugin {
   static final _unknownErrorCode = '${PurchasesErrorCode.unknownError.index}';
-  static final _configurationErrorCode = '${PurchasesErrorCode.configurationError.index}';
+  static final _configurationErrorCode =
+      '${PurchasesErrorCode.configurationError.index}';
   static const _purchasesHybridMappingsVersion = '17.50.0';
   static const _platformName = 'flutter';
   static const _pluginVersion = '9.14.0';
@@ -26,9 +27,10 @@ class PurchasesFlutterPlugin {
     }
     _initCompleter = Completer<void>();
 
-    final script = HTMLScriptElement()
-      ..type = 'text/javascript'
-      ..text = '''
+    final script =
+        HTMLScriptElement()
+          ..type = 'text/javascript'
+          ..text = '''
         window.after_rc_load_callback = async (callback) => {
           callback(await import("$_purchasesHybridMappingsUrl"));
         };
@@ -40,7 +42,7 @@ class PurchasesFlutterPlugin {
         "The current document doesn't have a head element which is required to insert a script.",
       );
     }
-    
+
     head.append(script);
 
     globalContext.callMethod(
@@ -71,8 +73,8 @@ class PurchasesFlutterPlugin {
   Future<dynamic> handleMethodCall(MethodCall call) async {
     if (_initCompleter == null) {
       throw PlatformException(
-          code: _unknownErrorCode,
-          message: 'Purchases SDK not loaded on method call: ${call.method}',
+        code: _unknownErrorCode,
+        message: 'Purchases SDK not loaded on method call: ${call.method}',
       );
     }
     await _initCompleter?.future;
@@ -183,7 +185,7 @@ class PurchasesFlutterPlugin {
     };
     final appUserId = arguments['appUserId'] as String?;
 
-    if(appUserId != null && appUserId.isNotEmpty) {
+    if (appUserId != null && appUserId.isNotEmpty) {
       options['appUserId'] = appUserId;
     }
 
@@ -218,8 +220,8 @@ class PurchasesFlutterPlugin {
   ) async {
     final placementIdentifier = arguments['placementIdentifier'] as String;
     return await _getNullableMapFromInstanceMethod(
-        'getCurrentOfferingForPlacement',
-        [placementIdentifier],
+      'getCurrentOfferingForPlacement',
+      [placementIdentifier],
     );
   }
 
@@ -248,9 +250,9 @@ class PurchasesFlutterPlugin {
   }
 
   Future<Map<String, dynamic>> _restorePurchases() async =>
-    // Web SDK doesn't support restore purchases, but returns current customer info
-    // to match behavior with other platforms
-    await _getCustomerInfo();
+      // Web SDK doesn't support restore purchases, but returns current customer info
+      // to match behavior with other platforms
+      await _getCustomerInfo();
 
   Future<void> _close() async {
     _getInstance().callMethod('close'.toJS);
@@ -271,18 +273,16 @@ class PurchasesFlutterPlugin {
   ) async {
     // Web SDK doesn't support trial/intro eligibility checking
     // Return unknown status for all products
-    final productIdentifiers =
-        List<String>.from(arguments['productIdentifiers']);
+    final productIdentifiers = List<String>.from(
+      arguments['productIdentifiers'],
+    );
     return Map.fromEntries(
       productIdentifiers.map(
-        (id) => MapEntry(
-          id,
-          {
-            'status': 0, // introEligibilityStatusUnknown
-            'description':
-                'Trial or intro price eligibility checking not supported on web',
-          },
-        ),
+        (id) => MapEntry(id, {
+          'status': 0, // introEligibilityStatusUnknown
+          'description':
+              'Trial or intro price eligibility checking not supported on web',
+        }),
       ),
     );
   }
@@ -319,29 +319,25 @@ class PurchasesFlutterPlugin {
   }
 
   Future<Map<String, dynamic>> _getVirtualCurrencies() async =>
-    await _getMapFromInstanceMethod('getVirtualCurrencies', []);
+      await _getMapFromInstanceMethod('getVirtualCurrencies', []);
 
   Future<void> _invalidateVirtualCurrenciesCache() async {
     _getInstance().callMethod('invalidateVirtualCurrenciesCache'.toJS);
   }
 
   Future<Map<String, dynamic>?> _getCachedVirtualCurrencies() async =>
       _getNullableMapFromInstanceSyncMethod('getCachedVirtualCurrencies');
-    
+
   // Helper functions to handle JS interop
 
-  Object? _callStaticMethod(
-    String methodName,
-    List<dynamic> args,
-  ) {
+  Object? _callStaticMethod(String methodName, List<dynamic> args) {
     final purchasesStatic = _getStaticPurchasesCommon();
     final processedArgs = _processArgs(args);
 
     try {
-      return purchasesStatic.callMethodVarArgs(
-          methodName.toJS,
-          processedArgs,
-      ).dartify();
+      return purchasesStatic
+          .callMethodVarArgs(methodName.toJS, processedArgs)
+          .dartify();
     } catch (e) {
       throw PlatformException(
         code: _unknownErrorCode,
@@ -350,10 +346,7 @@ class PurchasesFlutterPlugin {
     }
   }
 
-  JSObject _callInstanceMethod(
-      String methodName,
-      List<dynamic> args,
-  ) {
+  JSObject _callInstanceMethod(String methodName, List<dynamic> args) {
     final purchases = _getInstance();
     final jsArgs = _processArgs(args);
     return purchases.callMethodVarArgs(methodName.toJS, jsArgs);
@@ -373,7 +366,8 @@ class PurchasesFlutterPlugin {
   }
 
   JSObject _getStaticPurchasesCommon() {
-    final purchasesHybridMappings = globalContext['PurchasesHybridMappings'] as JSObject;
+    final purchasesHybridMappings =
+        globalContext['PurchasesHybridMappings'] as JSObject;
     JSObject? purchasesCommon;
     if (purchasesHybridMappings.has('PurchasesCommon')) {
       purchasesCommon = purchasesHybridMappings['PurchasesCommon'] as JSObject;
@@ -388,17 +382,16 @@ class PurchasesFlutterPlugin {
   }
 
   Future _callStaticMethodReturningPromise(
-      String methodName,
-      List<dynamic> args,
-      ) async {
+    String methodName,
+    List<dynamic> args,
+  ) async {
     final promise = _callStaticMethod(methodName, args) as JSPromise;
-    return promise.toDart
-        .catchError((error) => throw _processError(error));
+    return promise.toDart.catchError((error) => throw _processError(error));
   }
 
   Future<Map<String, dynamic>> _getMapFromInstanceMethod(
-      String methodName,
-      List<dynamic> args,
+    String methodName,
+    List<dynamic> args,
   ) async {
     final promise = _callInstanceMethod(methodName, args) as JSPromise;
 
@@ -408,8 +401,8 @@ class PurchasesFlutterPlugin {
   }
 
   Future<Map<String, dynamic>?> _getNullableMapFromInstanceMethod(
-      String methodName,
-      List<dynamic> args,
+    String methodName,
+    List<dynamic> args,
   ) async {
     final promise = _callInstanceMethod(methodName, args) as JSPromise;
 
@@ -423,12 +416,12 @@ class PurchasesFlutterPlugin {
         .catchError((error) => throw _processError(error));
   }
 
-  /// Calls a synchronous instance method on PurchasesCommon and converts 
+  /// Calls a synchronous instance method on PurchasesCommon and converts
   /// the result to a nullable Dart map.
   ///
   /// Returns null if the JS method returns null, otherwise returns the converted map.
   Map<String, dynamic>? _getNullableMapFromInstanceSyncMethod(
-      String methodName,
+    String methodName,
   ) {
     final result = _getInstance().callMethod(methodName.toJS);
     if (result == null) {
@@ -446,38 +439,40 @@ class PurchasesFlutterPlugin {
         final int arg => arg.toJS,
         final bool arg => arg.toJS,
         null => null,
-        _ => throw ArgumentError(
-          'Unsupported argument type: ${arg.runtimeType}',
-        ),
+        _ =>
+          throw ArgumentError('Unsupported argument type: ${arg.runtimeType}'),
       },
   ];
 
   PlatformException _processError(dynamic error) {
-    if (error is JSObject && error.has('code')) {
-      final errorMap = _convertJsRecordToMap(error);
-      final code = errorMap['code'];
-      final message = errorMap['message'];
-      final underlyingErrorMessage = errorMap['underlyingErrorMessage'];
-      final finalMessage = '$message. $underlyingErrorMessage';
-      return PlatformException(
-        code: '$code',
-        message: finalMessage,
-        details: errorMap,
-      );
-    } else {
-      return PlatformException(
-        code: _unknownErrorCode,
-        message: error.toString(),
-      );
+    final jsAny = error as JSAny?;
+    if (jsAny != null && jsAny.isA<JSObject>()) {
+      final jsObject = jsAny as JSObject;
+      if (jsObject.has('code')) {
+        final errorMap = _convertJsRecordToMap(jsObject);
+        final code = errorMap['code'];
+        final message = errorMap['message'];
+        final underlyingErrorMessage = errorMap['underlyingErrorMessage'];
+        final finalMessage = '$message. $underlyingErrorMessage';
+        return PlatformException(
+          code: '$code',
+          message: finalMessage,
+          details: errorMap,
+        );
+      }
     }
+    return PlatformException(
+      code: _unknownErrorCode,
+      message: error.toString(),
+    );
   }
 
   Map<String, dynamic> _convertJsRecordToMap(JSAny? jsRecord) {
     if (jsRecord == null) {
       throw ArgumentError('returned result cannot be null');
     } else {
       return Map<String, dynamic>.from(
-          jsRecord.dartify() as Map<dynamic, dynamic>,
+        jsRecord.dartify() as Map<dynamic, dynamic>,
       );
     }
   }